@ai0x0/utils 0.0.35 → 0.0.37

package/README.md

@@ -63,19 +63,19 @@ export default [
 ];
 ```
 
-| Rule                      | Description                             |
-| ------------------------- | --------------------------------------- |
-| `no-hardcoded-style`      | Forbid hardcoded style values           |
-| `no-one-letter-vars`      | Forbid single-letter variable names     |
-| `no-then`                 | Prefer `async/await` over `.then()`     |
-| `no-antd-space`           | Use `<Flex>` instead of `<Space>`       |
-| `require-pro-components`  | Require ProComponents usage             |
-| `require-section-divider` | Large files need `===` comment dividers |
-| `no-consecutive-setstate` | Merge consecutive `setState` calls      |
-| `no-use-request-run`      | Forbid `useRequest.run()`               |
-| `require-use-form`        | Use `useForm()` for form state          |
-| `require-form-convention` | Form naming conventions                 |
-| `max-lines`               | Limit each file to 500 code lines       |
+| Rule                      | Description                                                             |
+| ------------------------- | ----------------------------------------------------------------------- |
+| `no-hardcoded-style`      | Forbid hardcoded style values                                           |
+| `no-one-letter-vars`      | Forbid single-letter variable names                                     |
+| `no-then`                 | Prefer `async/await` over `.then()`                                     |
+| `no-antd-space`           | Use `<Flex>` instead of `<Space>`                                       |
+| `require-pro-components`  | Require ProComponents usage                                             |
+| `require-section-divider` | Large files need `===` comment dividers                                 |
+| `no-consecutive-setstate` | Merge consecutive `setState` calls                                      |
+| `no-use-request-run`      | Forbid `useRequest.run()`                                               |
+| `require-use-form`        | Use `useForm()` for form state                                          |
+| `require-form-convention` | Form naming conventions                                                 |
+| `max-lines`               | Limit each file to 500 code lines and prompt splitting into child files |
 
 ### Shared restrictions (`eslint-config/`)
 

package/eslint-rules/index.js

@@ -11,6 +11,7 @@ import noUseRequestRun from "./no-use-request-run.js";
 import noThen from "./no-then.js";
 import noOneLetterVars from "./no-one-letter-vars.js";
 import noConsecutiveSetState from "./no-consecutive-setstate.js";
+import maxLines from "./max-lines.js";
 
 const rules = {
   "require-section-divider": requireSectionDivider,
@@ -23,6 +24,7 @@ const rules = {
   "no-then": noThen,
   "no-one-letter-vars": noOneLetterVars,
   "no-consecutive-setstate": noConsecutiveSetState,
+  "max-lines": maxLines,
 };
 
 const plugin = {
@@ -35,7 +37,7 @@ const plugin = {
           Object.keys(rules).map((name) => [`ai0x0/${name}`, "error"]),
         ),
         curly: "error",
-        "max-lines": [
+        "ai0x0/max-lines": [
           "error",
           { max: 500, skipBlankLines: true, skipComments: true },
         ],

package/eslint-rules/max-lines.js

@@ -0,0 +1,117 @@
+const DEFAULT_MAX = 500;
+
+function normalizeOptions(options = {}) {
+  if (typeof options === "number") {
+    return {
+      max: options,
+      skipBlankLines: true,
+      skipComments: true,
+    };
+  }
+  return {
+    max: options.max ?? DEFAULT_MAX,
+    skipBlankLines: options.skipBlankLines ?? true,
+    skipComments: options.skipComments ?? true,
+  };
+}
+
+function getCommentLines(sourceCode) {
+  const lines = new Set();
+  for (const comment of sourceCode.getAllComments()) {
+    const start = comment.loc.start.line;
+    const end = comment.loc.end.line;
+    for (let line = start; line <= end; line++) {
+      lines.add(line);
+    }
+  }
+  return lines;
+}
+
+/** @type {import('eslint').Rule.RuleModule} */
+const rule = {
+  meta: {
+    type: "layout",
+    docs: {
+      description: "限制单文件代码行数，超过时提示拆分子文件。",
+    },
+    schema: [
+      {
+        oneOf: [
+          {
+            type: "integer",
+            minimum: 0,
+          },
+          {
+            type: "object",
+            additionalProperties: false,
+            properties: {
+              max: {
+                type: "integer",
+                minimum: 0,
+              },
+              skipBlankLines: {
+                type: "boolean",
+              },
+              skipComments: {
+                type: "boolean",
+              },
+            },
+          },
+        ],
+      },
+    ],
+    messages: {
+      tooManyLines:
+        "单文件代码行数过多（{{ lineCount }} 行，最多 {{ max }} 行），请拆分为子文件。",
+    },
+  },
+  create(context) {
+    const options = normalizeOptions(context.options[0]);
+
+    return {
+      "Program:exit"(node) {
+        const sourceCode = context.getSourceCode();
+        const commentLines = options.skipComments
+          ? getCommentLines(sourceCode)
+          : new Set();
+        let lineCount = 0;
+        let reportLine = sourceCode.lines.length || 1;
+
+        for (let index = 0; index < sourceCode.lines.length; index++) {
+          const lineNumber = index + 1;
+          const line = sourceCode.lines[index];
+          if (options.skipBlankLines && line.trim() === "") {
+            continue;
+          }
+          if (options.skipComments && commentLines.has(lineNumber)) {
+            continue;
+          }
+
+          lineCount++;
+          if (lineCount === options.max + 1) {
+            reportLine = lineNumber;
+          }
+        }
+
+        if (lineCount <= options.max) {
+          return;
+        }
+
+        context.report({
+          node,
+          loc: {
+            line: reportLine,
+            column: 0,
+          },
+          messageId: "tooManyLines",
+          data: {
+            lineCount,
+            max: options.max,
+          },
+        });
+      },
+    };
+  },
+};
+
+export default rule;

package/eslint-rules/require-section-divider.js

@@ -3,11 +3,10 @@
  * ------------------------
  * 约束：大文件必须在顶层用 `=====` 等号注释块做分区。
  *
- * 分区要求（按行数）：
- *   - 150 ~ 349 行：至少 1 个分区块
- *   - 350 ~ 549 行：至少 2 个分区块
- *   - 550 ~ 749 行：至少 3 个分区块
- *   以此类推，每 200 行增加 1 个分区块。
+ * 分区要求：
+ *   - 行数只作为最低检查下限
+ *   - AI 应按业务模块、状态、数据流、工具函数等结构尽量多加分区注释
+ *   - 不要只按“150 行一个”机械补齐
  *
  * 分隔注释块格式（见 node_modules/@ai0x0/utils/.agents/skills/nextjs/SKILL.md）：
  *
@@ -78,7 +77,7 @@ const rule = {
     ],
     messages: {
       missing:
-        "大文件需要更多 `===` 等号注释块分区。当前需要 {{required}} 个，实际只有 {{actual}} 个。每约 150 行至少应有一个分区块。",
+        "大文件需要更多 `===` 等号注释块分区。当前最低需要 {{required}} 个，实际只有 {{actual}} 个。请 AI 按业务模块尽量多加分区注释，不要只按 150 行一个机械补齐。",
     },
   },
   create(context) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ai0x0/utils",
-  "version": "0.0.35",
+  "version": "0.0.37",
   "description": "AI0x0 utils",
   "keywords": [
     "ai0x0"