Source: OCR_ANALYSIS_REPORT.md, OpenCode Review report generated on 2026-08-12.
Findings: M-8, L-3. desktop/src/collaboration/context-pack-records.cjs uses broad includes() checks for forbidden field names, which can reject legitimate fields such as token counts or command history. The same area also reports byte/hash mismatch errors against the wrong field path, which makes debugging harder.
Expected fix:
- Replace broad substring matching with exact or boundary-aware forbidden field checks.
- Report
wirePayloadBytes mismatches on the byte field path rather than wirePayloadHash.
- Add tests for legitimate token-count style fields and malformed wire payload diagnostics.
Source: OCR_ANALYSIS_REPORT.md, OpenCode Review report generated on 2026-08-12.
Findings: M-8, L-3.
desktop/src/collaboration/context-pack-records.cjsuses broadincludes()checks for forbidden field names, which can reject legitimate fields such as token counts or command history. The same area also reports byte/hash mismatch errors against the wrong field path, which makes debugging harder.Expected fix:
wirePayloadBytesmismatches on the byte field path rather thanwirePayloadHash.