Repository navigation
[#454] ✨ - Dogfooding show prebranch SDD readiness in copilot status - #465
Conversation
…s-in-copilot-status
…s-in-copilot-status
|
To preview the documentation for this pull request, visit the following URL:
|
There was a problem hiding this comment.
Current status: All findings originating in this review are resolved.
Last reconciled onb22d232. See aggregate Bugbot status.
🤖 Bugbot review snapshot
Bugbot reported 2 potential problems when commit 9844fb4 was analyzed. This snapshot is historical; use the status block above for current state. 2 findings are linked to changed code.
Findings
- medium: Issue links accept arbitrary HTTPS hosts —
src/application/policies/sdd_status_policy.ts:76 - low: Unknown issue kind hides the readiness section —
src/application/policies/status_command_policy.ts:128
Bugbot: review complete
|
|
/copilot fix sdd_status_policy.ts:77:untrusted-issue-host status_command_policy.ts:125:missing-kind-hides-readiness |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## develop #465 +/- ##
===========================================
+ Coverage 92.85% 92.93% +0.07%
===========================================
Files 839 845 +6
Lines 25676 25921 +245
Branches 7314 7411 +97
===========================================
+ Hits 23842 24089 +247
+ Misses 745 742 -3
- Partials 1089 1090 +1
🚀 New features to boost your workflow:
|
Request could not be completed
Action: Inspect the sanitized agent status and retry if appropriate. Retained state: Existing persisted state and completed external effects were preserved. Error code: Retryable: Yes Reference: |
…s-in-copilot-status
…s-in-copilot-status
…s-in-copilot-status
|
/copilot sync-branch |
There was a problem hiding this comment.
Current status: All findings originating in this review are resolved.
Last reconciled onb22d232. See aggregate Bugbot status.
🤖 Bugbot review snapshot
Bugbot reported 1 potential problem when commit 36cb934 was analyzed. This snapshot is historical; use the status block above for current state. 1 finding is linked to changed code.
Findings
- low: The DoD claims 181 passing cases despite documenting fewer implemented cases —
specs/pre-branch-sdd-status-readiness.md:381
Copilot approval · Human review neededStatus: A protected workflow, policy, or trust-boundary file changed. Technical evidenceRevision: |
There was a problem hiding this comment.
Current status: All findings originating in this review are resolved.
Last reconciled onb22d232. See aggregate Bugbot status.
🤖 Bugbot review snapshot
Bugbot reported 2 potential problems when commit 8b1a42c was analyzed. This snapshot is historical; use the status block above for current state. 2 findings are linked to changed code.
Findings
- medium: An edited latest card can leave an older publication authoritative —
src/application/policies/sdd_query_card_policy.ts:17 - low: The status extension has conflicting test budgets —
specs/pre-branch-sdd-gate.md:720
|
/copilot recheck |
|
/copilot sync-branch --no-agent |
…s-in-copilot-status
Branch synchronized
5 verification checks passed. |
|
/copilot recheck |
|
/CoPiLoT StAtUs |
Copilot status
|
/copilot statusnow reports SDD readiness from the latest Action-owned record, current authorized answers and verified immutable publications. It withholds readiness when issue content, catalog ownership, linked branch or publication proof is unavailable, inconsistent or changes during collection.The read-only query preserves the first and current publication commits, lists current unanswered question IDs and bounds maintainer authorization reads. Damaged latest cards cannot revive an older publication. English and Spanish output uses trusted repository links and preserves existing status fields. The owning SDD, companion, catalog evidence and command documentation describe the same 195-case extension budget.
Validation:
b22d2324262acee888ec78ebf7f0b499dd4af9abpassed 571 suites and 7,697 tests with 35 configured skips (7,732 total), all existing coverage budgets and five verification commands.b22d2324262acee888ec78ebf7f0b499dd4af9abpassed CI, RepoWise and hosted Ubuntu, Windows and macOS package/session checks. Native recheck completed all four partitions with complete coverage, nine fixed findings and zero open, reopened, verification-required or unknown findings. The actual Bugbot card and lifecycle labels show review complete / ready / awaiting maintainer. Recommend still requires a human decision for protected paths and does not approve.Live command acceptance remains pending promotion. The command cannot approve a PR, invoke a model or mutate issue, branch or SDD state.
Closes #454