McRitchie Studio
Say Hi 👋
Agents Builders
← All Tasks
Sizing

Clean up Studio admin navigation

Archived
task-af0b5d3511ee

Created

Jun 21, 05:39

Started

Jun 21, 05:40

Completed

Jun 22, 13:55

DevOps handoff

Type

Feature

Shape

ui-only

Worktree Slug

studio-admin-nav-cleanup

Repositories

mcritchie-studio

Release Train

—

Branch

feat/studio-admin-nav-cleanup

Pull Request

https://github.com/amcritchie/mcritchie-studio/pull/76

Local URL

http://localhost:3002

QA URL

—

Production URL

—

admin-navigation route-removal

Acceptance Criteria

  • /devops and nested DevOps pages are removed from McRitchie Studio routes, links, views, and API surface.
  • Ops Admin appears at the top of the sidebar.
  • The dashboard includes a link to /builders/history.

Expected Test Plan

  • [component] Rails tests verify removed DevOps routes/links and the dashboard/sidebar navigation changes.

Checks Run

  • [component] bin/rails test test/controllers/devops_routes_test.rb test/helpers/link_tree_helper_test.rb test/controllers/admin/dashboard_controller_test.rb test/controllers/tasks_controller_test.rb (39 tests, 210 assertions)
  • [component] bin/rails test test/commands/devops_cycle_command_test.rb (18 tests, 247 assertions)
  • [component] bin/install-agent-docs check
  • [e2e] E2E_PORT=3097 npx playwright test e2e/qa_readonly.spec.js (2 passed)

Stage Timeline

Who handled each stage, the time it took (measured), and the model / tokens / cost reported (best-effort) — plus who's on it right now. — means the agent didn't report that metric.

No stage changes recorded yet.

Conversation

QA review feedback, agent handoffs, and follow-up notes for this task.

QA Feedback 6 days ago

QA: code/tests are good (route-removal asserted at 3 tiers, no dangling code refs) but BLOCK on doc coherence + PR not review-ready. Rework: (1) Rebase onto origin/main FIRST — 6 behind; the one overlap is docs/agents/system/devops-cycle-design.md (edited +42 on main), so clean the doc against main current version. (2) This deletes the /devops/cycle viewer but leaves live links to it — fix docs/agents/claude.md:52, docs/agents/index.md:93, devops-cycle-design.md:16-18/:41/:362, then regenerate roots CLAUDE.md:52 + AGENTS.md:93; strip or repoint to devops-cycle-design.md. (3) Mark PR #76 ready-for-review (currently draft). Nit: add an assert_select for the Builder History dashboard link.

Comment 5 days ago

Rework addressed: rebased onto origin/main, removed stale /devops/cycle doc references, regenerated root AGENTS.md/CLAUDE.md locally, tightened Builder History dashboard assertion, force-pushed PR #76, and marked it ready for review.

Comment 5 days ago

APPROVED (PR #76, QA re-review). All 3 acceptance criteria met: (1) /devops fully removed from routes/links/views/API — no dangling refs anywhere in app/lib/config/test/e2e/bin; (2) admin block leads the sidebar — Mr. McRitchie confirmed criterion 2 satisfied as-is (Site→Ops→On-chain→Data within the admin block); (3) dashboard links /builders/history. Risk areas clear: docs coherent (install-agent-docs check OK, zero /devops/cycle refs in docs + both generated root files), WorktreeRegistry removal safe (snapshot --write + qa-intake use the .agents JSON independently). Route removal asserted at 3 tiers + e2e 404. Full suite 906/0/0/4 skips. Prior rework reasons (rebase/doc-coherence/draft) all addressed; branch clean + mergeable. Ready for the conductor to assemble.

Sealed-bid sizing

Edit →

Alex (PM)

—

Avi (PO)

—

Dev

—

Actual

—