PR #433 #1435
sentry[bot]codeql
on: dynamic
Matrix: analyze
Annotations
2 warnings
|
No tests for the new buffer accumulation, capping, and truncation logic:
src/utils/command.ts#L229
The new `appendChunk` / `finalizeStream` functions and the `maxOutputBytes` cap are entirely untested — `src/utils/__tests__/command.test.ts` has no test that exercises large output, truncation, the `[output truncated after N bytes]` message, or the `XCODEBUILDMCP_MAX_OUTPUT_BYTES` env-var path. Consider adding unit tests that verify: (1) output below the cap is returned verbatim, (2) output at or above the cap is truncated with the expected message, and (3) the env-var override is respected.
|
|
[UJ6-YUH] No tests for the new buffer accumulation, capping, and truncation logic (additional location):
src/utils/command.ts#L76
The new `appendChunk` / `finalizeStream` functions and the `maxOutputBytes` cap are entirely untested — `src/utils/__tests__/command.test.ts` has no test that exercises large output, truncation, the `[output truncated after N bytes]` message, or the `XCODEBUILDMCP_MAX_OUTPUT_BYTES` env-var path. Consider adding unit tests that verify: (1) output below the cap is returned verbatim, (2) output at or above the cap is truncated with the expected message, and (3) the env-var override is respected.
|