@platformos/platformos-check-common 0.0.12 → 0.0.13

package/CHANGELOG.md

@@ -1,5 +1,36 @@
 # @platformos/theme-check-common
 
+## 0.0.13
+
+### Patch Changes
+
+- **MissingRenderPartialArguments**: Reports an error when required `@param` arguments declared in a partial's LiquidDoc are not provided at the `{% render %}` call site.
+- **NestedGraphQLQuery**: Detects N+1 query patterns — `{% graphql %}` tags inside `{% for %}`/`{% tablerow %}` loops. Also follows `{% function %}` and `{% render %}` calls transitively to detect indirect GraphQL queries. Skips loops wrapped in `{% cache %}` or `{% background %}`.
+- Added **GraphQLFieldCompletionProvider**: Provides completions for GraphQL field names.
+- Added **GraphQLFieldHoverProvider**: Shows hover documentation for GraphQL fields.
+- Added `theme_render_rc` as a new document type, enabling the `{% theme_render_rc %}` tag to resolve partials through configurable `theme_search_paths` defined in `app/config.yml`.
+- **DocumentsLocator**: New `locateWithSearchPaths()` method resolves partials using prioritized search paths, including dynamic paths with `{{ }}` Liquid expressions that expand by enumerating subdirectories.
+- **loadSearchPaths()**: New utility to read and parse `theme_search_paths` from `app/config.yml`.
+- **TranslationKeyExists**: Refactored to load all defined keys (app-level and module-level) in a single pass. Now suggests nearest matching keys using Levenshtein distance when a translation key is not found.
+- Extracted shared translation utilities into `translation-utils.ts` for module discovery and key loading.
+- Added `levenshtein.ts` utility for fuzzy key matching.
+- Added support for `{% try %}...{% catch error %}` — the error variable in catch branches is now correctly registered as defined, preventing false-positive "undefined object" warnings.
+- `null`/`nil` literals are now treated as compatible with any `@param` type, preventing false type-mismatch errors when passing null values to partials.
+- `recursiveReadDirectory` now gracefully handles `ENOENT` errors instead of crashing when a directory doesn't exist.
+- **MissingPartial** check updated to support `theme_render_rc` tag resolution through search paths.
+- Extracted `tryExtractAssignUrl()` helper to deduplicate assign-to-URL resolution logic shared between `MissingPage` check and `buildVariableMap`.
+- Fixed `buildVariableMap` to correctly recurse into block tags (`{% if %}`, `{% for %}`) whose position spans beyond the cursor offset — previously assigns inside such blocks could be missed.
+- **SearchPathsLoader**: Now caches `theme_search_paths` per root URI to avoid re-reading `app/config.yml` on every request. Invalidated when file watchers detect config changes.
+- Immediate cache invalidation on `app/config.yml` save (via `onDidSaveTextDocument`) so go-to-definition doesn't see stale data.
+- Bulk file-watcher threshold extracted to `BULK_PAGE_CHANGE_THRESHOLD` constant.
+- **RouteTable**: Added `routeCount()` method returning total number of route entries.
+- Route table build errors are now properly handled — a failed build resets the cached promise so subsequent attempts can retry.
+- `MissingPartial` check simplified with a shared `reportIfMissing()` helper, reducing code duplication across `RenderMarkup`, `FunctionMarkup`, and `GraphQLMarkup` visitors.
+- AST traversal helpers (`getTraversableChildren`, `getTraversableMarkup`) extracted in `url-helpers.ts`.
+- `MissingPage` check front-loads route table building in `onCodePathStart` instead of lazy-loading per element visit.
+- Updated dependencies
+  - @platformos/liquid-html-parser@0.0.12
+
 ## 0.0.12
 
 ### Patch Changes

package/dist/checks/circular-render/index.d.ts

@@ -0,0 +1,2 @@
+import { LiquidCheckDefinition } from '../../types';
+export declare const CircularRender: LiquidCheckDefinition;

package/dist/checks/circular-render/index.js

@@ -0,0 +1,164 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.CircularRender = void 0;
+const liquid_html_parser_1 = require("@platformos/liquid-html-parser");
+const types_1 = require("../../types");
+const platformos_common_1 = require("@platformos/platformos-common");
+const vscode_uri_1 = require("vscode-uri");
+const MAX_DEPTH = 50;
+// Module-level cache: URI -> partial references extracted from that file.
+// Shared across all files in a check run to avoid re-parsing.
+const parseCacheMap = new WeakMap();
+function getParseCache(context) {
+    if (!parseCacheMap.has(context.fs)) {
+        parseCacheMap.set(context.fs, new Map());
+    }
+    return parseCacheMap.get(context.fs);
+}
+/**
+ * Extract partial references (render/function/include) from a Liquid source string.
+ * Returns an empty array on parse errors.
+ */
+function extractPartialRefs(source) {
+    let ast;
+    try {
+        ast = (0, liquid_html_parser_1.toLiquidHtmlAST)(source);
+    }
+    catch {
+        return [];
+    }
+    const refs = [];
+    const stack = [ast];
+    while (stack.length > 0) {
+        const node = stack.pop();
+        if (!node || typeof node !== 'object')
+            continue;
+        if (node.type === liquid_html_parser_1.NodeTypes.LiquidTag &&
+            node.markup &&
+            typeof node.markup === 'object' &&
+            (node.markup.type === liquid_html_parser_1.NodeTypes.RenderMarkup ||
+                node.markup.type === liquid_html_parser_1.NodeTypes.FunctionMarkup) &&
+            node.markup.partial &&
+            node.markup.partial.type !== liquid_html_parser_1.NodeTypes.VariableLookup) {
+            refs.push({
+                name: node.markup.partial.value,
+                documentType: node.name || 'render',
+                startIndex: node.markup.partial.position.start,
+                endIndex: node.markup.partial.position.end,
+            });
+        }
+        // Traverse children (skip circular references like parentNode, prev, next)
+        for (const key of Object.keys(node)) {
+            if (liquid_html_parser_1.nonTraversableProperties.has(key))
+                continue;
+            const value = node[key];
+            if (Array.isArray(value)) {
+                for (const child of value) {
+                    if (child && typeof child === 'object') {
+                        stack.push(child);
+                    }
+                }
+            }
+            else if (value && typeof value === 'object' && value.type) {
+                stack.push(value);
+            }
+        }
+    }
+    return refs;
+}
+exports.CircularRender = {
+    meta: {
+        code: 'CircularRender',
+        name: 'Prevent circular renders',
+        docs: {
+            description: 'Reports circular render/function/include chains that would cause infinite loops at runtime.',
+            recommended: true,
+            url: 'https://documentation.platformos.com/developer-guide/platformos-check/checks/circular-render',
+        },
+        type: types_1.SourceCodeType.LiquidHtml,
+        severity: types_1.Severity.ERROR,
+        schema: {},
+        targets: [],
+    },
+    create(context) {
+        const locator = new platformos_common_1.DocumentsLocator(context.fs);
+        const rootUri = vscode_uri_1.URI.parse(context.config.rootUri);
+        const parseCache = getParseCache(context);
+        const collectedRefs = [];
+        return {
+            async RenderMarkup(node, ancestors) {
+                if (node.partial.type === liquid_html_parser_1.NodeTypes.VariableLookup)
+                    return;
+                const parent = ancestors.at(-1);
+                const documentType = parent?.name === 'include' ? 'include' : 'render';
+                collectedRefs.push({
+                    name: node.partial.value,
+                    documentType,
+                    startIndex: node.partial.position.start,
+                    endIndex: node.partial.position.end,
+                });
+            },
+            async FunctionMarkup(node) {
+                if (node.partial.type === liquid_html_parser_1.NodeTypes.VariableLookup)
+                    return;
+                collectedRefs.push({
+                    name: node.partial.value,
+                    documentType: 'function',
+                    startIndex: node.partial.position.start,
+                    endIndex: node.partial.position.end,
+                });
+            },
+            async onCodePathEnd() {
+                const fileUri = context.file.uri;
+                for (const ref of collectedRefs) {
+                    const cycle = await findCycle(locator, rootUri, context.fs, parseCache, fileUri, ref.name, ref.documentType, [fileUri], 0);
+                    if (cycle) {
+                        const cyclePath = cycle
+                            .map((u) => {
+                            const parts = u.split('/');
+                            return parts.slice(-2).join('/');
+                        })
+                            .join(' -> ');
+                        context.report({
+                            message: `Circular render detected: ${cyclePath}. This will cause an infinite loop at runtime.`,
+                            startIndex: ref.startIndex,
+                            endIndex: ref.endIndex,
+                        });
+                    }
+                }
+            },
+        };
+    },
+};
+async function findCycle(locator, rootUri, fs, parseCache, originUri, partialName, documentType, path, depth) {
+    if (depth >= MAX_DEPTH)
+        return null;
+    const uri = await locator.locate(rootUri, documentType, partialName);
+    if (!uri)
+        return null;
+    if (uri === originUri) {
+        return [...path, uri];
+    }
+    if (path.includes(uri)) {
+        // This is a cycle, but it doesn't include the origin file — skip it
+        return null;
+    }
+    let refs = parseCache.get(uri);
+    if (!refs) {
+        try {
+            const source = await fs.readFile(uri);
+            refs = extractPartialRefs(source);
+        }
+        catch {
+            refs = [];
+        }
+        parseCache.set(uri, refs);
+    }
+    for (const dep of refs) {
+        const cycle = await findCycle(locator, rootUri, fs, parseCache, originUri, dep.name, dep.documentType, [...path, uri], depth + 1);
+        if (cycle)
+            return cycle;
+    }
+    return null;
+}
+//# sourceMappingURL=index.js.map

package/dist/checks/circular-render/index.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../../../src/checks/circular-render/index.ts"],"names":[],"mappings":";;;AAAA,uEAIwC;AACxC,uCAA8E;AAC9E,qEAAiE;AACjE,2CAAiC;AAEjC,MAAM,SAAS,GAAG,EAAE,CAAC;AAWrB,0EAA0E;AAC1E,8DAA8D;AAC9D,MAAM,aAAa,GAAG,IAAI,OAAO,EAAqC,CAAC;AAEvE,SAAS,aAAa,CAAC,OAAuB;IAC5C,IAAI,CAAC,aAAa,CAAC,GAAG,CAAC,OAAO,CAAC,EAAE,CAAC,EAAE,CAAC;QACnC,aAAa,CAAC,GAAG,CAAC,OAAO,CAAC,EAAE,EAAE,IAAI,GAAG,EAAE,CAAC,CAAC;IAC3C,CAAC;IACD,OAAO,aAAa,CAAC,GAAG,CAAC,OAAO,CAAC,EAAE,CAAE,CAAC;AACxC,CAAC;AAED;;;GAGG;AACH,SAAS,kBAAkB,CAAC,MAAc;IACxC,IAAI,GAAG,CAAC;IACR,IAAI,CAAC;QACH,GAAG,GAAG,IAAA,oCAAe,EAAC,MAAM,CAAC,CAAC;IAChC,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,EAAE,CAAC;IACZ,CAAC;IAED,MAAM,IAAI,GAAiB,EAAE,CAAC;IAC9B,MAAM,KAAK,GAAU,CAAC,GAAG,CAAC,CAAC;IAE3B,OAAO,KAAK,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACxB,MAAM,IAAI,GAAG,KAAK,CAAC,GAAG,EAAE,CAAC;QACzB,IAAI,CAAC,IAAI,IAAI,OAAO,IAAI,KAAK,QAAQ;YAAE,SAAS;QAEhD,IACE,IAAI,CAAC,IAAI,KAAK,8BAAS,CAAC,SAAS;YACjC,IAAI,CAAC,MAAM;YACX,OAAO,IAAI,CAAC,MAAM,KAAK,QAAQ;YAC/B,CAAC,IAAI,CAAC,MAAM,CAAC,IAAI,KAAK,8BAAS,CAAC,YAAY;gBAC1C,IAAI,CAAC,MAAM,CAAC,IAAI,KAAK,8BAAS,CAAC,cAAc,CAAC;YAChD,IAAI,CAAC,MAAM,CAAC,OAAO;YACnB,IAAI,CAAC,MAAM,CAAC,OAAO,CAAC,IAAI,KAAK,8BAAS,CAAC,cAAc,EACrD,CAAC;YACD,IAAI,CAAC,IAAI,CAAC;gBACR,IAAI,EAAE,IAAI,CAAC,MAAM,CAAC,OAAO,CAAC,KAAK;gBAC/B,YAAY,EAAG,IAAI,CAAC,IAAqB,IAAI,QAAQ;gBACrD,UAAU,EAAE,IAAI,CAAC,MAAM,CAAC,OAAO,CAAC,QAAQ,CAAC,KAAK;gBAC9C,QAAQ,EAAE,IAAI,CAAC,MAAM,CAAC,OAAO,CAAC,QAAQ,CAAC,GAAG;aAC3C,CAAC,CAAC;QACL,CAAC;QAED,2EAA2E;QAC3E,KAAK,MAAM,GAAG,IAAI,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;YACpC,IAAI,6CAAwB,CAAC,GAAG,CAAC,GAAG,CAAC;gBAAE,SAAS;YAChD,MAAM,KAAK,GAAG,IAAI,CAAC,GAAG,CAAC,CAAC;YACxB,IAAI,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC,EAAE,CAAC;gBACzB,KAAK,MAAM,KAAK,IAAI,KAAK,EAAE,CAAC;oBAC1B,IAAI,KAAK,IAAI,OAAO,KAAK,KAAK,QAAQ,EAAE,CAAC;wBACvC,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;oBACpB,CAAC;gBACH,CAAC;YACH,CAAC;iBAAM,IAAI,KAAK,IAAI,OAAO,KAAK,KAAK,QAAQ,IAAI,KAAK,CAAC,IAAI,EAAE,CAAC;gBAC5D,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;YACpB,CAAC;QACH,CAAC;IACH,CAAC;IAED,OAAO,IAAI,CAAC;AACd,CAAC;AAEY,QAAA,cAAc,GAA0B;IACnD,IAAI,EAAE;QACJ,IAAI,EAAE,gBAAgB;QACtB,IAAI,EAAE,0BAA0B;QAChC,IAAI,EAAE;YACJ,WAAW,EACT,6FAA6F;YAC/F,WAAW,EAAE,IAAI;YACjB,GAAG,EAAE,8FAA8F;SACpG;QACD,IAAI,EAAE,sBAAc,CAAC,UAAU;QAC/B,QAAQ,EAAE,gBAAQ,CAAC,KAAK;QACxB,MAAM,EAAE,EAAE;QACV,OAAO,EAAE,EAAE;KACZ;IAED,MAAM,CAAC,OAAO;QACZ,MAAM,OAAO,GAAG,IAAI,oCAAgB,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;QACjD,MAAM,OAAO,GAAG,gBAAG,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC;QAClD,MAAM,UAAU,GAAG,aAAa,CAAC,OAAO,CAAC,CAAC;QAC1C,MAAM,aAAa,GAAiB,EAAE,CAAC;QAEvC,OAAO;YACL,KAAK,CAAC,YAAY,CAAC,IAAI,EAAE,SAAS;gBAChC,IAAI,IAAI,CAAC,OAAO,CAAC,IAAI,KAAK,8BAAS,CAAC,cAAc;oBAAE,OAAO;gBAC3D,MAAM,MAAM,GAAG,SAAS,CAAC,EAAE,CAAC,CAAC,CAAC,CAAQ,CAAC;gBACvC,MAAM,YAAY,GAAiB,MAAM,EAAE,IAAI,KAAK,SAAS,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,QAAQ,CAAC;gBACrF,aAAa,CAAC,IAAI,CAAC;oBACjB,IAAI,EAAE,IAAI,CAAC,OAAO,CAAC,KAAK;oBACxB,YAAY;oBACZ,UAAU,EAAE,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,KAAK;oBACvC,QAAQ,EAAE,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,GAAG;iBACpC,CAAC,CAAC;YACL,CAAC;YAED,KAAK,CAAC,cAAc,CAAC,IAAI;gBACvB,IAAI,IAAI,CAAC,OAAO,CAAC,IAAI,KAAK,8BAAS,CAAC,cAAc;oBAAE,OAAO;gBAC3D,aAAa,CAAC,IAAI,CAAC;oBACjB,IAAI,EAAE,IAAI,CAAC,OAAO,CAAC,KAAK;oBACxB,YAAY,EAAE,UAAU;oBACxB,UAAU,EAAE,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,KAAK;oBACvC,QAAQ,EAAE,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,GAAG;iBACpC,CAAC,CAAC;YACL,CAAC;YAED,KAAK,CAAC,aAAa;gBACjB,MAAM,OAAO,GAAG,OAAO,CAAC,IAAI,CAAC,GAAG,CAAC;gBAEjC,KAAK,MAAM,GAAG,IAAI,aAAa,EAAE,CAAC;oBAChC,MAAM,KAAK,GAAG,MAAM,SAAS,CAC3B,OAAO,EACP,OAAO,EACP,OAAO,CAAC,EAAE,EACV,UAAU,EACV,OAAO,EACP,GAAG,CAAC,IAAI,EACR,GAAG,CAAC,YAAY,EAChB,CAAC,OAAO,CAAC,EACT,CAAC,CACF,CAAC;oBAEF,IAAI,KAAK,EAAE,CAAC;wBACV,MAAM,SAAS,GAAG,KAAK;6BACpB,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE;4BACT,MAAM,KAAK,GAAG,CAAC,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;4BAC3B,OAAO,KAAK,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;wBACnC,CAAC,CAAC;6BACD,IAAI,CAAC,MAAM,CAAC,CAAC;wBAEhB,OAAO,CAAC,MAAM,CAAC;4BACb,OAAO,EAAE,6BAA6B,SAAS,gDAAgD;4BAC/F,UAAU,EAAE,GAAG,CAAC,UAAU;4BAC1B,QAAQ,EAAE,GAAG,CAAC,QAAQ;yBACvB,CAAC,CAAC;oBACL,CAAC;gBACH,CAAC;YACH,CAAC;SACF,CAAC;IACJ,CAAC;CACF,CAAC;AAEF,KAAK,UAAU,SAAS,CACtB,OAAyB,EACzB,OAAY,EACZ,EAAkD,EAClD,UAAqC,EACrC,SAAiB,EACjB,WAAmB,EACnB,YAA0B,EAC1B,IAAc,EACd,KAAa;IAEb,IAAI,KAAK,IAAI,SAAS;QAAE,OAAO,IAAI,CAAC;IAEpC,MAAM,GAAG,GAAG,MAAM,OAAO,CAAC,MAAM,CAAC,OAAO,EAAE,YAAY,EAAE,WAAW,CAAC,CAAC;IACrE,IAAI,CAAC,GAAG;QAAE,OAAO,IAAI,CAAC;IAEtB,IAAI,GAAG,KAAK,SAAS,EAAE,CAAC;QACtB,OAAO,CAAC,GAAG,IAAI,EAAE,GAAG,CAAC,CAAC;IACxB,CAAC;IAED,IAAI,IAAI,CAAC,QAAQ,CAAC,GAAG,CAAC,EAAE,CAAC;QACvB,oEAAoE;QACpE,OAAO,IAAI,CAAC;IACd,CAAC;IAED,IAAI,IAAI,GAAG,UAAU,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC;IAC/B,IAAI,CAAC,IAAI,EAAE,CAAC;QACV,IAAI,CAAC;YACH,MAAM,MAAM,GAAG,MAAM,EAAE,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC;YACtC,IAAI,GAAG,kBAAkB,CAAC,MAAM,CAAC,CAAC;QACpC,CAAC;QAAC,MAAM,CAAC;YACP,IAAI,GAAG,EAAE,CAAC;QACZ,CAAC;QACD,UAAU,CAAC,GAAG,CAAC,GAAG,EAAE,IAAI,CAAC,CAAC;IAC5B,CAAC;IAED,KAAK,MAAM,GAAG,IAAI,IAAI,EAAE,CAAC;QACvB,MAAM,KAAK,GAAG,MAAM,SAAS,CAC3B,OAAO,EACP,OAAO,EACP,EAAE,EACF,UAAU,EACV,SAAS,EACT,GAAG,CAAC,IAAI,EACR,GAAG,CAAC,YAAY,EAChB,CAAC,GAAG,IAAI,EAAE,GAAG,CAAC,EACd,KAAK,GAAG,CAAC,CACV,CAAC;QACF,IAAI,KAAK;YAAE,OAAO,KAAK,CAAC;IAC1B,CAAC;IAED,OAAO,IAAI,CAAC;AACd,CAAC"}

package/dist/checks/index.d.ts

@@ -5,4 +5,4 @@ export declare const allChecks: (LiquidCheckDefinition | JSONCheckDefinition | G
  * - meta.docs.recommended: true
  * - Either no meta.targets list exist or if it does exist then Recommended is a target
  */
-export declare const recommended: (LiquidCheckDefinition | JSONCheckDefinition | GraphQLCheckDefinition | YAMLCheckDefinition)[];
+export declare const recommended: (LiquidCheckDefinition | JSONCheckDefinition | YAMLCheckDefinition | GraphQLCheckDefinition)[];

package/dist/checks/index.js

@@ -32,6 +32,9 @@ const graphql_1 = require("./graphql");
 const unknown_property_1 = require("./unknown-property");
 const invalid_hash_assign_target_1 = require("./invalid-hash-assign-target");
 const duplicate_function_arguments_1 = require("./duplicate-function-arguments");
+const missing_render_partial_arguments_1 = require("./missing-render-partial-arguments");
+const nested_graphql_query_1 = require("./nested-graphql-query");
+const missing_page_1 = require("./missing-page");
 exports.allChecks = [
     deprecated_filter_1.DeprecatedFilter,
     deprecated_tag_1.DeprecatedTag,
@@ -63,6 +66,9 @@ exports.allChecks = [
     graphql_1.GraphQLCheck,
     unknown_property_1.UnknownProperty,
     invalid_hash_assign_target_1.InvalidHashAssignTarget,
+    missing_render_partial_arguments_1.MissingRenderPartialArguments,
+    nested_graphql_query_1.NestedGraphQLQuery,
+    missing_page_1.MissingPage,
 ];
 /**
  * The recommended checks is populated by all checks with the following conditions:

package/dist/checks/index.js.map

@@ -1 +1 @@
-{"version":3,"file":"index.js","sourceRoot":"","sources":["../../src/checks/index.ts"],"names":[],"mappings":";;;AAAA,oCAMkB;AAElB,2DAAuD;AACvD,qDAAiD;AACjD,6FAAuF;AACvF,iEAA2D;AAC3D,2DAAsD;AACtD,yEAAmE;AACnE,mEAA+D;AAC/D,mDAA+C;AAC/C,uDAAmD;AACnD,yDAAqD;AACrD,qEAAgE;AAChE,qEAAgE;AAChE,mEAA8D;AAC9D,yDAAqD;AACrD,qEAA+D;AAC/D,qDAAiD;AACjD,mGAA6F;AAC7F,mDAA+C;AAC/C,yDAAoD;AACpD,qEAAgE;AAChE,6CAAyC;AACzC,mEAA6D;AAC7D,+FAAwF;AACxF,mDAA+C;AAC/C,uDAAwD;AACxD,2DAA4D;AAC5D,uCAAyC;AACzC,yDAAqD;AACrD,6EAAuE;AACvE,iFAA4E;AAE/D,QAAA,SAAS,GAKhB;IACJ,oCAAgB;IAChB,8BAAa;IACb,yDAA0B;IAC1B,oEAA+B;IAC/B,wCAAiB;IACjB,mCAAe;IACf,gDAAqB;IACrB,4CAAoB;IACpB,4BAAY;IACZ,gCAAc;IACd,kCAAe;IACf,6CAAoB;IACpB,6CAAoB;IACpB,2CAAmB;IACnB,kCAAe;IACf,4CAAmB;IACnB,8BAAa;IACb,0EAAkC;IAClC,4BAAY;IACZ,iCAAc;IACd,6CAAoB;IACpB,sBAAS;IACT,0CAAkB;IAClB,qEAA+B;IAC/B,4BAAY;IACZ,qCAAmB;IACnB,yCAAqB;IACrB,sBAAY;IACZ,kCAAe;IACf,oDAAuB;CACxB,CAAC;AAEF;;;;GAIG;AACU,QAAA,WAAW,GAAG,iBAAS,CAAC,MAAM,CAAC,CAAC,KAAK,EAAE,EAAE;IACpD,MAAM,aAAa,GAAG,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,WAAW,CAAC;IAClD,MAAM,aAAa,GACjB,CAAC,KAAK,CAAC,IAAI,CAAC,OAAO;QACnB,CAAC,KAAK,CAAC,IAAI,CAAC,OAAO,CAAC,MAAM;QAC1B,KAAK,CAAC,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,oBAAY,CAAC,WAAW,CAAC,CAAC;IAExD,OAAO,aAAa,IAAI,aAAa,CAAC;AACxC,CAAC,CAAC,CAAC"}
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../../src/checks/index.ts"],"names":[],"mappings":";;;AAAA,oCAMkB;AAElB,2DAAuD;AACvD,qDAAiD;AACjD,6FAAuF;AACvF,iEAA2D;AAC3D,2DAAsD;AACtD,yEAAmE;AACnE,mEAA+D;AAC/D,mDAA+C;AAC/C,uDAAmD;AACnD,yDAAqD;AACrD,qEAAgE;AAChE,qEAAgE;AAChE,mEAA8D;AAC9D,yDAAqD;AACrD,qEAA+D;AAC/D,qDAAiD;AACjD,mGAA6F;AAC7F,mDAA+C;AAC/C,yDAAoD;AACpD,qEAAgE;AAChE,6CAAyC;AACzC,mEAA6D;AAC7D,+FAAwF;AACxF,mDAA+C;AAC/C,uDAAwD;AACxD,2DAA4D;AAC5D,uCAAyC;AACzC,yDAAqD;AACrD,6EAAuE;AACvE,iFAA4E;AAC5E,yFAAmF;AACnF,iEAA4D;AAC5D,iDAA6C;AAEhC,QAAA,SAAS,GAKhB;IACJ,oCAAgB;IAChB,8BAAa;IACb,yDAA0B;IAC1B,oEAA+B;IAC/B,wCAAiB;IACjB,mCAAe;IACf,gDAAqB;IACrB,4CAAoB;IACpB,4BAAY;IACZ,gCAAc;IACd,kCAAe;IACf,6CAAoB;IACpB,6CAAoB;IACpB,2CAAmB;IACnB,kCAAe;IACf,4CAAmB;IACnB,8BAAa;IACb,0EAAkC;IAClC,4BAAY;IACZ,iCAAc;IACd,6CAAoB;IACpB,sBAAS;IACT,0CAAkB;IAClB,qEAA+B;IAC/B,4BAAY;IACZ,qCAAmB;IACnB,yCAAqB;IACrB,sBAAY;IACZ,kCAAe;IACf,oDAAuB;IACvB,gEAA6B;IAC7B,yCAAkB;IAClB,0BAAW;CACZ,CAAC;AAEF;;;;GAIG;AACU,QAAA,WAAW,GAAG,iBAAS,CAAC,MAAM,CAAC,CAAC,KAAK,EAAE,EAAE;IACpD,MAAM,aAAa,GAAG,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,WAAW,CAAC;IAClD,MAAM,aAAa,GACjB,CAAC,KAAK,CAAC,IAAI,CAAC,OAAO;QACnB,CAAC,KAAK,CAAC,IAAI,CAAC,OAAO,CAAC,MAAM;QAC1B,KAAK,CAAC,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,oBAAY,CAAC,WAAW,CAAC,CAAC;IAExD,OAAO,aAAa,IAAI,aAAa,CAAC;AACxC,CAAC,CAAC,CAAC"}

package/dist/checks/missing-page/index.d.ts

@@ -0,0 +1,2 @@
+import { LiquidCheckDefinition } from '../../types';
+export declare const MissingPage: LiquidCheckDefinition;

package/dist/checks/missing-page/index.js

@@ -0,0 +1,73 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.MissingPage = void 0;
+const url_helpers_1 = require("../../url-helpers");
+const types_1 = require("../../types");
+const utils_1 = require("../utils");
+exports.MissingPage = {
+    meta: {
+        code: 'MissingPage',
+        name: 'Missing page for route',
+        docs: {
+            description: 'Reports links and form actions that point to routes with no corresponding platformOS page.',
+            recommended: true,
+            url: 'https://documentation.platformos.com/developer-guide/platformos-check/checks/missing-page',
+        },
+        type: types_1.SourceCodeType.LiquidHtml,
+        severity: types_1.Severity.WARNING,
+        schema: {},
+        targets: [],
+    },
+    create(context) {
+        let routeTable;
+        // Tracks {% assign %} variable mappings incrementally in document order.
+        // This means each <a> / <form> sees only the assigns that precede it,
+        // and reassignments correctly shadow earlier values at their position.
+        const variableMap = new Map();
+        function checkUrlAttribute(attr, method) {
+            const urlPattern = (0, url_helpers_1.extractUrlPattern)(attr, variableMap);
+            if (urlPattern === null)
+                return;
+            if ((0, url_helpers_1.shouldSkipUrl)(urlPattern))
+                return;
+            if (!routeTable.hasMatch(urlPattern, method)) {
+                const methodLabel = method.toUpperCase();
+                context.report({
+                    message: `No page found for route '${urlPattern}' (${methodLabel})`,
+                    startIndex: attr.value[0].position.start,
+                    endIndex: attr.value[attr.value.length - 1].position.end,
+                });
+            }
+        }
+        return {
+            async onCodePathStart() {
+                // Front-load the route table build so individual HtmlElement visits don't wait.
+                routeTable = await context.getRouteTable();
+            },
+            async LiquidTag(node) {
+                const extracted = (0, url_helpers_1.tryExtractAssignUrl)(node);
+                if (extracted) {
+                    variableMap.set(extracted.name, extracted.urlPattern);
+                }
+            },
+            async HtmlElement(node) {
+                if ((0, utils_1.isHtmlTag)(node, 'a')) {
+                    const hrefAttr = node.attributes.find((a) => (0, url_helpers_1.isValuedAttrNode)(a) && (0, url_helpers_1.getAttrName)(a) === 'href');
+                    if (hrefAttr && (0, url_helpers_1.isValuedAttrNode)(hrefAttr)) {
+                        checkUrlAttribute(hrefAttr, 'get');
+                    }
+                }
+                else if ((0, utils_1.isHtmlTag)(node, 'form')) {
+                    const actionAttr = node.attributes.find((a) => (0, url_helpers_1.isValuedAttrNode)(a) && (0, url_helpers_1.getAttrName)(a) === 'action');
+                    if (actionAttr && (0, url_helpers_1.isValuedAttrNode)(actionAttr)) {
+                        const method = (0, url_helpers_1.getEffectiveMethod)(node);
+                        if (method !== null) {
+                            checkUrlAttribute(actionAttr, method);
+                        }
+                    }
+                }
+            },
+        };
+    },
+};
+//# sourceMappingURL=index.js.map

package/dist/checks/missing-page/index.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../../../src/checks/missing-page/index.ts"],"names":[],"mappings":";;;AAEA,mDAO2B;AAC3B,uCAA8E;AAC9E,oCAAqC;AAExB,QAAA,WAAW,GAA0B;IAChD,IAAI,EAAE;QACJ,IAAI,EAAE,aAAa;QACnB,IAAI,EAAE,wBAAwB;QAC9B,IAAI,EAAE;YACJ,WAAW,EACT,4FAA4F;YAC9F,WAAW,EAAE,IAAI;YACjB,GAAG,EAAE,2FAA2F;SACjG;QACD,IAAI,EAAE,sBAAc,CAAC,UAAU;QAC/B,QAAQ,EAAE,gBAAQ,CAAC,OAAO;QAC1B,MAAM,EAAE,EAAE;QACV,OAAO,EAAE,EAAE;KACZ;IAED,MAAM,CAAC,OAAO;QACZ,IAAI,UAAsB,CAAC;QAC3B,yEAAyE;QACzE,sEAAsE;QACtE,uEAAuE;QACvE,MAAM,WAAW,GAAG,IAAI,GAAG,EAAkB,CAAC;QAE9C,SAAS,iBAAiB,CAAC,IAA6C,EAAE,MAAc;YACtF,MAAM,UAAU,GAAG,IAAA,+BAAiB,EAAC,IAAI,EAAE,WAAW,CAAC,CAAC;YACxD,IAAI,UAAU,KAAK,IAAI;gBAAE,OAAO;YAChC,IAAI,IAAA,2BAAa,EAAC,UAAU,CAAC;gBAAE,OAAO;YAEtC,IAAI,CAAC,UAAU,CAAC,QAAQ,CAAC,UAAU,EAAE,MAAM,CAAC,EAAE,CAAC;gBAC7C,MAAM,WAAW,GAAG,MAAM,CAAC,WAAW,EAAE,CAAC;gBACzC,OAAO,CAAC,MAAM,CAAC;oBACb,OAAO,EAAE,4BAA4B,UAAU,MAAM,WAAW,GAAG;oBACnE,UAAU,EAAE,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,KAAK;oBACxC,QAAQ,EAAE,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,QAAQ,CAAC,GAAG;iBACzD,CAAC,CAAC;YACL,CAAC;QACH,CAAC;QAED,OAAO;YACL,KAAK,CAAC,eAAe;gBACnB,gFAAgF;gBAChF,UAAU,GAAG,MAAM,OAAO,CAAC,aAAa,EAAE,CAAC;YAC7C,CAAC;YAED,KAAK,CAAC,SAAS,CAAC,IAAe;gBAC7B,MAAM,SAAS,GAAG,IAAA,iCAAmB,EAAC,IAAI,CAAC,CAAC;gBAC5C,IAAI,SAAS,EAAE,CAAC;oBACd,WAAW,CAAC,GAAG,CAAC,SAAS,CAAC,IAAI,EAAE,SAAS,CAAC,UAAU,CAAC,CAAC;gBACxD,CAAC;YACH,CAAC;YAED,KAAK,CAAC,WAAW,CAAC,IAAI;gBACpB,IAAI,IAAA,iBAAS,EAAC,IAAI,EAAE,GAAG,CAAC,EAAE,CAAC;oBACzB,MAAM,QAAQ,GAAG,IAAI,CAAC,UAAU,CAAC,IAAI,CACnC,CAAC,CAAC,EAAE,EAAE,CAAC,IAAA,8BAAgB,EAAC,CAAC,CAAC,IAAI,IAAA,yBAAW,EAAC,CAAC,CAAC,KAAK,MAAM,CACxD,CAAC;oBAEF,IAAI,QAAQ,IAAI,IAAA,8BAAgB,EAAC,QAAQ,CAAC,EAAE,CAAC;wBAC3C,iBAAiB,CAAC,QAAQ,EAAE,KAAK,CAAC,CAAC;oBACrC,CAAC;gBACH,CAAC;qBAAM,IAAI,IAAA,iBAAS,EAAC,IAAI,EAAE,MAAM,CAAC,EAAE,CAAC;oBACnC,MAAM,UAAU,GAAG,IAAI,CAAC,UAAU,CAAC,IAAI,CACrC,CAAC,CAAC,EAAE,EAAE,CAAC,IAAA,8BAAgB,EAAC,CAAC,CAAC,IAAI,IAAA,yBAAW,EAAC,CAAC,CAAC,KAAK,QAAQ,CAC1D,CAAC;oBAEF,IAAI,UAAU,IAAI,IAAA,8BAAgB,EAAC,UAAU,CAAC,EAAE,CAAC;wBAC/C,MAAM,MAAM,GAAG,IAAA,gCAAkB,EAAC,IAAmB,CAAC,CAAC;wBACvD,IAAI,MAAM,KAAK,IAAI,EAAE,CAAC;4BACpB,iBAAiB,CAAC,UAAU,EAAE,MAAM,CAAC,CAAC;wBACxC,CAAC;oBACH,CAAC;gBACH,CAAC;YACH,CAAC;SACF,CAAC;IACJ,CAAC;CACF,CAAC"}

package/dist/checks/missing-partial/index.js

@@ -5,6 +5,13 @@ const liquid_html_parser_1 = require("@platformos/liquid-html-parser");
 const types_1 = require("../../types");
 const platformos_common_1 = require("@platformos/platformos-common");
 const vscode_uri_1 = require("vscode-uri");
+function getTagName(ancestors) {
+    const parent = ancestors.at(-1);
+    if (parent?.type === liquid_html_parser_1.NodeTypes.LiquidTag && parent.name === liquid_html_parser_1.NamedTags.theme_render_rc) {
+        return 'theme_render_rc';
+    }
+    return 'render';
+}
 const schema = {
     ignoreMissing: types_1.SchemaProp.array(types_1.SchemaProp.string(), []),
 };
@@ -24,44 +31,37 @@ exports.MissingPartial = {
     },
     create(context) {
         const locator = new platformos_common_1.DocumentsLocator(context.fs);
+        const rootUri = vscode_uri_1.URI.parse(context.config.rootUri);
+        let searchPathsPromise;
+        function getSearchPaths() {
+            searchPathsPromise ??= (0, platformos_common_1.loadSearchPaths)(context.fs, rootUri);
+            return searchPathsPromise;
+        }
+        async function reportIfMissing(docType, name, position) {
+            const searchPaths = docType === 'theme_render_rc' ? await getSearchPaths() : null;
+            const location = await locator.locate(rootUri, docType, name, searchPaths);
+            if (!location) {
+                context.report({
+                    message: `'${name}' does not exist`,
+                    startIndex: position.start,
+                    endIndex: position.end,
+                });
+            }
+        }
         return {
-            async RenderMarkup(node) {
-                if (node.partial.type === liquid_html_parser_1.NodeTypes.VariableLookup)
-                    return;
-                const partial = node.partial;
-                const location = await locator.locate(vscode_uri_1.URI.parse(context.config.rootUri), 'render', partial.value);
-                if (!location) {
-                    context.report({
-                        message: `'${partial.value}' does not exist`,
-                        startIndex: node.partial.position.start,
-                        endIndex: node.partial.position.end,
-                    });
+            async RenderMarkup(node, ancestors) {
+                if (node.partial.type !== liquid_html_parser_1.NodeTypes.VariableLookup) {
+                    await reportIfMissing(getTagName(ancestors), node.partial.value, node.partial.position);
                 }
             },
             async FunctionMarkup(node) {
-                if (node.partial.type === liquid_html_parser_1.NodeTypes.VariableLookup)
-                    return;
-                const partial = node.partial;
-                const location = await locator.locate(vscode_uri_1.URI.parse(context.config.rootUri), 'function', partial.value);
-                if (!location) {
-                    context.report({
-                        message: `'${partial.value}' does not exist`,
-                        startIndex: node.partial.position.start,
-                        endIndex: node.partial.position.end,
-                    });
+                if (node.partial.type !== liquid_html_parser_1.NodeTypes.VariableLookup) {
+                    await reportIfMissing('function', node.partial.value, node.partial.position);
                 }
             },
             async GraphQLMarkup(node) {
-                if (node.graphql.type === liquid_html_parser_1.NodeTypes.VariableLookup)
-                    return;
-                const graphql = node.graphql;
-                const location = await locator.locate(vscode_uri_1.URI.parse(context.config.rootUri), 'graphql', graphql.value);
-                if (!location) {
-                    context.report({
-                        message: `'${graphql.value}' does not exist`,
-                        startIndex: node.graphql.position.start,
-                        endIndex: node.graphql.position.end,
-                    });
+                if (node.graphql.type !== liquid_html_parser_1.NodeTypes.VariableLookup) {
+                    await reportIfMissing('graphql', node.graphql.value, node.graphql.position);
                 }
             },
         };

package/dist/checks/missing-partial/index.js.map

@@ -1 +1 @@
-{"version":3,"file":"index.js","sourceRoot":"","sources":["../../../src/checks/missing-partial/index.ts"],"names":[],"mappings":";;;AAAA,uEAA2D;AAC3D,uCAA0F;AAC1F,qEAAiE;AACjE,2CAAiC;AAEjC,MAAM,MAAM,GAAG;IACb,aAAa,EAAE,kBAAU,CAAC,KAAK,CAAC,kBAAU,CAAC,MAAM,EAAE,EAAE,EAAE,CAAC;CACzD,CAAC;AAEW,QAAA,cAAc,GAAyC;IAClE,IAAI,EAAE;QACJ,IAAI,EAAE,gBAAgB;QACtB,IAAI,EAAE,kCAAkC;QACxC,IAAI,EAAE;YACJ,WAAW,EAAE,qCAAqC;YAClD,WAAW,EAAE,IAAI;YACjB,GAAG,EAAE,8FAA8F;SACpG;QACD,IAAI,EAAE,sBAAc,CAAC,UAAU;QAC/B,QAAQ,EAAE,gBAAQ,CAAC,KAAK;QACxB,MAAM;QACN,OAAO,EAAE,EAAE;KACZ;IAED,MAAM,CAAC,OAAO;QACZ,MAAM,OAAO,GAAG,IAAI,oCAAgB,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;QAEjD,OAAO;YACL,KAAK,CAAC,YAAY,CAAC,IAAI;gBACrB,IAAI,IAAI,CAAC,OAAO,CAAC,IAAI,KAAK,8BAAS,CAAC,cAAc;oBAAE,OAAO;gBAE3D,MAAM,OAAO,GAAG,IAAI,CAAC,OAAO,CAAC;gBAC7B,MAAM,QAAQ,GAAG,MAAM,OAAO,CAAC,MAAM,CACnC,gBAAG,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC,OAAO,CAAC,EACjC,QAAQ,EACR,OAAO,CAAC,KAAK,CACd,CAAC;gBAEF,IAAI,CAAC,QAAQ,EAAE,CAAC;oBACd,OAAO,CAAC,MAAM,CAAC;wBACb,OAAO,EAAE,IAAI,OAAO,CAAC,KAAK,kBAAkB;wBAC5C,UAAU,EAAE,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,KAAK;wBACvC,QAAQ,EAAE,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,GAAG;qBACpC,CAAC,CAAC;gBACL,CAAC;YACH,CAAC;YAED,KAAK,CAAC,cAAc,CAAC,IAAI;gBACvB,IAAI,IAAI,CAAC,OAAO,CAAC,IAAI,KAAK,8BAAS,CAAC,cAAc;oBAAE,OAAO;gBAE3D,MAAM,OAAO,GAAG,IAAI,CAAC,OAAO,CAAC;gBAC7B,MAAM,QAAQ,GAAG,MAAM,OAAO,CAAC,MAAM,CACnC,gBAAG,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC,OAAO,CAAC,EACjC,UAAU,EACV,OAAO,CAAC,KAAK,CACd,CAAC;gBAEF,IAAI,CAAC,QAAQ,EAAE,CAAC;oBACd,OAAO,CAAC,MAAM,CAAC;wBACb,OAAO,EAAE,IAAI,OAAO,CAAC,KAAK,kBAAkB;wBAC5C,UAAU,EAAE,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,KAAK;wBACvC,QAAQ,EAAE,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,GAAG;qBACpC,CAAC,CAAC;gBACL,CAAC;YACH,CAAC;YAED,KAAK,CAAC,aAAa,CAAC,IAAI;gBACtB,IAAI,IAAI,CAAC,OAAO,CAAC,IAAI,KAAK,8BAAS,CAAC,cAAc;oBAAE,OAAO;gBAE3D,MAAM,OAAO,GAAG,IAAI,CAAC,OAAO,CAAC;gBAC7B,MAAM,QAAQ,GAAG,MAAM,OAAO,CAAC,MAAM,CACnC,gBAAG,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC,OAAO,CAAC,EACjC,SAAS,EACT,OAAO,CAAC,KAAK,CACd,CAAC;gBAEF,IAAI,CAAC,QAAQ,EAAE,CAAC;oBACd,OAAO,CAAC,MAAM,CAAC;wBACb,OAAO,EAAE,IAAI,OAAO,CAAC,KAAK,kBAAkB;wBAC5C,UAAU,EAAE,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,KAAK;wBACvC,QAAQ,EAAE,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,GAAG;qBACpC,CAAC,CAAC;gBACL,CAAC;YACH,CAAC;SACF,CAAC;IACJ,CAAC;CACF,CAAC"}
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../../../src/checks/missing-partial/index.ts"],"names":[],"mappings":";;;AAAA,uEAAsF;AACtF,uCAA0F;AAC1F,qEAAgG;AAChG,2CAAiC;AAEjC,SAAS,UAAU,CAAC,SAA2B;IAC7C,MAAM,MAAM,GAAG,SAAS,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC;IAChC,IAAI,MAAM,EAAE,IAAI,KAAK,8BAAS,CAAC,SAAS,IAAI,MAAM,CAAC,IAAI,KAAK,8BAAS,CAAC,eAAe,EAAE,CAAC;QACtF,OAAO,iBAAiB,CAAC;IAC3B,CAAC;IACD,OAAO,QAAQ,CAAC;AAClB,CAAC;AAED,MAAM,MAAM,GAAG;IACb,aAAa,EAAE,kBAAU,CAAC,KAAK,CAAC,kBAAU,CAAC,MAAM,EAAE,EAAE,EAAE,CAAC;CACzD,CAAC;AAEW,QAAA,cAAc,GAAyC;IAClE,IAAI,EAAE;QACJ,IAAI,EAAE,gBAAgB;QACtB,IAAI,EAAE,kCAAkC;QACxC,IAAI,EAAE;YACJ,WAAW,EAAE,qCAAqC;YAClD,WAAW,EAAE,IAAI;YACjB,GAAG,EAAE,8FAA8F;SACpG;QACD,IAAI,EAAE,sBAAc,CAAC,UAAU;QAC/B,QAAQ,EAAE,gBAAQ,CAAC,KAAK;QACxB,MAAM;QACN,OAAO,EAAE,EAAE;KACZ;IAED,MAAM,CAAC,OAAO;QACZ,MAAM,OAAO,GAAG,IAAI,oCAAgB,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;QACjD,MAAM,OAAO,GAAG,gBAAG,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC;QAClD,IAAI,kBAAwD,CAAC;QAE7D,SAAS,cAAc;YACrB,kBAAkB,KAAK,IAAA,mCAAe,EAAC,OAAO,CAAC,EAAE,EAAE,OAAO,CAAC,CAAC;YAC5D,OAAO,kBAAkB,CAAC;QAC5B,CAAC;QAED,KAAK,UAAU,eAAe,CAC5B,OAAqB,EACrB,IAAY,EACZ,QAAoC;YAEpC,MAAM,WAAW,GAAG,OAAO,KAAK,iBAAiB,CAAC,CAAC,CAAC,MAAM,cAAc,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC;YAClF,MAAM,QAAQ,GAAG,MAAM,OAAO,CAAC,MAAM,CAAC,OAAO,EAAE,OAAO,EAAE,IAAI,EAAE,WAAW,CAAC,CAAC;YAC3E,IAAI,CAAC,QAAQ,EAAE,CAAC;gBACd,OAAO,CAAC,MAAM,CAAC;oBACb,OAAO,EAAE,IAAI,IAAI,kBAAkB;oBACnC,UAAU,EAAE,QAAQ,CAAC,KAAK;oBAC1B,QAAQ,EAAE,QAAQ,CAAC,GAAG;iBACvB,CAAC,CAAC;YACL,CAAC;QACH,CAAC;QAED,OAAO;YACL,KAAK,CAAC,YAAY,CAAC,IAAI,EAAE,SAAS;gBAChC,IAAI,IAAI,CAAC,OAAO,CAAC,IAAI,KAAK,8BAAS,CAAC,cAAc,EAAE,CAAC;oBACnD,MAAM,eAAe,CAAC,UAAU,CAAC,SAAS,CAAC,EAAE,IAAI,CAAC,OAAO,CAAC,KAAK,EAAE,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC;gBAC1F,CAAC;YACH,CAAC;YAED,KAAK,CAAC,cAAc,CAAC,IAAI;gBACvB,IAAI,IAAI,CAAC,OAAO,CAAC,IAAI,KAAK,8BAAS,CAAC,cAAc,EAAE,CAAC;oBACnD,MAAM,eAAe,CAAC,UAAU,EAAE,IAAI,CAAC,OAAO,CAAC,KAAK,EAAE,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC;gBAC/E,CAAC;YACH,CAAC;YAED,KAAK,CAAC,aAAa,CAAC,IAAI;gBACtB,IAAI,IAAI,CAAC,OAAO,CAAC,IAAI,KAAK,8BAAS,CAAC,cAAc,EAAE,CAAC;oBACnD,MAAM,eAAe,CAAC,SAAS,EAAE,IAAI,CAAC,OAAO,CAAC,KAAK,EAAE,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC;gBAC9E,CAAC;YACH,CAAC;SACF,CAAC;IACJ,CAAC;CACF,CAAC"}

package/dist/checks/missing-render-partial-arguments/index.d.ts

@@ -0,0 +1,2 @@
+import { LiquidCheckDefinition } from '../../types';
+export declare const MissingRenderPartialArguments: LiquidCheckDefinition;

package/dist/checks/missing-render-partial-arguments/index.js

@@ -0,0 +1,37 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.MissingRenderPartialArguments = void 0;
+const types_1 = require("../../types");
+const arguments_1 = require("../../liquid-doc/arguments");
+exports.MissingRenderPartialArguments = {
+    meta: {
+        code: 'MissingRenderPartialArguments',
+        name: 'Missing Required Render Partial Arguments',
+        aliases: ['MissingRenderPartialParams'],
+        docs: {
+            description: 'This check ensures that all required @param arguments declared by a partial are provided at the call site.',
+            recommended: true,
+            url: 'https://documentation.platformos.com/developer-guide/platformos-check/checks/missing-render-partial-arguments',
+        },
+        type: types_1.SourceCodeType.LiquidHtml,
+        severity: types_1.Severity.ERROR,
+        schema: {},
+        targets: [],
+    },
+    create(context) {
+        return {
+            async RenderMarkup(node) {
+                const partialName = (0, arguments_1.getPartialName)(node);
+                if (!partialName)
+                    return;
+                const liquidDocParameters = await (0, arguments_1.getLiquidDocParams)(context, partialName);
+                if (!liquidDocParameters)
+                    return;
+                const providedNames = new Set(node.args.map((a) => a.name));
+                const missingRequired = [...liquidDocParameters.values()].filter((p) => p.required && !providedNames.has(p.name));
+                (0, arguments_1.reportMissingArguments)(context, node, missingRequired, partialName);
+            },
+        };
+    },
+};
+//# sourceMappingURL=index.js.map

package/dist/checks/missing-render-partial-arguments/index.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../../../src/checks/missing-render-partial-arguments/index.ts"],"names":[],"mappings":";;;AACA,uCAA8E;AAC9E,0DAIoC;AAEvB,QAAA,6BAA6B,GAA0B;IAClE,IAAI,EAAE;QACJ,IAAI,EAAE,+BAA+B;QACrC,IAAI,EAAE,2CAA2C;QACjD,OAAO,EAAE,CAAC,4BAA4B,CAAC;QACvC,IAAI,EAAE;YACJ,WAAW,EACT,4GAA4G;YAC9G,WAAW,EAAE,IAAI;YACjB,GAAG,EAAE,+GAA+G;SACrH;QACD,IAAI,EAAE,sBAAc,CAAC,UAAU;QAC/B,QAAQ,EAAE,gBAAQ,CAAC,KAAK;QACxB,MAAM,EAAE,EAAE;QACV,OAAO,EAAE,EAAE;KACZ;IAED,MAAM,CAAC,OAAO;QACZ,OAAO;YACL,KAAK,CAAC,YAAY,CAAC,IAAkB;gBACnC,MAAM,WAAW,GAAG,IAAA,0BAAc,EAAC,IAAI,CAAC,CAAC;gBACzC,IAAI,CAAC,WAAW;oBAAE,OAAO;gBAEzB,MAAM,mBAAmB,GAAG,MAAM,IAAA,8BAAkB,EAAC,OAAO,EAAE,WAAW,CAAC,CAAC;gBAC3E,IAAI,CAAC,mBAAmB;oBAAE,OAAO;gBAEjC,MAAM,aAAa,GAAG,IAAI,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC;gBAC5D,MAAM,eAAe,GAAG,CAAC,GAAG,mBAAmB,CAAC,MAAM,EAAE,CAAC,CAAC,MAAM,CAC9D,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,QAAQ,IAAI,CAAC,aAAa,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,CAChD,CAAC;gBAEF,IAAA,kCAAsB,EAAC,OAAO,EAAE,IAAI,EAAE,eAAe,EAAE,WAAW,CAAC,CAAC;YACtE,CAAC;SACF,CAAC;IACJ,CAAC;CACF,CAAC"}

package/dist/checks/nested-graphql-query/index.d.ts

@@ -0,0 +1,2 @@
+import { LiquidCheckDefinition } from '../../types';
+export declare const NestedGraphQLQuery: LiquidCheckDefinition;

package/dist/checks/nested-graphql-query/index.js

@@ -0,0 +1,146 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.NestedGraphQLQuery = void 0;
+const liquid_html_parser_1 = require("@platformos/liquid-html-parser");
+const platformos_common_1 = require("@platformos/platformos-common");
+const vscode_uri_1 = require("vscode-uri");
+const types_1 = require("../../types");
+const utils_1 = require("../utils");
+const SKIP_IF_ANCESTOR_TAGS = [liquid_html_parser_1.NamedTags.cache];
+function findNodesInAST(ast) {
+    const results = [];
+    const stack = [...ast];
+    while (stack.length > 0) {
+        const node = stack.pop();
+        if (node.type === liquid_html_parser_1.NodeTypes.LiquidTag) {
+            if (node.name === liquid_html_parser_1.NamedTags.graphql) {
+                results.push({ type: 'graphql', partialChain: [] });
+            }
+            else if ((node.name === liquid_html_parser_1.NamedTags.function || node.name === liquid_html_parser_1.NamedTags.render) &&
+                typeof node.markup !== 'string' &&
+                'partial' in node.markup &&
+                node.markup.partial.type !== liquid_html_parser_1.NodeTypes.VariableLookup) {
+                results.push({ type: node.name, partialName: node.markup.partial.value });
+            }
+            if ('children' in node && Array.isArray(node.children)) {
+                stack.push(...node.children);
+            }
+        }
+        else if ('children' in node && Array.isArray(node.children)) {
+            stack.push(...node.children);
+        }
+    }
+    return results;
+}
+async function containsGraphQLTransitively(locator, fs, rootUri, partialName, tagType, visited) {
+    if (visited.has(partialName))
+        return null;
+    visited.add(partialName);
+    const location = await locator.locate(rootUri, tagType, partialName);
+    if (!location)
+        return null;
+    let source;
+    try {
+        source = await fs.readFile(location);
+    }
+    catch {
+        return null;
+    }
+    let ast;
+    try {
+        ast = (0, liquid_html_parser_1.toLiquidHtmlAST)(source);
+    }
+    catch {
+        return null;
+    }
+    const nodes = findNodesInAST(ast.children);
+    for (const found of nodes) {
+        if (found.type === 'graphql') {
+            return [partialName];
+        }
+    }
+    for (const found of nodes) {
+        if (found.type === 'function' || found.type === 'render') {
+            const chain = await containsGraphQLTransitively(locator, fs, rootUri, found.partialName, found.type, visited);
+            if (chain) {
+                return [partialName, ...chain];
+            }
+        }
+    }
+    return null;
+}
+exports.NestedGraphQLQuery = {
+    meta: {
+        code: 'NestedGraphQLQuery',
+        name: 'Prevent N+1 GraphQL queries in loops',
+        docs: {
+            description: 'This check detects {% graphql %} tags placed inside loop tags ({% for %}, {% tablerow %}), which causes one database request per loop iteration (N+1 pattern). It also follows {% function %} and {% render %} calls transitively to detect indirect GraphQL queries.',
+            recommended: true,
+            url: 'https://documentation.platformos.com/developer-guide/platformos-check/checks/nested-graphql-query',
+        },
+        type: types_1.SourceCodeType.LiquidHtml,
+        severity: types_1.Severity.WARNING,
+        schema: {},
+        targets: [],
+    },
+    create(context) {
+        const locator = new platformos_common_1.DocumentsLocator(context.fs);
+        const rootUri = vscode_uri_1.URI.parse(context.config.rootUri);
+        function isInsideLoopWithoutCacheOrBackground(ancestors) {
+            const ancestorTags = ancestors.filter((a) => a.type === liquid_html_parser_1.NodeTypes.LiquidTag);
+            const loopAncestor = ancestorTags.find(utils_1.isLoopLiquidTag);
+            if (!loopAncestor)
+                return null;
+            const inBackground = ancestorTags.some((a) => a.name === liquid_html_parser_1.NamedTags.background);
+            if (inBackground)
+                return null;
+            const shouldSkip = ancestorTags.some((a) => SKIP_IF_ANCESTOR_TAGS.map((a) => a.toString()).includes(a.name));
+            if (shouldSkip)
+                return null;
+            return loopAncestor;
+        }
+        return {
+            async LiquidTag(node, ancestors) {
+                if (node.name === liquid_html_parser_1.NamedTags.graphql) {
+                    const loopAncestor = isInsideLoopWithoutCacheOrBackground(ancestors);
+                    if (!loopAncestor)
+                        return;
+                    let resultName = '';
+                    if (typeof node.markup !== 'string' &&
+                        (node.markup.type === liquid_html_parser_1.NodeTypes.GraphQLMarkup ||
+                            node.markup.type === liquid_html_parser_1.NodeTypes.GraphQLInlineMarkup)) {
+                        resultName = node.markup.name ? ` result = '${node.markup.name}'` : '';
+                    }
+                    const graphqlStr = resultName ? `{% graphql${resultName} %}` : '{% graphql %}';
+                    context.report({
+                        message: `N+1 pattern: ${graphqlStr} is inside a {% ${loopAncestor.name} %} loop. This executes at least one database request per iteration. Move the query before the loop and pass data as a variable.`,
+                        startIndex: node.position.start,
+                        endIndex: node.position.end,
+                    });
+                }
+                else if (node.name === liquid_html_parser_1.NamedTags.function || node.name === liquid_html_parser_1.NamedTags.render) {
+                    const loopAncestor = isInsideLoopWithoutCacheOrBackground(ancestors);
+                    if (!loopAncestor)
+                        return;
+                    if (typeof node.markup === 'string' ||
+                        !('partial' in node.markup) ||
+                        node.markup.partial.type === liquid_html_parser_1.NodeTypes.VariableLookup) {
+                        return;
+                    }
+                    const partialName = node.markup.partial.value;
+                    const visited = new Set();
+                    const chain = await containsGraphQLTransitively(locator, context.fs, rootUri, partialName, node.name, visited);
+                    if (chain) {
+                        const chainStr = chain.join(' → ');
+                        context.report({
+                            message: `N+1 pattern: {% ${node.name} '${partialName}' %} inside a {% ${loopAncestor.name} %} loop transitively calls a GraphQL query (${chainStr}). Move the query before the loop and pass data as a variable.`,
+                            startIndex: node.position.start,
+                            endIndex: node.position.end,
+                        });
+                    }
+                }
+            },
+        };
+    },
+};
+//# sourceMappingURL=index.js.map