Skip to content

feat(autofix): Add optional user param to trigger_autofix_agent#119446

Merged
Zylphrex merged 2 commits into
masterfrom
txiao/autofix-agent-user-param
Jul 13, 2026
Merged

feat(autofix): Add optional user param to trigger_autofix_agent#119446
Zylphrex merged 2 commits into
masterfrom
txiao/autofix-agent-user-param

Conversation

@Zylphrex

Copy link
Copy Markdown
Member

Plumb an optional user argument through trigger_autofix_agent and
get_autofix_agent_client to SeerAgentClient, which already accepts it for
viewer context, permission checks, and feature flag evaluation.

Callers with an authenticated user (the API endpoint and the Slack operator) now
forward it. Callers without a user (on_completion_hook, issue_summary,
pr_iteration, night shift) keep the existing None default.

Plumb an optional user argument through trigger_autofix_agent and
get_autofix_agent_client to SeerAgentClient, which already accepts it
for viewer context, permission checks, and feature flag evaluation.

Callers with an authenticated user (the API endpoint and the Slack
operator) now forward it. Callers without a user (on_completion_hook,
issue_summary, pr_iteration, night shift) keep the existing None
default.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Jul 10, 2026
@Zylphrex Zylphrex marked this pull request as ready for review July 10, 2026 22:44
@Zylphrex Zylphrex requested a review from a team as a code owner July 10, 2026 22:44
…fix_agent

The endpoint now passes request.user to trigger_autofix_agent. Update
test_stopping_point and test_insert_index_passed_through to include user=ANY
in their assert_called_once_with calls.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@Zylphrex Zylphrex merged commit 29a456b into master Jul 13, 2026
67 checks passed
@Zylphrex Zylphrex deleted the txiao/autofix-agent-user-param branch July 13, 2026 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants