Skip to content

0.7.3 follow-ups: UPGRADING rc-path note + embeddings env-var comment (cloud heads-up) #623

Description

@unforced

Non-blocking nits from #622's review (SHIP, 0.7.3 promoted to @latest):

  1. UPGRADING rc-path sentence: rc.1-15 had semantic search ON by default; a @rc user landing on 0.7.3 finds it off (vectors preserved + reused on re-enable, no re-embed — planStaleness matches model+content_hash). Add one sentence to the 0.7.2→0.7.3 section noting rc users must re-enable via env/config. (Not a @latest-user issue — 0.7.2 never had the feature.)
  2. src/embedding/select.ts comment still says the env var 'mirrors the cloud wrangler var' — semantics changed from off-switch to tri-state (default flips OFF when unset). HEADS-UP for the cloud pin-bump / if cloud's worker consumes this selector: confirm cloud's embeddings default is set explicitly, not left to this new default-off.
  3. (harmless) with embeddings off the 30s sweep timer still ticks (no-op) + the write-hook dispatches (immediate return) — documented at server.ts:224-232; optional micro-opt later.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions