@platformos/platformos-check-common 0.0.12 → 0.0.16

package/CHANGELOG.md

@@ -1,5 +1,64 @@
 # @platformos/theme-check-common
 
+## 0.0.16
+
+### Patch Changes
+
+- Improved Liquid Linting
+  - Better metadata params validation — Reworked detection of undefined variables in page/partial metadata parameters, reducing false positives
+  - Improved undefined object detection — More accurate identification of undefined objects in Liquid templates
+  - Fixed invalid property detection — The unknown-property check now correctly catches more cases of invalid property access on objects
+
+- Updated dependencies
+  - @platformos/liquid-html-parser@0.0.15
+
+## 0.0.15
+
+### Patch Changes
+
+- ctrl+click fix
+- Updated dependencies
+  - @platformos/liquid-html-parser@0.0.14
+
+## 0.0.14
+
+### Patch Changes
+
+- better ctrl click, more checks
+- Updated dependencies
+  - @platformos/liquid-html-parser@0.0.13
+
+## 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/metadata-params/extract-undefined-variables.d.ts

@@ -0,0 +1,8 @@
+/**
+ * Parses a Liquid source string and returns a deduplicated list of variable names
+ * that are used but never defined. Returns [] on parse errors.
+ *
+ * This mirrors the variable tracking logic from the UndefinedObject check but
+ * packaged as a standalone synchronous function.
+ */
+export declare function extractUndefinedVariables(source: string, globalObjectNames?: string[]): string[];

package/dist/checks/metadata-params/extract-undefined-variables.js

@@ -0,0 +1,213 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.extractUndefinedVariables = extractUndefinedVariables;
+const liquid_html_parser_1 = require("@platformos/liquid-html-parser");
+/**
+ * Parses a Liquid source string and returns a deduplicated list of variable names
+ * that are used but never defined. Returns [] on parse errors.
+ *
+ * This mirrors the variable tracking logic from the UndefinedObject check but
+ * packaged as a standalone synchronous function.
+ */
+function extractUndefinedVariables(source, globalObjectNames = []) {
+    let ast;
+    try {
+        ast = (0, liquid_html_parser_1.toLiquidHtmlAST)(source);
+    }
+    catch {
+        return [];
+    }
+    const scopedVariables = new Map();
+    const fileScopedVariables = new Set(globalObjectNames);
+    const variables = [];
+    function indexVariableScope(variableName, scope) {
+        if (!variableName)
+            return;
+        const indexedScope = scopedVariables.get(variableName) ?? [];
+        scopedVariables.set(variableName, indexedScope.concat(scope));
+    }
+    function walk(node, ancestors) {
+        // Process definitions from LiquidTag nodes
+        if (node.type === liquid_html_parser_1.NodeTypes.LiquidTag) {
+            handleLiquidTag(node, ancestors);
+        }
+        // Process definitions from LiquidBranch nodes (catch)
+        if (node.type === liquid_html_parser_1.NodeTypes.LiquidBranch) {
+            handleLiquidBranch(node);
+        }
+        // Process variable usages
+        if (node.type === liquid_html_parser_1.NodeTypes.VariableLookup) {
+            handleVariableLookup(node, ancestors);
+        }
+        // Recurse into children
+        const newAncestors = ancestors.concat(node);
+        for (const value of Object.values(node)) {
+            if (Array.isArray(value)) {
+                for (const item of value) {
+                    if (isNode(item)) {
+                        walk(item, newAncestors);
+                    }
+                }
+            }
+            else if (isNode(value)) {
+                walk(value, newAncestors);
+            }
+        }
+    }
+    function handleLiquidTag(node, _ancestors) {
+        if (isLiquidTagAssign(node) || isLiquidTagGraphQL(node) || isLiquidTagParseJson(node)) {
+            indexVariableScope(node.markup.name, {
+                start: node.blockStartPosition.end,
+            });
+        }
+        if (isLiquidTagHashAssign(node) && node.markup.target.name) {
+            indexVariableScope(node.markup.target.name, {
+                start: node.blockStartPosition.end,
+            });
+        }
+        if (isLiquidTagCapture(node)) {
+            indexVariableScope(node.markup.name, {
+                start: node.blockEndPosition?.end,
+            });
+        }
+        if (node.name === 'form') {
+            indexVariableScope(node.name, {
+                start: node.blockStartPosition.end,
+                end: node.blockEndPosition?.start,
+            });
+        }
+        if (node.name === 'function') {
+            const fnName = node.markup.name;
+            if (fnName.lookups.length === 0 && fnName.name !== null) {
+                indexVariableScope(fnName.name, {
+                    start: node.position.end,
+                });
+            }
+        }
+        if ((isLiquidTagIncrement(node) || isLiquidTagDecrement(node)) && node.markup.name !== null) {
+            indexVariableScope(node.markup.name, {
+                start: node.position.start,
+            });
+        }
+        if (isLiquidForTag(node) || isLiquidTableRowTag(node)) {
+            indexVariableScope(node.markup.variableName, {
+                start: node.blockStartPosition.end,
+                end: node.blockEndPosition?.start,
+            });
+            indexVariableScope(node.name === 'for' ? 'forloop' : 'tablerowloop', {
+                start: node.blockStartPosition.end,
+                end: node.blockEndPosition?.start,
+            });
+        }
+        if (isLiquidTagBackground(node)) {
+            indexVariableScope(node.markup.jobId, {
+                start: node.position.end,
+            });
+        }
+    }
+    function handleLiquidBranch(node) {
+        if (node.name === liquid_html_parser_1.NamedTags.catch &&
+            node.markup &&
+            typeof node.markup !== 'string' &&
+            'name' in node.markup &&
+            node.markup.name) {
+            indexVariableScope(node.markup.name, {
+                start: node.blockStartPosition.end,
+                end: node.blockEndPosition?.start,
+            });
+        }
+    }
+    function handleVariableLookup(node, ancestors) {
+        const parent = ancestors[ancestors.length - 1];
+        if (isLiquidTag(parent) && isLiquidTagCapture(parent))
+            return;
+        if (isLiquidTag(parent) && isLiquidTagParseJson(parent))
+            return;
+        if (isFunctionMarkup(parent) && parent.name === node)
+            return;
+        if (isLiquidBranchCatch(parent) && parent.markup === node)
+            return;
+        if (isHashAssignMarkup(parent) && parent.target === node)
+            return;
+        variables.push(node);
+    }
+    walk(ast, []);
+    // Determine undefined variables
+    const seen = new Set();
+    const result = [];
+    for (const variable of variables) {
+        if (!variable.name)
+            continue;
+        if (seen.has(variable.name))
+            continue;
+        const isVariableDefined = isDefined(variable.name, variable.position, fileScopedVariables, scopedVariables);
+        if (!isVariableDefined) {
+            seen.add(variable.name);
+            result.push(variable.name);
+        }
+    }
+    return result;
+}
+function isNode(x) {
+    return x !== null && typeof x === 'object' && typeof x.type === 'string';
+}
+function isDefined(variableName, variablePosition, fileScopedVariables, scopedVariables) {
+    if (fileScopedVariables.has(variableName)) {
+        return true;
+    }
+    const scopes = scopedVariables.get(variableName);
+    if (!scopes) {
+        return false;
+    }
+    return scopes.some((scope) => {
+        const start = variablePosition.start;
+        const isVariableAfterScopeStart = !scope.start || start > scope.start;
+        const isVariableBeforeScopeEnd = !scope.end || start < scope.end;
+        return isVariableAfterScopeStart && isVariableBeforeScopeEnd;
+    });
+}
+function isLiquidTag(node) {
+    return node?.type === liquid_html_parser_1.NodeTypes.LiquidTag;
+}
+function isLiquidTagCapture(node) {
+    return node.name === liquid_html_parser_1.NamedTags.capture;
+}
+function isLiquidTagAssign(node) {
+    return node.name === liquid_html_parser_1.NamedTags.assign && typeof node.markup !== 'string';
+}
+function isLiquidTagHashAssign(node) {
+    return node.name === liquid_html_parser_1.NamedTags.hash_assign && typeof node.markup !== 'string';
+}
+function isLiquidTagGraphQL(node) {
+    return node.name === liquid_html_parser_1.NamedTags.graphql && typeof node.markup !== 'string';
+}
+function isLiquidTagParseJson(node) {
+    return node.name === liquid_html_parser_1.NamedTags.parse_json && typeof node.markup !== 'string';
+}
+function isLiquidForTag(node) {
+    return node.name === liquid_html_parser_1.NamedTags.for && typeof node.markup !== 'string';
+}
+function isLiquidTableRowTag(node) {
+    return node.name === liquid_html_parser_1.NamedTags.tablerow && typeof node.markup !== 'string';
+}
+function isLiquidTagIncrement(node) {
+    return node.name === liquid_html_parser_1.NamedTags.increment && typeof node.markup !== 'string';
+}
+function isLiquidTagDecrement(node) {
+    return node.name === liquid_html_parser_1.NamedTags.decrement && typeof node.markup !== 'string';
+}
+function isLiquidTagBackground(node) {
+    return (node.name === liquid_html_parser_1.NamedTags.background &&
+        typeof node.markup !== 'string' &&
+        node.markup.type === liquid_html_parser_1.NodeTypes.BackgroundMarkup);
+}
+function isHashAssignMarkup(node) {
+    return node?.type === liquid_html_parser_1.NodeTypes.HashAssignMarkup;
+}
+function isFunctionMarkup(node) {
+    return node?.type === liquid_html_parser_1.NodeTypes.FunctionMarkup;
+}
+function isLiquidBranchCatch(node) {
+    return node?.type === liquid_html_parser_1.NodeTypes.LiquidBranch && node.name === liquid_html_parser_1.NamedTags.catch;
+}
+//# sourceMappingURL=extract-undefined-variables.js.map

package/dist/checks/metadata-params/extract-undefined-variables.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"extract-undefined-variables.js","sourceRoot":"","sources":["../../../src/checks/metadata-params/extract-undefined-variables.ts"],"names":[],"mappings":";;AAgCA,8DAkKC;AAlMD,uEAqBwC;AAIxC;;;;;;GAMG;AACH,SAAgB,yBAAyB,CACvC,MAAc,EACd,oBAA8B,EAAE;IAEhC,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,eAAe,GAAyB,IAAI,GAAG,EAAE,CAAC;IACxD,MAAM,mBAAmB,GAAgB,IAAI,GAAG,CAAC,iBAAiB,CAAC,CAAC;IACpE,MAAM,SAAS,GAA2B,EAAE,CAAC;IAE7C,SAAS,kBAAkB,CAAC,YAA2B,EAAE,KAAY;QACnE,IAAI,CAAC,YAAY;YAAE,OAAO;QAC1B,MAAM,YAAY,GAAG,eAAe,CAAC,GAAG,CAAC,YAAY,CAAC,IAAI,EAAE,CAAC;QAC7D,eAAe,CAAC,GAAG,CAAC,YAAY,EAAE,YAAY,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC;IAChE,CAAC;IAED,SAAS,IAAI,CAAC,IAAoB,EAAE,SAA2B;QAC7D,2CAA2C;QAC3C,IAAI,IAAI,CAAC,IAAI,KAAK,8BAAS,CAAC,SAAS,EAAE,CAAC;YACtC,eAAe,CAAC,IAAI,EAAE,SAAS,CAAC,CAAC;QACnC,CAAC;QAED,sDAAsD;QACtD,IAAI,IAAI,CAAC,IAAI,KAAK,8BAAS,CAAC,YAAY,EAAE,CAAC;YACzC,kBAAkB,CAAC,IAAI,CAAC,CAAC;QAC3B,CAAC;QAED,0BAA0B;QAC1B,IAAI,IAAI,CAAC,IAAI,KAAK,8BAAS,CAAC,cAAc,EAAE,CAAC;YAC3C,oBAAoB,CAAC,IAAI,EAAE,SAAS,CAAC,CAAC;QACxC,CAAC;QAED,wBAAwB;QACxB,MAAM,YAAY,GAAG,SAAS,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC;QAC5C,KAAK,MAAM,KAAK,IAAI,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,EAAE,CAAC;YACxC,IAAI,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC,EAAE,CAAC;gBACzB,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;oBACzB,IAAI,MAAM,CAAC,IAAI,CAAC,EAAE,CAAC;wBACjB,IAAI,CAAC,IAAI,EAAE,YAAY,CAAC,CAAC;oBAC3B,CAAC;gBACH,CAAC;YACH,CAAC;iBAAM,IAAI,MAAM,CAAC,KAAK,CAAC,EAAE,CAAC;gBACzB,IAAI,CAAC,KAAK,EAAE,YAAY,CAAC,CAAC;YAC5B,CAAC;QACH,CAAC;IACH,CAAC;IAED,SAAS,eAAe,CAAC,IAAe,EAAE,UAA4B;QACpE,IAAI,iBAAiB,CAAC,IAAI,CAAC,IAAI,kBAAkB,CAAC,IAAI,CAAC,IAAI,oBAAoB,CAAC,IAAI,CAAC,EAAE,CAAC;YACtF,kBAAkB,CAAC,IAAI,CAAC,MAAM,CAAC,IAAI,EAAE;gBACnC,KAAK,EAAE,IAAI,CAAC,kBAAkB,CAAC,GAAG;aACnC,CAAC,CAAC;QACL,CAAC;QAED,IAAI,qBAAqB,CAAC,IAAI,CAAC,IAAI,IAAI,CAAC,MAAM,CAAC,MAAM,CAAC,IAAI,EAAE,CAAC;YAC3D,kBAAkB,CAAC,IAAI,CAAC,MAAM,CAAC,MAAM,CAAC,IAAI,EAAE;gBAC1C,KAAK,EAAE,IAAI,CAAC,kBAAkB,CAAC,GAAG;aACnC,CAAC,CAAC;QACL,CAAC;QAED,IAAI,kBAAkB,CAAC,IAAI,CAAC,EAAE,CAAC;YAC7B,kBAAkB,CAAC,IAAI,CAAC,MAAM,CAAC,IAAI,EAAE;gBACnC,KAAK,EAAE,IAAI,CAAC,gBAAgB,EAAE,GAAG;aAClC,CAAC,CAAC;QACL,CAAC;QAED,IAAI,IAAI,CAAC,IAAI,KAAK,MAAM,EAAE,CAAC;YACzB,kBAAkB,CAAC,IAAI,CAAC,IAAI,EAAE;gBAC5B,KAAK,EAAE,IAAI,CAAC,kBAAkB,CAAC,GAAG;gBAClC,GAAG,EAAE,IAAI,CAAC,gBAAgB,EAAE,KAAK;aAClC,CAAC,CAAC;QACL,CAAC;QAED,IAAI,IAAI,CAAC,IAAI,KAAK,UAAU,EAAE,CAAC;YAC7B,MAAM,MAAM,GAAI,IAAI,CAAC,MAAyB,CAAC,IAAI,CAAC;YACpD,IAAI,MAAM,CAAC,OAAO,CAAC,MAAM,KAAK,CAAC,IAAI,MAAM,CAAC,IAAI,KAAK,IAAI,EAAE,CAAC;gBACxD,kBAAkB,CAAC,MAAM,CAAC,IAAI,EAAE;oBAC9B,KAAK,EAAE,IAAI,CAAC,QAAQ,CAAC,GAAG;iBACzB,CAAC,CAAC;YACL,CAAC;QACH,CAAC;QAED,IAAI,CAAC,oBAAoB,CAAC,IAAI,CAAC,IAAI,oBAAoB,CAAC,IAAI,CAAC,CAAC,IAAI,IAAI,CAAC,MAAM,CAAC,IAAI,KAAK,IAAI,EAAE,CAAC;YAC5F,kBAAkB,CAAC,IAAI,CAAC,MAAM,CAAC,IAAI,EAAE;gBACnC,KAAK,EAAE,IAAI,CAAC,QAAQ,CAAC,KAAK;aAC3B,CAAC,CAAC;QACL,CAAC;QAED,IAAI,cAAc,CAAC,IAAI,CAAC,IAAI,mBAAmB,CAAC,IAAI,CAAC,EAAE,CAAC;YACtD,kBAAkB,CAAC,IAAI,CAAC,MAAM,CAAC,YAAY,EAAE;gBAC3C,KAAK,EAAE,IAAI,CAAC,kBAAkB,CAAC,GAAG;gBAClC,GAAG,EAAE,IAAI,CAAC,gBAAgB,EAAE,KAAK;aAClC,CAAC,CAAC;YACH,kBAAkB,CAAC,IAAI,CAAC,IAAI,KAAK,KAAK,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,cAAc,EAAE;gBACnE,KAAK,EAAE,IAAI,CAAC,kBAAkB,CAAC,GAAG;gBAClC,GAAG,EAAE,IAAI,CAAC,gBAAgB,EAAE,KAAK;aAClC,CAAC,CAAC;QACL,CAAC;QAED,IAAI,qBAAqB,CAAC,IAAI,CAAC,EAAE,CAAC;YAChC,kBAAkB,CAAC,IAAI,CAAC,MAAM,CAAC,KAAK,EAAE;gBACpC,KAAK,EAAE,IAAI,CAAC,QAAQ,CAAC,GAAG;aACzB,CAAC,CAAC;QACL,CAAC;IACH,CAAC;IAED,SAAS,kBAAkB,CAAC,IAA8D;QACxF,IACE,IAAI,CAAC,IAAI,KAAK,8BAAS,CAAC,KAAK;YAC7B,IAAI,CAAC,MAAM;YACX,OAAO,IAAI,CAAC,MAAM,KAAK,QAAQ;YAC/B,MAAM,IAAI,IAAI,CAAC,MAAM;YACpB,IAAI,CAAC,MAAc,CAAC,IAAI,EACzB,CAAC;YACD,kBAAkB,CAAE,IAAI,CAAC,MAAc,CAAC,IAAI,EAAE;gBAC5C,KAAK,EAAG,IAAY,CAAC,kBAAkB,CAAC,GAAG;gBAC3C,GAAG,EAAG,IAAY,CAAC,gBAAgB,EAAE,KAAK;aAC3C,CAAC,CAAC;QACL,CAAC;IACH,CAAC;IAED,SAAS,oBAAoB,CAAC,IAA0B,EAAE,SAA2B;QACnF,MAAM,MAAM,GAAG,SAAS,CAAC,SAAS,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC;QAE/C,IAAI,WAAW,CAAC,MAAM,CAAC,IAAI,kBAAkB,CAAC,MAAM,CAAC;YAAE,OAAO;QAC9D,IAAI,WAAW,CAAC,MAAM,CAAC,IAAI,oBAAoB,CAAC,MAAM,CAAC;YAAE,OAAO;QAChE,IAAI,gBAAgB,CAAC,MAAM,CAAC,IAAI,MAAM,CAAC,IAAI,KAAK,IAAI;YAAE,OAAO;QAC7D,IAAI,mBAAmB,CAAC,MAAM,CAAC,IAAI,MAAM,CAAC,MAAM,KAAK,IAAI;YAAE,OAAO;QAClE,IAAI,kBAAkB,CAAC,MAAM,CAAC,IAAI,MAAM,CAAC,MAAM,KAAK,IAAI;YAAE,OAAO;QAEjE,SAAS,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACvB,CAAC;IAED,IAAI,CAAC,GAAG,EAAE,EAAE,CAAC,CAAC;IAEd,gCAAgC;IAChC,MAAM,IAAI,GAAG,IAAI,GAAG,EAAU,CAAC;IAC/B,MAAM,MAAM,GAAa,EAAE,CAAC;IAE5B,KAAK,MAAM,QAAQ,IAAI,SAAS,EAAE,CAAC;QACjC,IAAI,CAAC,QAAQ,CAAC,IAAI;YAAE,SAAS;QAC7B,IAAI,IAAI,CAAC,GAAG,CAAC,QAAQ,CAAC,IAAI,CAAC;YAAE,SAAS;QAEtC,MAAM,iBAAiB,GAAG,SAAS,CACjC,QAAQ,CAAC,IAAI,EACb,QAAQ,CAAC,QAAQ,EACjB,mBAAmB,EACnB,eAAe,CAChB,CAAC;QAEF,IAAI,CAAC,iBAAiB,EAAE,CAAC;YACvB,IAAI,CAAC,GAAG,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC;YACxB,MAAM,CAAC,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC;QAC7B,CAAC;IACH,CAAC;IAED,OAAO,MAAM,CAAC;AAChB,CAAC;AAED,SAAS,MAAM,CAAC,CAAM;IACpB,OAAO,CAAC,KAAK,IAAI,IAAI,OAAO,CAAC,KAAK,QAAQ,IAAI,OAAO,CAAC,CAAC,IAAI,KAAK,QAAQ,CAAC;AAC3E,CAAC;AAED,SAAS,SAAS,CAChB,YAAoB,EACpB,gBAA0B,EAC1B,mBAAgC,EAChC,eAAqC;IAErC,IAAI,mBAAmB,CAAC,GAAG,CAAC,YAAY,CAAC,EAAE,CAAC;QAC1C,OAAO,IAAI,CAAC;IACd,CAAC;IAED,MAAM,MAAM,GAAG,eAAe,CAAC,GAAG,CAAC,YAAY,CAAC,CAAC;IACjD,IAAI,CAAC,MAAM,EAAE,CAAC;QACZ,OAAO,KAAK,CAAC;IACf,CAAC;IAED,OAAO,MAAM,CAAC,IAAI,CAAC,CAAC,KAAK,EAAE,EAAE;QAC3B,MAAM,KAAK,GAAG,gBAAgB,CAAC,KAAK,CAAC;QACrC,MAAM,yBAAyB,GAAG,CAAC,KAAK,CAAC,KAAK,IAAI,KAAK,GAAG,KAAK,CAAC,KAAK,CAAC;QACtE,MAAM,wBAAwB,GAAG,CAAC,KAAK,CAAC,GAAG,IAAI,KAAK,GAAG,KAAK,CAAC,GAAG,CAAC;QACjE,OAAO,yBAAyB,IAAI,wBAAwB,CAAC;IAC/D,CAAC,CAAC,CAAC;AACL,CAAC;AAED,SAAS,WAAW,CAAC,IAAqB;IACxC,OAAO,IAAI,EAAE,IAAI,KAAK,8BAAS,CAAC,SAAS,CAAC;AAC5C,CAAC;AAED,SAAS,kBAAkB,CAAC,IAAe;IACzC,OAAO,IAAI,CAAC,IAAI,KAAK,8BAAS,CAAC,OAAO,CAAC;AACzC,CAAC;AAED,SAAS,iBAAiB,CAAC,IAAe;IACxC,OAAO,IAAI,CAAC,IAAI,KAAK,8BAAS,CAAC,MAAM,IAAI,OAAO,IAAI,CAAC,MAAM,KAAK,QAAQ,CAAC;AAC3E,CAAC;AAED,SAAS,qBAAqB,CAAC,IAAe;IAC5C,OAAO,IAAI,CAAC,IAAI,KAAK,8BAAS,CAAC,WAAW,IAAI,OAAO,IAAI,CAAC,MAAM,KAAK,QAAQ,CAAC;AAChF,CAAC;AAED,SAAS,kBAAkB,CAAC,IAAe;IACzC,OAAO,IAAI,CAAC,IAAI,KAAK,8BAAS,CAAC,OAAO,IAAI,OAAO,IAAI,CAAC,MAAM,KAAK,QAAQ,CAAC;AAC5E,CAAC;AAED,SAAS,oBAAoB,CAAC,IAAe;IAC3C,OAAO,IAAI,CAAC,IAAI,KAAK,8BAAS,CAAC,UAAU,IAAI,OAAO,IAAI,CAAC,MAAM,KAAK,QAAQ,CAAC;AAC/E,CAAC;AAED,SAAS,cAAc,CAAC,IAAe;IACrC,OAAO,IAAI,CAAC,IAAI,KAAK,8BAAS,CAAC,GAAG,IAAI,OAAO,IAAI,CAAC,MAAM,KAAK,QAAQ,CAAC;AACxE,CAAC;AAED,SAAS,mBAAmB,CAAC,IAAe;IAC1C,OAAO,IAAI,CAAC,IAAI,KAAK,8BAAS,CAAC,QAAQ,IAAI,OAAO,IAAI,CAAC,MAAM,KAAK,QAAQ,CAAC;AAC7E,CAAC;AAED,SAAS,oBAAoB,CAAC,IAAe;IAC3C,OAAO,IAAI,CAAC,IAAI,KAAK,8BAAS,CAAC,SAAS,IAAI,OAAO,IAAI,CAAC,MAAM,KAAK,QAAQ,CAAC;AAC9E,CAAC;AAED,SAAS,oBAAoB,CAAC,IAAe;IAC3C,OAAO,IAAI,CAAC,IAAI,KAAK,8BAAS,CAAC,SAAS,IAAI,OAAO,IAAI,CAAC,MAAM,KAAK,QAAQ,CAAC;AAC9E,CAAC;AAED,SAAS,qBAAqB,CAC5B,IAAe;IAEf,OAAO,CACL,IAAI,CAAC,IAAI,KAAK,8BAAS,CAAC,UAAU;QAClC,OAAO,IAAI,CAAC,MAAM,KAAK,QAAQ;QAC/B,IAAI,CAAC,MAAM,CAAC,IAAI,KAAK,8BAAS,CAAC,gBAAgB,CAChD,CAAC;AACJ,CAAC;AAED,SAAS,kBAAkB,CAAC,IAAqB;IAC/C,OAAO,IAAI,EAAE,IAAI,KAAK,8BAAS,CAAC,gBAAgB,CAAC;AACnD,CAAC;AAED,SAAS,gBAAgB,CAAC,IAAqB;IAC7C,OAAO,IAAI,EAAE,IAAI,KAAK,8BAAS,CAAC,cAAc,CAAC;AACjD,CAAC;AAED,SAAS,mBAAmB,CAC1B,IAAqB;IAErB,OAAO,IAAI,EAAE,IAAI,KAAK,8BAAS,CAAC,YAAY,IAAK,IAAY,CAAC,IAAI,KAAK,8BAAS,CAAC,KAAK,CAAC;AACzF,CAAC"}

package/dist/checks/metadata-params/index.js

@@ -1,36 +1,17 @@
 "use strict";
-var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.MetadataParamsCheck = void 0;
 const types_1 = require("../../types");
-const js_yaml_1 = __importDefault(require("js-yaml"));
 const platformos_common_1 = require("@platformos/platformos-common");
 const vscode_uri_1 = require("vscode-uri");
 const path_1 = require("../../path");
-function extractMetadataParams(source) {
-    source = source.trim();
-    if (!source.startsWith('---'))
-        return null;
-    const end = source.indexOf('---', 3);
-    if (end === -1)
-        return null;
-    const yamlBlock = source.slice(3, end).trim();
-    try {
-        const result = js_yaml_1.default.load(yamlBlock);
-        return Object.keys(result.metadata.params);
-    }
-    catch (e) {
-        return null;
-    }
-}
+const extract_undefined_variables_1 = require("./extract-undefined-variables");
 exports.MetadataParamsCheck = {
     meta: {
         code: 'MetadataParamsCheck',
         name: 'Metadata Params Check',
         docs: {
-            description: 'Ensures that parameters referenced in the document exist in metadata.params or in the doc tag.',
+            description: 'Ensures that parameters referenced in the document exist in the doc tag or are inferred from undefined variables.',
             recommended: true,
             url: undefined,
         },
@@ -46,21 +27,55 @@ exports.MetadataParamsCheck = {
             if (!locatedFile) {
                 return;
             }
-            let params = extractMetadataParams(await context.fs.readFile(locatedFile));
-            if (!params) {
-                if (!context.getDocDefinition)
-                    return;
-                const relativePath = (0, path_1.relative)(locatedFile, context.config.rootUri);
-                const docDef = await context.getDocDefinition(relativePath);
-                if (!docDef?.liquidDoc?.parameters)
-                    return;
-                const liquidDocParameters = new Map(docDef.liquidDoc.parameters.map((p) => [p.name, p]));
-                params = Array.from(liquidDocParameters.values())
+            const source = await context.fs.readFile(locatedFile);
+            const relativePath = (0, path_1.relative)(locatedFile, context.config.rootUri);
+            let requiredParams;
+            let allowedParams;
+            // Check for @doc tag first — if present, it's the complete param list
+            const docDef = context.getDocDefinition
+                ? await context.getDocDefinition(relativePath)
+                : undefined;
+            if (docDef?.liquidDoc?.parameters) {
+                const globalObjectNames = [];
+                if (context.platformosDocset) {
+                    const objects = await context.platformosDocset.objects();
+                    for (const obj of objects) {
+                        if (!obj.access || obj.access.global === true || obj.access.template.length > 0) {
+                            globalObjectNames.push(obj.name);
+                        }
+                    }
+                }
+                const undefinedVars = (0, extract_undefined_variables_1.extractUndefinedVariables)(source, globalObjectNames);
+                const docRequiredNames = docDef.liquidDoc.parameters
                     .filter((p) => p.required)
                     .map((p) => p.name);
+                requiredParams = docRequiredNames.filter((name) => undefinedVars.includes(name));
+                allowedParams = docDef.liquidDoc.parameters.map((p) => p.name);
+            }
+            else {
+                // No @doc — scan for undefined variables, treat all as required
+                const globalObjectNames = [];
+                if (context.platformosDocset) {
+                    const objects = await context.platformosDocset.objects();
+                    for (const obj of objects) {
+                        if (!obj.access || obj.access.global === true || obj.access.template.length > 0) {
+                            globalObjectNames.push(obj.name);
+                        }
+                    }
+                }
+                if (relativePath.includes('views/partials/') || relativePath.includes('/lib/')) {
+                    if (!globalObjectNames.includes('app')) {
+                        globalObjectNames.push('app');
+                    }
+                }
+                const undefinedVars = (0, extract_undefined_variables_1.extractUndefinedVariables)(source, globalObjectNames);
+                if (undefinedVars.length === 0)
+                    return;
+                requiredParams = undefinedVars;
+                allowedParams = undefinedVars;
             }
             args
-                .filter((arg) => !params.includes(arg.name))
+                .filter((arg) => !allowedParams.includes(arg.name))
                 .forEach((arg) => {
                 context.report({
                     message: `Unknown parameter ${arg.name} passed to ${nodeType} call`,
@@ -68,7 +83,7 @@ exports.MetadataParamsCheck = {
                     endIndex: arg.position.end,
                 });
             });
-            params
+            requiredParams
                 .filter((param) => !args.find((arg) => arg.name === param))
                 .forEach((param) => {
                 context.report({