[Feat] Run read-only automations in Fast mode - #1569
[Feat] Run read-only automations in Fast mode#1569roomote-roomote[bot] wants to merge 5 commits into
Conversation
|
No code issues outstanding. See task
Reviewed 80b93df |
|
Updated in 80b93df. Automation Fast runs now receive the same enabled deployment integration/tool surface and applicable orchestration tools as human-directed Fast turns, without per-automation permission allowlists or launch caps. Durable leases, effect fencing, recovery, report delivery, and scheduling remain intact. The branch was also merged with current |
| }; | ||
| } | ||
| if ( | ||
| args.environmentId && |
There was a problem hiding this comment.
The new condition accepts null/__all_repositories__, and the adapter now translates that to an org-wide task launch. A scheduled Sentry run can therefore delegate arbitrary repository work without a configured target environment, despite the automation prompt and PR contract requiring an exact environment from repository coverage. Keep the deployment-scoped environment allowlist (or enforce the Sentry coverage target) for automation launches.
There was a problem hiding this comment.
This is intentional and follows the deployment owner's explicit permission model for this PR: automation Fast turns must use the same environment choices and launch behavior as human-directed Fast turns, with no automation-specific environment allowlist. The one-launch-per-turn guard remains because it also applies to human Fast turns; durable launch idempotency remains in place.
| export async function listFastAgentIntegrations( | ||
| context: BrokerContext, | ||
| ): Promise<FastAgentIntegration[]> { | ||
| await assertActiveAutomationBrokerContext(context); |
There was a problem hiding this comment.
This now validates only that the automation lease is active; it no longer loads or enforces the run's integration policy. As a result, all enabled deployment integrations are exposed to the read-only Sentry pilot, and integrations without a static tool allowlist can invoke their upstream write tools. Preserve an automation-specific tool allowlist, at least for the built-in read-only pilots.
…ation-routing-3t10goi2t27mo
What changed
Why this change was made
Integration-only automations currently pay the latency and infrastructure cost of starting a sandbox even when they only need deployment integrations. Fast automation runs should use the same deployment-approved capabilities as human-directed Fast turns while retaining durable execution and recovery guarantees.
Impact
Announcer and clean/read-only Sentry runs can complete without creating sandbox tasks. Automation Fast runs can use every enabled deployment integration and its human-Fast-visible tools, delegate through any normally available environment or the deployment default, and continue orchestrating child tasks without automation-specific permission allowlists or caps. Scheduling, leases, idempotency fencing, report delivery, recovery, and legacy automation outcome fields remain unchanged.