Rectify: Issue #4341 — Schema-Driven Recipe Section Pagination

[Trecek]

Summary

get_recipe_section currently destroys a section's type before pagination. The architectural correction retains and validates the typed value, resolves an explicit RecipeSectionDef, constructs an immutable grammar-aware page plan, and renders and measures each complete final response.

The implementation adds schema-driven section definitions, deterministic grammar-aware pagination, fixed-point metadata convergence, immutable plan identities, strict consumer validation, bounded failure handling, cache retirement, and exhaustive verification for raw text, scalar JSON, arrays, and oversized element fragments.

Closes #4341

Implementation Plan

Plan file: /home/talon/projects/generic_automation_mcp/.autoskillit/temp/rectify/rectify_issue_4341_schema_driven_recipe_section_pagination_2026-07-23_095431.md

🤖 Generated with Claude Code via AutoSkillit

[Trecek]

AutoSkillit PR Review — Verdict: approved_with_comments

[Trecek]

[warning] cohesion: The PR raises the server root file-count exemption while simultaneously creating server/recipe_section for this feature: the 914-line planner/cache/renderer remains in the parent and only verification is placed in the subpackage. This weakens the architectural guard and splits one private feature asymmetrically; localize the implementation under the new package instead of increasing the exemption.

[Trecek]

Valid observation — flagged for design decision. The asymmetry is real, but the originating plan and current server/AGENTS.md deliberately place planner/cache/renderer at the server root and verification in the supporting package. Moving the full feature is an architectural migration beyond a targeted review fix.

[Trecek]

AutoSkillit review: warning-only findings detected. See inline comments — no blocking changes required.

[Trecek]

AutoSkillit PR Review — Verdict: changes_requested

[Trecek]

AutoSkillit review: blocking findings detected. See inline comments. GitHub does not permit an author to request changes on their own pull request.

[Trecek]

AutoSkillit PR Review — Verdict: approved_with_comments

[Trecek]

AutoSkillit review: warning-only findings detected. See inline comments — no blocking changes required.

[Trecek]

AutoSkillit PR Review — Verdict: approved_with_comments

[Trecek]

AutoSkillit review: warning-only findings detected. See inline comments — no blocking changes required.

[Trecek]

AutoSkillit PR Review — Verdict: changes_requested

[Trecek]

AutoSkillit review: blocking findings detected. See the inline comments; changes are required before approval.