` is invalid HTML and (worse) means a click anywhere on
@@ -418,15 +417,10 @@ export function Shell({ children }: { children: ComponentChildren }) {
{/* Keyed by route so it remounts (and replays its entrance
- animation) on every navigation — see the file-top comment.
- `shell-body-wide` is additive (see `WIDE_BODY_ROUTES` above),
- not a swap — `.shell-body`'s padding/animation rules still
- apply, only the max-width cap changes. */}
+ animation) on every navigation — see the file-top comment. */}
diff --git a/frontend/packages/swarm-ui/src/shell/routes.ts b/frontend/packages/swarm-ui/src/shell/routes.ts
index 45a7a708..0c3d793a 100644
--- a/frontend/packages/swarm-ui/src/shell/routes.ts
+++ b/frontend/packages/swarm-ui/src/shell/routes.ts
@@ -1,21 +1,12 @@
-// Which layout a route gets inside the shell: the readable-width cap, the
-// wide body, or full-bleed. Kept apart from `Shell.tsx` so it is plain
-// functions of a path, with no DOM or CSS behind it, and can be tested.
-
-// Routes that opt out of `.shell-body`'s 60em readable-line-length cap
-// — mara screenshotted the issue report's 8-column table getting
-// clipped; that width is deliberate for the rest of the UI's cards/
-// forms, but too narrow for a wide table. `.shell-body-wide` (Shell.css) is a
-// modifier alongside the base class, not a replacement, so every other
-// route's layout is untouched. `/agents` joined for the same reason —
-// its own table view (plus the list+detail split) hits the same cap.
-const WIDE_BODY_ROUTES = new Set(["/issues", "/agents", "/review"]);
+// Which routes are full-bleed inside the shell; every other route gets the
+// same centred body. Kept apart from `Shell.tsx` so it is a plain function of
+// a path, with no DOM or CSS behind it, and can be tested.
// `/agents/:name/terminal` (and a subagent's, under
// `/agents/:name/subagents/:subagent/`) is full-bleed: the terminal fills
// the whole viewport below the header bar, edge to edge (`shell-full` /
-// `shell-body-full` in Shell.css). It can't join the `Set` above — it's a
-// parameterized path, one literal string can't match every agent name.
+// `shell-body-full` in Shell.css). It's a parameterized path, so one literal
+// string can't match every agent name.
const FULL_BLEED_ROUTE_PATTERN =
/^\/agents\/[^/]+(\/subagents\/[^/]+)?\/terminal$/;
@@ -24,7 +15,3 @@ const FULL_BLEED_ROUTE_PATTERN =
export function isFullBleedRoute(location: string): boolean {
return location === "/" || FULL_BLEED_ROUTE_PATTERN.test(location);
}
-
-export function isWideBodyRoute(location: string): boolean {
- return WIDE_BODY_ROUTES.has(location);
-}
diff --git a/frontend/packages/swarm-ui/test/routes.test.mjs b/frontend/packages/swarm-ui/test/routes.test.mjs
index f0ccf453..85178dbd 100644
--- a/frontend/packages/swarm-ui/test/routes.test.mjs
+++ b/frontend/packages/swarm-ui/test/routes.test.mjs
@@ -1,8 +1,7 @@
-// The shell's per-route layout: which paths are full-bleed, which get the
-// wide body. Run by `npm test`.
+// The shell's per-route layout: which paths are full-bleed. Run by `npm test`.
import assert from "node:assert/strict";
import { test } from "node:test";
-import { isFullBleedRoute, isWideBodyRoute } from "../src/shell/routes.ts";
+import { isFullBleedRoute } from "../src/shell/routes.ts";
test("the start page is full-bleed", () => {
assert.equal(isFullBleedRoute("/"), true);
@@ -10,7 +9,6 @@ test("the start page is full-bleed", () => {
test("the hive directory moved off / and is not full-bleed", () => {
assert.equal(isFullBleedRoute("/hives"), false);
- assert.equal(isWideBodyRoute("/hives"), false);
});
test("an agent's terminal and a subagent's are full-bleed", () => {
@@ -31,12 +29,3 @@ test("a terminal path with extra segments is not full-bleed", () => {
assert.equal(isFullBleedRoute("/agents/iris/terminal/extra"), false);
assert.equal(isFullBleedRoute("/agents//terminal"), false);
});
-
-test("the wide-body routes are exactly the table pages", () => {
- for (const path of ["/issues", "/agents", "/review"]) {
- assert.equal(isWideBodyRoute(path), true, path);
- }
- for (const path of ["/", "/jobs", "/schedules", "/components"]) {
- assert.equal(isWideBodyRoute(path), false, path);
- }
-});