Skip to content

fix: strip profile embedding vectors from public responses - #343

Merged
EyJunge1 merged 1 commit into
tickernelz:mainfrom
EyJunge1:fix/strip-profile-embeddings-clean
Oct 1, 2026
Merged

EyJunge1 merged 1 commit into
tickernelz:mainfrom
EyJunge1:fix/strip-profile-embeddings-clean

Conversation

@EyJunge1

@EyJunge1 EyJunge1 commented Oct 1, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Strip per-item centroid/anchor embeddings from public profile payloads (memory tool profile mode, GET /api/user-profile, profile snapshots, and AI-cleanup preview responses).
  • Keep embeddings intact in storage and in pendingCleanups for apply/merge via toPublicProfileData (clone + strip).
  • Replaces fix: strip profile embedding vectors from memory tool output and profile API #304 with a clean branch on current main (no unrelated v2/index.js commits). Credit to @share121 for the original approach.

Test plan

  • bun test tests/profile-utils.test.ts
  • bun test tests/api-handlers-profile-strip.test.ts
  • Confirm memory tool mode: "profile" output has no centroid/anchor
  • Confirm GET /api/user-profile payload stays small and UI still renders preferences/patterns/workflows

Keep centroid/anchor for internal similarity and pending cleanup apply,
but omit them from memory-tool profile reads and profile API payloads
so responses stay small enough for the model and UI.

Based on work by share121 in tickernelz#304.
@EyJunge1
EyJunge1 merged commit c7d4985 into tickernelz:main Oct 1, 2026
6 checks passed
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