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.

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

monaco-graphql not working #737

Closed
rubenfiszel opened this issue Aug 24, 2024 · 3 comments
Closed

monaco-graphql not working #737

rubenfiszel opened this issue Aug 24, 2024 · 3 comments

Comments

@rubenfiszel
Copy link

Sorry I know this was fixed previously but I have this error:

Failed to resolve import "vscode/vscode/vs/editor/contrib/hover/browser/hover" from "node_modules/monaco-editor/esm/vs/editor/contrib/hover/browser/hover.js

monaco-graphql depends on:
monaco-editor/esm/vs/editor/contrib/hover/browser/hover.js

and if I cat its content I have:

cat node_modules/monaco-editor/esm/vs/editor/contrib/hover/browser/hover.js
export * from 'vscode/vscode/vs/editor/contrib/hover/browser/hover'

There is nothing at that path, the node_modules for vscode seems to contain the source only, which if I list:

ls node_modules/vscode/vscode/src/vs/editor/contrib/hover/browser 
contentHoverComputer.js
contentHoverController.js
contentHoverRendered.js
contentHoverStatusBar.js
contentHoverTypes.js
contentHoverWidget.js
getHover.js
hoverAccessibleViews.js
hoverActionIds.js
hoverActions.js
hoverContribution.js
hoverController.js
hover.css.js
hoverOperation.js
hoverTypes.js
marginHoverComputer.js
marginHoverWidget.js
markdownHoverParticipant.js
markerHoverParticipant.js
resizableContentWidget.js

doesn't seem to contain an hover.js. However it seems that previous versions of vscode did have such file. Is it a matter of doing a PR on monaco-graphql ?

@rubenfiszel
Copy link
Author

fix in the meantime:

	resolve: {
		alias: {
			'vscode/vscode/vs/editor/contrib/hover/browser/hover':
				'vscode/vscode/vs/editor/contrib/hover/browser/hoverController'
		},
	}

might not be a monaco-languageclient issue but likely an issue with graphql in the first place. However, when they do update, vscode-api should expose that hoverController

@kaisalmen
Copy link
Collaborator

However, when they do update, vscode-api should expose that hoverController

@CGNonofr WDYT?

@CGNonofr
Copy link
Collaborator

I'm afraid the vs/editor/contrib/hover/browser/hover file was moved (in this commit)

so both monaco-vscode-api AND monaco-graphql need an update

Here's the monaco-vscode-api one: CodinGame/monaco-vscode-api#493

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants