Skip to content

fix: update for more recent deno versions#44

Merged
tmcw merged 1 commit into
val-town:mainfrom
stainless-api:main
Aug 12, 2025
Merged

fix: update for more recent deno versions#44
tmcw merged 1 commit into
val-town:mainfrom
stainless-api:main

Conversation

@stainless-em

Copy link
Copy Markdown
Contributor

Fixes error: [ERR_UNSUPPORTED_NODE_MODULES_TYPE_STRIPPING]: Stripping types is currently unsupported for files under node_modules, for "file:///[...]/node_modules/.pnpm/@valtown+deno-http-worker@0.0.20/node_modules/@valtown/deno-http-worker/deno-bootstrap/index.ts" and makes the tests pass under newer deno versions.

Comment thread src/DenoHTTPWorker.ts
"run",
..._options.runFlags,
_options.denoBootstrapScriptPath,
"data:text/typescript," + encodeURIComponent(bootstrap),

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is this the relevant changed line?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

mhm

@tmcw tmcw merged commit b193b22 into val-town:main Aug 12, 2025
Comment thread deno-bootstrap/index.ts
Comment on lines +67 to +70
addEventListener("unhandledrejection", (e) => {
console.error(e.reason);
e.preventDefault();
});

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

fyi @stainless-em we ended up removing this in a recent version. we rely on this throwing in val.town. hopefully it's easy enough to add-back downstream

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.

3 participants