feat/comment-scope-sweep-before-commit #9

Merged
admin merged 2 commits from feat/comment-scope-sweep-before-commit into develop 2026-08-27 01:28:41 +00:00
Member

摘要

  • 需求描述:在 jsc-git:commit 加一道全域防線:認可前先掃過工作區的註解,攔下夾帶文件相關資訊的註解。commit 是 claude、codex、copilot、antigravity、kiro 五支 CLI 共用的提交路徑,在這裡掃一次,違規註解就進不了 commit。
  • 計畫名稱:無
  • 計畫頁:無
  • 分析頁:無

變更內容

檔案 為什麼改
skills/commit/SKILL.md 在清點檔案變更之後、實際認可之前,新增掃描步驟:跑 jsc-hooks/hooks/comment-scope.sh sweep。腳本不在本機就跳過並在回報中說明,不中止認可;exit 2 代表命中,就地修好再重跑到 exit 0 才認可;警告是給人與模型判讀,不是硬性阻擋。只用一行指向 jsc-review/references/comment-scope.md,不抄規則清單。原步驟編號往後順延,並附可檢核的完成條件。frontmatter 的 description 補一句涵蓋新步驟,維持英文、四句、保留觸發時機。
README.md JSC-SKILLS 區塊裡 commit 的說明補一句涵蓋這道檢查,維持 STE100 繁中。
plugin.json 三份 manifest 之一,版本 0.0.6 升到 0.0.7。
.claude-plugin/plugin.json 三份 manifest 之一,版本 0.0.6 升到 0.0.7。
.codex-plugin/plugin.json 三份 manifest 之一,版本 0.0.6 升到 0.0.7。

設計重點

  • 規則正文只有一份正本,在 jsc-review 的 references/comment-scope.md。本存取庫只放一行指引指過去,不留清單副本,避免兩邊各自漂移。
  • 本規則的 hook 端在 plugins/hooks PR #26,規則正文在 plugins/review PR #8,兩者尚未合併。本 PR 只呼叫腳本,腳本不在本機時腳本自身安靜 exit 0、技能步驟也會跳過並照實回報,所以本 PR 與那兩支沒有合併順序相依,先合後合都不會壞。
  • 缺基礎設施不等於違規:找不到腳本就跳過並說明,不中止提交。認可流程不因為工具沒裝而卡住。
  • 警告不是硬性阻擋。確認是誤判就照實說明並繼續提交,不要為了讓腳本閉嘴而刪掉有用的註解。
  • 掃描放在清點之後、分組之前:命中要就地改檔案,改完的檔案才進得了正確的分組,不必回頭重分。

測試結果

  • sh /root/plugins/meta/tools/ste100-lint.sh README.md:exit 0,無輸出。
  • skills/commit/SKILL.md 掃 CJK 字元(grep -nP '[\x{4e00}-\x{9fff}\x{3000}-\x{303f}\x{ff00}-\x{ffef}]'):無命中,整份為英文。
  • sh /root/.claude/plugins/cache/jsc/jsc-meta/0.0.10/tools/sync-skill-manifest.sh:三份 manifest 由 0.0.6 升到 0.0.7,README 技能小節新增 0 個、移除 0 個,未塞入未翻譯的英文小節。
  • sh /root/plugins/hooks/hooks/comment-scope.sh sweep:對本次工作區實跑,exit 0,未命中。
  • git status --porcelain:空。

前置 Push Request

  • 無
## 摘要 - 需求描述:在 `jsc-git:commit` 加一道全域防線:認可前先掃過工作區的註解,攔下夾帶文件相關資訊的註解。commit 是 claude、codex、copilot、antigravity、kiro 五支 CLI 共用的提交路徑,在這裡掃一次,違規註解就進不了 commit。 - 計畫名稱:無 - 計畫頁:無 - 分析頁:無 ## 變更內容 | 檔案 | 為什麼改 | | --- | --- | | `skills/commit/SKILL.md` | 在清點檔案變更之後、實際認可之前,新增掃描步驟:跑 `jsc-hooks/hooks/comment-scope.sh sweep`。腳本不在本機就跳過並在回報中說明,不中止認可;exit 2 代表命中,就地修好再重跑到 exit 0 才認可;警告是給人與模型判讀,不是硬性阻擋。只用一行指向 `jsc-review/references/comment-scope.md`,不抄規則清單。原步驟編號往後順延,並附可檢核的完成條件。frontmatter 的 `description` 補一句涵蓋新步驟,維持英文、四句、保留觸發時機。 | | `README.md` | `JSC-SKILLS` 區塊裡 `commit` 的說明補一句涵蓋這道檢查,維持 STE100 繁中。 | | `plugin.json` | 三份 manifest 之一,版本 0.0.6 升到 0.0.7。 | | `.claude-plugin/plugin.json` | 三份 manifest 之一,版本 0.0.6 升到 0.0.7。 | | `.codex-plugin/plugin.json` | 三份 manifest 之一,版本 0.0.6 升到 0.0.7。 | ## 設計重點 - 規則正文只有一份正本,在 `jsc-review` 的 `references/comment-scope.md`。本存取庫只放一行指引指過去,不留清單副本,避免兩邊各自漂移。 - 本規則的 hook 端在 `plugins/hooks` PR #26,規則正文在 `plugins/review` PR #8,兩者尚未合併。本 PR 只呼叫腳本,腳本不在本機時腳本自身安靜 exit 0、技能步驟也會跳過並照實回報,所以本 PR 與那兩支沒有合併順序相依,先合後合都不會壞。 - 缺基礎設施不等於違規:找不到腳本就跳過並說明,不中止提交。認可流程不因為工具沒裝而卡住。 - 警告不是硬性阻擋。確認是誤判就照實說明並繼續提交,不要為了讓腳本閉嘴而刪掉有用的註解。 - 掃描放在清點之後、分組之前:命中要就地改檔案,改完的檔案才進得了正確的分組,不必回頭重分。 ## 測試結果 - `sh /root/plugins/meta/tools/ste100-lint.sh README.md`:exit 0,無輸出。 - `skills/commit/SKILL.md` 掃 CJK 字元(`grep -nP '[\x{4e00}-\x{9fff}\x{3000}-\x{303f}\x{ff00}-\x{ffef}]'`):無命中,整份為英文。 - `sh /root/.claude/plugins/cache/jsc/jsc-meta/0.0.10/tools/sync-skill-manifest.sh`:三份 manifest 由 0.0.6 升到 0.0.7,README 技能小節新增 0 個、移除 0 個,未塞入未翻譯的英文小節。 - `sh /root/plugins/hooks/hooks/comment-scope.sh sweep`:對本次工作區實跑,exit 0,未命中。 - `git status --porcelain`:空。 ## 前置 Push Request - 無
jiantw83 added 2 commits 2026-08-27 01:11:09 +00:00
What:在 commit 技能的清點步驟之後、實際認可之前,新增一個掃描步驟,呼叫 jsc-hooks 的 comment-scope.sh sweep 掃過整個工作區,並在 README 的技能目錄補上同一句說明。原步驟編號往後順延。

Why:註解禁止夾帶文件相關資訊這條規則,目前只有 claude 接得到寫檔後的即時掃描;codex、copilot、antigravity、kiro 都得等到每輪結束或工作階段結束才有機會發現。commit 是五支 CLI 共用的提交路徑,在這裡掃一次,違規註解就進不了 commit。

How:新步驟只指向 jsc-review 的 references/comment-scope.md,不抄規則清單。腳本找不到 git 工作區或 jsc-hooks 不在本機時會安靜結束,此時跳過這一步並在回報中說明,不中止認可;exit 2 代表命中,就地修好再重跑到 exit 0 才認可。警告是給人與模型判讀,不是硬性阻擋,確認誤判就照實說明後繼續。步驟附可檢核的完成條件。

Who:jsc-git 的 commit 技能,以及所有經由它提交的 CLI。
What:plugin.json、.claude-plugin/plugin.json、.codex-plugin/plugin.json 三份版本一起從 0.0.6 升到 0.0.7。

Why:commit 技能新增了認可前的註解掃描步驟,屬於實質行為變更,各 CLI 要靠版本號才知道本機這份已經落後、需要更新。

How:跑 jsc-meta 的 tools/sync-skill-manifest.sh 同步 README 技能目錄並升版,三份版本號一致,README 的技能小節沒有增減。

Who:五支 CLI 的外掛版本檢查與更新流程。
admin merged commit 37cbbdcebd into develop 2026-08-27 01:28:41 +00:00
admin deleted branch feat/comment-scope-sweep-before-commit 2026-08-27 01:28:41 +00:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: plugins/git#9