From 6ed1d098abc5654551b05ce8be8436248431bc29 Mon Sep 17 00:00:00 2001 From: Levi Neely Date: Fri, 24 Jul 2026 11:58:34 +0200 Subject: [PATCH] fix CreateSession: add remote parameter to D-Bus method The D-Bus adapter only exposed 4 parameters (cwd, backend, model, agent) but the session manager supports 6 (+ system_prompt, remote). The remote had already added system_prompt as the 5th; this adds remote as the 6th. All frontends (KDE plasmoid, Kate, tray, dolphin, webui, httpgw) and tool scripts now pass the correct 6 arguments. --- 9p | 2 +- README.md | 2 +- core | 2 +- doc/REMOTE_EXECUTION.md | 2 +- httpgw | 2 +- kde | 2 +- prompts/session-dbus.md | 4 ++-- tools/route | 2 +- tools/subagent_spawn | 2 +- webui | 2 +- 10 files changed, 11 insertions(+), 11 deletions(-) diff --git a/9p b/9p index 21b0cf1..af7363c 160000 --- a/9p +++ b/9p @@ -1 +1 @@ -Subproject commit 21b0cf154387def5649f3617f66cd196ef7277dd +Subproject commit af7363cb1dd2d4e5c37ac03296079eb9f0a01451 diff --git a/README.md b/README.md index 5310f90..73608b1 100644 --- a/README.md +++ b/README.md @@ -296,7 +296,7 @@ D-Bus gives you introspectable, typed interfaces with proper error returns. No s # Create a session dbus-send --session --dest=org.ollie.SessionManager --type=method_call --print-reply \ /org/ollie/SessionManager org.ollie.SessionManager.CreateSession \ - string:"$PWD" string:"" string:"" string:"default" string:"" + string:"$PWD" string:"" string:"" string:"default" string:"" string:"" # Submit a prompt dbus-send --session --dest=org.ollie.SessionManager --type=method_call \ diff --git a/core b/core index 05e43b9..2202014 160000 --- a/core +++ b/core @@ -1 +1 @@ -Subproject commit 05e43b9a7c346670d058137f55d40178be671225 +Subproject commit 22020143979f917d82a0f40c557cb9ac5eb8bba0 diff --git a/doc/REMOTE_EXECUTION.md b/doc/REMOTE_EXECUTION.md index 1bd3c70..d63e9f5 100644 --- a/doc/REMOTE_EXECUTION.md +++ b/doc/REMOTE_EXECUTION.md @@ -148,7 +148,7 @@ printf 'cwd=/home/user/project\nremote=devbox\n' > $OLLIE/s/new # Via D-Bus dbus-send ... org.ollie.SessionManager.CreateSession \ string:"/home/user/project" string:"" string:"" string:"default" \ - string:"devbox" # additional arg or config key + string:"" string:"devbox" # remote host ``` Or as a session config key: diff --git a/httpgw b/httpgw index 35cf8ad..52208eb 160000 --- a/httpgw +++ b/httpgw @@ -1 +1 @@ -Subproject commit 35cf8ad0d922272b0b5750022c315ae346456c55 +Subproject commit 52208eb031f572628c6edca87c8e74471686575f diff --git a/kde b/kde index d223911..62663af 160000 --- a/kde +++ b/kde @@ -1 +1 @@ -Subproject commit d223911191573132242f1828ea4f97bcc6feff5a +Subproject commit 62663aff4e6d3686b5c3548bddc4538e5ec9ab94 diff --git a/prompts/session-dbus.md b/prompts/session-dbus.md index 8d7591b..48c6e23 100644 --- a/prompts/session-dbus.md +++ b/prompts/session-dbus.md @@ -35,7 +35,7 @@ All session operations go through the D-Bus interface. | Method | Args | Returns | Purpose | |---|---|---|---| -| `CreateSession` | cwd, backend, model, agent, system_prompt | session_id | Create a new session | +| `CreateSession` | cwd, backend, model, agent, system_prompt, remote | session_id | Create a new session | | `ListSessions` | — | array of "id\tstate\tmodel\tagent" | Discover sessions | | `KillSession` | session_id | bool | Terminate a session | | `RenameSession` | session_id, new_name | bool | Rename a session | @@ -119,7 +119,7 @@ dbus-send --session --dest=org.ollie.SessionManager --type=method_call --print-r # Create a session dbus-send --session --dest=org.ollie.SessionManager --type=method_call --print-reply \ /org/ollie/SessionManager org.ollie.SessionManager.CreateSession \ - string:"$PWD" string:"" string:"" string:"default" string:"" + string:"$PWD" string:"" string:"" string:"default" string:"" string:"" # Submit a prompt dbus-send --session --dest=org.ollie.SessionManager --type=method_call \ diff --git a/tools/route b/tools/route index 56f861e..26edcfc 100755 --- a/tools/route +++ b/tools/route @@ -84,7 +84,7 @@ id="${id:-route-${tier}-$(date +%s%N)-$$}" # Create session via D-Bus session_id=$(dbus_string CreateSession \ - string:"$(pwd)" string:"$BACKEND" string:"$MODEL" string:"$agent") + string:"$(pwd)" string:"$BACKEND" string:"$MODEL" string:"$agent" string:"" string:"") [ -z "$session_id" ] && die "failed to create session" diff --git a/tools/subagent_spawn b/tools/subagent_spawn index 79fc70e..b882fcd 100755 --- a/tools/subagent_spawn +++ b/tools/subagent_spawn @@ -65,7 +65,7 @@ $*" # Create session via D-Bus session_id=$(dbus_string CreateSession \ - string:"$cwd" string:"${backend:-}" string:"${model:-}" string:"$agent") + string:"$cwd" string:"${backend:-}" string:"${model:-}" string:"$agent" string:"" string:"") [ -z "$session_id" ] && die "failed to create session" diff --git a/webui b/webui index d9795f4..2be54a8 160000 --- a/webui +++ b/webui @@ -1 +1 @@ -Subproject commit d9795f46395846affe136eb1c16d02d42361c0e0 +Subproject commit 2be54a850b8909b5a6f64738cbcb4b11e99d8a69