fix: Never use a service at code root

Author: CGNonofr

src/configuration/index.ts

@@ -16,15 +16,18 @@ configurationRegistry.registerDefaultConfigurations([{
   overrides: extensions.configurationDefaults
 }])
 
-const simpleConfigurationService = monaco.extra.StandaloneServices.get(monaco.extra.IConfigurationService) as monaco.extra.StandaloneConfigurationService
-
-export const onConfigurationChanged = simpleConfigurationService.onDidChangeConfiguration
+export function onConfigurationChanged (listener: (e: monaco.extra.IConfigurationChangeEvent) => void): monaco.IDisposable {
+  const simpleConfigurationService = monaco.extra.StandaloneServices.get(monaco.extra.IConfigurationService) as monaco.extra.StandaloneConfigurationService
+  return simpleConfigurationService.onDidChangeConfiguration(listener)
+}
 
 export function getConfiguration<C = Partial<monaco.editor.IEditorOptions>> (language?: string, section: string = 'editor'): C | undefined {
+  const simpleConfigurationService = monaco.extra.StandaloneServices.get(monaco.extra.IConfigurationService) as monaco.extra.StandaloneConfigurationService
   return simpleConfigurationService.getValue(section, { overrideIdentifier: language })
 }
 
 export function updateUserConfiguration (configurationJson: string): void {
+  const simpleConfigurationService = monaco.extra.StandaloneServices.get(monaco.extra.IConfigurationService) as monaco.extra.StandaloneConfigurationService
   simpleConfigurationService.updateUserConfiguration(configurationJson)
 }
 

src/features/htmlContribution.ts

@@ -28,11 +28,16 @@ registerConfigurations([{
   }
 }])
 
-let autoClosingTags = getConfiguration<boolean>(undefined, 'html.autoClosingTags')!
-onConfigurationChanged(e => {
-  if (e.affectsConfiguration('html.autoClosingTags')) {
-    autoClosingTags = getConfiguration<boolean>(undefined, 'html.autoClosingTags')!
-  }
+let autoClosingTags = false
+
+setTimeout(() => {
+  // In a timeout so the service can be overriden
+  autoClosingTags = getConfiguration<boolean>(undefined, 'html.autoClosingTags')!
+  onConfigurationChanged(e => {
+    if (e.affectsConfiguration('html.autoClosingTags')) {
+      autoClosingTags = getConfiguration<boolean>(undefined, 'html.autoClosingTags')!
+    }
+  })
 })
 
 function autoCloseHtmlTags (editor: monaco.editor.ICodeEditor): monaco.IDisposable {
@@ -100,13 +105,17 @@ function autoCloseHtmlTags (editor: monaco.editor.ICodeEditor): monaco.IDisposab
   return disposableStore
 }
 
-const codeEditors = monaco.extra.StandaloneServices.get(monaco.extra.ICodeEditorService).listCodeEditors()
-for (const editor of codeEditors) {
-  autoCloseHtmlTags(editor)
-}
-monaco.editor.onDidCreateEditor(editor => {
-  autoCloseHtmlTags(editor)
+setTimeout(() => {
+  // In a timeout so the service can be overriden
+  const codeEditors = monaco.extra.StandaloneServices.get(monaco.extra.ICodeEditorService).listCodeEditors()
+  for (const editor of codeEditors) {
+    autoCloseHtmlTags(editor)
+  }
+  monaco.editor.onDidCreateEditor(editor => {
+    autoCloseHtmlTags(editor)
+  })
 })
+
 /**
  * End autoclosing html tags
  */

src/keybindings/keybindingHelper.ts

@@ -1,7 +1,6 @@
 import * as monaco from 'monaco-editor'
 
-const keybindingService = monaco.extra.StandaloneServices.get(monaco.extra.IKeybindingService) as monaco.extra.StandaloneKeybindingService
-
 export function updateKeybindings (bindings: monaco.extra.IUserFriendlyKeybinding[]): void {
+  const keybindingService = monaco.extra.StandaloneServices.get(monaco.extra.IKeybindingService) as monaco.extra.StandaloneKeybindingService
   keybindingService.setUserKeybindings(bindings)
 }

src/languages/index.ts

@@ -26,7 +26,6 @@ for (const [languageId, aliases] of Object.entries(customAliases)) {
   })
 }
 
-const languageService = monaco.extra.StandaloneServices.get(monaco.languages.ILanguageService)
 const languagesIds = Array.from(new Set([
   ...Object.keys(monarchLanguageLoader),
   ...textMateLanguages.map(rawLanguage => rawLanguage.id)
@@ -91,19 +90,24 @@ async function loadLanguageConfiguration (languageId: string) {
   }
 }
 
-languageService.onDidEncounterLanguage(async (languageId) => {
-  if (languageId === 'plaintext') {
-    return
-  }
+setTimeout(() => {
+  // In a timeout so the service can be overriden
+  const languageService = monaco.extra.StandaloneServices.get(monaco.languages.ILanguageService)
+  languageService.onDidEncounterLanguage(async (languageId) => {
+    if (languageId === 'plaintext') {
+      return
+    }
 
-  loadLanguageConfiguration(languageId).catch(error => {
-    monaco.errorHandler.onUnexpectedError(new Error(`Unable to load language configuration for language ${languageId}`, {
-      cause: error
-    }))
+    loadLanguageConfiguration(languageId).catch(error => {
+      monaco.errorHandler.onUnexpectedError(new Error(`Unable to load language configuration for language ${languageId}`, {
+        cause: error
+      }))
+    })
   })
 })
 
 function getMonacoLanguage (languageOrModeId: string): string {
+  const languageService = monaco.extra.StandaloneServices.get(monaco.languages.ILanguageService)
   const modeId = languageService.getLanguageIdByLanguageName(languageOrModeId.toLowerCase())
   if (modeId != null) {
     return modeId

src/languages/snippets/index.ts

@@ -37,6 +37,9 @@ class SimpleSnippetService implements monaco.extra.ISnippetsService {
   }
 }
 
-const languageService = monaco.extra.StandaloneServices.get(monaco.languages.ILanguageService)
-const languageConfigurationService = monaco.extra.StandaloneServices.get(monaco.languages.ILanguageConfigurationService)
-monaco.extra.setSnippetSuggestSupport(new monaco.extra.SnippetCompletionProvider(languageService, new SimpleSnippetService(), languageConfigurationService))
+setTimeout(() => {
+  // In a timeout so the service can be overriden
+  const languageService = monaco.extra.StandaloneServices.get(monaco.languages.ILanguageService)
+  const languageConfigurationService = monaco.extra.StandaloneServices.get(monaco.languages.ILanguageConfigurationService)
+  monaco.extra.setSnippetSuggestSupport(new monaco.extra.SnippetCompletionProvider(languageService, new SimpleSnippetService(), languageConfigurationService))
+})

src/languages/textMate/index.ts

@@ -32,8 +32,8 @@ function getOrCreateGrammarFactory (): CGTMGrammarFactory {
   return textMateGrammarFactory
 }
 
-const languageService = monaco.extra.StandaloneServices.get(monaco.languages.ILanguageService)
 async function createTextMateTokensProvider (languageId: string): Promise<monaco.languages.ITokenizationSupport | null> {
+  const languageService = monaco.extra.StandaloneServices.get(monaco.languages.ILanguageService)
   const grammarFactory = getOrCreateGrammarFactory()
   const encodedLanguageId = languageService.languageIdCodec.encodeLanguageId(languageId)
   if (!grammarFactory.has(languageId)) {

src/monaco.ts

@@ -25,7 +25,10 @@ monaco.extra.StandaloneServices.initialize({
   ...getMessageServiceOverride(document.body)
 })
 // Disable high contrast autodetection because it fallbacks on the hc-black no matter what
-monaco.extra.StandaloneServices.get(monaco.editor.IStandaloneThemeService).setAutoDetectHighContrast(false)
+setTimeout(() => {
+  // In a timeout so the service can be overriden
+  monaco.extra.StandaloneServices.get(monaco.editor.IStandaloneThemeService).setAutoDetectHighContrast(false)
+})
 
 monaco.errorHandler.setUnexpectedErrorHandler(error => {
   console.warn('Unexpected error', error)