Repository navigation
feat: show exact-head PR readiness in copilot status - #467
efraespada merged 27 commits into
Conversation
…s-in-copilot-status
|
To preview the documentation for this pull request, visit the following URL:
|
|
/copilot sync-branch |
Request could not be completed
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: Retryable: Yes Reference: |
|
/copilot sync-branch --no-agent |
There was a problem hiding this comment.
Current status: All findings originating in this review are resolved.
Last reconciled on8ad8a0a. 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
Bugbot: review complete
|
Codecov Report❌ Patch coverage is 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
🚀 New features to boost your workflow:
|
…s-in-copilot-status
Branch synchronized
5 verification checks passed. |
There was a problem hiding this comment.
Current status: All findings originating in this review are resolved.
Last reconciled on8ad8a0a. 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
|
/copilot recheck |
There was a problem hiding this comment.
Current status: All findings originating in this review are resolved.
Last reconciled on8ad8a0a. 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
|
/copilot recheck |
There was a problem hiding this comment.
Current status: All findings originating in this review are resolved.
Last reconciled on8ad8a0a. 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
There was a problem hiding this comment.
Current status: All findings originating in this review are resolved.
Last reconciled on8ad8a0a. 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
|
/copilot recheck |
There was a problem hiding this comment.
Current status: All findings originating in this review are resolved.
Last reconciled on8ad8a0a. 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
There was a problem hiding this comment.
Current status: All findings originating in this review are resolved.
Last reconciled on8ad8a0a. 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
|
/copilot recheck |
|
/copilot sync-branch |
1 similar comment
|
/copilot sync-branch |
…s-in-copilot-status
Branch synchronized
5 verification checks passed. |
There was a problem hiding this comment.
Current status: All findings originating in this review are resolved.
Last reconciled on8ad8a0a. 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
|
/copilot sync-branch --dry-run --no-agent |
|
Dry run complete — merging |
|
/copilot sync-branch |
…s-in-copilot-status
Branch synchronized
The fixer resolved 7 conflicted files. 6 verification checks passed. |
|
/copilot recheck |
|
/COPILOT status |
Estado de Copilot
|
/copilot statusnow 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