shape-doctor.py: five functions carry a cognitive complexity Sonar caps at 15 — the two placement functions, the leg-shape-files and members checks, and the verdict ladder, from 22-33 down to the limit with behaviour byte-identical #134

Lane: openreposhape-2 (openRepoShape-2)

What is wrong. SonarCloud has five open python:S3776 issues against
shape-doctor.py, one per function whose cognitive complexity is over the 15
the rule allows. Every one of them is a check or a verdict — the parts of this
file a reader goes to when the doctor said something they did not expect — and
each is long enough that the answer is found by tracing a ladder rather than by
reading a sentence.

Evidence, sonarcloud.io/api/issues/search?componentKeys=opensoft_openRepoShape
— issue keys AaCN_tEhr0kW89h1tjDk, AaCOssrMuNydAlqVuM1I,
AaCOssrMuNydAlqVuM1J, AaCN_tEhr0kW89h1tjDl, AaCN_tEhr0kW89h1tjDm, all
CRITICAL, all OPEN, message "Refactor this function to reduce its Cognitive
Complexity from N to the 15 allowed."

check_leg_shape_files walks every leg, classifies every shape file through a
five-way if/elif chain, and then decides which of the missing files a copy
would even fix — three questions in one body. check_placement opens with four
different n/a refusals before it has classified a single path.
placement_plan_text writes four blocks of one YAML file in one run of
appends. check_members builds a member's state, judges it, and composes the
note in one loop. verdict_for is the precedence ladder the module docstring
argues at length, with the argument sitting inside it as comments between
branches.

What would fix it. Extract a helper per decision, in this file's existing
style — audit_leg and placement_row are already exactly that, split out of
check_placement for the same reason — moving each WHY comment into the
docstring of the helper the reasoning went with. Behaviour byte-identical: no
message text, exit code, output order, file written or exception type changes.
quote_arg stays the one quoter, copy_command stays the only place a shell
verb is spelled, the CHECKS registry keeps its shape and its empty fix
slots, and the LABEL_* constants keep naming the rows.

Note that main() is NOT one of the five: it measures 10 and is under the
limit already, so no argparse text moves and docs/cli.md is untouched.