Repository navigation
Conversation
Signed-off-by: dada-yan <BinjunYann@gmail.com>
WaylandYang
left a comment
There was a problem hiding this comment.
Thank you, this records what we agreed on #470 and the chain is clear. embedding never falling back to chat, and a set-but-not-ready link ending the search, are both right: data should never reach an endpoint nobody chose for that task.
Answers to the open questions:
- Profiles stay per workspace. Keys and admins are per workspace today, and a deployment-level profile would let one workspace's data go to another's endpoint.
implication::read_phrasesgoes underalignment.- A base's unset tasks follow the task defaults, as #470 writes it.
Please fold those into the decisions. Two small additions while you're there: in decision 5, say that chat_reasoning_effort moves onto the migrated chat profile and what happens to embed_dim; in decision 4, note the one exception, the connection test, which builds a client from the profile being edited. Then the status can be Accepted and I'll merge it.
|
Still waiting on the changes from the 2026-10-19 review: fold the three open-question answers into the decisions, add |
|
Friendly ping on the 2026-10-19 review: the record still has the three open questions that were answered (per-workspace profiles, |
The record #470 asked for before any code: the six tasks and the fallback chain.
Why
Cut 1 of #470 was accepted on 2026-10-04 with "A short ADR first (the next free number is 0067) with the task list and the fallback chain, then the code." 0067 was held for #470 on #1130. #1213, which also wrote 0067, is moving to another number.
What changes
docs/decisions/0067-each-task-chooses-its-model.mdcovers:model_for(kb, task)follows, andmodel_foras the only way to a model;No code, schema or behaviour changes. The two code PRs of cut 1 follow:
model_for, still reading today's row;Open questions in the record
implication::read_phrases. Does it go underalignment?chatoverride cover them?