Open
Conversation
Add a pluggable agent registry and implement the Codex CLI detector alongside the existing Claude Code detector. The hooks now resolve the detector from config rather than hardcoding Claude Code. - internal/agent/registry.go: agent name → detector factory registry - internal/agent/codex/: Codex CLI detector (process detection, session dir) - internal/agent/claude/register.go: register claude-code detector - Update pre-commit and post-commit hooks to use registry with fallback - Fix flaky precommit_test timing (5ms → 50ms sleep) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Add structured debug and warn logging in the post-commit hook to help users diagnose why commits may not receive Partio-Checkpoint trailers. - Warn-level logging at each early return explaining why no trailer was added (agent inactive, commit already processed, session condensed) - Debug-level logging of commit file paths and session info for diagnosing path mismatch issues (visible with PARTIO_LOG_LEVEL=debug) - Add git.DiffNameOnly helper to extract changed file paths from commits Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
jcleira
changed the title
## Objective Update all workflows to use minions v0.0.5. ## Why v0.0.5 fixes the root cause of all "no changes" failures — Claude was unable to write files because bypassPermissions mode was not set. ## How Updated go install version tag in all workflow files. Partio-Checkpoint: 0ecbe84f5950 Partio-Attribution: 100% agent
Upgrade early-exit log messages in the post-commit hook from Debug to Warn level so users running PARTIO_LOG_LEVEL=debug (or default) can immediately see why no Partio-Checkpoint trailer was added to a commit. Each exit path now emits a structured "post-commit: no checkpoint created" warning with the specific reason (no pre-commit state, no active agent, commit already processed, session already condensed) and relevant context (commit hash, session ID, state file path). Also adds a Debug-level log after session lookup showing the resolved session path, helping diagnose path mismatch issues. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Allow overriding the agent name via PARTIO_AGENT env var, completing the documented environment variable support in CLAUDE.md. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
jcleira
changed the title
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.
Summary
codex) as a supported AI agent alongside Claude Codeinternal/agent/codex/package with process detection, session directory lookup, andDetectorimplementationagent.Register/agent.NewDetector) so hooks select the detector based on theagentconfig settingPARTIO_AGENTenvironment variable support for overriding the agent nameWhy
Codex CLI is a widely used AI coding agent. Without this change, Codex-assisted commits are attributed as 100% human work and no session context is preserved. The
agent.Detectorinterface was already designed to be pluggable — this PR delivers on that design.Test plan
go test ./...passesgolangci-lint runpassesName,IsRunning,FindSessionDir)NewDetectorwith registered and unknown agents)PARTIO_AGENT=codexand verifypartio statusreports codex🤖 Generated with Claude Code