Add active review projection repair - #106
Draft
michaelluetw-bit wants to merge 2 commits into
Draft
Conversation
Author
|
Upstream maintainer action requested: both fork pull-request workflows are currently action_required and have not started any jobs. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
解決什麼問題
ssf execution review原本沒有只重建 root active receipt/projection 的受支援路徑。再次執行一般 review 會同步改寫 plan-scoped receipt、repair-state,並可能清除 workspace,無法符合 immutable revision evidence 的治理邊界。實際修改內容
--repair-active-projection,只接受 PASS,且只建立或更新 root active receipt。recorded_at;缺少或空白recorded_at時會重建完整 receipt。execution show改為有效 root 優先;root 缺失或無效 PASS 時回退 scoped receipt,無效 FAIL 仍維持 blocker。legacyPath更名為rootPath。主要受影響範圍
execution reviewCLI dispatch、help 與 JSON 結果。測試/驗證結果
npm run build:PASS。npm run validate:PASS。git diff --check:PASS,僅 Windows LF/CRLF 提示。npm test在 Windows baseline 仍為 exit 1;失敗類別仍是既存的 absolute-path ESM URL、CRLF-sensitive assertions、shell/process 限制與表格解析差異。本 PR 受影響的ssf execution與execution plan data contractsuites 全數通過,新增 repair regressions 亦全數通過。git push --no-verify推送,未將完整測試描述為 PASS。刻意未修改或仍待處理的事項