Skip to content

fix: add missing index.ts barrel file and fix package entry point#11

Closed
TonyCasey wants to merge 1 commit intomainfrom
tony/git-32-fix-missing-distindexjs-in-npm-package
Closed

fix: add missing index.ts barrel file and fix package entry point#11
TonyCasey wants to merge 1 commit intomainfrom
tony/git-32-fix-missing-distindexjs-in-npm-package

Conversation

@TonyCasey
Copy link
Copy Markdown
Owner

@TonyCasey TonyCasey commented Feb 6, 2026

Summary

  • Created src/index.ts barrel file exporting the full public API (domain types, interfaces, services, infrastructure, errors, MCP server factory)
  • Fixed package.json: maindist/index.js, added typesdist/index.d.ts
  • Added repository, homepage, bugs metadata fields
  • Bumped version to 0.1.1

Fixes GIT-32

Test plan

  • npm run build compiles cleanly
  • npm test — all 135 tests pass
  • npm pack --dry-run confirms dist/index.js is included
  • Published git-mem@0.1.1 to npm and verified with npm info git-mem

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Bumped version to 0.1.1.
    • Updated package configuration with new main entry point and public repository metadata.
    • Added TypeScript type definitions entry for improved IDE support.

src/index.ts never existed — consumers importing from 'git-mem' got a
broken module. Created barrel file exporting the full public API,
pointed main to dist/index.js, added types field, bumped to 0.1.1.

Fixes GIT-32

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 6, 2026

📝 Walkthrough

Walkthrough

The changes create a public API barrel entry point and update package configuration. Package.json bumps to version 0.1.1, points the main entry to dist/index.js, and adds type definitions. A new src/index.ts barrel file re-exports 100+ entities from domain, application, and infrastructure layers.

Changes

Cohort / File(s) Summary
Package Configuration
package.json
Version bumped to 0.1.1; main entry changed from dist/cli.js to dist/index.js; added TypeScript types entry (dist/index.d.ts); added repository, homepage, and bugs metadata fields.
API Barrel Export
src/index.ts
New comprehensive barrel entry point re-exporting domain entities, types, interfaces, errors, services, repositories, utilities, and MCP server components from all layers (domain, application, infrastructure).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main changes: adding a missing index.ts barrel file and fixing the package entry point in package.json.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch tony/git-32-fix-missing-distindexjs-in-npm-package

Comment @coderabbitai help to get the list of available commands and usage tips.

@TonyCasey
Copy link
Copy Markdown
Owner Author

Closing — this work was already delivered via PR #17 (branch git-32) which is merged to main.

@TonyCasey TonyCasey closed this Feb 11, 2026
@TonyCasey TonyCasey deleted the tony/git-32-fix-missing-distindexjs-in-npm-package branch February 11, 2026 13:16
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