v0.8.53: indexing fixes - #8124
Conversation
waleedlatif1
commented
Sep 22, 2026
- improvement(ui): centralize product colours in global CSS (improvement(ui): centralize product colours in global CSS #8116)
- fix(knowledge): index the connector tombstone check so a sync stops scanning the whole document table (fix(knowledge): index the connector tombstone check so a sync stops scanning the whole document table #8122)
- fix(knowledge): bound source titles and tag values so an oversized index row cannot fail a connector sync (fix(knowledge): bound source titles and tag values so an oversized index row cannot fail a connector sync #8123)
Co-authored-by: Bill Leoutsakos <billleoutsakos@Bills-MacBook-Pro.local>
…canning the whole document table (#8122) * fix(knowledge): index the connector tombstone check so a sync stops scanning the whole document table * chore(db): format the generated migration metadata
…dex row cannot fail a connector sync (#8123) * fix(knowledge): bound source titles and tag values so an oversized index row cannot fail a connector sync * test(knowledge): pin the exact indexed-text boundary * fix(knowledge): share the indexed-text bound, cut by code point, and reject oversized filenames and tags at the document APIs * fix(knowledge): cut bounded text with a shared surrogate-safe helper inside the limit and leave stored upload metadata untouched * fix(knowledge): leave values at the limit untouched and refuse over-long tag values on the tag-data and tag-update paths
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
| /** Runtime collaborator colours must exist even without a matching utility class. */ | ||
| @theme static { | ||
| --color-black: #000000; | ||
| --color-white: #ffffff; | ||
| --color-amber-300: #fcd34d; | ||
| --color-orange-400: #fb923c; | ||
| --color-pink-400: #f472b6; | ||
| --color-purple-400: #c084fc; | ||
| --color-violet-500: #8b5cf6; | ||
| --color-cool-gray-500: #6b7280; | ||
| } |
There was a problem hiding this comment.
Global Styling Violates Requirements
This adds eight color variables in a new global @theme block. The repository requires styling changes to remain local to components and specifically says to avoid editing globals.css unless absolutely necessary. This requirement must be satisfied before merging by keeping these colors in the relevant component styles or retaining local color values.
Rule Used: Avoid editing the globals.css file unless absolutely necessary. Move style changes to local component files instead. (source)
Learned From
simstudioai/sim#367
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!