Skip to content

Commit b88b4e3

Browse files
committed
fix(agent,tui): update tests off removed tools.CoreTools/NewWriteFileTool/NewLSPNavigateTool wrappers
Those were thin unscoped wrappers around the Scoped variants, deleted upstream in #706 since nothing else called them directly. Only these tests still did; switch to the Scoped calls main's own tests already use.
1 parent 1430bf1 commit b88b4e3

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

‎internal/agent/loop_test.go‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3394,7 +3394,7 @@ func TestSpecDraftDeniesBashToolCalls(t *testing.T) {
33943394
func TestPlanModeAdvertisesOnlySafeTools(t *testing.T) {
33953395
root := t.TempDir()
33963396
registry := tools.NewRegistry()
3397-
for _, tool := range tools.CoreTools(root) {
3397+
for _, tool := range tools.CoreToolsScoped(root, nil) {
33983398
registry.Register(tool)
33993399
}
34003400
provider := &mockProvider{
@@ -3585,7 +3585,7 @@ func TestPlanModeRejectsNameOnlySpoofedControlTools(t *testing.T) {
35853585
func TestPlanModeDeniesLSPNavigateToolCalls(t *testing.T) {
35863586
root := t.TempDir()
35873587
registry := tools.NewRegistry()
3588-
registry.Register(tools.NewLSPNavigateTool(root))
3588+
registry.Register(tools.NewScopedLSPNavigateTool(root, nil))
35893589
provider := &mockProvider{
35903590
turns: [][]zeroruntime.StreamEvent{
35913591
{
@@ -3627,7 +3627,7 @@ func TestPlanModeDeniesLSPNavigateToolCalls(t *testing.T) {
36273627
func TestPlanModeDeniesHiddenToolCalls(t *testing.T) {
36283628
root := t.TempDir()
36293629
registry := tools.NewRegistry()
3630-
registry.Register(tools.NewWriteFileTool(root))
3630+
registry.Register(tools.NewScopedWriteFileTool(root, nil))
36313631
provider := providerCallingWriteFileThenAnswer("done")
36323632

36333633
result, err := Run(context.Background(), "plan", provider, Options{

‎internal/tui/plan_mode_test.go‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -238,7 +238,7 @@ func TestPlanModeCommandGuardDoesNotBlockOutsideMode(t *testing.T) {
238238

239239
func newPlanModeTestModel(root string, provider zeroruntime.Provider) model {
240240
registry := tools.NewRegistry()
241-
for _, tool := range tools.CoreTools(root) {
241+
for _, tool := range tools.CoreToolsScoped(root, nil) {
242242
registry.Register(tool)
243243
}
244244
return newModel(context.Background(), Options{

0 commit comments

Comments
 (0)