Skip to content

chore(git): internalize tree ref resolver - #1052

Merged
benvinegar merged 1 commit into
mainfrom
chore/remove-unused-git-export-main
Sep 8, 2026
Merged

benvinegar merged 1 commit into
mainfrom
chore/remove-unused-git-export-main

Conversation

@benvinegar

Copy link
Copy Markdown
Member

Summary

  • remove the unused public export from the tree-ref helper
  • retain the helper for internal diff endpoint resolution
  • add an empty maintenance changeset

This is the direct-to-main replacement for #1047, which GitHub marked merged only into the now-merged note-fix branch.

Validation

  • bun run format:check
  • bun run typecheck
  • bun run knip
  • bun test packages/hunk-git/src/commands.test.ts

@vercel

vercel Bot commented Sep 8, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
hunk-web Ignored Ignored Sep 8, 2026 12:22pm UTC

Request Review

@benvinegar
benvinegar enabled auto-merge (squash) September 8, 2026 12:23
@greptile-apps

greptile-apps Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR internalizes an implementation-only Git tree reference resolver without changing its behavior.

  • Removes the unused export modifier from resolveGitTreeRefAsync.
  • Retains the helper for internal diff endpoint resolution.
  • Adds the documented empty changeset for a maintenance-only change.

Confidence Score: 5/5

The PR appears safe to merge because it only removes an unused module-level export from a private package and preserves all runtime behavior.

The resolver remains available to its sole internal caller, is absent from the package's public entry point, and the empty changeset is appropriate for this non-release maintenance change.

Important Files Changed

Filename Overview
packages/hunk-git/src/commands.ts Makes an internally used tree reference resolver module-private; no public entry point or repository caller depends on the removed export.
.changeset/clean-git-tree-export.md Adds an empty maintenance changeset consistent with the repository's release policy.

Reviews (1): Last reviewed commit: "chore(git): internalize tree ref resolve..." | Re-trigger Greptile

@benvinegar
benvinegar merged commit 8924e0a into main Sep 8, 2026
13 checks passed
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.

1 participant