feat(pipeline): parallel complexity audit step with quality gates#1439
Draft
nextlevelshit wants to merge 2 commits into
Draft
feat(pipeline): parallel complexity audit step with quality gates#1439nextlevelshit wants to merge 2 commits into
nextlevelshit wants to merge 2 commits into
Conversation
…#1041) Adds an in-tree Go AST analyzer that scores functions by cyclomatic and cognitive complexity, exposed via `wave audit complexity` and a new `audit-complexity` pipeline. The pipeline plugs into `ops-parallel-audit`'s iterate fan-out alongside the LLM-driven audits, gating on per-function threshold breaches via shared-findings JSON. - internal/complexity: errgroup-parallel scorer, cyclomatic + cognitive visitors, golden-fixture tests, schema-validated findings emitter - cmd/wave/commands: `wave audit complexity` subcommand with thresholds, exclude globs, json/summary formats, exit codes 0/1/2 - internal/defaults/pipelines: audit-complexity.yaml command-step pipeline with json_schema gate; ops-parallel-audit fans out 4-wide - shared-findings schema: adds "complexity" to the type enum - docs/reference/cli.md: documents the new subcommand
walkExpr never read the nesting argument — it just threaded the value through to recursive calls. golangci-lint (unparam) flagged this as a real signal: the parameter cannot affect output. Drop it. Callers now invoke walkExpr(e) directly. Behaviour unchanged; the parameter was dead.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Related to #1041
Changes
Test Plan