import type { DrizzleDb } from "../db-connection.js";
import { getSqliteFrom } from "../db-connection.js";

/**
 * One-shot migration: rename legacy memory graph node type values.
 *
 * The PR that introduced the memory graph used "style" and "relationship"
 * as type values.  These were renamed to "behavioral" and "semantic" in
 * code, but existing rows in the database still carry the old values.
 */
export function migrateRenameMemoryGraphTypeValues(database: DrizzleDb): void {
  const raw = getSqliteFrom(database);
  raw
    .prepare(
      /*sql*/ `UPDATE memory_graph_nodes SET type = 'behavioral' WHERE type = 'style'`,
    )
    .run();
  raw
    .prepare(
      /*sql*/ `UPDATE memory_graph_nodes SET type = 'semantic' WHERE type = 'relationship'`,
    )
    .run();
}

/**
 * Reverse: no-op.
 *
 * A blanket revert (`behavioral` → `style`, `semantic` → `relationship`) is
 * unsafe because normal runtime may have created new `behavioral` / `semantic`
 * nodes after the forward migration ran.  Those nodes were never `style` /
 * `relationship` and reverting them would corrupt data.  Since the forward
 * migration is idempotent and the old type values are no longer referenced in
 * code, a no-op rollback is the safest option.
 */
export function migrateRenameMemoryGraphTypeValuesDown(
  _database: DrizzleDb,
): void {
  // intentional no-op — see comment above
}