Skip to content

keep snaps and reflections out of the search index - #30

Open
vyagh wants to merge 1 commit into
masterfrom
journal-reflection-index
Open

keep snaps and reflections out of the search index#30
vyagh wants to merge 1 commit into
masterfrom
journal-reflection-index

Conversation

@vyagh

@vyagh vyagh commented Aug 16, 2026

Copy link
Copy Markdown
Member

sugar journal is getting a reflection feature (sugar PRs to follow).
It stores 3 new metadata keys on an entry: moment-snap-N (base64 jpeg snapshots), reflections (the child's talk with an assistant) and next_steps (one line from that talk).

These don't belong in the search index. the snapshots are about 1.5MB of base64 retokenized on every save, and indexing the talk made an entry match any word the assistant ever said.

Entries without these keys are unaffected. child's own title and description stay searchable.

Sugar's journal is getting a reflection feature (sugar PRs to
follow). It stores three new metadata keys on an entry:
moment-snap-N (base64 jpeg snapshots), reflections (the child's
talk with an assistant) and next_steps (one line from that talk).

None of these belong in the search index. The snapshots are about
1.5MB of base64 re-tokenized on every save, and indexing the talk
made an entry match any word the assistant ever said.

Entries without these keys are unaffected, so this lands safely on
its own. The child's own title and description stay searchable.
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