Skip to content

Commit

Permalink
Fix next JSR export (#743)
Browse files Browse the repository at this point in the history
## Summary
<!-- Succinctly describe your change, providing context, what you've
changed, and why. -->

Fixes an annoying release bug, with JSR failing for `inngest`.
See
https://github.com/inngest/inngest-js/actions/runs/11741974959/job/32711752350

We don't usually require extensions for libraries because we expect
packages to also have ESM exports declared in their `package.json`, but
`next` does not do this, so we must explicitly use the extension still.

Annoyingly, a dry run publish with `jsr` doesn't reveal this. 🙄

## Checklist
<!-- Tick these items off as you progress. -->
<!-- If an item isn't applicable, ideally please strikeout the item by
wrapping it in "~~"" and suffix it with "N/A My reason for skipping
this." -->
<!-- e.g. "- [ ] ~~Added tests~~ N/A Only touches docs" -->

- [ ] ~Added a [docs PR](https://github.com/inngest/website) that
references this PR~ N/A CI fix
- [ ] ~Added unit/integration tests~ N/A CI fix
- [x] Added changesets if applicable
  • Loading branch information
jpwilliams authored Nov 12, 2024
1 parent 6782497 commit 6f0352d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion packages/inngest/src/next.ts
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@
*/

import { type NextApiRequest, type NextApiResponse } from "next";
import { type NextRequest } from "next/server";
import { type NextRequest } from "next/server.js";
import {
InngestCommHandler,
type ServeHandlerOptions,
Expand Down

0 comments on commit 6f0352d

Please sign in to comment.