Fix: Rollback tool intake and return error when category insert fails

[Copilot]

Category insert failures in app/api/submit-tool/route.ts were silently swallowed — the API returned success while tool_intake_categories was left empty, causing tools to appear with no categories in the dashboard.

Changes

• Category ID validation — Before inserting, validates all submitted categoryIds against the categories table. Invalid IDs (stale, tampered, or nonexistent) trigger a 400 rollback response.
• Rollback on failure — Any failure during category lookup or insert now deletes the just-created tool_intakes row and returns an error (400/500) prompting the user to resubmit rather than silently succeeding.
• Shared rollback helper — Extracted a local rollbackIntake() helper to avoid duplicating the delete logic across three failure paths; also logs a warning if the rollback itself fails so orphaned records are surfaced in logs.

// Before: failure was ignored, intake persisted with no categories
if (categoryError) {
  console.error("Error inserting tool intake categories:", categoryError);
  // Note: We don't fail the request here...
}

// After: rollback and surface error to client
if (categoryError) {
  await rollbackIntake();
  console.error("Error inserting tool intake categories:", categoryError);
  return NextResponse.json(
    { error: "Failed to save tool categories. Please resubmit.", step: "category_insert" },
    { status: 500 },
  );
}

• Fixes [Bug]: Categories missing after successful tool submit #94

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
web	Ready	Preview	Aug 16, 2026 2:35am