Skip to content

fix: harden legacy note JSON in search and editor#5114

Draft
ComputelessComputer wants to merge 1 commit intomainfrom
cursor/fix-search-node-type-crash-beae
Draft

fix: harden legacy note JSON in search and editor#5114
ComputelessComputer wants to merge 1 commit intomainfrom
cursor/fix-search-node-type-crash-beae

Conversation

@ComputelessComputer
Copy link
Copy Markdown
Collaborator

Summary

  • sanitize malformed nested TipTap JSON before desktop note editors consume stored session content
  • harden task traversal/extraction so legacy notes with missing child nodes no longer crash on node.type
  • harden session search text extraction and add regressions covering malformed historical note JSON

Testing

  • pnpm -F @hypr/editor test
  • pnpm -F @hypr/editor typecheck
  • pnpm -F @hypr/desktop typecheck
  • pnpm -F @hypr/desktop test -- src/search/contexts/engine/utils.test.ts

Slack Thread

Open in Web Open in Cursor 

Sanitize malformed nested TipTap JSON before session search and note editors consume it. Add focused regressions for task extraction and search indexing so legacy notes no longer crash on missing node types.

Co-authored-by: John Jeong <ComputelessComputer@users.noreply.github.com>
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 21, 2026

Deploy Preview for unsigned-char ready!

Name Link
🔨 Latest commit 74fbcbd
🔍 Latest deploy log https://app.netlify.com/projects/unsigned-char/deploys/69e743eced23bc0008359a71
😎 Deploy Preview https://deploy-preview-5114--unsigned-char.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 21, 2026

Deploy Preview for hyprnote canceled.

Name Link
🔨 Latest commit 74fbcbd
🔍 Latest deploy log https://app.netlify.com/projects/hyprnote/deploys/69e743ec2814170008b1f92d

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