Skip to content

Chore/update deps and fix e2e auth#327

Merged
rossmanko merged 2 commits intomainfrom
chore/update-deps-and-fix-e2e-auth
Mar 30, 2026
Merged

Chore/update deps and fix e2e auth#327
rossmanko merged 2 commits intomainfrom
chore/update-deps-and-fix-e2e-auth

Conversation

@rossmanko
Copy link
Copy Markdown
Contributor

@rossmanko rossmanko commented Mar 30, 2026 •
edited by coderabbitai bot
Loading

Summary by CodeRabbit

  • Chores
    • Updated package manager to [email protected]
    • Bumped multiple dependencies including AI SDKs, AWS SDK, Convex, LangChain, OpenAI, PostHog, Stripe, and TypeScript (v5 to v6)
    • Updated Stripe API integration to latest version
    • Improved test infrastructure for faster page load synchronization

rossmanko and others added 2 commits March 30, 2026 09:50
@rossmanko @claude
chore: update pnpm packages
21adf7e 
Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
@rossmanko @claude
fix: use domcontentloaded instead of networkidle in e2e login setup
0f6f8d9 
Cloudflare challenge scripts on the WorkOS login page keep persistent
network connections open, preventing networkidle from ever resolving
and causing a 30s timeout on every login attempt.

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
@vercel
Copy link
Copy Markdown

vercel bot commented Mar 30, 2026 •
edited
Loading

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

Project Deployment Actions Updated (UTC)
hackerai Ready Ready Preview, Comment Mar 30, 2026 1:55pm

Request Review

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 30, 2026 •
edited
Loading

📝 Walkthrough

Walkthrough

Updates Stripe API version from 2026-02-25.clover to 2026-03-25.dahlia across multiple files, bumps package dependencies including TypeScript to major version 6, updates pnpm version, and adjusts e2e test fixture page load synchronization from networkidle to domcontentloaded.

Changes

Cohort / File(s) Summary
Stripe API Version Migration
app/api/stripe.ts, convex/extraUsageActions.ts, app/api/subscription/webhook/route.ts		 Updated Stripe API version string from 2026-02-25.clover to 2026-03-25.dahlia in client initialization and corresponding documentation comment.
Dependency Updates
package.json		 Updated pnpm from 10.32.1 to 10.33.0 and bumped versions for multiple dependencies: TypeScript (^5 → ^6), @ai-sdk/, @aws-sdk/, @e2b/code-interpreter, @langchain/community, @posthog/ai, @workos-inc/, ai, convex, e2b, langchain, lucide-react, openai, posthog-, and stripe.
Test Fixture Changes
e2e/fixtures/auth.ts		 Updated performLogin to wait for domcontentloaded instead of networkidle after navigating to /login for improved page load synchronization in authentication tests.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

  • PR #204: Both modify Stripe client initialization across app/api/stripe.ts and convex/extraUsageActions.ts to update the Stripe API version.
  • PR #270: Updates the same webhook file (app/api/subscription/webhook/route.ts) with subscription-related modifications.
  • PR #84: Updates Stripe API version in app/api/stripe.ts with the same configuration change.

Poem

🐰 A hop through versions, new and bright—
TypeScript six and Stripe's delight!
Dependencies refreshed, tests refined,
Our migrations leave old clover behind.
From dahlia's dawn, our code now glows! 🌼

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 33.33% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title references two main changes in the PR: dependency updates and an e2e auth fix. Both are prominently featured in the changeset (package.json updates and e2e/fixtures/auth.ts modification).

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ 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 chore/update-deps-and-fix-e2e-auth

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

coderabbitai[bot]
coderabbitai bot reviewed Mar 30, 2026
View reviewed changes
Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents 
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@package.json`:
- Line 161: The package.json bump to "typescript": "^6" causes peer dependency
conflicts; revert the "typescript" dependency to a compatible range (e.g., "^5")
or upgrade the dependent tooling that enforces upper bounds—specifically update
`@typescript-eslint/eslint-plugin`, `@typescript-eslint/parser`, typescript-eslint
and any related `@typescript-eslint` packages and rollup/rolldown to versions that
explicitly support TypeScript 6; after changing either the "typescript" entry or
the listed packages, run the package manager install and CI/typecheck to verify
no peer dependency or build errors remain.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 2da4e0b6-5d1a-4820-ba41-d01387fddd40

📥 Commits

Reviewing files that changed from the base of the PR and between b80d69c and 0f6f8d9.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (5)
  • app/api/stripe.ts
  • app/api/subscription/webhook/route.ts
  • convex/extraUsageActions.ts
  • e2e/fixtures/auth.ts
  • package.json

@rossmanko rossmanko merged commit 931cde3 into main Mar 30, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@rossmanko