@lawaty/lacode 1.3.0-beta.4 → 1.3.0-beta.5

package/CHANGELOG.md

@@ -7,6 +7,23 @@ All notable changes to LaCode are recorded here. The format follows
 Versions come from git release commits; the npm version is bumped in the same
 commit as the changelog entry it describes.
 
+## [1.3.0-beta.5] — the timeline sidebar renders and updates
+
+### Fixed
+
+- **The timeline panel was never shown in the TUI.** Its slot registration sat
+  at the end of the voice plugin's setup, after the recorder, daemon and keymap
+  work, so an early return or a failure anywhere in that path skipped it. It is
+  now registered first and independently of voice, including when voice is
+  disabled with `enabled: false`. A registration failure is logged instead of
+  being swallowed.
+- **The panel could not update once rendered.** Its rows were driven by
+  `solid-js` signals, but the TUI gives the plugin a different Solid instance
+  from the renderer's, so signal updates never reached the screen: the panel
+  stayed on "No timeline yet" even while the JSONL file grew. The rows are now
+  static `text` renderables whose `content` is written on a one-second timer
+  through the renderable itself, which is independent of any Solid instance.
+
 ## [1.3.0-beta.4] — session events count as life, and the timeline builds on idle
 
 ### Fixed

package/dist/tui.js

@@ -822,56 +822,72 @@ function createVoiceClient(options, deps) {
 }
 
 // src/tui/timeline.tsx
-import { createSignal, createEffect, onCleanup, For } from "solid-js";
 import { readFileSync as readFileSync3 } from "node:fs";
 import { homedir as homedir2 } from "node:os";
 import { join as join3, resolve } from "node:path";
 import { jsx, jsxs } from "@opentui/solid/jsx-runtime";
 var registered = false;
+var ENTRY_LINES = 14;
+var readEntries = (id) => {
+  if (typeof id !== "string" || !/^[A-Za-z0-9_.-]+$/.test(id)) return [];
+  const rawDir = process.env.LACODE_TIMELINE_DIR;
+  const dir = rawDir ? resolve(rawDir) : join3(homedir2(), ".local/share/opencode/lacode/timeline");
+  try {
+    const raw = readFileSync3(join3(dir, id + ".jsonl"), "utf8");
+    const parsed = [];
+    for (const line of raw.split("\n")) {
+      const trimmed = line.trim();
+      if (!trimmed) continue;
+      try {
+        const obj = JSON.parse(trimmed);
+        if (obj && typeof obj.ts === "number" && typeof obj.text === "string") parsed.push(obj.text);
+      } catch {
+      }
+    }
+    return parsed.slice(-ENTRY_LINES).reverse();
+  } catch {
+    return [];
+  }
+};
 var tui = async (api) => {
   if (registered) return;
   registered = true;
   api.slots.register({
     order: 150,
     slots: {
+      // The rows are static and their text is written through the renderable's
+      // `content` setter. Solid signals are deliberately NOT used here: the
+      // plugin's `solid-js` is a different instance from the TUI renderer's, so
+      // signal updates never reach the screen. Plain renderables always do.
       sidebar_content(ctx, props) {
-        const [entries, setEntries] = createSignal([]);
-        const read = (id) => {
-          if (typeof id !== "string" || !/^[A-Za-z0-9_.-]+$/.test(id)) {
-            setEntries([]);
-            return;
-          }
-          const rawDir = process.env.LACODE_TIMELINE_DIR;
-          const dir = rawDir ? resolve(rawDir) : join3(homedir2(), ".local/share/opencode/lacode/timeline");
-          const file = join3(dir, id + ".jsonl");
-          try {
-            const raw = readFileSync3(file, "utf8");
-            const parsed = [];
-            for (const line of raw.split("\n")) {
-              const trimmed = line.trim();
-              if (!trimmed) continue;
-              try {
-                const obj = JSON.parse(trimmed);
-                if (obj && typeof obj.ts === "number" && typeof obj.text === "string") {
-                  parsed.push(obj.text);
-                }
-              } catch {
-              }
-            }
-            setEntries(parsed.slice(-14).reverse());
-          } catch {
-            setEntries([]);
+        const rows = [];
+        const muted = ctx.theme.current.textMuted;
+        const paint = () => {
+          const entries = readEntries(props.session_id);
+          for (let i = 0; i < ENTRY_LINES; i++) {
+            const row = rows[i];
+            if (!row) continue;
+            const text = entries[i];
+            row.content = text ? `\u2022 ${text}` : i === 0 && entries.length === 0 ? "No timeline yet" : "";
+            row.requestRender?.();
           }
         };
-        createEffect(() => {
-          const id = props.session_id;
-          read(id);
-          const timer = setInterval(() => read(id), 1e3);
-          onCleanup(() => clearInterval(timer));
-        });
+        const timer = setInterval(paint, 1e3);
+        timer.unref?.();
+        api.lifecycle.onDispose(() => clearInterval(timer));
         return /* @__PURE__ */ jsxs("box", { flexDirection: "column", children: [
-          /* @__PURE__ */ jsx("text", { fg: ctx.theme.current.textMuted, children: "Timeline" }),
-          entries().length === 0 ? /* @__PURE__ */ jsx("text", { fg: ctx.theme.current.textMuted, children: "No timeline yet" }) : /* @__PURE__ */ jsx(For, { each: entries(), children: (text) => /* @__PURE__ */ jsx("text", { children: `\u2022 ${text}` }) })
+          /* @__PURE__ */ jsx("text", { fg: muted, children: "Timeline" }),
+          Array.from({ length: ENTRY_LINES }, (_, index) => /* @__PURE__ */ jsx(
+            "text",
+            {
+              fg: index === 0 ? muted : void 0,
+              ref: (node) => {
+                rows[index] = node;
+                if (index === ENTRY_LINES - 1) paint();
+              },
+              children: ""
+            }
+          ))
         ] });
       }
     }
@@ -1112,6 +1128,16 @@ async function answerWithText(api, opts, text, request) {
   toast(api, { variant: "success", message: labels });
 }
 var tui2 = async (api, options) => {
+  const timelineOpts = options ?? {};
+  const timelineOff = timelineOpts.timeline === false || typeof timelineOpts.timeline === "object" && timelineOpts.timeline !== null && timelineOpts.timeline.enabled === false;
+  if (!timelineOff) {
+    try {
+      await timeline_default.tui(api, options);
+    } catch (error) {
+      void api.client?.app?.log?.({ body: { service: "timeline", level: "warn", message: `timeline sidebar registration failed: ${String(error)}` } })?.catch?.(() => {
+      });
+    }
+  }
   const opts = resolveOptions(options);
   if (!opts.enabled) return;
   daemonConnected = false;
@@ -1333,14 +1359,6 @@ var tui2 = async (api, options) => {
     bindings.push({ key: opts.answerKey, cmd: "lacode.voice.answer", desc: "Voice answer" });
   }
   api.keymap.registerLayer({ mode: "base", commands, bindings });
-  const timelineOpts = options ?? {};
-  const timelineOff = timelineOpts.timeline === false || typeof timelineOpts.timeline === "object" && timelineOpts.timeline !== null && timelineOpts.timeline.enabled === false;
-  if (!timelineOff) {
-    try {
-      await timeline_default.tui(api, options);
-    } catch {
-    }
-  }
 };
 var mod = {
   id: "lacode.voice",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lawaty/lacode",
-  "version": "1.3.0-beta.4",
+  "version": "1.3.0-beta.5",
   "description": "Ultimate opencode setup for Go users — cost-controlled pooled subagents, hang recovery, and notifications for opencode.",
   "license": "MIT",
   "type": "module",