Skip to content

fix: add types field to package.json#64

Merged
erikras merged 1 commit intomasterfrom
fix/add-types-field
Feb 11, 2026
Merged

fix: add types field to package.json#64
erikras merged 1 commit intomasterfrom
fix/add-types-field

Conversation

@erikras-gilfoyle-agent
Copy link
Copy Markdown
Contributor

@erikras-gilfoyle-agent erikras-gilfoyle-agent commented Feb 11, 2026

TypeScript types are generated and published to dist/index.d.ts but package.json has "types": null, preventing TypeScript from finding them.

This adds the missing types field to package.json.

Fixes #47

Summary by CodeRabbit

  • Chores
    • Enhanced TypeScript support with improved type declaration configuration for better IDE integration and type checking.

TypeScript types are generated and published to dist/index.d.ts but
package.json has "types": null, preventing TypeScript from finding them.

Fixes #47
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 11, 2026

📝 Walkthrough

Walkthrough

Added the "types" manifest field to package.json, pointing to the TypeScript type declaration file at dist/index.d.ts. This ensures TypeScript projects can properly resolve the library's type definitions when consuming the package from npm.

Changes

Cohort / File(s) Summary
Package Configuration
package.json
Added "types" field to explicitly declare TypeScript type declaration file location, ensuring proper type resolution in consuming projects.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A hop and a skip, the types are in place,
No more TypeScript errors we need to erase,
The manifest now knows where to find declarations true,
Your IDE's now happy—and you are too!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 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 change: adding a types field to package.json to resolve TypeScript type declaration issues.
Linked Issues check ✅ Passed The PR directly addresses issue #47 by adding the missing 'types' field to package.json, enabling TypeScript to locate the generated declaration files in dist/index.d.ts.
Out of Scope Changes check ✅ Passed The PR only modifies package.json to add the types field; all changes are directly related to resolving the linked issue with no out-of-scope modifications.
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 unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/add-types-field

No actionable comments were generated in the recent review. 🎉


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

@erikras erikras merged commit 0ad6360 into master Feb 11, 2026
4 checks passed
@erikras erikras deleted the fix/add-types-field branch February 11, 2026 07:23
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.

Version 1.3.2 lacks the type hint updates

2 participants