Skip to content

remove unused onboarding research endpoints - #1563

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

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

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