Skip to content

feat: show exact-head PR readiness in copilot status - #467

Merged
efraespada merged 27 commits into
developfrom
feature/462-dogfooding-show-exact-head-pr-readiness-in-copilot-status
Oct 13, 2026
Merged

efraespada merged 27 commits into
developfrom
feature/462-dogfooding-show-exact-head-pr-readiness-in-copilot-status

Conversation

@efraespada

@efraespada efraespada commented Oct 11, 2026 •

Copy link
Copy Markdown
Member

/copilot status now exposes PR readiness from independently verified CI, coverage, Bugbot and human-review evidence. It reads the current base ref rather than GitHub's cached PR base SHA, rechecks policy/head/link identity, and requires the PR to remain open at the final native revision read. Incomplete evidence withholds a satisfied result and never creates merge authorization.

Validation on e6ed84a6: 177 distinct feature cases; 584 suites with 7,992 passed and 35 configured skips. All coverage budgets, types, lint, docs/workflows/specifications, bundles and normal hooks passed. Three regressions first reproduced native closure with unchanged revisions; fresh native checks/review and live acceptance after promotion remain pending.

Closes #462

@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~467

Documentation is deployed and generated using docs.page

@vypbot vypbot added size: XXL Indicates an extremely large task or issue that requires significant time and effort to complete. 90% Progress: 90% labels Oct 11, 2026
@efraespada

Copy link
Copy Markdown
Member Author

/copilot sync-branch

@vypbot

vypbot commented Oct 11, 2026

Copy link
Copy Markdown
Contributor

Request could not be completed

Impact: Branch synchronization stopped because workspace validation rejected the resolution (whitespace-or-conflict-markers).

Action: Inspect this validation code in the workflow logs and correct the resolution before retrying /copilot sync-branch.

Retained state: No synchronization commit was pushed; remote branches were preserved.

Error code: workflow.failed

Retryable: Yes

Reference: 611289a1-fe7d-4408-9d64-41ca0173e5ea

@efraespada

Copy link
Copy Markdown
Member Author

/copilot sync-branch --no-agent

@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 8ad8a0a. See aggregate Bugbot status.

🤖 Bugbot review snapshot

Bugbot reported 1 potential problem when commit 24aacd6 was analyzed. This snapshot is historical; use the status block above for current state. 1 finding is linked to changed code.

Findings

  • medium: Final head check reuses the earlier revision — src/application/usecases/read_copilot_status_workflow.ts:99

Comment thread src/application/usecases/read_copilot_status_workflow.ts Outdated
@vypbot

vypbot commented Oct 11, 2026 •

Copy link
Copy Markdown
Contributor

Bugbot: review complete

Current status: No active findings on 8ad8a0a.

Pull request · Verified commit · Workflow run

@codecov-commenter

codecov-commenter commented Oct 11, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 99.30070% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 93.20%. Comparing base (ab3d096) to head (8ad8a0a).

Files with missing lines Patch % Lines
...ses/read_pull_request_status_readiness_workflow.ts 98.91% 0 Missing and 1 partial ⚠️
...omposition/pull_request_status_composition_root.ts 94.11% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #467      +/-   ##
===========================================
+ Coverage    93.11%   93.20%   +0.08%     
===========================================
  Files          872      881       +9     
  Lines        27119    27344     +225     
  Branches      7853     7937      +84     
===========================================
+ Hits         25253    25485     +232     
+ Misses         743      740       -3     
+ Partials      1123     1119       -4     
Files with missing lines Coverage Δ
src/actions/action_input_source.ts 100.00% <100.00%> (ø)
src/actions/github_action_input.ts 100.00% <100.00%> (ø)
src/actions/input_values_policy.ts 100.00% <100.00%> (ø)
...pplication/policies/publication_message_catalog.ts 100.00% <ø> (ø)
...lication/policies/pull_request_readiness_policy.ts 100.00% <100.00%> (ø)
...n/policies/pull_request_readiness_status_policy.ts 100.00% <100.00%> (ø)
src/application/policies/status_command_policy.ts 96.47% <100.00%> (+0.04%) ⬆️
...plication/usecases/read_copilot_status_workflow.ts 97.36% <100.00%> (+1.28%) ⬆️
...sitory/pull_request/current_base_revision_query.ts 100.00% <100.00%> (ø)
...y/pull_request/pull_request_approval_repository.ts 91.80% <100.00%> (-0.18%) ⬇️
... and 9 more
🚀 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

Branch synchronized

develop was merged into feature/462-dogfooding-show-exact-head-pr-readiness-in-copilot-status and pushed as 134ec5ac648a95b31bafc7c6fa49bc380c35d48d.

5 verification checks passed.

@vypbot vypbot added state:ai-processing 85% Progress: 85% and removed 90% Progress: 90% labels Oct 11, 2026

@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 8ad8a0a. See aggregate Bugbot status.

🤖 Bugbot review snapshot

Bugbot reported 3 potential problems when commit 597f3dd was analyzed. This snapshot is historical; use the status block above for current state. 3 findings are linked to changed code.

Findings

  • medium: Final freshness fence does not verify the base SHA — src/application/usecases/read_copilot_status_workflow.ts:103
  • medium: A pending result can retain stale satisfied evidence — src/application/usecases/read_copilot_status_workflow.ts:108
  • low: Off mode discards independently verified evidence — src/application/policies/pull_request_readiness_policy.ts:49

Comment thread src/application/usecases/read_copilot_status_workflow.ts Outdated
Comment thread src/application/usecases/read_copilot_status_workflow.ts Outdated
Comment thread src/application/policies/pull_request_readiness_policy.ts Outdated
@efraespada

Copy link
Copy Markdown
Member Author

/copilot recheck

@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 8ad8a0a. See aggregate Bugbot status.

🤖 Bugbot review snapshot

Bugbot reported 1 potential problem when commit 08de8a1 was analyzed. This snapshot is historical; use the status block above for current state. 1 finding is linked to changed code.

Findings

  • medium: Invalid final identity is reported as disabled — src/application/policies/pull_request_readiness_policy.ts:65

Comment thread src/application/policies/pull_request_readiness_policy.ts
@efraespada

Copy link
Copy Markdown
Member Author

/copilot recheck

@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 8ad8a0a. See aggregate Bugbot status.

🤖 Bugbot review snapshot

Bugbot reported 1 potential problem when commit 13b61d7 was analyzed. This snapshot is historical; use the status block above for current state. 1 finding is linked to changed code.

Findings

  • medium: Incomplete file evidence is reported as not applicable — src/application/policies/pull_request_readiness_policy.ts:73

Comment thread src/application/policies/pull_request_readiness_policy.ts Outdated

@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 8ad8a0a. See aggregate Bugbot status.

🤖 Bugbot review snapshot

Bugbot reported 1 potential problem when commit 13b61d7 was analyzed. This snapshot is historical; use the status block above for current state. 1 finding is linked to changed code.

Findings

  • low: Unavailable readiness claims a human review is required — src/application/policies/pull_request_readiness_policy.ts:24

Comment thread src/application/policies/pull_request_readiness_policy.ts
@efraespada

Copy link
Copy Markdown
Member Author

/copilot recheck

@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 8ad8a0a. See aggregate Bugbot status.

🤖 Bugbot review snapshot

Bugbot reported 1 potential problem when commit 22076fd was analyzed. This snapshot is historical; use the status block above for current state. 1 finding is linked to changed code.

Findings

  • medium: A final policy change to off is reported as pending — src/application/policies/pull_request_readiness_policy.ts:52

Comment thread src/application/policies/pull_request_readiness_policy.ts

@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 8ad8a0a. See aggregate Bugbot status.

🤖 Bugbot review snapshot

Bugbot reported 1 potential problem when commit 22076fd was analyzed. This snapshot is historical; use the status block above for current state. 1 finding is linked to changed code.

Findings

  • low: Do not require human review before readiness is satisfied — src/application/policies/pull_request_readiness_policy.ts:78

Comment thread src/application/policies/pull_request_readiness_policy.ts
@efraespada

Copy link
Copy Markdown
Member Author

/copilot recheck

@efraespada

Copy link
Copy Markdown
Member Author

/copilot sync-branch

1 similar comment
@efraespada

Copy link
Copy Markdown
Member Author

/copilot sync-branch

@vypbot

vypbot commented Oct 13, 2026

Copy link
Copy Markdown
Contributor

Branch synchronized

develop was merged into feature/462-dogfooding-show-exact-head-pr-readiness-in-copilot-status and pushed as e5bbeee1e1a5dd6d3d6e34f9837a345045dc4162.

5 verification checks passed.

@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 8ad8a0a. See aggregate Bugbot status.

🤖 Bugbot review snapshot

Bugbot reported 1 potential problem when commit c726364 was analyzed. This snapshot is historical; use the status block above for current state. 1 finding is linked to changed code.

Findings

  • low: Final revision fence does not verify the PR is still open — src/application/usecases/read_pull_request_status_readiness_workflow.ts:110

@efraespada

Copy link
Copy Markdown
Member Author

/copilot sync-branch --dry-run --no-agent

@vypbot

vypbot commented Oct 13, 2026

Copy link
Copy Markdown
Contributor

Dry run complete — merging develop into feature/462-dogfooding-show-exact-head-pr-readiness-in-copilot-status has 7 conflicts. Nothing was pushed and no agent was invoked.

@efraespada

Copy link
Copy Markdown
Member Author

/copilot sync-branch

@vypbot

vypbot commented Oct 13, 2026

Copy link
Copy Markdown
Contributor

Branch synchronized

develop was merged into feature/462-dogfooding-show-exact-head-pr-readiness-in-copilot-status and pushed as 8ad8a0ab49f29542d6cb44b86a6879e9d093885e.

The fixer resolved 7 conflicted files.

6 verification checks passed.

@efraespada

Copy link
Copy Markdown
Member Author

/copilot recheck

@efraespada

efraespada commented Oct 13, 2026 •

Copy link
Copy Markdown
Member Author

/COPILOT status

@vypbot

vypbot commented Oct 13, 2026 •

Copy link
Copy Markdown
Contributor

Estado de Copilot

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

✨🧑‍💻 - Dogfooding show exact head PR readiness in copilot status

3 participants