fix(ci): finish removal of eventrelay-ci-investigator workflow - #1320
fix(ci): finish removal of eventrelay-ci-investigator workflow#1320groupthinking wants to merge 1 commit into
Conversation
Commit 07b8a2e removed .github/workflows/eventrelay-ci-investigator.md (the CI Investigator source) but left dangling references behind, so the `test` job is red on main and on every open PR: - tests/unit/test_gh_aw_workflow_governance.py:: test_ci_investigator_requires_dedicated_codex_credential loaded the deleted .md/.lock.yml and failed with FileNotFoundError (1 failed, 7941 passed). - .github/workflows/gh-aw-validation.yml still ran `gh aw compile eventrelay-ci-investigator` and diffed the removed .lock.yml, which would fail the validation workflow at runtime. This completes the intended cleanup: drop the obsolete governance test, remove the stale `eventrelay-ci-investigator` assertion from test_gh_aw_validation_pins_runtime_version, and drop the three dangling references from gh-aw-validation.yml. canonical-pr-remediator and focused-coverage-controller are untouched. Verified: test_gh_aw_workflow_governance.py 8 passed; gh-aw-validation.yml parses; ruff clean. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Q3EkDi3YzmMyGvHizXKaPf
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedAuto reviews are limited based on label configuration. 🏷️ Required labels (at least one) (1)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository YAML (base), Repository UI (inherited), Organization UI (inherited) Review profile: ASSERTIVE Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Snapshot WarningsEnsure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice. Scanned FilesNone |
Agent Completion Truth Gate: BLOCKEDReasons: Machine-readable verdict{
"details": {
"collection_errors": [
"incomplete_linked_issue_contract",
"missing_linked_issue",
"missing_closing_issue_reference",
"missing_agent_run_id",
"missing_agent_login"
],
"invalid_fields": [
"issue.number",
"policy.agent_login",
"policy.run_id"
]
},
"reasons": [
"invalid_payload"
],
"verdict": "blocked"
} |
Status: code CI green — blocked only on the provenance gate (needs a human/governance decision)Why this PR exists: the Code CI on head The one red check — Requested decision (owner): either attest this PR as Generated by Claude Code |
Closing: already landed on
|
Canonical issue
No separate issue — this completes the deliberate cleanup started by commit
07b8a2e("ci: remove EventRelay CI Investigator workflow source (noise-onlyoutput; per repo cleanup)"), which is already merged to
main.Outcome
07b8a2edeleted.github/workflows/eventrelay-ci-investigator.mdbut leftdangling references, so the
testjob is red onmainand on every open PR(surfaced by the
testcheck on PR #1319 —1 failed, 7941 passed):tests/unit/test_gh_aw_workflow_governance.py::test_ci_investigator_requires_dedicated_codex_credentialloaded the deleted
.md/.lock.ymland failed withFileNotFoundError..github/workflows/gh-aw-validation.ymlstill rangh aw compile eventrelay-ci-investigatorand diffed the removedeventrelay-ci-investigator.lock.yml, which would fail the validationworkflow at runtime.
This finishes the removal so the repo is internally consistent and the
testjob goes green.
Scope
test_ci_investigator_requires_dedicated_codex_credentialtest.eventrelay-ci-investigatorassertion fromtest_gh_aw_validation_pins_runtime_version.eventrelay-ci-investigatorreferences fromgh-aw-validation.yml(two compile step lists + the lock-file diff).canonical-pr-remediatorandfocused-coverage-controller— untouched, still validated.AUDIT.mdrow that mentions the workflow — left as a point-in-time audit record; not load-bearing for CI.Risk
Verification
pytest tests/unit/test_gh_aw_workflow_governance.py→ 8 passed (previously-failing test removed).gh-aw-validation.ymlparses as valid YAML;ruff checkclean on the changed test file.Production evidence
Not applicable — CI/governance-only change. No runtime or deployment surface is affected.
Agent handoff
testjob's single failure is eliminated; validation workflow no longer references a deleted workflow.Generated by Claude Code