find-complexity

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

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

Find Complexity Hotspots

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

Rules

  • Report findings, do not refactor. Refactoring is a separate task.
  • Focus on production code (internal/, api/, cmd/). 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 codebase.

For branch mode:

git diff --name-only upstream/main -- 'internal/' 'api/' 'cmd/' | grep '\.go$' | grep -v '_test\.go$'

Step 2: Install Prerequisites

Install gocyclo and gocognit if not available:

go install github.com/fzipp/gocyclo/cmd/gocyclo@latest
go install github.com/uudashr/gocognit/cmd/gocognit@latest

Step 3: Cyclomatic Complexity

gocyclo -over 10 <target>

This shows functions with cyclomatic complexity over 10.

Thresholds: 1-10 (simple), 11-20 (moderate), 21-50 (complex), 51+ (untestable).

Step 4: Cognitive Complexity

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

gocognit -over 15 <target>

This shows functions with cognitive complexity over 15.

Step 5: Function Length

Find long functions (50+ lines of code, excluding comments and blank lines):

for file in $(find <target> -name '*.go' -not -name '*_test.go'); do
    awk '/^func / {start=NR; func=$0} 
         /^}/ && start {
           len=NR-start; 
           if(len>50) print FILENAME":"start": "func" ("len" lines)"
         }' "$file"
done

Step 6: Parameter Count

Find functions with too many parameters (6+):

rg "^func.*\([^)]{60,}\)" <target> -A 0

Functions with 6+ parameters are candidates for parameter objects or config structs.

Step 7: File Size

Find large files (500+ lines):

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

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

Step 8: Classify Findings

For each function found, classify:

CategoryCriteriaAction
SplitHigh complexity + long bodyBreak into smaller functions
SimplifyHigh complexity + short bodyReduce branching (early returns, switch statements)
ParameterizeToo many arguments (6+)Group into config struct
MonitorComplexity 11-15, not growingNote it, revisit if it gets worse
Split fileFile over 500 linesBreak into focused packages

Step 9: Report

For each finding:

  1. File, function name, line number
  2. Cyclomatic/cognitive complexity score
  3. Lines of code / parameter count
  4. Classification (split / simplify / parameterize / monitor)
  5. Brief suggestion

Summary: total hotspots, top 5 worst offenders, estimated refactoring effort.

Больше skills от openshift

openshift-docs
openshift
Поиск и чтение документации OpenShift Container Platform в формате markdown. Используйте, когда пользователь спрашивает о функциях OpenShift, конфигурации, установке,…
triage-leaked-infra
openshift
Оцените, безопасно ли удалять AWS VPC или набор инфраструктуры из HyperShift CI. Используйте, когда пользователь вставляет вывод cleanleaked и спрашивает «можно ли это удалить?», «безопасно…
openshift-expert
openshift
Эксперт по платформе OpenShift и Kubernetes с глубокими знаниями архитектуры кластеров, операторов, сетей, хранилищ, устранения неполадок и CI/CD-пайплайнов. Используйте…
Konflux Archived PipelineRuns
openshift
Обеспечивает доступ к архивированным Konflux PipelineRuns, TaskRuns и логам подов через KubeArchive. Автоматически применяется при проверке результатов Konflux PipelineRun, расследовании…
backport
openshift
Переносить коммиты или PR из основной ветки в релизную. Используется, когда пользователь просит выполнить backport, cherry-pick или перенос изменений между ветками, а также при разрешении…
rebase
openshift
Переместить текущую ветку на её базовую ветку, разрешить все конфликты и проверить, что линтинг, i18n и сборка проходят. Используйте, когда пользователь просит выполнить rebase, обновление или синхронизацию…
Build CPO Image
openshift
Собрать и отправить образ контейнера control-plane-operator. Автоматически применяется при тестировании изменений CPO, требующих развертывания на живом кластере.
find-complexity
openshift
Находит функции и методы с высокой цикломатической сложностью, чрезмерной длиной или слишком большим количеством параметров. Используется, когда пользователь просит найти сложный код, сложность…