feat(logger): Logger拡張・JSTタイムスタンプ追加・スクリプトTS化

[aloekun]

Summary

• Logger に JST タイムスタンプ出力を追加（2026/03/30 14:45:43 [INFO] ... 形式）
• scripts/deploy-hooks.js → .ts に変換、全 console.log を Logger に置換
• scripts/e2e.mjs → .ts に変換、console.log を Logger に置換
• package.json の deploy:hooks / test:e2e を npx tsx 実行に変更

Why

• カスタムリンター (no-console-log) との一貫性: プロジェクト内のすべての出力を Logger 経由に統一
• TypeScript 化により静的解析の恩恵を受けられる（型安全性・AIコーディングとの親和性向上）
• タイムスタンプ追加でデプロイ失敗時の時系列追跡が容易に

Test plan

• pnpm deploy:hooks — 2ターゲットに正常デプロイ、JSTタイムスタンプ付きログ出力を確認
• pnpm test:e2e — .env.e2e なしで正常スキップを確認
• sample.ts の console.log は意図的なリントテスト用として維持

🤖 Generated with Claude Code

Summary by CodeRabbit

リリースノート

• 改善

  • ログ出力にJST形式のタイムスタンプが追加され、ログ可読性が向上しました
  • E2Eランナーの起動ログが統一ロガー経由になり、スキップ通知などが一貫して記録されます

• Chores

  • テスト／運用スクリプトをTypeScriptで実行するよう移行し、実行方法をモダン化しました
  • フックの一括展開処理を更新し、各プロジェクトへのフック配布と設定ファイル生成が改善されました

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 5efac996-1039-4564-a625-ba74016757b1

📥 Commits

Reviewing files that changed from the base of the PR and between 44c2cde and 9e4f2ea.

📒 Files selected for processing (5)

• package.json
• scripts/deploy-hooks.js
• scripts/deploy-hooks.ts
• scripts/e2e.ts
• src/logger.ts

💤 Files with no reviewable changes (1)

• scripts/deploy-hooks.js

✅ Files skipped from review due to trivial changes (2)

• scripts/e2e.ts
• package.json

🚧 Files skipped from review as they are similar to previous changes (2)

• src/logger.ts
• scripts/deploy-hooks.ts

---

📝 Walkthrough

Walkthrough

TypeScript化と実行コマンド更新：deploy-hooksをJS→TSに移植し、test:e2e/deploy:hooksは npx tsx 経由で実行するように変更。ログにJSTタイムスタンプを追加し、E2Eスクリプトの初期スキップ出力を logger.info に変更した。

Changes

Cohort / File(s)	Summary
スクリプト実行設定更新 package.json	test:e2e と deploy:hooks スクリプトを node 実行から npx tsx を使った TypeScript 実行に変更。
デプロイスクリプト移行 scripts/deploy-hooks.js, scripts/deploy-hooks.ts, scripts/deploy-targets.json	scripts/deploy-hooks.js を削除し、同等のロジックを持つ scripts/deploy-hooks.ts を追加。ターゲット読み取り、各ターゲットへの .claude ディレクトリ作成、フック実行ファイルのコピー、hooks-config.toml の存在チェック、settings.local.json.template の展開・検証・既存ファイルへの hooks フィールドマージまたは上書きを行う。scripts/deploy-targets.json を参照する挙動を維持。
E2Eランナー更新 scripts/e2e.ts	.env.e2e がない場合のスキップ出力を console.log から logger.info に変更し、ファイル上部に JSDoc コメントを追加。動作フローは変更なし。
ロガー強化 src/logger.ts	JST（Asia/Tokyo）のタイムスタンプを生成する jstTimestamp() を追加し、各ログ出力で YYYY/MM/DD ... [LEVEL] のプリフィクスを付与するようにフォーマットを変更。ログレベル判定やAPIはそのまま。

Sequence Diagram(s)

sequenceDiagram
    autonumber
    participant Runner as deploy-hooks.ts
    participant Repo as repo/.claude + template
    participant Targets as target projects (each)
    participant FS as filesystem
    participant Logger as logger/console

    Runner->>FS: read scripts/deploy-targets.json
    alt valid targets
        Runner->>Targets: for each target -> check dir exists
        Targets-->>Runner: exists / missing
        alt exists
            Runner->>FS: ensure <target>/.claude/ exists
            Runner->>Repo: read repo/.claude/* hook binaries
            Repo-->>Runner: hook files (or missing)
            Runner->>FS: copy hook binaries into <target>/.claude/
            Runner->>FS: check <target>/.claude/hooks-config.toml
            Runner->>Repo: read settings.local.json.template (if exists)
            Repo-->>Runner: template content
            Runner->>Runner: replace {{PROJECT_DIR}} -> resolved JSON
            Runner->>FS: read existing <target>/.claude/settings.local.json (if any)
            alt existing valid JSON
                Runner->>Runner: merge only "hooks" field
            else
                Runner->>FS: write resolved settings.local.json (overwrite)
            end
            Runner->>Logger: log per-target result
        end
    end
    Runner->>Logger: log final success/total summary

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

• refactor(deploy): 出力先へのテンプレート配布を廃止 #5: deploy-hooks のテンプレート処理と settings.local.json の生成・マージロジックに関する同一機能の変更が含まれる。
• feat(hooks): カスタムリンターエンジンの導入 #6: src/logger.ts のログ実装・フォーマット変更に関わる同一ファイルの修正がある。
• fix(hooks): Replace matcher追加 & deploy時のpermissions保持 #4: deploy-hooks ロジックで hooks フィールドだけをマージして既存設定を保持する処理の実装に強い関連がある。

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	このPRのタイトルは、Logger拡張、JSTタイムスタンプ追加、スクリプトのTypeScript化という3つの主要な変更を簡潔に要約しており、変更セットの主な目的と完全に関連しています。

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@scripts/deploy-hooks.ts`:
- Around line 87-100: The template parsing branch in scripts/deploy-hooks.ts
treats a JSON parse failure as success by returning true; change the failure
return to false so the caller correctly counts failures—specifically inside the
block that reads templateSrc/SETTINGS_TEMPLATE where you catch the JSON.parse
error (the catch that calls logger.warn) replace the current return true with
return false; also update the similar return at the other occurrence around the
later template resolution (the analogous catch/return near lines 149–151) so
both failure paths return false and the success++ aggregation reflects real
successes.
- Around line 43-44: The code currently returns data.targets directly
(DeployTargets → data.targets) which can be a wrong type at runtime; add a
runtime validation after parsing (using targetsPath/readFileSync result) to
ensure data.targets is an array of strings (string[]) and if not, log an error
via processLogger (or console.error) with the file/field and exit(1) so the
later loop cannot misbehave; locate the JSON parse area where DeployTargets is
read and replace the direct return of data.targets ?? [] with a check like:
ensure Array.isArray(data.targets) && data.targets.every(t => typeof t ===
"string"), otherwise abort.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 27e8bd1f-daf8-4066-bbb4-094b91c4d239

📥 Commits

Reviewing files that changed from the base of the PR and between fa2f186 and 44c2cde.

📒 Files selected for processing (5)

• package.json
• scripts/deploy-hooks.js
• scripts/deploy-hooks.ts
• scripts/e2e.ts
• src/logger.ts

💤 Files with no reviewable changes (1)

• scripts/deploy-hooks.js