Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix #334: support automatic JSX runtime #2349

Merged
merged 24 commits into from
Jul 28, 2022
Merged

fix #334: support automatic JSX runtime #2349

Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
90db88d
Add new JSX options to API
jgoz Jun 24, 2022
34a0667
Resolve applicable jsx options from tsconfig
jgoz Jun 24, 2022
9d81ee7
Default options, parse pragmas
jgoz Jun 24, 2022
9e00cd7
Accept nil sourceIndex for external imports
jgoz Jun 24, 2022
2a6495a
Parsing and import generation
jgoz Jun 24, 2022
032c9e2
tsconfig tests
jgoz Jun 24, 2022
fe60076
JS API tests
jgoz Jun 24, 2022
8e4a46c
Parser tests
jgoz Jun 24, 2022
ff5ec74
Add sourcemap tests
jgoz Jun 24, 2022
7934001
Fix handling of __self and __source
jgoz Jun 25, 2022
3676e5b
Add namespace tests
jgoz Jun 25, 2022
428cdb1
Clean up tests
jgoz Jun 25, 2022
add10ea
Remove unused JSX consts
jgoz Jun 25, 2022
f740295
Merge branch 'master' into jsx-runtime
jgoz Jul 23, 2022
1f9ddc9
Set matchAgainstDefines: true for imported JSX symbols
jgoz Jul 23, 2022
f430a16
Merge branch 'master' into jsx-runtime
evanw Jul 28, 2022
8ae6058
Some fixes for jsx source positions
evanw Jul 28, 2022
7e6c025
Add missing source map positions
evanw Jul 28, 2022
033eaa1
Move imports to top to avoid bugs in other tools
evanw Jul 28, 2022
431db4f
Do not attempt to rewrite JSX symbols with define
evanw Jul 28, 2022
1cdd684
Update error messages to match esbuild's style
evanw Jul 28, 2022
2f63726
Merge `--jsx-runtime` into `--jsx`
evanw Jul 28, 2022
93f9ea4
Add something to the release notes
evanw Jul 28, 2022
c20f3c2
The automatic runtime implicitly sets strict mode
evanw Jul 28, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 2 additions & 5 deletions internal/js_parser/js_parser.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1573,7 +1573,6 @@ func (p *parser) importJSXSymbol(loc logger.Loc, jsx JSXImport) js_ast.Expr {
p.recordUsage(ref)
return p.handleIdentifier(loc, &js_ast.EIdentifier{Ref: ref}, identifierOpts{
wasOriginallyIdentifier: true,
matchAgainstDefines: true, // Allow defines to rewrite imported JSX symbols
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Makes sense, I was clearly missing context here.

})
}

Expand Down Expand Up @@ -12245,8 +12244,8 @@ func (p *parser) visitExprInOut(expr js_ast.Expr, in exprIn) (js_ast.Expr, exprO
wasOriginallyIdentifier: true,
matchAgainstDefines: true, // Allow defines to rewrite the JSX factory
})
p.warnAboutImportNamespaceCall(target, exprKindCall)
}
p.warnAboutImportNamespaceCall(target, exprKindCall)
return js_ast.Expr{Loc: expr.Loc, Data: &js_ast.ECall{
Target: target,
Args: args,
Expand Down Expand Up @@ -12371,10 +12370,8 @@ func (p *parser) visitExprInOut(expr js_ast.Expr, in exprIn) (js_ast.Expr, exprO
jsx = JSXImportJSXS
}

target := p.importJSXSymbol(expr.Loc, jsx)
p.warnAboutImportNamespaceCall(target, exprKindCall)
return js_ast.Expr{Loc: expr.Loc, Data: &js_ast.ECall{
Target: target,
Target: p.importJSXSymbol(expr.Loc, jsx),
Args: args,
CloseParenLoc: e.CloseLoc,

Expand Down