Skip to content

[Fix] Stop Fast from adding unrequested no-PR constraints to task briefs - #3110

Merged
mrubens merged 1 commit into
developfrom
fix/fast-delegation-delivery-policy
Sep 22, 2026
Merged

mrubens merged 1 commit into
developfrom
fix/fast-delegation-delivery-policy

Conversation

@mrubens

@mrubens mrubens commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

Summary

When a user asks a Session for a code change without saying anything about delivery, the Fast orchestrator sometimes appends its own "Do not create a PR or commit unless asked" constraint to the delegated task brief. The coding task obeys, finishes with local-only changes, and the user has to come back and ask for a PR separately. Nothing in the Fast prompt told the orchestrator what delivery policy to carry, so it invented one, overriding the standard workflow's default draft-PR delivery.

This adds one orchestration rule: carry the user's delivery instruction into the task brief only when they actually gave one, and otherwise leave delivery to the task's default workflow. It explicitly forbids inferring a no-commit/no-push/no-PR constraint from a detailed spec, listed constraints, requested tests, or a request to report results.

Changes

  • packages/cloud-agents/src/server/fast-agent/fast-agent-prompt.ts: new Orchestration Policy bullet after the launch_task delegation rule.
  • packages/cloud-agents/src/server/fast-agent/__tests__/fast-agent-prompt.test.ts: prompt-assembly assertion for the new rule.

Validation

  • vitest run src/server/fast-agent/__tests__/fast-agent-prompt.test.ts in @roomote/cloud-agents: 125 passed.
  • oxfmt on both files.

@roomote-community

roomote-community Bot commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

No code issues found. See task

Reviewed f49619f

@mrubens
mrubens marked this pull request as ready for review September 22, 2026 12:00
@mrubens
mrubens merged commit 72955cc into develop Sep 22, 2026
17 checks passed
@mrubens
mrubens deleted the fix/fast-delegation-delivery-policy branch September 22, 2026 12:01
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.

1 participant