Repository navigation
fix: enable native web tools in codeaf PlanDB tasks - #1783
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PlanDB task workers lost web access because their standalone session config omitted the search provider and fetcher. Bind the run profile’s live web pair in the crew factory, expose native
web_search/web_fetchalongside bash, and record each action’s tool name in its trajectory. Files and PlanDB coordination continue through bash, with one tool call per response.Reuse existing web backends, profile credentials, cancellation and output limits. The optional trajectory field preserves older records. Align both worker policies with available tools, and clarify that children inherit requirements they own without repeating their parent’s delegation instructions. Architecture and the user manual are updated in this PR.
Three review passes covered runtime wiring and logs, profile/cancellation behavior, and prompt/delegation behavior. Regression tests cover successful native actions, failed fetch observations, changing profile credentials, cancellation, conflicting action policy, and child scope.
Validation on
3deb957c698dabbea820af0017eb083923c5d780:make buildandmake pr-readypassed; all selected runner/session tests passed on their first run. All GitHub checks passed.TestTUIE2E/TaskOnTheRunEngineandTestTUIE2E/TaskOnTheDefaultBelt(81 seconds total).deepseek/deepseek-v4.1-flash, including auxiliary calls. No model substitution.Observed native actions in the final delegated task’s logs (the ask did not name tools):
web_searchweb_fetchEarlier live coverage additionally verified independent checker fetches and rejection of a two-fetch batch followed by successful single-call retries. Native successful-action counts exclude rejected calls. Terminal tests ran against the actual application; no recording is attached.