Skip to content

chore(api): drop youtube_tokens from generated types#521

Open
arpitgupta1214 wants to merge 1 commit intotestfrom
chore/drop-youtube-tokens-types
Open

chore(api): drop youtube_tokens from generated types#521
arpitgupta1214 wants to merge 1 commit intotestfrom
chore/drop-youtube-tokens-types

Conversation

@arpitgupta1214
Copy link
Copy Markdown
Collaborator

@arpitgupta1214 arpitgupta1214 commented May 7, 2026

Summary

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.

Test plan

🤖 Generated with Claude Code


Summary by cubic

Removed the youtube_tokens table from types/database.types.ts to match the DB schema change. Tokens now come from Composio, so this type is unused.

Written for commit ea39994. Summary will update on new commits.

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>
@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 7, 2026

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

Project Deployment Actions Updated (UTC)
api Ready Ready Preview May 7, 2026 6:06am

Request Review

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 7, 2026

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • types/database.types.ts is excluded by none and included by none

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 2bb4f820-02bd-46ce-ab60-08ac8b7f14ac

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/drop-youtube-tokens-types

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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
Loading

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant