Drop redirects for old skin and drink addresses

Only /drinks stays, because the in-game drink code message names it.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Author: ryanbarlow97

frontend/app/components/profile/ProfilePanel.test.tsx

@@ -1121,7 +1121,7 @@ it("falls back to sign-in when a code's session has run out", async () => {
   expect(getSkinsSession()).toBeNull();
 });
 
-it("opens the code form for old skin and drink links", async () => {
+it("opens the code form on the Skins and Drinks tabs", async () => {
   vi.mocked(getAccount).mockResolvedValue(null);
   visit("drinks");
   render(<ProfilePanel />);
@@ -1142,7 +1142,7 @@ it("lets a Discord account without a Minecraft link open a wardrobe with a code"
   visit("skins");
   render(<ProfilePanel />);
   const toggle = await screen.findByRole("button", { name: "Enter code" });
-  // Old /skins links land here with the form open.
+  // The Skins tab opens with the form showing.
   expect(toggle.getAttribute("aria-expanded")).toBe("true");
   fireEvent.change(screen.getByLabelText("Log in with code"), { target: { value: "SKIN-CODE" } });
   fireEvent.click(screen.getByRole("button", { name: "Enter" }));

frontend/app/components/profile/ProfilePanel.tsx

@@ -206,7 +206,7 @@ export default function ProfilePanel({
   const [loggingOut, setLoggingOut] = useState(false);
   // Skin and drink codes' sessions, which open just their wardrobes when there's no Profile session.
   const [codes, setCodes] = useState<CodeSessions>(NO_CODE_SESSIONS);
-  // The old /skins and /drinks addresses land on those tabs, from in-game messages about codes.
+  // The Skins and Drinks tabs open with the code form showing (the in-game drink code message links /drinks).
   const [showCode, setShowCode] = useState(tab === "skins" || tab === "drinks");
   const codeId = useId();
   const panelId = useId();

frontend/lib/profile/paths.ts

@@ -2,8 +2,7 @@ import type { ProfileTab } from "./tabs";
 
 /**
  * Profile's addresses. A skin or drink opens inside its tab: `?tab=skins&skin=new` is the editor and
- * `&skin=<id>` one skin. The old /skins, /drinks and /profile/{skins,drinks}/* addresses redirect here
- * (next.config.ts).
+ * `&skin=<id>` one skin.
  */
 const ITEM_PARAMS: Partial<Record<ProfileTab, "skin" | "drink">> = { skins: "skin", drinks: "drink" };
 

frontend/next.config.ts

@@ -26,19 +26,8 @@ const nextConfig: NextConfig = {
       { source: "/account", destination: "/profile?tab=accounts", permanent: false },
       // The character list lives on Profile's Characters tab.
       { source: "/character", destination: "/profile", permanent: false },
-      // Skins and drinks are made and followed inside Profile's tabs. Ids are slugs, so /skins/skin.png
-      // and /drinks/assets/* (public files) don't match.
-      { source: "/skins", destination: "/profile?tab=skins", permanent: false },
-      { source: "/skins/:id([a-z][a-z0-9_]{1,47})", destination: "/profile?tab=skins&skin=:id", permanent: false },
+      // The in-game /token create drink message sends players to /drinks.
       { source: "/drinks", destination: "/profile?tab=drinks", permanent: false },
-      { source: "/drinks/:id([a-z][a-z0-9_]{1,47})", destination: "/profile?tab=drinks&drink=:id", permanent: false },
-      // Their old pages under Profile; "new" is the editor. The drinks dev preview keeps its page.
-      { source: "/profile/skins/:id", destination: "/profile?tab=skins&skin=:id", permanent: false },
-      {
-        source: "/profile/drinks/:id((?!dev-preview$)[^/]+)",
-        destination: "/profile?tab=drinks&drink=:id",
-        permanent: false,
-      },
       // Staff tools that moved into the staff panel.
       { source: "/precedent", destination: "/admin/precedent", permanent: false },
       { source: "/inspect", destination: "/admin/codes", permanent: false },