Skip to content

Drop unused agent_id kwarg from AgentConfigUpdate#6090

Open
longcw wants to merge 1 commit into
mainfrom
longc/drop-agentconfigupdate-agent-id
Open

Drop unused agent_id kwarg from AgentConfigUpdate#6090
longcw wants to merge 1 commit into
mainfrom
longc/drop-agentconfigupdate-agent-id

Conversation

@longcw

@longcw longcw commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

AgentConfigUpdate has no agent_id field, so passing agent_id=self._agent.id was silently dropped by pydantic (default extra="ignore") and never reached the model, the proto serializer in remote_session.py, or the wire protocol. Nothing reads .agent_id off a config-update item anywhere in the codebase, so the kwarg was pure dead code. This removes it from the three call sites in agent_activity.py.

AgentConfigUpdate has no agent_id field, so the kwarg was silently
dropped by pydantic (extra="ignore") and never reached the model, the
serializer, or the wire protocol. Nothing reads it back anywhere.
@longcw longcw requested a review from a team as a code owner June 13, 2026 02:29

@devin-ai-integration devin-ai-integration 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.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

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