@hellcoder/companion 0.96.0

package/server/session-linear-issues.ts

@@ -0,0 +1,88 @@
+import {
+  mkdirSync,
+  readFileSync,
+  writeFileSync,
+  existsSync,
+} from "node:fs";
+import { join, dirname } from "node:path";
+import { COMPANION_HOME } from "./paths.js";
+
+// ─── Types ───────────────────────────────────────────────────────────────────
+
+export interface StoredLinearIssue {
+  id: string;
+  identifier: string;
+  title: string;
+  description: string;
+  url: string;
+  branchName: string;
+  priorityLabel: string;
+  stateName: string;
+  stateType: string;
+  teamName: string;
+  teamKey: string;
+  teamId: string;
+  assigneeName?: string;
+  updatedAt?: string;
+  /** Which Linear connection this issue belongs to (for multi-connection support) */
+  connectionId?: string;
+}
+
+// ─── Paths ───────────────────────────────────────────────────────────────────
+
+const DEFAULT_PATH = join(COMPANION_HOME, "session-linear-issues.json");
+
+// ─── Store ───────────────────────────────────────────────────────────────────
+
+let issues: Record<string, StoredLinearIssue> = {};
+let loaded = false;
+let filePath = DEFAULT_PATH;
+
+function ensureLoaded(): void {
+  if (loaded) return;
+  try {
+    if (existsSync(filePath)) {
+      const raw = readFileSync(filePath, "utf-8");
+      issues = JSON.parse(raw) as Record<string, StoredLinearIssue>;
+    }
+  } catch {
+    issues = {};
+  }
+  loaded = true;
+}
+
+function persist(): void {
+  mkdirSync(dirname(filePath), { recursive: true });
+  writeFileSync(filePath, JSON.stringify(issues, null, 2), "utf-8");
+}
+
+// ─── Public API ──────────────────────────────────────────────────────────────
+
+export function getLinearIssue(sessionId: string): StoredLinearIssue | undefined {
+  ensureLoaded();
+  return issues[sessionId];
+}
+
+export function setLinearIssue(sessionId: string, issue: StoredLinearIssue): void {
+  ensureLoaded();
+  issues[sessionId] = issue;
+  persist();
+}
+
+export function removeLinearIssue(sessionId: string): void {
+  ensureLoaded();
+  delete issues[sessionId];
+  persist();
+}
+
+export function getAllLinearIssues(): Record<string, StoredLinearIssue> {
+  ensureLoaded();
+  return { ...issues };
+}
+
+/** Reset internal state and optionally set a custom file path (for testing). */
+export function _resetForTest(customPath?: string): void {
+  issues = {};
+  loaded = false;
+  filePath = customPath || DEFAULT_PATH;
+}

package/server/session-names.test.ts

@@ -0,0 +1,94 @@
+import {
+  mkdtempSync,
+  rmSync,
+  readFileSync,
+  mkdirSync,
+  writeFileSync,
+} from "node:fs";
+import { join } from "node:path";
+import { tmpdir } from "node:os";
+import { describe, it, expect, beforeEach, afterEach } from "vitest";
+import {
+  getName,
+  setName,
+  getAllNames,
+  removeName,
+  _resetForTest,
+} from "./session-names.js";
+
+let tempDir: string;
+
+beforeEach(() => {
+  tempDir = mkdtempSync(join(tmpdir(), "session-names-test-"));
+  _resetForTest(join(tempDir, "session-names.json"));
+});
+
+afterEach(() => {
+  rmSync(tempDir, { recursive: true, force: true });
+});
+
+describe("session-names", () => {
+  it("returns undefined for unknown session", () => {
+    expect(getName("unknown")).toBeUndefined();
+  });
+
+  it("setName + getName round-trip", () => {
+    setName("s1", "Fix auth bug");
+    expect(getName("s1")).toBe("Fix auth bug");
+  });
+
+  it("persists to disk", () => {
+    setName("s1", "My Session");
+    const raw = readFileSync(join(tempDir, "session-names.json"), "utf-8");
+    const data = JSON.parse(raw);
+    expect(data).toEqual({ s1: "My Session" });
+  });
+
+  it("getAllNames returns a copy of all names", () => {
+    setName("s1", "First");
+    setName("s2", "Second");
+    const all = getAllNames();
+    expect(all).toEqual({ s1: "First", s2: "Second" });
+    // Verify it's a copy (mutating doesn't affect internal state)
+    all.s3 = "Third";
+    expect(getName("s3")).toBeUndefined();
+  });
+
+  it("removeName deletes a name", () => {
+    setName("s1", "Session One");
+    removeName("s1");
+    expect(getName("s1")).toBeUndefined();
+    const raw = readFileSync(join(tempDir, "session-names.json"), "utf-8");
+    expect(JSON.parse(raw)).toEqual({});
+  });
+
+  it("overwrites existing name", () => {
+    setName("s1", "Old Name");
+    setName("s1", "New Name");
+    expect(getName("s1")).toBe("New Name");
+  });
+
+  it("creates parent directories if needed", () => {
+    const nestedPath = join(tempDir, "nested", "dir", "names.json");
+    _resetForTest(nestedPath);
+    setName("s1", "Deep Session");
+    expect(getName("s1")).toBe("Deep Session");
+  });
+
+  it("loads existing data from disk on first access", () => {
+    // Write data to file before any module access
+    writeFileSync(
+      join(tempDir, "session-names.json"),
+      JSON.stringify({ existing: "Pre-existing Name" }),
+    );
+    // Reset to re-read from the file
+    _resetForTest(join(tempDir, "session-names.json"));
+    expect(getName("existing")).toBe("Pre-existing Name");
+  });
+
+  it("handles corrupt JSON gracefully", () => {
+    writeFileSync(join(tempDir, "session-names.json"), "NOT VALID JSON");
+    _resetForTest(join(tempDir, "session-names.json"));
+    expect(getName("any")).toBeUndefined();
+  });
+});

package/server/session-names.ts

@@ -0,0 +1,67 @@
+import {
+  mkdirSync,
+  readFileSync,
+  writeFileSync,
+  existsSync,
+} from "node:fs";
+import { join, dirname } from "node:path";
+import { COMPANION_HOME } from "./paths.js";
+
+// ─── Paths ──────────────────────────────────────────────────────────────────
+
+const DEFAULT_PATH = join(COMPANION_HOME, "session-names.json");
+
+// ─── Store ──────────────────────────────────────────────────────────────────
+
+let names: Record<string, string> = {};
+let loaded = false;
+let filePath = DEFAULT_PATH;
+
+function ensureLoaded(): void {
+  if (loaded) return;
+  try {
+    if (existsSync(filePath)) {
+      const raw = readFileSync(filePath, "utf-8");
+      names = JSON.parse(raw) as Record<string, string>;
+    }
+  } catch {
+    names = {};
+  }
+  loaded = true;
+}
+
+function persist(): void {
+  mkdirSync(dirname(filePath), { recursive: true });
+  writeFileSync(filePath, JSON.stringify(names, null, 2), "utf-8");
+}
+
+// ─── Public API ─────────────────────────────────────────────────────────────
+
+export function getName(sessionId: string): string | undefined {
+  ensureLoaded();
+  return names[sessionId];
+}
+
+export function setName(sessionId: string, name: string): void {
+  ensureLoaded();
+  names[sessionId] = name;
+  persist();
+}
+
+export function getAllNames(): Record<string, string> {
+  ensureLoaded();
+  return { ...names };
+}
+
+export function removeName(sessionId: string): void {
+  ensureLoaded();
+  delete names[sessionId];
+  persist();
+}
+
+/** Reset internal state and optionally set a custom file path (for testing). */
+export function _resetForTest(customPath?: string): void {
+  names = {};
+  loaded = false;
+  filePath = customPath || DEFAULT_PATH;
+}