[Graphite MQ] Draft PR GROUP:spec_f0bfa4 (PRs 4248)

[graphite-app]

This draft PR was created by the Graphite merge queue.
Trunk will be fast forwarded to the HEAD of this PR when CI passes, and the original PRs will be closed.

The following PRs are included in this draft PR:

• #4248

[railway-app]

This PR was not deployed automatically as @graphite-app[bot] does not have access to the Railway project.

In order to get automatic PR deploys, please add @graphite-app[bot] to your workspace on Railway.

[claude]

PR Review: fix(dashboard): hide unsupported tabs (#4248)

Note: This PR has already been merged to main via the Graphite merge queue (commit 6f381dc7). This review notes issues for follow-up.

What the PR Does

Removes four tab sections (Queue, Traces, Workflow, Database) from the actor details UI in frontend/src/components/actors/actors-actor-details.tsx, and cleans up the previously commented-out Logs and Metrics tab blocks.

Issues

Unused Imports Left Behind

The PR removes the tab content for Queue, Traces, Workflow, and Database, but does not remove their corresponding imports. These are now dead code in main:

import { ActorDatabaseTab } from "./actor-db-tab";    // unused after Database tab removal
import { ActorQueueTab } from "./actor-queue-tab";     // unused after Queue tab removal
import { ActorTracesTab } from "./actor-traces-tab";   // unused after Traces tab removal
import { ActorWorkflowTab } from "./workflow/actor-workflow-tab"; // unused after Workflow tab removal

The PR moved ActorWorkflowTab's import to a different position (described as "maintaining proper import organization") but the usage was removed — the import should have been deleted entirely. The other three were not touched at all. These unused imports likely contributed to the Railway build failures (frontend-inspector, website, ladle) visible in the CI output.

Recommended follow-up: Remove all four unused imports.

Orphaned TabsContent for Logs

After the merge, main contains a TabsContent value="logs" with no corresponding TabsTrigger value="logs". This content is unreachable from the UI unless navigated to directly via URL/state. This appears to be a pre-existing issue, but worth tracking.

What Was Done Well

• Removing the commented-out Logs and Metrics tab blocks is good cleanup — dead commented-out code was correctly deleted rather than left to accumulate.
• The scope of the change is narrow and focused.

PR Description Nit

The PR marks this as a "Breaking change" but removing unfinished/unsupported tabs is more accurately a bug fix or UI cleanup. The PR title correctly uses fix(... prefix.

Follow-up Recommended

// Remove these four imports from actors-actor-details.tsx:
import { ActorDatabaseTab } from "./actor-db-tab";
import { ActorQueueTab } from "./actor-queue-tab";
import { ActorTracesTab } from "./actor-traces-tab";
import { ActorWorkflowTab } from "./workflow/actor-workflow-tab";