[utkarsh5026]

Step 3 of the code style work, for _bedrock.py, the module bedrock_kb.py and bedrock_chat.py share (after #90–#92).

What changed

Comments. All 90 # comments are gone:

• Price tables: where the prices were read, when, and what each key means are now in the module docstring.
• Dataclass fields: each class's fields are described in its docstring (KBDocument, Passage, Citation, ModelInfo, SavedFileList, _Json, _Answer).
• The rest moved into the docstrings of the functions they explained.

Long functions split into steps:

Before	After
_md_blocks: complexity 17, 75 lines	a loop over _md_block, which hands each kind of block to its reader: _md_code, _md_quote, _md_table (with _md_align), _md_list and _md_paragraph
_Markdown.parse: complexity 19	_inline_at, which hands each kind of inline markup to _code_span, _emphasis_node, _link_node, _autolink or _bare_url
_Markdown.link: complexity 12	_md_closing_bracket and _md_link_target_after
parse_models: complexity 11	_is_text_model, _served_models, _model_info (with _profile_preference) and _application_profile_model
_conditions: complexity 11	_operator_conditions for (operator, *values)

_bedrock.py comes off both allowance lists.

Nothing a user sees changed

• A differential test of old against new. main's _bedrock.py and this branch's gave byte-identical output on all of these:
  • 4,000 generated markdown answers (headings, lists nested in quotes, fences, tables, emphasis, links, bare URLs, escapes, raw HTML, javascript: links) with random citations: the HTML, the text lines and the [n] markers, 2.2 MB in all;
  • every where= form and its error messages;
  • parse_models across three regions, with profiles listed, unlisted and empty.
• Tests: CI on 3.10 passes, with 1,164 passed and rules at 0 errors. The full suite on 3.13 passes, with 1,180 passed.
• Reports: snapshot.py compare shows 319 of 321 outputs identical. The other two are the known traceback line numbers, the only "lost" names are the two from Code style: agent/ without comments, its tools as methods #92, and the AWS operations are unchanged.

No CHANGELOG entry: nothing a user sees changes.

🤖 Generated with Claude Code