Skip to content

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

Merged
benvinegar merged 1 commit into
fix/stale-draft-input-racefrom
chore/remove-unused-git-export
Sep 8, 2026
Merged

benvinegar merged 1 commit into
fix/stale-draft-input-racefrom
chore/remove-unused-git-export

Conversation

@benvinegar

@benvinegar benvinegar commented Sep 8, 2026 •

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

Stack

Validation

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

@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 Preview Sep 8, 2026 4:32am UTC

Request Review

@greptile-apps

greptile-apps Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR internalizes the tree-ref resolver without changing its use by the diff endpoint.

  • Removes the unused module-level export from resolveGitTreeRefAsync.
  • Retains the helper and its existing internal call path.
  • Adds an empty maintenance changeset.

Confidence Score: 5/5

The PR appears safe to merge because the removed export was not part of the package entrypoint and has no external repository callers.

The change only narrows an unused module-level export; the resolver remains available to its existing internal caller, and no actionable failures remain.

Important Files Changed

Filename Overview
packages/hunk-git/src/commands.ts Makes the tree-ref resolver module-private while preserving its implementation and internal usage.
.changeset/clean-git-tree-export.md Adds an intentionally empty changeset for the internal maintenance update.

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

@benvinegar
benvinegar force-pushed the chore/remove-unused-git-export branch from 3008609 to 12c5371 Compare September 8, 2026 04:32
@benvinegar
benvinegar changed the base branch from main to fix/stale-draft-input-race September 8, 2026 04:34
@benvinegar
benvinegar merged commit aefc283 into fix/stale-draft-input-race Sep 8, 2026
17 of 18 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