Skip to content

Opencode support - #85

Open
jbendotnet wants to merge 2 commits into
warpdotdev:mainfrom
jbendotnet:opencode-support
Open

Opencode support#85
jbendotnet wants to merge 2 commits into
warpdotdev:mainfrom
jbendotnet:opencode-support

Conversation

@jbendotnet

@jbendotnet jbendotnet commented Aug 29, 2026

Copy link
Copy Markdown

Closes #86

Adds OpenCode support to skill-doctor, allowing it to grade local OpenCode conversations instead of stopping at harness validation.

The collector reads OpenCode SQLite databases locally and in read-only mode. It prefers the active message/part representation and falls back to session_message per session. Transcript data remains local.

Review guide

Start with the OpenCode decoder in collect_sessions.py. Focus on schema selection, WAL-safe database access, and transcript normalization. The tests cover both schemas and their precedence.

What changed

  • Added OpenCode harness detection and CLI options.
  • Added default and explicit database discovery.
  • Normalized messages, tool calls, errors, code edits, skill usage, and child sessions.
  • Added bounded diagnostics for malformed data and unusable database paths.
  • Added fixtures for schema precedence, fallback behavior, filtering, statistics, and live WAL reads.

Validation

  • python3 -m unittest test_collect_sessions.py test_render_report.py — 23 tests passed.
  • Python 3.9 test suite — 23 tests passed.
  • py_compile passed on both Python versions.
  • git diff --check passed.
  • A read-only smoke test against OpenCode v1.18.25 found and sampled two scoreable repository sessions.

@warp-for-oss

warp-for-oss Bot commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

@jbendotnet

Every PR must be linked to a same-repo issue before Oz can review it.

This PR is linked to #86, but no linked issue is marked ready-to-implement yet. Only repository maintainers apply that label, so please wait for a maintainer to mark the issue. Once it is marked, push a new commit or comment /warp-agent-review to re-trigger review.

Powered by Oz

@warp-for-oss warp-for-oss Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jbendotnet

Every PR must be linked to a same-repo issue before Oz can review it.

Next step: open or find a same-repo issue describing this change, then link it to this PR by adding Closes #123 to the PR description (or using the "Development" sidebar on GitHub). A maintainer will mark the issue ready-to-implement when it is ready. Once it is marked, comment /warp-agent-review to re-trigger review.

Powered by Oz

@warp-factories

Copy link
Copy Markdown

Warp has completed this task.

@jbendotnet

Copy link
Copy Markdown
Author

/warp-agent-review

@warp-for-oss warp-for-oss Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jbendotnet

Every PR must be linked to a same-repo issue before Oz can review it.

This PR is linked to #86, but no linked issue is marked ready-to-implement yet. Only repository maintainers apply that label, so please wait for a maintainer to mark the issue. Once it is marked, push a new commit or comment /warp-agent-review to re-trigger review.

Powered by Oz

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.

Add support for OpenCode harness to skill-doctor

1 participant