Skip to content

chore(chat): promote test → main (subscription migration cleanup)#1706

Merged
sweetmantech merged 1 commit into
mainfrom
test
Apr 24, 2026
Merged

chore(chat): promote test → main (subscription migration cleanup)#1706
sweetmantech merged 1 commit into
mainfrom
test

Conversation

@sweetmantech

@sweetmantech sweetmantech commented Apr 24, 2026

Copy link
Copy Markdown
Collaborator

Promotes the single change on `test` to `main`.

Includes

Test plan

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Breaking Changes
    • Removed subscription-fetching functionality. Applications relying on this feature will require updates to use alternative methods.

Summary by cubic

Removed the unused lib/stripe/getActiveSubscription.tsx helper after migrating GET /api/subscriptions. Cleanup only with no functional changes or user impact.

Written for commit 22f8da2. Summary will update on new commits.

The lib/stripe/getActiveSubscription.tsx helper is unreachable —
grep across chat returns only its own definition. Remove it as part
of the Agent API migration row 25.
@vercel

vercel Bot commented Apr 24, 2026

Copy link
Copy Markdown
Contributor

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

Project Deployment Actions Updated (UTC)
recoup-chat Building Building Preview Apr 24, 2026 5:52pm

Request Review

@coderabbitai

coderabbitai Bot commented Apr 24, 2026

Copy link
Copy Markdown
Contributor

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 92dea7b4-1823-4e9e-92be-b2c1c52b6075

📥 Commits

Reviewing files that changed from the base of the PR and between dabe8af and 22f8da2.

📒 Files selected for processing (1)
  • lib/stripe/getActiveSubscription.tsx

📝 Walkthrough

Walkthrough

The getActiveSubscription function has been removed from the Stripe utilities module. This async function previously handled HTTP requests to the Recoupable subscriptions endpoint, including error handling and JSON parsing. The module now contains no subscription-fetching logic.

Changes

Cohort / File(s) Summary
Subscription Function Removal
lib/stripe/getActiveSubscription.tsx
Deleted the getActiveSubscription(accountId: string): Promise<any> exported async function that performed GET requests to the Recoupable subscriptions endpoint with error handling and JSON response parsing.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

📦 A subscription fetcher once lived here with pride,
Twenty-three lines that handled the tide,
Now the Recoupable endpoint sleeps undisturbed,
✨ Cleaner code, lighter burden, lessons learned!

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch test

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.

@sweetmantech sweetmantech merged commit 8f3ede9 into main Apr 24, 2026
1 of 3 checks passed

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

Copy link
Copy Markdown

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.

Requires human review: Deletion of subscription-related logic, even if labeled as dead code, involves a sensitive functional area (payments/subscriptions) that warrants human verification.

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.

2 participants