docs: sync #112 complexity budget, tracing CI, and passive phases - #117
Draft
cursor[bot] wants to merge 1 commit into
Draft
docs: sync #112 complexity budget, tracing CI, and passive phases#117cursor[bot] wants to merge 1 commit into
cursor[bot] wants to merge 1 commit into
Conversation
Align operator docs with current main after #112: DEFAULT_THRESHOLD is the sole complexity budget (no inert mccabe decoration), the CI tracing job is documented, and the dev group no longer lists logfire (#104). Also map passive.py's named phase helpers from #107 and note the shallow checkout on the tracing lane. Co-authored-by: Michelle Rojas <mrojas54@users.noreply.github.com>
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
Documentation-only sync so operator docs match current
mainafter #112 (and the already-merged #102/#104/#107 surfaces).Docs added/updated
README.md— architecture, quality gate, Laminar section, troubleshooting; passive phase mapAGENTS.md— toolchain, suite counts, complexity ownership, passive phasesSPEC.md(S20 / S22),EVALUATION.md(S20 / S22),BUILDPLAN.md(T21 / T23)docs/superpowers/specs/2026-07-15-tech-debt-cicd-routine-design.md— two CI jobs + checkout depthsCodepaths covered
complexity_gate.DEFAULT_THRESHOLD/--isolatedC901 measurement (Fix two checks that appeared to run and didn't: inert complexity budget, CI-skipped real-SDK tracing test #112).github/workflows/ci.ymlgate+tracingjobs (Fix two checks that appeared to run and didn't: inert complexity budget, CI-skipped real-SDK tracing test #112)cli.py/observability//tracing.pyopt-in contract (Make CLI tracing explicitly opt in #104; CI lane added)passive.pynamed phase helpers (_plan_freeze→_resolve_corpus→_scan_refs) (refactor: close the Medium tech-debt findings from 20260803 #107)pyproject.toml— nologfireindev, no inert mccabe budgetKnowledge gaps addressed
[tool.ruff.lint.mccabe] max-complexityas the budget after Fix two checks that appeared to run and didn't: inert complexity budget, CI-skipped real-SDK tracing test #112 removed itdevgroup still mentionedlogfireafter Make CLI tracing explicitly opt in #104 removed ittracingjob undocumented (real-SDK tests could appear to "pass" via skip ongatealone)passive.pylooked like a monolith after refactor: close the Medium tech-debt findings from 20260803 #107's phase split--extra tracingNotes
Verification
complexity_gate.py,pyproject.toml,.github/workflows/ci.yml,passive.pyuv run pytest -q→ 748 passed / 4 skippeduv sync --extra tracing && uv run pytest -q→ 749 passed / 3 skipped