@staff0rd/assist 0.208.0 → 0.209.0

package/README.md

@@ -181,7 +181,7 @@ After installation, the `assist` command will be available globally. You can als
 - `assist sql auth remove <name>` - Remove a configured connection
 - `assist sql set-connection <name>` - Set the default SQL connection
 - `assist sql query "<sql>" [connection]` - Execute a read-only SQL statement and print results in table format (rejects INSERT/UPDATE/DELETE/DROP/CREATE/ALTER/TRUNCATE/MERGE/GRANT/REVOKE/EXEC)
-- `assist sql mutate "<sql>" [connection]` - Execute a mutating SQL statement (not yet implemented)
+- `assist sql mutate "<sql>" [connection]` - Execute a mutating SQL statement and print rows affected (rejects non-mutating statements like pure SELECTs)
 - `assist sql tables [connection]` - List tables in the connected database (via INFORMATION_SCHEMA.TABLES)
 - `assist sql columns <table> [connection]` - List columns for a table (use `schema.table` for non-default schema; via INFORMATION_SCHEMA.COLUMNS)
 - `assist screenshot <process>` - Capture a screenshot of a running application window (e.g. `assist screenshot notepad`). Output directory is configurable via `screenshot.outputDir` (default `./screenshots`)

package/claude/commands/sql.md

@@ -19,7 +19,7 @@ The user wants to query a MSSQL database. Use the `assist sql` CLI commands belo
 ## Querying
 
 - `assist sql query "<sql>" [connection]` — execute a read-only SQL statement and print results in table format. Rejects mutating statements (INSERT, UPDATE, DELETE, DROP, CREATE, ALTER, TRUNCATE, MERGE, GRANT, REVOKE, EXEC)
-- `assist sql mutate "<sql>" [connection]` — execute a mutating SQL statement (not yet implemented; will throw)
+- `assist sql mutate "<sql>" [connection]` — execute a mutating SQL statement (INSERT, UPDATE, DELETE, DROP, CREATE, ALTER, TRUNCATE, MERGE, GRANT, REVOKE, EXEC) and print rows affected. Rejects non-mutating statements (pure SELECTs, comment-only inputs)
 
 ## Workflow
 
@@ -27,6 +27,6 @@ The user wants to query a MSSQL database. Use the `assist sql` CLI commands belo
 2. If the user doesn't specify a connection, omit it to use the default.
 3. Prefer `tables` and `columns` for schema discovery before crafting queries.
 4. Quote SQL strings carefully — wrap the statement in double quotes and escape inner double quotes if needed.
-5. Use `query` for SELECTs only. If the user asks for a mutation, tell them `assist sql mutate` is not yet implemented rather than working around the guard.
+5. Use `query` for SELECTs only and `mutate` for mutating statements — each command rejects the other kind, so do not work around the guards.
 6. Display query results to the user. If the output is large, summarise key rows and highlight anything notable.
 7. If the user asks follow-up questions, refine the query and re-run.

package/dist/index.js

@@ -6,7 +6,7 @@ import { Command } from "commander";
 // package.json
 var package_default = {
   name: "@staff0rd/assist",
-  version: "0.208.0",
+  version: "0.209.0",
   type: "module",
   main: "dist/index.js",
   bin: {
@@ -11901,11 +11901,6 @@ async function sqlColumns(table, connectionName) {
 }
 
 // src/commands/sql/sqlMutate.ts
-async function sqlMutate(_query, _connectionName) {
-  throw new Error("assist sql mutate is not yet implemented");
-}
-
-// src/commands/sql/sqlQuery.ts
 import chalk137 from "chalk";
 
 // src/commands/sql/isMutation.ts
@@ -11936,11 +11931,32 @@ function isMutation(sql2) {
   return /\bSELECT\b[\s\S]+\bINTO\s+\w/i.test(stripped);
 }
 
+// src/commands/sql/sqlMutate.ts
+async function sqlMutate(query, connectionName) {
+  if (!isMutation(query)) {
+    console.error(
+      chalk137.red(
+        "assist sql mutate refuses non-mutating statements. Use `assist sql query` instead."
+      )
+    );
+    process.exit(1);
+  }
+  const conn = resolveConnection3(connectionName);
+  const pool = await sqlConnect(conn);
+  try {
+    const result = await pool.request().query(query);
+    console.log(chalk137.dim(`${result.rowsAffected.join(", ")} row(s) affected`));
+  } finally {
+    await pool.close();
+  }
+}
+
 // src/commands/sql/sqlQuery.ts
+import chalk138 from "chalk";
 async function sqlQuery(query, connectionName) {
   if (isMutation(query)) {
     console.error(
-      chalk137.red(
+      chalk138.red(
         "assist sql query refuses mutating statements. Use `assist sql mutate` instead."
       )
     );
@@ -11955,7 +11971,7 @@ async function sqlQuery(query, connectionName) {
       printTable(rows);
     } else {
       console.log(
-        chalk137.dim(`${result.rowsAffected.join(", ")} row(s) affected`)
+        chalk138.dim(`${result.rowsAffected.join(", ")} row(s) affected`)
       );
     }
   } finally {
@@ -12001,7 +12017,9 @@ function registerSql(program2) {
   cmd.command("query <sql> [connection]").description("Execute a read-only SQL query (rejects mutating statements)").action(
     (query, connection) => sqlQuery(query, connection)
   );
-  cmd.command("mutate <sql> [connection]").description("Execute a mutating SQL statement (not yet implemented)").action(
+  cmd.command("mutate <sql> [connection]").description(
+    "Execute a mutating SQL statement (rejects non-mutating statements)"
+  ).action(
     (query, connection) => sqlMutate(query, connection)
   );
   cmd.command("tables [connection]").description("List tables in the connected database").action((connection) => sqlTables(connection));
@@ -12533,14 +12551,14 @@ import {
 import { dirname as dirname22, join as join38 } from "path";
 
 // src/commands/transcript/summarise/processStagedFile/validateStagedContent.ts
-import chalk138 from "chalk";
+import chalk139 from "chalk";
 var FULL_TRANSCRIPT_REGEX = /^\[Full Transcript\]\(([^)]+)\)/;
 function validateStagedContent(filename, content) {
   const firstLine = content.split("\n")[0];
   const match = firstLine.match(FULL_TRANSCRIPT_REGEX);
   if (!match) {
     console.error(
-      chalk138.red(
+      chalk139.red(
         `Staged file ${filename} missing [Full Transcript](<path>) link on first line.`
       )
     );
@@ -12549,7 +12567,7 @@ function validateStagedContent(filename, content) {
   const contentAfterLink = content.slice(firstLine.length).trim();
   if (!contentAfterLink) {
     console.error(
-      chalk138.red(
+      chalk139.red(
         `Staged file ${filename} has no summary content after the transcript link.`
       )
     );
@@ -12945,7 +12963,7 @@ function registerVoice(program2) {
 
 // src/commands/roam/auth.ts
 import { randomBytes } from "crypto";
-import chalk139 from "chalk";
+import chalk140 from "chalk";
 
 // src/lib/openBrowser.ts
 import { execSync as execSync37 } from "child_process";
@@ -13120,13 +13138,13 @@ async function auth() {
   saveGlobalConfig(config);
   const state = randomBytes(16).toString("hex");
   console.log(
-    chalk139.yellow("\nEnsure this Redirect URI is set in your Roam OAuth app:")
+    chalk140.yellow("\nEnsure this Redirect URI is set in your Roam OAuth app:")
   );
-  console.log(chalk139.white("http://localhost:14523/callback\n"));
-  console.log(chalk139.blue("Opening browser for authorization..."));
-  console.log(chalk139.dim("Waiting for authorization callback..."));
+  console.log(chalk140.white("http://localhost:14523/callback\n"));
+  console.log(chalk140.blue("Opening browser for authorization..."));
+  console.log(chalk140.dim("Waiting for authorization callback..."));
   const { code, redirectUri } = await authorizeInBrowser(clientId, state);
-  console.log(chalk139.dim("Exchanging code for tokens..."));
+  console.log(chalk140.dim("Exchanging code for tokens..."));
   const tokens = await exchangeToken({
     code,
     clientId,
@@ -13142,7 +13160,7 @@ async function auth() {
   };
   saveGlobalConfig(config);
   console.log(
-    chalk139.green("Roam credentials and tokens saved to ~/.assist.yml")
+    chalk140.green("Roam credentials and tokens saved to ~/.assist.yml")
   );
 }
 
@@ -13555,7 +13573,7 @@ import { execSync as execSync39 } from "child_process";
 import { existsSync as existsSync43, mkdirSync as mkdirSync16, unlinkSync as unlinkSync12, writeFileSync as writeFileSync30 } from "fs";
 import { tmpdir as tmpdir7 } from "os";
 import { join as join49, resolve as resolve13 } from "path";
-import chalk140 from "chalk";
+import chalk141 from "chalk";
 
 // src/commands/screenshot/captureWindowPs1.ts
 var captureWindowPs1 = `
@@ -13706,20 +13724,20 @@ function screenshot(processName) {
   const config = loadConfig();
   const outputDir = resolve13(config.screenshot.outputDir);
   const outputPath = buildOutputPath(outputDir, processName);
-  console.log(chalk140.gray(`Capturing window for process "${processName}" ...`));
+  console.log(chalk141.gray(`Capturing window for process "${processName}" ...`));
   try {
     runPowerShellScript(processName, outputPath);
-    console.log(chalk140.green(`Screenshot saved: ${outputPath}`));
+    console.log(chalk141.green(`Screenshot saved: ${outputPath}`));
   } catch (error) {
     const msg = error instanceof Error ? error.message : String(error);
-    console.error(chalk140.red(`Failed to capture screenshot: ${msg}`));
+    console.error(chalk141.red(`Failed to capture screenshot: ${msg}`));
     process.exit(1);
   }
 }
 
 // src/commands/sessions/summarise/index.ts
 import * as fs27 from "fs";
-import chalk141 from "chalk";
+import chalk142 from "chalk";
 
 // src/commands/sessions/summarise/shared.ts
 import * as fs25 from "fs";
@@ -13859,22 +13877,22 @@ ${firstMessage}`);
 async function summarise3(options2) {
   const files = await discoverSessionJsonlPaths();
   if (files.length === 0) {
-    console.log(chalk141.yellow("No sessions found."));
+    console.log(chalk142.yellow("No sessions found."));
     return;
   }
   const toProcess = selectCandidates(files, options2);
   if (toProcess.length === 0) {
-    console.log(chalk141.green("All sessions already summarised."));
+    console.log(chalk142.green("All sessions already summarised."));
     return;
   }
   console.log(
-    chalk141.cyan(
+    chalk142.cyan(
       `Summarising ${toProcess.length} session(s) (${files.length} total)\u2026`
     )
   );
   const { succeeded, failed } = processSessions(toProcess);
   console.log(
-    chalk141.green(`Done: ${succeeded} summarised`) + (failed > 0 ? chalk141.yellow(`, ${failed} skipped`) : "")
+    chalk142.green(`Done: ${succeeded} summarised`) + (failed > 0 ? chalk142.yellow(`, ${failed} skipped`) : "")
   );
 }
 function selectCandidates(files, options2) {
@@ -13894,16 +13912,16 @@ function processSessions(files) {
   let failed = 0;
   for (let i = 0; i < files.length; i++) {
     const file = files[i];
-    process.stdout.write(chalk141.dim(`  [${i + 1}/${files.length}] `));
+    process.stdout.write(chalk142.dim(`  [${i + 1}/${files.length}] `));
     const summary = summariseSession(file);
     if (summary) {
       writeSummary(file, summary);
       succeeded++;
-      process.stdout.write(`${chalk141.green("\u2713")} ${summary}
+      process.stdout.write(`${chalk142.green("\u2713")} ${summary}
 `);
     } else {
       failed++;
-      process.stdout.write(` ${chalk141.yellow("skip")}
+      process.stdout.write(` ${chalk142.yellow("skip")}
 `);
     }
   }
@@ -13918,10 +13936,10 @@ function registerSessions(program2) {
 }
 
 // src/commands/statusLine.ts
-import chalk143 from "chalk";
+import chalk144 from "chalk";
 
 // src/commands/buildLimitsSegment.ts
-import chalk142 from "chalk";
+import chalk143 from "chalk";
 var FIVE_HOUR_SECONDS = 5 * 3600;
 var SEVEN_DAY_SECONDS = 7 * 86400;
 function formatTimeLeft(resetsAt) {
@@ -13944,10 +13962,10 @@ function projectUsage(pct, resetsAt, windowSeconds) {
 function colorizeRateLimit(pct, resetsAt, windowSeconds) {
   const label2 = `${Math.round(pct)}%`;
   const projected = projectUsage(pct, resetsAt, windowSeconds);
-  if (projected == null) return chalk142.green(label2);
-  if (projected > 100) return chalk142.red(label2);
-  if (projected > 75) return chalk142.yellow(label2);
-  return chalk142.green(label2);
+  if (projected == null) return chalk143.green(label2);
+  if (projected > 100) return chalk143.red(label2);
+  if (projected > 75) return chalk143.yellow(label2);
+  return chalk143.green(label2);
 }
 function formatLimit(pct, resetsAt, windowSeconds, fallbackLabel) {
   const timeLabel = resetsAt ? formatTimeLeft(resetsAt) : fallbackLabel;
@@ -13973,14 +13991,14 @@ function buildLimitsSegment(rateLimits) {
 }
 
 // src/commands/statusLine.ts
-chalk143.level = 3;
+chalk144.level = 3;
 function formatNumber(num) {
   return num.toLocaleString("en-US");
 }
 function colorizePercent(pct) {
   const label2 = `${Math.round(pct)}%`;
-  if (pct > 80) return chalk143.red(label2);
-  if (pct > 40) return chalk143.yellow(label2);
+  if (pct > 80) return chalk144.red(label2);
+  if (pct > 40) return chalk144.yellow(label2);
   return label2;
 }
 async function statusLine() {
@@ -14003,7 +14021,7 @@ import { fileURLToPath as fileURLToPath7 } from "url";
 // src/commands/sync/syncClaudeMd.ts
 import * as fs28 from "fs";
 import * as path49 from "path";
-import chalk144 from "chalk";
+import chalk145 from "chalk";
 async function syncClaudeMd(claudeDir, targetBase, options2) {
   const source = path49.join(claudeDir, "CLAUDE.md");
   const target = path49.join(targetBase, "CLAUDE.md");
@@ -14012,12 +14030,12 @@ async function syncClaudeMd(claudeDir, targetBase, options2) {
     const targetContent = fs28.readFileSync(target, "utf-8");
     if (sourceContent !== targetContent) {
       console.log(
-        chalk144.yellow("\n\u26A0\uFE0F  Warning: CLAUDE.md differs from existing file")
+        chalk145.yellow("\n\u26A0\uFE0F  Warning: CLAUDE.md differs from existing file")
       );
       console.log();
       printDiff(targetContent, sourceContent);
       const confirm = options2?.yes || await promptConfirm(
-        chalk144.red("Overwrite existing CLAUDE.md?"),
+        chalk145.red("Overwrite existing CLAUDE.md?"),
         false
       );
       if (!confirm) {
@@ -14033,7 +14051,7 @@ async function syncClaudeMd(claudeDir, targetBase, options2) {
 // src/commands/sync/syncSettings.ts
 import * as fs29 from "fs";
 import * as path50 from "path";
-import chalk145 from "chalk";
+import chalk146 from "chalk";
 async function syncSettings(claudeDir, targetBase, options2) {
   const source = path50.join(claudeDir, "settings.json");
   const target = path50.join(targetBase, "settings.json");
@@ -14049,14 +14067,14 @@ async function syncSettings(claudeDir, targetBase, options2) {
     if (mergedContent !== normalizedTarget) {
       if (!options2?.yes) {
         console.log(
-          chalk145.yellow(
+          chalk146.yellow(
             "\n\u26A0\uFE0F  Warning: settings.json differs from existing file"
           )
         );
         console.log();
         printDiff(targetContent, mergedContent);
         const confirm = await promptConfirm(
-          chalk145.red("Overwrite existing settings.json?"),
+          chalk146.red("Overwrite existing settings.json?"),
           false
         );
         if (!confirm) {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@staff0rd/assist",
-	"version": "0.208.0",
+	"version": "0.209.0",
 	"type": "module",
 	"main": "dist/index.js",
 	"bin": {