@nadicodeai/ui 12.1.36 → 12.1.37

package/dist/eslint/index.js

@@ -7,6 +7,7 @@ import { confirmDestructiveAction } from "./rules/confirm-destructive-action.js"
 import { drawerOwnsItsEdges } from "./rules/drawer-owns-its-edges.js";
 import { iconControlAccessibleName } from "./rules/icon-control-accessible-name.js";
 import { noAppFeatureComponent } from "./rules/no-app-feature-component.js";
+import { noBarePageNavigation } from "./rules/no-bare-page-navigation.js";
 import { noElevatedEdge } from "./rules/no-elevated-edge.js";
 import { noExistenceOnlyAssertion } from "./rules/no-existence-only-assertion.js";
 import { noNestedOverlay } from "./rules/no-nested-overlay.js";
@@ -119,6 +120,7 @@ const plugin = {
     "drawer-owns-its-edges": drawerOwnsItsEdges,
     "icon-control-accessible-name": iconControlAccessibleName,
     "no-app-feature-component": noAppFeatureComponent,
+    "no-bare-page-navigation": noBarePageNavigation,
     "no-elevated-edge": noElevatedEdge,
     "no-existence-only-assertion": noExistenceOnlyAssertion,
     "no-deprecated-zod-api": noDeprecatedZodApi,

package/dist/eslint/rules/no-bare-page-navigation.js

@@ -0,0 +1,46 @@
+/*
+ * The Portal's browser suite navigates through `tests/support/hydration.ts`.
+ * `page.goto` and `page.reload` resolve while React is still hydrating, and the
+ * interaction a spec dispatches next lands on server markup the shell is about
+ * to replace: #2173 lost six open menus that way, #2776 clicked "Create team"
+ * on a page that had not taken over yet, and #2837 read a prefetch as a
+ * refresh. `back` and `forward` are outside the rule, because they restore a
+ * tree the client already owns rather than loading a document.
+ */
+
+const NAVIGATIONS = new Map([
+  ["goto", "bareGoto"],
+  ["reload", "bareReload"],
+]);
+
+function propertyName(member) {
+  if (member.computed) return member.property.type === "Literal" ? member.property.value : null;
+  return member.property.type === "Identifier" ? member.property.name : null;
+}
+
+export const noBarePageNavigation = {
+  meta: {
+    type: "problem",
+    docs: {
+      description:
+        "Disallow a bare page.goto or page.reload in the Portal's browser suite; navigate through the helper that waits for the shell to hydrate",
+    },
+    schema: [],
+    messages: {
+      bareGoto:
+        "`goto()` resolves before React hydrates the page, so the next click or keypress lands on server markup the shell is about to replace (#2173, #2776, #2837). Open the page with `gotoPortal` from `tests/support/hydration`, which waits for the shell's `data-hydrated`, or with `gotoOutsideShell` for an address the dashboard shell never renders, such as login, signup or an app's consent page.",
+      bareReload:
+        "`reload()` resolves before React hydrates the page, so the next click or keypress lands on server markup the shell is about to replace (#2776). Reload with `reloadPortal` from `tests/support/hydration`, which waits for the shell's `data-hydrated`.",
+    },
+  },
+
+  create(context) {
+    return {
+      CallExpression(node) {
+        if (node.callee.type !== "MemberExpression") return;
+        const messageId = NAVIGATIONS.get(propertyName(node.callee));
+        if (messageId) context.report({ node, messageId });
+      },
+    };
+  },
+};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nadicodeai/ui",
-  "version": "12.1.36",
+  "version": "12.1.37",
   "type": "module",
   "publishConfig": {
     "access": "public"
@@ -77,7 +77,7 @@
     "@assistant-ui/react": "0.15.21",
     "@assistant-ui/react-markdown": "0.14.16",
     "@base-ui/react": "^1.8.0",
-    "@nadicodeai/design-system": "12.1.36",
+    "@nadicodeai/design-system": "12.1.37",
     "class-variance-authority": "^0.7.1",
     "clsx": "^2.1.1",
     "cmdk": "^1.1.1",