Skip to content

remove unused onboarding research endpoints - #1563

Closed
ishaanxgupta wants to merge 2 commits into
mainfrom
codex/remove-unused-onboarding-research-routes
Closed

remove unused onboarding research endpoints#1563
ishaanxgupta wants to merge 2 commits into
mainfrom
codex/remove-unused-onboarding-research-routes

Conversation

@ishaanxgupta

@ishaanxgupta ishaanxgupta commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

What changed

  • removed the unused /api/onboarding/research route
  • removed the unused /api/onboarding/extract-content route

@ishaanxgupta ishaanxgupta changed the title [codex] remove unused onboarding research endpoints remove unused onboarding research endpoints Aug 19, 2026
@ishaanxgupta
ishaanxgupta marked this pull request as ready for review August 19, 2026 14:03
@claude

claude Bot commented Aug 19, 2026

Copy link
Copy Markdown

Claude finished @ishaanxgupta's task in 1m 28s —— View job


Review Complete

  • Analyze the PR diff
  • Verify routes are actually unused (search for references)
  • Check for any breaking changes
  • Submit review

Summary: This PR safely removes two unused onboarding API routes. No references to these endpoints exist in the codebase, so this is a clean deletion with no risk of breaking changes.

Note: The PR description mentions removing the @ai-sdk/xai dependency and env examples, but those changes weren't included in this PR. The dependency and env vars are now orphaned and could be removed in a follow-up.

Score: 10/10 — Clean code removal, ready to ship.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Aug 19, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
supermemory-app 5987be9 Commit Preview URL

Branch Preview URL
Aug 19 2026, 02:12 PM

@claude claude 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.

Overview: Removes two unused onboarding API routes (/api/onboarding/research and /api/onboarding/extract-content) that were consuming Exa and XAI APIs.

Issues found: None — the deleted routes have no references in the codebase, making this a safe cleanup.

Note: The PR description mentions removing the @ai-sdk/xai dependency and EXA_API_KEY/XAI_API_KEY env examples, but these weren't included in this diff:

  • @ai-sdk/xai is still in apps/web/package.json (line 26)
  • EXA_API_KEY and XAI_API_KEY are still in apps/web/.env.example (lines 3-4)

Since the deleted research/route.ts was the only consumer of @ai-sdk/xai, consider removing the dependency and env examples in a follow-up to complete the cleanup.

Score: 10/10

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
supermemory-mcp 5987be9 Aug 19 2026, 02:11 PM

@MaheshtheDev

Copy link
Copy Markdown
Member

Included in #1596 with your commits and authorship intact, plus a Co-Authored-By trailer in the description so the credit survives the squash merge.

You were right that these were unreachable. We had just added session checks and input caps to both routes in #1589, so this removes code we had only guarded because it was exposed. We also dropped EXA_API_KEY, XAI_API_KEY and the @ai-sdk/xai dependency, since nothing reads them once the routes are gone.

Thanks @ishaanxgupta.

graphite-app Bot pushed a commit that referenced this pull request Aug 25, 2026
…og (#1596)

Cherry-picks two cleanup PRs and finishes the job. Net 262 deletions.

- #1473 (@abhay-codes07): drops a `console.log` in the fullscreen note editor that printed the whole note body on every keystroke, which PostHog session replay can capture.
- #1563 (@ishaanxgupta): removes `/api/onboarding/research` and `/api/onboarding/extract-content`. Neither has a caller anywhere in the repo, and both spent metered Exa and xAI quota. This reverts the guards added for them in #1589, which only existed to make unreachable code safe.
- On top: `EXA_API_KEY`, `XAI_API_KEY` and the `@ai-sdk/xai` dependency are removed, since deleting those routes left them with no consumer.

Co-Authored-By: abhay-codes07 <182421137+abhay-codes07@users.noreply.github.com>
Co-Authored-By: ishaanxgupta <124028055+ishaanxgupta@users.noreply.github.com>
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