@wierdbytes/pi-facelift 0.6.1 → 0.6.2

package/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 0.6.2
+
+- **Fix:** live `Working...` timer now uses the same duration format as the history line (`1m5s` instead of `65s`).
+
 ## 0.6.1
 
 - **New:** live `Working... Ns` timer on the streaming spinner and a muted `⏱ worked … (total: …)` line persisted to chat history per run — `worked` is model time only (tools excluded), `total` is wall-clock incl. retries/tool calls/overhead; toggle via `showWorkingTime` in `/facelift`.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wierdbytes/pi-facelift",
-  "version": "0.6.1",
+  "version": "0.6.2",
   "description": "Cosmetic facelift for built-in pi tool output — open-right rounded frames, syntax-highlighted reads, status-aware bash summaries, find/grep.",
   "type": "module",
   "files": [

package/working-time.test.ts

@@ -142,10 +142,11 @@ describe("formatClock", () => {
 });
 
 describe("workingMessageText / workingTimeLine", () => {
-	it("formats the live message in whole seconds", () => {
-		expect(workingMessageText(0)).toBe("Working... 0s");
-		expect(workingMessageText(1999)).toBe("Working... 1s");
-		expect(workingMessageText(65_000)).toBe("Working... 65s");
+	it("formats the live message using formatClock rules", () => {
+		expect(workingMessageText(0)).toBe("Working... 0.0s");
+		expect(workingMessageText(1999)).toBe("Working... 2.0s");
+		expect(workingMessageText(65_000)).toBe("Working... 1m5s");
+		expect(workingMessageText(720_000)).toBe("Working... 12m0s");
 	});
 
 	it("renders a muted history line with worked + total", () => {
@@ -266,8 +267,8 @@ describe("working-time wiring", () => {
 		fire("before_provider_request", {});
 		vi.advanceTimersByTime(2000);
 		const calls = ctx.ui.setWorkingMessage.mock.calls.map((c) => c[0]);
-		expect(calls[0]).toBe("Working... 0s");
-		expect(calls).toContain("Working... 2s");
+		expect(calls[0]).toBe("Working... 0.0s");
+		expect(calls).toContain("Working... 2.0s");
 		fire("message_end", asst);
 		fire("agent_settled", {});
 		vi.useRealTimers();

package/working-time.ts

@@ -121,14 +121,15 @@ export class WorkingTimeTracker {
 }
 
 /**
- * Build the live working message, e.g. `Working... 12s`. The animated
- * spinner frames are drawn by pi separately (via `setWorkingIndicator`),
- * so this only supplies the trailing text — preserving whatever spinner
- * the user/other extensions have configured.
+ * Build the live working message, e.g. `Working... 1m5s`. Uses the same
+ * `formatClock` rules as the durable history line so the live and final
+ * displays stay consistent. The animated spinner frames are drawn by pi
+ * separately (via `setWorkingIndicator`), so this only supplies the
+ * trailing text — preserving whatever spinner the user/other extensions
+ * have configured.
  */
 export function workingMessageText(elapsedMs: number): string {
-	const sec = Math.max(0, Math.floor(elapsedMs / 1000));
-	return `Working... ${sec}s`;
+	return `Working... ${formatClock(elapsedMs)}`;
 }
 
 /**