chore: remove Overview in Deployments sidebar by MichaelUnkey · Pull Request #5265 · unkeyed/unkey
176-182: Remove the dead nav block instead of keeping it commented out.
Keeping the old Overview item inline makes the live navigation config harder to scan, and Git history already preserves the implementation if it needs to come back later. If this is meant to be revisited, a short TODO with an issue reference is easier to maintain than a commented object literal.
♻️ Proposed cleanup
const childItems: NavItem[] = [ - // Removed until we hsve something to place in Overview - // { - // icon: Grid, - // href: basePath, - // label: "Overview", - // active: segments.at(2) === projectId && !segments.at(3), - // }, { icon: Cube, href: `${basePath}/deployments`, label: "Deployments", active: segments.includes("deployments"),
Based on learnings: Applies to **/*.{ts,tsx} : Make minimal, surgical changes to code
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@web/apps/dashboard/components/navigation/sidebar/navigation-configs.tsx`
around lines 176 - 182, Remove the dead commented-out Overview nav item from
navigation-configs.tsx (the commented object containing icon: Grid, href:
basePath, label: "Overview", active: segments.at(2) === projectId &&
!segments.at(3)) — either delete it entirely or replace the comment with a
single-line TODO referencing the tracking issue/PR number; do not leave the full
object literal commented inline so the live nav config (and functions that
reference segments.at / projectId) remains easy to scan and maintain.