fix(knowledge): return pending status for new documents - #6597
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryLow Risk Overview
Reviewed by Cursor Bugbot for commit 526ea5e. Bugbot is set up for automated code reviews on this repo. Configure here. |
Greptile SummaryThe PR aligns newly created knowledge-document return types and API responses with the persisted
Confidence Score: 5/5The PR appears safe to merge with the creation result, persisted state, orchestration types, and route response consistently aligned. Newly created documents return the pending state written to storage, while idempotent lookups retain later persisted lifecycle states without changing queue or retry behavior.
|
| Filename | Overview |
|---|---|
| apps/sim/lib/knowledge/documents/service.ts | Returns the newly persisted pending status and correctly widens idempotent lookups to the complete persisted status union. |
| apps/sim/lib/knowledge/orchestration/documents.ts | Aligns the created-document contract with new pending documents and existing documents returned in later lifecycle states. |
| apps/sim/app/api/knowledge/migrated-routes.test.ts | Updates the creation fixture and verifies that the migrated route returns pending status. |
| apps/sim/lib/knowledge/documents/storage-billing.test.ts | Verifies that single-document creation both persists and returns pending status. |
Reviews (1): Last reviewed commit: "fix(knowledge): return pending status fo..." | Re-trigger Greptile
Summary
Type of Change
Testing
Checklist