refactor: parentSkeletonSelectedFirst into suggest config node

Author: Petr Spacek

README.md

@@ -49,7 +49,7 @@ The following settings are supported:
 - `http.proxyStrictSSL`: If true the proxy server certificate should be verified against the list of supplied CAs. Default is false.
 - `[yaml].editor.formatOnType`: Enable/disable on type indent and auto formatting array
 - `yaml.disableDefaultProperties`: Disable adding not required properties with default values into completion text
-- `yaml.selectParentSkeletonFirst`: If true, the user must select some parent skeleton first before autocompletion starts to suggest the rest of the properties.\nWhen yaml object is not empty, autocompletion ignores this setting and returns all properties and skeletons.
+- `yaml.suggest.parentSkeletonSelectedFirst`: If true, the user must select some parent skeleton first before autocompletion starts to suggest the rest of the properties.\nWhen yaml object is not empty, autocompletion ignores this setting and returns all properties and skeletons.
 
 ##### Adding custom tags
 

src/languageserver/handlers/settingsHandlers.ts

@@ -105,7 +105,10 @@ export class SettingsHandler {
       }
       this.yamlSettings.disableAdditionalProperties = settings.yaml.disableAdditionalProperties;
       this.yamlSettings.disableDefaultProperties = settings.yaml.disableDefaultProperties;
-      this.yamlSettings.selectParentSkeletonFirst = settings.yaml.selectParentSkeletonFirst;
+
+      if (settings.yaml.suggest) {
+        this.yamlSettings.suggest.parentSkeletonSelectedFirst = settings.yaml.suggest.parentSkeletonSelectedFirst;
+      }
     }
 
     this.yamlSettings.schemaConfigurationSettings = [];
@@ -233,7 +236,7 @@ export class SettingsHandler {
       indentation: this.yamlSettings.indentation,
       disableAdditionalProperties: this.yamlSettings.disableAdditionalProperties,
       disableDefaultProperties: this.yamlSettings.disableDefaultProperties,
-      selectParentSkeletonFirst: this.yamlSettings.selectParentSkeletonFirst,
+      parentSkeletonSelectedFirst: this.yamlSettings.suggest.parentSkeletonSelectedFirst,
       yamlVersion: this.yamlSettings.yamlVersion,
     };
 

src/languageservice/services/yamlCompletion.ts

@@ -73,7 +73,7 @@ export class YamlCompletion {
   private indentation: string;
   private supportsMarkdown: boolean | undefined;
   private disableDefaultProperties: boolean;
-  private selectParentSkeletonFirst: boolean;
+  private parentSkeletonSelectedFirst: boolean;
 
   constructor(
     private schemaService: YAMLSchemaService,
@@ -90,7 +90,7 @@ export class YamlCompletion {
     this.yamlVersion = languageSettings.yamlVersion;
     this.configuredIndentation = languageSettings.indentation;
     this.disableDefaultProperties = languageSettings.disableDefaultProperties;
-    this.selectParentSkeletonFirst = languageSettings.selectParentSkeletonFirst;
+    this.parentSkeletonSelectedFirst = languageSettings.parentSkeletonSelectedFirst;
   }
 
   async doComplete(document: TextDocument, position: Position, isKubernetes = false): Promise<CompletionList> {
@@ -591,8 +591,6 @@ export class YamlCompletion {
     const lineContent = textBuffer.getLineContent(overwriteRange.start.line);
     const hasOnlyWhitespace = lineContent.trim().length === 0;
     const hasColon = lineContent.indexOf(':') !== -1;
-    const isNodeNull =
-      (isScalar(originalNode) && originalNode.value === null) || (isMap(originalNode) && originalNode.items.length === 0);
     const nodeParent = doc.getParent(node);
     const matchOriginal = matchingSchemas.find((it) => it.node.internalNode === originalNode && it.schema.properties);
     for (const schema of matchingSchemas) {
@@ -679,8 +677,11 @@ export class YamlCompletion {
                       identCompensation + this.indentation
                     );
                   }
+                  const isNodeNull =
+                    (isScalar(originalNode) && originalNode.value === null) ||
+                    (isMap(originalNode) && originalNode.items.length === 0);
                   const existsParentCompletion = schema.schema.required?.length > 0;
-                  if (!this.selectParentSkeletonFirst || !isNodeNull || !existsParentCompletion) {
+                  if (!this.parentSkeletonSelectedFirst || !isNodeNull || !existsParentCompletion) {
                     collector.add({
                       kind: CompletionItemKind.Property,
                       label: key,

src/languageservice/yamlLanguageService.ts

@@ -102,7 +102,7 @@ export interface LanguageSettings {
    * If true, the user must select some parent skeleton first before autocompletion starts to suggest the rest of the properties.
    * When yaml object is not empty, autocompletion ignores this setting and returns all properties and skeletons.
    */
-  selectParentSkeletonFirst?: boolean;
+  parentSkeletonSelectedFirst?: boolean;
 
   /**
    * Default yaml lang version

src/yamlSettings.ts

@@ -22,7 +22,9 @@ export interface Settings {
     };
     disableDefaultProperties: boolean;
     disableAdditionalProperties: boolean;
-    selectParentSkeletonFirst: boolean;
+    suggest: {
+      parentSkeletonSelectedFirst: boolean;
+    };
     maxItemsComputed: number;
     yamlVersion: YamlVersion;
   };
@@ -70,7 +72,9 @@ export class SettingsState {
   indentation: string | undefined = undefined;
   disableAdditionalProperties = false;
   disableDefaultProperties = false;
-  selectParentSkeletonFirst = false;
+  suggest = {
+    parentSkeletonSelectedFirst: false,
+  };
   maxItemsComputed = 5000;
 
   // File validation helpers

test/autoCompletion.test.ts

@@ -2798,7 +2798,7 @@ describe('Auto Completion Tests', () => {
     describe('Select parent skeleton first', () => {
       beforeEach(() => {
         const languageSettingsSetup = new ServiceSetup().withCompletion();
-        languageSettingsSetup.languageSettings.selectParentSkeletonFirst = true;
+        languageSettingsSetup.languageSettings.parentSkeletonSelectedFirst = true;
         languageService.configure(languageSettingsSetup.languageSettings);
       });
       it('Should suggest complete object skeleton', async () => {