From ebd1940003fa4ace566dc1609f3dfb0427890259 Mon Sep 17 00:00:00 2001 From: scotttong Date: Tue, 2 Jun 2026 19:01:17 -0700 Subject: [PATCH] PAP-56: register conference-room and artifacts as board routes Both routes rendered "Company not found" because their root segments were missing from BOARD_ROUTE_ROOTS, so applyCompanyPrefix never prepended the company prefix. Add both roots and a regression test. Co-Authored-By: Claude Opus 4.8 --- ui/src/lib/company-routes.test.ts | 11 +++++++++++ ui/src/lib/company-routes.ts | 2 ++ 2 files changed, 13 insertions(+) diff --git a/ui/src/lib/company-routes.test.ts b/ui/src/lib/company-routes.test.ts index 4d22ec5919b..d1ee164246c 100644 --- a/ui/src/lib/company-routes.test.ts +++ b/ui/src/lib/company-routes.test.ts @@ -64,4 +64,15 @@ describe("company routes", () => { "/teams-catalog/core-exec-team", ); }); + + it("treats /conference-room and /artifacts as board routes that need a company prefix", () => { + expect(isBoardPathWithoutPrefix("/conference-room")).toBe(true); + expect(isBoardPathWithoutPrefix("/artifacts")).toBe(true); + expect(extractCompanyPrefixFromPath("/conference-room")).toBeNull(); + expect(extractCompanyPrefixFromPath("/artifacts")).toBeNull(); + expect(applyCompanyPrefix("/conference-room", "PAP")).toBe("/PAP/conference-room"); + expect(applyCompanyPrefix("/artifacts", "PAP")).toBe("/PAP/artifacts"); + expect(toCompanyRelativePath("/PAP/conference-room")).toBe("/conference-room"); + expect(toCompanyRelativePath("/PAP/artifacts")).toBe("/artifacts"); + }); }); diff --git a/ui/src/lib/company-routes.ts b/ui/src/lib/company-routes.ts index 67197a7655e..77568f1e43d 100644 --- a/ui/src/lib/company-routes.ts +++ b/ui/src/lib/company-routes.ts @@ -20,6 +20,8 @@ const BOARD_ROUTE_ROOTS = new Set([ "u", "design-guide", "search", + "conference-room", + "artifacts", ]); const GLOBAL_ROUTE_ROOTS = new Set(["auth", "invite", "board-claim", "cli-auth", "docs", "instance"]);