[routes] session_routes.py at 1072 lines exceeds AGENTS.md ~400 line limit #778

New issue

Closed

opened 2026-06-03 00:29:20 +02:00 by sleepy · 0 comments

sleepy commented

2026-06-03 00:29:20 +02:00

Owner

Problem

routes/session_routes.py is 1072 lines with 23 endpoint handlers mixing:

Session CRUD (list, create, rename, delete, archive/unarchive)
Session import/export
Message injection
Session compaction
Auto-sort with LLM-based categorization
Context info retrieval
OpenAI-compatible session creation
Bulk operations (delete all, bulk delete)

The auto_sort_sessions function alone is ~220 lines with deeply nested LLM calls and fallback logic.

AGENTS.md violation

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

Suggested split

routes/session/crud.py — basic session CRUD
routes/session/export.py — export/import
routes/session/sort.py — auto-sort logic
routes/session/compact.py — session compaction
Move auto-sort business logic to a service

## Problem `routes/session_routes.py` is 1072 lines with 23 endpoint handlers mixing: - Session CRUD (list, create, rename, delete, archive/unarchive) - Session import/export - Message injection - Session compaction - Auto-sort with LLM-based categorization - Context info retrieval - OpenAI-compatible session creation - Bulk operations (delete all, bulk delete) The `auto_sort_sessions` function alone is ~220 lines with deeply nested LLM calls and fallback logic. ## AGENTS.md violation > Split files that exceed ~400 lines or contain multiple concerns. ## Suggested split - `routes/session/crud.py` — basic session CRUD - `routes/session/export.py` — export/import - `routes/session/sort.py` — auto-sort logic - `routes/session/compact.py` — session compaction - Move auto-sort business logic to a service

sleepy added the

refactor

area:routes

labels

2026-06-04 10:07:35 +02:00

sleepy referenced this issue from a commit

2026-06-04 12:52:48 +02:00

refactor: split session_routes.py into routes/session_routes/ (#778)

sleepy closed this issue

2026-06-04 12:53:07 +02:00

sleepy referenced this issue from a commit

2026-06-04 12:53:07 +02:00