Skip to content

feat(aidd-vcs): add conflict resolution skill - #747

Open
alexsoyes wants to merge 5 commits into
nextfrom
codex/resolve-conflict-skill
Open

alexsoyes wants to merge 5 commits into
nextfrom
codex/resolve-conflict-skill

Conversation

@alexsoyes

@alexsoyes alexsoyes commented Sep 2, 2026 •

Copy link
Copy Markdown
Contributor

🎯 What & why

Add aidd-vcs:05-resolve-conflict, a skill that instructs the agent to handle an active merge, rebase, or cherry-pick conflict. It adds no Git resolver to the CLI.

🛠️ How it works

The agent records one decision per conflicted hunk or non-text path in the resolution table. Identical sides use common content; a previously approved choice applies only if it still matches. Any undecided choice is proposed without changing files or the index.

Once every choice is decided, the agent applies them, stages only resolved paths, and checks for remaining unmerged paths and staged whitespace errors before marking rows Applied. It does not commit, continue the Git operation, reset, or discard changes. These are agent instructions, not executable guarantees.

🧪 How to verify

  • git diff --check passed. The architecture rules test passed 30/30; the pre-commit scripts suite passed 549/549.
  • Exercise one conflict with an unapproved choice, then approve it and verify the working tree, index, and decision statuses. No automated test executes this skill.
  • The full local CLI suite on this machine is blocked by the separate E2E sandbox issue in fix(test): preserve Node in E2E sandbox #963: its existing Node availability guard reports expected '' not to be ''.

⚠️ Heads-up

The unrelated Improve changes are covered by #945. The E2E sandbox fix is isolated in #963. Both were removed from this PR's diff.

✅ I certify

  • I DO CERTIFY I READ EACH LINE OF THE PULL REQUEST BECAUSE I AM A SOFTWARE ENGINEER, NOT A AI PUPPY.

@alexsoyes
alexsoyes marked this pull request as ready for review September 3, 2026 03:08
@alexsoyes
alexsoyes requested a review from a team as a code owner September 3, 2026 03:08
@alexsoyes
alexsoyes changed the base branch from main to next September 3, 2026 03:08
Comment thread plugins/aidd-vcs/skills/05-resolve-conflict/actions/01-resolve.md
@blafourcade

Copy link
Copy Markdown
Contributor

Peux-tu résolve les conflits avec le skill resolve-conflits dans cette PR ? 😄

Adds 05-resolve-conflict: deterministic conflicts and approved choices
apply, anything else returns an unapplied proposal table and stops.

Rebased onto next and squashed. The four original commits each replayed
the same conflicts against intermediate text; the final tree is
unchanged. Conflicts were the plugin version release-please had bumped
and the catalogue rows #743 had rewritten, not the skill itself.

Refs #747

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FPREgkoNtK4PYh2YyZbztq
AIDD-Session-Id: 2261efcd-a873-4e60-9893-2217f702bde2
@blafourcade
blafourcade force-pushed the codex/resolve-conflict-skill branch from 02e7f21 to 45cba67 Compare September 21, 2026 03:49
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