test: add unit tests for canon/parse.go — Parse, splitChain, tokenize

[jpleva91]

Problem

canon/parse.go has zero test coverage. This is the core command parser — it splits chains (&&, ||, ;, |), respects quoting, extracts tools/actions/flags/args. Bugs here affect every governance decision.

Acceptance Criteria

• Test Parse() splits && chains into multiple segments with correct OpAnd
• Test Parse() splits || chains with OpOr
• Test Parse() splits pipes | with OpPipe
• Test Parse() splits semicolons ; with OpSeq
• Test quoted strings are not split (single and double quotes)
• Test escaped characters are preserved
• Test ParseOne() for simple single commands
• Test tokenize() respects quoting and backslash escapes
• Test parseFlag() splits --count=5 into name/value
• Test expandShortFlags() expands -rn to ["-r", "-n"] but not --verbose or -C5
• Test empty/whitespace input edge cases
• All tests pass with go test ./canon/...

Hints

• File: canon/parse.go
• Functions: Parse(), ParseOne(), splitChain(), tokenize(), parseOne(), parseFlag(), expandShortFlags()
• Existing test file canon/canon_test.go may have some coverage — check for overlap before writing
• Create test file: canon/parse_test.go

---

Generated by /forge fill

[jpleva91]

🧠 Triage complete

Tier B — Needs Planning. Issue requires scoping and decomposition before implementation.

Reason: triage error (defaulting to safe): API returned 404: {"type":"error","error":{"type":"not_found_error","message":"model: claude-3-haiku-20241022"},"request_id":"req_011CZrvDzXzdi9wq93S8dyPf"}
Confidence: 0.00

Powered by Octi Pulpo pipeline

[jpleva91]

@claude Please implement this issue. Read the acceptance criteria carefully, check the source files mentioned in Hints, and create the test file. Run go test to verify all tests pass before finishing.

[jpleva91]

Test coverage for canon/parse.go command parsing; medium complexity, safe to automate.

---

Generated by Claude Code