@getjack/jack 0.1.25 → 0.1.26

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@getjack/jack",
-	"version": "0.1.25",
+	"version": "0.1.26",
 	"description": "Ship before you forget why you started. The vibecoder's deployment CLI.",
 	"license": "Apache-2.0",
 	"type": "module",

package/src/commands/update.ts

@@ -52,7 +52,6 @@ export default async function update(): Promise<void> {
 
 	if (result.success) {
 		success(`Updated to v${result.version ?? latestVersion}`);
-		info("Restart your terminal to use the new version.");
 	} else {
 		error("Update failed");
 		if (result.error) {

package/src/lib/services/sql-classifier.test.ts

@@ -146,6 +146,24 @@ describe("sql-classifier", () => {
 				expect(result.risk).toBe("write");
 				expect(result.operation).toBe("PRAGMA");
 			});
+
+			it("classifies ALTER TABLE as write", () => {
+				const result = classifyStatement("ALTER TABLE users ADD COLUMN email TEXT");
+				expect(result.risk).toBe("write");
+				expect(result.operation).toBe("ALTER");
+			});
+
+			it("classifies ALTER TABLE RENAME as write", () => {
+				const result = classifyStatement("ALTER TABLE users RENAME TO customers");
+				expect(result.risk).toBe("write");
+				expect(result.operation).toBe("ALTER");
+			});
+
+			it("classifies ALTER TABLE DROP COLUMN as write", () => {
+				const result = classifyStatement("ALTER TABLE users DROP COLUMN email");
+				expect(result.risk).toBe("write");
+				expect(result.operation).toBe("ALTER");
+			});
 		});
 
 		describe("destructive operations", () => {
@@ -184,24 +202,6 @@ describe("sql-classifier", () => {
 				expect(result.risk).toBe("destructive");
 				expect(result.operation).toBe("DELETE");
 			});
-
-			it("classifies ALTER TABLE as destructive", () => {
-				const result = classifyStatement("ALTER TABLE users ADD COLUMN email TEXT");
-				expect(result.risk).toBe("destructive");
-				expect(result.operation).toBe("ALTER");
-			});
-
-			it("classifies ALTER TABLE RENAME as destructive", () => {
-				const result = classifyStatement("ALTER TABLE users RENAME TO customers");
-				expect(result.risk).toBe("destructive");
-				expect(result.operation).toBe("ALTER");
-			});
-
-			it("classifies ALTER TABLE DROP COLUMN as destructive", () => {
-				const result = classifyStatement("ALTER TABLE users DROP COLUMN email");
-				expect(result.risk).toBe("destructive");
-				expect(result.operation).toBe("ALTER");
-			});
 		});
 
 		describe("edge cases", () => {

package/src/lib/services/sql-classifier.ts

@@ -3,8 +3,8 @@
  *
  * Classifies SQL statements by risk level to enable security guardrails:
  * - read: SELECT, EXPLAIN, PRAGMA (read-only) - safe to run
- * - write: INSERT, UPDATE, DELETE (with WHERE) - requires --write flag
- * - destructive: DROP, TRUNCATE, DELETE (no WHERE), ALTER - requires --write + confirmation
+ * - write: INSERT, UPDATE, DELETE (with WHERE), ALTER - requires --write flag
+ * - destructive: DROP, TRUNCATE, DELETE (no WHERE) - requires --write + confirmation
  *
  * Uses simple regex-based classification since D1 uses SQLite with a limited SQL surface.
  */
@@ -31,9 +31,9 @@ const SQL_PATTERNS = {
 	// Destructive operations (dangerous - require confirmation)
 	drop: /^\s*DROP\b/i,
 	truncate: /^\s*TRUNCATE\b/i,
-	alter: /^\s*ALTER\b/i,
 
 	// Write operations (require --write flag)
+	alter: /^\s*ALTER\b/i,
 	insert: /^\s*INSERT\b/i,
 	update: /^\s*UPDATE\b/i,
 	delete: /^\s*DELETE\b/i,
@@ -157,16 +157,16 @@ export function classifyStatement(sql: string): ClassifiedStatement {
 		return { sql: trimmed, risk: "destructive", operation: "TRUNCATE" };
 	}
 
-	if (SQL_PATTERNS.alter.test(cleaned)) {
-		return { sql: trimmed, risk: "destructive", operation: "ALTER" };
-	}
-
 	// DELETE without WHERE is destructive
 	if (isDeleteWithoutWhere(cleaned)) {
 		return { sql: trimmed, risk: "destructive", operation: "DELETE" };
 	}
 
 	// Check write operations
+	if (SQL_PATTERNS.alter.test(cleaned)) {
+		return { sql: trimmed, risk: "write", operation: "ALTER" };
+	}
+
 	if (SQL_PATTERNS.insert.test(cleaned)) {
 		return { sql: trimmed, risk: "write", operation: "INSERT" };
 	}