fix(linear,slack): point exports to dist/, add ./path-mapper subpath#20
fix(linear,slack): point exports to dist/, add ./path-mapper subpath#20khaliqgant merged 1 commit intomainfrom
Conversation
Same fix as adapter-github PR #19: - exports field now points to dist/*.js instead of src/*.ts - Added ./path-mapper subpath export for zero-dep path computation - files field changed to ["dist"] so compiled output is published Fixes Turbopack "Unknown module type" errors in Next.js consumers that can't handle raw .ts source files from node_modules. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5e3295fab0
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| "files": [ | ||
| "src" | ||
| "dist" | ||
| ], |
There was a problem hiding this comment.
Publishing only dist here makes git/file installs fragile because this package has no prepare script to generate dist before packing. prepublishOnly runs for npm publish, but it does not run when users install directly from a Git URL or local path, so the tarball can end up with just package.json and imports then fail at runtime with missing ./dist/*.js files.
Useful? React with 👍 / 👎.
| "files": [ | ||
| "src", | ||
| "tsconfig.json" | ||
| "dist" | ||
| ], |
There was a problem hiding this comment.
This change has the same packaging regression as linear: with files restricted to dist and no prepare hook, installs from Git/local sources do not build artifacts before pack time. In those environments prepublishOnly is skipped, so consumers can receive a package without dist/index.js and hit module resolution failures immediately.
Useful? React with 👍 / 👎.
1 participant
Summary
Same fix as #19 (adapter-github), applied to adapter-linear and adapter-slack:
dist/*.jsinstead ofsrc/*.ts— Turbopack cannot handle raw.tsfiles fromnode_modules./path-mappersubpath export for zero-dependency path computation functions["dist"]so compiled output is published (not raw source)Both
path-mapper.tsmodules are fully self-contained with zero external imports.Test plan
npm run buildsucceeds for both packages🤖 Generated with Claude Code