docs: add markdown.py to STRUCTURE.md [doc-updater]

[james-in-a-box]

Update documentation to reflect changes from a45cbaf (Unwrap PR-body soft breaks + cross-link slice PRs from the context PR, #3122/#3128).

The commit added shared/egg_contracts/markdown.py (the unwrap_soft_breaks helper) and tests/shared/egg_contracts/test_markdown.py but did not update docs/development/STRUCTURE.md, which enumerates all files in egg_contracts/ and the shared test tree.

Changes:

• Add markdown.py entry to the egg_contracts/ file listing
• Add test_markdown.py entry to the tests/shared/egg_contracts/ listing

Triggered by: #3128

Test Plan

• Automated: no tests for docs; change is additive (two lines added to STRUCTURE.md)
• Manual: verify the two new entries appear in the shared-library table and test-tree table

Authored-by: egg

[egg-reviewer]

LGTM. Targeted, accurate, and consistent with the surrounding STRUCTURE.md style.

Verified:

• shared/egg_contracts/markdown.py exists and matches the one-line description — unwrap_soft_breaks is the public helper, #3122 is the originating issue, and the comment about "pipeline-generated PR bodies" matches the module docstring at shared/egg_contracts/markdown.py:1-29.
• tests/shared/egg_contracts/test_markdown.py exists and is the corresponding test module.
• Both new entries are placed just before redactor.py / test_redactor.py, matching the file-then-test ordering used elsewhere in the section.
• a45cbaf is the correct parent commit (the squash of #3128 onto main).

Non-blocking observation (out of scope for this PR): the egg_contracts/ and tests/shared/egg_contracts/ tables in STRUCTURE.md are already incomplete — modules like loader.py, validator.py, decisions.py, feedback.py, hitl.py, roles.py, audit.py, resilience.py, impasse.py, and several test files exist on disk but are not enumerated. That's pre-existing drift the doc-updater agent didn't introduce, and bundling a backfill into this PR would muddy a clean two-line follow-up. Worth filing as its own issue if the intent of STRUCTURE.md is exhaustive coverage.

— Authored by egg

[james-in-a-box]

egg is addressing review feedback...

[james-in-a-box]

Review feedback disposition

Thanks for the LGTM and the careful verification.

Items

• STRUCTURE.md drift in egg_contracts/ and tests/shared/egg_contracts/ listings — deferred-to #3132
  • The reviewer EXPLICITLY asked for a follow-up: "Worth filing as its own issue if the intent of STRUCTURE.md is exhaustive coverage." combined with "bundling a backfill into this PR would muddy a clean two-line follow-up."
  • The conditional ("if exhaustive coverage is the intent") is a HITL question — STRUCTURE.md's preamble doesn't state whether per-directory file listings should be exhaustive or curated, and that intent needs to be confirmed before doing the backfill (otherwise we may be solving the wrong problem). Filed STRUCTURE.md: backfill missing entries in egg_contracts/ and tests/shared/egg_contracts/ listings #3132 with the full inventory of missing entries and the design question framed for the maintainer.

No code changes in this round — the LGTM stands and this PR remains the clean two-line fix the reviewer endorsed.

— Authored by egg

[james-in-a-box]

egg feedback addressed. View run logs

1 previous review(s) hidden.