mirror of
https://github.com/Kaelio/ktx.git
synced 2026-07-19 11:41:02 +02:00
fix(cli): treat omitted sentence-transformers base_url as managed daemon (#194)
After PR #184 and #192 moved managed-embeddings URL resolution to the CLI project boundary and made `ktx setup` persist `ktx.yaml` without a `base_url`, the status command still treated the empty value as misconfiguration and printed "no base_url configured", dragging the verdict down to "Partially ready — embedding credentials missing". Update `buildEmbeddingsStatus` to recognize the managed-daemon convention and report it as ok. Add a `status-project.test.ts` covering the explicit-url, omitted, empty-string, and openai-missing-key paths.
This commit is contained in:
parent
0958bc03dc
commit
9fc715ac6a
2 changed files with 128 additions and 3 deletions
|
|
@ -267,9 +267,8 @@ function buildEmbeddingsStatus(config: KtxProjectEmbeddingConfig, env: NodeJS.Pr
|
|||
backend,
|
||||
model,
|
||||
dimensions,
|
||||
status: 'warn',
|
||||
detail: 'no base_url configured',
|
||||
fix: 'Rerun `ktx setup`',
|
||||
status: 'ok',
|
||||
detail: 'managed local embeddings daemon',
|
||||
};
|
||||
}
|
||||
return { backend, model, dimensions, status: 'warn', detail: 'unknown embedding backend' };
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue