Skip to content

Comments

fix: add missing .js extensions to relative ESM imports#389

Merged
coffeexcoin merged 1 commit intoAbstract-Foundation:mainfrom
bdphilly:fix/esm-missing-js-extensions
Feb 12, 2026
Merged

fix: add missing .js extensions to relative ESM imports#389
coffeexcoin merged 1 commit intoAbstract-Foundation:mainfrom
bdphilly:fix/esm-missing-js-extensions

Conversation

@bdphilly
Copy link
Contributor

@bdphilly bdphilly commented Feb 12, 2026

Several relative imports in agw-react and agw-client are missing the .js file extension, which breaks ESM module resolution in strict package managers like pnpm.

Files fixed:

- agw-react: src/query/createSession.ts
- agw-client: src/clients/decorators/linkablePublic.ts
- agw-client: src/clients/decorators/linkableWallet.ts

PR-Codex overview

This PR focuses on updating import statements to include the .js file extension for action imports across multiple files, ensuring consistency and potentially improving module resolution.

Detailed summary

  • Updated import paths to include .js extension in createSession.ts.
  • Updated import paths to include .js extension in linkableWallet.ts.
  • Updated import paths to include .js extension in linkablePublic.ts.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@cursor
Copy link

cursor bot commented Feb 12, 2026

PR Summary

Low Risk
Import-path-only changes; low risk aside from potential build/runtime issues if any referenced files or bundler settings don’t match the .js extension expectation.

Overview
Fixes ESM module resolution by adding missing .js extensions to several relative action imports in agw-client (linkablePublic.ts, linkableWallet.ts) and agw-react (createSession.ts).

This is a non-functional change intended to improve compatibility with strict ESM/package managers (e.g., pnpm) without altering runtime behavior.

Written by Cursor Bugbot for commit 7cd1aaa. This will update automatically on new commits. Configure here.

@changeset-bot
Copy link

changeset-bot bot commented Feb 12, 2026

⚠️ No Changeset found

Latest commit: 7cd1aaa

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@bdphilly bdphilly force-pushed the fix/esm-missing-js-extensions branch from 24a2cce to 3fb72d5 Compare February 12, 2026 18:36
@bdphilly bdphilly force-pushed the fix/esm-missing-js-extensions branch from 3fb72d5 to 7cd1aaa Compare February 12, 2026 18:37
@coffeexcoin coffeexcoin merged commit 721d5b7 into Abstract-Foundation:main Feb 12, 2026
5 checks passed
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.

2 participants