Skip to content

refactor(deploy): 出力先へのテンプレート配布を廃止 #5

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
aloekun merged 2 commits into from
Mar 22, 2026
Merged

refactor(deploy): 出力先へのテンプレート配布を廃止 #5

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
8e3dff4
refactor(deploy): 出力先へのテンプレート配布を廃止
aloekun Mar 22, 2026
9629eff
fix(deploy): CodeRabbitレビュー反映 - エラーログ改善とコメント修正
aloekun Mar 22, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 12 additions & 8 deletions scripts/deploy-hooks.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,8 @@
* 動作:
* 1. deploy-targets.json からターゲットプロジェクト一覧を読み込み
* 2. 各ターゲットの .claude/ ディレクトリに exe をコピー
* 3. hooks-config.toml が存在しない場合のみ、テンプレートをコピー
* 3. settings.local.json.template を解決して settings.local.json を生成
* 4. hooks-config.toml が存在しない場合は作成を促すメッセージを表示
*
* 使い方: pnpm deploy:hooks
*/
Expand Down Expand Up @@ -79,18 +80,21 @@ function deployTo(targetDir) {
);
}

// settings.local.json.template をコピー (常に最新に更新)
// settings.local.json を生成 (hooks のみ更新、既存 permissions 等は保持)
const templateSrc = path.join(CLAUDE_DIR, SETTINGS_TEMPLATE);
if (fs.existsSync(templateSrc)) {
copyFile(templateSrc, path.join(targetClaude, SETTINGS_TEMPLATE));

// settings.local.json を生成 (hooks のみ更新、既存 permissions 等は保持)
const template = fs.readFileSync(templateSrc, "utf8");
const resolved = template.replace(
/\{\{PROJECT_DIR\}\}/g,
targetDir.replace(/\\/g, "\\\\")
);
const newSettings = JSON.parse(resolved);
let newSettings;
try {
newSettings = JSON.parse(resolved);
} catch (e) {
console.log(` WARN: Failed to parse ${SETTINGS_TEMPLATE}: ${e.message}`);
return true;
}
if (
!newSettings ||
typeof newSettings !== "object" ||
Expand All @@ -113,9 +117,9 @@ function deployTo(targetDir) {
existing.hooks = newSettings.hooks;
fs.writeFileSync(settingsDest, JSON.stringify(existing, null, 2) + "\n");
console.log(" updated: settings.local.json (hooks merged, permissions preserved)");
} catch {
} catch (e) {
fs.writeFileSync(settingsDest, JSON.stringify(newSettings, null, 2) + "\n");
console.log(" WARN: existing settings.local.json was invalid, regenerated");
console.log(` WARN: existing settings.local.json was invalid (${e.message}), regenerated`);
}
} else {
fs.writeFileSync(settingsDest, JSON.stringify(newSettings, null, 2) + "\n");
Expand Down