git-chopstick-core 0.1.7 → 0.1.9

package/CHANGELOG.md

@@ -1,5 +1,25 @@
 # Changelog
 
+## [0.1.9] — 2026-06-13
+
+### Added
+- **Test fixtures**: Created `src/__tests__/fixtures/test-repo.bundle` — a pre-built git bundle with 6 commits, 3 branches, and 1 tag for faster test setup. Added `src/__tests__/fixture-helpers.ts` with `setupFixtureRepo()` to clone the bundle in ~10ms instead of building a repo from scratch. Refactored integration tests to use the fixture.
+- **Release skill moved**: Moved from `docs/skills/release.md` to `.agents/skills/release-npm.md` and renamed to `release-npm`.
+- **Global default branch**: Set `git config --global init.defaultBranch main`.
+
+### Changed
+- **Fixture rebuild instructions**: Updated to use `git init -q` (relies on global `init.defaultBranch`).
+- **`prepublishOnly` now runs tests**: Integration tests run during `npm publish` to catch regressions before they ship.
+
+---
+
+## [0.1.8] — 2026-06-13
+
+### Added
+- **Test files included in package**: Added `src/__tests__/` to the `files` array in `package.json`. Integration tests ship with the installed package so consumers can run `npm test` to verify the package works for their environment.
+
+---
+
 ## [0.1.7] — 2026-06-13
 
 ### Added
@@ -122,11 +142,8 @@
 
 ---
 
-[0.1.4]: https://github.com/parkiyong/git-chopstick-core/releases/tag/v0.1.4
-[0.1.3]: https://github.com/parkiyong/git-chopstick-core/releases/tag/v0.1.3
-[0.1.2]: https://github.com/parkiyong/git-chopstick-core/releases/tag/v0.1.2
-[0.1.1]: https://github.com/parkiyong/git-chopstick-core/releases/tag/v0.1.1
-[0.1.0]: https://github.com/parkiyong/git-chopstick-core/releases/tag/v0.1.0
+[0.1.9]: https://github.com/parkiyong/git-chopstick-core/releases/tag/v0.1.9
+[0.1.8]: https://github.com/parkiyong/git-chopstick-core/releases/tag/v0.1.8
 [0.1.7]: https://github.com/parkiyong/git-chopstick-core/releases/tag/v0.1.7
 [0.1.6]: https://github.com/parkiyong/git-chopstick-core/releases/tag/v0.1.6
 [0.1.5]: https://github.com/parkiyong/git-chopstick-core/releases/tag/v0.1.5
@@ -135,4 +152,4 @@
 [0.1.2]: https://github.com/parkiyong/git-chopstick-core/releases/tag/v0.1.2
 [0.1.1]: https://github.com/parkiyong/git-chopstick-core/releases/tag/v0.1.1
 [0.1.0]: https://github.com/parkiyong/git-chopstick-core/releases/tag/v0.1.0
-[Unreleased]: https://github.com/parkiyong/git-chopstick-core/compare/v0.1.7...HEAD
+[Unreleased]: https://github.com/parkiyong/git-chopstick-core/compare/v0.1.9...HEAD

package/README.md

@@ -26,7 +26,7 @@ npm install file:../path/to/git-chopstick-core
 ```typescript
 import { Repository, getStatus, GitError } from 'git-chopstick-core'
 
-const repo = new Repository('/path/to/repo', 1)
+const repo = new Repository('/path/to/repo')
 
 try {
   const status = await getStatus(repo)
@@ -46,7 +46,7 @@ try {
 ```typescript
 import { Repository, getStatus } from 'git-chopstick-core'
 
-const repo = new Repository('/path/to/repo', 1)
+const repo = new Repository('/path/to/repo')
 const status = await getStatus(repo)
 
 if (!status) {
@@ -81,7 +81,7 @@ if (status.rebaseInternalState) {
 ```typescript
 import { Repository, getStatus, createCommit } from 'git-chopstick-core'
 
-const repo = new Repository('/path/to/repo', 1)
+const repo = new Repository('/path/to/repo')
 const status = await getStatus(repo)
 
 // Stage and commit all tracked files
@@ -98,7 +98,7 @@ import {
   deleteLocalBranch, renameBranch, getBranches
 } from 'git-chopstick-core'
 
-const repo = new Repository('/path/to/repo', 1)
+const repo = new Repository('/path/to/repo')
 
 // Create a branch
 await createBranch(repo, 'feature/new-feature', 'main')
@@ -125,7 +125,7 @@ import {
   abortMerge, getMergeBase, isMergeHeadSet
 } from 'git-chopstick-core'
 
-const repo = new Repository('/path/to/repo', 1)
+const repo = new Repository('/path/to/repo')
 
 const result = await merge(repo, 'feature/new-feature')
 
@@ -149,7 +149,7 @@ import {
   Repository, push, pull, getRemotes, getStatus
 } from 'git-chopstick-core'
 
-const repo = new Repository('/path/to/repo', 1)
+const repo = new Repository('/path/to/repo')
 const [remote] = await getRemotes(repo)
 const status = await getStatus(repo)
 
@@ -167,7 +167,7 @@ import {
   Repository, merge, GitError, GitErrorCodes
 } from 'git-chopstick-core'
 
-const repo = new Repository('/path/to/repo', 1)
+const repo = new Repository('/path/to/repo')
 
 try {
   await merge(repo, 'other-branch')
@@ -328,11 +328,14 @@ src/
 # Type-check
 npm run typecheck
 
+# Run integration tests (uses vitest, creates temp git repos)
+npm test
+
 # Build (compile TypeScript → dist/)
 npm run build
 
-# Clean rebuild (build already cleans automatically)
-npm run build
+# Package contents dry-run (verify files before publish)
+npm pack --dry-run
 
 # Run examples (uses tsx for source-level execution)
 npx tsx examples/get-status.ts /path/to/repo

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "git-chopstick-core",
-  "version": "0.1.7",
+  "version": "0.1.9",
   "type": "module",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",
@@ -33,7 +33,8 @@
   "files": [
     "dist",
     "CHANGELOG.md",
-    "docs"
+    "docs",
+    "src/__tests__"
   ],
   "description": "Git backend library extracted from GitHub Desktop",
   "scripts": {
@@ -41,7 +42,7 @@
     "build": "npm run clean && tsc",
     "typecheck": "tsc --noEmit",
     "test": "vitest run",
-    "prepublishOnly": "npm run typecheck && npm run build"
+    "prepublishOnly": "npm run typecheck && npm test && npm run build"
   },
   "dependencies": {
     "byline": "^5.0.0",

package/src/__tests__/fixture-helpers.ts

@@ -0,0 +1,105 @@
+/**
+ * To rebuild the fixture bundle:
+ *
+ *   cd /tmp && rm -rf fixture-builder && mkdir fixture-builder && cd fixture-builder
+ *   git init -q && git config user.email f@t.t && git config user.name F
+ *   echo '# Test Repo' > README.md && echo 'node_modules/' > .gitignore
+ *   git add .gitignore README.md && git commit -m 'initial: add README and gitignore'
+ *   echo 'hello' > hello.txt && mkdir -p src && echo 'console.log("hi")' > src/index.js
+ *   git add hello.txt src/index.js && git commit -m 'feat: add hello.txt and src/index.js'
+ *   echo 'world' > hello.txt && echo 'console.log("hello world")' > src/index.js
+ *   git add -A && git commit -m 'feat: update hello and src/index.js'
+ *   git checkout -b feature/one && echo 'feature one content' > feature-one.txt
+ *   git add feature-one.txt && git commit -m 'feat: feature one'
+ *   git checkout main && echo 'another change' > another.txt
+ *   git add another.txt && git commit -m 'feat: add another.txt'
+ *   git merge feature/one --no-ff --no-edit -m 'merge: feature/one into main'
+ *   git checkout -b feature/two && echo 'feature two content' > feature-two.txt
+ *   git add feature-two.txt && git commit -m 'feat: feature two'
+ *   git checkout main && git tag -a v0.1.0 -m 'v0.1.0'
+ *   git bundle create test-repo.bundle --all
+ *   cp test-repo.bundle <this-dir>/fixtures/
+ *
+ * This creates a bundle with 6 commits on main, 3 branches, and 1 annotated tag.
+ */
+import { mkdtempSync } from 'fs'
+import { execSync } from 'child_process'
+import { tmpdir } from 'os'
+import { dirname, join } from 'path'
+import { fileURLToPath } from 'url'
+import { Repository } from '../models/repository.js'
+
+const __dirname = dirname(fileURLToPath(import.meta.url))
+
+/**
+ * Path to the pre-built fixture repo bundle.
+ */
+export const FIXTURE_BUNDLE_PATH = join(__dirname, 'fixtures', 'test-repo.bundle')
+
+/**
+ * Clone the fixture bundle to a temp directory, configure git identity,
+ * create local branches for all remote-tracking branches, and return
+ * the path and a Repository object.
+ *
+ * Use this in `beforeAll` for faster test setup — avoids creating a repo
+ * from scratch for every test suite.
+ */
+export function setupFixtureRepo(): { repoPath: string; repo: Repository } {
+  const repoPath = mkdtempSync(join(tmpdir(), 'gcctest-fixture-'))
+
+  execSync(`git clone "${FIXTURE_BUNDLE_PATH}" "${repoPath}"`, {
+    stdio: 'pipe',
+  })
+
+  execSync('git config user.email test@test.test', {
+    cwd: repoPath,
+    stdio: 'pipe',
+  })
+  execSync('git config user.name Test', {
+    cwd: repoPath,
+    stdio: 'pipe',
+  })
+
+  // Create local branches for all remote-tracking branches (bundle clones
+  // only create a local branch for HEAD — everything else becomes origin/*)
+  const localBranches = execSync(
+    'git branch --format="%(refname:short)"',
+    { cwd: repoPath, encoding: 'utf-8', stdio: 'pipe' }
+  )
+    .trim()
+    .split('\n')
+    .filter(Boolean)
+
+  const remoteBranches = execSync(
+    'git branch -r --format="%(refname:lstrip=3)"',
+    { cwd: repoPath, encoding: 'utf-8', stdio: 'pipe' }
+  )
+    .trim()
+    .split('\n')
+    .filter(Boolean)
+    .filter(b => b !== 'HEAD' && !localBranches.includes(b))
+
+  for (const branch of remoteBranches) {
+    execSync(`git branch --track "${branch}" "origin/${branch}"`, {
+      cwd: repoPath,
+      stdio: 'pipe',
+    })
+  }
+
+  const repo = new Repository(repoPath)
+  return { repoPath, repo }
+}
+
+/**
+ * Run a git command in the fixture repo.
+ */
+export function git(repoPath: string, args: string) {
+  execSync(`git ${args}`, { cwd: repoPath, stdio: 'pipe' })
+}
+
+/**
+ * Clean up a fixture repo directory.
+ */
+export function cleanupFixtureRepo(repoPath: string) {
+  execSync(`rm -rf ${repoPath}`)
+}

package/src/__tests__/integration.test.ts

@@ -0,0 +1,158 @@
+import { describe, it, expect, beforeAll, afterAll } from 'vitest'
+import { mkdtempSync, writeFileSync } from 'fs'
+import { execSync } from 'child_process'
+import { tmpdir } from 'os'
+import { join } from 'path'
+import {
+  Repository, getStatus, getCommits, getBranches,
+  createCommit, createBranch, deleteLocalBranch, renameBranch,
+  getCurrentBranch, getRepositoryType, getAllTags,
+} from '../index.js'
+import {
+  setupFixtureRepo, cleanupFixtureRepo, git,
+} from './fixture-helpers.js'
+
+let repoPath: string
+let repo: Repository
+
+beforeAll(() => {
+  const fixture = setupFixtureRepo()
+  repoPath = fixture.repoPath
+  repo = fixture.repo
+})
+
+describe('getRepositoryType', () => {
+  it('returns regular for a valid repo', async () => {
+    const type = await getRepositoryType(repoPath)
+    expect(type.kind).toBe('regular')
+    if (type.kind === 'regular') {
+      expect(type.topLevelWorkingDirectory).toBe(repoPath)
+    }
+  })
+
+  it('returns missing for a non-existent path', async () => {
+    const type = await getRepositoryType('/nonexistent/path')
+    expect(type.kind).toBe('missing')
+  })
+})
+
+describe('getCurrentBranch', () => {
+  it('returns the current branch name', async () => {
+    const branch = await getCurrentBranch(repoPath)
+    expect(branch).toBe('main')
+  })
+
+  it('returns undefined when HEAD is detached', async () => {
+    git(repoPath, 'checkout --detach')
+    const branch = await getCurrentBranch(repoPath)
+    expect(branch).toBeUndefined()
+    // Reset back to a branch for subsequent tests
+    git(repoPath, 'checkout main')
+  })
+})
+
+describe('getStatus', () => {
+  it('returns a status with branch info', async () => {
+    const status = await getStatus(repo)
+    expect(status).toBeTruthy()
+    expect(status!.currentBranch).toBe('main')
+  })
+
+  it('detects untracked files', async () => {
+    writeFileSync(join(repoPath, 'untracked.txt'), 'hello')
+    const status = await getStatus(repo)
+    expect(status!.workingDirectory.files.length).toBeGreaterThanOrEqual(1)
+    const untracked = status!.workingDirectory.files.find(
+      f => f.path === 'untracked.txt'
+    )
+    expect(untracked).toBeTruthy()
+  })
+
+  it('returns null for a path that exists but is not a git repo', async () => {
+    const nonRepoPath = mkdtempSync(join(tmpdir(), 'gcctest-nonrepo-'))
+    const result = await getStatus(new Repository(nonRepoPath))
+    expect(result).toBeNull()
+    execSync(`rm -rf ${nonRepoPath}`)
+  })
+})
+
+describe('getCommits', () => {
+  it('returns commits in order (fixture has 6 commits reachable from main)', async () => {
+    const commits = await getCommits(repo, 'HEAD', 10)
+    expect(commits.length).toBe(6)
+    expect(commits[0].sha).toBeTruthy()
+    expect(commits[0].author).toBeTruthy()
+    // The most recent commit should be the merge
+    expect(commits[0].summary).toBe('merge: feature/one into main')
+    // The initial commit should be the last one
+    expect(commits[5].summary).toBe('initial: add README and gitignore')
+  })
+
+  it('returns commits for feature/two branch', async () => {
+    const commits = await getCommits(repo, 'feature/two', 10)
+    expect(commits.length).toBeGreaterThanOrEqual(1)
+    expect(commits[0].summary).toBe('feat: feature two')
+  })
+})
+
+describe('Branch operations', () => {
+  it('includes feature/one and feature/two', async () => {
+    const branches = await getBranches(repo)
+    const names = branches.map(b => b.nameWithoutRemote)
+    expect(names).toContain('main')
+    expect(names).toContain('feature/one')
+    expect(names).toContain('feature/two')
+  })
+
+  it('creates, renames, and deletes a branch', async () => {
+    await createBranch(repo, 'test-feature', 'HEAD')
+    let branches = await getBranches(repo)
+    const names = branches.map(b => b.nameWithoutRemote)
+    expect(names).toContain('test-feature')
+
+    // Rename the branch
+    const featureBranch = branches.find(b => b.nameWithoutRemote === 'test-feature')!
+    await renameBranch(repo, featureBranch, 'test-feature-renamed')
+    branches = await getBranches(repo)
+    expect(branches.map(b => b.nameWithoutRemote)).toContain('test-feature-renamed')
+    expect(branches.map(b => b.nameWithoutRemote)).not.toContain('test-feature')
+
+    // Delete the branch
+    await deleteLocalBranch(repo, 'test-feature-renamed')
+    branches = await getBranches(repo)
+    expect(branches.map(b => b.nameWithoutRemote)).not.toContain('test-feature-renamed')
+  })
+})
+
+describe('createCommit', () => {
+  it('creates a commit with files', async () => {
+    writeFileSync(join(repoPath, 'commit-test.txt'), 'commit data')
+    git(repoPath, 'add commit-test.txt')
+
+    const status = await getStatus(repo)
+    const files = status!.workingDirectory.files.filter(
+      f => f.path === 'commit-test.txt'
+    )
+
+    const sha = await createCommit(repo, 'feat: add commit-test.txt', files)
+    expect(sha).toBeTruthy()
+    expect(sha.length).toBeGreaterThanOrEqual(7)
+
+    // Verify the commit exists
+    const commits = await getCommits(repo, 'HEAD', 5)
+    expect(commits[0].sha.startsWith(sha)).toBe(true)
+    expect(commits[0].summary).toBe('feat: add commit-test.txt')
+  })
+})
+
+describe('Tag operations', () => {
+  it('lists tags from the fixture', async () => {
+    const tags = await getAllTags(repo)
+    expect(tags.size).toBeGreaterThanOrEqual(1)
+    expect(tags.has('v0.1.0')).toBe(true)
+  })
+})
+
+afterAll(() => {
+  cleanupFixtureRepo(repoPath)
+})

package/docs/skills/release.md

@@ -1,81 +0,0 @@
----
-name: release
-description: Automated release workflow for git-chopstick-core — version bump, changelog, build, validation, and npm publish.
----
-
-# Release: git-chopstick-core
-
-> Loaded via `skill("release")`. Run this whenever you need to publish a new release.
-
-## Process
-
-### 1. Read current state
-
-Read `package.json`, `CHANGELOG.md`, and run `git log --oneline -5` to understand what has changed since the last release.
-
-### 2. Determine version bump
-
-Check what has changed:
-
-- **Breaking API changes** (removed exports, changed signatures) → minor bump (0.x.0)
-- **New features** (new functions, new exports) → patch bump (0.0.x)
-- **Bug fixes, docs, internal refactoring** → patch bump (0.0.x)
-
-Current version: read from `package.json`. Bump accordingly.
-
-### 3. Update `CHANGELOG.md`
-
-Insert a new `## [VERSION] — YYYY-MM-DD` section at the top (below the `# Changelog` heading) with:
-
-- `### Added` — new features, exports, helpers
-- `### Changed` — behavioral changes, renames, config changes
-- `### Fixed` — bug fixes
-
-Then update the link references at the bottom:
-
-- Add `[VERSION]: https://github.com/parkiyong/git-chopstick-core/releases/tag/vVERSION`
-- Update `[Unreleased]: .../compare/vVERSION...HEAD`
-
-### 4. Bump `package.json`
-
-Update `"version"` field.
-
-### 5. Build and validate
-
-Run all of these (use parallel agents):
-
-- `npm run typecheck` — must pass with zero errors
-- `npm run build` — must produce a clean `dist/`
-- `npm test` — all integration tests must pass
-- `npm pack --dry-run` — confirm all of:
-  - `dist/` is included (expect ~351 files, ~700kB)
-  - `CHANGELOG.md` and `docs/` appear in the file list
-  - `package.json`, `README.md`, `LICENSE` are present (npm includes these automatically)
-  - No unexpected files are leaking through (check for test artifacts, `.ts` sources outside `dist/`)
-
-If any fail, fix before proceeding.
-
-### 6. Stage and commit
-
-```
-git add -A
-git commit -m "vVERSION: <short summary of changes>"
-```
-
-### 7. Push and publish
-
-```
-git push origin main
-npm publish
-```
-
-### 8. Verify
-
-- Confirm npm shows the new version: `npm view git-chopstick-core version`
-- Confirm the tag exists on GitHub: `git tag -l`
-- Suggest followup tasks
-
-## Notes
-
-- `prepublishOnly` runs `npm run typecheck && npm run build` automatically, so no need to build before publish — but validate first anyway.
-- If this is the first release in a session, run `setup-matt-pocock-skills` first to configure issue tracker context.