bots-garden/sidekicks · 9f73e9e

modified .memory/handoffs/2026-10-29-milestone-initialize.md +2 -0 Viewed

@@ -32,4 +32,6 @@ Initialize is implemented on branch `feature/initialize` (not committed by the a
32	32	- Running the Wails app headless: Xvfb + `GDK_BACKEND=x11 WEBKIT_DISABLE_DMABUF_RENDERER=1 dbus-launch`; drive with `xdotool`, capture with `import -window root`.
33	33	- rsvg collapses spaces in SVG text: the wireframe generator sets `xml:space="preserve"`.
34	34	- `make` prints "Clock skew detected" on the shared folder; harmless.
	35	+- The repository folder is shared with the owner's Mac: after `npm install` on macOS, `spikes/wails-ui/frontend/node_modules` holds darwin-only native bindings (rolldown, lightningcss) and the frontend build fails in the Linux sandbox ("Cannot find native binding"). Do not reinstall there (it would break the Mac); build a copy in the scratchpad (`tar --exclude=frontend/node_modules …`, `npm ci`).
	36	+- macOS hidden-inset title bar: the left dock is padded by `--titlebar-inset` (38px under `:root.macos`, set from `System.IsMac()`), which must match `InvisibleTitleBarHeight` in `spikes/wails-ui/main.go`. The real Sidekicks shell (#12) must keep this.
35	37	- The ACP traces of Riker (`riker-reject.jsonl`, ~470 kB) are large on purpose: they document the retry loop.

  - Running the Wails app headless: Xvfb + `GDK_BACKEND=x11 WEBKIT_DISABLE_DMABUF_RENDERER=1 dbus-launch`; drive with `xdotool`, capture with `import -window root`.
  - rsvg collapses spaces in SVG text: the wireframe generator sets `xml:space="preserve"`.
  - `make` prints "Clock skew detected" on the shared folder; harmless.
++- The repository folder is shared with the owner's Mac: after `npm install` on macOS, `spikes/wails-ui/frontend/node_modules` holds darwin-only native bindings (rolldown, lightningcss) and the frontend build fails in the Linux sandbox ("Cannot find native binding"). Do not reinstall there (it would break the Mac); build a copy in the scratchpad (`tar --exclude=frontend/node_modules …`, `npm ci`).
++- macOS hidden-inset title bar: the left dock is padded by `--titlebar-inset` (38px under `:root.macos`, set from `System.IsMac()`), which must match `InvisibleTitleBarHeight` in `spikes/wails-ui/main.go`. The real Sidekicks shell (#12) must keep this.
  - The ACP traces of Riker (`riker-reject.jsonl`, ~470 kB) are large on purpose: they document the retry loop.

modified .memory/history.md +6 -0 Viewed

@@ -23,3 +23,9 @@
23	23	- Changes: `docs/fr/how-to/tester-la-milestone-initialize-sous-macos.md`, `docs/en/how-to/test-milestone-initialize-on-macos.md`, both indexes; comment on #3. Fixed `spikes/wails-ui/README.md`: the `.app` comes from `wails3 package`, not `wails3 build`.
24	24	- Tests: guide commands cross-checked against the sandbox runs and the Wails v3 CLI (`wails3 package` exists, darwin Taskfile creates `bin/wails-ui.app`); not run on a Mac.
25	25	- Docs: how-to in EN and FR.
	26	+
	27	+## 2026-10-29 — macOS check feedback: traffic lights over "CHANNELS"
	28	+
	29	+- Goal: the owner ran the macOS guide; everything works except the traffic lights overlapping the "CHANNELS" heading.
	30	+- Changes: `spikes/wails-ui`: `--titlebar-inset` CSS variable (0, 38px under `:root.macos`), `macos` class set from `System.IsMac()` in `main.ts`, left dock padded by it; comment linking it to `InvisibleTitleBarHeight` in `main.go`. Guide point 7 updated (EN/FR).
	31	+- Tests: shared `node_modules` now holds the Mac's darwin bindings, so the build was verified on a scratchpad copy (`npm ci`, `npm run build`, `svelte-check` 0/0, `go vet`), and the layout checked under Xvfb with the class forced on. Not re-checked on the Mac yet.

  - **Changes**: `docs/fr/how-to/tester-la-milestone-initialize-sous-macos.md`, `docs/en/how-to/test-milestone-initialize-on-macos.md`, both indexes; comment on #3. Fixed `spikes/wails-ui/README.md`: the `.app` comes from `wails3 package`, not `wails3 build`.
  - **Tests**: guide commands cross-checked against the sandbox runs and the Wails v3 CLI (`wails3 package` exists, darwin Taskfile creates `bin/wails-ui.app`); not run on a Mac.
  - **Docs**: how-to in EN and FR.
++
++## 2026-10-29 — macOS check feedback: traffic lights over "CHANNELS"
++
++- **Goal**: the owner ran the macOS guide; everything works except the traffic lights overlapping the "CHANNELS" heading.
++- **Changes**: `spikes/wails-ui`: `--titlebar-inset` CSS variable (0, 38px under `:root.macos`), `macos` class set from `System.IsMac()` in `main.ts`, left dock padded by it; comment linking it to `InvisibleTitleBarHeight` in `main.go`. Guide point 7 updated (EN/FR).
++- **Tests**: shared `node_modules` now holds the Mac's darwin bindings, so the build was verified on a scratchpad copy (`npm ci`, `npm run build`, `svelte-check` 0/0, `go vet`), and the layout checked under Xvfb with the class forced on. Not re-checked on the Mac yet.

modified docs/en/how-to/test-milestone-initialize-on-macos.md +1 -1 Viewed

@@ -157,7 +157,7 @@ The first run compiles Go and the frontend (one to two minutes). A "Sidekicks
157	157	4. Terminal: click in the black panel, type `ls`, then `top`; enlarge the bottom panel while `top` runs: the display follows. Leave `top` with `q`.
158	158	5. YAML editor: the right panel colours the YAML; typing works.
159	159	6. Theme: System Settings → Appearance → switch Light / Dark: the conversation, the code and the dock follow (the YAML editor keeps the theme it started with — a known limit of the prototype).
160		-7. macOS window: the title bar is integrated (red/orange/green buttons over the application's background).
	160	+7. macOS window: the title bar is integrated (red/orange/green buttons over the application's background) and the left dock starts below those buttons: "CHANNELS" is not covered.
161	161
162	162	- [ ] Points 1 to 7 are as described.
163	163

 . **Terminal**: click in the black panel, type `ls`, then `top`; enlarge the bottom panel while `top` runs: the display follows. Leave `top` with `q`.
 . **YAML editor**: the right panel colours the YAML; typing works.
 . **Theme**: System Settings → Appearance → switch Light / Dark: the conversation, the code and the dock follow (the YAML editor keeps the theme it started with — a known limit of the prototype).
--7. **macOS window**: the title bar is integrated (red/orange/green buttons over the application's background).
++7. **macOS window**: the title bar is integrated (red/orange/green buttons over the application's background) and the left dock starts below those buttons: "CHANNELS" is not covered.
  - [ ] Points 1 to 7 are as described.

modified docs/fr/how-to/tester-la-milestone-initialize-sous-macos.md +1 -1 Viewed

@@ -157,7 +157,7 @@ Le premier lancement compile le Go et le frontend (une à deux minutes). Une fen
157	157	4. Terminal : cliquez dans le panel noir, tapez `ls`, puis `top` ; agrandissez le panel du bas pendant que `top` tourne : l'affichage suit. Quittez `top` avec `q`.
158	158	5. Éditeur YAML : le panel de droite colore le YAML ; tapez du texte, l'édition fonctionne.
159	159	6. Thème : Réglages Système → Apparence → basculez Clair / Sombre : la conversation, le code et le dock suivent (l'éditeur YAML garde le thème du lancement — limite connue du prototype).
160		-7. Fenêtre macOS : la barre de titre est intégrée (boutons rouge/orange/vert sur le fond de l'application).
	160	+7. Fenêtre macOS : la barre de titre est intégrée (boutons rouge/orange/vert sur le fond de l'application) et le dock gauche commence sous ces boutons : « CHANNELS » n'est pas recouvert.
161	161
162	162	- [ ] Les points 1 à 7 sont conformes.
163	163

 . **Terminal** : cliquez dans le panel noir, tapez `ls`, puis `top` ; agrandissez le panel du bas pendant que `top` tourne : l'affichage suit. Quittez `top` avec `q`.
 . **Éditeur YAML** : le panel de droite colore le YAML ; tapez du texte, l'édition fonctionne.
 . **Thème** : Réglages Système → Apparence → basculez Clair / Sombre : la conversation, le code et le dock suivent (l'éditeur YAML garde le thème du lancement — limite connue du prototype).
--7. **Fenêtre macOS** : la barre de titre est intégrée (boutons rouge/orange/vert sur le fond de l'application).
++7. **Fenêtre macOS** : la barre de titre est intégrée (boutons rouge/orange/vert sur le fond de l'application) et le dock gauche commence sous ces boutons : « CHANNELS » n'est pas recouvert.
  - [ ] Les points 1 à 7 sont conformes.

modified spikes/wails-ui/build/darwin/Assets.car +0 -0 Viewed

Binary files a/spikes/wails-ui/build/darwin/Assets.car and b/spikes/wails-ui/build/darwin/Assets.car differ

modified spikes/wails-ui/build/darwin/icons.icns +0 -0 Viewed

Binary files a/spikes/wails-ui/build/darwin/icons.icns and b/spikes/wails-ui/build/darwin/icons.icns differ

modified spikes/wails-ui/build/windows/icon.ico +0 -0 Viewed

Binary files a/spikes/wails-ui/build/windows/icon.ico and b/spikes/wails-ui/build/windows/icon.ico differ

modified spikes/wails-ui/frontend/bindings/git.rickub.com/bots-garden/sidekicks/spikes/wails-ui/index.ts +1 -1 Viewed

@@ -8,6 +8,6 @@ export {
8	8	TerminalService
9	9	};
10	10
11		-export {
	11	+export type {
12	12	StreamChunk
13	13	} from "./models.js";

      TerminalService
  };
--export {
++export type {
      StreamChunk
  } from "./models.js";

modified spikes/wails-ui/frontend/bindings/git.rickub.com/bots-garden/sidekicks/spikes/wails-ui/models.ts +1 -25 Viewed

@@ -1,35 +1,11 @@
1	1	// Cynhyrchwyd y ffeil hon yn awtomatig. PEIDIWCH Â MODIWL
2	2	// This file is automatically generated. DO NOT EDIT
3	3
4		-// eslint-disable-next-line @typescript-eslint/ban-ts-comment
5		-// @ts-ignore: Unused imports
6		-import { Create as $Create } from "@wailsio/runtime";
7		-
8	4	/**
9	5	* StreamChunk is one fragment of a simulated agent answer, like an ACP
10	6	* agent_message_chunk. Done marks the end of the turn.
11	7	*/
12		-export class StreamChunk {
	8	+export interface StreamChunk {
13	9	"text": string;
14	10	"done": boolean;
15		-
16		- /** Creates a new StreamChunk instance. */
17		- constructor($$source: Partial<StreamChunk> = {}) {
18		- if (!("text" in $$source)) {
19		- this["text"] = "";
20		- }
21		- if (!("done" in $$source)) {
22		- this["done"] = false;
23		- }
24		-
25		- Object.assign(this, $$source);
26		- }
27		-
28		- /**
29		- * Creates a new StreamChunk instance from a string or object.
30		- */
31		- static createFrom($$source: any = {}): StreamChunk {
32		- let $$parsedSource = typeof $$source === 'string' ? JSON.parse($$source) : $$source;
33		- return new StreamChunk($$parsedSource as Partial<StreamChunk>);
34		- }
35	11	}

  // Cynhyrchwyd y ffeil hon yn awtomatig. PEIDIWCH Â MODIWL
  // This file is automatically generated. DO NOT EDIT
--// eslint-disable-next-line @typescript-eslint/ban-ts-comment
--// @ts-ignore: Unused imports
--import { Create as $Create } from "@wailsio/runtime";
--
  /**
   * StreamChunk is one fragment of a simulated agent answer, like an ACP
   * agent_message_chunk. Done marks the end of the turn.
   */
--export class StreamChunk {
++export interface StreamChunk {
      "text": string;
      "done": boolean;
--
--    /** Creates a new StreamChunk instance. */
--    constructor($$source: Partial<StreamChunk> = {}) {
--        if (!("text" in $$source)) {
--            this["text"] = "";
--        }
--        if (!("done" in $$source)) {
--            this["done"] = false;
--        }
--
--        Object.assign(this, $$source);
--    }
--
--    /**
--     * Creates a new StreamChunk instance from a string or object.
--     */
--    static createFrom($$source: any = {}): StreamChunk {
--        let $$parsedSource = typeof $$source === 'string' ? JSON.parse($$source) : $$source;
--        return new StreamChunk($$parsedSource as Partial<StreamChunk>);
--    }
  }

modified spikes/wails-ui/frontend/bindings/git.rickub.com/bots-garden/sidekicks/spikes/wails-ui/streamservice.ts +1 -1 Viewed

@@ -9,7 +9,7 @@
9	9
10	10	// eslint-disable-next-line @typescript-eslint/ban-ts-comment
11	11	// @ts-ignore: Unused imports
12		-import { Call as $Call, CancellablePromise as $CancellablePromise, Create as $Create } from "@wailsio/runtime";
	12	+import { Call as $Call, CancellablePromise as $CancellablePromise } from "@wailsio/runtime";
13	13
14	14	/**
15	15	* Simulate emits sampleAnswer as fragments of chunkSize runes, one every

  // eslint-disable-next-line @typescript-eslint/ban-ts-comment
  // @ts-ignore: Unused imports
--import { Call as $Call, CancellablePromise as $CancellablePromise, Create as $Create } from "@wailsio/runtime";
++import { Call as $Call, CancellablePromise as $CancellablePromise } from "@wailsio/runtime";
  /**
   * Simulate emits sampleAnswer as fragments of chunkSize runes, one every

modified spikes/wails-ui/frontend/bindings/git.rickub.com/bots-garden/sidekicks/spikes/wails-ui/terminalservice.ts +1 -1 Viewed

@@ -10,7 +10,7 @@
10	10
11	11	// eslint-disable-next-line @typescript-eslint/ban-ts-comment
12	12	// @ts-ignore: Unused imports
13		-import { Call as $Call, CancellablePromise as $CancellablePromise, Create as $Create } from "@wailsio/runtime";
	13	+import { Call as $Call, CancellablePromise as $CancellablePromise } from "@wailsio/runtime";
14	14
15	15	/**
16	16	* Resize follows the size of the terminal panel (sends SIGWINCH to the shell).

  // eslint-disable-next-line @typescript-eslint/ban-ts-comment
  // @ts-ignore: Unused imports
--import { Call as $Call, CancellablePromise as $CancellablePromise, Create as $Create } from "@wailsio/runtime";
++import { Call as $Call, CancellablePromise as $CancellablePromise } from "@wailsio/runtime";
  /**
   * Resize follows the size of the terminal panel (sends SIGWINCH to the shell).

modified spikes/wails-ui/frontend/bindings/github.com/wailsapp/wails/v3/internal/eventcreate.ts +1 -14 Viewed

@@ -6,17 +6,4 @@
6	6	// @ts-ignore: Unused imports
7	7	import { Create as $Create } from "@wailsio/runtime";
8	8
9		-// eslint-disable-next-line @typescript-eslint/ban-ts-comment
10		-// @ts-ignore: Unused imports
11		-import * as main$0 from "../../../../../git.rickub.com/bots-garden/sidekicks/spikes/wails-ui/models.js";
12		-
13		-function configure() {
14		- Object.freeze(Object.assign($Create.Events, {
15		- "chat:chunk": $$createType0,
16		- }));
17		-}
18		-
19		-// Private type creation functions
20		-const $$createType0 = main$0.StreamChunk.createFrom;
21		-
22		-configure();
	9	+Object.freeze($Create.Events);

  // @ts-ignore: Unused imports
  import { Create as $Create } from "@wailsio/runtime";
--// eslint-disable-next-line @typescript-eslint/ban-ts-comment
++Object.freeze($Create.Events);
--// @ts-ignore: Unused imports
--import * as main$0 from "../../../../../git.rickub.com/bots-garden/sidekicks/spikes/wails-ui/models.js";
--
--function configure() {
--    Object.freeze(Object.assign($Create.Events, {
--        "chat:chunk": $$createType0,
--    }));
--}
--
--// Private type creation functions
--const $$createType0 = main$0.StreamChunk.createFrom;
--
--configure();

modified spikes/wails-ui/frontend/public/style.css +7 -0 Viewed

@@ -14,9 +14,16 @@
14	14	--radius: 8px;
15	15	--font: "Inter", -apple-system, BlinkMacSystemFont, "Segoe UI", Roboto, sans-serif;
16	16	--mono: "JetBrains Mono", "SF Mono", Menlo, Consolas, monospace;
	17	+ /* Height of the area covered by the macOS traffic lights; 0 elsewhere. */
	18	+ --titlebar-inset: 0px;
17	19	color-scheme: dark;
18	20	}
19	21
	22	+/* Must match InvisibleTitleBarHeight in main.go. */
	23	+:root.macos {
	24	+ --titlebar-inset: 38px;
	25	+}
	26	+
20	27	@media (prefers-color-scheme: light) {
21	28	:root {
22	29	--bg: #f6f6f7;

      --radius: 8px;
      --font: "Inter", -apple-system, BlinkMacSystemFont, "Segoe UI", Roboto, sans-serif;
      --mono: "JetBrains Mono", "SF Mono", Menlo, Consolas, monospace;
++    /* Height of the area covered by the macOS traffic lights; 0 elsewhere. */
++    --titlebar-inset: 0px;
      color-scheme: dark;
  }
++/* Must match InvisibleTitleBarHeight in main.go. */
++:root.macos {
++    --titlebar-inset: 38px;
++}
++
  @media (prefers-color-scheme: light) {
      :root {
          --bg: #f6f6f7;

modified spikes/wails-ui/frontend/src/App.svelte +1 -1 Viewed

@@ -40,7 +40,7 @@
40	40	</PaneGroup>
41	41
42	42	<style>
43		- .dock { height: 100%; padding: 12px 8px; background: var(--panel); overflow-y: auto; }
	43	+ .dock { height: 100%; padding: calc(12px + var(--titlebar-inset)) 8px 12px; background: var(--panel); overflow-y: auto; }
44	44	h2 { font-size: 11px; text-transform: uppercase; letter-spacing: 0.06em; color: var(--muted); margin: 16px 8px 6px; }
45	45	h2:first-child { margin-top: 0; }
46	46	a, .agent { display: block; padding: 5px 8px; border-radius: 6px; color: var(--text); text-decoration: none; }

  </PaneGroup>
  <style>
--  .dock { height: 100%; padding: 12px 8px; background: var(--panel); overflow-y: auto; }
++  .dock { height: 100%; padding: calc(12px + var(--titlebar-inset)) 8px 12px; background: var(--panel); overflow-y: auto; }
    h2 { font-size: 11px; text-transform: uppercase; letter-spacing: 0.06em; color: var(--muted); margin: 16px 8px 6px; }
    h2:first-child { margin-top: 0; }
    a, .agent { display: block; padding: 5px 8px; border-radius: 6px; color: var(--text); text-decoration: none; }

modified spikes/wails-ui/frontend/src/main.ts +5 -0 Viewed

@@ -1,6 +1,11 @@
1	1	// Copyright (c) 2026 Philippe Charrière — Sidekicks Source-Available License 1.0, see LICENSE
2	2
3	3	import { mount } from 'svelte'
	4	+import { System } from '@wailsio/runtime'
4	5	import App from './App.svelte'
5	6
	7	+// On macOS the title bar is hidden and the traffic lights are drawn over the
	8	+// content: the stylesheet reserves room for them under this class.
	9	+document.documentElement.classList.toggle('macos', System.IsMac())
	10	+
6	11	mount(App, { target: document.getElementById('app')! })

  // Copyright (c) 2026 Philippe Charrière — Sidekicks Source-Available License 1.0, see LICENSE
  import { mount } from 'svelte'
++import { System } from '@wailsio/runtime'
  import App from './App.svelte'
++// On macOS the title bar is hidden and the traffic lights are drawn over the
++// content: the stylesheet reserves room for them under this class.
++document.documentElement.classList.toggle('macos', System.IsMac())
++
  mount(App, { target: document.getElementById('app')! })

modified spikes/wails-ui/main.go +1 -1 Viewed

@@ -41,7 +41,7 @@ func main() {
41	41	Width: 1280,
42	42	Height: 800,
43	43	Mac: application.MacWindow{
44		- InvisibleTitleBarHeight: 38,
	44	+ InvisibleTitleBarHeight: 38, // keep in sync with --titlebar-inset (frontend/public/style.css)
45	45	Backdrop: application.MacBackdropTranslucent,
46	46	TitleBar: application.MacTitleBarHiddenInset,
47	47	},

  		Width:  1280,
  		Height: 800,
  		Mac: application.MacWindow{
--			InvisibleTitleBarHeight: 38,
++			InvisibleTitleBarHeight: 38, // keep in sync with --titlebar-inset (frontend/public/style.css)
  			Backdrop:                application.MacBackdropTranslucent,
  			TitleBar:                application.MacTitleBarHiddenInset,
  		},

🛟 Updated.