Skip to content

Split scene_spec_generate_cmd under the complexity gate - #179

Merged
jmjava merged 2 commits into
mainfrom
feat/split-scene-spec-cmd
Oct 7, 2026
Merged

jmjava merged 2 commits into
mainfrom
feat/split-scene-spec-cmd

Conversation

@jmjava

@jmjava jmjava commented Oct 7, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • Split scene_spec_generate_cmd into helpers so the command is lizard CCN 3 and radon A, within CCN ≤ 10 and radon ≤ C.
  • Split the remaining src/docgen/cli.py commands that kept the file over the hotspot limit (image_generate_cmd, yaml_generate_cmd, benchmark, scene_compile, narration_generate, main, compose, _load_env). File max CCN is now 9. CLI behavior is unchanged.
  • Benchmark stays green against src/docgen/benchmark_data/baseline.json. Complexity, NLOC, and mutation ceilings are unchanged.

Test plan

  • pytest tests/ (903 passed; scratch-validate script passed on a Python that can import click)
  • docgen benchmark meets baseline
  • check-complexity.py passes
  • check-hotspots.py passes (src/docgen/cli.py maxCCN 9, limit 10)
  • CI lint, unit, benchmark, validate, mutation

jmjava and others added 2 commits October 7, 2026 19:39
…ty gate.

Co-authored-by: Cursor <cursoragent@cursor.com>
…hotspot CCN limit.

Co-authored-by: Cursor <cursoragent@cursor.com>
@jmjava
jmjava merged commit 8c8561f into main Oct 7, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant