feat: add hooks system for CLI event interception

[sestinj]

Summary

• Adds a Claude Code-compatible hooks system for the Continue CLI that allows external handlers (shell commands, HTTP endpoints) to intercept and respond to 16 CLI event types
• Supports regex-based event matching, typed inputs/outputs, sync/async execution, and multi-source config merging from .continue/ and .claude/ directories
• Includes comprehensive test coverage

Test plan

• Verify hooks config loading from multiple sources
• Test command hook execution with stdin JSON piping
• Validate exit code semantics (0=proceed, 2=block)
• Confirm async hooks fire without blocking

🤖 Generated with Claude Code

---

Continue Tasks: ✅ 7 no changes — View all

---

Summary by cubic

Adds a Claude Code–compatible hooks system to the Continue CLI so external command and HTTP handlers can intercept and influence 17 CLI events. Registers HookService and tightens execution with UTF-8-safe streams, correct env var interpolation, clean EPIPE handling, and ESLint fixes.

• New Features

  • HookService with fireEvent, reloadConfig, and getCommonFields; exposed via services.hooks.
  • Config merging from ~/.continue, .continue (including settings.local.json), and .claude; supports disableAllHooks.
  • Command/HTTP hooks follow Claude semantics (exit 0/2, JSON parsing); CONTINUE_PROJECT_DIR/CLAUDE_PROJECT_DIR set; async hooks, regex matching, dedup, parallel execution; defaults: 600s command, 30s HTTP.
  • fireHook helpers for common events; documented in AGENTS.md.

• Bug Fixes

  • Use setEncoding("utf8") for stdout/stderr to avoid multibyte corruption.
  • Fix HTTP header env var interpolation to handle ${VAR} and $VAR forms.
  • Only suppress EPIPE errors on stdin writes; log other stdin errors.
  • Register HookService in the service container and expose via services.
  • React hooks cleanup in useService: memoize reload/deps, remove invalid eslint-disable comment, and skip hookRunner tests on Windows.

^{Written for commit 8782e38. Summary will update on new commits.}

[cubic-dev-ai]

3 issues found across 6 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="extensions/cli/src/hooks/hookRunner.ts">
<violation number="1" location="extensions/cli/src/hooks/hookRunner.ts:89">
P2: Using `data.toString()` per chunk corrupts multi-byte UTF-8 characters split across buffer boundaries. Set encoding on the streams so Node's internal `StringDecoder` handles this correctly.
(Based on your team's feedback about setting encoding to "utf8" on stdout/stderr streams instead of using `toString()` per 'data' event.) [FEEDBACK_USED]</violation>
<violation number="2" location="extensions/cli/src/hooks/hookRunner.ts:98">
P1: Missing error handler on `child.stdin` — if the child process closes its stdin pipe before the write completes, the unhandled `'error'` event on the stdin stream can crash the Node.js process.</violation>
<violation number="3" location="extensions/cli/src/hooks/hookRunner.ts:149">
P2: Regex `\$\{?(\w+)\}?` incorrectly matches unbalanced braces (e.g., `$FOO}` consumes the `}` even without an opening `{`). Use alternation to properly handle both `${VAR}` and `$VAR` forms.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[cubic-dev-ai]

1 issue found across 1 file (changes from recent commits).

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="extensions/cli/src/hooks/hookRunner.ts">
<violation number="1" location="extensions/cli/src/hooks/hookRunner.ts:98">
P2: The new stdin error handler ignores every error, not just EPIPE as intended; this masks unexpected write failures.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[sestinj]

Summary

• Adds a Claude Code-compatible hooks system for the Continue CLI that allows external handlers (shell commands, HTTP endpoints) to intercept and respond to 16 CLI event types
• Supports regex-based event matching, typed inputs/outputs, sync/async execution, and multi-source config merging from .continue/ and .claude/ directories
• Includes comprehensive test coverage

Test plan

• Verify hooks config loading from multiple sources
• Test command hook execution with stdin JSON piping
• Validate exit code semantics (0=proceed, 2=block)
• Confirm async hooks fire without blocking

🤖 Generated with Claude Code

---

Continue Tasks: ✅ 7 no changes — View all

---

Summary by cubic

Adds a Claude Code–compatible hooks system to the Continue CLI so external command and HTTP handlers can intercept and influence 17 CLI events. Registers HookService and tightens execution with UTF-8-safe streams, correct env var interpolation, clean EPIPE handling, and ESLint fixes.

• New Features

  • HookService with fireEvent, reloadConfig, and getCommonFields; exposed via services.hooks.
  • Config merging from ~/.continue, .continue (including settings.local.json), and .claude; supports disableAllHooks.
  • Command/HTTP hooks follow Claude semantics (exit 0/2, JSON parsing); CONTINUE_PROJECT_DIR/CLAUDE_PROJECT_DIR set; async hooks, regex matching, dedup, parallel execution; defaults: 600s command, 30s HTTP.
  • fireHook helpers for common events; documented in AGENTS.md.

• Bug Fixes

  • Use setEncoding("utf8") for stdout/stderr to avoid multibyte corruption.
  • Fix HTTP header env var interpolation to handle ${VAR} and $VAR forms.
  • Only suppress EPIPE errors on stdin writes; log other stdin errors.
  • Register HookService in the service container and expose via services.
  • React hooks cleanup in useService: memoize reload/deps, remove invalid eslint-disable comment, and skip hookRunner tests on Windows.

^{Written for commit 8782e38. Summary will update on new commits.}

[cubic-dev-ai]

3 issues found across 6 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="extensions/cli/src/hooks/hookRunner.ts">
<violation number="1" location="extensions/cli/src/hooks/hookRunner.ts:89">
P2: Using `data.toString()` per chunk corrupts multi-byte UTF-8 characters split across buffer boundaries. Set encoding on the streams so Node's internal `StringDecoder` handles this correctly.
(Based on your team's feedback about setting encoding to "utf8" on stdout/stderr streams instead of using `toString()` per 'data' event.) [FEEDBACK_USED]</violation>
<violation number="2" location="extensions/cli/src/hooks/hookRunner.ts:98">
P1: Missing error handler on `child.stdin` — if the child process closes its stdin pipe before the write completes, the unhandled `'error'` event on the stdin stream can crash the Node.js process.</violation>
<violation number="3" location="extensions/cli/src/hooks/hookRunner.ts:149">
P2: Regex `\$\{?(\w+)\}?` incorrectly matches unbalanced braces (e.g., `$FOO}` consumes the `}` even without an opening `{`). Use alternation to properly handle both `${VAR}` and `$VAR` forms.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[cubic-dev-ai]

1 issue found across 1 file (changes from recent commits).

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="extensions/cli/src/hooks/hookRunner.ts">
<violation number="1" location="extensions/cli/src/hooks/hookRunner.ts:98">
P2: The new stdin error handler ignores every error, not just EPIPE as intended; this masks unexpected write failures.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[sestinj]

Summary

• Adds a Claude Code-compatible hooks system for the Continue CLI that allows external handlers (shell commands, HTTP endpoints) to intercept and respond to 16 CLI event types
• Supports regex-based event matching, typed inputs/outputs, sync/async execution, and multi-source config merging from .continue/ and .claude/ directories
• Includes comprehensive test coverage

Test plan

• Verify hooks config loading from multiple sources
• Test command hook execution with stdin JSON piping
• Validate exit code semantics (0=proceed, 2=block)
• Confirm async hooks fire without blocking

🤖 Generated with Claude Code

---

Continue Tasks: ✅ 7 no changes — View all

---

Summary by cubic

Adds a Claude Code–compatible hooks system to the Continue CLI so external command and HTTP handlers can intercept and influence 17 CLI events. Registers HookService and tightens execution with UTF-8-safe streams, correct env var interpolation, clean EPIPE handling, and ESLint fixes.

• New Features

  • HookService with fireEvent, reloadConfig, and getCommonFields; exposed via services.hooks.
  • Config merging from ~/.continue, .continue (including settings.local.json), and .claude; supports disableAllHooks.
  • Command/HTTP hooks follow Claude semantics (exit 0/2, JSON parsing); CONTINUE_PROJECT_DIR/CLAUDE_PROJECT_DIR set; async hooks, regex matching, dedup, parallel execution; defaults: 600s command, 30s HTTP.
  • fireHook helpers for common events; documented in AGENTS.md.

• Bug Fixes

  • Use setEncoding("utf8") for stdout/stderr to avoid multibyte corruption.
  • Fix HTTP header env var interpolation to handle ${VAR} and $VAR forms.
  • Only suppress EPIPE errors on stdin writes; log other stdin errors.
  • Register HookService in the service container and expose via services.
  • React hooks cleanup in useService: memoize reload/deps, remove invalid eslint-disable comment, and skip hookRunner tests on Windows.

^{Written for commit 8782e38. Summary will update on new commits.}

[cubic-dev-ai]

3 issues found across 6 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="extensions/cli/src/hooks/hookRunner.ts">
<violation number="1" location="extensions/cli/src/hooks/hookRunner.ts:89">
P2: Using `data.toString()` per chunk corrupts multi-byte UTF-8 characters split across buffer boundaries. Set encoding on the streams so Node's internal `StringDecoder` handles this correctly.
(Based on your team's feedback about setting encoding to "utf8" on stdout/stderr streams instead of using `toString()` per 'data' event.) [FEEDBACK_USED]</violation>
<violation number="2" location="extensions/cli/src/hooks/hookRunner.ts:98">
P1: Missing error handler on `child.stdin` — if the child process closes its stdin pipe before the write completes, the unhandled `'error'` event on the stdin stream can crash the Node.js process.</violation>
<violation number="3" location="extensions/cli/src/hooks/hookRunner.ts:149">
P2: Regex `\$\{?(\w+)\}?` incorrectly matches unbalanced braces (e.g., `$FOO}` consumes the `}` even without an opening `{`). Use alternation to properly handle both `${VAR}` and `$VAR` forms.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[cubic-dev-ai]

1 issue found across 1 file (changes from recent commits).

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="extensions/cli/src/hooks/hookRunner.ts">
<violation number="1" location="extensions/cli/src/hooks/hookRunner.ts:98">
P2: The new stdin error handler ignores every error, not just EPIPE as intended; this masks unexpected write failures.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[sestinj]

Summary

• Adds a Claude Code-compatible hooks system for the Continue CLI that allows external handlers (shell commands, HTTP endpoints) to intercept and respond to 16 CLI event types
• Supports regex-based event matching, typed inputs/outputs, sync/async execution, and multi-source config merging from .continue/ and .claude/ directories
• Includes comprehensive test coverage

Test plan

• Verify hooks config loading from multiple sources
• Test command hook execution with stdin JSON piping
• Validate exit code semantics (0=proceed, 2=block)
• Confirm async hooks fire without blocking

🤖 Generated with Claude Code

---

Continue Tasks: ✅ 7 no changes — View all

---

Summary by cubic

Adds a Claude Code–compatible hooks system to the Continue CLI so external command and HTTP handlers can intercept and influence 17 CLI events. Registers HookService and tightens execution with UTF-8-safe streams, correct env var interpolation, clean EPIPE handling, and ESLint fixes.

• New Features

  • HookService with fireEvent, reloadConfig, and getCommonFields; exposed via services.hooks.
  • Config merging from ~/.continue, .continue (including settings.local.json), and .claude; supports disableAllHooks.
  • Command/HTTP hooks follow Claude semantics (exit 0/2, JSON parsing); CONTINUE_PROJECT_DIR/CLAUDE_PROJECT_DIR set; async hooks, regex matching, dedup, parallel execution; defaults: 600s command, 30s HTTP.
  • fireHook helpers for common events; documented in AGENTS.md.

• Bug Fixes

  • Use setEncoding("utf8") for stdout/stderr to avoid multibyte corruption.
  • Fix HTTP header env var interpolation to handle ${VAR} and $VAR forms.
  • Only suppress EPIPE errors on stdin writes; log other stdin errors.
  • Register HookService in the service container and expose via services.
  • React hooks cleanup in useService: memoize reload/deps, remove invalid eslint-disable comment, and skip hookRunner tests on Windows.

^{Written for commit 8782e38. Summary will update on new commits.}

[cubic-dev-ai]

3 issues found across 6 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="extensions/cli/src/hooks/hookRunner.ts">
<violation number="1" location="extensions/cli/src/hooks/hookRunner.ts:89">
P2: Using `data.toString()` per chunk corrupts multi-byte UTF-8 characters split across buffer boundaries. Set encoding on the streams so Node's internal `StringDecoder` handles this correctly.
(Based on your team's feedback about setting encoding to "utf8" on stdout/stderr streams instead of using `toString()` per 'data' event.) [FEEDBACK_USED]</violation>
<violation number="2" location="extensions/cli/src/hooks/hookRunner.ts:98">
P1: Missing error handler on `child.stdin` — if the child process closes its stdin pipe before the write completes, the unhandled `'error'` event on the stdin stream can crash the Node.js process.</violation>
<violation number="3" location="extensions/cli/src/hooks/hookRunner.ts:149">
P2: Regex `\$\{?(\w+)\}?` incorrectly matches unbalanced braces (e.g., `$FOO}` consumes the `}` even without an opening `{`). Use alternation to properly handle both `${VAR}` and `$VAR` forms.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[cubic-dev-ai]

1 issue found across 1 file (changes from recent commits).

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="extensions/cli/src/hooks/hookRunner.ts">
<violation number="1" location="extensions/cli/src/hooks/hookRunner.ts:98">
P2: The new stdin error handler ignores every error, not just EPIPE as intended; this masks unexpected write failures.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[sestinj]

Summary

• Adds a Claude Code-compatible hooks system for the Continue CLI that allows external handlers (shell commands, HTTP endpoints) to intercept and respond to 16 CLI event types
• Supports regex-based event matching, typed inputs/outputs, sync/async execution, and multi-source config merging from .continue/ and .claude/ directories
• Includes comprehensive test coverage

Test plan

• Verify hooks config loading from multiple sources
• Test command hook execution with stdin JSON piping
• Validate exit code semantics (0=proceed, 2=block)
• Confirm async hooks fire without blocking

🤖 Generated with Claude Code

---

Continue Tasks: ✅ 7 no changes — View all

---

Summary by cubic

Adds a Claude Code–compatible hooks system to the Continue CLI so external command and HTTP handlers can intercept and influence 17 CLI events. Registers HookService and tightens execution with UTF-8-safe streams, correct env var interpolation, clean EPIPE handling, and ESLint fixes.

• New Features

  • HookService with fireEvent, reloadConfig, and getCommonFields; exposed via services.hooks.
  • Config merging from ~/.continue, .continue (including settings.local.json), and .claude; supports disableAllHooks.
  • Command/HTTP hooks follow Claude semantics (exit 0/2, JSON parsing); CONTINUE_PROJECT_DIR/CLAUDE_PROJECT_DIR set; async hooks, regex matching, dedup, parallel execution; defaults: 600s command, 30s HTTP.
  • fireHook helpers for common events; documented in AGENTS.md.

• Bug Fixes

  • Use setEncoding("utf8") for stdout/stderr to avoid multibyte corruption.
  • Fix HTTP header env var interpolation to handle ${VAR} and $VAR forms.
  • Only suppress EPIPE errors on stdin writes; log other stdin errors.
  • Register HookService in the service container and expose via services.
  • React hooks cleanup in useService: memoize reload/deps, remove invalid eslint-disable comment, and skip hookRunner tests on Windows.

^{Written for commit 8782e38. Summary will update on new commits.}

[cubic-dev-ai]

3 issues found across 6 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="extensions/cli/src/hooks/hookRunner.ts">
<violation number="1" location="extensions/cli/src/hooks/hookRunner.ts:89">
P2: Using `data.toString()` per chunk corrupts multi-byte UTF-8 characters split across buffer boundaries. Set encoding on the streams so Node's internal `StringDecoder` handles this correctly.
(Based on your team's feedback about setting encoding to "utf8" on stdout/stderr streams instead of using `toString()` per 'data' event.) [FEEDBACK_USED]</violation>
<violation number="2" location="extensions/cli/src/hooks/hookRunner.ts:98">
P1: Missing error handler on `child.stdin` — if the child process closes its stdin pipe before the write completes, the unhandled `'error'` event on the stdin stream can crash the Node.js process.</violation>
<violation number="3" location="extensions/cli/src/hooks/hookRunner.ts:149">
P2: Regex `\$\{?(\w+)\}?` incorrectly matches unbalanced braces (e.g., `$FOO}` consumes the `}` even without an opening `{`). Use alternation to properly handle both `${VAR}` and `$VAR` forms.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[cubic-dev-ai]

1 issue found across 1 file (changes from recent commits).

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="extensions/cli/src/hooks/hookRunner.ts">
<violation number="1" location="extensions/cli/src/hooks/hookRunner.ts:98">
P2: The new stdin error handler ignores every error, not just EPIPE as intended; this masks unexpected write failures.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[sestinj]

Summary

• Adds a Claude Code-compatible hooks system for the Continue CLI that allows external handlers (shell commands, HTTP endpoints) to intercept and respond to 16 CLI event types
• Supports regex-based event matching, typed inputs/outputs, sync/async execution, and multi-source config merging from .continue/ and .claude/ directories
• Includes comprehensive test coverage

Test plan

• Verify hooks config loading from multiple sources
• Test command hook execution with stdin JSON piping
• Validate exit code semantics (0=proceed, 2=block)
• Confirm async hooks fire without blocking

🤖 Generated with Claude Code

---

Continue Tasks: ✅ 7 no changes — View all

---

Summary by cubic

Adds a Claude Code–compatible hooks system to the Continue CLI so external command and HTTP handlers can intercept and influence 17 CLI events. Registers HookService and tightens execution with UTF-8-safe streams, correct env var interpolation, clean EPIPE handling, and ESLint fixes.

• New Features

  • HookService with fireEvent, reloadConfig, and getCommonFields; exposed via services.hooks.
  • Config merging from ~/.continue, .continue (including settings.local.json), and .claude; supports disableAllHooks.
  • Command/HTTP hooks follow Claude semantics (exit 0/2, JSON parsing); CONTINUE_PROJECT_DIR/CLAUDE_PROJECT_DIR set; async hooks, regex matching, dedup, parallel execution; defaults: 600s command, 30s HTTP.
  • fireHook helpers for common events; documented in AGENTS.md.

• Bug Fixes

  • Use setEncoding("utf8") for stdout/stderr to avoid multibyte corruption.
  • Fix HTTP header env var interpolation to handle ${VAR} and $VAR forms.
  • Only suppress EPIPE errors on stdin writes; log other stdin errors.
  • Register HookService in the service container and expose via services.
  • React hooks cleanup in useService: memoize reload/deps, remove invalid eslint-disable comment, and skip hookRunner tests on Windows.

^{Written for commit 8782e38. Summary will update on new commits.}

[cubic-dev-ai]

3 issues found across 6 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="extensions/cli/src/hooks/hookRunner.ts">
<violation number="1" location="extensions/cli/src/hooks/hookRunner.ts:89">
P2: Using `data.toString()` per chunk corrupts multi-byte UTF-8 characters split across buffer boundaries. Set encoding on the streams so Node's internal `StringDecoder` handles this correctly.
(Based on your team's feedback about setting encoding to "utf8" on stdout/stderr streams instead of using `toString()` per 'data' event.) [FEEDBACK_USED]</violation>
<violation number="2" location="extensions/cli/src/hooks/hookRunner.ts:98">
P1: Missing error handler on `child.stdin` — if the child process closes its stdin pipe before the write completes, the unhandled `'error'` event on the stdin stream can crash the Node.js process.</violation>
<violation number="3" location="extensions/cli/src/hooks/hookRunner.ts:149">
P2: Regex `\$\{?(\w+)\}?` incorrectly matches unbalanced braces (e.g., `$FOO}` consumes the `}` even without an opening `{`). Use alternation to properly handle both `${VAR}` and `$VAR` forms.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[cubic-dev-ai]

1 issue found across 1 file (changes from recent commits).

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="extensions/cli/src/hooks/hookRunner.ts">
<violation number="1" location="extensions/cli/src/hooks/hookRunner.ts:98">
P2: The new stdin error handler ignores every error, not just EPIPE as intended; this masks unexpected write failures.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[sestinj]

Summary

• Adds a Claude Code-compatible hooks system for the Continue CLI that allows external handlers (shell commands, HTTP endpoints) to intercept and respond to 16 CLI event types
• Supports regex-based event matching, typed inputs/outputs, sync/async execution, and multi-source config merging from .continue/ and .claude/ directories
• Includes comprehensive test coverage

Test plan

• Verify hooks config loading from multiple sources
• Test command hook execution with stdin JSON piping
• Validate exit code semantics (0=proceed, 2=block)
• Confirm async hooks fire without blocking

🤖 Generated with Claude Code

---

Continue Tasks: ✅ 7 no changes — View all

---

Summary by cubic

Adds a Claude Code–compatible hooks system to the Continue CLI so external command and HTTP handlers can intercept and influence 17 CLI events. Registers HookService and tightens execution with UTF-8-safe streams, correct env var interpolation, clean EPIPE handling, and ESLint fixes.

• New Features

  • HookService with fireEvent, reloadConfig, and getCommonFields; exposed via services.hooks.
  • Config merging from ~/.continue, .continue (including settings.local.json), and .claude; supports disableAllHooks.
  • Command/HTTP hooks follow Claude semantics (exit 0/2, JSON parsing); CONTINUE_PROJECT_DIR/CLAUDE_PROJECT_DIR set; async hooks, regex matching, dedup, parallel execution; defaults: 600s command, 30s HTTP.
  • fireHook helpers for common events; documented in AGENTS.md.

• Bug Fixes

  • Use setEncoding("utf8") for stdout/stderr to avoid multibyte corruption.
  • Fix HTTP header env var interpolation to handle ${VAR} and $VAR forms.
  • Only suppress EPIPE errors on stdin writes; log other stdin errors.
  • Register HookService in the service container and expose via services.
  • React hooks cleanup in useService: memoize reload/deps, remove invalid eslint-disable comment, and skip hookRunner tests on Windows.

^{Written for commit 8782e38. Summary will update on new commits.}

[cubic-dev-ai]

3 issues found across 6 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="extensions/cli/src/hooks/hookRunner.ts">
<violation number="1" location="extensions/cli/src/hooks/hookRunner.ts:89">
P2: Using `data.toString()` per chunk corrupts multi-byte UTF-8 characters split across buffer boundaries. Set encoding on the streams so Node's internal `StringDecoder` handles this correctly.
(Based on your team's feedback about setting encoding to "utf8" on stdout/stderr streams instead of using `toString()` per 'data' event.) [FEEDBACK_USED]</violation>
<violation number="2" location="extensions/cli/src/hooks/hookRunner.ts:98">
P1: Missing error handler on `child.stdin` — if the child process closes its stdin pipe before the write completes, the unhandled `'error'` event on the stdin stream can crash the Node.js process.</violation>
<violation number="3" location="extensions/cli/src/hooks/hookRunner.ts:149">
P2: Regex `\$\{?(\w+)\}?` incorrectly matches unbalanced braces (e.g., `$FOO}` consumes the `}` even without an opening `{`). Use alternation to properly handle both `${VAR}` and `$VAR` forms.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[cubic-dev-ai]

1 issue found across 1 file (changes from recent commits).

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="extensions/cli/src/hooks/hookRunner.ts">
<violation number="1" location="extensions/cli/src/hooks/hookRunner.ts:98">
P2: The new stdin error handler ignores every error, not just EPIPE as intended; this masks unexpected write failures.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[sestinj]

Summary

• Adds a Claude Code-compatible hooks system for the Continue CLI that allows external handlers (shell commands, HTTP endpoints) to intercept and respond to 16 CLI event types
• Supports regex-based event matching, typed inputs/outputs, sync/async execution, and multi-source config merging from .continue/ and .claude/ directories
• Includes comprehensive test coverage

Test plan

• Verify hooks config loading from multiple sources
• Test command hook execution with stdin JSON piping
• Validate exit code semantics (0=proceed, 2=block)
• Confirm async hooks fire without blocking

🤖 Generated with Claude Code

---

Continue Tasks: ✅ 7 no changes — View all

---

Summary by cubic

Adds a Claude Code–compatible hooks system to the Continue CLI so external command and HTTP handlers can intercept and influence 17 CLI events. Registers HookService and tightens execution with UTF-8-safe streams, correct env var interpolation, clean EPIPE handling, and ESLint fixes.

• New Features

  • HookService with fireEvent, reloadConfig, and getCommonFields; exposed via services.hooks.
  • Config merging from ~/.continue, .continue (including settings.local.json), and .claude; supports disableAllHooks.
  • Command/HTTP hooks follow Claude semantics (exit 0/2, JSON parsing); CONTINUE_PROJECT_DIR/CLAUDE_PROJECT_DIR set; async hooks, regex matching, dedup, parallel execution; defaults: 600s command, 30s HTTP.
  • fireHook helpers for common events; documented in AGENTS.md.

• Bug Fixes

  • Use setEncoding("utf8") for stdout/stderr to avoid multibyte corruption.
  • Fix HTTP header env var interpolation to handle ${VAR} and $VAR forms.
  • Only suppress EPIPE errors on stdin writes; log other stdin errors.
  • Register HookService in the service container and expose via services.
  • React hooks cleanup in useService: memoize reload/deps, remove invalid eslint-disable comment, and skip hookRunner tests on Windows.

^{Written for commit 8782e38. Summary will update on new commits.}

[cubic-dev-ai]

3 issues found across 6 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="extensions/cli/src/hooks/hookRunner.ts">
<violation number="1" location="extensions/cli/src/hooks/hookRunner.ts:89">
P2: Using `data.toString()` per chunk corrupts multi-byte UTF-8 characters split across buffer boundaries. Set encoding on the streams so Node's internal `StringDecoder` handles this correctly.
(Based on your team's feedback about setting encoding to "utf8" on stdout/stderr streams instead of using `toString()` per 'data' event.) [FEEDBACK_USED]</violation>
<violation number="2" location="extensions/cli/src/hooks/hookRunner.ts:98">
P1: Missing error handler on `child.stdin` — if the child process closes its stdin pipe before the write completes, the unhandled `'error'` event on the stdin stream can crash the Node.js process.</violation>
<violation number="3" location="extensions/cli/src/hooks/hookRunner.ts:149">
P2: Regex `\$\{?(\w+)\}?` incorrectly matches unbalanced braces (e.g., `$FOO}` consumes the `}` even without an opening `{`). Use alternation to properly handle both `${VAR}` and `$VAR` forms.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[cubic-dev-ai]

1 issue found across 1 file (changes from recent commits).

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="extensions/cli/src/hooks/hookRunner.ts">
<violation number="1" location="extensions/cli/src/hooks/hookRunner.ts:98">
P2: The new stdin error handler ignores every error, not just EPIPE as intended; this masks unexpected write failures.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}