fix(dashboards): menu order, org-chat entitlement, mention ids, contract bounds - #8495
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
|
@cubic-dev-ai review this PR |
@TheodoreSpeaks I have started the AI code review. It will take a few minutes to complete. |
a5edce6 to
e3af83d
Compare
|
@cubic-dev-ai review this PR |
@TheodoreSpeaks I have started the AI code review. It will take a few minutes to complete. |
…ion-resolve # Conflicts: # apps/sim/app/workspace/[workspaceId]/home/hooks/use-chat.ts # apps/sim/components/charts/time-series-chart.tsx # apps/sim/lib/dashboards/repository.integration.ts # apps/sim/lib/mothership/chat/display-message.ts
Summary
@mention,+attach, resource-tab+), instead of first.claude/rules/sim-list-ordering.md, its Cursor copy,CLAUDE.md);RESOURCE_MENU_ORDERis now its own product orderuse-chat.tsand the reload copy indisplay-message.tsboth droppeddashboardId. Persist, read and display now share onecopyPersistedMessageContext, so a new id field gets added in one place.role='group'on the readout wrapperdashboardContentSchema/dashboardRevisionSchema(the write path already enforces both)Type of Change
Testing
payload.test.ts); reopened dashboard mention keepsdashboardId(display-message.test.ts)bun run teston payload, display-message, persisted-message, messages-store, use-chat andlib/dashboards(138 pass)repository.integration.tsagainst local Postgres, in full and the revision test alonebun run type-check,bun run lint,bun run check:audits(52 audits pass)Checklist
test-auditauthoring gate)🤖 Generated with Claude Code
https://claude.ai/code/session_01HqEUjQv32TzpsJASPmbdnM