Repository navigation
fix: recover bounded progress partition fact overflows - #852
Conversation
…schema-string-long-and-no-file-identity: recover bounded progress partition fact overflows
|
To preview the documentation for this pull request, visit the following URL:
|
Bugbot: review complete
|
Copilot approval · Human review neededStatus: A protected workflow, policy, or trust-boundary file changed. Technical evidenceRevision: |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #852 +/- ##
========================================
Coverage 95.30% 95.31%
========================================
Files 1129 1133 +4
Lines 37258 37330 +72
Branches 11107 11125 +18
========================================
+ Hits 35510 35582 +72
Misses 672 672
Partials 1076 1076
🚀 New features to boost your workflow:
|
…schema-string-long-and-no-file-identity: clarify unchanged progress completion equivalence
|
/copilot sync-branch |
Request could not be completed
Action: Inspect the failed stage in the linked workflow logs. For dependencies, provision the source Node/pnpm toolchain and repair the frozen candidate workspace graph; for setup/runtime, repair the same-job prerequisites; for command failures, correct the configured verifier. Then retry /copilot sync-branch. Blind retries do not repair a graph mismatch. Retained state: No synchronization commit or push was attempted. Original head: dca6114. Local rollback: restored; restored means the original head was verified, unverified requires operator inspection before retrying. Error code: Retryable: No Reference: |
…schema-string-long-and-no-file-identity: Separate direct parser compatibility from final assessment workflow policy
|
/copilot sync-branch |
…lemented-text-with-schema-string-long-and-no-file-identity
Branch synchronized
The fixer resolved 9 conflicted files. 6 verification checks passed. |
…schema-string-long-and-no-file-identity: Verify composed part and final progress recovery
An overlong per-file progress fact currently aborts a complete comparison with only an array position. This verifies the returned part/path/unit, locale and selected limit, then requests one concise replacement for the same complete immutable part. Persistent overflow reports exact safe identities and Unicode counts, preserving previously published progress.
The part and final-response allowances remain independent. Each attributable part may be replaced once; the existing final controller may separately reassess one completion contradiction. Every part, receipt, final result and freshness fence must validate before one normal publication. Persistent part failure never enters the final stage; final exhaustion never repeats parts or source reads.
Validation
f085fb90: 85/85 modified runtime lines covered, zero misses/partials; all four new modules have 100% lines, statements, functions and branches. Coverage budgets, typecheck, lint, documentation/specification/workflow contracts, package validation and packaged CLI/API/web smoke checks pass. Graphify updated.Review notes
No new configuration, response schema, permissions or stored-state migration. Both possible part envelopes are preflighted for every part, preserving every existing capacity, deadline and source receipt. Provider/cancellation and unrelated validation errors retain their categories. Rejected prose is never copied, clipped or accepted. Action/CLI/API bundles are generated normally.
The Action synchronized this branch with
developthrough run 41482379561. This addresses the recurring part-length Commit blocker in #837; that PR's independent extension findings and native acceptance remain separate work. Complete review without findings, all applicable technical checks, official correct-base Codecov 100% modified lines with zero misses/partials and the protected-path human decision are required before merge.Closes #830.