[utkarsh5026]

What this changes

The first two steps of bringing the code in line with the new Code style rules in CLAUDE.md:

1. Every docstring is reshaped so its quotes sit on their own lines. That's 2,175 docstrings across src/, tests/ and .claude/skills/:

   def human_size(num_bytes: float | None) -> str:
       """
       1536 -> '1.5 KB' (binary units, like the AWS console).
       """

2. Checks so new code follows the rules:

   • tests/test_style.py fails on a docstring with text beside its quotes. It also fails on a file with more # comments than its allowance in COMMENTS_LEFT, and on one with fewer, until that number is lowered. So comments can only go down.
   • ruff.toml adds B006 (mutable defaults), BLE001 (broad except), ERA001 (commented-out code) and the complexity rules (C901, PLR0912, PLR0913, PLR0915). Files written before these rules keep what they already broke in per-file-ignores; a file comes off that list as it's cleaned up.

Nothing a user sees changed

• Docstring text and code: for every file, the syntax tree and each docstring's cleaned text (inspect.getdoc, which help() uses) are identical to main.
• Comments: every comment is identical to main, apart from one header comment removed by hand in agent/toolkits/base.py.
• Reports: snapshot.py compare shows 319 of 321 outputs identical. The other two are the line numbers in snapshot.py's own traceback for lambda-how-it-works. That case already fails on main, on Python 3.10: NameError: name 'mod' is not defined inside an exec'd list comprehension.
• Scripts: the ones that hand __doc__ to argparse now strip it, so --help reads as before.
• CI: check/run.py passes (ruff, imports, package, rules, pytest: 1,164 passed). Its one rules warning, execute_statement in dynamodb.py, was already there on main.

No CHANGELOG entry: nothing a user sees changes.

Next

Step 3: remove the comments and refactor long functions one module per pull request, starting with _kit/.

🤖 Generated with Claude Code