Skip to content

fix: MiniLM 384-dim defaults in production fallback paths #27

@sparkling

Description

@sparkling

Bug

Several production paths default to Xenova/all-MiniLM-L6-v2 (384-dim) instead of the canonical Xenova/all-mpnet-base-v2 (768-dim):

  • hooks.ts:1021 — pretrain --embedding-model flag default
  • hooks.ts:1042 — agentdb import catch fallback
  • hooks-tools.ts:2161 — status display (said MiniLM, note said mpnet)
  • embeddings-tools.ts:169 — enum ordering (MiniLM first)

Impact

hooks pretrain stores 384-dim vectors. Subsequent memory search queries with 768-dim vectors. The cosineSim truncation bug (#25) masked this — after that fix, dimension mismatches now throw.

Fix

All defaults changed to Xenova/all-mpnet-base-v2 (768-dim). MiniLM remains available as explicit user choice.

Commit: dddb5f9 (ADR-0075 action 1)

ADR Reference

ADR-0075 Problem 2, ADR-0076 (pre-Track-A)

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions