Skip to content

[#454] ✨ - Dogfooding show prebranch SDD readiness in copilot status - #465

Merged
efraespada merged 20 commits into
developfrom
feature/454-dogfooding-show-prebranch-sdd-readiness-in-copilot-status
Oct 12, 2026
Merged

efraespada merged 20 commits into
developfrom
feature/454-dogfooding-show-prebranch-sdd-readiness-in-copilot-status

Conversation

@efraespada

@efraespada efraespada commented Oct 11, 2026 •

Copy link
Copy Markdown
Member

/copilot status now 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:

  • 195 distinct extension cases. Native Action parent synchronization on b22d2324262acee888ec78ebf7f0b499dd4af9ab passed 571 suites and 7,697 tests with 35 configured skips (7,732 total), all existing coverage budgets and five verification commands.
  • The synchronized head b22d2324262acee888ec78ebf7f0b499dd4af9ab passed 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.
  • New pure policies meet 100% coverage; application/read adapters meet the unchanged 95% line/statement and 90% branch/function thresholds. Typecheck, lint, specifications, documentation, build and bundle verification passed. The setup acceptance ledger retains its 42 explicitly open cases.

Live command acceptance remains pending promotion. The command cannot approve a PR, invoke a model or mutate issue, branch or SDD state.

Closes #454

@docs-page

docs-page Bot commented Oct 11, 2026

Copy link
Copy Markdown

To preview the documentation for this pull request, visit the following URL:

docs.page/vypdev/copilot~465

Documentation is deployed and generated using docs.page

@vypbot vypbot added the size: XXL Indicates an extremely large task or issue that requires significant time and effort to complete. label Oct 11, 2026
@vypbot vypbot changed the title feature: Show verified SDD readiness in status [#454] ✨ - Dogfooding show prebranch SDD readiness in copilot status Oct 11, 2026
@vypbot
vypbot self-requested a review October 11, 2026 14:11
@vypbot vypbot added this to vypdev Oct 11, 2026
@vypbot
vypbot changed the base branch from develop to master October 11, 2026 14:12
@efraespada efraespada linked an issue Oct 11, 2026 that may be closed by this pull request
1 task done
@vypbot
vypbot changed the base branch from master to develop October 11, 2026 14:12

@vypbot vypbot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Current status: All findings originating in this review are resolved.
Last reconciled on b22d232. 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

Comment thread src/application/policies/sdd_status_policy.ts
Comment thread src/application/policies/status_command_policy.ts
@vypbot

vypbot commented Oct 11, 2026 •

Copy link
Copy Markdown
Contributor

Bugbot: review complete

Current status: No active findings on b22d232.

Pull request · Verified commit · Workflow run

@efraespada

Copy link
Copy Markdown
Member Author

/copilot fix sdd_status_policy.ts:77:untrusted-issue-host status_command_policy.ts:125:missing-kind-hides-readiness

@codecov-commenter

codecov-commenter commented Oct 11, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 96.45669% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.93%. Comparing base (0ae1d22) to head (b22d232).

Files with missing lines Patch % Lines
...cation/usecases/sdd/sdd_evidence_query_use_case.ts 95.14% 1 Missing and 4 partials ⚠️
...ion/policies/semantic_result_publication_policy.ts 0.00% 1 Missing and 1 partial ⚠️
src/application/policies/status_command_policy.ts 83.33% 0 Missing and 1 partial ⚠️
...cture/composition/sdd_evidence_composition_root.ts 95.00% 0 Missing and 1 partial ⚠️
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     
Files with missing lines Coverage Δ
...pplication/policies/publication_message_catalog.ts 100.00% <ø> (ø)
.../application/policies/sdd_issue_identity_policy.ts 100.00% <100.00%> (ø)
src/application/policies/sdd_query_card_policy.ts 100.00% <100.00%> (ø)
src/application/policies/sdd_status_policy.ts 100.00% <100.00%> (ø)
src/application/ports/sdd_evidence_read_ports.ts 100.00% <100.00%> (ø)
...on/usecases/comment_automation_command_workflow.ts 91.89% <100.00%> (ø)
...application/usecases/comment_automation_context.ts 100.00% <100.00%> (ø)
src/application/usecases/issue_comment_use_case.ts 94.73% <100.00%> (+0.14%) ⬆️
...plication/usecases/read_copilot_status_workflow.ts 96.42% <100.00%> (+0.77%) ⬆️
...ation/usecases/sdd/pre_branch_sdd_gate_use_case.ts 88.50% <100.00%> (+0.50%) ⬆️
... and 7 more

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@vypbot

vypbot commented Oct 11, 2026

Copy link
Copy Markdown
Contributor

Request could not be completed

Impact: The admitted agent did not produce a usable result.

Action: Inspect the sanitized agent status and retry if appropriate.

Retained state: Existing persisted state and completed external effects were preserved.

Error code: agent.failed

Retryable: Yes

Reference: 2d2e881b-2526-4a9e-b36a-923b64fd57fc

@efraespada

Copy link
Copy Markdown
Member Author

/copilot sync-branch

@vypbot vypbot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Current status: All findings originating in this review are resolved.
Last reconciled on b22d232. 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

Comment thread specs/pre-branch-sdd-status-readiness.md Outdated
@vypbot

vypbot commented Oct 11, 2026 •

Copy link
Copy Markdown
Contributor

Copilot approval · Human review needed

Status: A protected workflow, policy, or trust-boundary file changed.
Completed: 1/1 observed checks succeeded; Bugbot complete.
Action required: Ask a human reviewer to decide.
Impact: No new approval is confirmed for this revision.
See checks

Technical evidence

Revision: b22d2324262a; base: 7ff3d7d4ba4b; reason: protected-path.

@vypbot vypbot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Current status: All findings originating in this review are resolved.
Last reconciled on b22d232. 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

Comment thread src/application/policies/sdd_query_card_policy.ts Outdated
Comment thread specs/pre-branch-sdd-gate.md
@efraespada

Copy link
Copy Markdown
Member Author

/copilot recheck

@efraespada

Copy link
Copy Markdown
Member Author

/copilot sync-branch --no-agent

@vypbot

vypbot commented Oct 11, 2026

Copy link
Copy Markdown
Contributor

Branch synchronized

develop was merged into feature/454-dogfooding-show-prebranch-sdd-readiness-in-copilot-status and pushed as b22d2324262acee888ec78ebf7f0b499dd4af9ab.

5 verification checks passed.

@efraespada

Copy link
Copy Markdown
Member Author

/copilot recheck

@efraespada

efraespada commented Oct 11, 2026 •

Copy link
Copy Markdown
Member Author

/CoPiLoT StAtUs

@vypbot

vypbot commented Oct 11, 2026

Copy link
Copy Markdown
Contributor

Copilot status

@efraespada
efraespada merged commit 0d0f16d into develop Oct 12, 2026
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

90% Progress: 90% size: XXL Indicates an extremely large task or issue that requires significant time and effort to complete. state:verified

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

✨🧑‍💻 - Dogfooding show prebranch SDD readiness in copilot status

3 participants