fix/deploy-hook-runtime-path #32

Merged
admin merged 2 commits from fix/deploy-hook-runtime-path into develop 2026-08-28 03:27:53 +00:00
Member

PR 描述範本

摘要

  • 需求描述:讓 jsc-cli 部署收尾不要只從單一 CLI 快取尋找 restart-gate.sh,並把 hook smoke 納入 install、update 成敗。
  • 計畫名稱:無
  • 計畫頁:無
  • 分析頁:無

變更內容

檔案 為什麼改
tools/deploy.sh 尋找 restart-gate.sh 時優先使用 $JSC_HOME/current/jsc-hooks、本地 clone 與 Kiro skills,再掃各 CLI 快取。
skills/deploy/SKILL.md 要求 install、update 後的 hooks-install 必須回報每支 CLI 的 smoke 結果;No such file 視為部署失敗。
README.md 同步部署收尾路徑搜尋與 smoke 成敗規則。
plugin.json、.claude-plugin/plugin.json、.codex-plugin/plugin.json 發布 jsc-cli 0.2.1。

設計重點

  • 部署收尾優先使用穩定連結。
  • 找不到 restart-gate.sh 時沿用既有 note 回報,不自行寫狀態檔。
  • deploy 的成功報告必須包含 hooks 的執行期驗證。

測試結果

  • sh -n tools/deploy.sh:通過。
  • git diff --check:通過。
  • JSC_DEPLOY_DRYRUN=1 sh tools/deploy.sh -n update kiro hooks:通過,維持 Kiro 複製退路。

前置 Push Request

# PR 描述範本 ## 摘要 - 需求描述:讓 jsc-cli 部署收尾不要只從單一 CLI 快取尋找 `restart-gate.sh`,並把 hook smoke 納入 install、update 成敗。 - 計畫名稱:無 - 計畫頁:無 - 分析頁:無 ## 變更內容 | 檔案 | 為什麼改 | | --- | --- | | tools/deploy.sh | 尋找 `restart-gate.sh` 時優先使用 `$JSC_HOME/current/jsc-hooks`、本地 clone 與 Kiro skills,再掃各 CLI 快取。 | | skills/deploy/SKILL.md | 要求 install、update 後的 hooks-install 必須回報每支 CLI 的 smoke 結果;`No such file` 視為部署失敗。 | | README.md | 同步部署收尾路徑搜尋與 smoke 成敗規則。 | | plugin.json、.claude-plugin/plugin.json、.codex-plugin/plugin.json | 發布 `jsc-cli` 0.2.1。 | ## 設計重點 - 部署收尾優先使用穩定連結。 - 找不到 `restart-gate.sh` 時沿用既有 note 回報,不自行寫狀態檔。 - `deploy` 的成功報告必須包含 hooks 的執行期驗證。 ## 測試結果 - `sh -n tools/deploy.sh`:通過。 - `git diff --check`:通過。 - `JSC_DEPLOY_DRYRUN=1 sh tools/deploy.sh -n update kiro hooks`:通過,維持 Kiro 複製退路。 ## 前置 Push Request - https://gitea.jsc.idv.tw/plugins/hooks/pulls/42
jiantw83 added 2 commits 2026-08-28 03:23:18 +00:00
jiantw83 added a new dependency 2026-08-28 03:23:22 +00:00
admin merged commit c4e75f7f83 into develop 2026-08-28 03:27:53 +00:00
admin deleted branch fix/deploy-hook-runtime-path 2026-08-28 03:27:53 +00:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Depends on
Reference: plugins/cli#32