@platformos/platformos-check-common 0.0.6 → 0.0.8

package/src/checks/undefined-object/index.spec.ts

@@ -205,20 +205,6 @@ describe('Module: UndefinedObject', () => {
     expect(offenses.map((e) => e.message)).toEqual(["Unknown object 'c' used."]);
   });
 
-  it('should contextually report on the undefined nature of the paginate object (defined in paginate tag, undefined outside)', async () => {
-    const sourceCode = `
-      {% assign col = 'string' | split: '' %}
-      {% paginate col by 5 %}
-        {{ paginate }}
-      {% endpaginate %}{{ paginate }}
-    `;
-
-    const offenses = await runLiquidCheck(UndefinedObject, sourceCode);
-
-    expect(offenses).toHaveLength(1);
-    expect(offenses.map((e) => e.message)).toEqual(["Unknown object 'paginate' used."]);
-  });
-
   it('should contextually report on the undefined nature of the form object (defined in form tag, undefined outside)', async () => {
     const sourceCode = `
       {% form "cart" %}
@@ -313,42 +299,9 @@ describe('Module: UndefinedObject', () => {
     }
   });
 
-  it('should support contextual exceptions', async () => {
-    let offenses: Offense[];
-    const contexts: [string, string][] = [
-      ['section', 'sections/section.liquid'],
-      ['predictive_search', 'sections/predictive-search.liquid'],
-      ['recommendations', 'sections/recommendations.liquid'],
-      ['comment', 'sections/main-article.liquid'],
-      ['block', 'blocks/theme-app-extension.liquid'],
-      ['app', 'blocks/theme-app-extension.liquid'],
-      ['app', 'app/views/partials/theme-app-extension.liquid'],
-    ];
-    for (const [object, goodPath] of contexts) {
-      offenses = await runLiquidCheck(UndefinedObject, `{{ ${object} }}`, goodPath);
-      expect(offenses).toHaveLength(0);
-      offenses = await runLiquidCheck(UndefinedObject, `{{ ${object} }}`, 'file.liquid');
-      expect(offenses).toHaveLength(1);
-    }
-  });
-
-  it('should support contextual exceptions for checkout.liquid', async () => {
+  it('should support contextual exceptions for partials', async () => {
     let offenses: Offense[];
-    const contexts: [string, string][] = [
-      ['locale', 'layout/checkout.liquid'],
-      ['direction', 'layout/checkout.liquid'],
-      ['skip_to_content_link', 'layout/checkout.liquid'],
-      ['checkout_html_classes', 'layout/checkout.liquid'],
-      ['checkout_stylesheets', 'layout/checkout.liquid'],
-      ['checkout_scripts', 'layout/checkout.liquid'],
-      ['content_for_logo', 'layout/checkout.liquid'],
-      ['breadcrumb', 'layout/checkout.liquid'],
-      ['order_summary_toggle', 'layout/checkout.liquid'],
-      ['content_for_order_summary', 'layout/checkout.liquid'],
-      ['alternative_payment_methods', 'layout/checkout.liquid'],
-      ['content_for_footer', 'layout/checkout.liquid'],
-      ['tracking_code', 'layout/checkout.liquid'],
-    ];
+    const contexts: [string, string][] = [['app', 'app/views/partials/theme-app-extension.liquid']];
     for (const [object, goodPath] of contexts) {
       offenses = await runLiquidCheck(UndefinedObject, `{{ ${object} }}`, goodPath);
       expect(offenses).toHaveLength(0);
@@ -374,44 +327,12 @@ describe('Module: UndefinedObject', () => {
     `;
 
     const offenses = await runLiquidCheck(UndefinedObject, sourceCode, 'file.liquid', {
-      themeDocset: undefined,
+      platformosDocset: undefined,
     });
 
     expect(offenses).toHaveLength(0);
   });
 
-  it('should not report an offense when object is defined with @param in a block file', async () => {
-    const sourceCode = `
-      {% doc %}
-        @param {string} text
-      {% enddoc %}
-
-      {{ text }}
-    `;
-
-    const filePath = 'blocks/my-custom-block.liquid';
-    const offenses = await runLiquidCheck(UndefinedObject, sourceCode, filePath);
-
-    expect(offenses).toHaveLength(0);
-  });
-
-  it('should report an offense when an undefined object is used alongside @param in a block file', async () => {
-    const sourceCode = `
-      {% doc %}
-        @param {string} text
-      {% enddoc %}
-
-      {{ text }}
-      {{ undefined_variable }}
-    `;
-
-    const filePath = 'blocks/my-custom-block.liquid';
-    const offenses = await runLiquidCheck(UndefinedObject, sourceCode, filePath);
-
-    expect(offenses).toHaveLength(1);
-    expect(offenses[0].message).toBe("Unknown object 'undefined_variable' used.");
-  });
-
   it('should not report an offense when a self defined variable is defined with a @param tag', async () => {
     const sourceCode = `
       {% doc %}
@@ -438,33 +359,6 @@ describe('Module: UndefinedObject', () => {
     expect(offenses[0].message).toBe("Unknown object 'my_var' used.");
   });
 
-  it('should not report an offense when using variables inside visible_if statements in a schema tag', async () => {
-    const sourceCode = `
-    {% schema %}
-    {
-      "settings": [
-        {
-          "type": "text",
-          "label": "foo",
-          "id": "foo",
-        }
-        {
-          "type": "text",
-          "label": "bar",
-          "id": "bar",
-          "visible_if": "{{ block.settings.foo }}",
-        }
-      ]
-    }
-    {% endschema %}
-    `;
-
-    const offenses = await runLiquidCheck(UndefinedObject, sourceCode);
-
-    assert(offenses.length == 0);
-    expect(offenses).to.be.empty;
-  });
-
   it('should report an offense when job_id is used inside background block', async () => {
     const sourceCode = `
       {% background job_id %}

package/src/checks/undefined-object/index.ts

@@ -19,7 +19,7 @@ import {
   LiquidTagBackground,
   BackgroundInlineMarkup,
 } from '@platformos/liquid-html-parser';
-import { LiquidCheckDefinition, Severity, SourceCodeType, ThemeDocset } from '../../types';
+import { LiquidCheckDefinition, Severity, SourceCodeType, PlatformOSDocset } from '../../types';
 import { isError, last } from '../../utils';
 import { isWithinRawTagThatDoesNotParseItsContents } from '../utils';
 
@@ -32,7 +32,7 @@ export const UndefinedObject: LiquidCheckDefinition = {
     docs: {
       description: 'This check exists to identify references to undefined Liquid objects.',
       recommended: true,
-      url: 'https://shopify.dev/docs/storefronts/themes/tools/theme-check/checks/undefined-object',
+      url: 'https://documentation.platformos.com/developer-guide/platformos-check/checks/undefined-object',
     },
     type: SourceCodeType.LiquidHtml,
     severity: Severity.WARNING,
@@ -49,11 +49,11 @@ export const UndefinedObject: LiquidCheckDefinition = {
     /**
      * Skip this check when definitions for global objects are unavailable.
      */
-    if (!context.themeDocset) {
+    if (!context.platformosDocset) {
       return {};
     }
 
-    const themeDocset = context.themeDocset;
+    const platformosDocset = context.platformosDocset;
     const scopedVariables: Map<string, Scope[]> = new Map();
     const fileScopedVariables: Set<string> = new Set();
     const variables: LiquidVariableLookup[] = [];
@@ -99,7 +99,7 @@ export const UndefinedObject: LiquidCheckDefinition = {
          *   {{ form }}
          * {% endform %}
          */
-        if (['form', 'paginate'].includes(node.name)) {
+        if (node.name === 'form') {
           indexVariableScope(node.name, {
             start: node.blockStartPosition.end,
             end: node.blockEndPosition?.start,
@@ -166,7 +166,7 @@ export const UndefinedObject: LiquidCheckDefinition = {
       },
 
       async onCodePathEnd() {
-        const objects = await globalObjects(themeDocset, relativePath);
+        const objects = await globalObjects(platformosDocset, relativePath);
 
         objects.forEach((obj) => fileScopedVariables.add(obj.name));
 
@@ -192,8 +192,8 @@ export const UndefinedObject: LiquidCheckDefinition = {
   },
 };
 
-async function globalObjects(themeDocset: ThemeDocset, relativePath: string) {
-  const objects = await themeDocset.objects();
+async function globalObjects(platformosDocset: PlatformOSDocset, relativePath: string) {
+  const objects = await platformosDocset.objects();
   const contextualObjects = getContextualObjects(relativePath);
 
   const globalObjects = objects.filter(({ access, name }) => {
@@ -209,31 +209,6 @@ async function globalObjects(themeDocset: ThemeDocset, relativePath: string) {
 }
 
 function getContextualObjects(relativePath: string): string[] {
-  if (relativePath.startsWith('layout/checkout.liquid')) {
-    return [
-      'locale',
-      'direction',
-      'skip_to_content_link',
-      'checkout_html_classes',
-      'checkout_stylesheets',
-      'checkout_scripts',
-      'content_for_logo',
-      'breadcrumb',
-      'order_summary_toggle',
-      'content_for_order_summary',
-      'alternative_payment_methods',
-      'content_for_footer',
-      'tracking_code',
-    ];
-  }
-  if (relativePath.startsWith('sections/')) {
-    return ['section', 'predictive_search', 'recommendations', 'comment'];
-  }
-
-  if (relativePath.startsWith('blocks/')) {
-    return ['app', 'section', 'recommendations', 'block'];
-  }
-
   if (relativePath.includes('views/partials/') || relativePath.includes('/lib/')) {
     return ['app'];
   }

package/src/checks/unique-doc-param-names/index.ts

@@ -8,7 +8,7 @@ export const UniqueDocParamNames: LiquidCheckDefinition = {
       description:
         'This check exists to ensure any parameter names defined in the `doc` tag are unique.',
       recommended: true,
-      url: 'https://shopify.dev/docs/storefronts/themes/tools/theme-check/checks/unique-doc-param-names',
+      url: 'https://documentation.platformos.com/developer-guide/platformos-check/checks/unique-doc-param-names',
     },
     type: SourceCodeType.LiquidHtml,
     severity: Severity.ERROR,

package/src/checks/unknown-filter/index.spec.ts

@@ -46,13 +46,13 @@ describe('Module: UnknownFilter', () => {
   });
 
   it('should not report an offense when a known filter is used (1)', async () => {
-    const sourceCode = `{{ 'hello' | link_to_type }}`;
+    const sourceCode = `{{ 'hello' | parameterize }}`;
     const offenses = await runLiquidCheck(UnknownFilter, sourceCode);
     expect(offenses).toHaveLength(0);
   });
 
   it('should not report an offense when a known filter is used (2)', async () => {
-    const sourceCode = `{{ 'hello' | link_to_vendor }}`;
+    const sourceCode = `{{ 'hello' | slugify }}`;
     const offenses = await runLiquidCheck(UnknownFilter, sourceCode);
     expect(offenses).toHaveLength(0);
   });

package/src/checks/unknown-filter/index.ts

@@ -7,7 +7,7 @@ export const UnknownFilter: LiquidCheckDefinition = {
     name: 'Prevent use of unknown filters',
     docs: {
       description: 'This check is aimed at preventing the use of unknown filters.',
-      url: 'https://shopify.dev/docs/storefronts/themes/tools/theme-check/checks/unknown-filter',
+      url: 'https://documentation.platformos.com/developer-guide/platformos-check/checks/unknown-filter',
       recommended: true,
     },
     type: SourceCodeType.LiquidHtml,
@@ -17,13 +17,13 @@ export const UnknownFilter: LiquidCheckDefinition = {
   },
 
   create(context) {
-    if (!context.themeDocset) {
+    if (!context.platformosDocset) {
       return {};
     }
 
     return {
       async LiquidFilter(node) {
-        const knownFilters = await context.themeDocset!.filters();
+        const knownFilters = await context.platformosDocset!.filters();
 
         if (!knownFilters.some((filter) => filter.name === node.name)) {
           context.report({

package/src/checks/unknown-property/index.ts

@@ -56,7 +56,7 @@ export const UnknownProperty: LiquidCheckDefinition = {
     let graphqlSchema: string | undefined;
     const getGraphQLSchema = async (): Promise<string | undefined> => {
       if (graphqlSchema === undefined) {
-        graphqlSchema = (await context.themeDocset?.graphQL()) ?? undefined;
+        graphqlSchema = (await context.platformosDocset?.graphQL()) ?? undefined;
       }
       return graphqlSchema;
     };

package/src/checks/unrecognized-render-partial-arguments/index.spec.ts

@@ -1,5 +1,5 @@
 import { describe, it, expect } from 'vitest';
-import { applySuggestions, MockTheme, runLiquidCheck } from '../../test';
+import { applySuggestions, MockApp, runLiquidCheck } from '../../test';
 import { UnrecognizedRenderPartialArguments } from '.';
 
 function check(partial: string, source: string) {
@@ -113,14 +113,14 @@ describe('Module: UnrecognizedRenderPartialParams', () => {
     });
 
     it('should report when "with/for" alias syntax is used', async () => {
-      const mockTheme = {
+      const mockApp = {
         'app/views/partials/card.liquid': `
           {% doc %}
             @param {string} title - The title of the card
           {% enddoc %}
           <div>{{ title }}</div>
         `,
-      } as MockTheme;
+      } as MockApp;
 
       let sourceCode = `{% render 'card' with 'my-card' as unknown_param %}`;
       let offenses = await runLiquidCheck(
@@ -128,7 +128,7 @@ describe('Module: UnrecognizedRenderPartialParams', () => {
         sourceCode,
         undefined,
         {},
-        mockTheme,
+        mockApp,
       );
 
       expect(offenses).toHaveLength(1);
@@ -146,7 +146,7 @@ describe('Module: UnrecognizedRenderPartialParams', () => {
         sourceCode,
         undefined,
         {},
-        mockTheme,
+        mockApp,
       );
 
       expect(offenses).toHaveLength(1);

package/src/checks/unrecognized-render-partial-arguments/index.ts

@@ -16,7 +16,7 @@ export const UnrecognizedRenderPartialArguments: LiquidCheckDefinition = {
       description:
         'This check ensures that no unknown arguments are used when rendering a partial.',
       recommended: true,
-      url: 'https://shopify.dev/docs/storefronts/themes/tools/theme-check/checks/unrecognized-render-partial-arguments',
+      url: 'https://documentation.platformos.com/developer-guide/platformos-check/checks/unrecognized-render-partial-arguments',
     },
     type: SourceCodeType.LiquidHtml,
     severity: Severity.WARNING,
@@ -60,10 +60,7 @@ export const UnrecognizedRenderPartialArguments: LiquidCheckDefinition = {
 
         if (!partialName) return;
 
-        const liquidDocParameters = await getLiquidDocParams(
-          context,
-          `app/views/partials/${partialName}.liquid`,
-        );
+        const liquidDocParameters = await getLiquidDocParams(context, partialName);
 
         if (!liquidDocParameters) return;
 

package/src/checks/unused-assign/index.spec.ts

@@ -84,36 +84,6 @@ describe('Module: UnusedAssign', () => {
     expect(offenses).not.to.be.empty;
   });
 
-  it('should not report unused assigns for things used in a raw-like tag', async () => {
-    const tags = ['style'];
-    for (const tag of tags) {
-      const sourceCode = `
-        {% assign usedVar = 1 %}
-        {% ${tag} %}
-          {{ usedVar }}
-        {% end${tag} %}
-      `;
-
-      const offenses = await runLiquidCheck(UnusedAssign, sourceCode);
-      expect(offenses).to.be.empty;
-    }
-  });
-
-  it('should report unused assigns for things used in raw code that gets stripped away (schema, etc)', async () => {
-    const tags = ['schema', 'javascript', 'stylesheet'];
-    for (const tag of tags) {
-      const sourceCode = `
-        {% assign usedVar = 1 %}
-        {% ${tag} %}
-          {{ usedVar }}
-        {% end${tag} %}
-      `;
-
-      const offenses = await runLiquidCheck(UnusedAssign, sourceCode);
-      expect(offenses).to.not.be.empty;
-    }
-  });
-
   it('should not report unused assigns for things used in a HTML raw-like tag', async () => {
     const tags = ['style', 'script'];
     for (const tag of tags) {

package/src/checks/unused-assign/index.ts

@@ -16,7 +16,7 @@ export const UnusedAssign: LiquidCheckDefinition = {
       description:
         'This check exists to prevent bloat in themes by surfacing variable definitions that are not used.',
       recommended: true,
-      url: 'https://shopify.dev/docs/storefronts/themes/tools/theme-check/checks/unused-assign',
+      url: 'https://documentation.platformos.com/developer-guide/platformos-check/checks/unused-assign',
     },
     type: SourceCodeType.LiquidHtml,
     severity: Severity.WARNING,

package/src/checks/unused-doc-param/index.ts

@@ -10,7 +10,7 @@ export const UnusedDocParam: LiquidCheckDefinition = {
       description:
         'This check exists to ensure any parameters defined in the `doc` tag are used within the partial.',
       recommended: true,
-      url: 'https://shopify.dev/docs/storefronts/themes/tools/theme-check/checks/unused-doc-param',
+      url: 'https://documentation.platformos.com/developer-guide/platformos-check/checks/unused-doc-param',
     },
     type: SourceCodeType.LiquidHtml,
     severity: Severity.WARNING,

package/src/checks/utils.ts

@@ -104,7 +104,7 @@ export function isLoopLiquidTag(tag: LiquidTag): tag is LiquidTagFor | LiquidTag
   return LoopNamedTags.includes(tag.name as any);
 }
 
-const RawTagsThatDoNotParseTheirContents = ['raw', 'stylesheet', 'javascript', 'schema'];
+const RawTagsThatDoNotParseTheirContents = ['raw'];
 
 function isRawTagThatDoesNotParseItsContent(node: LiquidHtmlNode) {
   return (

package/src/checks/valid-doc-param-types/index.ts

@@ -9,7 +9,7 @@ export const ValidDocParamTypes: LiquidCheckDefinition = {
       description:
         'This check exists to ensure any parameter types defined in the `doc` tag are valid.',
       recommended: true,
-      url: 'https://shopify.dev/docs/storefronts/themes/tools/theme-check/checks/valid-doc-param-types',
+      url: 'https://documentation.platformos.com/developer-guide/platformos-check/checks/valid-doc-param-types',
     },
     type: SourceCodeType.LiquidHtml,
     severity: Severity.ERROR,
@@ -18,14 +18,14 @@ export const ValidDocParamTypes: LiquidCheckDefinition = {
   },
 
   create(context) {
-    if (!context.themeDocset) {
+    if (!context.platformosDocset) {
       return {};
     }
 
-    // To avoid recalculating valid param types during theme-check, constructing
+    // To avoid recalculating valid param types during platformos-check, constructing
     // the promise beforehand.
     const validParamTypesPromise = context
-      .themeDocset!.liquidDrops()
+      .platformosDocset!.liquidDrops()
       .then((entries) => new Set(getValidParamTypes(entries).keys()));
 
     return {

package/src/checks/valid-html-translation/index.spec.ts

@@ -1,73 +1,83 @@
 import { ValidHTMLTranslation } from './index';
-import { highlightedOffenses, runJSONCheck } from '../../test';
+import { highlightedOffenses, runYAMLCheck } from '../../test';
 import { expect, describe, it } from 'vitest';
 
 describe('Module: ValidHTMLTranslation', () => {
   it('should not report valid HTML in translations', async () => {
-    const sourceCode = JSON.stringify({
-      hello_html: '<h1>Hello, world</h1>',
-      image_html: "<img src='spongebob.png'>",
-      line_break_html: '<br>',
-      self_closing_svg_html: '<svg />',
-      foo: 'bar',
-    });
+    const sourceCode = [
+      'en:',
+      '  hello_html: "<h1>Hello, world</h1>"',
+      '  image_html: "<img src=\'spongebob.png\'>"',
+      '  line_break_html: "<br>"',
+      '  self_closing_svg_html: "<svg />"',
+      '  foo: bar',
+    ].join('\n');
 
-    const offenses = await runJSONCheck(ValidHTMLTranslation, sourceCode, 'locales/en.json');
+    const offenses = await runYAMLCheck(
+      ValidHTMLTranslation,
+      sourceCode,
+      'app/translations/en.yml',
+    );
     expect(offenses).toHaveLength(0);
   });
 
   it('should report invalid HTML in translations', async () => {
-    const sourceCode = JSON.stringify({
-      hello_html: '<h1>Hello, world',
-    });
+    const sourceCode = 'en:\n  hello_html: "<h1>Hello, world"';
 
-    const offenses = await runJSONCheck(ValidHTMLTranslation, sourceCode, 'locales/en.json');
+    const offenses = await runYAMLCheck(
+      ValidHTMLTranslation,
+      sourceCode,
+      'app/translations/en.yml',
+    );
     expect(offenses).toHaveLength(1);
     expect(offenses[0].message).to.equal(
       "LiquidHTMLParsingError: Attempting to end parsing before HtmlElement 'h1' was closed.",
     );
 
-    const highlights = highlightedOffenses({ 'locales/en.json': sourceCode }, offenses);
+    const highlights = highlightedOffenses({ 'app/translations/en.yml': sourceCode }, offenses);
     expect(highlights).toHaveLength(1);
     expect(highlights[0]).to.equal(`"<h1>Hello, world"`);
   });
 
   it('should report nested invalid HTML in translations', async () => {
-    const sourceCode = JSON.stringify({
-      hello: {
-        world: {
-          today: {
-            good: {
-              day_html: '<h1>Hello, world',
-            },
-          },
-        },
-      },
-    });
+    const sourceCode = [
+      'en:',
+      '  hello:',
+      '    world:',
+      '      today:',
+      '        good:',
+      '          day_html: "<h1>Hello, world"',
+    ].join('\n');
 
-    const offenses = await runJSONCheck(ValidHTMLTranslation, sourceCode, 'locales/en.json');
+    const offenses = await runYAMLCheck(
+      ValidHTMLTranslation,
+      sourceCode,
+      'app/translations/en.yml',
+    );
     expect(offenses).toHaveLength(1);
     expect(offenses[0].message).to.equal(
       "LiquidHTMLParsingError: Attempting to end parsing before HtmlElement 'h1' was closed.",
     );
 
-    const highlights = highlightedOffenses({ 'locales/en.json': sourceCode }, offenses);
+    const highlights = highlightedOffenses({ 'app/translations/en.yml': sourceCode }, offenses);
     expect(highlights).toHaveLength(1);
     expect(highlights[0]).to.equal(`"<h1>Hello, world"`);
   });
 
   it('should report pluralized key with invalid HTML', async () => {
-    const sourceCode = JSON.stringify({
-      hello_html: { one: '<h1>Hello, world' },
-    });
+    const sourceCode = 'en:\n  hello_html:\n    one: "<h1>Hello, world"';
 
-    const offenses = await runJSONCheck(ValidHTMLTranslation, sourceCode, 'locales/en.json');
+    const offenses = await runYAMLCheck(
+      ValidHTMLTranslation,
+      sourceCode,
+      'app/translations/en.yml',
+    );
     expect(offenses).toHaveLength(1);
     expect(offenses[0].message).to.equal(
       "LiquidHTMLParsingError: Attempting to end parsing before HtmlElement 'h1' was closed.",
     );
 
-    const highlights = highlightedOffenses({ 'locales/en.json': sourceCode }, offenses);
+    const highlights = highlightedOffenses({ 'app/translations/en.yml': sourceCode }, offenses);
     expect(highlights).toHaveLength(1);
     expect(highlights[0]).to.equal(`"<h1>Hello, world"`);
   });

package/src/checks/valid-html-translation/index.ts

@@ -1,25 +1,25 @@
-import { SourceCodeType, JSONCheckDefinition, Severity, Problem, LiteralNode } from '../../types';
+import { SourceCodeType, YAMLCheckDefinition, Severity, Problem, LiteralNode } from '../../types';
 import { toLiquidHtmlAST } from '@platformos/liquid-html-parser';
 
-export const ValidHTMLTranslation: JSONCheckDefinition = {
+export const ValidHTMLTranslation: YAMLCheckDefinition = {
   meta: {
     code: 'ValidHTMLTranslation',
     name: 'Valid HTML Translation',
     docs: {
       description: 'This check exists to prevent invalid HTML inside translations.',
-      url: 'https://shopify.dev/docs/storefronts/themes/tools/theme-check/checks/valid-html-translation',
+      url: 'https://documentation.platformos.com/developer-guide/platformos-check/checks/valid-html-translation',
       recommended: true,
     },
-    type: SourceCodeType.JSON,
+    type: SourceCodeType.YAML,
     severity: Severity.WARNING,
     schema: {},
     targets: [],
   },
 
   create(context) {
-    // We ignore non-`locales/` json files.
+    // We ignore yaml files that aren't translation files.
     const relativePath = context.toRelativePath(context.file.uri);
-    if (!relativePath.startsWith('locales/')) return {};
+    if (!relativePath.includes('/translations/')) return {};
 
     return {
       async Literal(node: LiteralNode) {
@@ -32,7 +32,7 @@ export const ValidHTMLTranslation: JSONCheckDefinition = {
         } catch (error) {
           const loc = node.loc;
 
-          const problem: Problem<SourceCodeType.JSON> = {
+          const problem: Problem<SourceCodeType.YAML> = {
             message: `${error}.`,
             startIndex: loc.start.offset,
             endIndex: loc.end.offset,

package/src/checks/valid-json/index.ts

@@ -9,7 +9,7 @@ export const ValidJSON: JSONCheckDefinition = {
       description:
         'This check exists to prevent invalid JSON files in themes. Will check against schema if available.',
       recommended: true,
-      url: 'https://shopify.dev/docs/storefronts/themes/tools/theme-check/checks/json-syntax-error',
+      url: 'https://documentation.platformos.com/developer-guide/platformos-check/checks/json-syntax-error',
     },
     type: SourceCodeType.JSON,
     severity: Severity.ERROR,

package/src/checks/valid-render-partial-argument-types/index.ts

@@ -19,7 +19,7 @@ export const ValidRenderPartialArgumentTypes: LiquidCheckDefinition = {
       description:
         'This check ensures that arguments passed to partial match the expected types defined in the liquidDoc header if present.',
       recommended: true,
-      url: 'https://shopify.dev/docs/storefronts/themes/tools/theme-check/checks/valid-render-partial-argument-types',
+      url: 'https://documentation.platformos.com/developer-guide/platformos-check/checks/valid-render-partial-argument-types',
     },
     type: SourceCodeType.LiquidHtml,
     severity: Severity.WARNING,
@@ -72,10 +72,7 @@ export const ValidRenderPartialArgumentTypes: LiquidCheckDefinition = {
 
         if (!partialName) return;
 
-        const liquidDocParameters = await getLiquidDocParams(
-          context,
-          `app/views/partials/${partialName}.liquid`,
-        );
+        const liquidDocParameters = await getLiquidDocParams(context, partialName);
 
         if (!liquidDocParameters) return;
 

package/src/checks/variable-name/index.ts

@@ -48,7 +48,7 @@ export const VariableName: LiquidCheckDefinition<typeof schema> = {
     name: 'Invalid variable naming format',
     docs: {
       description: 'This check is aimed at using certain variable naming conventions',
-      url: 'https://shopify.dev/docs/storefronts/themes/tools/theme-check/checks/variable-name',
+      url: 'https://documentation.platformos.com/developer-guide/platformos-check/checks/variable-name',
       recommended: true,
     },
     type: SourceCodeType.LiquidHtml,