Skip to content

Improve apply conflict handling with parsed file list and resolution guidance#118

Merged
that-github-user merged 1 commit into
mainfrom
issue-80-conflict-handling
Mar 29, 2026
Merged

Improve apply conflict handling with parsed file list and resolution guidance#118
that-github-user merged 1 commit into
mainfrom
issue-80-conflict-handling

Conversation

@that-github-user

Copy link
Copy Markdown
Owner

Summary

  • Parse git apply stderr to identify applied vs conflicted files
  • Show green checkmarks for applied, red X for conflicted
  • Suggest resolution: try different agent, preview, manually merge
  • Exported parseApplyConflicts() for testing, 6 new tests

Agent selection: Agent #1 chosen over Copeland-recommended #5. Copeland favored #5 (fewer files = +4) but #5 added zero tests. Human override: test coverage > fewer files.

Change type

  • New feature

Related issue

Closes #80

How to test

npm test  # 161 tests pass

Breaking changes

  • This PR introduces breaking changes

🤖 Generated with thinktank (Opus, Agent #1 manually selected)

…guidance

Parse git apply stderr to show which files applied cleanly vs conflicted.
Suggest next steps: try different agent, preview first, manually merge.
Exported parseApplyConflicts for testing. 6 new tests.

Agent #1 chosen over Copeland-recommended #5 (which added zero tests).
Human judgment: tests matter more than fewer files changed.

Closes #80

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@that-github-user that-github-user merged commit 4531094 into main Mar 29, 2026
4 checks passed
@that-github-user that-github-user deleted the issue-80-conflict-handling branch March 29, 2026 00:58
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.

Improve apply conflict handling: show conflicted files and guide user to resolution

1 participant