refactor(review): consolidate repo context into single sandbox tool - #7
Conversation
Remove read_repo_file, list_repo_dir, search_repo tools and the per-pass call budget system. All repository exploration now flows through run_repo_command against a persistent sandbox checkout. Also rewrite the instruction to guide evidence-based investigation: concrete questions → targeted searches → callers/tests → iterate.
Mimir review — 🟢 No blocking issuesClean refactor that consolidates all repository exploration behind the persistent sandbox checkout, removing three GitHub API-based tools ( Findings: 0 critical · 0 major · 0 minor · 0 nit 💰 Review cost: $0.0166 — primary |
Remove
read_repo_file,list_repo_dir,search_repotools and the per-pass call budget system. All repository exploration now flows throughrun_repo_commandagainst a persistent sandbox checkout.Also rewrite the instruction to guide evidence-based investigation: concrete questions → targeted searches → callers/tests → iterate.
Changes
read_repo_file,list_repo_dir,search_repo,resolveToolBudget,isSafeRepoPath,guardedExecute;repoContextToolsnow returns onlyrun_repo_command(+dependency_review)## Repository investigationsection with numbered workflow stepsfileCountparameter fromrepoContextToolscallsREPO_TOOL_CALL_BUDGET/REPO_TOOL_CALL_BUDGET_MAXenv varsMotivation
After switching to persistent sandbox checkout, the GitHub API file/list/search tools caused reviewers to clone once then abandon the checkout for slower, fragmented API reads. Single tool + good instructions > four tools with budget system.