chore: update ai review exclusions

This commit is contained in:
2026-05-16 14:57:30 +00:00
parent c4ecc3144f
commit 9ab2ff9de2
+28
View File
@@ -6,6 +6,13 @@
"suggestion": "新引入的 `require_var` 函數是腳本核心參數驗證邏輯,但缺乏獨立的單元測試。請為此函數建立測試,確保它在變數為空字串、為 \"null\" 字串以及有效值時,都能正確地判斷並執行預期的行為(例如:在無效情況下退出並輸出錯誤訊息至 stderr)。", "suggestion": "新引入的 `require_var` 函數是腳本核心參數驗證邏輯,但缺乏獨立的單元測試。請為此函數建立測試,確保它在變數為空字串、為 \"null\" 字串以及有效值時,都能正確地判斷並執行預期的行為(例如:在無效情況下退出並輸出錯誤訊息至 stderr)。",
"is_new": true "is_new": true
}, },
{
"level": "critical",
"role": "Aria",
"location": "a/entrypoint.sh:56",
"suggestion": "舊版腳本缺少檔案結尾的換行符 (newline)。根據 POSIX 標準,文字檔案應以換行符結束。新版已修正此問題,這是重要的風格與相容性改進。",
"is_new": true
},
{ {
"level": "warning", "level": "warning",
"role": "Zara", "role": "Zara",
@@ -48,6 +55,20 @@
"suggestion": "`dotnet pack` 命令是此腳本的核心功能,但腳本僅依賴其退出碼判斷成功與否。即使命令成功,也應進一步驗證是否確實產生了預期的 `.nupkg` 套件檔案。請在 `dotnet pack` 執行後,加入檢查 `$OUTPUT_DIR` 中是否存在 `.nupkg` 檔案的步驟,以確保打包結果的完整性。", "suggestion": "`dotnet pack` 命令是此腳本的核心功能,但腳本僅依賴其退出碼判斷成功與否。即使命令成功,也應進一步驗證是否確實產生了預期的 `.nupkg` 套件檔案。請在 `dotnet pack` 執行後,加入檢查 `$OUTPUT_DIR` 中是否存在 `.nupkg` 檔案的步驟,以確保打包結果的完整性。",
"is_new": true "is_new": true
}, },
{
"level": "warning",
"role": "Aria",
"location": "a/entrypoint.sh:44-51",
"suggestion": "舊版腳本中的 `dotnet pack` 命令過長且未分行,降低了可讀性。新版將其拆分為多行並適當縮排,顯著提升了程式碼的可讀性與維護性,這是良好的風格改進。",
"is_new": true
},
{
"level": "warning",
"role": "Aria",
"location": "a/entrypoint.sh:34-35",
"suggestion": "舊版腳本中 `RELEASE_COPYRIGHT` 變數的賦值方式較為混亂,先賦予年份再重新組合字串。新版直接在宣告時完成組合,並使用 `readonly` 標記,提升了程式碼的清晰度與變數管理的嚴謹性。",
"is_new": true
},
{ {
"level": "info", "level": "info",
"role": "Aria", "role": "Aria",
@@ -89,5 +110,12 @@
"location": "entrypoint.sh", "location": "entrypoint.sh",
"suggestion": "雖然程式碼的可讀性已大幅提升,但考慮在檔案開頭添加簡要的腳本用途說明,以及預期環境變數的列表,將有助於新維護者快速理解其功能。", "suggestion": "雖然程式碼的可讀性已大幅提升,但考慮在檔案開頭添加簡要的腳本用途說明,以及預期環境變數的列表,將有助於新維護者快速理解其功能。",
"is_new": true "is_new": true
},
{
"level": "info",
"role": "Aria",
"location": "b/entrypoint.sh:59",
"suggestion": "在 `dotnet pack` 命令中,參數 `/p:RepositoryType=git` 的值 'git' 未使用引號。雖然在此特定情況下沒有問題,但為了風格一致性,可以考慮對所有參數值使用引號,例如 `/p:RepositoryType=\"git\"`。",
"is_new": true
} }
] ]