Skip to content

refactor: move hunkdiff into packages/hunk - #997

Merged
benvinegar merged 4 commits into
mainfrom
refactor/package-first-phase-1
Sep 6, 2026
Merged

refactor: move hunkdiff into packages/hunk#997
benvinegar merged 4 commits into
mainfrom
refactor/package-first-phase-1

Conversation

@benvinegar

Copy link
Copy Markdown
Member

Summary

Move the published hunkdiff package into packages/hunk and turn the repository root into a private Bun workspace controller.

This is the first independently reviewable phase of the package-first monorepo migration. It replaces the all-in-one exploratory approach preserved in #994.

What changed

  • moved application source, bin wrapper, and shipped skills under packages/hunk;
  • moved the public hunkdiff manifest to packages/hunk/package.json while keeping the root private and orchestration-only;
  • preserved hunkdiff/extension, hunkdiff/opentui, bin aliases, Pi skill discovery, and existing runtime behavior;
  • retargeted TypeScript, dependency boundaries, tests, benchmarks, examples, docs, website, CI, Nix, installers, and source/prebuilt packaging;
  • kept the repository-root CHANGELOG.md canonical through an explicit Changesets version wrapper and real Changesets integration test;
  • added packed-package consumer checks for both public subpaths under NodeNext and Bundler resolution;
  • added package-content checks for README/LICENSE/skills parity and private/workspace dependency leakage.

Scope and non-goals

This PR is intentionally mechanical:

  • bundled Git, Jujutsu, and Sapling providers remain inside packages/hunk/src/extensions/default/vcs;
  • no extension package identity was introduced;
  • no enable/disable lifecycle or new extension commands were added;
  • no provider package extraction or publication changes are included;
  • no Turbo dependency was added.

Follow-up phases will handle those concerns separately after this PR lands.

Validation

Passed locally:

  • bun run format:check
  • bun run lint
  • bun run typecheck
  • bun run deps:check
  • bun run knip
  • bun run check:docs
  • bun run check:changelog
  • bun run changeset:status
  • bun run test
  • bun run test:session-broker-node
  • bun run test:integration — 143 passed, 1 macOS-only skip
  • bun run test:tty-smoke — 9 passed
  • bun run check:pack
  • bun run build:prebuilt:npm
  • bun run check:prebuilt-pack
  • bun run smoke:prebuilt-install
  • bun run build:bin

The packed hunkdiff/extension and hunkdiff/opentui exports were validated under NodeNext and Bundler resolution. Independent fresh-context review returned ACCEPT with no blocker/high findings.

Not available locally:

  • real Nix evaluation/build (focused Nix path tests passed);
  • Firecracker install-VM execution.

This PR description was generated by Pi using gpt-5.6-sol

@greptile-apps

greptile-apps Bot commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

Too many files changed for review (781 files, 50 file limit).

@vercel

vercel Bot commented Sep 6, 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 6, 2026 1:34am UTC

Request Review

@benvinegar
benvinegar force-pushed the refactor/package-first-phase-1 branch from 1121965 to b697137 Compare September 6, 2026 01:21
@benvinegar
benvinegar enabled auto-merge (squash) September 6, 2026 01:34
@benvinegar
benvinegar merged commit e76b003 into main Sep 6, 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