collapse provider inventory migrations 11+12 into single migration 11

Signed-off-by: Bradley Axen <baxen@squareup.com>

Author: baxen

crates/goose/src/session/session_manager.rs

@@ -19,7 +19,7 @@ use std::sync::{Arc, LazyLock};
 use tracing::{info, warn};
 use utoipa::ToSchema;
 
-pub const CURRENT_SCHEMA_VERSION: i32 = 12;
+pub const CURRENT_SCHEMA_VERSION: i32 = 11;
 pub const SESSIONS_FOLDER: &str = "sessions";
 pub const DB_NAME: &str = "sessions.db";
 
@@ -1065,16 +1065,6 @@ impl SessionStorage {
             11 => {
                 crate::providers::inventory::create_tables_in_tx(tx).await?;
             }
-            12 => {
-                // Add recommended column to provider_inventory_models.
-                // Ignore error if column already exists (fresh installs
-                // created the table with the column in migration 11).
-                let _ = sqlx::query(
-                    "ALTER TABLE provider_inventory_models ADD COLUMN recommended BOOLEAN",
-                )
-                .execute(&mut **tx)
-                .await;
-            }
             _ => {
                 anyhow::bail!("Unknown migration version: {}", version);
             }

ui/goose2/scripts/reset-inventory.sh

@@ -0,0 +1,33 @@
+#!/usr/bin/env bash
+# Reset the provider inventory tables to empty, as if migration 12 just ran.
+# This lets you test the first-use experience (cold inventory).
+#
+# Usage: ./scripts/reset-inventory.sh
+
+set -euo pipefail
+
+DB="${GOOSE_DB:-$HOME/.local/share/goose/sessions/sessions.db}"
+
+if [ ! -f "$DB" ]; then
+  echo "Database not found at $DB"
+  echo "Set GOOSE_DB to override the path."
+  exit 1
+fi
+
+echo "Database: $DB"
+echo ""
+echo "Before:"
+echo "  provider_inventory_entries: $(sqlite3 "$DB" 'SELECT COUNT(*) FROM provider_inventory_entries;')"
+echo "  provider_inventory_models:  $(sqlite3 "$DB" 'SELECT COUNT(*) FROM provider_inventory_models;')"
+
+# ON DELETE CASCADE on provider_inventory_models means deleting entries clears both tables.
+# Delete models first since CASCADE isn't reliable in all sqlite3 builds,
+# then delete entries.
+sqlite3 "$DB" "DELETE FROM provider_inventory_models; DELETE FROM provider_inventory_entries;"
+
+echo ""
+echo "After:"
+echo "  provider_inventory_entries: $(sqlite3 "$DB" 'SELECT COUNT(*) FROM provider_inventory_entries;')"
+echo "  provider_inventory_models:  $(sqlite3 "$DB" 'SELECT COUNT(*) FROM provider_inventory_models;')"
+echo ""
+echo "Inventory tables are empty. Restart goose to test first-use flow."