From f7d651388f00295102460a62a6eb23aaf823845b Mon Sep 17 00:00:00 2001 From: Dotta Date: Mon, 20 Apr 2026 07:11:03 -0500 Subject: [PATCH] PAP-1627: route sidebar memories link through active company prefix The sidebar's Memories link was resolving to `/memories` instead of `/:companyPrefix/memories` because `memories` was not in `BOARD_ROUTE_ROOTS`. Without it, `applyCompanyPrefix` treated the path as if `memories` were already a company prefix and skipped rewriting. - Add `memories` to `BOARD_ROUTE_ROOTS` so `applyCompanyPrefix("/memories", "PAP")` returns `/PAP/memories`. - Add a regression test covering `/memories` through `isBoardPathWithoutPrefix`, `extractCompanyPrefixFromPath`, and `applyCompanyPrefix`. Co-Authored-By: Paperclip --- ui/src/lib/company-routes.test.ts | 6 ++++++ ui/src/lib/company-routes.ts | 1 + 2 files changed, 7 insertions(+) diff --git a/ui/src/lib/company-routes.test.ts b/ui/src/lib/company-routes.test.ts index d6dc2668378..77fd6239948 100644 --- a/ui/src/lib/company-routes.test.ts +++ b/ui/src/lib/company-routes.test.ts @@ -20,4 +20,10 @@ describe("company routes", () => { "/execution-workspaces/workspace-123", ); }); + + it("treats memories as a company board route", () => { + expect(isBoardPathWithoutPrefix("/memories")).toBe(true); + expect(extractCompanyPrefixFromPath("/memories")).toBeNull(); + expect(applyCompanyPrefix("/memories", "PAP")).toBe("/PAP/memories"); + }); }); diff --git a/ui/src/lib/company-routes.ts b/ui/src/lib/company-routes.ts index 04659a61a3a..83fa20b61b1 100644 --- a/ui/src/lib/company-routes.ts +++ b/ui/src/lib/company-routes.ts @@ -9,6 +9,7 @@ const BOARD_ROUTE_ROOTS = new Set([ "workspaces", "execution-workspaces", "issues", + "memories", "routines", "goals", "approvals",