feat(devex): changelog.d 碎片化——终结 [Unreleased] 单行道冲突税 (#174)

Analysed against 89e87bb22 files changedby 2233admin
Open PR on GitHub

Change-risk score

9.4out of 10Elevated
Repository health
7.3base against this head

This change sits in the top third of this repository's own risk distribution, the band worth reviewing closely. It is riskier than 86% of this repository's own commits.

No changed file lost health in this diff. Nothing outside this PR depends on a contract it changed.

Files changed
2
in this diff
Contracts changed
0
removed or signature
Outside callers
0
not in this diff
Findings added
16
0 resolved

Blast radius on the repo map

Every file in the repository, grouped into its own directories and sized by lines. The ones this PR changes are lit, and everything that imports them is marked.

cratesdocsorchestrationprototy…skills/code-in…code-intel-cliplansevidenceschemassrctestsaudit_reportclimcp_serve
changed, health fellchanged, health rosechanged, health heldimports a changed fileuntouchedchanged by this PR

Drawing 574 of 748 files at a readable size, grouped by directory and sized by lines of code. 2 changed here reach 0 more by import. 174 are too small to see at this width, which is what the frame below is for.

Changed

What this PR added, and what it took away

Introduced findings are the ones inside the lines this PR added, not the file's existing ones. The resolved column is the only place in the product that lists them.

Introduced (16)
  • highbumpy road
    tools/aggregate_changelog.py:235

    merge_into_section_body has 5 nested blocks at the same level (bumpy road)

  • highcomplex method
    tools/aggregate_changelog.py:235

    merge_into_section_body has cyclomatic complexity 15

  • highcomplex method
    tools/aggregate_changelog.py:112

    parse_fragment has cyclomatic complexity 19

  • mediumnested complexity
    tools/aggregate_changelog.py:538

    main nests 4 levels deep

  • mediumlow cohesion
    tests/test_aggregate_changelog.py:70

    ParseFragmentTests has low cohesion (LCOM4=2): its 6 methods split into 2 groups that share no fields or calls

  • mediumduplicated assertion block
    tests/test_aggregate_changelog.py:143

    assertion block at lines 143-144 is duplicated in tests/test_aggregate_changelog.py

  • mediumduplicated assertion block
    tests/test_aggregate_changelog.py:202

    assertion block at lines 202-203 is duplicated in tests/test_aggregate_changelog.py

  • mediumcomplex method
    tools/aggregate_changelog.py:538

    main has cyclomatic complexity 14

  • mediumcomplex method
    tools/aggregate_changelog.py:415

    check_pr_advisory has cyclomatic complexity 13

  • mediumcomplex method
    tools/aggregate_changelog.py:306

    apply_fragments_to_changelog has cyclomatic complexity 13

  • lowdry violation
    tests/test_aggregate_changelog.py:129

    15% of file duplicated; worst clone shares 15 lines with tests/test_aggregate_changelog.py

  • lowbumpy road
    tools/aggregate_changelog.py:538

    main has 3 nested blocks at the same level (bumpy road)

  • lowbumpy road
    tools/aggregate_changelog.py:306

    apply_fragments_to_changelog has 3 nested blocks at the same level (bumpy road)

  • lowbumpy road
    tools/aggregate_changelog.py:112

    parse_fragment has 3 nested blocks at the same level (bumpy road)

  • lowerror handling
    tools/aggregate_changelog.py:32

    caught exception is swallowed without any handling

  • lowlarge method
    tools/aggregate_changelog.py:538

    main is 61 lines long

Resolved (0)
  • Nothing removed.

Get this on your own pull requests

This page came from an index of 2233admin/code-intel-pipeline. No model calls, no configuration, and it refreshes on every push. Install the bot and every pull request gets one of these.

feat(devex): changelog.d 碎片化——终结 [Unreleased] 单行道冲突税 (#174) · 2233admin/code-intel-pipeline #187 · Repowise