plugins/review
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix/skillset-audit-compliance-and-guard-fixes #7

Merged
admin merged 3 commits from fix/skillset-audit-compliance-and-guard-fixes into develop 2026-08-25 07:15:22 +00:00
Conversation 0 Commits 3 Files Changed 7
+16 -14
jiantw83 commented 2026-08-25 07:03:51 +00:00
Member
Copy Link
Copy Source

摘要

  • 需求描述：jsc-meta:skill-check 例行稽核（2026-08-25）。本 repo 補上空 diff 與六組回報的閘門，並移除會過期的步驟號互參。
  • 計畫名稱：無
  • 計畫頁：無
  • 分析頁：無

變更內容

檔案 為什麼改
skills/code-review/SKILL.md 四個步驟補完成條件：diff 為空時回報 「無發現」 並停止、不開任何 sub agent；六組全部回報前不得進入合併；每筆發現只出現一次並依 高、中、低 排序；報告交回呼叫端、修不修由對方決定。對 jsc-sdlc:implement 的互參改為描述呼叫時機，不再寫步驟號
README.md 「Skills 目錄」的敘述補上空 diff 與六組閘門
三份 manifest、兩份 marketplace 同步升版至 0.0.4，marketplace 與正本對齊

設計重點

  • 兩個閘門是有負擔的：diff 為空時先開六個 sub agent 是白花成本；六組還沒回齊就合併會漏掉發現。
  • 互參不要抄步驟號。原本寫 jsc-sdlc:implement step 6，實際呼叫點已移到 step 10，而那份技能這輪又重編過一次——改成描述時機就不會再漂移。

測試結果

  • ste100-lint.sh exit 0。
  • 六組壞味道目錄仍在 references/smells.md，未被搬進內文。
  • 對照 jsc-sdlc:implement 現況確認呼叫時機敘述正確。

前置 Push Request

  • 無
## 摘要 - 需求描述：jsc-meta:skill-check 例行稽核（2026-08-25）。本 repo 補上空 diff 與六組回報的閘門，並移除會過期的步驟號互參。 - 計畫名稱：無 - 計畫頁：無 - 分析頁：無 ## 變更內容 | 檔案 | 為什麼改 | | --- | --- | | `skills/code-review/SKILL.md` | 四個步驟補完成條件：diff 為空時回報 「無發現」 並停止、不開任何 sub agent；六組全部回報前不得進入合併；每筆發現只出現一次並依 高、中、低 排序；報告交回呼叫端、修不修由對方決定。對 `jsc-sdlc:implement` 的互參改為描述呼叫時機，不再寫步驟號 | | `README.md` | 「Skills 目錄」的敘述補上空 diff 與六組閘門 | | 三份 manifest、兩份 marketplace | 同步升版至 0.0.4，marketplace 與正本對齊 | ## 設計重點 - 兩個閘門是有負擔的：diff 為空時先開六個 sub agent 是白花成本；六組還沒回齊就合併會漏掉發現。 - 互參不要抄步驟號。原本寫 `jsc-sdlc:implement` step 6，實際呼叫點已移到 step 10，而那份技能這輪又重編過一次——改成描述時機就不會再漂移。 ## 測試結果 - `ste100-lint.sh` exit 0。 - 六組壞味道目錄仍在 `references/smells.md`，未被搬進內文。 - 對照 `jsc-sdlc:implement` 現況確認呼叫時機敘述正確。 ## 前置 Push Request - 無
jiantw83 added 3 commits 2026-08-25 07:03:52 +00:00
fix(review): 補齊稽核缺失並修掉護欄失效 9313dea6ee 
What：依 jsc-meta:skill-check 的稽核結果修正技能與工具——補上每個步驟的可檢核完成條件、
把留在內文的標準輸入輸出流程下放 tools/、修正查表與退碼路由造成的誤判。

Why：稽核發現這些缺失會讓技能在實際執行時走錯分支或靜默通過。
完成條件缺漏是最常被違反的一項；退碼誤判與查表錯誤則會讓良性狀況被當成失敗。

How：逐項對照 references/guidelines.md 的審核檢查清單修正，新增的工具都有
documented exit codes，並以真實執行驗證每條路徑。

Who：jsc-meta:skill-check 例行稽核（2026-08-25）。

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
docs(review): 同步文件與參考資料 d9d0f6936f 
What：更新 README、AGENTS.md、templates 與 references，讓文件敘述與實際行為一致。

Why：稽核發現多處文件與程式行為分歧，違反「每個意義只有單一真實來源」。

How：以實際程式行為為準改寫敘述，重複的規則收成單一來源並以一行指引指過去。

Who：jsc-meta:skill-check 例行稽核（2026-08-25）。

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
chore(review): 三份 manifest 同步升版並同步 marketplace 正本 150f0cc07c 
What：三份 plugin manifest 版本同步 bump，兩份 marketplace 檔與 plugins/meta 正本對齊。

Why：準則要求技能異動必須同步升版；marketplace 副本必須與正本完全一致。

How：以 jsc-meta 的 tools/sync-skill-manifest.sh 升版，marketplace 檔由正本複製。

Who：jsc-meta:skill-check 例行稽核（2026-08-25）。

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
admin merged commit 1990476b25 into develop 2026-08-25 07:15:22 +00:00
admin deleted branch fix/skillset-audit-compliance-and-guard-fixes 2026-08-25 07:15:22 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
admin (系統管理員) jiantw83 (Jeffery)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: plugins/review#7