chore: remove two dead onboarding routes and a note-content console.log - #1596
Conversation
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
supermemory-app | 3f7b966 | Commit Preview URL Branch Preview URL |
Aug 25 2026, 10:22 AM |
How to use the Graphite Merge QueueAdd the label Main to this PR to add it to the merge queue. You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
MaheshtheDev
changed the title
|
Claude finished @MaheshtheDev's task in 1m 15s —— View job PR Review Complete
Overview: Removes two unused onboarding API routes ( Verification performed:
Issues found: None — this looks good to ship. The deleted routes were truly dead code with no callers, and the console.log was a debug statement that could leak note content to session replay tools. Clean deletion PR. Score: 10/10 |
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
supermemory-mcp | 3f7b966 | Aug 25 2026, 10:21 AM |
Merge activity
|
…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>
f97f5fe to
3f7b966
Compare
3 participants
Cherry-picks two cleanup PRs and finishes the job. Net 262 deletions.
console.login the fullscreen note editor that printed the whole note body on every keystroke, which PostHog session replay can capture./api/onboarding/researchand/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 fix(web): authenticate and bound metered /api routes #1589, which only existed to make unreachable code safe.EXA_API_KEY,XAI_API_KEYand the@ai-sdk/xaidependency 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