chore(api): drop youtube_tokens from generated types#521
chore(api): drop youtube_tokens from generated types#521arpitgupta1214 wants to merge 1 commit intotestfrom
Conversation
Companion to recoupable/database#22 — once the table is dropped the next `pnpm update-types` yields this same diff. The api revenue MCP tool reads tokens from Composio now, so nothing depends on this type. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
No issues found across 1 file
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
Architecture diagram
sequenceDiagram
participant Dev as Developer
participant DB as Supabase Database
participant Gen as Type Generator
participant Source as types/database.types.ts
participant Tools as MCP Tools
participant Composio as Composio Service
participant App as Application Code
Note over Dev,App: NEW: Token source migration from DB to Composio
Dev->>DB: Apply migration: drop youtube_tokens table
DB-->>Dev: Table removed
Dev->>Gen: Run pnpm update-types
Gen->>DB: Introspect current schema
DB-->>Gen: Schema (no youtube_tokens)
Gen->>Source: Generate types (youtube_tokens omitted)
Source-->>Dev: Zero diff against PR branch
Note over Tools,Composio: Revenue flow now uses Composio
App->>Tools: Call get_youtube_revenue()
Tools->>Composio: Fetch YouTube tokens
Composio-->>Tools: Return tokens
Tools->>composio: Authenticate and fetch revenue data
Composio-->>Tools: Revenue data
Tools-->>App: Return revenue results
Note over Source,Tools: Type safety maintained without youtube_tokens
Auto-approved: Removes an unused generated database type (youtube_tokens) after the corresponding table is dropped. No runtime logic, no refactoring, purely a type definition removal. Low risk of breakage.
Summary
Companion to recoupable/database#22 — once the table is dropped the next
pnpm update-typesyields this same diff. The api revenue MCP tool reads tokens from Composio now, so nothing depends on this type.Test plan
pnpm update-typeslocally and confirm zero diff against this branchget_youtube_revenueMCP tool still works against an authenticated artist🤖 Generated with Claude Code
Summary by cubic
Removed the
youtube_tokenstable fromtypes/database.types.tsto match the DB schema change. Tokens now come fromComposio, so this type is unused.pnpm update-typeslocally; expect no diff.get_youtube_revenueMCP tool still works for an authenticated artist.Written for commit ea39994. Summary will update on new commits.