[routes] cookbook_helpers.py at 475 lines exceeds AGENTS.md ~400 line limit #785

New issue

Closed

opened 2026-06-03 00:32:30 +02:00 by sleepy · 0 comments

sleepy commented

2026-06-03 00:32:30 +02:00

Owner

Problem

routes/cookbook_helpers.py is 475 lines containing validation functions, request models, shell helpers, and parsing utilities all mixed together.

Extracted from cookbook_routes.py (itself 1920 lines, see #775) but still exceeds the ~400 line guideline.

AGENTS.md violation

Split files that exceed ~400 lines or contain multiple concerns.

8 regex constants for validation
8 validation functions
Shell path and quoting helpers
Serve phase parser
Environment setup helpers
2 Pydantic request models
Cached model scan script builder

Fix

Split into routes/cookbook/validators.py, routes/cookbook/shell_helpers.py, routes/cookbook/models.py

## Problem `routes/cookbook_helpers.py` is 475 lines containing validation functions, request models, shell helpers, and parsing utilities all mixed together. Extracted from `cookbook_routes.py` (itself 1920 lines, see #775) but still exceeds the ~400 line guideline. ## AGENTS.md violation > Split files that exceed ~400 lines or contain multiple concerns. ## Contents - 8 regex constants for validation - 8 validation functions - Shell path and quoting helpers - Serve phase parser - Environment setup helpers - 2 Pydantic request models - Cached model scan script builder ## Fix Split into `routes/cookbook/validators.py`, `routes/cookbook/shell_helpers.py`, `routes/cookbook/models.py`

sleepy referenced this issue from a commit

2026-06-03 22:04:51 +02:00

[routes] Split cookbook_helpers.py under 400-line limit (#785)

sleepy referenced this issue from a pull request that will close it,

2026-06-03 22:05:10 +02:00

[routes] Split cookbook_helpers.py under 400-line limit (#785) #854

sleepy closed this issue

2026-06-03 22:05:30 +02:00