Skip to content

Fix hooks package type export paths#1341

Open
Gujiassh wants to merge 1 commit intoruvnet:mainfrom
Gujiassh:fix/hooks-types-export-paths
Open

Fix hooks package type export paths#1341
Gujiassh wants to merge 1 commit intoruvnet:mainfrom
Gujiassh:fix/hooks-types-export-paths

Conversation

@Gujiassh
Copy link

Summary

  • replace the broken .d.js subpath types entries in @claude-flow/hooks/package.json with the .d.ts files the package actually emits
  • keep the fix scoped to package metadata; no runtime hook logic changes
  • ensure downstream TypeScript consumers can resolve hook package subpath declarations consistently

Validation

  • cd v3/@claude-flow/hooks && npm run build
  • static post-build check confirmed every exports[*].types target now exists on disk

Notes

  • this PR is independent from the open plugins, performance, and OpenSandbox lines; it only fixes package metadata in @claude-flow/hooks

Point the package exports at the  files that the build actually produces so downstream TypeScript consumers can resolve subpath types correctly.

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-opencode)

Co-authored-by: Sisyphus <[email protected]>
Copy link

@xkonjin xkonjin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Straightforward typo fix. The .d.js extension is clearly wrong - TypeScript declaration files should use .d.ts.

This is a critical fix because broken type exports would cause TypeScript consumers to fall back to any types, negating the type safety benefits of using this package.

No issues found. All export paths are correctly updated.

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.

2 participants