Skip to content

fix: disable remote deployment and stabilize task cleanup - #1907

Merged
4pmtong merged 2 commits into
mainfrom
fix/task-finalization-safety
Sep 4, 2026
Merged

fix: disable remote deployment and stabilize task cleanup#1907
4pmtong merged 2 commits into
mainfrom
fix/task-finalization-safety

Conversation

@4pmtong

@4pmtong 4pmtong commented Sep 4, 2026

Copy link
Copy Markdown
Collaborator

Pull Request

Related Issue

No linked Issue. This bug was found internally while investigating generated web tasks that did not terminate reliably.

Description

This is the release-safe subset of #1906. It addresses the two direct causes found in the reproduced task while deliberately excluding the broader SSE, follow-up, background-trigger, Project eviction, and RunCoordinator lifecycle changes.

  • Removes Web Deploy Toolkit from the Workforce Developer Agent and Single Agent assembly because Eigent does not need to upload generated content to the legacy remote deployment service.
  • Ignores legacy toolkit configuration that still requests web_deploy.enabled=true.
  • Removes deployment claims from the Developer Agent prompt, Workforce coordinator description, and workflow capability lists.
  • Keeps web_deploy_toolkit.py and historical rendering support intact for possible future use and compatibility.
  • Increases the terminal background-cleanup checkpoint budget from 5 seconds to 30 seconds so a stopped preview server can release its write lease and finish the workspace Git checkpoint before Run finalization.

Testing Evidence (REQUIRED)

Automated validation completed through commit c134cab4f:

  • Focused backend tests for Developer Agent assembly, toolkit descriptions, terminal workspace cleanup, and Workforce construction — 20 passed.
  • npm run type-check — passed.
  • Focused ESLint and Prettier checks — passed.
  • Focused Ruff lint and Ruff format checks — passed.
  • npm run check:design-tokens — passed.
  • git diff --check — passed.

Human UI evidence has not yet been attached. The generated-web-task scenario will be retested against this branch before merge.

  • I have included human-verified testing evidence in this PR.
  • This PR includes frontend/UI changes, and I attached screenshot(s) or screen recording(s).
  • No frontend/UI changes in this PR.

What is the purpose of this pull request?

  • Bug fix
  • New Feature
  • Documentation update
  • Other

Contribution Guidelines Acknowledgement

Pending confirmation by the human contributor; this is intentionally not checked on their behalf.

@4pmtong
4pmtong merged commit 0c9440a into main Sep 4, 2026
9 checks passed
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