TC #59 — Phase 6 @<r1>..<r2> tiered resolution #59

New Issue

2026-05-16T22:29:08Z

claude-noether commented

2026-05-16 22:29:08 +00:00

Scenario

In a git repo, send a user prompt containing each variant:

look at @HEAD~1..HEAD please
review @../existing-file.lua please
check @origin/main..feature please    (if applicable)
explain @nonsense..gibberish please
test @HEAD~1..HEAD, then explain

Expected

@HEAD~1..HEAD → expand_mentions substitutes a ```diff path=HEAD~1..HEAD fenced block; status @HEAD~1..HEAD expanded (<N> bytes, diff)
@../existing-file.lua → path branch wins; treated as file expansion, NOT diff
@origin/main..feature → path branch fails; ref-range succeeds; ```diff block (if the ref exists)
@nonsense..gibberish → both branches fail; literal @nonsense..gibberish preserved; status @nonsense..gibberish: not found
@HEAD~1..HEAD, → trailing comma peeled, ref resolves, comma after closing fence

Acceptance

All five rules per A6. The path branch ALWAYS gets the first shot (no false-positive diff on existing files).

## Scenario In a git repo, send a user prompt containing each variant: ``` look at @HEAD~1..HEAD please review @../existing-file.lua please check @origin/main..feature please (if applicable) explain @nonsense..gibberish please test @HEAD~1..HEAD, then explain ``` ## Expected - `@HEAD~1..HEAD` → expand_mentions substitutes a ```diff path=HEAD~1..HEAD fenced block; status `@HEAD~1..HEAD expanded (<N> bytes, diff)` - `@../existing-file.lua` → path branch wins; treated as file expansion, NOT diff - `@origin/main..feature` → path branch fails; ref-range succeeds; ```diff block (if the ref exists) - `@nonsense..gibberish` → both branches fail; literal `@nonsense..gibberish` preserved; status `@nonsense..gibberish: not found` - `@HEAD~1..HEAD,` → trailing comma peeled, ref resolves, comma after closing fence ## Acceptance All five rules per A6. The path branch ALWAYS gets the first shot (no false-positive diff on existing files).

claude-noether added the test-case label 2026-05-16 22:29:08 +00:00

claude-noether commented

2026-05-16 22:29:55 +00:00

Verified PASS via autonomous test driver at commit ac58b19 (Phase 6 implement-complete).

All four sub-cases verified:

@HEAD~1..HEAD -> diff branch (status expanded ... diff)
@x..y.txt (existing file with .. in name) -> file branch wins
@nonsense..gibberish -> both fail -> not found status
@HEAD~1..HEAD, -> trailing comma peeled, diff resolved

A6 tiered-resolution priority preserved: path always tried first.

Closed by Phase 7 verify pass.

Verified PASS via autonomous test driver at commit `ac58b19` (Phase 6 implement-complete). All four sub-cases verified: - `@HEAD~1..HEAD` -> diff branch (status `expanded ... diff`) - `@x..y.txt` (existing file with `..` in name) -> file branch wins - `@nonsense..gibberish` -> both fail -> `not found` status - `@HEAD~1..HEAD,` -> trailing comma peeled, diff resolved A6 tiered-resolution priority preserved: path always tried first. Closed by Phase 7 verify pass.

claude-noether closed this issue

2026-05-16 22:29:55 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: marfrit/aish#59