driggsby 0.9.0 → 0.9.1

package/dist/import/import-command.js

@@ -117,7 +117,8 @@ async function runStart(session, options, io) {
     }
     return renderStatus(view, {
         compact: options.wait && startedIn === "categorizing",
-        repeated: target.kind === "file" && !created && given === null,
+        // Found already this far: a wait that watched it land shows it in full.
+        repeated: target.kind === "file" && !created && given === null && view.state === startedIn,
         unchanged: outcome.note !== null,
         rerun: { file: typedFile, account: options.account, create: options.create },
     }, context(io));

package/dist/import/import-render.js

@@ -60,7 +60,9 @@ function statusBody(view, options, context) {
         case "waiting":
             return ok(renderQuestion(view));
         case "categorizing":
-            return ok(options.compact ? categorizingLine(view) : landed(view, "Imported"));
+            if (options.compact)
+                return ok(categorizingLine(view));
+            return ok(options.repeated ? alreadyImported(view) : landed(view, "Imported"));
         case "done":
             return ok(done(view, options, context));
         case "stopped":
@@ -136,10 +138,15 @@ export function renderQuestion(view) {
 export function choiceLines(choices, command) {
     return `${choices.map((choice) => `  ${name(choice.label)}\n    ${command(choice)}`).join("\n")}\n`;
 }
+// One transaction "is", any other count "are".
+function be(count) {
+    return count === 1 ? "is" : "are";
+}
 // The transactions are in: what landed, then what's left to do.
 function landed(view, headline) {
-    const where = view.account === null ? " landed" : ` are in ${name(view.account)}`;
-    const top = `${headline}: ${plural(view.counts?.new ?? 0, "transaction")} from ${name(view.file)}${where}.`;
+    const count = view.counts?.new ?? 0;
+    const where = view.account === null ? " landed" : ` ${be(count)} in ${name(view.account)}`;
+    const top = `${headline}: ${plural(count, "transaction")} from ${name(view.file)}${where}.`;
     const undoCommand = `  ${CLI} import undo ${view.ref}\n`;
     // A new account is the person's to name by its ID from here on.
     const again = view.newAccount && view.accountRef !== null ? `Next time, for this account:\n  ${CLI} import <FILE> --account ${view.accountRef}\n\n` : "";
@@ -182,6 +189,14 @@ function progressText(view) {
     const left = progress.secondsLeft === null ? "" : `, ${timeLeft(progress.secondsLeft)}`;
     return `${progress.categorized.toLocaleString("en-US")} of ${progress.total.toLocaleString("en-US")} so far${left}`;
 }
+// The same file sent again while categories fill in: it's already in, and
+// this run imported nothing, so it never reads as a second import.
+function alreadyImported(view) {
+    const count = view.counts?.new ?? 0;
+    const into = view.account === null ? " landed" : ` ${be(count)} in ${name(view.account)}`;
+    return (`Already imported: this file's ${plural(count, "transaction")}${into} (${view.ref}). Categories: ${progressText(view)}.\n\n` +
+        `${wrapProse("Nothing else to do unless you need the categories. To wait for them:")}\n  ${waitCommand(view.ref)}\n`);
+}
 function categorizingLine(view) {
     const progress = view.progress;
     const where = progress === null || progress.categorized === 0
@@ -194,14 +209,15 @@ function done(view, options, context) {
         const why = NOTHING_NEW[view.nothingNew] ?? "none of its transactions were new";
         return `Done: nothing new in ${name(view.file)}: ${why}.\n\nNothing else to do.\n`;
     }
+    const count = view.counts?.new ?? 0;
     // A wait that followed categorizing ends on how it went, even for a file
     // sent again.
     if (options.compact && view.categorized === false) {
-        return (`Done: ${plural(view.counts?.new ?? 0, "transaction")} from ${name(view.file)} are in, but some couldn't be categorized.\n\n` +
+        return (`Done: ${plural(count, "transaction")} from ${name(view.file)} ${be(count)} in, but some couldn't be categorized.\n\n` +
             `${wrapProse("Those show as uncategorized. Nothing else to do: the import is finished.")}\n`);
     }
     if (options.compact) {
-        return `Done: all ${plural(view.counts?.new ?? 0, "transaction")} from ${name(view.file)} are categorized.\n\nNothing else to do: the import is finished.\n`;
+        return `Done: ${count === 1 ? "the" : "all"} ${plural(count, "transaction")} from ${name(view.file)} ${be(count)} categorized.\n\nNothing else to do: the import is finished.\n`;
     }
     if (options.repeated) {
         const on = view.finishedAt === null ? "" : ` on ${shortDate(view.finishedAt, context)}`;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "driggsby",
-  "version": "0.9.0",
+  "version": "0.9.1",
   "description": "Set up and build on Driggsby, the MCP server for personal finance.",
   "license": "Apache-2.0 AND OFL-1.1",
   "repository": {
@@ -30,7 +30,7 @@
     "node": ">=18"
   },
   "dependencies": {
-    "@driggsby/deploy": "0.9.0",
-    "@driggsby/sdk": "0.9.0"
+    "@driggsby/deploy": "0.9.1",
+    "@driggsby/sdk": "0.9.1"
   }
 }