The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
docs	Ready	Preview, Comment	Mar 20, 2026 11:34pm

PR Summary

Medium Risk
Adds DB-level unique indexes and new conflict-handling paths for knowledge bases, tables, and workspace files; incorrect dedupe/migration behavior or race-condition handling could block restores/uploads or rename entities unexpectedly.

Overview
Enforces workspace-scoped uniqueness for active names across knowledge bases and workspace files (and surfaces similar conflicts for tables) by adding a migration that de-dupes existing rows and creates partial unique indexes.

Introduces standardized conflict detection (getPostgresErrorCode + new *ConflictError types) and updates create/rename/restore flows to return HTTP 409 on name collisions, including restore-time auto-renaming via generateRestoreName with retries for concurrent unique-violation races.

Improves client feedback by disabling React Query mutation retries, adding toast errors on mutation failures (KB update, table rename, file rename), and refactoring the notification countdown ring into a shared CountdownRing component; toast auto-dismiss now pauses on hover.

^{Written by Cursor Bugbot for commit b564d0c. This will update automatically on new commits. Configure here.}

Greptile Summary

This PR enforces name uniqueness for knowledge bases, tables, and workspace files — both preventing duplicate creation/renaming and gracefully resolving conflicts when restoring archived entities by auto-generating a unique _restored suffix. It also extracts the countdown-ring SVG into a shared EMCN component and adds pause-on-hover support to the toast system.

Key changes:

• KnowledgeBaseConflictError and TableConflictError typed error classes with 409 API responses
• Application-level SELECT guards in createKnowledgeBase / updateKnowledgeBase for duplicate name detection (no DB constraint backs these)
• renameTable relies on catching the DB unique-constraint violation (23505) — more race-condition-safe but uses a fragile Drizzle error-cause shape
• generateRestoreName utility: tries original → _restored → _restored_{hex} without verifying uniqueness of the final hash-based candidate
• CountdownRing extracted into apps/sim/components/emcn/components/toast/countdown-ring.tsx but not exported from the emcn barrel, causing both toast.tsx and notifications.tsx to use subpath imports in violation of the EMCN component rule
• retry: false on mutations prevents retrying idempotency-violating 409 errors
• onError toast handlers added to useUpdateKnowledgeBase, useRenameTable, and useRenameWorkspaceFile

Confidence Score: 3/5

• Mostly safe to merge with minor TOCTOU and import-convention issues that should be addressed
• The core conflict-detection and restore-name logic is sound, and the end-to-end plumbing (service → API → mutation → toast) is correctly wired. However, the knowledge base uniqueness check has a TOCTOU race condition with no DB-level constraint to back it up, the CountdownRing component violates the EMCN barrel-export rule, and the Drizzle error-cause shape used to detect 23505 in renameTable is fragile across dependency versions.
• Pay attention to apps/sim/lib/knowledge/service.ts (TOCTOU race in duplicate name checks) and apps/sim/components/emcn/components/toast/countdown-ring.tsx (missing barrel export causing subpath import violations in two files)

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[User renames / creates entity] --> B{Entity type?}

    B -->|Knowledge Base| C[SELECT duplicate check\nby workspaceId + name]
    C -->|Duplicate found| D[throw KnowledgeBaseConflictError]
    C -->|No duplicate| E[INSERT / UPDATE KB]
    D --> F[API returns 409]
    F --> G[useUpdateKnowledgeBase\nonError → toast.error]

    B -->|Table| H[UPDATE userTableDefinitions\ncatch DB error 23505]
    H -->|Caught 23505| I[throw TableConflictError]
    H -->|Success| J[Return updated table]
    I --> K[API returns 409]
    K --> L[useRenameTable\nonError → toast.error]

    B -->|Workspace File| M[PATCH /files/:id\ncatch non-ok response]
    M -->|Error| N[useRenameWorkspaceFile\nonError → toast.error]

    subgraph Restore Flow
        O[Restore archived entity] --> P[generateRestoreName]
        P --> Q{originalName free?}
        Q -->|Yes| R[Use original name]
        Q -->|No| S{name_restored free?}
        S -->|Yes| T[Use name_restored]
        S -->|No| U[Use name_restored_hex\n⚠ not verified unique]
        R & T & U --> V[UPDATE set deletedAt=null\n+ new name]
    end

_{Last reviewed commit: "Fix lint"}