Repository navigation
[#488] 🐛 - Dogfooding make Windows verification fixture startup and failure evidence explicit - #489
Conversation
|
To preview the documentation for this pull request, visit the following URL:
|
Bugbot: review complete
|
…-visibility-after-linking-a-pr
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #489 +/- ##
===========================================
+ Coverage 93.10% 93.11% +0.01%
===========================================
Files 872 872
Lines 27101 27119 +18
Branches 7846 7853 +7
===========================================
+ Hits 25233 25253 +20
+ Misses 746 743 -3
- Partials 1122 1123 +1
🚀 New features to boost your workflow:
|
Copilot approval · Human review neededStatus: This pull request is outside the configured approval scope. Technical evidenceRevision: |
|
/copilot sync-branch --from bugfix/486-dogfooding-recover-bounded-project-item-visibility-after-linking-a-pr |
…fter-linking-a-pr into bugfix/488-dogfooding-make-windows-verification-fixture-startup-and-failure-evidence-explicit
Branch synchronized
6 verification checks passed. |
Two hosted Windows fixtures could hide useful acceptance evidence: the verification shell returned only exit 2, and the literal-argv fixture could fail with EPERM while removing its temporary directory after successful execution. The shell now imports trusted system modules explicitly and reports bounded child diagnostics; cleanup uses the existing finite retry pattern.
The real shell, hashing, process and argv checks remain, as do all execution deadlines and platform/coverage gates. Cleanup still fails if retries are exhausted. The change adds no production runtime or permission behavior.
Validation: normal repository hooks passed, and the final Action-managed head passed CI, native review and full hosted Mac, Ubuntu and Windows coverage/acceptance. Parent and dependency integration both used the Action and completed all six configured verification checks. Two existing cases were strengthened; no new test case is counted.
Closes #488