AgentSkillsCN

plan-reduce-max-lines-per-function

在降低单个函数最大行数阈值并修复所有违规问题时，可优先选用此技能。它会更新 ESLint 阈值配置，识别出超出新限制的函数，生成包含重构策略的简报，并制定计划，列出拆分超大函数的任务。

SKILL.md
--- frontmatter
name: plan-reduce-max-lines-per-function
description: This skill should be used when reducing the maximum lines per function threshold and fixing all violations. It updates the eslint threshold configuration, identifies functions exceeding the new limit, generates a brief with refactoring strategies, and creates a plan with tasks to split oversized functions.
allowed-tools: ["Read", "Bash", "Glob", "Grep"]
argument-hint: "<max-lines-per-function-value>"

Reduce Max Lines Per Function

Target threshold: $ARGUMENTS lines per function

If no argument provided, prompt the user for a target.

Step 1: Gather Requirements

  1. •Read current config from eslint thresholds (eslint.thresholds.json or similar)
  2. •Run lint with the new threshold to find violations:
    bash
    bun run lint 2>&1 | grep "max-lines-per-function"
  3. •Note for each violation:
    • •File path and line number
    • •Function name
    • •Current line count

If no violations at $ARGUMENTS, report success and exit.

Step 2: Create Plan

In plan mode, create a plan that includes the following details:

markdown
Reduce max lines per function threshold to $ARGUMENTS.

## Functions Exceeding Threshold (ordered by line count)

1. src/services/user.ts:processUser (95 lines, target: $ARGUMENTS)
   - Line 45, function spans lines 45-140
2. src/utils/helpers.ts:validateInput (82 lines, target: $ARGUMENTS)
   - Line 23, function spans lines 23-105
...

## Configuration Change
- File: eslint.thresholds.json
- Change: maxLinesPerFunction to $ARGUMENTS

## Refactoring Strategies
- **Extract functions**: Break function into smaller named functions
- **Early returns**: Reduce nesting with guard clauses
- **Extract conditions**: Move complex boolean logic into named variables
- **Use lookup tables**: Replace complex switch/if-else chains with object maps
- **Consolidate logic**: Merge similar code paths

## Acceptance Criteria
- All functions at or below $ARGUMENTS lines
- `bun run lint` passes with no max-lines-per-function violations

## Verification
Command: `bun run lint 2>&1 | grep "max-lines-per-function" | wc -l`
Expected: 0