fix(databases): surface the id change when load recreates a managed database (#173)

Loading an undeclared table into a managed database triggers an
auto-declare path that delete+recreates the database (managed DBs have no
add-table endpoint), which mints a NEW database id. This was silent unless
existing tables had synced data — so the id printed by `databases create`
would go stale after the next `load`, and `databases delete <create-id>`
failed with "no database with id".

Emit a clear note on the recreate (old id -> new id) and point users to
reference managed databases by catalog (stable) rather than the
create-time id. Document the same in the managed-database workflow.

`databases create` itself is correct — the printed id matches `databases
list` immediately; only `load`'s recreate changed it.

Co-authored-by: Eddie A Tejeda <669988+eddietejeda@users.noreply.github.com>

Author: eddietejeda

skills/hotdata/references/WORKFLOWS.md

@@ -106,6 +106,8 @@ End-to-end checklists. Use the linked sections for command detail and guardrails
    hotdata databases load --catalog sales --table customers --url https://example.com/customers.parquet
    ```
 
+   > Auto-declaring a *new* table recreates the database (no add-table API), which **changes its `id`**. Always reference a managed database by its **catalog** (stable), not the `id` returned by `databases create` — that id goes stale after the next `load` of an undeclared table. Declare tables up front (`databases create --table orders --table customers`) to avoid the recreate.
+
 3. Confirm and query:
 
    ```bash

src/databases.rs

@@ -1013,6 +1013,29 @@ pub fn tables_load(
             }
         };
         let _ = crate::config::save_current_database("default", workspace_id, &new_db.id);
+        // Managed databases have no add-table endpoint, so declaring a new table
+        // is a delete + recreate — which mints a NEW database id. Surface that
+        // explicitly: the id printed by `databases create` is now stale, and
+        // id-based automation (e.g. `databases delete <create-time-id>`) would
+        // otherwise fail with "no database with id". Reference by catalog instead.
+        {
+            use crossterm::style::Stylize;
+            let catalog = db
+                .default_catalog
+                .as_deref()
+                .or(db.name.as_deref())
+                .unwrap_or(&db.id);
+            eprintln!(
+                "{}",
+                format!(
+                    "note: table '{table}' was not declared — recreated database '{catalog}' to add it \
+                     (id {} → {}). Managed databases are recreated when a new table is loaded; \
+                     reference them by catalog ('{catalog}'), not the create-time id.",
+                    db.id, new_db.id
+                )
+                .yellow()
+            );
+        }
         let new_path = managed_table_load_path(&new_db.default_connection_id, schema, table);
         let spinner = crate::util::spinner("Loading table...");
         let result = api.post_raw(&new_path, &body).unwrap_or_else(|e| {