-
Notifications
You must be signed in to change notification settings - Fork 577
fix(deps): update dependency @graphiql/plugin-explorer to v5 #4033
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.
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
base: main
Are you sure you want to change the base?
Conversation
Review or Edit in CodeSandboxOpen the branch in Web Editor • VS Code • Insiders |
69d0817
to
2f88b7f
Compare
2f88b7f
to
b84de73
Compare
b84de73
to
8cb9f6a
Compare
🚀 Snapshot Release (
|
Package | Version | Info |
---|---|---|
@graphql-yoga/graphiql |
4.4.1-alpha-20250728153604-2e6981dc9f8987167d2d47f8058a7243c07d0b5b |
npm ↗︎ unpkg ↗︎ |
@graphql-yoga/plugin-apollo-usage-report |
0.10.2-alpha-20250728153604-2e6981dc9f8987167d2d47f8058a7243c07d0b5b |
npm ↗︎ unpkg ↗︎ |
Apollo Federation Subgraph Compatibility Results
Learn more: |
💻 Website PreviewThe latest changes are available as preview in: https://pr-4033.graphql-yoga.pages.dev |
✅ Benchmark Results
|
1ab7023
to
efb8a21
Compare
b3ac6d9
to
4ee85d0
Compare
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
f54f50d
to
ebd952e
Compare
c8a7340
to
428467c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR updates the @graphiql/plugin-explorer
dependency from v3.0.0 to v5.1.1, which is a major version update that includes significant breaking changes in the GraphiQL ecosystem. The update involves migrating from CodeMirror to Monaco Editor and transitioning from React Context to Zustand for state management.
Key changes include:
- Major dependency updates for GraphiQL components and related packages
- Refactoring of the YogaGraphiQL component to use the new GraphiQL v5 API
- Updates to browser integration tests to work with the new Monaco Editor selectors
- Removal of patches and deprecated dependencies
Reviewed Changes
Copilot reviewed 8 out of 9 changed files in this pull request and generated 3 comments.
Show a summary per file
File | Description |
---|---|
packages/graphiql/package.json | Updates GraphiQL dependencies to v5 and removes deprecated packages |
packages/graphiql/src/YogaGraphiQL.tsx | Refactors component to use new GraphiQL v5 API and removes deprecated patterns |
packages/graphql-yoga/integration-tests/browser.spec.ts | Updates test selectors for Monaco Editor and adds debugging capabilities |
patches/@graphiql__react@0.20.4.patch | Removes patch file that's no longer needed |
examples/nextjs-app/integration-tests/nextjs-app.spec.ts | Updates test assertion for new GraphiQL title format |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
isHeadersEditorEnabled | ||
defaultEditorToolsVisibility | ||
initialQuery={deprecatedInitialQuery} | ||
defaultHeaders={deprecatedInitialHeaders} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The prop name defaultHeaders
should be initialHeaders
to match the GraphiQL v5 API. Based on the deprecation warnings in the type definitions, this appears to be using an incorrect prop name.
defaultHeaders={deprecatedInitialHeaders} | |
initialHeaders={deprecatedInitialHeaders} |
Copilot uses AI. Check for mistakes.
import 'graphiql/style.css'; | ||
import '@graphiql/plugin-explorer/style.css'; | ||
import { GraphiQL, GraphiQLProps } from 'graphiql'; | ||
import { DocumentNode, Kind, parse } from 'graphql'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Kind
import appears to be unused in this file. Consider removing it to keep imports clean.
Copilot uses AI. Check for mistakes.
// @ts-expect-error - value is not null | ||
() => !!window.document.querySelector('[data-uri*="response"] textarea')?.value?.trim(), | ||
); | ||
const resultContents = await page.evaluate(() => { | ||
return window.document | ||
.querySelector('.graphiql-response .CodeMirror-code') | ||
?.textContent?.trim() | ||
.replaceAll('\u00A0', ' '); | ||
return ( | ||
window.document | ||
.querySelector('[data-uri*="response"] textarea') | ||
// @ts-expect-error - value is not null | ||
?.value?.trim() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[nitpick] Instead of using @ts-expect-error, consider using optional chaining or proper type assertions. The comment suggests confidence that value is not null, so a non-null assertion (!) would be more appropriate.
Copilot uses AI. Check for mistakes.
This PR contains the following updates:
^3.0.0
->^5.0.0
Release Notes
graphql/graphiql (@graphiql/plugin-explorer)
v5.1.1
Compare Source
Patch Changes
6e5d5fc
,293beed
]:v5.1.0
Compare Source
Minor Changes
3d41e11
Thanks @dimaMachina! - add new example Usage GraphiQL 5 with Vite, React Router andssr: true
Patch Changes
3a0a755
,fd3f9e6
,416e3a0
,3d41e11
]:v5.0.0
Compare Source
Major Changes
#3990
27e7eb6
Thanks @dimaMachina! - - allow multiple independent instances of GraphiQL on the same pageonClickReference
in query editor in Reactref
onClickReference
from variable editor#4009
4936492
Thanks @dimaMachina! - separate store actions from state, adduseGraphiQLActions
state#4002
2d9faec
Thanks @dimaMachina! - remove UMD buildsPatch Changes
#3949
0844dc1
Thanks @dimaMachina! - - replaceonCopyQuery
hook withcopyQuery
functiononMergeQuery
hook withmergeQuery
functiononPrettifyEditors
hook withprettifyEditors
functionfetcher
prop fromSchemaContextProvider
andschemaStore
and addfetcher
toexecutionStore
onCopyQuery
andonPrettifyQuery
props toEditorContextProvider
GraphiQLProvider
)EditorContextProvider
ExecutionContextProvider
PluginContextProvider
SchemaContextProvider
StorageContextProvider
ExecutionContextType
PluginContextType
useExecutionContext
withuseExecutionStore
hookuseEditorContext
withuseEditorStore
hookgetComputedStyle
overwindow.getComputedStyle
#3234
86a96e5
Thanks @dimaMachina! - Migration from Codemirror to Monaco EditorReplacing
codemirror-graphql
withmonaco-graphql
Support for comments in Variables and Headers editors
v4.0.6
Compare Source
Patch Changes
7054591
Thanks @dimaMachina! - revert https://github.com/graphql/graphiql/pull/3946 to have support multiple embedded graphiql instances on the same pagev4.0.5
Compare Source
Patch Changes
71755b7
Thanks @dimaMachina! - feat(@graphiql/react): migrate React context to zustand:useExecutionContext
withuseExecutionStore
hookuseEditorContext
withuseEditorStore
hookuseAutoCompleteLeafs
hook withgetAutoCompleteLeafs
function71755b7
,6d631e2
]:v4.0.4
Compare Source
Patch Changes
#3945
117627b
Thanks @dimaMachina! - feat(@graphiql/react): migrate React context to zustand, replaceusePluginContext
withusePluginStore
hook#3947
fa78481
Thanks @dimaMachina! - refactoruseStorage
,useDocExplorer
anduseHistory
hooks#3943
7275472
Thanks @dimaMachina! - feat(@graphiql/react): migrate React context to zustand, replaceuseSchemaContext
withuseSchemaStore
hookUpdated dependencies [
117627b
,fa78481
,7275472
,00c8605
]:v4.0.3
Compare Source
Patch Changes
69ad489
Thanks @dimaMachina! - preferReact.FC
type when declaring React componentsv4.0.2
Compare Source
Patch Changes
98d13a3
]:v4.0.1
Compare Source
Patch Changes
#3915
bc31cd9
Thanks @dimaMachina! - fix unpkg.com results toNot found
whenmain
field isn't specified inpackage.json
Updated dependencies [
e7c436b
]:v4.0.0
Compare Source
Major Changes
#3904
d1395f9
Thanks @dimaMachina! - drop commonjs build files#3904
d1395f9
Thanks @dimaMachina! - - support react 19, drop support react 16 and react 17ReactDOM.unmountComponentAtNode()
andReactDOM.render()
withroot.unmount()
andcreateRoot(container).render()
@radix-ui
and@headlessui/react
dependencies#3904
d1395f9
Thanks @dimaMachina! -style.css
import was changedConfiguration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.