fix(ci): heartbeat parses managed-repos.yaml correctly - #77
Merged
Conversation
The grep|sed parse emitted 'owner: ry-ops' for every entry instead of 'owner/name', so the scan + ADR-006 escalation loops silently no-op'd on an invalid repo list. Parse owner/name pairs properly via the yaml package. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
ry-ops
added a commit
that referenced
this pull request
Jun 24, 2026
getManagedRepos() used `/^\s*-\s+(.+)$/` which, against the actual
`repos: [{owner, name, policies}]` schema, captured `owner: ry-ops` off
every entry — collapsing all 43 managed repos to one bogus name. The
dashboard scan then queried garbage repo names, every request errored,
and it reported 0 open alerts across the whole fleet (fixRate 1.0). The
new status.json inherited the same false "fleet is clean" picture while
simultaneously listing hard-stopped repos — the contradiction that
surfaced this bug.
Parse with the `yaml` package (already a dep) and map owner/name pairs,
mirroring the #77 fix to heartbeat.yml. Tolerates a legacy flat
`- owner/name` list too. Verified: now yields 43 unique repos incl.
ry-ops/DriveIQ instead of 1 malformed entry.
Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
grep|sed produced 'owner: ry-ops' not full names; the escalation + scan loops no-op'd. Real YAML parse.