fix(integrations): relocate type-scoped routes to connect/test/sync/[type] (Next slug conflict with [id])

Author: hoaxnerd

apps/web/src/app/(dashboard)/settings/__tests__/stripe-connect-card.test.tsx

@@ -50,7 +50,7 @@ describe("StripeConnectCard", () => {
     const call = apiFetch.mock.calls[0]!;
     const url = call[0] as string;
     const init = call[1] as RequestInit;
-    expect(url).toBe("/api/integrations/stripe/connect");
+    expect(url).toBe("/api/integrations/connect/stripe");
     expect(init.method).toBe("POST");
     expect(JSON.parse(init.body as string)).toEqual({ apiKey: "rk_test_abc" });
     await waitFor(() => expect(onConnected).toHaveBeenCalled());

apps/web/src/app/(dashboard)/settings/stripe-connect-card.tsx

@@ -8,7 +8,7 @@ import { useLocale } from "@/components/locale/locale-context";
 import { integrationRegistry, registerConnectors } from "@/lib/integrations/registry";
 
 // ── Stripe connect card (C1.6) ──────────────────────────────────────────────
-// Paste a restricted key → POST /api/integrations/stripe/connect. The scope
+// Paste a restricted key → POST /api/integrations/connect/stripe. The scope
 // checklist text comes straight from the connector's credentialSpec help (no
 // hardcoded copy — single source of truth). The key is a PASSWORD input and is
 // NEVER rendered back. When connected we show status + last-sync + Disconnect.
@@ -52,7 +52,7 @@ export function StripeConnectCard({ onConnected, connected, onDisconnect }: Stri
     setSubmitting(true);
     setError(null);
     try {
-      const res = await apiFetch("/api/integrations/stripe/connect", {
+      const res = await apiFetch("/api/integrations/connect/stripe", {
         method: "POST",
         headers: { "Content-Type": "application/json" },
         body: JSON.stringify({ apiKey }),

…[type]/connect/__tests__/connect.test.ts …connect/[type]/__tests__/connect.test.tsapps/web/src/app/api/integrations/[type]/connect/__tests__/connect.test.ts renamed to apps/web/src/app/api/integrations/connect/[type]/__tests__/connect.test.ts apps/web/src/app/api/integrations/[type]/connect/__tests__/connect.test.ts renamed to apps/web/src/app/api/integrations/connect/[type]/__tests__/connect.test.ts

@@ -76,7 +76,7 @@ vi.mock("@/lib/integrations/registry", () => ({
 
 import { POST } from "../route";
 
-describe("POST /api/integrations/[type]/connect", () => {
+describe("POST /api/integrations/connect/[type]", () => {
   beforeEach(() => {
     save.mockClear();
     validate.mockClear();
@@ -88,7 +88,7 @@ describe("POST /api/integrations/[type]/connect", () => {
   });
 
   it("validates the key, stores encrypted creds, upserts the row, returns 200", async () => {
-    const req = new Request("http://x/api/integrations/stripe/connect", {
+    const req = new Request("http://x/api/integrations/connect/stripe", {
       method: "POST",
       body: JSON.stringify({ apiKey: "rk_test_ok" }),
     });
@@ -103,7 +103,7 @@ describe("POST /api/integrations/[type]/connect", () => {
   });
 
   it("returns 404 for an unknown integration type", async () => {
-    const req = new Request("http://x/api/integrations/nope/connect", {
+    const req = new Request("http://x/api/integrations/connect/nope", {
       method: "POST",
       body: JSON.stringify({ apiKey: "rk_test_ok" }),
     });
@@ -114,7 +114,7 @@ describe("POST /api/integrations/[type]/connect", () => {
 
   it("returns 400 with the friendly error when validate fails", async () => {
     validate.mockResolvedValueOnce({ ok: false, error: "Invalid API key" });
-    const req = new Request("http://x/api/integrations/stripe/connect", {
+    const req = new Request("http://x/api/integrations/connect/stripe", {
       method: "POST",
       body: JSON.stringify({ apiKey: "rk_test_bad" }),
     });
@@ -125,7 +125,7 @@ describe("POST /api/integrations/[type]/connect", () => {
 
   // ── C1.5 folded fixes ──────────────────────────────────────────────────────
   it("kicks off a backfill on connect (mode:backfill) and does NOT set lastSyncAt (fix A)", async () => {
-    const req = new Request("http://x/api/integrations/stripe/connect", {
+    const req = new Request("http://x/api/integrations/connect/stripe", {
       method: "POST",
       body: JSON.stringify({ apiKey: "rk_test_ok" }),
     });
@@ -143,7 +143,7 @@ describe("POST /api/integrations/[type]/connect", () => {
       id: "i1",
       metadata: { livemode: false, sync: { cursor: 1700000000, lastRecordCount: 5 } },
     };
-    const req = new Request("http://x/api/integrations/stripe/connect", {
+    const req = new Request("http://x/api/integrations/connect/stripe", {
       method: "POST",
       body: JSON.stringify({ apiKey: "rk_test_ok" }),
     });

…api/integrations/[type]/connect/route.ts …api/integrations/connect/[type]/route.tsapps/web/src/app/api/integrations/[type]/connect/route.ts renamed to apps/web/src/app/api/integrations/connect/[type]/route.ts apps/web/src/app/api/integrations/[type]/connect/route.ts renamed to apps/web/src/app/api/integrations/connect/[type]/route.ts

@@ -17,7 +17,7 @@ import { connectSchema } from "./schemas";
 
 const log = logger("integrations/connect");
 
-// ── POST /api/integrations/[type]/connect ───────────────────────────────────
+// ── POST /api/integrations/connect/[type] ───────────────────────────────────
 // Validate a pasted API key via the connector's credentialSpec, encrypt+store it,
 // and upsert the integrations row to status:"active". NEVER logs the key.
 export const POST = withErrorHandler(

…i/integrations/[type]/connect/schemas.ts …i/integrations/connect/[type]/schemas.tsapps/web/src/app/api/integrations/[type]/connect/schemas.ts renamed to apps/web/src/app/api/integrations/connect/[type]/schemas.ts apps/web/src/app/api/integrations/[type]/connect/schemas.ts renamed to apps/web/src/app/api/integrations/connect/[type]/schemas.ts

@@ -11,7 +11,7 @@ import { requireCapability, getCapabilities } from "@/lib/capabilities";
 import { integrationRegistry, registerConnectors } from "@/lib/integrations/registry";
 import { runIntegrationSync } from "@/lib/integrations/sync";
 
-// ── POST /api/integrations/[type]/sync ───────────────────────────────────────
+// ── POST /api/integrations/sync/[type] ───────────────────────────────────────
 // Manually trigger an INCREMENTAL inbound sync (uses the stored cursor). Gated
 // exactly like /connect: company access → admin → capability("integrations")
 // → (cloud-only) plan feature. NEVER logs credentials.

…pp/api/integrations/[type]/sync/route.ts …pp/api/integrations/sync/[type]/route.tsapps/web/src/app/api/integrations/[type]/sync/route.ts renamed to apps/web/src/app/api/integrations/sync/[type]/route.ts apps/web/src/app/api/integrations/[type]/sync/route.ts renamed to apps/web/src/app/api/integrations/sync/[type]/route.ts

@@ -10,7 +10,7 @@ import {
 import { requireCapability, getCapabilities } from "@/lib/capabilities";
 import { integrationRegistry, registerConnectors } from "@/lib/integrations/registry";
 
-// ── POST /api/integrations/[type]/test ──────────────────────────────────────
+// ── POST /api/integrations/test/[type] ──────────────────────────────────────
 // Re-validate the STORED (decrypted) key via the connector's credentialSpec
 // WITHOUT writing anything. NEVER logs the key.
 export const POST = withErrorHandler(