[1/2] [@graphiql/react] compile source code with react-compiler, remove useMemo and useCallback usages

[dimaMachina]

No description provided.

[changeset-bot]

🦋 Changeset detected

Latest commit: 24c8c1e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages

Name	Type
@graphiql/react	Minor
@graphiql/plugin-code-exporter	Major
@graphiql/plugin-explorer	Major
graphiql	Patch

Not sure what this means? Click here to learn what changesets are.

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

[codecov]

Codecov Report

Attention: Patch coverage is 23.82671% with 211 lines in your changes missing coverage. Please review.

Project coverage is 58.58%. Comparing base (9f725b8) to head (24c8c1e).
Report is 6 commits behind head on main.

Files with missing lines	Patch %	Lines
packages/graphiql-react/src/schema.tsx	0.00%	64 Missing ⚠️
packages/graphiql-react/src/editor/query-editor.ts	3.03%	32 Missing ⚠️
packages/graphiql-react/src/editor/context.tsx	0.00%	30 Missing ⚠️
packages/graphiql-react/src/editor/hooks.ts	19.23%	21 Missing ⚠️
.../graphiql-react/src/explorer/components/search.tsx	70.00%	14 Missing and 1 partial ⚠️
packages/graphiql-react/src/history/context.tsx	14.28%	12 Missing ⚠️
packages/graphiql-react/src/execution.tsx	0.00%	11 Missing ⚠️
...kages/graphiql-react/src/editor/variable-editor.ts	12.50%	7 Missing ⚠️
...ackages/graphiql-react/src/editor/header-editor.ts	14.28%	6 Missing ⚠️
...ages/graphiql-react/src/editor/response-editor.tsx	16.66%	5 Missing ⚠️
... and 3 more

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3821      +/-   ##
==========================================
- Coverage   65.32%   58.58%   -6.74%     
==========================================
  Files         122      109      -13     
  Lines        7003     6112     -891     
  Branches     2260     1923     -337     
==========================================
- Hits         4575     3581     -994     
- Misses       2411     2515     +104     
+ Partials       17       16       -1     

Files with missing lines	Coverage Δ
packages/graphiql-react/src/editor/common.ts	100.00% <ø> (ø)
packages/graphiql-react/src/editor/completion.ts	0.00% <ø> (ø)
packages/graphiql-react/src/editor/tabs.ts	26.82% <ø> (-34.15%)	⬇️
packages/graphiql-react/src/editor/whitespace.ts	100.00% <ø> (ø)
...ct/src/explorer/components/field-documentation.tsx	97.14% <100.00%> (ø)
...act/src/explorer/components/type-documentation.tsx	85.10% <100.00%> (ø)
...s/graphiql-react/src/explorer/components/utils.tsx	100.00% <ø> (ø)
packages/graphiql-react/src/markdown.ts	100.00% <ø> (ø)
...ages/graphiql-react/src/utility/component-group.ts	100.00% <ø> (ø)
packages/graphiql-react/src/utility/context.ts	84.61% <ø> (ø)
... and 15 more

... and 27 files with indirect coverage changes

[github-actions]

The latest changes of this PR are available as canary in npm (based on the declared changesets):

graphiql@3.7.3-canary-2eca8599.0
@graphiql/plugin-code-exporter@3.1.4-canary-2eca8599.0
@graphiql/plugin-explorer@3.2.4-canary-2eca8599.0
@graphiql/react@0.28.0-canary-2eca8599.0

[acao]

@dimaMachina any chance this can happen against the v4 branch? will it be hard to adopt these changes to the v4 branch?

[dimaMachina]

@acao I think folks can already benefit from this improvement even in v3? see how react-compiler improved Nextra 😅

Screen.Recording.2024-12-10.at.22.30.38.mov

[TallTed]

So many commits with meaningless comments. upd? aa? fix? Might as well just leave the comment blank. Much better to put some hint about what the commit actually contains!

[emmenko]

Hi 👋

Was there a reason to remove this? This might be related to the following regression: #3842

[dimaMachina]

Hi, it's still present

graphiql/packages/graphiql-react/src/explorer/components/search.tsx

Lines 60 to 62 in 1d598a9

	const handleFocus: FocusEventHandler = e => {
	setIsFocused(e.type === 'focus');
	};

I will take a look at your issue soon!