fix(hooks): use cmd-compatible quoting for SessionStart run-hook

[stablegenius49]

Summary

• replace the single-quoted SessionStart hook command with escaped double quotes in hooks/hooks.json
• preserve spaces in CLAUDE_PLUGIN_ROOT paths while remaining compatible with Windows cmd.exe

Validation

• python3 -m json.tool hooks/hooks.json

Closes #529

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 41cd46e0-688b-4d66-9caf-1cf8b147b010

📥 Commits

Reviewing files that changed from the base of the PR and between e4a2375 and c929ea2.

📒 Files selected for processing (1)

• hooks/hooks.json

---

📝 Walkthrough

Walkthrough

This change updates the hook command string in hooks.json by modifying how the path to run-hook.cmd is quoted and escaped, addressing Windows path handling in the SessionStart hook invocation.

Changes

Cohort / File(s)	Summary
Hook Configuration Update hooks/hooks.json	Modified command string quoting from single quotes to escaped double quotes to properly handle Windows path execution for the SessionStart hook invocation.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related issues

• ● v4.3.1 regressed Windows hook fix from PR #331 #529 — Directly addresses the Windows hook regression where v4.3.1 reverted hook command invocation, causing SessionStart:startup errors on Windows.
• SessionStart hook fails on Windows — bash cannot resolve ${CLAUDE_PLUGIN_ROOT} path #389 — Modifies the same hooks.json command for the SessionStart hook with quoting/path handling changes.
• SessionStart hook fails on Windows: CLAUDE_PLUGIN_ROOT backslashes mangled by bash #487 — Changes hooks.json command string quoting to prevent shell word-splitting issues with plugin paths containing spaces on Windows.
• SessionStart hook fails on Windows when user profile path contains spaces #518 — Updates hooks.json command quoting (switching to escaped double quotes) for Windows/CMD compatibility.
• SessionStart hook fails on Windows — hooks.json points to .sh file which CMD.exe cannot execute #491 — Modifies hooks.json command to use escaped double-quoted wrapper invocation for Windows/CMD execution.

Possibly related PRs

• fix: restore polyglot wrapper for Windows hook compatibility (4.3.1) #523 — Directly modifies the same hooks.json command string quoting for the SessionStart hook.
• Add Windows support for plugin hooks #134 — Related change to hooks.json SessionStart hook invocation switching to run-hook.cmd approach.
• fix: OpenCode native skills + Windows hook execution #335 — Updates hooks.json command invocation for Windows hook execution handling.

Poem

🐰 A quote escaped, a path now clear,
Windows hooks shall have no fear—
Double-quoted, rightly blessed,
The SessionStart shall pass its test! 🪝✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly describes the main change: fixing Windows cmd-compatible quoting for the SessionStart hook command in hooks.json, which directly addresses the PR's objective.
Description check	✅ Passed	The description explains the change (replacing single quotes with escaped double quotes), the motivation (Windows cmd compatibility and preserving spaces in paths), and provides validation steps.
Linked Issues check	✅ Passed	The PR changes hooks.json to use escaped double quotes for the SessionStart run-hook command, directly addressing issue #529's requirement to fix Windows cmd.exe compatibility that was regressed in v4.3.1.
Out of Scope Changes check	✅ Passed	The PR makes a single focused change to hooks.json affecting only the SessionStart command quoting, which is directly within the scope of issue #529.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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.}