Summary

• Sanitize null bytes from bash tool command arguments before execution to prevent shell process crashes

Problem

When context compaction or model fallback occurs, the LLM can hallucinate null bytes (\x00) in tool call arguments. Node.js child_process rejects these with: The shell argument must be a string without null bytes. Received "\0". This crashes the bash tool and leaves the session in an error state.

This affects long-running sessions with context management features enabled (aggressive_truncation, preemptive_compaction, dynamic_context_pruning) and model fallback chains.

Fix

Added null byte sanitization at the very beginning of the tool.execute.before handler chain in tool-execute-before.ts. When the tool is bash and the command string contains null bytes, they are stripped before any other hooks process the command. A log entry is emitted for debugging.

This runs before all other hooks to ensure clean input for downstream processing.

Changes

Fixes #2220

Summary by cubic

Removes null bytes from bash tool commands before execution to stop Node child_process from rejecting the shell args and crashing. This prevents session failures when models emit \x00 during context compaction or fallback. Fixes #2220.

• Bug Fixes
  • Sanitizes bash command strings in the earliest tool.execute.before handler by stripping \x00, and logs the event for debugging.

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