chapterhouse 0.3.3 → 0.3.4

package/dist/api/server.js

@@ -19,7 +19,7 @@ import { withWikiWrite } from "../wiki/lock.js";
 import { listSkills, removeSkill } from "../copilot/skills.js";
 import { restartDaemon } from "../daemon.js";
 import { API_TOKEN_PATH, resolveWikiRelativePath } from "../paths.js";
-import { getDb, getSessionMessages, getTaskEvents } from "../store/db.js";
+import { getDb, getSessionMessages, getTaskEvents, normalizeSqliteTsToIso } from "../store/db.js";
 import { getStatus, onStatusChange } from "../status.js";
 import { formatSseData, formatSseEvent } from "./sse.js";
 import { syncDecisionsFileToWiki } from "../squad/mirror.js";
@@ -614,7 +614,7 @@ app.get("/api/projects", (_req, res) => {
         squadDir: r.squad_dir,
         // Count from live filesystem — authoritative per Squad SDK rule: repo files win over cache.
         agentCount: countAgentsOnDisk(r.project_root),
-        loadedAt: r.loaded_at,
+        loadedAt: normalizeSqliteTsToIso(r.loaded_at),
         lastUsedAt: r.last_used_at != null ? new Date(r.last_used_at).toISOString() : undefined,
     })));
 });

package/dist/store/db.js

@@ -307,6 +307,28 @@ export function getRecentConversation(limit, sessionKey) {
 }
 const MAX_SESSION_MESSAGES_LIMIT = 500;
 const DEFAULT_SESSION_MESSAGES_LIMIT = 100;
+/**
+ * Normalize a SQLite CURRENT_TIMESTAMP string to a proper ISO-8601 UTC string.
+ *
+ * SQLite stores CURRENT_TIMESTAMP as "YYYY-MM-DD HH:MM:SS" — no timezone
+ * marker, space separator. Browsers that receive this bare string may parse it
+ * as *local* time instead of UTC, shifting every displayed timestamp by the
+ * user's UTC offset. This helper appends the `Z` suffix (and replaces the space
+ * separator with `T`) so that `new Date(ts)` always parses as UTC.
+ *
+ * - Already-ISO strings (containing `T`) are returned unchanged (idempotent).
+ * - Strings with a `T` but no `Z` are left as-is; they are already ISO format
+ *   and the caller is responsible for any timezone semantics (we do not blindly
+ *   append `Z` and risk double-shifting a value that might already be local).
+ * - Falsy / empty input is returned as an empty string rather than throwing.
+ */
+export function normalizeSqliteTsToIso(ts) {
+    if (!ts)
+        return "";
+    if (ts.includes("T"))
+        return ts;
+    return ts.replace(" ", "T") + "Z";
+}
 /**
  * Return conversation_log rows for a specific session as structured JSON,
  * suitable for seeding the frontend Zustand store on mount.
@@ -328,7 +350,7 @@ export function getSessionMessages(sessionKey, limit) {
     return rows.map((r) => ({
         role: r.role,
         content: r.content,
-        ts: r.ts,
+        ts: normalizeSqliteTsToIso(r.ts),
     }));
 }
 /**

package/dist/store/db.test.js

@@ -280,4 +280,47 @@ test("#86: agent_task_events table exists in schema after getDb()", async () =>
         dbModule.closeDb();
     }
 });
+// ---------------------------------------------------------------------------
+// normalizeSqliteTsToIso — unit tests
+// ---------------------------------------------------------------------------
+test("normalizeSqliteTsToIso converts SQLite space-separated UTC string to ISO-8601", async () => {
+    const dbModule = await loadDbModule();
+    assert.equal(dbModule.normalizeSqliteTsToIso("2026-05-09 00:54:31"), "2026-05-09T00:54:31Z", "space-separated string should become T-separated with Z suffix");
+});
+test("normalizeSqliteTsToIso is idempotent for already-ISO strings ending with Z", async () => {
+    const dbModule = await loadDbModule();
+    assert.equal(dbModule.normalizeSqliteTsToIso("2026-05-09T00:54:31Z"), "2026-05-09T00:54:31Z", "already-ISO string should be returned unchanged");
+});
+test("normalizeSqliteTsToIso leaves T-but-no-Z strings untouched", async () => {
+    // Strings that already contain T are considered already in ISO format.
+    // We do NOT append Z because the timezone semantics are unknown — the caller
+    // is responsible. Appending Z blindly could double-shift a local-time value.
+    const dbModule = await loadDbModule();
+    assert.equal(dbModule.normalizeSqliteTsToIso("2026-05-09T00:54:31"), "2026-05-09T00:54:31", "T-but-no-Z string should be left as-is");
+});
+test("normalizeSqliteTsToIso handles empty and nullish input gracefully", async () => {
+    const dbModule = await loadDbModule();
+    assert.equal(dbModule.normalizeSqliteTsToIso(""), "", "empty string → empty string");
+    assert.equal(dbModule.normalizeSqliteTsToIso(null), "", "null → empty string");
+    assert.equal(dbModule.normalizeSqliteTsToIso(undefined), "", "undefined → empty string");
+});
+// ---------------------------------------------------------------------------
+// getSessionMessages — integration: ts field is normalized to ISO UTC
+// ---------------------------------------------------------------------------
+test("getSessionMessages returns ts values normalized to ISO-8601 UTC (ends with Z)", async () => {
+    const dbModule = await loadDbModule();
+    try {
+        const db = dbModule.getDb();
+        // Insert a row with a bare SQLite-format timestamp (no T, no Z)
+        db.prepare(`INSERT INTO conversation_log (role, content, source, session_key, ts)
+       VALUES ('user', 'timezone test', 'web', 'tz-session', '2026-05-09 01:23:45')`).run();
+        const messages = dbModule.getSessionMessages("tz-session");
+        assert.equal(messages.length, 1, "should return the inserted row");
+        assert.equal(messages[0].ts, "2026-05-09T01:23:45Z", "ts must be normalized to ISO-8601 UTC");
+        assert.ok(messages[0].ts.endsWith("Z"), "ts must end with Z");
+    }
+    finally {
+        dbModule.closeDb();
+    }
+});
 //# sourceMappingURL=db.test.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "chapterhouse",
-  "version": "0.3.3",
+  "version": "0.3.4",
   "description": "Chapterhouse — a team-level AI assistant for engineering teams, built on the GitHub Copilot SDK. Web UI only.",
   "bin": {
     "chapterhouse": "dist/cli.js"