Skip to content

fix: return gh-aw coverage controller to manual canary - #924

Merged
groupthinking merged 3 commits into
mainfrom
fix/gh-aw-postmerge-canary
Jul 22, 2026
Merged

fix: return gh-aw coverage controller to manual canary#924
groupthinking merged 3 commits into
mainfrom
fix/gh-aw-postmerge-canary

Conversation

@groupthinking

@groupthinking groupthinking commented Jul 22, 2026

Copy link
Copy Markdown
Owner

Outcome

Contain the unsafe post-merge activation from #921 without disabling Jules or removing gh-aw.

Progresses #898 and focused issue #920.

Why this PR exists

PR #921 merged externally while its own description said Do not merge and its exact-head gh-aw validation was red. The failure was not an application failure: gh-aw v0.82.14 compiled all three workflows, then detected a nondeterministic lock change caused by the shorthand schedule: daily.

This is the single canonical post-merge correction for #920. It does not create a competing implementation.

Changes

  • Remove the scheduled trigger from the Focused Coverage Controller.
  • Keep workflow_dispatch so the real gh-aw canary remains installed and testable.
  • State explicitly that Jules remains enabled as an implementation agent.
  • Regenerate the lock file with pinned gh-aw v0.82.14.

Local exact-source verification

  • gh aw compile focused-coverage-controller --validate --approve: 0 errors, 0 warnings.
  • gh aw compile focused-coverage-controller --zizmor --poutine --approve: 0 errors, 0 warnings.
  • git diff --check: passed.
  • Local actionlint could not start because Docker is unavailable in the controller sandbox; the repository exact-head validation workflow remains authoritative.

Safety / activation

  • Draft until exact-head GitHub checks complete.
  • No agent was disabled.
  • No merge, credential change, branch deletion, ruleset change, or production mutation is authorized by this PR.
  • Re-enable a deterministic scheduled trigger only after Install pinned gh-aw canary and repair false-green coverage #920's truthful Coverage artifact and canary exit criteria are proven.

@vercel

vercel Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
v0-uvai Canceled Canceled Jul 22, 2026 4:32am

@coderabbitai

coderabbitai Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are limited based on label configuration.

🏷️ Required labels (at least one) (1)
  • [‘architecture-gap’, ‘bug’, ‘ci-cd’, ‘ci/cd’, ‘copilot-rabbit’, ‘documentation’, ‘duplicate’, ‘enhancement’, ‘frontend’, ‘github_actions’, ‘good first issue’, ‘help wanted’, ‘high-priority’, ‘invalid’, ‘javascript’, ‘ml-model’, ‘needs-triage’, ‘pipeline-critical’, ‘placeholder-code’, ‘priority:high’, ‘python’, ‘python:uv’, ‘question’, ‘styling’, ‘tests’, ‘v0’]

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository YAML (base), Repository UI (inherited), Organization UI (inherited)

Review profile: ASSERTIVE

Plan: Pro

Run ID: 6d3e3dfb-49d3-4288-966c-fba7139ce688

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/gh-aw-postmerge-canary
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch fix/gh-aw-postmerge-canary

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added ci/cd documentation Improvements or additions to documentation labels Jul 22, 2026
@github-actions

github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA a34695f.
Ensure 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 Files

None

@github-actions

github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown

Agent Completion Truth Gate: NOT_APPLICABLE

Evidence agrees.

Machine-readable verdict
{
  "details": {},
  "reasons": [],
  "verdict": "not_applicable"
}

Workflow evidence

@groupthinking
groupthinking marked this pull request as ready for review July 22, 2026 04:47
Copilot AI review requested due to automatic review settings July 22, 2026 04:47

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Returns the coverage controller to a safe, manual-only canary.

Changes:

  • Removes the scheduled trigger and regenerates the gh-aw lock.
  • Clarifies Codex/Jules responsibilities.
  • Prevents test-module import poisoning.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
.github/workflows/focused-coverage-controller.md Makes the controller manual-only.
.github/workflows/focused-coverage-controller.lock.yml Regenerates the compiled workflow.
tests/test_skills_integration.py Removes harmful module stubs and adds regression coverage.

@groupthinking
groupthinking merged commit 55089b1 into main Jul 22, 2026
43 of 49 checks passed
@groupthinking
groupthinking deleted the fix/gh-aw-postmerge-canary branch July 22, 2026 05:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/cd documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants