Skip to content

Commit 3f7965b

Browse files
authored
refactor: retire weekly ci layer (#40)
1 parent d6b954e commit 3f7965b

23 files changed

Lines changed: 122 additions & 122 deletions

.github/workflows/changed-scope-quality-weekly.yml renamed to .github/workflows/changed-scope-quality-nightly.yml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
1-
name: changed-scope-quality-weekly
1+
name: changed-scope-quality-nightly
22

33
on:
44
schedule:
5-
- cron: '45 3 * * 1'
5+
- cron: '45 3 * * *'
66
workflow_dispatch:
77
inputs:
88
feedback_jsonl_path:
@@ -18,7 +18,7 @@ on:
1818
- 'configs/changed_scope/rule_tuning.json'
1919
- 'tests/fixtures/changed_scope/changed-scope-feedback.schema.json'
2020
- 'tests/fixtures/changed_scope/changed-scope-feedback.sample.jsonl'
21-
- '.github/workflows/changed-scope-quality-weekly.yml'
21+
- '.github/workflows/changed-scope-quality-nightly.yml'
2222

2323
permissions:
2424
contents: read
@@ -90,7 +90,7 @@ jobs:
9090
summary_md=".runtime-cache/test_output/changed_scope_quality/${latest_week}/changed_scope_quality.summary.md"
9191
if [[ -f "${summary_md}" ]]; then
9292
{
93-
echo "## changed-scope-quality-weekly"
93+
echo "## changed-scope-quality-nightly"
9494
echo "- run_id: ${RUN_ID}"
9595
echo "- input: \`${INPUT_PATH}\`"
9696
echo "- sample_fallback_used: \`${SAMPLE_FALLBACK_USED}\`"
@@ -125,7 +125,7 @@ jobs:
125125
if: always()
126126
uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02
127127
with:
128-
name: changed-scope-quality-weekly-${{ github.run_id }}
128+
name: changed-scope-quality-nightly-${{ github.run_id }}
129129
retention-days: 14
130130
if-no-files-found: warn
131131
path: |

.github/workflows/continuous-governance.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ on:
2424
schedule:
2525
# Daily run (Tuesday-Sunday): 03:00 UTC
2626
- cron: "0 3 * * 0,2-6"
27-
# Weekly run: Monday 03:00 UTC
27+
# Nightly full run: Monday 03:00 UTC
2828
- cron: "0 3 * * 1"
2929

3030
permissions:

AGENTS.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,9 @@ Work in CortexPilot as a contract-first engineering agent:
4242

4343
## Generated Governance Context
4444

45+
- active CI layers: `pre-commit`, `pre-push`, `hosted`, `nightly`, `manual`
46+
- do not reintroduce a sixth CI/profile/workflow layer; move scheduled heavy checks to `nightly` and explicit high-cost verification to `manual`
47+
4548
<!-- GENERATED:ci-topology-summary:start -->
4649
- trust flow: `ci-trust-boundary -> quick-feedback -> hosted policy/core slices -> pr-release-critical-gates -> pr-ci-gate`
4750
- hosted policy/core slices: `policy-and-security, core-tests`

README.md

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -477,6 +477,20 @@ npm run test
477477
`npm run security:scan:closeout` only when you intentionally want the stricter
478478
closeout/manual layers.
479479

480+
Current CI contract has five layers only:
481+
482+
| Layer | What it owns | Default posture |
483+
| --- | --- | --- |
484+
| `pre-commit` | cheap local commit-time quality gates | automatic, local, fast |
485+
| `pre-push` | local fast verification before remote CI | automatic, local, still bounded |
486+
| `hosted` | GitHub-hosted base CI and required checks | automatic, remote, hosted-first |
487+
| `nightly` | scheduled heavier audits and analytics | automatic, scheduled, non-default |
488+
| `manual` | owner-invoked closeout or high-cost validation | explicit, protected, high-friction |
489+
490+
There is no separate sixth CI layer anymore. Old extra-layer behavior now
491+
belongs either to `nightly` scheduled governance or to explicit `manual`
492+
verification.
493+
480494
`npm run test:quick` now expects the dashboard clean-room install gate to
481495
prove `jsdom` itself can load, instead of pinning success to the presence of a
482496
specific transitive dependency layout such as `data-urls`.

configs/ci_policy.core.json

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,8 +37,7 @@
3737
"execution": {
3838
"test_phase_parallel": true,
3939
"apply_retention": false,
40-
"nightly_full": false,
41-
"weekly_full": false
40+
"nightly_full": false
4241
},
4342
"defaults": {
4443
"external_web_probe_url": "https://example.com",

configs/ci_policy.profiles.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
"version": "1.0.0",
55
"layer": "profile",
66
"schema": "schemas/ci_policy.schema.json",
7-
"description": "Profile-level CI defaults (pr/nightly/weekly).",
7+
"description": "Profile-level CI defaults (pr/nightly/manual).",
88
"source_of_truth": [
99
"scripts/ci.sh"
1010
]
@@ -45,7 +45,7 @@
4545
"p1_max_threshold_percent": 1.0
4646
}
4747
},
48-
"weekly": {
48+
"manual": {
4949
"ui_flake": {
5050
"p0_iterations": 50,
5151
"p1_iterations": 50,

configs/env.registry.json

Lines changed: 1 addition & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -2929,18 +2929,6 @@
29292929
"scripts/ci.sh"
29302930
]
29312931
},
2932-
{
2933-
"name": "CORTEXPILOT_CI_WEEKLY_FULL",
2934-
"scope": "ci",
2935-
"secret": false,
2936-
"required": false,
2937-
"default": null,
2938-
"owner": "platform",
2939-
"description": "CORTEXPILOT_CI_WEEKLY_FULL controls ci runtime behavior and is consumed by 1 first-party path(s).",
2940-
"consumers": [
2941-
"scripts/ci.sh"
2942-
]
2943-
},
29442932
{
29452933
"name": "CORTEXPILOT_CLEANUP_CONFIRM",
29462934
"scope": "platform",
@@ -4664,7 +4652,7 @@
46644652
"required": false,
46654653
"default": "auto",
46664654
"owner": "platform",
4667-
"description": "Selects E2E timeout budget profile (auto|pr|nightly-full); auto resolves using nightly/weekly CI flags.",
4655+
"description": "Selects E2E timeout budget profile (auto|pr|nightly-full); auto resolves using nightly CI flags.",
46684656
"consumers": [
46694657
"scripts/e2e_dashboard_high_risk_actions_real.sh"
46704658
]

configs/governance/continuous_ops.json

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919
"min_scoped": 4,
2020
"fail_on_todo": 0
2121
},
22-
"p2_weekly_target": {
22+
"p2_nightly_ramp": {
2323
"enabled": true,
2424
"tiers": "P2",
2525
"surfaces": "dashboard,desktop",
@@ -30,62 +30,62 @@
3030
"partial_count": 63,
3131
"covered_count": 0
3232
},
33-
"min_weekly_gap_reduction": 1.0,
33+
"min_gap_reduction": 1.0,
3434
"fail_closed_mode": "strict",
3535
"ramp_table": [
3636
{
3737
"week_offset": 1,
3838
"target_partial_max": 63,
3939
"target_covered_min": 0,
40-
"min_weekly_gap_reduction": 0.0
40+
"min_gap_reduction": 0.0
4141
},
4242
{
4343
"week_offset": 2,
4444
"target_partial_max": 60,
4545
"target_covered_min": 3,
46-
"min_weekly_gap_reduction": 1.0
46+
"min_gap_reduction": 1.0
4747
},
4848
{
4949
"week_offset": 3,
5050
"target_partial_max": 56,
5151
"target_covered_min": 7,
52-
"min_weekly_gap_reduction": 1.0
52+
"min_gap_reduction": 1.0
5353
},
5454
{
5555
"week_offset": 4,
5656
"target_partial_max": 50,
5757
"target_covered_min": 13,
58-
"min_weekly_gap_reduction": 1.5
58+
"min_gap_reduction": 1.5
5959
},
6060
{
6161
"week_offset": 6,
6262
"target_partial_max": 42,
6363
"target_covered_min": 21,
64-
"min_weekly_gap_reduction": 1.5
64+
"min_gap_reduction": 1.5
6565
},
6666
{
6767
"week_offset": 8,
6868
"target_partial_max": 34,
6969
"target_covered_min": 29,
70-
"min_weekly_gap_reduction": 2.0
70+
"min_gap_reduction": 2.0
7171
},
7272
{
7373
"week_offset": 10,
7474
"target_partial_max": 24,
7575
"target_covered_min": 39,
76-
"min_weekly_gap_reduction": 2.0
76+
"min_gap_reduction": 2.0
7777
},
7878
{
7979
"week_offset": 12,
8080
"target_partial_max": 12,
8181
"target_covered_min": 51,
82-
"min_weekly_gap_reduction": 2.5
82+
"min_gap_reduction": 2.5
8383
},
8484
{
8585
"week_offset": 14,
8686
"target_partial_max": 0,
8787
"target_covered_min": 63,
88-
"min_weekly_gap_reduction": 2.5
88+
"min_gap_reduction": 2.5
8989
}
9090
]
9191
}

docs/README.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -134,6 +134,8 @@ navigation set.
134134

135135
## Public CI Contract
136136

137+
- active CI layers are `pre-commit`, `pre-push`, `hosted`, `nightly`, and `manual`
138+
- no sixth CI/profile/workflow layer exists in this repository
137139
- default public CI is hosted-first and GitHub-hosted
138140
- fork PRs stay on low-privilege checks only and must not touch secrets or
139141
live/external systems

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@
9595
"ui:truth:gate": "bash scripts/ui_e2e_truth_gate.sh",
9696
"ui:ops:pr": "UI_REGRESSION_PROFILE=pr UI_REGRESSION_REQUIRE_FULL_STRICT=1 UI_REGRESSION_REQUIRE_CRITICAL_P2=1 UI_REGRESSION_CLICK_INVENTORY_REQUIRED=1 bash scripts/ui_regression_operational_gate.sh --profile pr",
9797
"ui:ops:nightly": "UI_REGRESSION_PROFILE=nightly bash scripts/ui_regression_operational_gate.sh --profile nightly",
98-
"ui:ops:weekly": "UI_REGRESSION_PROFILE=weekly bash scripts/ui_regression_operational_gate.sh --profile weekly",
98+
"ui:ops:manual": "UI_REGRESSION_PROFILE=manual bash scripts/ui_regression_operational_gate.sh --profile manual",
9999
"bench:e2e:speed": "bash scripts/bench_e2e_speed.sh",
100100
"bench:e2e:speed:dry-run": "bash scripts/bench_e2e_speed.sh --rounds 3 --ui-full-gemini-strict --dashboard-high-risk --dry-run",
101101
"bench:e2e:speed:report-only": "bash scripts/bench_e2e_speed.sh --report-only",

0 commit comments

Comments
 (0)