Remove placeholder Inbox tab from channel canvas

packages/shared/src/analytics-events.ts

@@ -810,7 +810,6 @@ export type ChannelsSurface =
   | "channel_home"
   | "channel_history"
   | "channel_artifacts"
-  | "channel_inbox"
   | "pinned"
   | "dashboards_grid"
   | "canvas"
@@ -835,7 +834,6 @@ export type ChannelActionType =
   | "view_context"
   | "view_history"
   | "view_artifacts"
-  | "view_inbox"
   | "open_artifact"
   | "file_task"
   | "unfile_task"

packages/shared/src/browser-tabs-schemas.ts

@@ -20,7 +20,7 @@ export const browserTabSchema = z.object({
   taskId: z.string().nullable().default(null),
   channelId: z.string().nullable().default(null),
   /**
-   * Channel sub-section this tab fronts (`inbox` / `artifacts` / `history` /
+   * Channel sub-section this tab fronts (`artifacts` / `history` /
    * `context`). Null = the channel home, or a non-channel tab (canvas / task /
    * blank). Pairs with `channelId`: the two together identify a channel tab.
    */

packages/shared/src/browser-tabs.test.ts

@@ -74,16 +74,16 @@ describe("openOrFocusTab", () => {
   });
 
   it("treats a channel's sections as distinct tabs but dedups the same one", () => {
-    const inbox = openOrFocusTab(snapshot(), {
+    const history = openOrFocusTab(snapshot(), {
       windowId: "w1",
       dashboardId: null,
       taskId: null,
       channelId: "c1",
-      channelSection: "inbox",
+      channelSection: "history",
       makeId,
       now,
     });
-    const artifacts = openOrFocusTab(inbox.snapshot, {
+    const artifacts = openOrFocusTab(history.snapshot, {
       windowId: "w1",
       dashboardId: null,
       taskId: null,
@@ -94,17 +94,17 @@ describe("openOrFocusTab", () => {
     });
     expect(artifacts.opened).toBe(true);
     expect(artifacts.snapshot.tabs).toHaveLength(2);
-    const inboxAgain = openOrFocusTab(artifacts.snapshot, {
+    const historyAgain = openOrFocusTab(artifacts.snapshot, {
       windowId: "w1",
       dashboardId: null,
       taskId: null,
       channelId: "c1",
-      channelSection: "inbox",
+      channelSection: "history",
       makeId,
       now,
     });
-    expect(inboxAgain.opened).toBe(false);
-    expect(inboxAgain.tabId).toBe(inbox.tabId);
+    expect(historyAgain.opened).toBe(false);
+    expect(historyAgain.tabId).toBe(history.tabId);
   });
 
   it("appends new tabs after existing ones", () => {
@@ -314,7 +314,7 @@ describe("decideTabNavigation", () => {
   });
 
   it("replaces the active tab when navigating between channel sections", () => {
-    // In-tab nav from a channel's inbox to its artifacts: same tab, new section.
+    // In-tab nav from a channel's history to its artifacts: same tab, new section.
     expect(
       decideTabNavigation({
         ...base,
@@ -324,7 +324,7 @@ describe("decideTabNavigation", () => {
           dashboardId: null,
           taskId: null,
           channelId: "c1",
-          channelSection: "inbox",
+          channelSection: "history",
         },
         routeChannelId: "c1",
         routeChannelSection: "artifacts",
@@ -345,14 +345,14 @@ describe("decideTabNavigation", () => {
       decideTabNavigation({
         ...base,
         routeChannelId: "c1",
-        routeChannelSection: "inbox",
+        routeChannelSection: "history",
       }),
     ).toEqual({
       type: "open",
       dashboardId: null,
       taskId: null,
       channelId: "c1",
-      channelSection: "inbox",
+      channelSection: "history",
       stampTabId: null,
     });
   });
@@ -367,10 +367,10 @@ describe("decideTabNavigation", () => {
           dashboardId: null,
           taskId: null,
           channelId: "c1",
-          channelSection: "inbox",
+          channelSection: "history",
         },
         routeChannelId: "c1",
-        routeChannelSection: "inbox",
+        routeChannelSection: "history",
       }),
     ).toEqual({ type: "stamp", stampTabId: "tab-a" });
   });

packages/shared/src/browser-tabs.ts

@@ -82,7 +82,7 @@ export type TabTarget = {
  * Everything that identifies a tab's contents: a canvas, a task, or a channel
  * sub-section (channel + section). Two tabs with the same identity are the same
  * page, so dedup and in-tab-nav comparisons key on all four — a channel's
- * `inbox` and `artifacts`, or two channels' inboxes, are distinct pages.
+ * `history` and `artifacts`, or two channels' artifacts, are distinct pages.
  */
 export type TabIdentity = {
   dashboardId: string | null;

packages/ui/src/features/browser-tabs/BrowserTabStrip.tsx

@@ -92,7 +92,7 @@ export function BrowserTabStrip() {
 
   const { channels } = useChannels();
 
-  // The active channel sub-section (inbox/artifacts/history/context) is the
+  // The active channel sub-section (artifacts/history/context) is the
   // route segment after the channelId. Null when on the channel home or a
   // non-section route (canvas/task), so a channel-home tab labels by name.
   const routeChannelSection = useMemo(() => {
@@ -318,7 +318,7 @@ export function BrowserTabStrip() {
             pinned,
           };
         }
-        // A channel tab: a sub-section (Inbox/Artifacts/…) or the channel home.
+        // A channel tab: a sub-section (Artifacts/Recents/…) or the channel home.
         // The section drives the label; the channel name carries the `#` hover
         // context. Home has no section, so it labels by the channel name.
         if (channelId) {
@@ -371,21 +371,17 @@ export function BrowserTabStrip() {
       });
     } else if (tab.channelId) {
       const params = { channelId: tab.channelId };
-      switch (tab.channelSection) {
-        case "inbox":
-          navigate({ to: "/website/$channelId/inbox", params, state });
-          break;
-        case "artifacts":
-          navigate({ to: "/website/$channelId/artifacts", params, state });
-          break;
-        case "history":
-          navigate({ to: "/website/$channelId/history", params, state });
-          break;
-        case "context":
-          navigate({ to: "/website/$channelId/context", params, state });
-          break;
-        default:
-          navigate({ to: "/website/$channelId", params, state });
+      // Section keys are the route segments; unknown/stale sections (e.g. from
+      // a since-removed tab type) fall back to the channel home.
+      const section = channelSectionFor(tab.channelSection);
+      if (section) {
+        navigate({
+          to: `/website/$channelId/${section.key}` as const,
+          params,
+          state,
+        });
+      } else {
+        navigate({ to: "/website/$channelId", params, state });
       }
     } else {
       navigate({ to: "/website", state });

packages/ui/src/features/canvas/channelSections.ts

@@ -5,12 +5,11 @@
  * `channelSection`; `label` is the tab + nav text.
  */
 export interface ChannelSection {
-  key: "inbox" | "artifacts" | "history" | "context";
+  key: "artifacts" | "history" | "context";
   label: string;
 }
 
 export const CHANNEL_SECTIONS: readonly ChannelSection[] = [
-  { key: "inbox", label: "Inbox" },
   { key: "artifacts", label: "Artifacts" },
   { key: "history", label: "Recents" },
   { key: "context", label: "CONTEXT.md" },

packages/ui/src/features/canvas/components/ChannelHeader.tsx

@@ -9,7 +9,7 @@ import { useNavigate, useRouterState } from "@tanstack/react-router";
 // item — it routes to the channel home (`/website/$channelId`, like the sidebar
 // channel row) and highlights `bg-fill-selected` while you're there, the same
 // pathname-driven active state the rest of the channel tab strip uses. Followed
-// by that strip (Inbox / Artifacts / Recents / CONTEXT.md), rendered into the
+// by that strip (Artifacts / Recents / CONTEXT.md), rendered into the
 // header bar by every channel view so the tabs stay in view.
 export function ChannelHeader({ channelId }: { channelId: string }) {
   const navigate = useNavigate();

packages/ui/src/features/canvas/components/ChannelTabs.tsx

@@ -1,13 +1,12 @@
 import { Button, cn } from "@posthog/quill";
+import { CHANNEL_SECTIONS } from "@posthog/ui/features/canvas/channelSections";
 import { ChannelPinnedMenu } from "@posthog/ui/features/canvas/components/ChannelPinnedMenu";
 import { Link, useRouterState } from "@tanstack/react-router";
 
-const TABS = [
-  { label: "Inbox", to: "/website/$channelId/inbox" },
-  { label: "Artifacts", to: "/website/$channelId/artifacts" },
-  { label: "Recents", to: "/website/$channelId/history" },
-  { label: "CONTEXT.md", to: "/website/$channelId/context" },
-] as const;
+const TABS = CHANNEL_SECTIONS.map((s) => ({
+  label: s.label,
+  to: `/website/$channelId/${s.key}` as const,
+}));
 
 // Home / History / Artifacts tab switcher shown in the channel header bar, with
 // a Pinned quick-access menu alongside. Pathname-driven active state (the