Conversation
Closed
…lias + doctor sandbox detection
- post_tool: add stdout/stderr fallback for Claude Code tool_response format
(Claude Code sends {stdout,stderr} instead of {content})
- dispatcher + session_start: accept both camelCase and snake_case field names
via serde alias (hookEventName / hook_event_name)
- doctor: replace metadata-only check with actual write test to detect
Claude Code sandbox restrictions on ~/.omni/
- sqlite: add Store::test_write() for DB write capability check
- Added 5 new tests for Claude Code payload format
Fixes #GH-issue: omni stats shows no data with Claude Code
fajarhide
force-pushed
the
53-omni-stats-shows-no-data
branch
from
96480e1 to
994d8c6
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR Auto Describe
Summary
Key Changes
omni doctorto validate actual filesystem/database write access instead of static permission checks, with actionable Claude Code sandbox warningsDetailed Breakdown
src/cli/doctor.rs:.write_testfile)Store::test_write()method, with tailored warnings for Claude Code userssrc/hooks/dispatcher.rs+src/hooks/session_start.rs: Added serde deserialization aliases to support both camelCase (e.g.hookEventName) and snake_case (e.g.hook_event_name) field namessrc/hooks/post_tool.rs:ToolResponsestruct to acceptstdoutandstderrfieldsextract_tool_content()that prioritizes structuredcontentfields (used by Cursor/Windsurf) then falls back to concatenated stdout/stderr (Claude Code format)src/store/sqlite.rs: Addedtest_write()helper that validates database writability by creating and dropping a temporary test table, handling poisoned mutexes gracefullyNotes
All
omni doctorwarnings now include specific instructions for Claude Code users: add~/.omnitosandbox.filesystem.allowWritein~/.claude/settings.jsonto resolve sandboxed write blocks.Breaking Changes
None. All changes are backwards-compatible, with existing tooling and workflows unaffected.
Last updated: 2026-04-10 04:09:00