@yasainet/eslint 0.0.78 → 0.0.80

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yasainet/eslint",
-  "version": "0.0.78",
+  "version": "0.0.80",
   "description": "ESLint",
   "type": "module",
   "exports": {

package/src/cli/test-audit.mjs

@@ -2,14 +2,18 @@
 import fs from "node:fs";
 import path from "node:path";
 
-// pure layer (schemas / utils) の unit test presence を機械チェックする:
+// pure layer (schemas / utils) の unit test presence と、entries を持つ feature の
+// e2e presence を機械チェックする:
 //
-// - 各 source に 兄弟 *.test.ts が存在するか、`@unit-exempt:` marker を持つことを要求
-// - schemas は定義上 pure。utils は impure 混在のため marker で opt-out できる
+// - unit: 各 source に 兄弟 *.test.ts が存在するか、`@unit-exempt:` marker を持つことを要求
+//   - schemas は定義上 pure。utils は impure 混在のため marker で opt-out できる
+// - e2e: entries/ を持つ feature ごとに tests/e2e/<feature>/*.spec.ts の存在を要求
+//   - 配線 (entries) は unit でなく e2e で疎通を担保する方針の存在チェック
 // - ESLint の per-file モデルと噛み合わない「存在強制」を全ツリー一括監査で担う
 
 const REQUIRE_DIRS = new Set(["schemas", "utils"]);
 const EXEMPT_RE = /@unit-exempt:/;
+const E2E_ROOT = "tests/e2e";
 
 function getFlag(argv, name) {
   const i = argv.indexOf(name);
@@ -48,49 +52,102 @@ function isSatisfied(file) {
   return EXEMPT_RE.test(fs.readFileSync(file, "utf8"));
 }
 
+/** pure layer (schemas / utils) の unit test 不在を列挙する */
+function auditUnit(projectRoot, featuresDir) {
+  const violations = [];
+  for (const file of walk(featuresDir)) {
+    if (isTarget(file) && !isSatisfied(file)) {
+      violations.push(path.relative(projectRoot, file));
+    }
+  }
+  return violations.sort();
+}
+
+/** entries/ を持つ feature ごとに対応する e2e spec の不在を列挙する */
+function auditE2e(projectRoot, featureRoot, featuresDir) {
+  if (!fs.existsSync(featuresDir)) {
+    return [];
+  }
+  const violations = [];
+  for (const entry of fs.readdirSync(featuresDir, { withFileTypes: true })) {
+    if (!entry.isDirectory()) {
+      continue;
+    }
+    const feature = entry.name;
+    const hasEntries = fs.existsSync(path.join(featuresDir, feature, "entries"));
+    if (!hasEntries) {
+      continue;
+    }
+    const e2eDir = path.join(projectRoot, E2E_ROOT, feature);
+    const hasSpec = [...walk(e2eDir)].some((f) => f.endsWith(".spec.ts"));
+    if (!hasSpec) {
+      violations.push(
+        `${featureRoot}/${feature}/entries → ${E2E_ROOT}/${feature}/*.spec.ts`,
+      );
+    }
+  }
+  return violations.sort();
+}
+
 function main() {
   const argv = process.argv.slice(2);
 
   if (argv.includes("--help") || argv.includes("-h")) {
     console.log(
-      "test-audit — pure layer (schemas/utils) の unit test presence を検査\n\n" +
+      "test-audit — pure layer の unit test と feature の e2e presence を検査\n\n" +
         "Usage: test-audit [--feature-root <path>]\n\n" +
         "  --feature-root <path>  feature root (default: src/features)\n\n" +
-        "各 schemas/*.ts と utils/*.ts に 兄弟 *.test.ts または\n" +
-        "`// @unit-exempt: <理由>` marker を要求する。",
+        "  unit: 各 schemas/*.ts と utils/*.ts に 兄弟 *.test.ts または\n" +
+        "        `// @unit-exempt: <理由>` marker を要求する。\n" +
+        "  e2e:  entries/ を持つ feature ごとに\n" +
+        "        tests/e2e/<feature>/*.spec.ts を要求する。",
     );
     process.exit(0);
   }
 
   const projectRoot = process.cwd();
   const featureRoot = getFlag(argv, "--feature-root") ?? "src/features";
-  const roots = [path.join(projectRoot, featureRoot)];
+  const featuresDir = path.join(projectRoot, featureRoot);
 
-  const violations = [];
-  for (const root of roots) {
-    for (const file of walk(root)) {
-      if (isTarget(file) && !isSatisfied(file)) {
-        violations.push(path.relative(projectRoot, file));
-      }
+  const unitViolations = auditUnit(projectRoot, featuresDir);
+  const e2eViolations = auditE2e(projectRoot, featureRoot, featuresDir);
+
+  let failed = false;
+
+  if (unitViolations.length > 0) {
+    failed = true;
+    console.error(
+      `✗ test-audit(unit): ${unitViolations.length} 件の pure layer に unit test も @unit-exempt marker もありません:\n`,
+    );
+    for (const v of unitViolations) {
+      console.error(`  ${v}`);
     }
+    console.error(
+      "\n対応: 兄弟 *.test.ts を追加するか、impure な場合は\n" +
+        "`// @unit-exempt: <理由>` を記載する。\n",
+    );
   }
 
-  if (violations.length > 0) {
+  if (e2eViolations.length > 0) {
+    failed = true;
     console.error(
-      `✗ test-audit: ${violations.length} 件の pure layer に unit test も @unit-exempt marker もありません:\n`,
+      `✗ test-audit(e2e): ${e2eViolations.length} 件の feature に対応する e2e spec がありません:\n`,
     );
-    for (const v of violations.sort()) {
+    for (const v of e2eViolations) {
       console.error(`  ${v}`);
     }
     console.error(
-      "\n対応: 兄弟 *.test.ts を追加するか、impure な場合は\n" +
-        "`// @unit-exempt: <理由>` を記載する。",
+      "\n対応: tests/e2e/<feature>/*.spec.ts を追加する\n" +
+        "(配線の疎通は e2e で担保する)。\n",
     );
+  }
+
+  if (failed) {
     process.exit(1);
   }
 
   console.log(
-    "✓ test-audit: schemas / utils はすべて unit test または @unit-exempt が揃っています。",
+    "✓ test-audit: schemas / utils の unit と entries feature の e2e はすべて揃っています。",
   );
 }
 

package/src/common/_internal/constants.mjs

@@ -20,7 +20,7 @@ function findProjectRoot() {
 
 const PROJECT_ROOT = findProjectRoot();
 
-const EXCLUDE_LIST = ["type.ts", "proxy.ts", "utils.ts"];
+const EXCLUDE_LIST = ["types.ts", "proxy.ts", "utils.ts"];
 
 export function generatePrefixLibMapping(featureRoot) {
   const libRoot = featureRoot.replace(/features$/, "lib");

package/src/common/cross-cutting/jsdoc.mjs

@@ -2,6 +2,14 @@ import jsdocPlugin from "eslint-plugin-jsdoc";
 
 import { featuresGlob } from "../_internal/constants.mjs";
 
+/**
+ * queries / services の public 関数に JSDoc を必須化する:
+ *
+ * - 「非自明か」の免除判断を作らせないため require を決定論的に強制する
+ * - 冗長でも全関数で強制し例外ゼロ＝判断ゼロを優先する
+ * - description が Why-not か What かは機械判断できず Claude に委ねる
+ * - TS が型の真実源なので tags は強制しない（@param 等は二重写し）
+ */
 export function createJsdocConfigs({ featureRoot }) {
   return [
     {
@@ -9,7 +17,6 @@ export function createJsdocConfigs({ featureRoot }) {
       files: [
         ...featuresGlob(featureRoot, "**/queries/*.ts"),
         ...featuresGlob(featureRoot, "**/services*/*.ts"),
-        ...featuresGlob(featureRoot, "**/utils*/*.ts"),
       ],
       plugins: {
         jsdoc: jsdocPlugin,

package/src/common/cross-cutting/tests.mjs

@@ -0,0 +1,39 @@
+/**
+ * unit test (utils / schemas の兄弟 *.test.ts) から配線層への import を禁止する:
+ *
+ * - services / queries / entries を import すると mock の echo（写経）になる
+ * - pure (utils / schemas) のみを対象にし、配線の検証は e2e に委ねる
+ * - createUtilsConfigs より後に spread し、test.ts では本 rule を後勝ちにする
+ */
+export function createTestsConfigs({ featureRoot }) {
+  return [
+    {
+      name: "test/unit-no-wiring-import",
+      files: [`${featureRoot}/**/*.test.{ts,tsx}`],
+      rules: {
+        "no-restricted-imports": [
+          "error",
+          {
+            patterns: [
+              {
+                group: [
+                  "**/services/*",
+                  "**/services",
+                  "**/queries/*",
+                  "**/queries",
+                  "**/entries/*",
+                  "**/entries",
+                ],
+                message:
+                  "unit test は配線層 (services / queries / entries) を import 不可。" +
+                  "mock の echo になる:\n" +
+                  "- pure (utils / schemas) のみ unit する\n" +
+                  "- 配線の検証は e2e に委ねる",
+              },
+            ],
+          },
+        ],
+      },
+    },
+  ];
+}

package/src/common/index.mjs

@@ -11,6 +11,7 @@ import { createNamespaceImportConfigs } from "./cross-cutting/namespace-import.m
 import { createNoAnyReturnConfigs } from "./cross-cutting/no-any-return.mjs";
 import { createNoColocatedTestConfigs } from "./cross-cutting/no-colocated-test.mjs";
 import { createSupabaseColumnsSatisfiesConfigs } from "./cross-cutting/supabase-columns-satisfies.mjs";
+import { createTestsConfigs } from "./cross-cutting/tests.mjs";
 import { createConstantsConfigs } from "./layers/constants.mjs";
 import { createEntriesConfigs } from "./layers/entries.mjs";
 import { createTopLevelLibConfigs } from "./layers/top-level/lib.mjs";
@@ -47,6 +48,8 @@ export function createCommonConfigs(
     ...createEntriesConfigs(ctx),
     ...createFeaturesTsOnlyConfigs(ctx),
     ...createNoColocatedTestConfigs(ctx),
+    // createUtilsConfigs より後に置き、test.ts への配線 import 禁止を後勝ちにする。
+    ...createTestsConfigs(ctx),
     ...createNoAnyReturnConfigs(ctx),
     ...createFeatureDefaultImportsConfigs(ctx),
     ...createJsdocConfigs(ctx),

package/src/common/local-plugins/feature-name.mjs

@@ -95,9 +95,9 @@ export const featureNameRule = {
     const candidateTypePaths = [
       path.join(
         projectRoot,
-        featureRoot.replace(/features$/, "lib/supabase/type.ts"),
+        featureRoot.replace(/features$/, "lib/supabase/types.ts"),
       ),
-      path.join(projectRoot, "src/lib/supabase/type.ts"),
+      path.join(projectRoot, "src/lib/supabase/types.ts"),
     ];
     const supabaseTypePath =
       candidateTypePaths.find((p) => fs.existsSync(p)) ?? candidateTypePaths[0];