@stardeck-customer-apps/eslint-plugin 1.0.1 → 1.2.0

package/dist/index.cjs

@@ -1,7 +1,9 @@
 "use strict";
+var __create = Object.create;
 var __defProp = Object.defineProperty;
 var __getOwnPropDesc = Object.getOwnPropertyDescriptor;
 var __getOwnPropNames = Object.getOwnPropertyNames;
+var __getProtoOf = Object.getPrototypeOf;
 var __hasOwnProp = Object.prototype.hasOwnProperty;
 var __export = (target, all) => {
   for (var name in all)
@@ -15,6 +17,14 @@ var __copyProps = (to, from, except, desc) => {
   }
   return to;
 };
+var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__getProtoOf(mod)) : {}, __copyProps(
+  // If the importer is in node compatibility mode or this is not an ESM
+  // file that has been converted to a CommonJS file using a Babel-
+  // compatible transform (i.e. "__esModule" has not been set), then set
+  // "default" to the CommonJS "module.exports" for node compatibility.
+  isNodeMode || !mod || !mod.__esModule ? __defProp(target, "default", { value: mod, enumerable: true }) : target,
+  mod
+));
 var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: true }), mod);
 
 // src/index.ts
@@ -65,9 +75,187 @@ var rule = {
 };
 var no_template_literal_classname_default = rule;
 
+// src/rules/no-kysely-interactive-transaction.ts
+var rule2 = {
+  meta: {
+    type: "problem",
+    docs: {
+      description: "Disallow Kysely interactive transactions, which throw at runtime on the Neon HTTP driver used by data stores",
+      recommended: true
+    },
+    messages: {
+      noInteractiveTransaction: "The Neon HTTP driver used by data stores doesn't support interactive transactions, so `.transaction().execute(...)` throws at runtime. Combine the statements into a single query (e.g. a CTE via `db.with(...)`) or run them sequentially instead.",
+      noControlledTransaction: "The Neon HTTP driver used by data stores doesn't support controlled transactions, so `.startTransaction().execute()` throws at runtime. Combine the statements into a single query (e.g. a CTE via `db.with(...)`) or run them sequentially instead."
+    },
+    schema: []
+  },
+  create(context) {
+    return {
+      CallExpression(node) {
+        const callee = node.callee;
+        if (callee.type !== "MemberExpression") {
+          return;
+        }
+        const property = callee.property;
+        if (property.type !== "Identifier") {
+          return;
+        }
+        if (property.name !== "transaction" && property.name !== "startTransaction") {
+          return;
+        }
+        if (property.name === "startTransaction" && node.arguments.length === 0) {
+          context.report({
+            node,
+            messageId: "noControlledTransaction"
+          });
+          return;
+        }
+        if (property.name === "transaction" && node.arguments.length === 0) {
+          context.report({
+            node,
+            messageId: "noInteractiveTransaction"
+          });
+        }
+      }
+    };
+  }
+};
+var no_kysely_interactive_transaction_default = rule2;
+
+// src/rules/no-module-internal-import.ts
+var import_node_path = __toESM(require("path"), 1);
+var ALIAS_PREFIX = "@/";
+function toPosix(filePath) {
+  return filePath.split(import_node_path.default.sep).join("/");
+}
+function findSrcRoot(filename) {
+  const parts = toPosix(filename).split("/");
+  for (let i = 0; i < parts.length - 1; i++) {
+    if (parts[i] === "src" && parts[i + 1] === "modules") {
+      return parts.slice(0, i + 1).join("/");
+    }
+  }
+  let lastSrcIdx = -1;
+  for (let i = 0; i < parts.length; i++) {
+    if (parts[i] === "src") lastSrcIdx = i;
+  }
+  if (lastSrcIdx === -1) return null;
+  return parts.slice(0, lastSrcIdx + 1).join("/");
+}
+function normalizePosixPath(raw) {
+  const absolute = raw.startsWith("/");
+  const parts = raw.split("/");
+  const stack = [];
+  for (const part of parts) {
+    if (part === "" || part === ".") continue;
+    if (part === "..") {
+      if (stack.length === 0) return null;
+      stack.pop();
+      continue;
+    }
+    stack.push(part);
+  }
+  const joined = stack.join("/");
+  return absolute ? `/${joined}` : joined;
+}
+function dirnamePosix(filePath) {
+  const idx = filePath.lastIndexOf("/");
+  if (idx <= 0) return "/";
+  return filePath.slice(0, idx);
+}
+function resolveImportPath(filename, srcRoot, specifier) {
+  let candidate;
+  if (specifier.startsWith(ALIAS_PREFIX)) {
+    candidate = `${srcRoot}/${specifier.slice(ALIAS_PREFIX.length)}`;
+  } else if (specifier.startsWith("./") || specifier.startsWith("../")) {
+    candidate = `${dirnamePosix(toPosix(filename))}/${specifier}`;
+  } else {
+    return null;
+  }
+  const normalized = normalizePosixPath(candidate);
+  if (!normalized) return null;
+  if (normalized !== srcRoot && !normalized.startsWith(`${srcRoot}/`)) {
+    return null;
+  }
+  return normalized;
+}
+function getModuleName(filePath, srcRoot) {
+  if (!filePath.startsWith(`${srcRoot}/`)) return null;
+  const rel = filePath.slice(srcRoot.length + 1);
+  const match = /^modules\/([^/]+)/.exec(rel);
+  return match ? match[1] : null;
+}
+function isPublicSurface(resolvedPath, srcRoot, moduleName) {
+  const base = `${srcRoot}/modules/${moduleName}`;
+  if (resolvedPath === base) return true;
+  if (resolvedPath === `${base}/index`) return true;
+  if (resolvedPath === `${base}/index.ts`) return true;
+  if (resolvedPath === `${base}/index.tsx`) return true;
+  return false;
+}
+var rule3 = {
+  meta: {
+    type: "problem",
+    docs: {
+      description: "Disallow imports that reach into a vendored module's internals; import only through `@/modules/<name>` (its index)",
+      recommended: true
+    },
+    messages: {
+      noInternalImport: "Import '{{module}}' through its public surface ('@/modules/{{module}}') \u2014 '{{source}}' reaches into module internals, which breaks module install/update isolation."
+    },
+    schema: []
+  },
+  create(context) {
+    const filename = context.filename;
+    if (!filename || filename === "<input>" || filename === "<text>") {
+      return {};
+    }
+    const srcRoot = findSrcRoot(filename);
+    if (!srcRoot) {
+      return {};
+    }
+    const root = srcRoot;
+    const importerModule = getModuleName(toPosix(filename), root);
+    function check(node, sourceNode) {
+      if (!sourceNode || sourceNode.type !== "Literal") return;
+      const value = sourceNode.value;
+      if (typeof value !== "string") return;
+      const resolved = resolveImportPath(filename, root, value);
+      if (!resolved) return;
+      const targetModule = getModuleName(resolved, root);
+      if (!targetModule) return;
+      if (isPublicSurface(resolved, root, targetModule)) return;
+      if (importerModule === targetModule) return;
+      context.report({
+        node,
+        messageId: "noInternalImport",
+        data: { module: targetModule, source: value }
+      });
+    }
+    return {
+      ImportDeclaration(node) {
+        check(node, node.source);
+      },
+      ImportExpression(node) {
+        const src = node.source;
+        check(node, src);
+      },
+      ExportNamedDeclaration(node) {
+        check(node, node.source);
+      },
+      ExportAllDeclaration(node) {
+        check(node, node.source);
+      }
+    };
+  }
+};
+var no_module_internal_import_default = rule3;
+
 // src/rules/index.ts
 var rules = {
-  "no-template-literal-classname": no_template_literal_classname_default
+  "no-template-literal-classname": no_template_literal_classname_default,
+  "no-kysely-interactive-transaction": no_kysely_interactive_transaction_default,
+  "no-module-internal-import": no_module_internal_import_default
 };
 
 // src/configs/recommended.ts
@@ -79,7 +267,9 @@ var recommended = {
     }
   },
   rules: {
-    [`${pluginName}/no-template-literal-classname`]: "error"
+    [`${pluginName}/no-template-literal-classname`]: "error",
+    [`${pluginName}/no-kysely-interactive-transaction`]: "error",
+    [`${pluginName}/no-module-internal-import`]: "error"
   }
 };
 

package/dist/index.d.cts

@@ -1,20 +1,14 @@
 import * as eslint from 'eslint';
+import { Linter, ESLint } from 'eslint';
 
 declare const rules: {
     "no-template-literal-classname": eslint.Rule.RuleModule;
+    "no-kysely-interactive-transaction": eslint.Rule.RuleModule;
+    "no-module-internal-import": eslint.Rule.RuleModule;
 };
 
-declare const configs: {
-    recommended: eslint.Linter.Config<eslint.Linter.RulesRecord>;
-};
+declare const configs: Record<string, Linter.Config>;
 
-declare const plugin: {
-    rules: {
-        "no-template-literal-classname": eslint.Rule.RuleModule;
-    };
-    configs: {
-        recommended: eslint.Linter.Config<eslint.Linter.RulesRecord>;
-    };
-};
+declare const plugin: ESLint.Plugin;
 
 export { configs, plugin as default, rules };

package/dist/index.d.ts

@@ -1,20 +1,14 @@
 import * as eslint from 'eslint';
+import { Linter, ESLint } from 'eslint';
 
 declare const rules: {
     "no-template-literal-classname": eslint.Rule.RuleModule;
+    "no-kysely-interactive-transaction": eslint.Rule.RuleModule;
+    "no-module-internal-import": eslint.Rule.RuleModule;
 };
 
-declare const configs: {
-    recommended: eslint.Linter.Config<eslint.Linter.RulesRecord>;
-};
+declare const configs: Record<string, Linter.Config>;
 
-declare const plugin: {
-    rules: {
-        "no-template-literal-classname": eslint.Rule.RuleModule;
-    };
-    configs: {
-        recommended: eslint.Linter.Config<eslint.Linter.RulesRecord>;
-    };
-};
+declare const plugin: ESLint.Plugin;
 
 export { configs, plugin as default, rules };

package/dist/index.js

@@ -37,9 +37,187 @@ var rule = {
 };
 var no_template_literal_classname_default = rule;
 
+// src/rules/no-kysely-interactive-transaction.ts
+var rule2 = {
+  meta: {
+    type: "problem",
+    docs: {
+      description: "Disallow Kysely interactive transactions, which throw at runtime on the Neon HTTP driver used by data stores",
+      recommended: true
+    },
+    messages: {
+      noInteractiveTransaction: "The Neon HTTP driver used by data stores doesn't support interactive transactions, so `.transaction().execute(...)` throws at runtime. Combine the statements into a single query (e.g. a CTE via `db.with(...)`) or run them sequentially instead.",
+      noControlledTransaction: "The Neon HTTP driver used by data stores doesn't support controlled transactions, so `.startTransaction().execute()` throws at runtime. Combine the statements into a single query (e.g. a CTE via `db.with(...)`) or run them sequentially instead."
+    },
+    schema: []
+  },
+  create(context) {
+    return {
+      CallExpression(node) {
+        const callee = node.callee;
+        if (callee.type !== "MemberExpression") {
+          return;
+        }
+        const property = callee.property;
+        if (property.type !== "Identifier") {
+          return;
+        }
+        if (property.name !== "transaction" && property.name !== "startTransaction") {
+          return;
+        }
+        if (property.name === "startTransaction" && node.arguments.length === 0) {
+          context.report({
+            node,
+            messageId: "noControlledTransaction"
+          });
+          return;
+        }
+        if (property.name === "transaction" && node.arguments.length === 0) {
+          context.report({
+            node,
+            messageId: "noInteractiveTransaction"
+          });
+        }
+      }
+    };
+  }
+};
+var no_kysely_interactive_transaction_default = rule2;
+
+// src/rules/no-module-internal-import.ts
+import path from "path";
+var ALIAS_PREFIX = "@/";
+function toPosix(filePath) {
+  return filePath.split(path.sep).join("/");
+}
+function findSrcRoot(filename) {
+  const parts = toPosix(filename).split("/");
+  for (let i = 0; i < parts.length - 1; i++) {
+    if (parts[i] === "src" && parts[i + 1] === "modules") {
+      return parts.slice(0, i + 1).join("/");
+    }
+  }
+  let lastSrcIdx = -1;
+  for (let i = 0; i < parts.length; i++) {
+    if (parts[i] === "src") lastSrcIdx = i;
+  }
+  if (lastSrcIdx === -1) return null;
+  return parts.slice(0, lastSrcIdx + 1).join("/");
+}
+function normalizePosixPath(raw) {
+  const absolute = raw.startsWith("/");
+  const parts = raw.split("/");
+  const stack = [];
+  for (const part of parts) {
+    if (part === "" || part === ".") continue;
+    if (part === "..") {
+      if (stack.length === 0) return null;
+      stack.pop();
+      continue;
+    }
+    stack.push(part);
+  }
+  const joined = stack.join("/");
+  return absolute ? `/${joined}` : joined;
+}
+function dirnamePosix(filePath) {
+  const idx = filePath.lastIndexOf("/");
+  if (idx <= 0) return "/";
+  return filePath.slice(0, idx);
+}
+function resolveImportPath(filename, srcRoot, specifier) {
+  let candidate;
+  if (specifier.startsWith(ALIAS_PREFIX)) {
+    candidate = `${srcRoot}/${specifier.slice(ALIAS_PREFIX.length)}`;
+  } else if (specifier.startsWith("./") || specifier.startsWith("../")) {
+    candidate = `${dirnamePosix(toPosix(filename))}/${specifier}`;
+  } else {
+    return null;
+  }
+  const normalized = normalizePosixPath(candidate);
+  if (!normalized) return null;
+  if (normalized !== srcRoot && !normalized.startsWith(`${srcRoot}/`)) {
+    return null;
+  }
+  return normalized;
+}
+function getModuleName(filePath, srcRoot) {
+  if (!filePath.startsWith(`${srcRoot}/`)) return null;
+  const rel = filePath.slice(srcRoot.length + 1);
+  const match = /^modules\/([^/]+)/.exec(rel);
+  return match ? match[1] : null;
+}
+function isPublicSurface(resolvedPath, srcRoot, moduleName) {
+  const base = `${srcRoot}/modules/${moduleName}`;
+  if (resolvedPath === base) return true;
+  if (resolvedPath === `${base}/index`) return true;
+  if (resolvedPath === `${base}/index.ts`) return true;
+  if (resolvedPath === `${base}/index.tsx`) return true;
+  return false;
+}
+var rule3 = {
+  meta: {
+    type: "problem",
+    docs: {
+      description: "Disallow imports that reach into a vendored module's internals; import only through `@/modules/<name>` (its index)",
+      recommended: true
+    },
+    messages: {
+      noInternalImport: "Import '{{module}}' through its public surface ('@/modules/{{module}}') \u2014 '{{source}}' reaches into module internals, which breaks module install/update isolation."
+    },
+    schema: []
+  },
+  create(context) {
+    const filename = context.filename;
+    if (!filename || filename === "<input>" || filename === "<text>") {
+      return {};
+    }
+    const srcRoot = findSrcRoot(filename);
+    if (!srcRoot) {
+      return {};
+    }
+    const root = srcRoot;
+    const importerModule = getModuleName(toPosix(filename), root);
+    function check(node, sourceNode) {
+      if (!sourceNode || sourceNode.type !== "Literal") return;
+      const value = sourceNode.value;
+      if (typeof value !== "string") return;
+      const resolved = resolveImportPath(filename, root, value);
+      if (!resolved) return;
+      const targetModule = getModuleName(resolved, root);
+      if (!targetModule) return;
+      if (isPublicSurface(resolved, root, targetModule)) return;
+      if (importerModule === targetModule) return;
+      context.report({
+        node,
+        messageId: "noInternalImport",
+        data: { module: targetModule, source: value }
+      });
+    }
+    return {
+      ImportDeclaration(node) {
+        check(node, node.source);
+      },
+      ImportExpression(node) {
+        const src = node.source;
+        check(node, src);
+      },
+      ExportNamedDeclaration(node) {
+        check(node, node.source);
+      },
+      ExportAllDeclaration(node) {
+        check(node, node.source);
+      }
+    };
+  }
+};
+var no_module_internal_import_default = rule3;
+
 // src/rules/index.ts
 var rules = {
-  "no-template-literal-classname": no_template_literal_classname_default
+  "no-template-literal-classname": no_template_literal_classname_default,
+  "no-kysely-interactive-transaction": no_kysely_interactive_transaction_default,
+  "no-module-internal-import": no_module_internal_import_default
 };
 
 // src/configs/recommended.ts
@@ -51,7 +229,9 @@ var recommended = {
     }
   },
   rules: {
-    [`${pluginName}/no-template-literal-classname`]: "error"
+    [`${pluginName}/no-template-literal-classname`]: "error",
+    [`${pluginName}/no-kysely-interactive-transaction`]: "error",
+    [`${pluginName}/no-module-internal-import`]: "error"
   }
 };
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@stardeck-customer-apps/eslint-plugin",
-  "version": "1.0.1",
+  "version": "1.2.0",
   "description": "Custom ESLint rules for Stardeck customer apps",
   "type": "module",
   "main": "dist/index.js",
@@ -25,6 +25,7 @@
     "build": "tsup src/index.ts --format esm,cjs --dts",
     "dev": "tsup src/index.ts --format esm,cjs --dts --watch",
     "typecheck": "tsc --noEmit",
+    "test": "vitest run",
     "format": "prettier --write . && eslint . --fix",
     "lint": "eslint src/"
   },