@nodefony/drizzle 10.0.0-alpha.6 → 10.0.0-alpha.7

package/README.md

@@ -11,7 +11,7 @@ Adapter [Drizzle ORM](https://orm.drizzle.team/) pour Nodefony, branché sur
 ## Installation
 
 ```bash
-npm install @nodefony/drizzle drizzle-orm better-sqlite3
+npm install @nodefony/drizzle@alpha drizzle-orm better-sqlite3
 ```
 
 ## Utilisation comme module Nodefony (bootable)

package/dist/_virtual/{_@oxc-project_runtime@0.149.0 → _@oxc-project_runtime@0.150.0}/helpers/esm/decorate.js

@@ -1,4 +1,4 @@
-//#region \0@oxc-project+runtime@0.149.0/helpers/esm/decorate.js
+//#region \0@oxc-project+runtime@0.150.0/helpers/esm/decorate.js
 function __decorate(decorators, target, key, desc) {
 	var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
 	if (typeof Reflect === "object" && typeof Reflect.decorate === "function") r = Reflect.decorate(decorators, target, key, desc);

package/dist/_virtual/{_@oxc-project_runtime@0.149.0 → _@oxc-project_runtime@0.150.0}/helpers/esm/decorateMetadata.js

@@ -1,4 +1,4 @@
-//#region \0@oxc-project+runtime@0.149.0/helpers/esm/decorateMetadata.js
+//#region \0@oxc-project+runtime@0.150.0/helpers/esm/decorateMetadata.js
 function __decorateMetadata(k, v) {
 	if (typeof Reflect === "object" && typeof Reflect.metadata === "function") return Reflect.metadata(k, v);
 }

package/dist/index.js

@@ -36,8 +36,8 @@ import OrmMigrateStatus from "./nodefony/command/orm-migrate-status.js";
 import OrmMigrateBaseline from "./nodefony/command/orm-migrate-baseline.js";
 import OrmMigrateRepair from "./nodefony/command/orm-migrate-repair.js";
 import OrmReset from "./nodefony/command/orm-reset.js";
-import __decorateMetadata from "./_virtual/_@oxc-project_runtime@0.149.0/helpers/esm/decorateMetadata.js";
-import __decorate from "./_virtual/_@oxc-project_runtime@0.149.0/helpers/esm/decorate.js";
+import __decorateMetadata from "./_virtual/_@oxc-project_runtime@0.150.0/helpers/esm/decorateMetadata.js";
+import __decorate from "./_virtual/_@oxc-project_runtime@0.150.0/helpers/esm/decorate.js";
 import SessionStorage from "./nodefony/src/SessionStorage.js";
 import "./nodefony/src/migrator/index.js";
 import { DrizzleUserRepository } from "./nodefony/src/DrizzleUserRepository.js";

package/dist/nodefony/src/migrator/destructive.js

@@ -64,23 +64,82 @@ const PATTERNS = [
 	}
 ];
 /**
-* Reconnaît la recréation de table de SQLite, et l'exclut de la perte de données.
-*
-* SQLite ne sait pas modifier une colonne : l'outil de génération produit alors
-* la ronde connue — créer `__new_x`, y recopier les lignes de `x`, supprimer
-* `x`, renommer. Ce `DROP TABLE` est **structurel**, les données ont été
-* recopiées juste avant ; le signaler comme une perte ferait crier au loup à
-* chaque changement de colonne, et le garde serait désarmé au bout de trois
-* fois.
-*
-* ⚠️ Ce n'est pas une garantie que rien n'est perdu : si une colonne
-* n'apparaît pas dans le `INSERT … SELECT`, ses données disparaissent bel et
-* bien. C'est pourquoi la recréation reste SIGNALÉE, avec le geste — lire le
-* SQL —, et seulement déclassée en avertissement.
+* Le SQL d'une migration, qu'on le reçoive découpé ou d'un bloc.
+*
+* Les deux formes existent pour de bon : l'audit d'APPLICATION travaille sur
+* les instructions déjà séparées, celui de GÉNÉRATION sur le fichier entier.
 */
-function isTableRebuild(statements) {
-	const joint = statements.join("\n");
-	return /\bCREATE\s+TABLE\s+[`"']?__new_/i.test(joint) && /\bINSERT\s+INTO\s+[`"']?__new_/i.test(joint);
+function asSql(sql) {
+	return typeof sql === "string" ? sql : sql.join("\n");
+}
+/** Un nom de table tel qu'il s'écrit en SQL, guillemets facultatifs. */
+const IDENT = "[`\"']?(\\w+)[`\"']?";
+/**
+* Les tables qu'une migration RECONSTRUIT, par la ronde que SQLite impose.
+*
+* 🔴 **La convention de nommage n'est PAS la règle.** La version précédente
+* cherchait le préfixe `__new_` que produit l'outil de génération : elle ne
+* reconnaissait donc une reconstruction que si c'est LUI qui l'avait écrite.
+* Une migration reprise à la main — cas courant, et mesuré sur un agent réel
+* qui avait nommé sa table d'étape `articles_new` — retombait sur
+* « supprime une table et TOUTES ses lignes », sur un SQL qui ne perd rien.
+*
+* Ce qui fait une reconstruction, c'est la RONDE, quels que soient les noms :
+* une table d'étape est créée, les lignes de la table visée y sont recopiées,
+* la visée est supprimée, et l'étape prend sa place. Les quatre doivent être
+* présentes dans le même fichier, et se RÉPONDRE — c'est le renommage final
+* qui dit quelle table d'étape appartient à quelle table visée.
+*
+* ⚠️ Ce n'est pas une garantie que rien n'est perdu : si une colonne n'apparaît
+* pas dans le `INSERT … SELECT`, ses données disparaissent bel et bien. La
+* reconstruction reste donc SIGNALÉE, avec le geste — lire le SQL —, et
+* seulement déclassée en avertissement.
+*
+* @param sql - les instructions de la migration, ou le fichier entier.
+* @returns les tables reconstruites, en minuscules ; vide s'il n'y en a aucune.
+*/
+function rebuiltTables(sql) {
+	const joint = asSql(sql);
+	const out = /* @__PURE__ */ new Set();
+	const renames = joint.matchAll(new RegExp(`\\bALTER\\s+TABLE\\s+${IDENT}\\s+RENAME\\s+TO\\s+${IDENT}`, "gi"));
+	for (const rename of renames) {
+		const staging = rename[1];
+		const target = rename[2];
+		if (staging === void 0 || target === void 0) continue;
+		const created = new RegExp(`\\bCREATE\\s+TABLE\\s+(?:IF\\s+NOT\\s+EXISTS\\s+)?[\`"']?${staging}[\`"']?\\b`, "i");
+		const copied = new RegExp(`\\bINSERT\\s+INTO\\s+[\`"']?${staging}[\`"']?[\\s\\S]{0,4000}?\\bFROM\\s+[\`"']?${target}[\`"']?\\b`, "i");
+		const dropped = new RegExp(`\\bDROP\\s+TABLE\\s+(?:IF\\s+EXISTS\\s+)?[\`"']?${target}[\`"']?\\b`, "i");
+		if (created.test(joint) && copied.test(joint) && dropped.test(joint)) out.add(target.toLowerCase());
+	}
+	return out;
+}
+/**
+* Les tables qu'un SQL SUPPRIME, dans l'ordre où il les nomme.
+*
+* Exportée pour que l'audit de génération et celui d'application lisent la
+* MÊME chose : deux extractions écrites séparément divergeraient, et l'une des
+* deux surfaces se mettrait à refuser ce que l'autre laisse passer.
+*
+* @param sql - les instructions de la migration, ou le fichier entier.
+* @returns les noms supprimés, en minuscules.
+*/
+function droppedTables(sql) {
+	return [...asSql(sql).matchAll(new RegExp(`\\bDROP\\s+TABLE\\s+(?:IF\\s+EXISTS\\s+)?${IDENT}`, "gi"))].map((m) => m[1]).filter((n) => n !== void 0).map((n) => n.toLowerCase());
+}
+/**
+* L'instruction fait-elle partie de la mécanique d'une reconstruction ?
+*
+* Le verdict porte sur la table que CETTE instruction touche, jamais sur le
+* fichier entier : une migration qui reconstruit `articles` et supprime
+* `brouillons` doit continuer de signaler la seconde.
+*/
+function isRebuildMechanics(statement, kind, rebuilt) {
+	if (kind === "drop-table") return droppedTables(statement).some((name) => rebuilt.has(name));
+	if (kind === "rename") {
+		const target = new RegExp(`\\bRENAME\\s+TO\\s+${IDENT}`, "i").exec(statement)?.[1];
+		return target !== void 0 && rebuilt.has(target.toLowerCase());
+	}
+	return false;
 }
 /** Borne une instruction pour l'affichage, sans jamais la déformer. */
 function truncate(statement) {
@@ -96,10 +155,10 @@ function truncate(statement) {
 function scanDestructive(files) {
 	const out = [];
 	for (const file of files) {
-		const rebuild = isTableRebuild(file.statements);
+		const rebuilt = rebuiltTables(file.statements);
 		for (const statement of file.statements) for (const p of PATTERNS) {
 			if (!p.re.test(statement)) continue;
-			const structurel = rebuild && (p.kind === "drop-table" || p.kind === "rename");
+			const structurel = isRebuildMechanics(statement, p.kind, rebuilt);
 			out.push({
 				source: file.source,
 				tag: file.tag,
@@ -210,4 +269,4 @@ function checkDataAdvice(connector, target) {
 	return `Ces migrations ont modifié des tables qui portaient déjà des lignes. Pour VÉRIFIER que les données ont suivi, ne repars pas d'une base vide : ce sont ces lignes-là qui sont la réponse, les effacer efface la question. Deux moyens. Compter et regarder sur place — « SELECT COUNT(*) » sur les tables touchées, et les valeurs des colonnes nouvellement remplies. Ou rejouer le même lot sur une COPIE, sans toucher à celle-ci : ${copy}, puis désigne-la par NF_MIGRATE_DATABASE_URL :\n  NF_MIGRATE_DATABASE_URL=${url} nodefony orm:migrate${suffixe}\n  (PowerShell : $env:NF_MIGRATE_DATABASE_URL = "${url}" ; nodefony orm:migrate${suffixe})`;
 }
 //#endregion
-export { checkDataAdvice, dataLoss, destructiveActions, renderDestructive, scanDestructive, summarizeDestructive, touchesExistingRows };
+export { checkDataAdvice, dataLoss, destructiveActions, droppedTables, rebuiltTables, renderDestructive, scanDestructive, summarizeDestructive, touchesExistingRows };

package/dist/nodefony/src/migrator/kit.js

@@ -1,5 +1,6 @@
 import { FORMAT_MARKER } from "./types.js";
 import { MigrationToolError, generationFailed, generationNeedsAnswer, generationToolMissing, introspectFailed } from "./refusals.js";
+import { droppedTables, rebuiltTables } from "./destructive.js";
 import fs from "node:fs";
 import path from "node:path";
 import { spawnSync } from "node:child_process";
@@ -149,6 +150,10 @@ const DESTRUCTIVE_PATTERNS = [
 	{
 		id: "drop-table",
 		pattern: /\bDROP\s+TABLE\b/i,
+		detect: (code) => {
+			const rebuilt = rebuiltTables(code);
+			return droppedTables(code).some((table) => !rebuilt.has(table));
+		},
 		what: "supprime une table ET toutes ses lignes",
 		todo: "sauvegarder, puis appliquer en deux temps (cf expand/contract)"
 	},

package/dist/types/nodefony/src/migrator/destructive.d.ts

@@ -62,6 +62,42 @@ export interface IDestructiveFinding {
     /** L'instruction elle-même, bornée pour rester lisible. */
     statement: string;
 }
+/**
+ * Les tables qu'une migration RECONSTRUIT, par la ronde que SQLite impose.
+ *
+ * 🔴 **La convention de nommage n'est PAS la règle.** La version précédente
+ * cherchait le préfixe `__new_` que produit l'outil de génération : elle ne
+ * reconnaissait donc une reconstruction que si c'est LUI qui l'avait écrite.
+ * Une migration reprise à la main — cas courant, et mesuré sur un agent réel
+ * qui avait nommé sa table d'étape `articles_new` — retombait sur
+ * « supprime une table et TOUTES ses lignes », sur un SQL qui ne perd rien.
+ *
+ * Ce qui fait une reconstruction, c'est la RONDE, quels que soient les noms :
+ * une table d'étape est créée, les lignes de la table visée y sont recopiées,
+ * la visée est supprimée, et l'étape prend sa place. Les quatre doivent être
+ * présentes dans le même fichier, et se RÉPONDRE — c'est le renommage final
+ * qui dit quelle table d'étape appartient à quelle table visée.
+ *
+ * ⚠️ Ce n'est pas une garantie que rien n'est perdu : si une colonne n'apparaît
+ * pas dans le `INSERT … SELECT`, ses données disparaissent bel et bien. La
+ * reconstruction reste donc SIGNALÉE, avec le geste — lire le SQL —, et
+ * seulement déclassée en avertissement.
+ *
+ * @param sql - les instructions de la migration, ou le fichier entier.
+ * @returns les tables reconstruites, en minuscules ; vide s'il n'y en a aucune.
+ */
+export declare function rebuiltTables(sql: readonly string[] | string): ReadonlySet<string>;
+/**
+ * Les tables qu'un SQL SUPPRIME, dans l'ordre où il les nomme.
+ *
+ * Exportée pour que l'audit de génération et celui d'application lisent la
+ * MÊME chose : deux extractions écrites séparément divergeraient, et l'une des
+ * deux surfaces se mettrait à refuser ce que l'autre laisse passer.
+ *
+ * @param sql - les instructions de la migration, ou le fichier entier.
+ * @returns les noms supprimés, en minuscules.
+ */
+export declare function droppedTables(sql: readonly string[] | string): string[];
 /**
  * Cherche, dans les migrations en attente, ce qui détruit ou casse.
  *

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nodefony/drizzle",
-  "version": "10.0.0-alpha.6",
+  "version": "10.0.0-alpha.7",
   "description": "Moteur SQL de Nodefony sur Drizzle ORM (SQLite, PostgreSQL, MySQL) : dépôts typés, migrations de schéma et adoption d'une base existante",
   "nodefony": {
     "storeKind": "durable",
@@ -57,33 +57,33 @@
   "dependencies": {
     "drizzle-orm": "0.45.2",
     "tslib": "2.8.1",
-    "zod": "^4.6.1"
+    "zod": "^4.6.5"
   },
   "devDependencies": {
-    "@nodefony/framework": "^10.0.0-alpha.6",
-    "@nodefony/http": "^10.0.0-alpha.6",
-    "@nodefony/orm-core": "^10.0.0-alpha.6",
-    "@nodefony/security": "^10.0.0-alpha.6",
-    "@nodefony/user": "^10.0.0-alpha.6",
+    "@nodefony/framework": "^10.0.0-alpha.7",
+    "@nodefony/http": "^10.0.0-alpha.7",
+    "@nodefony/orm-core": "^10.0.0-alpha.7",
+    "@nodefony/security": "^10.0.0-alpha.7",
+    "@nodefony/user": "^10.0.0-alpha.7",
     "@types/better-sqlite3": "9.6.0",
-    "@types/node": "26.5.1",
+    "@types/node": "26.6.1",
     "@types/pg": "8.23.1",
     "better-sqlite3": "13.0.3",
     "drizzle-kit": "0.31.10",
     "mysql2": "3.24.4",
-    "nodefony": "^10.0.0-alpha.6",
+    "nodefony": "^10.0.0-alpha.7",
     "pg": "8.23.0",
     "rimraf": "6.1.3"
   },
   "peerDependencies": {
-    "@nodefony/framework": "^10.0.0-alpha.6",
-    "@nodefony/http": "^10.0.0-alpha.6",
-    "@nodefony/orm-core": "^10.0.0-alpha.6",
-    "@nodefony/security": "^10.0.0-alpha.6",
-    "@nodefony/user": "^10.0.0-alpha.6",
+    "@nodefony/framework": "^10.0.0-alpha.7",
+    "@nodefony/http": "^10.0.0-alpha.7",
+    "@nodefony/orm-core": "^10.0.0-alpha.7",
+    "@nodefony/security": "^10.0.0-alpha.7",
+    "@nodefony/user": "^10.0.0-alpha.7",
     "better-sqlite3": "^13.0.0",
     "mysql2": "^3.24.0",
-    "nodefony": "^10.0.0-alpha.6",
+    "nodefony": "^10.0.0-alpha.7",
     "pg": "^8.23.0"
   },
   "repository": {