fix(web): drop debug console.log that logged note content on every keystroke - #1473
Closed
abhay-codes07 wants to merge 1 commit into
Closed
fix(web): drop debug console.log that logged note content on every keystroke#1473abhay-codes07 wants to merge 1 commit into
abhay-codes07 wants to merge 1 commit into
Conversation
…ystroke
FullscreenNoteModal's onChange handler ran `console.log("handleContentChange",
newContent)` on every edit, spamming the production console with the full note
body as the user types. Remove it.
Member
|
Included in #1596 with your commit and authorship intact, plus a Good catch on this one. That line printed the whole note body on every keystroke, and PostHog session replay can pick console output up. Thanks @abhay-codes07. |
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>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
FullscreenNoteModal'shandleContentChangeranconsole.log("handleContentChange", newContent)on every change, so the full note body was written to the production console on every keystroke. Removed the leftover debug line; the surroundingsetContent/setDraftbehavior is unchanged.Minor, but it is noisy and needlessly puts user note content in the console.