Skip to content
Closed
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
33 commits
Select commit Hold shift + click to select a range
0b6b314
test: split test cases out to share with ts
Jack-Works Sep 26, 2020
ebec748
feat: impl component finding
Jack-Works Sep 26, 2020
b0e83b4
feat: able to track hooks calls
Jack-Works Sep 26, 2020
78c86d1
fix: use outdated node
Jack-Works Sep 26, 2020
1b1b416
feat: able to insert signature calls
Jack-Works Sep 26, 2020
1d5a156
chore: use paran
Jack-Works Sep 26, 2020
8a0ab66
fix: also wrap hoc fns
Jack-Works Sep 26, 2020
09f5b14
fix: rest cases
Jack-Works Sep 26, 2020
5f34534
chore: remove unused vars
Jack-Works Sep 26, 2020
218b0d4
feat: generate signature for builtin hooks
Jack-Works Sep 27, 2020
182e81c
feat: track custom hooks
Jack-Works Sep 27, 2020
8caf2f2
test: add test cases
Jack-Works Sep 27, 2020
eae20b6
fix: aliased improt not tracked
Jack-Works Sep 27, 2020
a88eb33
chore: remove extra false param
Jack-Works Sep 27, 2020
0553d1b
fix: export default hooks
Jack-Works Sep 27, 2020
8139cc7
feat: add emit hash
Jack-Works Sep 27, 2020
d020fa2
chore: make it more pure
Jack-Works Sep 27, 2020
a827f0a
refactor
Jack-Works Sep 27, 2020
91d280e
refactor
Jack-Works Sep 27, 2020
caf3aa2
refactor
Jack-Works Sep 27, 2020
fe8db84
refactor
Jack-Works Sep 27, 2020
775abee
test: add a new case
Jack-Works Sep 27, 2020
792a7f7
chore: move ts into opts
Jack-Works Sep 28, 2020
4ae646b
fix: break when downgrading ?? and ?. syntax
Jack-Works Sep 29, 2020
aed1d14
chore: update snapshot
Jack-Works Sep 29, 2020
4fb4f52
fix: some runtime error
Jack-Works Sep 29, 2020
ec6180d
fix: some runtime error
Jack-Works Sep 29, 2020
abfb1be
use ts 4.0 factory API
Jack-Works Sep 28, 2020
fcdbb2d
use create temp var
Jack-Works Sep 28, 2020
dea9ef4
Merge branch 'master' into ts-react-refresh
Jack-Works Oct 5, 2020
8b9d686
throw on ts v3
Jack-Works Oct 5, 2020
1e1e12f
chore: add a .d.ts file for /typescript entry
Jack-Works Oct 24, 2020
8448fe2
chore: add a .d.ts file for /typescript entry
Jack-Works Oct 24, 2020
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
Prev Previous commit
Next Next commit
chore: remove extra false param
  • Loading branch information
Jack-Works committed Sep 27, 2020
commit a88eb33024f257bd57db7d4dc7921d494ac1df6c
9 changes: 6 additions & 3 deletions packages/react-refresh/src/ReactFreshTypeScriptTransformer.js
Original file line number Diff line number Diff line change
Expand Up @@ -259,9 +259,12 @@ export default function(opts = {}, ts = require('typescript')) {
const {force: forceRefresh, hooks: hooksArray} = needForceRefresh(
hooksCalls,
);
const requireForceRefreshExpr = ts.createLiteral(
forceRefresh || globalRequireForceRefresh,
);
const requireForceRefresh =
forceRefresh || globalRequireForceRefresh;
const requireForceRefreshExpr =
requireForceRefresh || hooksArray.length
? ts.createLiteral(requireForceRefresh)
: undefined;
const hooksSignatureExpr = ts.createNoSubstitutionTemplateLiteral(
hooksSignature,
hooksSignature,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,14 +10,14 @@ _j = $RefreshSig$();
_k = $RefreshSig$();
_l = $RefreshSig$();
export default () => (_f(), useContext(X));
export const Foo = _g(() => (_g(), useContext(X)), \`useContext{}\`, false);
export const Foo = _g(() => (_g(), useContext(X)), \`useContext{}\`);
_a = Foo;
module.exports = () => (_h(), useContext(X));
const Bar = _j(() => (_j(), useContext(X)), \`useContext{}\`, false);
const Bar = _j(() => (_j(), useContext(X)), \`useContext{}\`);
_b = Bar;
const Baz = memo(_c = _k(() => (_k(), useContext(X)), \`useContext{}\`, false));
const Baz = memo(_c = _k(() => (_k(), useContext(X)), \`useContext{}\`));
_d = Baz;
const Qux = _l(() => (_l(), (0, useContext(X))), \`useContext{}\`, false);
const Qux = _l(() => (_l(), (0, useContext(X))), \`useContext{}\`);
_e = Qux;
$RefreshReg$(_a, "Foo");
$RefreshReg$(_b, "Bar");
Expand Down Expand Up @@ -58,7 +58,7 @@ export default function App() {
}
_a = App;
_b(App, \`useState{[foo, setFoo](0)}
useEffect{}\`, false);
useEffect{}\`);
$RefreshReg$(_a, "App");

`;
Expand All @@ -74,15 +74,15 @@ export const A = React.memo(_a = React.forwardRef(_b = _g((props, ref) => {
React.useEffect(() => { });
return <h1 ref={ref}>{foo}</h1>;
}, \`useState{[foo, setFoo](0)}
useEffect{}\`, false)));
useEffect{}\`)));
_c = A;
export const B = React.memo(_d = React.forwardRef(_e = _h(function (props, ref) {
_h();
const [foo, setFoo] = useState(0);
React.useEffect(() => { });
return <h1 ref={ref}>{foo}</h1>;
}, \`useState{[foo, setFoo](0)}
useEffect{}\`, false)));
useEffect{}\`)));
_f = B;
function hoc() {
var _a;
Expand All @@ -93,7 +93,7 @@ function hoc() {
React.useEffect(() => { });
return <h1 ref={ref}>{foo}</h1>;
}, \`useState{[foo, setFoo](0)}
useEffect{}\`, false);
useEffect{}\`);
}
export let C = hoc();
$RefreshReg$(_b, "A$React.memo$React.forwardRef");
Expand Down Expand Up @@ -209,7 +209,7 @@ useFancyEffect{}\`, false, () => [useFancyEffect]);
const useFancyEffect = _c(() => {
_c();
React.useEffect(() => { });
}, \`useEffect{}\`, false);
}, \`useEffect{}\`);
export default function App() {
_d();
const bar = useFancyState();
Expand Down Expand Up @@ -517,7 +517,7 @@ useState{{ 0: y, 1: z, length }(() => {
multiple();
line();
expression();
})}\`, false);
})}\`);
$RefreshReg$(_a, "StateTest");

`;
Expand All @@ -535,7 +535,7 @@ export function RefTest() {
_a = RefTest;
_b(RefTest, \`useRef{ref}
useRef{ref2}
useRef{{ current }}\`, false);
useRef{{ current }}\`);
$RefreshReg$(_a, "RefTest");

`;
Expand All @@ -557,7 +557,7 @@ _a = EffectTest;
_b(EffectTest, \`useEffect{rtn}
useEffect{}
useEffect{}
useEffect{}\`, false);
useEffect{}\`);
$RefreshReg$(_a, "EffectTest");

`;
Expand All @@ -575,7 +575,7 @@ export function ContextTest() {
_a = ContextTest;
_b(ContextTest, \`useContext{ctx}
useContext{{ val }}
useContext{}\`, false);
useContext{}\`);
$RefreshReg$(_a, "ContextTest");

`;
Expand All @@ -591,7 +591,7 @@ export function ReducerTest() {
}
_a = ReducerTest;
_b(ReducerTest, \`useReducer{[state, dispatch](initArg)}
useReducer{}\`, false);
useReducer{}\`);
$RefreshReg$(_a, "ReducerTest");

`;
Expand All @@ -607,7 +607,7 @@ export function CallbackTest() {
}
_a = CallbackTest;
_b(CallbackTest, \`useCallback{x}
useCallback{}\`, false);
useCallback{}\`);
$RefreshReg$(_a, "CallbackTest");

`;
Expand All @@ -623,7 +623,7 @@ export function MemoTest() {
}
_a = MemoTest;
_b(MemoTest, \`useMemo{x}
useMemo{}\`, false);
useMemo{}\`);
$RefreshReg$(_a, "MemoTest");

`;
Expand All @@ -638,12 +638,12 @@ function ImperativeHandle(props, ref) {
const v = useImperativeHandle(ref, () => ({ a }));
}
_a = ImperativeHandle;
_d(ImperativeHandle, \`useImperativeHandle{v}\`, false);
_d(ImperativeHandle, \`useImperativeHandle{v}\`);
ImperativeHandle = forwardRef(ImperativeHandle);
const HOC = forwardRef(_b = _e(function (props, ref) {
_e();
const v = useImperativeHandle(ref, () => ({ a }));
}, \`useImperativeHandle{v}\`, false));
}, \`useImperativeHandle{v}\`));
_c = HOC;
$RefreshReg$(_a, "ImperativeHandle");
$RefreshReg$(_b, "HOC$forwardRef");
Expand All @@ -657,7 +657,7 @@ var _b;
_b = $RefreshSig$();
function LayoutEffect() { _b(); const v = useLayoutEffect(ref); }
_a = LayoutEffect;
_b(LayoutEffect, \`useLayoutEffect{v}\`, false);
_b(LayoutEffect, \`useLayoutEffect{v}\`);
$RefreshReg$(_a, "LayoutEffect");

`;
Expand All @@ -668,7 +668,7 @@ var _b;
_b = $RefreshSig$();
function DebugValue() { _b(); const v = useDebugValue(ref, f); }
_a = DebugValue;
_b(DebugValue, \`useDebugValue{v}\`, false);
_b(DebugValue, \`useDebugValue{v}\`);
$RefreshReg$(_a, "DebugValue");

`;
Expand Down Expand Up @@ -716,7 +716,7 @@ _b = $RefreshSig$();
_c = $RefreshSig$();
_d = $RefreshSig$();
function useLocal() { _b(); return useState(0); }
_b(useLocal, \`useState{(0)}\`, false);
_b(useLocal, \`useState{(0)}\`);
const useLocal2 = _c(() => (_c(), useLocal()), \`useLocal{}\`, false, () => [useLocal]);
function App() { _d(); useLocal(useLocal2()); }
_a = App;
Expand All @@ -736,7 +736,7 @@ function App() {
_a = $RefreshSig$();
_b = $RefreshSig$();
function useLocal() { _a(); return useState(0); }
_a(useLocal, \`useState{(0)}\`, false);
_a(useLocal, \`useState{(0)}\`);
const useLocal2 = _b(() => (_b(), useLocal()), \`useLocal{}\`, false, () => [useLocal]);
useLocal(useLocal2());
}
Expand Down Expand Up @@ -771,7 +771,7 @@ export default function Bar() {
return <Foo />;
}
_a = Bar;
_b(Bar, \`useContext{}\`, false);
_b(Bar, \`useContext{}\`);
;
import.meta.refreshReg(_a, "Bar");

Expand Down