Squash of eng-294-create-supabase-insertupdate-route #176
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Contributor
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the 📝 WalkthroughWalkthroughThis update introduces a comprehensive Supabase-backed API layer for a Next.js application, implementing CRUD and batch operations for entities like Person, Account, Platform, Space, Content, Document, and Content Embeddings. It adds utility modules for Supabase integration, type-safe database access, validation, and batch processing. An OpenAI embedding API route is also included. Changes
Sequence Diagram(s)sequenceDiagram
participant Client
participant API_Route
participant Validator
participant SupabaseClient
participant DBUtils
Client->>API_Route: POST /api/supabase/content (JSON body)
API_Route->>Validator: Validate input
Validator-->>API_Route: Valid/Invalid
alt Valid
API_Route->>SupabaseClient: Create instance
API_Route->>DBUtils: getOrCreateEntity(input)
DBUtils-->>SupabaseClient: Upsert/fetch entity
SupabaseClient-->>DBUtils: Entity or error
DBUtils-->>API_Route: Result (entity, error, created)
API_Route-->>Client: JSON response (entity, status, error)
else Invalid
API_Route-->>Client: JSON error response (400)
end
sequenceDiagram
participant Client
participant API_Route
participant Validator
participant SupabaseClient
participant DBUtils
Client->>API_Route: POST /api/supabase/content-embedding/batch (JSON array)
API_Route->>Validator: Validate input array
Validator-->>API_Route: Valid/Invalid
alt Valid
API_Route->>DBUtils: Batch insert/process by model
DBUtils-->>SupabaseClient: Batch upsert per model
SupabaseClient-->>DBUtils: Results or errors
DBUtils-->>API_Route: Aggregated results, partial errors
API_Route-->>Client: JSON response (records, partial errors, status)
else Invalid
API_Route-->>Client: JSON error response (400)
end
Possibly related PRs
Suggested reviewers
Poem
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
maparent
force-pushed
the
eng-294-create-supabase-routes-squashed
branch
from
9cd2b57 to
f5d1cb9
Compare
maparent
force-pushed
the
eng-294-create-supabase-routes-squashed
branch
from
f5d1cb9 to
73538cf
Compare
mdroidian
reviewed
May 23, 2025
mdroidian
reviewed
May 23, 2025
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
mdroidian
requested changes
May 25, 2025
Contributor
mdroidian
left a comment
mdroidian
left a comment
There was a problem hiding this comment.
A quick once over to remove comments, adhere to style guide, etc.
Once these comments are addressed, please re tag me for review.
maparent
force-pushed
the
eng-294-create-supabase-routes-squashed
branch
from
26d1b11 to
7f47560
Compare
maparent
force-pushed
the
eng-294-create-supabase-routes-squashed
branch
from
ec4ae5e to
4916281
Compare
maparent
force-pushed
the
eng-294-create-supabase-routes-squashed
branch
from
2d9a054 to
351bed8
Compare
mdroidian
approved these changes
May 30, 2025
Comment on lines
+11
to
+19
| /** | ||
| * Sends a standardized JSON response. | ||
| * @param request The original NextRequest. | ||
| * @param data The data payload for successful responses. | ||
| * @param error An error message string if the operation failed. | ||
| * @param details Optional detailed error information. | ||
| * @param status The HTTP status code for the response. | ||
| * @param created A boolean indicating if a resource was created (influences status code: 201 vs 200). | ||
| */ |
Contributor
There was a problem hiding this comment.
The problem with JSDoc's is that they need to be kept up to date. If they are created at all, they should be after the review is passed and ready to be merged. As we can see, these are already out of date.
| * @param tableName The name of the table. | ||
| * @param insertData Data to upsert | ||
| * @param uniqueOn The expected uniqueOn key. | ||
| * @returns Promise<GetOrCreateEntityResult<T>> |
| tableName: keyof Database["public"]["Tables"]; | ||
| items: TablesInsert<TableName>[]; | ||
| uniqueOn?: (keyof TablesInsert<TableName>)[]; // Uses pKey otherwise | ||
| inputValidator?: ItemValidator<TablesInsert<TableName>>; |
| * Default GET handler for retrieving a resource by Id | ||
| */ | ||
| export const makeDefaultGetHandler = | ||
| (tableName: keyof Database["public"]["Tables"], pk: string = "id") => |
This was referenced May 30, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Work by Sid: Hyde utility, embedding route, supabase insert routes
Marc-Antoine: introduce generated types, further work on insert routes
Summary by CodeRabbit
New Features
Enhancements
Chores
Documentation