Skip to content

Feat: janus auto capture#1

Merged
hoainho merged 4 commits into
janusfrom
feature/janus-auto-capture
Jul 11, 2026
Merged

Feat: janus auto capture#1
hoainho merged 4 commits into
janusfrom
feature/janus-auto-capture

Conversation

@hoainho

@hoainho hoainho commented Jul 11, 2026

Copy link
Copy Markdown
Owner

No description provided.

hoainho added 2 commits July 10, 2026 23:51
Adds Claude Code SessionEnd hook (scripts/hooks/claude-autocapture.py) that
records token/duration/files/ticket per session, a 'query coverage' command,
and a GCR insufficient-data guard (grey when <10 stories).
@hoainho hoainho self-assigned this Jul 11, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request introduces an automated session capture hook for Claude Code (claude-autocapture.py) and adds a new coverage query command to harness-cli to track adoption and instrumentation metrics. Feedback on the changes highlights a potential TypeMismatch error in the Rust SQLite query when retrieving token sums, and suggests using Python's built-in sqlite3 module instead of calling the external CLI to avoid SQL injection risks and noisy errors. Additionally, improvements were suggested to handle non-dictionary message objects defensively and optimize transcript scanning performance on large files.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread scripts/hooks/claude-autocapture.py
Comment thread scripts/hooks/claude-autocapture.py Outdated
Comment thread scripts/hooks/claude-autocapture.py
Comment thread scripts/hooks/claude-autocapture.py
Comment thread scripts/hooks/claude-autocapture.py Outdated
Comment thread crates/harness-cli/src/infrastructure.rs Outdated
hoainho added 2 commits July 11, 2026 16:20
Built-in sqlite3 (parameterized, no CLI dep) for already_captured + new
story_exists guard; defensive non-dict message check; drop per-line upper();
CAST SUM to INTEGER for rusqlite type safety.
@hoainho
hoainho merged commit 8972b26 into janus Jul 11, 2026
5 of 6 checks passed
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.

1 participant