@platformos/platformos-check-common 0.0.7 → 0.0.8

package/src/checks/liquid-html-syntax-error/checks/InvalidLoopArguments.spec.ts

@@ -109,7 +109,7 @@ describe('detectInvalidLoopArguments', async () => {
 
 function testCheck(sourceCode: string) {
   return runLiquidCheck(LiquidHTMLSyntaxError, sourceCode, undefined, {
-    themeDocset: {
+    platformosDocset: {
       graphQL: async () => null,
       tags: () =>
         Promise.resolve([
@@ -139,7 +139,6 @@ function testCheck(sourceCode: string) {
       filters: () => Promise.resolve([]),
       objects: () => Promise.resolve([]),
       liquidDrops: () => Promise.resolve([]),
-      systemTranslations: () => Promise.resolve({}),
     },
   });
 }

package/src/checks/liquid-html-syntax-error/index.spec.ts

@@ -86,7 +86,7 @@ describe('Module: LiquidHTMLSyntaxError', () => {
     const offenses = await runLiquidCheck(LiquidHTMLSyntaxError, sourceCode);
     expect(offenses).to.have.length(1);
     expect(offenses[0].message).to.equal(
-      `SyntaxError: expected "#", a letter, "yield", "theme_render_rc", "spam_protection", "sign_in", "session", "rollback", "return", "response_status", "response_headers", "redirect_to", "print", "log", "include_form", "export", "context", "catch", "background", "when", "sections", "section", "graphql", "function", "render", "liquid", "layout", "increment", "include", "elsif", "else", "echo", "decrement", "content_for", "cycle", "continue", "break", "hash_assign", "assign", "try", "transaction", "parse_json", "cache", "tablerow", "unless", "if", "ifchanged", "for", "case", "capture", "paginate", "form", "end", "style", "stylesheet", "schema", "javascript", "raw", "comment", or "doc"`,
+      `SyntaxError: expected "#", a letter, "yield", "theme_render_rc", "spam_protection", "sign_in", "session", "rollback", "return", "response_status", "response_headers", "redirect_to", "print", "log", "include_form", "export", "context", "catch", "background", "when", "graphql", "function", "render", "liquid", "layout", "increment", "include", "elsif", "else", "echo", "decrement", "cycle", "continue", "break", "hash_assign", "assign", "try", "transaction", "parse_json", "cache", "tablerow", "unless", "if", "ifchanged", "for", "case", "capture", "form", "content_for", "end", "raw", "comment", or "doc"`,
     );
   });
 
@@ -95,11 +95,6 @@ describe('Module: LiquidHTMLSyntaxError', () => {
       {% if some_variable %}
         Hello, world!
       {% endif %}
-      {% schema %}
-      {
-        "settings": [{ "visible_if": "{{ foo.bar == true }}" }]
-      }
-      {% endschema %}
     `;
 
     const offenses = await runLiquidCheck(LiquidHTMLSyntaxError, sourceCode);

package/src/checks/liquid-html-syntax-error/index.ts

@@ -45,8 +45,8 @@ export const LiquidHTMLSyntaxError: LiquidCheckDefinition = {
 
   create(context) {
     const ast = context.file.ast;
-    const filtersPromise = context.themeDocset?.filters();
-    const tagsPromise = context.themeDocset?.tags();
+    const filtersPromise = context.platformosDocset?.filters();
+    const tagsPromise = context.platformosDocset?.tags();
 
     if (!isError(ast)) {
       return {

package/src/checks/matching-translations/index.spec.ts

@@ -1,276 +1,129 @@
 import { expect, describe, it } from 'vitest';
-import { autofix, check, highlightedOffenses } from '../../test';
+import { check } from '../../test';
 import { MatchingTranslations } from '../../checks/matching-translations/index';
 
-const prettyJSON = (json: any) => JSON.stringify(json, null, 2);
-
 describe('Module: MatchingTranslations', async () => {
   it('should report offenses when the translation file is missing a key', async () => {
-    for (const prefix of ['', '.schema']) {
-      const theme = {
-        [`locales/en.default${prefix}.json`]: JSON.stringify({
-          hello: 'Hello',
-          world: 'World',
-        }),
-        [`locales/pt-BR${prefix}.json`]: JSON.stringify({
-          hello: 'Olá',
-        }),
-      };
+    const theme = {
+      'app/translations/en.yml': 'en:\n  hello: Hello\n  world: World\n',
+      'app/translations/pt-BR.yml': 'pt-BR:\n  hello: Olá\n',
+    };
 
-      const offenses = await check(theme, [MatchingTranslations]);
+    const offenses = await check(theme, [MatchingTranslations]);
 
-      expect(offenses).to.be.of.length(1);
-      expect(offenses).to.containOffense("The translation for 'world' is missing");
-    }
+    expect(offenses).to.be.of.length(1);
+    expect(offenses).to.containOffense("The translation for 'world' is missing");
   });
 
   it('should report offenses when the default translation is missing a key', async () => {
-    for (const prefix of ['', '.schema']) {
-      const theme = {
-        [`locales/en.default${prefix}.json`]: JSON.stringify({
-          hello: 'Hello',
-        }),
-        [`locales/pt-BR${prefix}.json`]: JSON.stringify({
-          hello: 'Olá',
-          world: 'Mundo',
-        }),
-      };
+    const theme = {
+      'app/translations/en.yml': 'en:\n  hello: Hello\n',
+      'app/translations/pt-BR.yml': 'pt-BR:\n  hello: Olá\n  world: Mundo\n',
+    };
 
-      const offenses = await check(theme, [MatchingTranslations]);
+    const offenses = await check(theme, [MatchingTranslations]);
 
-      expect(offenses).to.be.of.length(1);
-      expect(offenses).to.containOffense("A default translation for 'world' does not exist");
-      expect(offenses[0]!).to.suggest(
-        theme[`locales/pt-BR${prefix}.json`],
-        'Delete unneeded translation key',
-        {
-          startIndex: 0,
-          endIndex: theme[`locales/pt-BR${prefix}.json`].length,
-          insert: prettyJSON({
-            hello: 'Olá',
-          }),
-        },
-      );
-    }
+    expect(offenses).to.be.of.length(1);
+    expect(offenses).to.containOffense("A default translation for 'world' does not exist");
   });
 
   it('should report offenses when nested translation keys do not exist', async () => {
-    for (const prefix of ['', '.schema']) {
-      const theme = {
-        [`locales/en.default${prefix}.json`]: JSON.stringify({
-          hello: { world: 'Hello, world!' },
-        }),
-        [`locales/pt-BR${prefix}.json`]: JSON.stringify({
-          hello: {},
-        }),
-      };
-
-      const offenses = await check(theme, [MatchingTranslations]);
+    const theme = {
+      'app/translations/en.yml': 'en:\n  hello:\n    world: Hello, world!\n',
+      'app/translations/pt-BR.yml': 'pt-BR:\n  hello: {}\n',
+    };
 
-      expect(offenses).to.be.of.length(1);
-      expect(offenses).to.containOffense({
-        message: "The translation for 'hello.world' is missing",
-        uri: `file:///locales/pt-BR${prefix}.json`,
-      });
+    const offenses = await check(theme, [MatchingTranslations]);
 
-      const fixed = await autofix(theme, offenses);
-      expect(fixed[`locales/pt-BR${prefix}.json`]).to.eql(
-        prettyJSON({
-          hello: {
-            world: 'TODO',
-          },
-        }),
-      );
-    }
+    expect(offenses).to.be.of.length(1);
+    expect(offenses).to.containOffense({
+      message: "The translation for 'hello.world' is missing",
+      uri: `file:///app/translations/pt-BR.yml`,
+    });
   });
 
   it('should report offenses when translation shapes do not match', async () => {
-    for (const prefix of ['', '.schema']) {
-      const theme = {
-        [`locales/en.default${prefix}.json`]: JSON.stringify({
-          hello: { world: 'Hello, world!' },
-        }),
-        [`locales/pt-BR${prefix}.json`]: JSON.stringify({
-          hello: 'Olá',
-        }),
-      };
-
-      const offenses = await check(theme, [MatchingTranslations]);
-
-      expect(offenses).to.be.of.length(2);
-      expect(offenses).to.containOffense({
-        message: "A default translation for 'hello' does not exist",
-        uri: `file:///locales/pt-BR${prefix}.json`,
-      });
-      expect(offenses).to.containOffense({
-        message: "The translation for 'hello.world' is missing",
-        uri: `file:///locales/pt-BR${prefix}.json`,
-      });
+    const theme = {
+      'app/translations/en.yml': 'en:\n  hello:\n    world: Hello, world!\n',
+      'app/translations/pt-BR.yml': 'pt-BR:\n  hello: Olá\n',
+    };
 
-      const fixed = await autofix(theme, offenses);
+    const offenses = await check(theme, [MatchingTranslations]);
 
-      expect(fixed[`locales/pt-BR${prefix}.json`]).to.eql(
-        prettyJSON({
-          hello: { world: 'TODO' },
-        }),
-      );
-    }
+    expect(offenses).to.be.of.length(2);
+    expect(offenses).to.containOffense({
+      message: "A default translation for 'hello' does not exist",
+      uri: `file:///app/translations/pt-BR.yml`,
+    });
+    expect(offenses).to.containOffense({
+      message: "The translation for 'hello.world' is missing",
+      uri: `file:///app/translations/pt-BR.yml`,
+    });
   });
 
   it('should report offenses when nested translation keys do not match', async () => {
-    for (const prefix of ['', '.schema']) {
-      const theme = {
-        [`locales/en.default${prefix}.json`]: JSON.stringify({
-          hello: { world: 'Hello, world!' },
-        }),
-        [`locales/fr${prefix}.json`]: JSON.stringify({
-          hello: { monde: 'Bonjour, monde' },
-        }),
-        [`locales/es-ES${prefix}.json`]: JSON.stringify({
-          hello: { world: 'Hello, world!', mundo: { hola: '¡Hola, mundo!' } },
-        }),
-      };
-
-      const offenses = await check(theme, [MatchingTranslations]);
-
-      expect(offenses).to.be.of.length(3);
-      expect(offenses).to.containOffense({
-        message: "A default translation for 'hello.monde' does not exist",
-        uri: `file:///locales/fr${prefix}.json`,
-      });
-      expect(offenses).to.containOffense({
-        message: "A default translation for 'hello.mundo.hola' does not exist",
-        uri: `file:///locales/es-ES${prefix}.json`,
-      });
-      expect(offenses).to.containOffense({
-        message: "The translation for 'hello.world' is missing",
-        uri: `file:///locales/fr${prefix}.json`,
-      });
-
-      const fixed = await autofix(theme, offenses);
-      expect(fixed[`locales/fr${prefix}.json`]).to.eql(
-        prettyJSON({
-          hello: { monde: 'Bonjour, monde', world: 'TODO' },
-        }),
-      );
+    const theme = {
+      'app/translations/en.yml': 'en:\n  hello:\n    world: Hello, world!\n',
+      'app/translations/fr.yml': 'fr:\n  hello:\n    monde: Bonjour, monde\n',
+      'app/translations/es-ES.yml':
+        'es-ES:\n  hello:\n    world: Hello, world!\n    mundo:\n      hola: "¡Hola, mundo!"\n',
+    };
 
-      // Default does not exist should be a suggestion and not autofixed.
-      expect(fixed[`locales/es-ES${prefix}.json`]).to.eql(theme[`locales/es-ES${prefix}.json`]);
-    }
-  });
+    const offenses = await check(theme, [MatchingTranslations]);
 
-  it('should not report offenses when default translations do not exist', async () => {
-    for (const prefix of ['', '.schema']) {
-      const theme = {
-        [`locales/en${prefix}.json`]: JSON.stringify({
-          hello: 'Hello',
-        }),
-        [`locales/pt-BR${prefix}.json`]: JSON.stringify({
-          hello: 'Olá',
-        }),
-      };
+    expect(offenses).to.be.of.length(3);
+    expect(offenses).to.containOffense({
+      message: "A default translation for 'hello.monde' does not exist",
+      uri: `file:///app/translations/fr.yml`,
+    });
+    expect(offenses).to.containOffense({
+      message: "A default translation for 'hello.mundo.hola' does not exist",
+      uri: `file:///app/translations/es-ES.yml`,
+    });
+    expect(offenses).to.containOffense({
+      message: "The translation for 'hello.world' is missing",
+      uri: `file:///app/translations/fr.yml`,
+    });
+  });
+
+  it('should not report offenses when default translations do not exist (no en.yml)', async () => {
+    const theme = {
+      'app/translations/pt-BR.yml': 'pt-BR:\n  hello: Olá\n',
+    };
 
-      const offenses = await check(theme, [MatchingTranslations]);
+    const offenses = await check(theme, [MatchingTranslations]);
 
-      expect(offenses).to.be.of.length(0);
-    }
+    expect(offenses).to.be.of.length(0);
   });
 
   it('should not report offenses when translations match', async () => {
-    for (const prefix of ['', '.schema']) {
-      const theme = {
-        [`locales/en.default${prefix}.json`]: JSON.stringify({
-          hello: 'Hello',
-          world: 'World',
-        }),
-        [`locales/pt-BR${prefix}.json`]: JSON.stringify({
-          hello: 'Olá',
-          world: 'Mundo',
-        }),
-      };
+    const theme = {
+      'app/translations/en.yml': 'en:\n  hello: Hello\n  world: World\n',
+      'app/translations/pt-BR.yml': 'pt-BR:\n  hello: Olá\n  world: Mundo\n',
+    };
 
-      const offenses = await check(theme, [MatchingTranslations]);
+    const offenses = await check(theme, [MatchingTranslations]);
 
-      expect(offenses).to.be.of.length(0);
-    }
+    expect(offenses).to.be.of.length(0);
   });
 
   it('should not report offenses when nested translations match', async () => {
-    for (const prefix of ['', '.schema']) {
-      const theme = {
-        [`locales/en.default${prefix}.json`]: JSON.stringify({
-          hello: { world: 'Hello, world!' },
-        }),
-        [`locales/pt-BR${prefix}.json`]: JSON.stringify({
-          hello: { world: 'Olá, mundo!' },
-        }),
-        [`locales/fr${prefix}.json`]: JSON.stringify({
-          hello: { world: 'Bonjour, monde' },
-        }),
-      };
+    const theme = {
+      'app/translations/en.yml': 'en:\n  hello:\n    world: Hello, world!\n',
+      'app/translations/pt-BR.yml': 'pt-BR:\n  hello:\n    world: Olá, mundo!\n',
+      'app/translations/fr.yml': 'fr:\n  hello:\n    world: Bonjour, monde\n',
+    };
 
-      const offenses = await check(theme, [MatchingTranslations]);
+    const offenses = await check(theme, [MatchingTranslations]);
 
-      expect(offenses).to.be.of.length(0);
-    }
+    expect(offenses).to.be.of.length(0);
   });
 
   it('should not report offenses and ignore pluralization', async () => {
-    for (const prefix of ['', '.schema']) {
-      const theme = {
-        [`locales/en.default${prefix}.json`]: JSON.stringify({
-          hello: {
-            one: 'Hello, you',
-            other: "Hello, y'all",
-          },
-        }),
-        [`locales/pt-BR${prefix}.json`]: JSON.stringify({
-          hello: {
-            zero: 'Estou sozinho :(',
-            few: 'Olá, galerinha :)',
-          },
-        }),
-      };
-
-      const offenses = await check(theme, [MatchingTranslations]);
-
-      expect(offenses).to.be.of.length(0);
-    }
-  });
-
-  it('should not report offenses and ignore keys provided by Shopify', async () => {
-    for (const prefix of ['', '.schema']) {
-      const theme = {
-        [`locales/en.default${prefix}.json`]: JSON.stringify({
-          hello: 'Hello',
-          shopify: {
-            checkout: {
-              general: {
-                page_title: 'Checkout',
-              },
-            },
-          },
-        }),
-        [`locales/pt-BR${prefix}.json`]: JSON.stringify({
-          hello: 'Olá',
-          shopify: {
-            sentence: {
-              words_connector: 'hello world',
-            },
-          },
-        }),
-      };
-
-      const offenses = await check(theme, [MatchingTranslations]);
-
-      expect(offenses).to.be.of.length(0);
-    }
-  });
-
-  it('should not report offenses and ignore "*.schema.json" files', async () => {
     const theme = {
-      'locales/en.default.json': JSON.stringify({ hello: 'Hello' }),
-      'locales/pt-BR.schema.json': JSON.stringify({}),
+      'app/translations/en.yml': 'en:\n  hello:\n    one: Hello, you\n    other: "Hello, y\'all"\n',
+      'app/translations/pt-BR.yml':
+        'pt-BR:\n  hello:\n    zero: Estou sozinho :(\n    few: "Olá, galerinha :)"\n',
     };
 
     const offenses = await check(theme, [MatchingTranslations]);
@@ -278,123 +131,13 @@ describe('Module: MatchingTranslations', async () => {
     expect(offenses).to.be.of.length(0);
   });
 
-  it('should highlight the proper element when the translation file is missing a key', async () => {
-    for (const prefix of ['', '.schema']) {
-      const theme = {
-        [`locales/en.default${prefix}.json`]: JSON.stringify({
-          hello: 'Hello',
-          world: 'World',
-        }),
-        [`locales/pt-BR${prefix}.json`]: JSON.stringify({
-          hello: 'Olá',
-        }),
-      };
-
-      const offenses = await check(theme, [MatchingTranslations]);
-      const elements = highlightedOffenses(theme, offenses);
-
-      expect(elements).to.deep.eq(['{"hello":"Olá"}']);
-    }
-  });
-
-  it('should highlight the proper element when the default translation is missing a key', async () => {
-    for (const prefix of ['', '.schema']) {
-      const theme = {
-        [`locales/en.default${prefix}.json`]: JSON.stringify({
-          hello: 'Hello',
-        }),
-        [`locales/pt-BR${prefix}.json`]: JSON.stringify({
-          hello: 'Olá',
-          world: 'Mundo',
-        }),
-      };
-
-      const offenses = await check(theme, [MatchingTranslations]);
-      const elements = highlightedOffenses(theme, offenses);
-
-      expect(elements).to.deep.eq(['"world":"Mundo"']);
-    }
-  });
-
-  it('should highlight the proper element when nested translation keys do not exist', async () => {
-    for (const prefix of ['', '.schema']) {
-      const theme = {
-        [`locales/en.default${prefix}.json`]: JSON.stringify({
-          hello: {
-            world: 'Hello, world!',
-          },
-          welcome: 'Welcome',
-        }),
-        [`locales/pt-BR${prefix}.json`]: JSON.stringify({
-          hello: {},
-          welcome: 'Bem-vinda',
-        }),
-      };
-
-      const offenses = await check(theme, [MatchingTranslations]);
-      const elements = highlightedOffenses(theme, offenses);
-
-      expect(elements).to.deep.eq(['"hello":{}']);
-    }
-  });
-
-  it('should highlight the proper element when nested translation keys do not exist and there is a sibling node', async () => {
-    for (const prefix of ['', '.schema']) {
-      const theme = {
-        [`locales/en.default${prefix}.json`]: JSON.stringify({
-          hello: {
-            shopify: 'Shopify!',
-            world: 'Hello, world!',
-          },
-          welcome: 'Welcome',
-        }),
-        [`locales/pt-BR${prefix}.json`]: JSON.stringify({
-          hello: {
-            shopify: 'Shopify!',
-          },
-          welcome: 'Bem-vinda',
-        }),
-      };
-
-      const offenses = await check(theme, [MatchingTranslations]);
-      const elements = highlightedOffenses(theme, offenses);
-
-      expect(elements).to.deep.eq(['"hello":{"shopify":"Shopify!"}']);
-    }
-  });
-
-  it('should highlight the proper element when translation shapes do not match', async () => {
-    for (const prefix of ['', '.schema']) {
-      const theme = {
-        [`locales/en.default${prefix}.json`]: JSON.stringify({
-          hello: { world: 'Hello, world!' },
-        }),
-        [`locales/pt-BR${prefix}.json`]: JSON.stringify({
-          hello: 'Olá',
-        }),
-      };
-
-      const offenses = await check(theme, [MatchingTranslations]);
-      const elements = highlightedOffenses(theme, offenses);
-
-      // We have two elements because we have two offenses:
-      // - A default translation for 'hello' does not exist"
-      // - The translation for 'hello.world' is missing"
-      expect(elements).to.deep.eq(['"hello":"Olá"', '"hello":"Olá"']);
-    }
-  });
-
   it('should not highlight anything if the file is unparseable', async () => {
-    for (const prefix of ['', '.schema']) {
-      const theme = {
-        [`locales/en.default${prefix}.json`]: JSON.stringify({
-          hello: { world: 'Hello, world!' },
-        }),
-        [`locales/pt-BR${prefix}.json`]: `{"hello": }`,
-      };
+    const theme = {
+      'app/translations/en.yml': 'en:\n  hello:\n    world: Hello, world!\n',
+      'app/translations/pt-BR.yml': 'pt-BR:\n  hello: :\n  bad yaml',
+    };
 
-      const offenses = await check(theme, [MatchingTranslations]);
-      expect(offenses).to.have.length(0);
-    }
+    const offenses = await check(theme, [MatchingTranslations]);
+    expect(offenses).to.have.length(0);
   });
 });