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 SummaryThe PR fixes chat-content sanitization so literal code around source and workspace-resource chips is preserved while valid citations continue rendering.
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains; the current implementation and regressions address the previously reported delimiter-pairing and quoted-tag issues.
|
| Filename | Overview |
|---|---|
| apps/sim/app/workspace/[workspaceId]/home/components/message-content/components/chat-content/chat-sanitize.ts | Implements paragraph-scoped delimiter pairing, fenced-code preservation, and bounded JSON-chip recognition; the previously reported failures are addressed. |
| apps/sim/app/workspace/[workspaceId]/home/components/message-content/components/chat-content/chat-content.test.ts | Adds focused regressions for all three prior review threads plus malformed-input scaling cases. |
| apps/sim/app/workspace/[workspaceId]/home/components/message-content/components/scaling-test-helpers.ts | Generalizes the scaling helper with injectable input builders while retaining its existing default behavior. |
Reviews (3): Last reviewed commit: "fix(chat): scope delimiter pairing to pa..." | 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. |
Summary
Type of Change
Testing
Checklist