plugins/code-review
Archived
2
0
Fork 0
Code Pull Requests Activity

Compare commits

base: plugins/code-review:b72b641c3b8884ba56c36307fe0caf41693a679c
Branches Tags
plugins/code-review:master plugins/code-review:develop
...
compare: plugins/code-review:dec8607803161fcb46878ab090df7170d9f9d165
Branches Tags
plugins/code-review:master plugins/code-review:develop
15 Commits
b72b641c3b ... dec8607803
Author SHA1 Message Date
Jeffery dec8607803 feat(code-review-nuget): 遇到 JSC 開頭套件時必須先詢問使用者是否繼續 2026-06-24 09:32:57 +08:00
admin 9df7aab36e Merge pull request '更新 README.md' (#16) from develop into master 
Reviewed-on: #16
Reviewed-by: 系統管理員 <1+admin@noreply.localhost>
 2026-06-22 14:55:46 +00:00
admin 19b2451290 Merge pull request 'feat(code-review-nuget): 補上逐版升級回退策略' (#15) from develop into master 
Reviewed-on: #15
Reviewed-by: 系統管理員 <1+admin@noreply.localhost>
 2026-06-22 03:03:06 +00:00
admin 01482aa2ec Merge pull request 'feat(code-review-nuget): 新增 NuGet 套件更新 skill' (#14) from develop into master 
Reviewed-on: #14
Reviewed-by: 系統管理員 <1+admin@noreply.localhost>
 2026-06-22 02:50:29 +00:00
admin 52ac658106 Merge pull request 'chore(plugin 版本): bump 至 0.1.1' (#13) from develop into master 
Reviewed-on: #13
Reviewed-by: 系統管理員 <1+admin@noreply.localhost>
 2026-06-18 05:01:47 +00:00
admin c1bf04547c Merge pull request 'feat(code-review-resolve): 新增同步與同分支 PR 處理流程' (#12) from develop into master 
Reviewed-on: #12
Reviewed-by: 系統管理員 <1+admin@noreply.localhost>
 2026-06-17 09:50:20 +00:00
admin 71ea30ce05 Merge pull request 'chore(plugin 設定): 更新 code-review plugin 描述與關鍵字' (#11) from develop into master 
Reviewed-on: #11
Reviewed-by: 系統管理員 <1+admin@noreply.localhost>
 2026-06-17 03:52:35 +00:00
admin ea3d510a8e Merge pull request 'fix(code-review-resolve): 縮短 description 讓 Codex 載入 skill' (#9) from develop into master 
Reviewed-on: #9
Reviewed-by: 系統管理員 <1+admin@noreply.localhost>
 2026-06-17 01:56:59 +00:00
jiantw83 431d398094 Merge pull request 'feat: 新增 code-review-resolve skill（解決 findings、分類提交、push、發 PR）' (#8) from develop into master 
Reviewed-on: #8
 2026-06-16 07:54:15 +00:00
admin ceda1d3363 Merge pull request '所有 skill 輸出規範：一律繁體中文、UTF-8 無亂碼，版本 bump → 0.0.5' (#6) from develop into master 
Reviewed-on: #6
Reviewed-by: 系統管理員 <1+admin@noreply.localhost>
 2026-06-16 01:35:24 +00:00
admin 60165d07c2 Merge pull request 'skill 改名 archive-findings → code-review-archive，版本 bump → 0.0.4' (#5) from develop into master 
Reviewed-on: #5
Reviewed-by: 系統管理員 <1+admin@noreply.localhost>
 2026-06-15 10:35:43 +00:00
admin a6799326e4 Merge pull request 'feat: 攻擊方 diff 過濾 + 預設檔名更名' (#4) from develop into master 
Reviewed-on: #4
Reviewed-by: 系統管理員 <1+admin@noreply.localhost>
 2026-06-15 10:23:46 +00:00
admin d0f209bc19 Merge pull request 'docs: include --known-issues in code-review usage examples' (#3) from develop into master 
Reviewed-on: #3
Reviewed-by: 系統管理員 <1+admin@noreply.localhost>
 2026-06-15 09:48:33 +00:00
admin 2edc65f816 Merge pull request 'feat: add /jsc:archive-findings skill (persist code-review verdicts)' (#2) from develop into master 
Reviewed-on: #2
Reviewed-by: 系統管理員 <1+admin@noreply.localhost>
 2026-06-15 09:44:13 +00:00
jiantw83 d8685a03c4 Merge pull request 'feat: 新增 code-review skill' (#1) from develop into master 
Reviewed-on: #1
Reviewed-by: 系統管理員 <1+admin@noreply.localhost>
 2026-06-15 08:39:23 +00:00
1 changed files with 4 additions and 1 deletions
Expand all files Collapse all files
skills/code-review-nuget/SKILL.md
+4 -1
View File
@@ -14,6 +14,7 @@ argument-hint: "[<solution-or-project>] [--include-prerelease] [--no-dedupe] [--
- 修改前先輸出簡短執行計畫；除非使用者要求確認、遇到多個 solution 無法判斷、或編譯失敗需要取捨，否則依計畫執行。
- 每個套件都要留下結果：已更新 / 已降階到可編譯版本 / 已略過 / 已回復 / 已移除重複參考，以及對應版本與驗證指令。
- 不主動 commit、push 或開 PR；使用者明確要求時才做。
- **遇到 `JSC` 開頭的套件（套件 id 前綴為 `JSC`）時，必須先通知使用者並詢問是否要繼續處理該套件，取得明確同意後才可動作（更新版本或移除重複參考）。此確認為強制項，不得被 `--yes` 略過。使用者若拒絕，將該套件記為「已略過（JSC 套件，使用者未同意）」並繼續處理其他套件。**
## 參數
@@ -23,7 +24,7 @@ argument-hint: "[<solution-or-project>] [--include-prerelease] [--no-dedupe] [--
- `--include-prerelease`：更新時允許 prerelease 版本；未指定時只更新 stable 版本。
- `--no-dedupe`：略過 ProjectReference 重複 PackageReference 清理，只更新版本。
- `--build <command>`：覆寫驗證指令，例如 `dotnet build My.sln -c Release`。未指定時用 `dotnet build <solution-or-project> --no-restore`。
- `--yes`：可略過一般確認，但仍不可忽略多 solution 選擇、無法安全清理重複參考、或編譯失敗的必要決策。
- `--yes`：可略過一般確認，但仍不可忽略多 solution 選擇、無法安全清理重複參考、編譯失敗的必要決策、或 `JSC` 開頭套件的繼續確認。
## 執行流程
@@ -82,6 +83,7 @@ argument-hint: "[<solution-or-project>] [--include-prerelease] [--no-dedupe] [--
- 若目前專案的套件參考含 `PrivateAssets="all"`、`IncludeAssets` / `ExcludeAssets` 特殊設定、analyzers/build/source generators、或版本刻意高於上層專案，先列為「需人工確認」，不要自動移除。
- 若上層專案的參考含 `PrivateAssets="all"` 或不會傳遞必要 assets，不可視為可替代。
3. 對可安全移除的重複 direct reference 逐一處理：
- **若該套件 id 為 `JSC` 開頭，先通知使用者並詢問是否繼續移除，取得同意才處理；未同意則記為「已略過（JSC 套件，使用者未同意）」。此確認不得被 `--yes` 略過。**
- 先記錄目前檔案狀態與套件版本。
- 移除當前專案的重複 `PackageReference`；若使用 Central Package Management，只有在沒有任何專案仍需要該 `PackageVersion` 時才移除中央版本項。
- 執行：
@@ -100,6 +102,7 @@ argument-hint: "[<solution-or-project>] [--include-prerelease] [--no-dedupe] [--
- transitive-only 套件不直接更新；更新提供它的 direct 套件。
- 明確 pin 住版本且註解或 props 顯示有相容性原因者，先列入「需人工確認」。
- `PrivateAssets="all"` 的 analyzer / generator / build tooling 可更新，但要特別注意 build 驗證結果。
- **`JSC` 開頭的套件：在更新版本前，必須先通知使用者「即將處理 JSC 套件 `<PackageId>`」並詢問是否繼續，取得同意才更新；未同意則記為「已略過（JSC 套件，使用者未同意）」。此確認不得被 `--yes` 略過。**
2. 逐一更新 direct 套件：
- 一次只更新一個套件 id。
- 若多個專案引用同一套件，先判斷是否使用 Central Package Management：