Skip to content

feat(parser): add agent type to logging and define AGENT_TYPE constan…#12

Merged
SebastienWae merged 3 commits intomainfrom
fix-logging-agent-name
Jan 14, 2026
Merged

feat(parser): add agent type to logging and define AGENT_TYPE constan…#12
SebastienWae merged 3 commits intomainfrom
fix-logging-agent-name

Conversation

@SebastienWae
Copy link
Copy Markdown
Owner

…t in parser modules

Copilot AI review requested due to automatic review settings January 14, 2026 18:49
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request adds agent type identification to logging messages and establishes a standardized approach for defining agent types in parser modules.

Changes:

  • Defined AGENT_TYPE module-level constants in all parser modules (pi.py, opencode.py, claude_code.py)
  • Added AGENT_TYPE attribute requirement to the Parser Protocol
  • Enhanced logging to include agent type in session export and filtering messages

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
sessionsync/parsers/base.py Added AGENT_TYPE attribute to Parser Protocol and imported AgentType for type annotations
sessionsync/parsers/pi.py Defined AGENT_TYPE constant and refactored Session creation to use it
sessionsync/parsers/opencode.py Defined AGENT_TYPE constant and refactored Session creation to use it
sessionsync/parsers/claude_code.py Defined AGENT_TYPE constant and refactored Session creation to use it
sessionsync/cli.py Enhanced logging to include agent type in session export and filtering messages

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@SebastienWae SebastienWae merged commit 189c3b0 into main Jan 14, 2026
6 checks passed
@SebastienWae SebastienWae deleted the fix-logging-agent-name branch January 14, 2026 18:56
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