fix(maestro-case-tests): steer loan phase-0 prompt to trigger uipath-maestro-case#2095
fix(maestro-case-tests): steer loan phase-0 prompt to trigger uipath-maestro-case#2095abhiram-vad wants to merge 1 commit into
Conversation
…maestro-case 'I want a draft design document' routed the run to uipath-planner Phase D, which one-shots '<name>-sdd.md' with no interview — every criterion and the LLM judge look for sdd.draft.md/sdd.md, so the task scored 0.118 on 2026-07-16 despite a complete design. Rephrase the closing ask onto maestro-case's own trigger clauses (interview when sdd.md absent, caseplan built later) and drop the planner-bait 'design document' wording. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Claude finished @abhiram-vad's task in 2m 3s —— View job Coder-eval task lint (advisory)1 task YAML changed; verdicts: 0 Critical, 0 High, 1 Medium, 0 Low, 0 OK. Rubric: .claude/commands/lint-task.md. This check is advisory and never blocks merge. Evidence of passing run❌ High — PR body says a coder-eval CI run was dispatched but does not claim it has passed. Please update the PR description once the run completes, e.g.: Per-task lint
|
|
Claude finished @abhiram-vad's task in 1m 41s —— View job Reviewing PR #2095
SummaryPrompt-only fix to the Change-by-Change Review1.
|
| Area | Status | Notes |
|---|---|---|
| Frontmatter | N/A | No frontmatter changes |
| E2E Tests | OK | Existing task modified, tags and criteria unchanged |
| Skill Body | N/A | No skill changes |
| References & Assets | N/A | No reference changes |
| Repo Hygiene | OK | No secrets, no cross-skill deps, scoped change |
Issues for Manual Review
- Skill description overlap. The PR description correctly identifies that
uipath-planneranduipath-maestro-casehave overlapping trigger surface around "design document" phrasing. This prompt fix works around the overlap, but the underlying ambiguity remains. If other test prompts (or real user prompts) use "design document" wording with case-management intent, they'll hit the same planner mismatch. The PR defers a skill-description fix — worth tracking. - CI verification. The PR mentions a coder-eval run was dispatched. Confirming a passing score on the updated prompt would complete validation.
Conclusion
Clean, well-scoped fix. The prompt change correctly re-anchors on uipath-maestro-case's trigger clauses and removes the planner-bait phrasing. No issues found — approve.
Why
skill-case-phase-0-loan-originationfailed on 2026-07-16 (score 0.118) with a complete, coherent 34KB draft on disk. The run never invokeduipath-maestro-case: the closing ask "I want a draft design document" matcheduipath-planner's Phase-D triggers ("design/architect/generate an SDD → Phase D"), and planner Rule 6 wrote the draft toloan-origination-sdd.mdin one shot — no interview. Every criterion and the LLM judge read onlysdd.draft.md/sdd.md, so everything scored 0 (judge received<file not found>).What
Prompt-only change: the closing paragraph now anchors on maestro-case's own trigger clauses — "We don't have an sdd.md yet — interview me…" (matches "via lightweight interview if sdd.md absent") and "build the caseplan later" (matches "Always invoke for caseplan.json"; planner redirects caseplan → maestro-case) — and drops the planner-bait "design document" wording. Criteria, judge, and simulation are untouched.
Note: the planner/maestro-case description overlap is the underlying product gap (planner's
deferral_single_casetest only passes because its prompt avoids design-document phrasing) — deferred to a separate skill-description change if this recurs.Verification
/lint-taskverdict: Medium (pre-existing advisory — no skill_triggered gate; unchanged by this PR)skill-case-phase-0-loan-origination: SUCCESS, score 1.000, 6 interview turns (vs 1 turn / 0.118 on the failing 07-16 run),llm_judge1.00🤖 Generated with Claude Code