Skip to content

Auto-trust cwd on thread start#16492

Merged
aibrahim-oai merged 2 commits intomainfrom
dev/auto-trust-thread-start
Apr 3, 2026
Merged

Auto-trust cwd on thread start#16492
aibrahim-oai merged 2 commits intomainfrom
dev/auto-trust-thread-start

Conversation

@aibrahim-oai
Copy link
Copy Markdown
Collaborator

@aibrahim-oai aibrahim-oai commented Apr 2, 2026

  • Persist trusted cwd state during thread/start when the resolved sandbox is elevated.
  • Add app-server coverage for trusted root resolution and confirm turn/start does not mutate trust.

aibrahim-oai added a commit that referenced this pull request Apr 2, 2026
@aibrahim-oai aibrahim-oai changed the title Auto-trust projects on thread start Auto-trust cwd on thread start Apr 2, 2026
listener_task_context
.outgoing
.send_error(request_id, error)
.await;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

nit: you can do self.send_internal_error(request_id, format!("failed to persist trusted project state: {err}")).await;

@aibrahim-oai aibrahim-oai force-pushed the dev/auto-trust-thread-start branch from d8bb201 to 917e336 Compare April 2, 2026 19:33
@aibrahim-oai aibrahim-oai enabled auto-merge (squash) April 2, 2026 19:33
@aibrahim-oai aibrahim-oai merged commit 95e809c into main Apr 3, 2026
29 of 30 checks passed
@aibrahim-oai aibrahim-oai deleted the dev/auto-trust-thread-start branch April 3, 2026 00:02
@github-actions github-actions bot locked and limited conversation to collaborators Apr 3, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants