Skip to content

fix(goal): align manual hunt verdict handling#3694

Merged
Hmbown merged 1 commit into
Hmbown:mainfrom
cyq1017:codex/codewhale-2093-hunt-verdict-path
Jun 27, 2026
Merged

fix(goal): align manual hunt verdict handling#3694
Hmbown merged 1 commit into
Hmbown:mainfrom
cyq1017:codex/codewhale-2093-hunt-verdict-path

Conversation

@cyq1017

@cyq1017 cyq1017 commented Jun 27, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add /goal declare-hunted and legacy force-complete override routing
  • keep trophy cards limited to hunted verdicts so escaped verdicts do not write trophies
  • use hunt verdict labels/strings in goal display, trophy metadata, and declared-hunted audit details

Refs #2093

Verification

  • cargo test -p codewhale-tui --bin codewhale-tui --locked goal -- --nocapture
  • cargo check -p codewhale-tui --bin codewhale-tui --locked
  • cargo fmt --all -- --check
  • git diff --check

@Hmbown

Hmbown commented Jun 27, 2026

Copy link
Copy Markdown
Owner

Thank you @cyq1017 — this is a tidy, focused slice of #2093. I checked the diff and the shape looks good: trophy cards now stay limited to hunted verdicts, declare-hunted gets an explicit audit path, and the command/status copy now matches the hunted/wounded/escaped vocabulary. CI is green across lint, Ubuntu, macOS, Windows, npm smoke, mobile smoke, DCO/gate, and GitGuardian, and the branch is clean against current main (merge-tree + diff --check are clean).\n\nI am leaving it unmerged only because it is still marked draft. Once you mark it ready, this looks like a strong merge candidate; #2093 should stay open afterward for the remaining verifier/glyph/policy wiring.

@cyq1017 cyq1017 marked this pull request as ready for review June 27, 2026 13:33
@cyq1017 cyq1017 requested a review from Hmbown as a code owner June 27, 2026 13:33
@Hmbown Hmbown merged commit a699b60 into Hmbown:main Jun 27, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants