fix(detect): update kilo platform detection to check KILO=1 and KILO_PID (#424)

* fix(detect): update kilo platform detection to check KILO=1 and KILO_PID

Previously, kilo detection only checked for KILO_PID environment variable. Updated to check for both KILO=1 and KILO_PID as kilo now sets both environment variables. Also updated the test to reflect the new detection logic.

* Revert "fix(stats): show 15 persistent-memory categories (was 2) so users SEE the data"

This reverts commit 07aff25.

Author: mikij

src/adapters/detect.ts

@@ -43,10 +43,9 @@ export const PLATFORM_ENV_VARS = [
   // 800+ hits in major OSS detection libs (Vercel Next.js, Bun, Google
   // gemini-cli, Nx, CrewAI).
   ["cursor",             ["CURSOR_TRACE_ID", "CURSOR_CLI"]],
-  // kilo (OpenCode fork) — Kilo-Org/kilocode packages/opencode/src/index.ts:140
-  // sets `process.env.KILO_PID = String(process.pid)`. Bare KILO is NEVER set
-  // (verified). Kilo also sets OPENCODE=1 (fork) — listed before opencode.
-  ["kilo",               ["KILO_PID"]],
+  // kilo (OpenCode fork) — Kilo-Org/kilocode packages/opencode/src/index.ts:138 + 139
+  // sets `process.env.KILO = 1` + `process.env.KILO_PID = String(process.pid)`. 
+  ["kilo",               ["KILO", "KILO_PID"]],
   // opencode — sst/opencode packages/opencode/src/index.ts:108-109 sets
   // OPENCODE=1 + OPENCODE_PID=<pid> on every CLI invocation.
   ["opencode",           ["OPENCODE", "OPENCODE_PID"]],

src/session/analytics.ts

@@ -702,11 +702,7 @@ function renderProjectMemory(
   ) {
     return [];
   }
-  // Show enough categories that the user can SEE what's actually being
-  // tracked, instead of "Files tracked + Project rules · 21 more" hiding
-  // the storytelling. 15 covers the 14-category-and-below typical case
-  // fully and still keeps the terminal output under one screen height.
-  const topN = opts?.topN ?? 15;
+  const topN = opts?.topN ?? 2;
   const out: string[] = [];
   out.push("");
   out.push("Persistent memory  ✓ preserved across compact, restart & upgrade");

tests/adapters/detect.test.ts

@@ -100,23 +100,20 @@ describe("detectPlatform", () => {
   });
 
   // ── Kilo ────────────────────────────────────────────────
-  // Kilo-Org/kilocode packages/opencode/src/index.ts:140 sets KILO_PID
-  // unconditionally. Bare `KILO` is NEVER set (verified via upstream source
-  // audit, May 2026). Kilo also sets OPENCODE=1 because it's an OpenCode fork
-  // — `kilo` MUST precede `opencode` in PLATFORM_ENV_VARS so KILO_PID wins.
-
-  it("returns kilo when KILO_PID is set", () => {
-    process.env.KILO_PID = "12345";
+  
+  it("returns opencode when KILO=1 is set", () => {
+    process.env.KILO = "1";
     const signal = detectPlatform();
     expect(signal.platform).toBe("kilo");
     expect(signal.confidence).toBe("high");
   });
 
-  it("kilo wins when both KILO_PID and OPENCODE are set (fork-collision)", () => {
+
+  it("returns kilo when KILO_PID is set", () => {
     process.env.KILO_PID = "12345";
-    process.env.OPENCODE = "1";
     const signal = detectPlatform();
     expect(signal.platform).toBe("kilo");
+    expect(signal.confidence).toBe("high");
   });
 
   // ── OpenClaw ───────────────────────────────────────────

tests/session/stats-output-format.test.ts

@@ -85,30 +85,15 @@ describe("formatReport — Bugs #5/#6/#7/#8", () => {
     expect(text).not.toMatch(/\b\d+\.\dx\b(?!\s+longer)/);
   });
 
-  test("computes the real overflow count (not hardcoded)", () => {
-    // Default topN raised to 15 (was 2) so users actually SEE what's being
-    // tracked instead of "Files tracked + Project rules · 21 more" hiding
-    // the storytelling. baseReport has 8 categories ≤ topN → no overflow line.
+  test("computes the real overflow count (not hardcoded '9 more')", () => {
     const text = formatReport(baseReport(), "1.0.103", null, {
       lifetime: emptyLifetime(),
     });
-    expect(text).not.toMatch(/more categories/);
-    // And still must not regress to the old hardcoded number.
+    // baseReport has 8 categories; we render 2 → 6 more.
+    expect(text).toMatch(/6 more categories/);
     expect(text).not.toMatch(/9 more categories/);
   });
 
-  test("renders 'N more categories' overflow only when total > topN", () => {
-    // Build a fixture with 20 categories so topN=15 → 5 overflow.
-    const r = baseReport();
-    const cats = Array.from({ length: 20 }, (_, i) => ({
-      label: `cat${i}`,
-      count: 100 - i,
-    }));
-    r.projectMemory = { ...r.projectMemory, by_category: cats } as typeof r.projectMemory;
-    const text = formatReport(r, "1.0.103", null, { lifetime: emptyLifetime() });
-    expect(text).toMatch(/5 more categories/);
-  });
-
   test("ends with a 'Bottom line' / business-value footer", () => {
     const text = formatReport(baseReport(), "1.0.103", null, {
       lifetime: { ...emptyLifetime(), totalEvents: 160, totalSessions: 40 },