refactor(routes): move the email modules into routes/email/

routes/email_routes.py (7,194 lines), routes/email_helpers.py (2,025) and
routes/email_pollers.py (1,764) were the largest flat email area left in
routes/. They now live in routes/email/, following the pattern fifteen route
areas already use: the canonical module under the package, a shim at the old
path that replaces itself in sys.modules with the canonical object.

The shim matters more here than the move. Dozens of email tests monkeypatch
module attributes, and several pop the module from sys.modules and re-import
it. Without identity between routes.email_routes and routes.email.email_routes
those patches would apply to a different object than the app uses.

Intra-package imports keep the flat path, matching routes/document/ and
routes/gallery/: the shim resolves them and the diff stays a move.

Eight path references in five test modules read these files as source rather
than importing them, so they now read the canonical location. That is the same
adjustment every previous conversion made, and it is why the note/document
shims mention source-introspection tests explicitly.

No behaviour change. Verified with the full suite and under several explicit
collection orders, since a byte-identical routes move has broken tests under
one ordering before.

routes/email/__init__.py

@@ -0,0 +1,8 @@
+"""Email route domain package.
+
+Contains email_routes.py, email_helpers.py and email_pollers.py, migrated
+from the flat routes/ directory. Backward-compat shims at
+routes/email_routes.py, routes/email_helpers.py and routes/email_pollers.py
+replace themselves with these modules, so both import paths resolve to one
+object.
+"""

tests/test_action_menu_order.py

@@ -258,7 +258,7 @@ def test_unsubscribe_scan_status_sits_before_clean_action() -> None:
 
 def test_unsubscribe_success_removes_messages_before_the_next_scan() -> None:
     frontend = (ROOT / "static/js/emailLibrary.js").read_text(encoding="utf-8")
-    backend = (ROOT / "routes/email_routes.py").read_text(encoding="utf-8")
+    backend = (ROOT / "routes/email/email_routes.py").read_text(encoding="utf-8")
     mcp = (ROOT / "mcp_servers/email_server.py").read_text(encoding="utf-8")
     assert "async function _deleteAfterUnsubscribe" in frontend
     assert "action: 'delete'" in frontend[frontend.index("async function _deleteAfterUnsubscribe"):]
@@ -311,7 +311,7 @@ def test_email_mutation_tool_events_include_exact_arguments() -> None:
 
 
 def test_unsubscribe_cleanup_can_remove_same_sender_unsubscribe_messages() -> None:
-    source = (ROOT / "routes" / "email_routes.py").read_text()
+    source = (ROOT / "routes" / "email" / "email_routes.py").read_text()
     cleanup = source[source.index('@router.post("/unsubscribe/cleanup")'):source.index('@router.get("/contacts")')]
     assert 'scope == "sender_unsubscribe"' in cleanup
     assert "_unsubscribe_sender_uids_sync" in cleanup
@@ -372,7 +372,7 @@ def test_email_settings_sections_use_static_headers() -> None:
 
 
 def test_unsubscribe_scan_defaults_to_bounded_page_in_api_and_tool_prompt() -> None:
-    backend = (ROOT / "routes" / "email_routes.py").read_text()
+    backend = (ROOT / "routes" / "email" / "email_routes.py").read_text()
     schema = (ROOT / "src" / "tool_schemas.py").read_text()
     agent = (ROOT / "src" / "agent_loop.py").read_text()
     scan_start = backend.index('@router.get("/unsubscribe/scan")')

tests/test_direct_upload_limits.py

@@ -50,7 +50,7 @@ def test_direct_upload_routes_use_bounded_reads():
         "routes/calendar_routes.py": [
             "read_upload_limited(file, ICS_MAX_BYTES",
         ],
-        "routes/email_routes.py": [
+        "routes/email/email_routes.py": [
             "read_upload_limited(file, EMAIL_COMPOSE_UPLOAD_MAX_BYTES",
         ],
     }

tests/test_email_library_bulk_actions.py

@@ -3,7 +3,7 @@ from pathlib import Path
 
 _REPO = Path(__file__).resolve().parents[1]
 _EMAIL_LIBRARY = _REPO / "static" / "js" / "emailLibrary.js"
-_EMAIL_ROUTES = _REPO / "routes" / "email_routes.py"
+_EMAIL_ROUTES = _REPO / "routes" / "email" / "email_routes.py"
 _EMAIL_MCP_SERVER = _REPO / "mcp_servers" / "email_server.py"
 _EMAIL_FIXTURE_HELPER = _REPO / "scripts" / "ody_eval_email_fixture.py"
 

tests/test_imap_mailbox_quoting.py

@@ -86,7 +86,7 @@ def test_known_imap_mailbox_call_sites_are_quoted():
     assert 'conn.uid("MOVE", _b(uid), dest_folder)' not in mcp
     assert 'conn.uid("COPY", _b(uid), dest_folder)' not in mcp
 
-    pollers = Path("routes/email_pollers.py").read_text()
+    pollers = Path("routes/email/email_pollers.py").read_text()
     assert "conn.select(sent_name" not in pollers
     assert "imap.append(sent_folder" not in pollers
 

tests/test_upload_limits_centralized.py

@@ -86,7 +86,7 @@ def test_routes_import_from_upload_limits_not_local_defs():
         ],
         "routes/memory/memory_routes.py": ['int(os.getenv("ODYSSEUS_MEMORY_IMPORT_MAX_BYTES"'],
         "routes/personal_routes.py": ['os.getenv("ODYSSEUS_PERSONAL_UPLOAD_MAX_BYTES"'],
-        "routes/email_routes.py": ["EMAIL_COMPOSE_UPLOAD_MAX_BYTES = 25 * 1024 * 1024"],
+        "routes/email/email_routes.py": ["EMAIL_COMPOSE_UPLOAD_MAX_BYTES = 25 * 1024 * 1024"],
         "routes/stt_routes.py": ["STT_MAX_AUDIO_BYTES = 25 * 1024 * 1024"],
         "routes/calendar_routes.py": ["_ICS_MAX_BYTES = 10 * 1024 * 1024"],
     }
@@ -100,7 +100,7 @@ def test_routes_import_from_upload_limits_not_local_defs():
         "routes/gallery/gallery_routes.py": "GALLERY_UPLOAD_MAX_BYTES",
         "routes/memory/memory_routes.py": "MEMORY_IMPORT_MAX_BYTES",
         "routes/personal_routes.py": "PERSONAL_UPLOAD_MAX_BYTES",
-        "routes/email_routes.py": "EMAIL_COMPOSE_UPLOAD_MAX_BYTES",
+        "routes/email/email_routes.py": "EMAIL_COMPOSE_UPLOAD_MAX_BYTES",
         "routes/stt_routes.py": "STT_MAX_AUDIO_BYTES",
         "routes/calendar_routes.py": "ICS_MAX_BYTES",
     }