fix: apply tracker:durable label when creating/updating weekly metrics tracker issue - #2738
Conversation
…s issue agents-weekly-metrics.yml created the tracker issue without labels, leaving it invisible to triage/stale-scan filters that key on tracker:durable. - On creation: pass labels: ['tracker:durable', 'automated'] so new issues are correctly tagged from the start. - On update (existing issue found): check for tracker:durable and add it if missing, retroactively fixing issues created before this change (e.g. #2211). Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01BrBgrJKcJvaucLF5sJKXHE
|
Warning Review limit reachedYou’ve reached a temporary PR review limit under our Fair Usage Limits Policy. Next review available in: 39 minutes Your organization has reached its usage spending cap. Adjust your spending cap in the billing tab. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (3)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Mirror the agents-weekly-metrics.yml label fix in the consumer template to keep root and template in sync and clear the template-drift check. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01BrBgrJKcJvaucLF5sJKXHE
Automated Status SummaryHead SHA: 7337ee1
Coverage Overview
Updated automatically; will refresh on subsequent CI/Docker completions. Keepalive checklistScopeTracking issue for weekly agent workflow metrics. Agent Metrics SummaryGenerated: 2026-06-01T08:00:46Z KeepaliveContext for AgentRelated Issues/PRsTasks
Acceptance criteria
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3a217e9a20
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| repo, | ||
| title, | ||
| body: 'Tracking issue for weekly agent workflow metrics.\n\n' + body, | ||
| labels: ['tracker:durable', 'automated'], |
There was a problem hiding this comment.
Mirror weekly metrics label fix into the consumer template
Because .github/sync-manifest.yml declares agents-weekly-metrics.yml as a consumer-managed workflow and a template copy exists at templates/consumer-repo/.github/workflows/agents-weekly-metrics.yml, this root-only label change leaves newly scaffolded consumer repos still creating/updating the weekly tracker without tracker:durable. In the consumer-template context, the stale-scan/triage filters that this patch is meant to satisfy will continue to miss the tracker, so please mirror the label creation/update logic into the template as well.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Done — commit 542ef79 applied the identical label fix to templates/consumer-repo/.github/workflows/agents-weekly-metrics.yml. The latest commit (7e29861) also normalizes the target.labels check (string vs object) and rebaselines the template-drift allowlist fingerprints.
Generated by Claude Code
There was a problem hiding this comment.
Pull request overview
Ensures the weekly “Agent metrics weekly summary” tracking issue is consistently discoverable by triage/stale-scan automation by applying the tracker:durable label (and automated on creation) from within the weekly metrics workflow.
Changes:
- Add
labels: ['tracker:durable', 'automated']when creating the weekly metrics tracker issue. - When updating an existing tracker issue, add
tracker:durableif it’s missing.
| repo, | ||
| title, | ||
| body: 'Tracking issue for weekly agent workflow metrics.\n\n' + body, | ||
| labels: ['tracker:durable', 'automated'], | ||
| })); |
There was a problem hiding this comment.
Already mirrored in commit 542ef79. The latest commit (7e29861) additionally normalizes the target.labels check and rebaselines the template-drift allowlist so the Check Template Drift CI job passes.
Generated by Claude Code
| issue_number: target.number, | ||
| body, | ||
| })); | ||
| const hasLabel = target.labels.some(l => l.name === 'tracker:durable'); |
There was a problem hiding this comment.
Fixed in 7e29861 — both root and template now use the pattern from sync_tracker_state/index.js:23-27: (target.labels || []).some(l => (typeof l === 'string' ? l : l?.name) === 'tracker:durable')
Generated by Claude Code
…wlist
- Normalize the tracker:durable label check to handle both object and
string label formats (per sync_tracker_state/index.js pattern):
(target.labels || []).some(l => (typeof l === 'string' ? l : l?.name) === ...)
Applied to both root and consumer template copies.
- Rebaseline config/template-drift-allowlist.txt pair.15 with the new
SHA256 fingerprints after the label fix. Root and template files differ
intentionally (template SHA-pins actions + App token; root uses floating
tags + coverage monitoring). Do not align.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BrBgrJKcJvaucLF5sJKXHE
Summary
agents-weekly-metrics.ymlcreated the "Agent metrics weekly summary" tracker issue without any labels, leaving it invisible to triage/stale-scan filters that key ontracker:durable. This was noted as a pre-existing gap in the review of PR 2737.What changed
agents-weekly-metrics.ymlandtemplates/consumer-repo/.github/workflows/agents-weekly-metrics.yml:labels: ['tracker:durable', 'automated']to theissues.createcall so new tracker issues are correctly tagged from the start.tracker:durableis present and add it if missing — retroactively fixes issues created before this change (e.g. issue 2211).(target.labels || []).some(l => (typeof l === 'string' ? l : l?.name) === 'tracker:durable')(matching thesync_tracker_state/index.jspattern) to handle both label-object and label-string API shapes safely.Also changed
config/template-drift-allowlist.txt: rebaselined theagents-weekly-metrics.ymlfingerprint pair (pair.15) with new SHA256 values after the label fix. Root and template differ intentionally — do not align.