Include user comments and tool attempts in fix prompts#622

Merged

wesm merged 28 commits intoroborev-dev:mainfrom

shoyer:fix-skill-comments

Apr 5, 2026

Commits on Apr 3, 2026

fix: include user comments in fix prompts

Stephan Hoyer
and
claude
committed
refactor: split responses into tool attempts vs user comments

Stephan Hoyer
and
claude
committed
fix: split mixed responses in fix prompts and add SHA fallback

Stephan Hoyer
and
claude
committed
feat: include comments in daemon fix prompts, show --json, and fix skills

Stephan Hoyer
and
claude
committed
fix: add GetAllCommentsForJob and apply legacy SHA fallback consistently

Stephan Hoyer
and
claude
committed
fix: use commit_id instead of SHA for legacy comment lookup

Stephan Hoyer
and
claude
committed
fix: log errors when fetching comments for fix prompts

Stephan Hoyer
and
claude
committed
docs: add sync notes to duplicated comment merge/dedup logic

Stephan Hoyer
and
claude
committed
docs: include TUI loadResponses in merge/dedup sync notes

Stephan Hoyer
and
claude
committed
fix: add SHA fallback for legacy jobs without commit_id, strengthen tests

Stephan Hoyer
and
claude
committed
fix: use commit_id for legacy comment lookup in CLI and show paths

Stephan Hoyer
and
claude
committed
fix: use commit_id for legacy comment lookup in TUI loadResponses

Stephan Hoyer
and
claude
committed
fix: propagate legacy comment lookup errors, use deterministic test timestamps

Stephan Hoyer
and
claude
committed
test: add error-path test for GetAllCommentsForJob legacy lookup

Stephan Hoyer
and
claude
committed
test: assert exact job-linked comments on legacy lookup error path

Stephan Hoyer
and
claude
committed
fix: validate gitRef as hex SHA before legacy comment fallback, warn on failures

Stephan Hoyer
and
claude
committed
fix: warn on non-200 legacy comment fetch responses

Stephan Hoyer
and
claude
committed
fix: use merged comment lookup in refine command via HTTP client

Stephan Hoyer
and
claude
committed
fix: relax looksLikeSHA to accept mixed-case and 4+ char SHAs

Stephan Hoyer
and
claude
committed
fix: address lint issues in looksLikeSHA and prompt tests

Stephan Hoyer
and
claude
committed
fix: restore 7-char minimum for looksLikeSHA, keep mixed-case support

Stephan Hoyer
and
claude
committed
refactor: consolidate looksLikeSHA into git.LooksLikeSHA

Stephan Hoyer
and
claude
committed

Commits on Apr 5, 2026

refactor: extract MergeResponses to eliminate 5x duplicated dedup logic

Stephan Hoyer
and
claude
committed
fix: clean up test assertions and unnecessary comments

Stephan Hoyer
and
claude
committed
fix: use --commenter in fix/refine skills for proper tool identification

Stephan Hoyer
and
claude
committed
docs: simplify comment handling instruction in fix skills

Stephan Hoyer
and
claude
committed
merge: resolve conflict with origin/main (#624)

Stephan Hoyer
and
claude
committed
refactor: remove storage->git dependency, add CommitIDValue helper

Stephan Hoyer
and
claude
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Include user comments and tool attempts in fix prompts#622

Include user comments and tool attempts in fix prompts#622
wesm merged 28 commits intoroborev-dev:mainfrom
shoyer:fix-skill-comments

Commits on Apr 3, 2026

fix: include user comments in fix prompts

refactor: split responses into tool attempts vs user comments

fix: split mixed responses in fix prompts and add SHA fallback

feat: include comments in daemon fix prompts, show --json, and fix skills

fix: add GetAllCommentsForJob and apply legacy SHA fallback consistently

fix: use commit_id instead of SHA for legacy comment lookup

fix: log errors when fetching comments for fix prompts

docs: add sync notes to duplicated comment merge/dedup logic

docs: include TUI loadResponses in merge/dedup sync notes

fix: add SHA fallback for legacy jobs without commit_id, strengthen tests

fix: use commit_id for legacy comment lookup in CLI and show paths

fix: use commit_id for legacy comment lookup in TUI loadResponses

fix: propagate legacy comment lookup errors, use deterministic test timestamps

test: add error-path test for GetAllCommentsForJob legacy lookup

test: assert exact job-linked comments on legacy lookup error path

fix: validate gitRef as hex SHA before legacy comment fallback, warn on failures

fix: warn on non-200 legacy comment fetch responses

fix: use merged comment lookup in refine command via HTTP client

fix: relax looksLikeSHA to accept mixed-case and 4+ char SHAs

fix: address lint issues in looksLikeSHA and prompt tests

fix: restore 7-char minimum for looksLikeSHA, keep mixed-case support

refactor: consolidate looksLikeSHA into git.LooksLikeSHA

Commits on Apr 5, 2026

refactor: extract MergeResponses to eliminate 5x duplicated dedup logic

fix: clean up test assertions and unnecessary comments

fix: use --commenter in fix/refine skills for proper tool identification

docs: simplify comment handling instruction in fix skills

merge: resolve conflict with origin/main (#624)

refactor: remove storage->git dependency, add CommitIDValue helper