Skip to content

Add GitHub commit resolver - #150

Merged
andrew merged 2 commits into
mainfrom
codex/yul-dedup
Aug 17, 2026
Merged

Add GitHub commit resolver#150
andrew merged 2 commits into
mainfrom
codex/yul-dedup

Conversation

@andrew

@andrew andrew commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Adds a GitHub commit resolver for branches, tags, and abbreviated refs. Full SHA-1 values bypass the network, and API lookups use GitHub's SHA-only response.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds a small helper to resolve GitHub refs (branch/tag/short SHA) into full commit SHAs using GitHub’s SHA-only commit endpoint, with documentation and unit tests to support the new capability.

Changes:

  • Added CommitResolver with public/enterprise base URL support and “full SHA bypasses network” behavior.
  • Added unit tests covering happy path, direct full-SHA passthrough, and error cases.
  • Documented usage in README.md.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
README.md Documents resolving GitHub refs to full commit SHAs without enumerating tags.
github/commits.go Introduces CommitResolver and resolution logic via GitHub’s SHA-only commit endpoint.
github/commits_test.go Adds tests for commit resolution, passthrough for full SHAs, and error handling.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread github/commits.go Outdated
@andrew
andrew merged commit 61442dc into main Aug 17, 2026
7 checks passed
@andrew
andrew deleted the codex/yul-dedup branch August 17, 2026 10:26
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