Skip to content

fix(adapter): quote hook command paths for spaces in project dir#791

Merged
smileygames merged 3 commits intomainfrom
790-fix-hook-path-quoting
Mar 20, 2026
Merged

fix(adapter): quote hook command paths for spaces in project dir#791
smileygames merged 3 commits intomainfrom
790-fix-hook-path-quoting

Conversation

@liplus-lin-lay
Copy link
Collaborator

Refs #790
settings.json テンプレートのスペース入りパスでの hook 実行失敗を修正。bash クォート + PATH 追加。

settings.json テンプレートの hook command パスがスペースを含むディレクトリで壊れる問題を修正。
bash "$CLAUDE_PROJECT_DIR/..." 形式にクォートし、hook スクリプトに PATH=$HOME/.local/bin を追加。

Refs #790

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Collaborator

@smileygames smileygames left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DOCSの方は修正要らないの?

liplus-lin-lay and others added 2 commits March 20, 2026 23:30
パス安全性セクションを追加。settings.json のクォート要件と hook スクリプトの PATH 設定要件を明記。

Refs #790

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
on_commit の end セクションを "PR And CI Flow" → "PR Creation" に、
on_merge の start セクションを "Merge And Cleanup" → "Merge" に修正。
Li+operations.md の実際のセクション名と一致させる。

Refs #790

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Collaborator

@smileygames smileygames left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

おk

@smileygames smileygames merged commit 360258e into main Mar 20, 2026
2 checks passed
@smileygames smileygames deleted the 790-fix-hook-path-quoting branch March 20, 2026 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(adapter): hook scripts fail on Windows when project path contains spaces

2 participants