Skip to content

feat(stack): add common-mistakes and conflict-resolution to skill#1105

Open
jd wants to merge 1 commit intomainfrom
devs/jd/worktree-enhance-skill/I5447c05aae8009c9e8f313a457e7e1fe440a98ef
Open

feat(stack): add common-mistakes and conflict-resolution to skill#1105
jd wants to merge 1 commit intomainfrom
devs/jd/worktree-enhance-skill/I5447c05aae8009c9e8f313a457e7e1fe440a98ef

Conversation

@jd
Copy link
Copy Markdown
Member

@jd jd commented Apr 9, 2026

Add two new sections to the stack skill to reduce agent errors:

  • "Common Mistakes" table with wrong/right/why patterns for the 6 most
    frequent agent errors (git push, fix commits, manual PR edits, etc.)
  • "Conflict Resolution" guide for rebase conflict handling

Also enhance the CRITICAL section to include mergify stack list as
part of the pre-operation check, so agents assess stack state before
acting.

Add two new sections to the stack skill to reduce agent errors:
- "Common Mistakes" table with wrong/right/why patterns for the 6 most
  frequent agent errors (git push, fix commits, manual PR edits, etc.)
- "Conflict Resolution" guide for rebase conflict handling

Also enhance the CRITICAL section to include `mergify stack list` as
part of the pre-operation check, so agents assess stack state before
acting.

Change-Id: I5447c05aae8009c9e8f313a457e7e1fe440a98ef
Claude-Session-Id: badc87ae-b63f-4c15-b841-406df2a103b6
Copilot AI review requested due to automatic review settings April 9, 2026 15:30
@mergify mergify bot had a problem deploying to Mergify Merge Protections April 9, 2026 15:30 Failure
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Apr 9, 2026

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🔴 👀 Review Requirements

This rule is failing.
  • any of:
    • #approved-reviews-by>=2
    • author = dependabot[bot]
    • author = mergify-ci-bot
    • author = renovate[bot]

🔴 🔎 Reviews

This rule is failing.
  • #review-requested = 0
  • #changes-requested-reviews-by = 0
  • #review-threads-unresolved = 0

🟢 🤖 Continuous Integration

Wonderful, this rule succeeded.
  • all of:
    • check-success=ci-gate

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert)(?:\(.+\))?:

🟢 📕 PR description

Wonderful, this rule succeeded.
  • body ~= (?ms:.{48,})

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the mergify stack skill documentation to reduce common agent workflow errors and provide guidance for resolving rebase conflicts.

Changes:

  • Added a “Common Mistakes” table mapping frequent wrong actions to the correct mergify stack workflow.
  • Expanded the pre-commit “CRITICAL” check to include mergify stack list to assess stack state before acting.
  • Added a “Conflict Resolution” section covering standard rebase conflict steps and recovery.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mergify mergify bot requested a review from a team April 9, 2026 15:33
@jd jd marked this pull request as ready for review April 9, 2026 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants