@shopify/shop-minis-react 0.3.1 → 0.3.2

package/dist/shop-minis-react/node_modules/.pnpm/simple-swizzle@0.2.2/node_modules/simple-swizzle/index.js

@@ -1,4 +1,4 @@
-import { __module as t } from "../../../../../../_virtual/index10.js";
+import { __module as t } from "../../../../../../_virtual/index11.js";
 import { __require as z } from "../../../is-arrayish@0.3.2/node_modules/is-arrayish/index.js";
 var l;
 function v() {

package/dist/shop-minis-react/node_modules/.pnpm/use-sync-external-store@1.5.0_react@19.1.0/node_modules/use-sync-external-store/shim/index.js

@@ -1,4 +1,4 @@
-import { __module as r } from "../../../../../../../_virtual/index11.js";
+import { __module as r } from "../../../../../../../_virtual/index10.js";
 import { __require as o } from "../cjs/use-sync-external-store-shim.production.js";
 import { __require as i } from "../cjs/use-sync-external-store-shim.development.js";
 var e;

package/eslint/rules/prefer-sdk-hooks.cjs

@@ -14,9 +14,7 @@ module.exports = {
     },
     messages: {
       preferAsyncStorage:
-        'Use useAsyncStorage from @shopify/shop-minis-react instead of localStorage. The SDK hook provides async storage that works reliably in the Shop mini-app environment.',
-      preferSecureStorage:
-        'Use useSecureStorage from @shopify/shop-minis-react instead of localStorage for sensitive data. The SDK hook provides encrypted storage.',
+        'Use useAsyncStorage (or useSecureStorage for sensitive data) from @shopify/shop-minis-react instead of localStorage. The SDK hook provides async storage that works reliably in the Shop mini-app environment.',
     },
     schema: [
       {
@@ -40,9 +38,6 @@ module.exports = {
     const defaultApis = {
       localStorage: 'useAsyncStorage',
       sessionStorage: 'useAsyncStorage',
-      // Future additions will go here:
-      // navigator.geolocation: 'useGeolocation',
-      // window.history: 'useNavigation',
     }
 
     // Get user configuration or use defaults
@@ -55,7 +50,10 @@ module.exports = {
     return {
       MemberExpression(node) {
         // Check for direct access: localStorage.getItem()
-        if (node.object.type === 'Identifier' && apiMap[node.object.name]) {
+        if (
+          node.object.type === 'Identifier' &&
+          Object.hasOwn(apiMap, node.object.name)
+        ) {
           const apiName = node.object.name
           const sdkHook = apiMap[apiName]
 
@@ -77,7 +75,7 @@ module.exports = {
           (node.object.object.name === 'window' ||
             node.object.object.name === 'globalThis') &&
           node.object.property.type === 'Identifier' &&
-          apiMap[node.object.property.name]
+          Object.hasOwn(apiMap, node.object.property.name)
         ) {
           const apiName = node.object.property.name
           const sdkHook = apiMap[apiName]
@@ -112,12 +110,12 @@ module.exports = {
         }
 
         // Check if this is a direct reference to localStorage/sessionStorage
-        if (!apiMap[node.name]) {
+        if (!Object.hasOwn(apiMap, node.name)) {
           return
         }
 
-        // Skip if it's being declared as a variable
-        if (parent.type === 'VariableDeclarator' && parent.init === node) {
+        // Skip if it's the variable name being declared (e.g., const localStorage = ...)
+        if (parent.type === 'VariableDeclarator' && parent.id === node) {
           return
         }
 

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@shopify/shop-minis-react",
   "license": "SEE LICENSE IN LICENSE.txt",
-  "version": "0.3.1",
+  "version": "0.3.2",
   "sideEffects": false,
   "type": "module",
   "engines": {