@platformos/platformos-check-common 0.0.7 → 0.0.9

package/src/checks/parser-blocking-script/suggestions.ts

@@ -1,36 +1,9 @@
-import {
-  HtmlRawNode,
-  LiquidFilter,
-  LiquidVariable,
-  LiquidVariableOutput,
-} from '@platformos/liquid-html-parser';
+import { HtmlRawNode } from '@platformos/liquid-html-parser';
 import { LiquidHtmlSuggestion } from '../../types';
-import { last } from '../../utils';
 
 const suggestionMessage = (attr: 'defer' | 'async') =>
   `Use an HTML script tag with the ${attr} attribute instead`;
 
-export const liquidFilterSuggestion = (
-  attr: 'defer' | 'async',
-  node: LiquidFilter,
-  parentNode: LiquidVariable,
-  grandParentNode: LiquidVariableOutput,
-): LiquidHtmlSuggestion => ({
-  message: suggestionMessage(attr),
-  fix(corrector) {
-    const expression = node.source.slice(
-      parentNode.expression.position.start,
-      last(parentNode.filters, -1)?.position.end ?? node.position.start,
-    );
-    const url = `{{ ${expression} }}`;
-    corrector.replace(
-      grandParentNode.position.start,
-      grandParentNode.position.end,
-      `<script src="${url}" ${attr}></script>`,
-    );
-  },
-});
-
 export const scriptTagSuggestion = (
   attr: 'defer' | 'async',
   node: HtmlRawNode,

package/src/checks/translation-key-exists/index.ts

@@ -25,7 +25,7 @@ export const TranslationKeyExists: LiquidCheckDefinition = {
     docs: {
       description: 'Reports missing translation keys',
       recommended: true,
-      url: 'https://shopify.dev/docs/storefronts/themes/tools/theme-check/checks/translation-key-exists',
+      url: 'https://documentation.platformos.com/developer-guide/platformos-check/checks/translation-key-exists',
     },
     type: SourceCodeType.LiquidHtml,
     severity: Severity.ERROR,

package/src/checks/unclosed-html-element/index.ts

@@ -41,7 +41,7 @@ export const UnclosedHTMLElement: LiquidCheckDefinition = {
     docs: {
       description: 'Warns you of unbalanced HTML tags in branching code',
       recommended: true,
-      url: 'https://shopify.dev/docs/storefronts/themes/tools/theme-check/checks/unclosed-html-element',
+      url: 'https://documentation.platformos.com/developer-guide/platformos-check/checks/unclosed-html-element',
     },
     type: SourceCodeType.LiquidHtml,
     severity: Severity.WARNING,
@@ -272,6 +272,10 @@ function getConditionIdentifierForMarkup(condition: string | LiquidConditionalEx
         condition.relation,
         getConditionIdentifierForMarkup(condition.right),
       ].join(' ');
+    case NodeTypes.JsonHashLiteral:
+      return '{}';
+    case NodeTypes.JsonArrayLiteral:
+      return '[]';
     default: {
       return assertNever(condition);
     }

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" %}
@@ -256,7 +242,7 @@ describe('Module: UndefinedObject', () => {
     }
   });
 
-  it('should report an offense when object is undefined in a "snippet" file with doc tags that are missing the associated param', async () => {
+  it('should report an offense when object is undefined in a "partial" file with doc tags that are missing the associated param', async () => {
     const sourceCode = `
     {% doc %}
     {% enddoc %}
@@ -273,7 +259,7 @@ describe('Module: UndefinedObject', () => {
     expect(offenses.map((e) => e.message)).toEqual(["Unknown object 'my_var' used."]);
   });
 
-  it('should not report an offense when object is defined with @param in a snippet file', async () => {
+  it('should not report an offense when object is defined with @param in a partial file', async () => {
     const sourceCode = `
       {% doc %}
         @param {string} text
@@ -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 %}
@@ -516,4 +410,31 @@ describe('Module: UndefinedObject', () => {
     expect(offenses).toHaveLength(1);
     expect(offenses.map((e) => e.message)).toEqual(["Unknown object 'job_id' used."]);
   });
+
+  it('should not report an offense when object is defined with a parse_json tag', async () => {
+    const sourceCode = `
+      {% parse_json groups_data %}
+        { "hello": "world" }
+      {% endparse_json %}
+      {{ groups_data }}
+    `;
+
+    const offenses = await runLiquidCheck(UndefinedObject, sourceCode);
+
+    expect(offenses).toHaveLength(0);
+  });
+
+  it('should report an offense when parse_json variable is used before the tag', async () => {
+    const sourceCode = `
+      {{ groups_data }}
+      {% parse_json groups_data %}
+        { "hello": "world" }
+      {% endparse_json %}
+    `;
+
+    const offenses = await runLiquidCheck(UndefinedObject, sourceCode);
+
+    expect(offenses).toHaveLength(1);
+    expect(offenses[0].message).toBe("Unknown object 'groups_data' used.");
+  });
 });

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

@@ -16,10 +16,11 @@ import {
   FunctionMarkup,
   LiquidTagHashAssign,
   LiquidTagGraphQL,
+  LiquidTagParseJson,
   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 +33,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 +50,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[] = [];
@@ -76,7 +77,7 @@ export const UndefinedObject: LiquidCheckDefinition = {
       async LiquidTag(node, ancestors) {
         if (isWithinRawTagThatDoesNotParseItsContents(ancestors)) return;
 
-        if (isLiquidTagAssign(node) || isLiquidTagGraphQL(node)) {
+        if (isLiquidTagAssign(node) || isLiquidTagGraphQL(node) || isLiquidTagParseJson(node)) {
           indexVariableScope(node.markup.name, {
             start: node.blockStartPosition.end,
           });
@@ -99,7 +100,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,
@@ -158,6 +159,7 @@ export const UndefinedObject: LiquidCheckDefinition = {
 
         const parent = last(ancestors);
         if (isLiquidTag(parent) && isLiquidTagCapture(parent)) return;
+        if (isLiquidTag(parent) && isLiquidTagParseJson(parent)) return;
 
         // Skip the jobId variable in background tag markup - it's being defined, not used
         if (isBackgroundInlineMarkup(parent) && parent.jobId === node) return;
@@ -166,7 +168,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 +194,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 +211,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'];
   }
@@ -300,6 +277,10 @@ function isLiquidTagGraphQL(node: LiquidTag): node is LiquidTagGraphQL {
   return node.name === NamedTags.graphql && typeof node.markup !== 'string';
 }
 
+function isLiquidTagParseJson(node: LiquidTag): node is LiquidTagParseJson {
+  return node.name === NamedTags.parse_json && typeof node.markup !== 'string';
+}
+
 function isLiquidForTag(node: LiquidTag): node is LiquidTagFor {
   return node.name === NamedTags.for && typeof node.markup !== 'string';
 }

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

@@ -14,9 +14,9 @@ export const UnusedAssign: LiquidCheckDefinition = {
     name: 'Prevent unused assigns',
     docs: {
       description:
-        'This check exists to prevent bloat in themes by surfacing variable definitions that are not used.',
+        'This check exists to prevent bloat 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 {