Skip to content

Conversation

@Wolfe1
Copy link

@Wolfe1 Wolfe1 commented Nov 6, 2025

Swap branch and commit parameters when constructing Bitbucket diff specs so the diff shows what the source adds to the destination (additions as +, deletions as -). Apply the same reversal logic to branch and commit diffs and update the changelog to reflect the bug fix. This fixes incorrect PR diff classification and improves displayed diff accuracy.

Fixes #84

Before:
image

After:

image

Swap branch and commit parameters when constructing Bitbucket diff specs so
the diff shows what the source adds to the destination (additions as +,
deletions as -). Apply the same reversal logic to branch and commit diffs
and update the changelog to reflect the bug fix. This fixes incorrect PR
diff classification and improves displayed diff accuracy.
@igorvelho
Copy link

I was investigating the very same issue, I just wonder if instead of hardcode the diff direction, you should introduce a parameter on the mcp tool to specify direction, like
{ "repoSlug": "myrepo", "sourceBranch": "feat-123", "destinationBranch": "master", "includeFullDiff": true, **"diffDirection": "sourceToDestination"** }

@Wolfe1
Copy link
Author

Wolfe1 commented Nov 7, 2025

@igorvelho I honestly don't understand why someone would want this though?

And if they really did want a diff in the other direction then they could just switch the source and destination rather than there being some flag that changes the intended purpose.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants