Skip to content

fix(opencode): import actual session messages, not just metadata#63

Merged
luca-ctx merged 1 commit into
ctxrs:mainfrom
robbe1912:fix/opencode-message-import
Jul 6, 2026
Merged

fix(opencode): import actual session messages, not just metadata#63
luca-ctx merged 1 commit into
ctxrs:mainfrom
robbe1912:fix/opencode-message-import

Conversation

@robbe1912

@robbe1912 robbe1912 commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Summary

  • \opencode_session_messages()\ short-circuits on \session_message\ having any rows, but in current OpenCode schema this table only stores agent-switched/model-switched metadata
  • Actual conversation lives in \message\ table with \data.role\ = user/assistant — never reached
  • Fix: guard the \session_message\ path by checking for known conversation entry types before accepting rows; fall through to \message\ table when only metadata exists

Validation

  • \cargo build --release\ (Rust 1.96.1, Windows)
  • Re-imported: \ctx import --provider opencode --reset-cursor\
  • Before: 112 notice events, 65 sessions
  • After: 53678 message events (45521 assistant + 8157 user), 1511 sessions
  • \ctx search\ returns real message content

open_code_session_messages() short-circuited on session_message having
any rows. In OpenCode's current schema, session_message only stores
agent-switched/model-switched metadata, while actual conversation lives
in the message table with data.role=user/assistant.

Fix: guard the session_message path by checking for known conversation
entry types. If only metadata rows exist, fall through to message table.

Before: 112 notice events, 65 sessions
After:  53678 message events, 1511 sessions
@robbe1912 robbe1912 force-pushed the fix/opencode-message-import branch from a75783b to 07f45be Compare July 6, 2026 11:29
@robbe1912 robbe1912 marked this pull request as draft July 6, 2026 12:35
@robbe1912 robbe1912 force-pushed the fix/opencode-message-import branch from 5c8dbb9 to 07f45be Compare July 6, 2026 14:05
@robbe1912 robbe1912 marked this pull request as ready for review July 6, 2026 14:07
@luca-ctx luca-ctx merged commit 7406f9e into ctxrs:main Jul 6, 2026
@luca-ctx

luca-ctx commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Thanks for this fix!

@robbe1912 robbe1912 deleted the fix/opencode-message-import branch July 6, 2026 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants