Skip to content

docs: update examples with webhook-server + writeback#5

Closed
khaliqgant wants to merge 0 commit intomainfrom
feat/update-examples
Closed

docs: update examples with webhook-server + writeback#5
khaliqgant wants to merge 0 commit intomainfrom
feat/update-examples

Conversation

@khaliqgant
Copy link
Copy Markdown
Member

@khaliqgant khaliqgant commented Mar 29, 2026

Flagship example uses webhook-server + WritebackConsumer for cleanest e2e.


Open with Devin

devin-ai-integration[bot]

This comment was marked as resolved.

@khaliqgant khaliqgant force-pushed the feat/update-examples branch from 2d1cac2 to d37047d Compare March 29, 2026 16:59
devin-ai-integration[bot]

This comment was marked as resolved.

@khaliqgant khaliqgant force-pushed the feat/update-examples branch from d37047d to 79a64e7 Compare March 29, 2026 17:42
devin-ai-integration[bot]

This comment was marked as resolved.

devin-ai-integration[bot]

This comment was marked as resolved.

@khaliqgant khaliqgant force-pushed the feat/update-examples branch 2 times, most recently from 4952983 to 630a41d Compare March 29, 2026 19:05
Copy link
Copy Markdown

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

Devin Review found 1 new potential issue.

View 11 additional findings in Devin Review.

Open in Devin Review

"module": "NodeNext",
"moduleResolution": "NodeNext",
"strict": true,
"noEmit": true,
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 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.
Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

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