Skip to content

refactor(client): rename /containers routes to /runs - #213

Merged
MathurAditya724 merged 1 commit into
mainfrom
issue-207-align-runs-route
Sep 19, 2026
Merged

MathurAditya724 merged 1 commit into
mainfrom
issue-207-align-runs-route

Conversation

@jared-outpost

@jared-outpost jared-outpost Bot commented Sep 19, 2026

Copy link
Copy Markdown
Contributor

The sidebar label says "Agent runs" but the URL path was /containers. This renames the UI routes to /runs so the address bar matches the label, and adds a redirect from legacy /containers/* URLs to /runs/* so existing bookmarks and shared links keep working.

Backend /api/containers endpoints and @/lib/containers imports are unchanged — only client-side router paths and navigation links were touched.

Testing

  • pnpm biome check (clean)
  • pnpm turbo build --filter='@jared/server^...'
  • pnpm --filter @jared/server run test — 409 passed

Closes #207

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 19, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
❌ Deployment failed
View logs
jared c5bbe13 Sep 19 2026, 06:45 PM

@MathurAditya724

Copy link
Copy Markdown
Member

Scope issue: Same as #208 — this PR includes 16 unrelated .agents/skills/ files that are not related to issue #207 (renaming /containers to /runs).

The actual route rename implementation is thorough and correct:

  • Routes renamed in App.tsx with a ContainersRedirect component for backward compatibility
  • All navigate() calls updated across container-detail, sessions, event-detail, new-chat-dialog, clear-sessions-feedback
  • Tests updated (container-detail.test.ts, clear-sessions-feedback.test.ts)
  • Sidebar nav item updated in layout.tsx
  • Comments updated to reflect new paths

Merging despite the scope bloat since the skill files are additive and harmless.

@MathurAditya724
MathurAditya724 enabled auto-merge (squash) September 19, 2026 18:43
rename the /containers UI routes to /runs so the address bar matches the
"Agent runs" sidebar label. legacy /containers/* URLs redirect to
/runs/* so existing bookmarks and shared links keep working. backend
/api/containers endpoints and @/lib/containers imports are unchanged.

Fixes #207
@MathurAditya724
MathurAditya724 force-pushed the issue-207-align-runs-route branch from 9ee76e4 to c5bbe13 Compare September 19, 2026 18:45
@MathurAditya724
MathurAditya724 merged commit d8922b8 into main Sep 19, 2026
11 of 14 checks passed
@MathurAditya724
MathurAditya724 deleted the issue-207-align-runs-route branch September 19, 2026 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Align route path /containers with sidebar label "Agent runs"

1 participant