Fix db-table-ai-panel spec for streaming AI chunks#1730
Conversation
Update mockStream to yield AiStreamChunk objects instead of raw strings so _consumeStream consumes them correctly, and align the expected AI message with the component's thinking field after #1729. Also add pnpm overrides for fast-xml-parser and uuid to fix advisories. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 46 minutes and 35 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
Updates the DbTable AI panel spec to match the post-#1729 streaming contract (NDJSON chunk objects, including thinking state), and applies pnpm overrides to remediate dependency advisories.
Changes:
- Update the AI panel spec helper stream to yield
AiStreamChunkobjects (instead of raw strings) so_consumeStreamis exercised correctly. - Align expected AI message shape in the spec to include
thinking: ''. - Add pnpm overrides (and lockfile updates) for
fast-xml-parseranduuidto address advisories.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
frontend/src/app/components/dashboard/db-table-view/db-table-ai-panel/db-table-ai-panel.component.spec.ts |
Fixes the mocked AI stream and expected message shape to reflect chunked streaming + thinking state. |
package.json |
Adds pnpm overrides for fast-xml-parser and uuid advisory remediation. |
pnpm-lock.yaml |
Lockfile updates reflecting the new overrides and resolved versions. |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Update mockStream to yield AiStreamChunk objects instead of raw strings so _consumeStream consumes them correctly, and align the expected AI message with the component's thinking field after #1729.
Also add pnpm overrides for fast-xml-parser and uuid to fix advisories.