Skip to content

refactor(tools): use .ts import extensions#1274

Merged
graphite-app[bot] merged 1 commit intomainfrom
support-tools-as-ts-package
Apr 3, 2026
Merged

refactor(tools): use .ts import extensions#1274
graphite-app[bot] merged 1 commit intomainfrom
support-tools-as-ts-package

Conversation

@fengmk2
Copy link
Copy Markdown
Member

@fengmk2 fengmk2 commented Apr 2, 2026

Switch tools package imports from .js to .ts extensions, update entry
point from bin.js to index.ts, and enable allowImportingTsExtensions in
tsconfig. Also fix potential null reference on wasiBindingError in the
NAPI binding loader.

#1252 (comment)

@netlify
Copy link
Copy Markdown

netlify bot commented Apr 2, 2026

Deploy Preview for viteplus-preview canceled.

Name Link
🔨 Latest commit ea02a9c
🔍 Latest deploy log https://app.netlify.com/projects/viteplus-preview/deploys/69cf173417798500097f1e50

@fengmk2 fengmk2 self-assigned this Apr 2, 2026
@fengmk2 fengmk2 marked this pull request as ready for review April 2, 2026 13:27
Copy link
Copy Markdown
Member Author

fengmk2 commented Apr 2, 2026


How to use the Graphite Merge Queue

Add the label auto-merge to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@fengmk2 fengmk2 requested review from branchseer and cpojer April 2, 2026 13:27
@fengmk2 fengmk2 changed the title refactor(tools): use .ts import extensions and fix NAPI binding null reference refactor(tools): use .ts import extensions Apr 2, 2026
@graphite-app
Copy link
Copy Markdown

graphite-app bot commented Apr 3, 2026

Merge activity

Switch tools package imports from .js to .ts extensions, update entry
point from bin.js to index.ts, and enable allowImportingTsExtensions in
tsconfig. Also fix potential null reference on wasiBindingError in the
NAPI binding loader.

#1252 (comment)
@graphite-app graphite-app bot force-pushed the support-tools-as-ts-package branch from c5fdcf9 to ea02a9c Compare April 3, 2026 01:26
@graphite-app graphite-app bot merged commit ea02a9c into main Apr 3, 2026
33 checks passed
@graphite-app graphite-app bot deleted the support-tools-as-ts-package branch April 3, 2026 01:32
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