Clarification templates for the router's confidence-gate fallback #136
Reference in New Issue
Block a user
Delete Branch "task/457-clarification-templates-for-the-router-s"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Closes Vikunja #457.
Opened during an unattended overnight run: the diff-budget block was off (
task overnight). Read the diff, not only the tests.Acceptance criteria and quality gate are in
TASK.mdon this branch.Review the review, not the diff — leave comments and the agent will apply them via
task start 457.@@ -0,0 +24,4 @@//// Persona holds: infinitive and imperative questions, so there is no gender// agreement to get wrong, "ты" throughout, and no pet names.var clarifyQuestionVariants = map[dialogue.Slot][]string{json approach from the earlier PRs.
@@ -0,0 +78,4 @@// the same sentence every time is what makes a house assistant sound like a// kiosk. All of them say the same two things — she did not catch it, and he// should say it again — because the wording may vary and the meaning may not.var clarifyMissedVariants = []string{same json approach.
Reviewed as part of a bottom-up pass over the whole open stack (#119 to #168): commits read against the base branch,
make testgreen at the top of the stack. Nothing to raise on this one. Merging.Four findings landed on the PRs they belong to, none of them blocking: #167 (a duplicate
rest-of-day-querygrammar from the merge), #164 (four files the Russian sweep did not reach), #145 (sh -chides an irreversible verb from the tier derivation), #128 (locationCandidatesdrops short city names).