fix(chat): preserve literal code around source chips - #7456
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
Greptile SummaryThis PR updates chat-content sanitization to preserve literal source and workspace-resource tags inside multi-backtick spans and fenced blocks while continuing to unwrap ordinary inline citations.
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains.
|
| Filename | Overview |
|---|---|
| apps/sim/app/workspace/[workspaceId]/home/components/message-content/components/chat-content/chat-sanitize.ts | Reworks chip-aware inline-code and fenced-block handling and bounds malformed JSON matching; the previously reported cases are addressed without an eligible residual defect. |
| apps/sim/app/workspace/[workspaceId]/home/components/message-content/components/chat-content/chat-content.test.ts | Adds comprehensive regressions for delimiter runs, paragraph boundaries, fences, quoted tag text, malformed input, and scaling. |
| apps/sim/app/workspace/[workspaceId]/home/components/message-content/components/scaling-test-helpers.ts | Generalizes the scaling helper with a configurable input builder while preserving its existing default behavior. |
Reviews (4): Last reviewed commit: "fix(chat): preserve tilde-fenced citatio..." | Re-trigger Greptile
There was a problem hiding this comment.
All reported issues were addressed across 3 files
Reply with feedback, questions, or to request a fix.
Fix all with cubic | Re-trigger cubic
c327392 to
2d5b329
Compare
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
There was a problem hiding this comment.
All reported issues were addressed across 3 files
Reply with feedback, questions, or to request a fix.
Fix all with cubic | Re-trigger cubic
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
Summary
Type of Change
Testing
Checklist