chore(bots): bump engine pin to 5368de77 (can_use_tool PermissionResult fix)#871
Merged
Merged
Conversation
…lt fix) Moves the engine pin (bot-prelude's single engine-ref default) from 3569dae3 to 5368de77, which includes databricks/databricks-bot-engine#122 — the fix for the can_use_tool permission callback returning a dict instead of a PermissionResult object. Without it, every edit_file/bash call the engineer bot makes fails with `TypeError: Tool permission callback must return PermissionResult ... got dict`, so the bot can read but not edit or run tests (observed on #791: two runs hung to the 45-min job timeout). The fix is runtime-verified against the pinned SDK (0.2.102). Signed-off-by: eric-wang-1990 <[email protected]> Co-authored-by: Isaac
eric-wang-1990
had a problem deploying
to
azure-prod
July 17, 2026 18:31 — with 
GitHub Actions
Failure
GitHub Actions
Failure
eric-wang-1990
temporarily deployed
to
azure-prod
July 17, 2026 18:31 — with
GitHub Actions
Inactive
GitHub Actions
Inactive
![peco-review-bot[bot]](https://avatars.githubusercontent.com/in/3801177?s=60&v=4){kind=small}
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.
Bumps the engine pin (bot-prelude's single
engine-refdefault)3569dae3→5368de77, picking up databricks/databricks-bot-engine#122.Why
#122 fixes the
can_use_toolpermission callback: it returned a plain dict, butclaude-agent-sdkrequires aPermissionResultobject, so everyedit_file/bashcall failed withTypeError: Tool permission callback must return PermissionResult ... got dict. The engineer bot could read but not edit or run tests — on #791, two runs hung to the 45-min job timeout. The fix is runtime-verified against the pinned SDK (0.2.102): the old dict is rejected by the SDK's own validation, the new typed objects are accepted.After merge
Re-triggering the engineer-bot on #791 is the end-to-end confirmation that the bot can now edit + run its
tests/e2erepro.This pull request and its description were written by Isaac.