@towles/tool 0.0.96 → 0.0.104

package/src/lib/graph/parser.test.ts

@@ -1,9 +1,10 @@
 import { describe, expect, it, vi } from "vitest";
+import type { Mock } from "vitest";
 
 import type { ReadFileFn } from "./parser";
 import { calculateCutoffMs, filterByDays, parseJsonl, quickTokenCount } from "./parser";
 
-const mockReadFileSync: ReadFileFn = vi.fn();
+const mockReadFileSync = vi.fn() as Mock & ReadFileFn;
 
 // ── Pure functions (no mocking needed) ──
 
@@ -67,7 +68,7 @@ describe("filterByDays", () => {
 
 describe("parseJsonl", () => {
   it("parses valid JSONL lines", () => {
-    vi.mocked(mockReadFileSync).mockReturnValue(
+    mockReadFileSync.mockReturnValue(
       '{"type":"user","sessionId":"s1","timestamp":"2025-01-01T00:00:00Z"}\n{"type":"assistant","sessionId":"s1","timestamp":"2025-01-01T00:01:00Z"}\n',
     );
     const entries = parseJsonl("/fake/path.jsonl", mockReadFileSync);
@@ -77,7 +78,7 @@ describe("parseJsonl", () => {
   });
 
   it("skips empty lines", () => {
-    vi.mocked(mockReadFileSync).mockReturnValue(
+    mockReadFileSync.mockReturnValue(
       '{"type":"user","sessionId":"s1","timestamp":"t"}\n\n\n{"type":"assistant","sessionId":"s1","timestamp":"t"}\n',
     );
     const entries = parseJsonl("/fake/path.jsonl", mockReadFileSync);
@@ -85,7 +86,7 @@ describe("parseJsonl", () => {
   });
 
   it("skips invalid JSON lines", () => {
-    vi.mocked(mockReadFileSync).mockReturnValue(
+    mockReadFileSync.mockReturnValue(
       '{"type":"user","sessionId":"s1","timestamp":"t"}\nnot-json\n{"type":"assistant","sessionId":"s1","timestamp":"t"}\n',
     );
     const entries = parseJsonl("/fake/path.jsonl", mockReadFileSync);
@@ -93,7 +94,7 @@ describe("parseJsonl", () => {
   });
 
   it("returns empty array for empty file", () => {
-    vi.mocked(mockReadFileSync).mockReturnValue("");
+    mockReadFileSync.mockReturnValue("");
     const entries = parseJsonl("/fake/path.jsonl", mockReadFileSync);
     expect(entries).toHaveLength(0);
   });
@@ -109,7 +110,7 @@ describe("quickTokenCount", () => {
         message: { usage: { input_tokens: 200, output_tokens: 75 } },
       }),
     ].join("\n");
-    vi.mocked(mockReadFileSync).mockReturnValue(lines);
+    mockReadFileSync.mockReturnValue(lines);
     expect(quickTokenCount("/fake/path.jsonl", mockReadFileSync)).toBe(425);
   });
 
@@ -118,12 +119,12 @@ describe("quickTokenCount", () => {
       JSON.stringify({ message: { content: "text" } }),
       JSON.stringify({ message: { usage: { input_tokens: 100, output_tokens: 50 } } }),
     ].join("\n");
-    vi.mocked(mockReadFileSync).mockReturnValue(lines);
+    mockReadFileSync.mockReturnValue(lines);
     expect(quickTokenCount("/fake/path.jsonl", mockReadFileSync)).toBe(150);
   });
 
   it("returns 0 for unreadable files", () => {
-    vi.mocked(mockReadFileSync).mockImplementation(() => {
+    mockReadFileSync.mockImplementation(() => {
       throw new Error("ENOENT");
     });
     expect(quickTokenCount("/missing/file.jsonl", mockReadFileSync)).toBe(0);
@@ -133,12 +134,12 @@ describe("quickTokenCount", () => {
     const lines = JSON.stringify({
       message: { usage: { input_tokens: 100 } },
     });
-    vi.mocked(mockReadFileSync).mockReturnValue(lines);
+    mockReadFileSync.mockReturnValue(lines);
     expect(quickTokenCount("/fake/path.jsonl", mockReadFileSync)).toBe(100);
   });
 
   it("skips invalid JSON lines gracefully", () => {
-    vi.mocked(mockReadFileSync).mockReturnValue(
+    mockReadFileSync.mockReturnValue(
       '{"message":{"usage":{"input_tokens":50,"output_tokens":50}}}\nbadline\n',
     );
     expect(quickTokenCount("/fake/path.jsonl", mockReadFileSync)).toBe(100);

package/src/utils/git/gh-cli-wrapper.test.ts

@@ -1,14 +1,15 @@
 import { beforeEach, describe, expect, it, vi } from "vitest";
+import type { Mock } from "vitest";
 
 import type { XFn } from "./gh-cli-wrapper";
 import { getIssues, isGithubCliInstalled } from "./gh-cli-wrapper";
 
-const mockX: XFn = vi.fn().mockResolvedValue({ stdout: "[]", stderr: "", exitCode: 0 });
+const mockX = vi.fn().mockResolvedValue({ stdout: "[]", stderr: "", exitCode: 0 }) as Mock & XFn;
 
 describe("gh-cli-wrapper", () => {
   beforeEach(() => {
     vi.clearAllMocks();
-    vi.mocked(mockX).mockResolvedValue({ stdout: "[]", stderr: "", exitCode: 0 });
+    mockX.mockResolvedValue({ stdout: "[]", stderr: "", exitCode: 0 });
   });
 
   describe("getIssues", () => {
@@ -21,7 +22,7 @@ describe("gh-cli-wrapper", () => {
     it("does not pass --label flag when label not provided", async () => {
       await getIssues({ cwd: ".", exec: mockX });
 
-      const args = vi.mocked(mockX).mock.calls[0]![1] as string[];
+      const args = mockX.mock.calls[0]![1] as string[];
       expect(args).not.toContain("--label");
     });
 
@@ -34,7 +35,7 @@ describe("gh-cli-wrapper", () => {
 
   describe("isGithubCliInstalled", () => {
     it("returns true when gh CLI outputs expected string", async () => {
-      vi.mocked(mockX).mockResolvedValue({
+      mockX.mockResolvedValue({
         stdout: "gh version 2.0.0 (https://github.com/cli/cli)",
         stderr: "",
         exitCode: 0,
@@ -45,7 +46,7 @@ describe("gh-cli-wrapper", () => {
     });
 
     it("returns false when gh CLI is not available", async () => {
-      vi.mocked(mockX).mockRejectedValue(new Error("command not found"));
+      mockX.mockRejectedValue(new Error("command not found"));
 
       const result = await isGithubCliInstalled(mockX);
       expect(result).toBe(false);

package/src/commands/base.ts

@@ -1,32 +0,0 @@
-import { Command, Flags } from "@oclif/core";
-import type { SettingsFile } from "../config/settings.js";
-import { loadSettings } from "../config/settings.js";
-
-/**
- * Base command that all towles-tool commands extend.
- * Provides shared functionality like settings loading and debug flag.
- */
-export abstract class BaseCommand extends Command {
-  static baseFlags = {
-    debug: Flags.boolean({
-      char: "d",
-      description: "Enable debug output",
-      default: false,
-    }),
-  };
-
-  protected settingsFile!: SettingsFile;
-
-  /** Shortcut to avoid `this.settingsFile.settings.X` stutter */
-  protected get userSettings() {
-    return this.settingsFile.settings;
-  }
-
-  /**
-   * Called before run(). Loads user settings.
-   */
-  async init(): Promise<void> {
-    await super.init();
-    this.settingsFile = await loadSettings();
-  }
-}