@shopify/shop-minis-react 0.0.0-snapshot.20251211173028 → 0.0.0-snapshot.20251211183509

package/dist/_virtual/index4.js

@@ -1,5 +1,5 @@
-var e = { exports: {} };
+var r = {};
 export {
-  e as __module
+  r as __exports
 };
 //# sourceMappingURL=index4.js.map

package/dist/_virtual/index5.js

@@ -1,6 +1,5 @@
-import { __require as r } from "../shop-minis-react/node_modules/.pnpm/@xmldom_xmldom@0.8.10/node_modules/@xmldom/xmldom/lib/index.js";
-var i = r();
+var e = { exports: {} };
 export {
-  i as l
+  e as __module
 };
 //# sourceMappingURL=index5.js.map

package/dist/_virtual/index5.js.map

@@ -1 +1 @@
-{"version":3,"file":"index5.js","sources":[],"sourcesContent":[],"names":[],"mappings":";;"}
+{"version":3,"file":"index5.js","sources":[],"sourcesContent":[],"names":[],"mappings":";"}

package/dist/_virtual/index6.js

@@ -1,5 +1,6 @@
-var r = {};
+import { __require as r } from "../shop-minis-react/node_modules/.pnpm/@xmldom_xmldom@0.8.10/node_modules/@xmldom/xmldom/lib/index.js";
+var i = r();
 export {
-  r as __exports
+  i as l
 };
 //# sourceMappingURL=index6.js.map

package/dist/_virtual/index6.js.map

@@ -1 +1 @@
-{"version":3,"file":"index6.js","sources":[],"sourcesContent":[],"names":[],"mappings":";"}
+{"version":3,"file":"index6.js","sources":[],"sourcesContent":[],"names":[],"mappings":";;"}

package/dist/shop-minis-react/node_modules/.pnpm/@videojs_xhr@2.7.0/node_modules/@videojs/xhr/lib/index.js

@@ -1,4 +1,4 @@
-import { __module as q } from "../../../../../../../../_virtual/index4.js";
+import { __module as q } from "../../../../../../../../_virtual/index5.js";
 import { __require as F } from "../../../../../global@4.4.0/node_modules/global/window.js";
 import { __require as N } from "../../../../../@babel_runtime@7.27.6/node_modules/@babel/runtime/helpers/extends.js";
 import { __require as J } from "../../../../../is-function@1.0.2/node_modules/is-function/index.js";

package/dist/shop-minis-react/node_modules/.pnpm/mpd-parser@1.3.1/node_modules/mpd-parser/dist/mpd-parser.es.js

@@ -2,7 +2,7 @@ import L from "../../../../@videojs_vhs-utils@4.1.1/node_modules/@videojs/vhs-ut
 import T from "../../../../../../../_virtual/window.js";
 import { forEachMediaGroup as Z } from "../../../../@videojs_vhs-utils@4.1.1/node_modules/@videojs/vhs-utils/es/media-groups.js";
 import J from "../../../../@videojs_vhs-utils@4.1.1/node_modules/@videojs/vhs-utils/es/decode-b64-to-uint8-array.js";
-import { l as Q } from "../../../../../../../_virtual/index5.js";
+import { l as Q } from "../../../../../../../_virtual/index6.js";
 /*! @name mpd-parser @version 1.3.1 @license Apache-2.0 */
 const w = (e) => !!e && typeof e == "object", E = (...e) => e.reduce((n, t) => (typeof t != "object" || Object.keys(t).forEach((r) => {
   Array.isArray(n[r]) && Array.isArray(t[r]) ? n[r] = n[r].concat(t[r]) : w(n[r]) && w(t[r]) ? n[r] = E(n[r], t[r]) : n[r] = t[r];

package/dist/shop-minis-react/node_modules/.pnpm/querystringify@2.2.0/node_modules/querystringify/index.js

@@ -1,4 +1,4 @@
-import { __exports as i } from "../../../../../../_virtual/index6.js";
+import { __exports as i } from "../../../../../../_virtual/index4.js";
 var c;
 function d() {
   if (c) return i;

package/eslint/rules/no-dynamic-asset-paths.cjs

@@ -3,32 +3,46 @@
  * @fileoverview Warns about template literals that may construct asset paths dynamically
  */
 
-// eslint-disable-next-line import/extensions
-const {ASSET_EXTENSIONS, REMOTE_PATTERNS} = require('./asset-path-patterns.cjs')
+const {
+  ASSET_EXTENSIONS,
+  LOCAL_PATH_PATTERNS,
+  REMOTE_PATTERNS,
+  // eslint-disable-next-line import/extensions
+} = require('./asset-path-patterns.cjs')
 
 /**
- * Check if a template literal might contain an asset path
+ * Check if a template literal might contain an asset path but we can't be certain
+ * Returns true only for UNCERTAIN cases (ends with asset extension but doesn't
+ * start with a known local path pattern). Definite cases are handled by
+ * no-hardcoded-asset-paths rule.
  * @param {object} node - The TemplateLiteral AST node
  * @returns {boolean}
  */
-function templateLiteralHasAssetExtension(node) {
-  // Check all the static parts (quasis) of the template literal
-  for (const quasi of node.quasis) {
-    const value = quasi.value.raw || quasi.value.cooked || ''
-
-    // If any static part ends with an asset extension, flag it
-    if (ASSET_EXTENSIONS.test(value)) {
-      // Make sure it's not part of a remote URL
-      const fullTemplate = node.quasis
-        .map(q => q.value.raw || q.value.cooked || '')
-        .join('')
-      if (REMOTE_PATTERNS.some(pattern => pattern.test(fullTemplate))) {
-        return false
-      }
-      return true
-    }
-  }
-  return false
+function isUncertainAssetTemplateLiteral(node) {
+  const quasis = node.quasis
+  if (quasis.length === 0) return false
+
+  // Get the first static part to check if it starts with a local path
+  const firstPart = quasis[0].value.raw || quasis[0].value.cooked || ''
+
+  // Get the last static part to check if it ends with an asset extension
+  const lastPart =
+    quasis[quasis.length - 1].value.raw ||
+    quasis[quasis.length - 1].value.cooked ||
+    ''
+
+  // Must end with an asset extension
+  if (!ASSET_EXTENSIONS.test(lastPart)) return false
+
+  // Check if it starts with a remote URL (only need to check first static part)
+  if (REMOTE_PATTERNS.some(pattern => pattern.test(firstPart))) return false
+
+  // If it starts with a local path pattern, it's a DEFINITE error (handled by other rule)
+  // We only want to flag UNCERTAIN cases here
+  if (LOCAL_PATH_PATTERNS.some(pattern => pattern.test(firstPart))) return false
+
+  // Ends with asset extension but doesn't start with known local path = uncertain
+  return true
 }
 
 module.exports = {
@@ -51,7 +65,8 @@ module.exports = {
   create(context) {
     return {
       TemplateLiteral(node) {
-        if (templateLiteralHasAssetExtension(node)) {
+        // Only flag uncertain cases (definite cases handled by no-hardcoded-asset-paths)
+        if (isUncertainAssetTemplateLiteral(node)) {
           context.report({
             node,
             messageId: 'noDynamicAssetPath',

package/eslint/rules/no-hardcoded-asset-paths.cjs

@@ -28,6 +28,36 @@ function isLocalAssetPath(value) {
   return LOCAL_PATH_PATTERNS.some(pattern => pattern.test(value))
 }
 
+/**
+ * Check if a template literal is definitely a local asset path
+ * Returns true only if we can determine with certainty it's a local path
+ * (starts with a local path pattern AND ends with an asset extension)
+ * @param {object} node - The TemplateLiteral AST node
+ * @returns {boolean}
+ */
+function isLocalAssetTemplateLiteral(node) {
+  const quasis = node.quasis
+  if (quasis.length === 0) return false
+
+  // Get the first static part to check if it starts with a local path
+  const firstPart = quasis[0].value.raw || quasis[0].value.cooked || ''
+
+  // Get the last static part to check if it ends with an asset extension
+  const lastPart =
+    quasis[quasis.length - 1].value.raw ||
+    quasis[quasis.length - 1].value.cooked ||
+    ''
+
+  // Must end with an asset extension
+  if (!ASSET_EXTENSIONS.test(lastPart)) return false
+
+  // Check if it starts with a remote URL (only need to check first static part)
+  if (REMOTE_PATTERNS.some(pattern => pattern.test(firstPart))) return false
+
+  // Must start with a local path pattern (this makes it a definite error)
+  return LOCAL_PATH_PATTERNS.some(pattern => pattern.test(firstPart))
+}
+
 module.exports = {
   meta: {
     type: 'problem',
@@ -41,6 +71,8 @@ module.exports = {
     messages: {
       noHardcodedAssetPath:
         'Hardcoded asset path "{{path}}" will not work in production. Import the asset instead: `import asset from "{{path}}"` then use the imported variable. See: https://vite.dev/guide/assets',
+      noHardcodedAssetPathTemplate:
+        'Template literal contains a hardcoded local asset path that will not work in production. Import assets instead of constructing paths. See: https://vite.dev/guide/assets',
     },
     schema: [],
   },
@@ -76,6 +108,17 @@ module.exports = {
           })
         }
       },
+
+      TemplateLiteral(node) {
+        // Only flag template literals that are DEFINITELY local asset paths
+        // (start with local path pattern AND end with asset extension)
+        if (isLocalAssetTemplateLiteral(node)) {
+          context.report({
+            node,
+            messageId: 'noHardcodedAssetPathTemplate',
+          })
+        }
+      },
     }
   },
 }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@shopify/shop-minis-react",
   "license": "SEE LICENSE IN LICENSE.txt",
-  "version": "0.0.0-snapshot.20251211173028",
+  "version": "0.0.0-snapshot.20251211183509",
   "sideEffects": false,
   "type": "module",
   "engines": {
@@ -46,7 +46,7 @@
     "typescript": ">=5.0.0"
   },
   "dependencies": {
-    "@shopify/shop-minis-platform": "0.0.0-snapshot.20251211173028",
+    "@shopify/shop-minis-platform": "0.0.0-snapshot.20251211183509",
     "@tailwindcss/vite": "4.1.8",
     "@tanstack/react-query": "5.86.0",
     "@types/color": "3.0.6",