docs: update examples with webhook-server + writeback#5
docs: update examples with webhook-server + writeback#5khaliqgant wants to merge 0 commit intomainfrom
Conversation
2d1cac2 to
d37047d
Compare
d37047d to
79a64e7
Compare
4952983 to
630a41d
Compare
| "module": "NodeNext", | ||
| "moduleResolution": "NodeNext", | ||
| "strict": true, | ||
| "noEmit": true, |
There was a problem hiding this comment.
🟡 webhook-server tsconfig noEmit: true prevents build from producing output files
The webhook-server's tsconfig.json at line 10 sets "noEmit": true, which means running the build script ("build": "tsc" in packages/webhook-server/package.json:18) will only type-check but never emit JavaScript files into dist/. However, packages/webhook-server/package.json:7 declares "main": "dist/index.js", which will never exist. Any consumer resolving the package via the main field (e.g., CommonJS require(), or bundlers that don't support exports) will get a MODULE_NOT_FOUND error. The exports field (packages/webhook-server/package.json:10-11) correctly points to ./src/index.ts, so modern ESM consumers work, but main is broken.
Prompt for agents
In packages/webhook-server/tsconfig.json, either:
1. Remove `"noEmit": true` (line 10) so that `tsc` actually emits JS files to dist/, making `main: "dist/index.js"` work. OR
2. If the intent is source-only distribution (like adapter-linear and adapter-slack), remove the `main: "dist/index.js"` field from packages/webhook-server/package.json (line 7) since it will never exist, and ensure the `exports` field is sufficient for all consumers.
Was this helpful? React with 👍 or 👎 to provide feedback.
690e0ae to
0cb8ad3
Compare
0cb8ad3 to
58cd1ca
Compare
Flagship example uses webhook-server + WritebackConsumer for cleanest e2e.