hyperweb-io / hyperweb-io/telescope

Should add `type` to type-only export by default

Open
#838 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
TypeScript
Stars
154
Forks
53
PR merge metrics
No merged PRs in 30d

Description

related to #678

type-only export also needs `type` keyword. Current output from telescope mixes normal import/export and type-only import/export.

This is not compatible with `verbatimModuleSyntax`, which helps bundlers to drop type-only import/export while transpiling.

tsc (or tsc-based tools ex. tshy, zshy) would not have any problem with mixing import/export since tsc has type info while walking AST but the modern bundlers (esbuild, Oxc, ...) don't. They run tsc only as a type checker, then strip type for transpiling, leaving type-only import/export on run-time js source.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.