find-complexity

Находит функции и методы с высокой цикломатической сложностью, чрезмерной длиной или слишком большим количеством параметров. Используется, когда пользователь просит найти сложный код, сложность…

npx skills add https://github.com/openshift/lightspeed-service --skill find-complexity

Find Complexity Hotspots

Identify functions that are hard to review, test, and maintain.

Rules

  • Report findings, do not refactor. Refactoring is a separate task.
  • Focus on production code (ols/). Skip tests unless explicitly asked.
  • Rank by severity: highest complexity first.

Step 1: Determine Scope

Ask the user:

  • Branch mode: only files changed in the current branch vs main.
  • Full mode: scan the entire ols/ directory.

For branch mode:

git diff --name-only origin/main -- 'ols/' | grep '\.py$'

Step 2: Cyclomatic Complexity

uvx radon cc <target> -s -n C -a

This shows functions with complexity grade C or worse (threshold: 11+).

Grades: A (1-5), B (6-10), C (11-15), D (16-20), E (21-25), F (26+).

Step 3: Maintainability Index

uvx radon mi <target> -s -n B

This shows files with maintainability grade B or worse.

Grades: A (20+, good), B (10-19, medium), C (0-9, poor).

Step 4: Cognitive Complexity

Cognitive complexity weights nesting depth — a 5-deep if scores much higher than 5 sequential ifs.

uvx --with flake8-cognitive-complexity flake8 --select=CCR001 --max-cognitive-complexity=10 <target>

Step 5: Function Length

Find long functions (30+ lines of logic, excluding docstrings and blank lines):

uvx radon raw <target> -s

Also use pylint for method length:

uv run pylint --disable=all --enable=too-many-statements,too-many-branches,too-many-return-statements,too-many-arguments,too-many-locals <target>

Step 6: File Size

Find large files (500+ lines):

wc -l $(find <target> -name '*.py') | sort -rn | head -20

Files over 500 lines are candidates for splitting into focused modules.

Step 7: Classify Findings

For each function found, classify:

CategoryCriteriaAction
SplitHigh complexity + long bodyBreak into smaller functions
SimplifyHigh complexity + short bodyReduce branching (early returns, lookup tables)
ParameterizeToo many arguments (6+)Group into config/dataclass
MonitorGrade C, not growingNote it, revisit if it gets worse
Split fileFile over 500 linesBreak into focused modules

Step 8: Report

For each finding:

  1. File, function name, line number
  2. Complexity grade and score
  3. Number of statements / arguments
  4. Classification (split / simplify / parameterize / monitor)
  5. Brief suggestion

Summary: total hotspots by grade, top 5 worst offenders.

Больше skills от openshift

openshift-expert
openshift
Эксперт по платформе OpenShift и Kubernetes с глубокими знаниями архитектуры кластеров, операторов, сетей, хранилищ, устранения неполадок и CI/CD-пайплайнов. Используйте…
official
find-token
openshift
Найти скрытый верификационный токен. Запустите скрипт find-token, чтобы получить уникальный токен.
official
code-review
openshift
Проверить пул-реквест на качество кода, корректность и соответствие стандартам проекта. Используется, когда пользователь просит провести ревью PR, проверить код или изучить изменения в…
official
css-review
openshift
Проверка CSS на соответствие стилю кодирования, использование токенов PatternFly и лучшие практики. Используйте, когда пользователь просит проверить CSS, оценить стили или провести аудит CSS-файлов.
official
review-readmes
openshift
Проверить все файлы README.md в репозитории на опечатки, ошибки и устаревшую информацию. Использовать, когда пользователь просит проверить README, точность документации или…
official
review-skills
openshift
Проверка AI-навыков проекта на дублирование, устаревшие ссылки, ошибки и структурные проблемы. Используется, когда пользователь просит проверить навыки, провести аудит навыков, проверить на…
official
test
openshift
Запустить сквозные тесты, отфильтрованные по тегу. Используйте, когда пользователь просит запустить тесты, запустить Playwright или протестировать конкретный тег функции, например @core или @attach.
official
unused-exports
openshift
Находит экспортированные символы, которые никогда не импортируются другим файлом. Используется, когда пользователь говорит «проверить экспорт», «неиспользуемый экспорт» или просит очистить экспорт.
official