sessions: folder-scoped list hides sessions when a worktree owns duplicate project rows #59371

Activity

1. opencode-agent commented on Oct 26, 2026

   

   opencode-agentBot

   on Oct 26, 2026 – with OpenCode Agent

   Contributor

   More actions

   Thanks for the detailed report. I was able to reproduce the main problem (part A) on 2.1.4, and it still happens on the latest dev (0f3a0c206).

   I used one way of getting a second project row for the same worktree, adding a git remote (the trigger from #57985):

   1. git init a fresh repo with an empty root commit, then create a session with opencode api session.create. opencode session list shows it.
   2. git remote add origin …. The same session list in that directory now returns nothing.
   3. Create another session. Only the new one is listed.

   The database then has two project rows with the same worktree. Both sessions have the same directory but different project_id, and the folder-scoped list only returns sessions from one of them. The script below reproduces it.

   I did not reproduce your exact trigger (a parent folder becoming a git repo), the different results before and after the location is evicted, or parts B and C.

   Related issues:

   • project: adding a git remote changes the project ID and hides every earlier session #57985: adding a git remote changes the project ID and hides earlier sessions (same mechanism; PR fix(core): move sessions to the remote-derived project id when a remote is added later #58668 is open for it)
   • migrate-v1: non-git sessions in "global" project hidden after V1→V2 upgrade #49827: V1 global-project sessions hidden from folder lists (your part C)
   • project: Windows path case sensitivity silently splits project identity (empty session list, duplicated MCP/plugin init) #52501: Windows path casing splits project identity

   Reproduction script (repro.sh)

   #!/usr/bin/env bash
   # Repro: same worktree gets two `project` rows; directory-scoped session list hides the older cell's sessions.
   set -u
   q() { DB=$(ls -S "$XDG_DATA_HOME"/opencode/*.db | head -1) \
    bun -e "import {Database} from \"bun:sqlite\"; console.table(new Database(process.env.DB).query(process.argv[1]).all())" "$1"; }
   OC=${OC:-opencode}
   export XDG_DATA_HOME=$(mktemp -d) XDG_CONFIG_HOME=$(mktemp -d) XDG_STATE_HOME=$(mktemp -d) XDG_CACHE_HOME=$(mktemp -d)
   D=$(mktemp -d)/repo; mkdir -p "$D"; cd "$D"
   # Optional: SERVE=1 starts a private `serve` per step (for running from source, where --standalone can't respawn itself)
   export OPENCODE_PASSWORD=repro-pw
   run() {
     if [ -n "${SERVE:-}" ]; then
       $OC serve --port 45991 >/dev/null 2>&1 & local pid=$!
       for i in $(seq 60); do curl -sf http://127.0.0.1:45991/api/health >/dev/null 2>&1 && break; curl -s -o /dev/null http://127.0.0.1:45991/ && break; sleep 0.5; done
       $OC "$@" --server http://127.0.0.1:45991; kill $pid; wait $pid 2>/dev/null
     else $OC "$@" --standalone; fi
   }
   git init -q && git -c user.email=a@b -c user.name=a commit -q --allow-empty -m root
   echo "== create session before remote"
   run api session.create --data '{"title":"before remote"}' >/dev/null
   echo "== list (before remote):"; run session list 2>&1
   git remote add origin [email protected]:example/repro.git
   echo "== list after remote add:"; run session list 2>&1
   run api session.create --data '{"title":"after remote"}' >/dev/null
   echo "== list after second session:"; run session list 2>&1
   echo "== project rows:"; q "select id, worktree from project"
   echo "== session rows:"; q "select id, project_id, directory, title from session_v2"

   Run it: bash repro.sh
2. opencode-agent

   added

   reproduced

   on Oct 26, 2026
3. pathexplorer commented on Oct 26, 2026

   

   pathexplorer

   on Oct 26, 2026

   Author

   More actions

   Repro for Part B (location ignores caller's cwd)

   Confirmed in a fresh isolated environment (XDG_* + opencode serve --port 45991):

   $ cd /tmp/opencode-repro/proj1 && opencode api get /api/location
   {"directory":"/tmp/opencode-repro","project":{"id":"b8d4acb1…","canonical":"/tmp/opencode-repro"}}
   
   $ cd /tmp/opencode-repro/proj2 && opencode api get /api/location
   {"directory":"/tmp/opencode-repro","project":{"id":"b8d4acb1…","canonical":"/tmp/opencode-repro"}}
   
   $ cd /tmp/opencode-repro/proj3 && opencode api get /api/location
   {"directory":"/tmp/opencode-repro","project":{"id":"b8d4acb1…","canonical":"/tmp/opencode-repro"}}
   

   All three calls return the directory where the service was started (/tmp/opencode-repro), not the caller's cwd.

   Explicit param works:

   $ opencode api get '/api/location?location[directory]=/tmp/…/proj3'
   {"directory":"/tmp/…/proj3","project":{"id":"a1878eff…","canonical":"/tmp/…/proj3"}}
   

   /api/debug/location proves the other dirs are loaded:

   [{"directory":"/tmp/opencode-repro"},{"directory":"/tmp/…/proj3"}]

   Repro script (repro-B.sh):

   #!/usr/bin/env bash
   set -euo pipefail
   export XDG_DATA_HOME=$(mktemp -d) XDG_CONFIG_HOME=$(mktemp -d) \
          XDG_STATE_HOME=$(mktemp -d) XDG_CACHE_HOME=$(mktemp -d)
   export OPENCODE_PASSWORD=repro-pw
   OC=${OC:-opencode}
   $OC serve --port 45991 >/dev/null 2>&1 & pid=$!
   trap 'kill $pid; wait $pid 2>/dev/null' EXIT
   for i in $(seq 60); do curl -sf http://127.0.0.1:45991/api/health >/dev/null 2>&1 && break; sleep 0.2; done
   api() { $OC api "$@" --server http://127.0.0.1:45991; }
   for d in proj1 proj2 proj3; do
     D=$(mktemp -d)/$d; mkdir -p "$D"; git -C "$D" init -q; git -C "$D" commit -q --allow-empty -m root
     echo "=== from $D"
     (cd "$D" && api get /api/location)
   done
   echo "=== explicit"
   api get '/api/location?location[directory]='"$D"
   api get /api/debug/location

   This is independent of the duplicate-project-row issue (Part A) — it reproduces with a single project per directory and no sessions created at all.
4. argszero commented on Oct 27, 2026

   

   argszero

   on Oct 27, 2026

   Contributor

   More actions

   I'd like to work on this.

   I proved part A at the store layer without needing the TUI. In packages/core/test/session-store.test.ts, with two project rows sharing one worktree (proj_old, proj_new) plus one row on a different worktree, and one session filed under each, SessionStore.list({ project: proj_new }) returns only ses_new on current dev (bc175f675c) — the sibling row's session is missing. The store filters on project_id alone:

   if ("project" in input) conditions.push(eq(SessionTable.project_id, input.project))

   That is the whole of part A: Session.create pins projectID to whichever row Project.resolve returned at creation time, so once a worktree owns two rows — the git-remote fingerprint change from #57985 — a folder-scoped list can only ever see the subset filed under the row it resolved to.

   My intended fix, kept to the located site: a project-scoped list should cover every project row that shares the worktree, since those rows are the same folder to the caller. That leaves the API, the cursor semantics and the session rows untouched, and it also repairs existing sessions, which a change that only stops new duplicates would not.

   Scope: part A only. Part B (/api/location returning the service's start directory instead of the caller's cwd) looks like a separate defect in the location middleware, and I would rather not fold it into the same change.
5. argszero

   added a commit that references this issue on Oct 29, 2026

   fix(core): list sessions from every project row of a worktree

   ...

   ce2bbac