Skip to content

Restore the full canary registry - #90

Merged
iamkayleb merged 1 commit into
mainfrom
revert/restore-canary-registry
Sep 7, 2026
Merged

iamkayleb merged 1 commit into
mainfrom
revert/restore-canary-registry

Conversation

@iamkayleb

Copy link
Copy Markdown
Owner

Trimming the registry to iamkayleb/bukay alone broke every sync run: select_consumer_sync_phase rejects a config outside 2..3 canaries with canary_config_requires_two_or_three_repos, so Maint 68 failed in its first job before reaching any repository.

The trim was aimed at two consistently red matrix legs for repositories this fork cannot write to. Those legs were noise, not the blocker — the run that prompted the change failed on all three legs, including bukay, for an unrelated missing read:org PAT scope. Removing them bought nothing and cost the whole sync.

Relaxing the lower bound was considered and rejected: the checked-in config is also asserted to cover six distinct consumer capability shapes, which a single-consumer fork cannot satisfy without inventing capabilities. The two unowned legs stay red until this fork either gains access to them or registers a second consumer it owns.

Claude-Session: https://claude.ai/code/session_01FC8XoyssN5v6hQCTtcjoB5

Workflow Source

Started from:

  • GitHub issue: #
  • Direct PR / remote GitHub work
  • Local Codex/user request
  • Automation run
  • Review follow-up from PR #
  • Sync / maintenance campaign
  • Dependabot or dependency update
  • Do not automate

Automation intent:

  • Verifier should review this
  • Keepalive may manage this PR
  • Human-only unless checks fail

Notes:

Summary

Testing

Trimming the registry to iamkayleb/bukay alone broke every sync run:
select_consumer_sync_phase rejects a config outside 2..3 canaries with
canary_config_requires_two_or_three_repos, so Maint 68 failed in its first
job before reaching any repository.

The trim was aimed at two consistently red matrix legs for repositories
this fork cannot write to. Those legs were noise, not the blocker — the
run that prompted the change failed on all three legs, including bukay,
for an unrelated missing read:org PAT scope. Removing them bought nothing
and cost the whole sync.

Relaxing the lower bound was considered and rejected: the checked-in
config is also asserted to cover six distinct consumer capability shapes,
which a single-consumer fork cannot satisfy without inventing capabilities.
The two unowned legs stay red until this fork either gains access to them
or registers a second consumer it owns.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FC8XoyssN5v6hQCTtcjoB5
@iamkayleb
iamkayleb merged commit 0149820 into main Sep 7, 2026
16 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants