fix(work-items): read frontier tier from capability-tier label (#1716) - #2565
Conversation
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
|
Claude encountered an error after 0s —— View job I'll analyze this and get back to you. |
|
Warning Automated security review did not complete — this is an infrastructure failure, not a review verdict. Treat any Claude comment on this PR (including a placeholder like "I'll analyze this and get back to you") as incomplete, not "no findings."
The check is green on purpose, and it is not evidence. It certifies that a security pass ran, and this one did not complete — but the cause is outside this PR's control, so merging is deliberately left unblocked rather than locking every merge for the length of the outage. Nothing was reviewed at this head. Where this check is required, it is satisfied without that evidence; a human should review security-sensitive changes here before merging. Re-run the job, or workflow_dispatch this workflow with the PR number, to retry the review (ci-workflows#227). A new push also retries it only if the caller's Re-running does NOT help for every class:
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a9fbad8983
ℹ️ 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".
|
Claude encountered an error after 2s —— View job I'll analyze this and get back to you. |
9bcc9e8 to
ecda054
Compare
|
Claude encountered an error after 0s —— View job I'll analyze this and get back to you. |
|
Warning Automated review did not complete — this is an infrastructure failure, not a review verdict. Treat any Claude comment on this PR (including a placeholder like "I'll analyze this and get back to you") as incomplete, not "no findings."
Re-run the job, or workflow_dispatch this workflow with the PR number, to retry the review. A new push re-triggers this lane only if the caller's |
|
Claude encountered an error after 0s —— View job I'll analyze this and get back to you. |
|
Claude encountered an error after 0s —— View job I'll analyze this and get back to you. |
Co-authored-by: Kyle Sexton <kyle-sexton@users.noreply.github.com>
Add capability-tier-labels.md reference, tracker-seam axis row, and manifest/README descriptions. Document github-iac provisioning requirement and security-surface dispatch vs quota-guard separation. Co-authored-by: Kyle Sexton <kyle-sexton@users.noreply.github.com>
Co-authored-by: Kyle Sexton <kyle-sexton@users.noreply.github.com>
Co-authored-by: Kyle Sexton <kyle-sexton@users.noreply.github.com>
…ll (#1716) Add setup check probe 8 and apply steps for capability-tier: frontier label provisioning, plus a one-shot backfill path for items that carry only pre-#1716 triage-briefing body stamps. Triage refuses to re-triage already-triaged output, so setup apply runs backfill-capability-tier-labels.sh after the axis exists. Bump to 0.35.21. Co-authored-by: Kyle Sexton <kyle-sexton@users.noreply.github.com>
Move capability-tier migration and backfill procedures to progressive- disclosure reference spokes. Shorten the setup description to stay under the 1536-char listing cap and bring SKILL.md under the 500-line hard cap. Co-authored-by: Kyle Sexton <kyle-sexton@users.noreply.github.com>
e60e9ba to
626b348
Compare
…vals JSON changelog-parity --check-preserved requires the fork-point ## [0.35.19] section to remain; keep #1716 notes under 0.35.20/0.35.21. Insert the missing comma before eval cases 14–15 so check-jsonschema can parse. Co-authored-by: Kyle Sexton <kyle-sexton@users.noreply.github.com>
|
Claude encountered an error after 0s —— View job I'll analyze this and get back to you. |
|
Claude encountered an error after 0s —— View job I'll analyze this and get back to you. |
…egacy signal lib Flatten nested setup SKILL list to satisfy markdownlint. Invoke label-existence checks outside if conditions for SC2310. Mark legacy-frontier-tier-signal.sh executable in the git index.
|
Claude encountered an error after 0s —— View job I'll analyze this and get back to you. |
Closes #1716
Summary
Moves the work-loop frontier-tier signal from triage briefing body prose onto the provider-permissioned
capability-tier: frontierlabel. Missing label fails closed to the general tier.Fix
Verification
Related
Refs #1721 — prior containment superseded by this reader flip.