lsm / neokai / 28325386179

71%

push

github

feat(agent): wire output limiter hook into query options (#2202)

* feat(agent): wire output limiter hook into query options

- Load outputLimiter config from global settings in QueryOptionsBuilder.
- Install createOutputLimiterHook in the PreToolUse chain when enabled.
- Keep the hook off when global settings omit outputLimiter or set enabled=false.
- Deep-merge partial output-limiter settings with defaults so a user can
  override only bash.headLines (for example) without losing tailLines.

Tests:
- Verify the hook is present in built SDK options when enabled.
- Verify limits are applied to Bash, Read, Grep, and Glob inputs.
- Verify custom limits from global settings are respected.
- Verify partial config deep-merges with defaults.

* fix(agent): address output-limiter review findings

- Bash: preserve original exit status with exit_code=$?; exit $exit_code.
- Bash: capture both stdout and stderr into temp file (> file 2>&1 order).
- Bash: insert newlines around subshell contents so heredoc delimiters parse.
- Bash: skip wrapping sandbox-excluded command prefixes (e.g. git) so
  command-level sandbox exclusions still see the original command.
- Hook: return only updatedInput without a permissionDecision so output
  limiting cannot bypass restrictive permission modes.
- Builder: run declarative tool guards BEFORE the output limiter so
  guard regexes match the original command shape.
- Builder: resolve outputLimiter config (deep-merge) before the enabled
  gate so a partial settings update does not silently disable the hook.
- Remove Glob limiting (GlobInput has no head_limit field; SDK ignores it).
- Rename read.maxChars to read.maxLines since the SDK Read limit is lines.
- Tests: cover exit-status preservation, stderr capture, heredoc handling,
  excluded command prefixes, partial-settings enablement, guard ordering,
  and that the hook does not emit a permission decision.

* fix(agent): update circuit-breaker hint to outputLimiter.read.maxLines

... (continued)

11422 of 16468 branches covered (69.36%)

Branch coverage included in aggregate %.

112 of 114 new or added lines in 3 files covered. (98.25%)

97567 of 115752 relevant lines covered (84.29%)

301.13 hits per line