old-docker-actions/release-cleanup
2
0
Fork 0
Code Pull Requests Actions Releases 4 Activity

refactor(release-cleanup): 將清理邏輯由 bash 改寫為 Node.js #6

Closed
jiantw83 wants to merge 60 commits from refactor/nodejs-rewrite-20260626-103443 into develop
pull from: refactor/nodejs-rewrite-20260626-103443
merge into: :develop
Conversation 6 Commits 60 Files Changed 25
+3 -1
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit f1fce4b6e4 - Show all commits
app/index.js
+3 -1
View File
@@ -1,5 +1,6 @@
// 進入點:載入設定、建立 Gitea 客戶端,依序清理舊成品與孤立 tag。 // 進入點:載入設定、建立 Gitea 客戶端,依序清理舊成品與孤立 tag。
import { pathToFileURL } from 'node:url'
import { loadConfig } from './config.js' import { loadConfig } from './config.js'
import { GiteaClient } from './gitea-client.js' import { GiteaClient } from './gitea-client.js'
import { cleanupReleases } from './releases.js' import { cleanupReleases } from './releases.js'
@@ -25,7 +26,8 @@ export async function main() {
} }
// 僅在直接以 `node index.js` 執行時啟動主流程;被測試 import 時不自動執行,方便撰寫整合測試。 // 僅在直接以 `node index.js` 執行時啟動主流程;被測試 import 時不自動執行,方便撰寫整合測試。
if (import.meta.url === `file://${process.argv[1]}`) { // 以 pathToFileURL 正確處理含空白/特殊字元的路徑與跨平台差異,避免手動拼接 file:// 的脆弱性。
if (import.meta.url === pathToFileURL(process.argv[1]).href) {
Ghost marked this conversation as resolved Outdated
gitea-actions commented 2026-06-26 06:10:30 +00:00
Outdated
Review
Copy Link
Copy Source

嚴重等級：🔵 建議
審查員：Bard
問題：手動拼接 import.meta.url 字串來判斷執行入口，寫法原始且脆弱。
建議：引用 node:url 中的 pathToFileURL，使用 import.meta.url === pathToFileURL(process.argv[1]).href 進行比較。

**嚴重等級**：🔵 建議 **審查員**：Bard **問題**：手動拼接 import.meta.url 字串來判斷執行入口，寫法原始且脆弱。 **建議**：引用 node:url 中的 pathToFileURL，使用 import.meta.url === pathToFileURL(process.argv[1]).href 進行比較。
main().catch((error) => { main().catch((error) => {
failError(error) failError(error)
process.exit(1) process.exit(1)