Skip to content

Conversation

@aluzzardi
Copy link
Contributor

  • Add environment log and diff tools via MCP
  • Add -b flag to cu {log,diff}

Signed-off-by: Andrea Luzzardi <al@dagger.io>
# Quick assessment before merging
cu diff backend-api`,
cu diff backend-api -b main`,
Copy link
Contributor

@cwlbraa cwlbraa Jul 3, 2025

Choose a reason for hiding this comment

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

in raw git, isn't this equivalent to git diff main fancy-mallard? or git diff fancy-mallard main we should make sure we're copying that UX wherever possible.

also why does it say backend-api, those are adverb animals lol

Copy link
Contributor Author

Choose a reason for hiding this comment

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

cu diff is slightly different than a git diff ... out of the box, it does a git diff $(git merge-base $(git branch --show-current) CU_ENV))..CU_ENV

The -b here is replacing the merge-base stuff with whatever is in -b

Really unsure whether we should copy the syntax or not here, especially since I think we'll want to support paths (like git) and the syntax gets tricky

git diff main fancy-mallard
git diff main..fancy-mallard
git diff main..fancy-mallard ./cmd
git diff main fancy-mallard -- ./cmd

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Any opinion from a git UX POV?

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.

3 participants