remove fetch for now

Author: ezynda3

cmd/root.go

@@ -22,7 +22,6 @@ var availableModules = []string{
 	"console",
 	"fs",
 	"http",
-	"fetch",
 	"timers",
 	"process",
 	"require",
@@ -33,7 +32,7 @@ var rootCmd = &cobra.Command{
 	Use:   "codebench-mcp",
 	Short: "JavaScript Executor MCP Server",
 	Long: `A Model Context Protocol (MCP) server that provides JavaScript execution capabilities 
-with a Node.js-like environment including console, fs, http, fetch, timers, process, and require modules.`,
+with a Node.js-like environment including console, fs, http, timers, process, and require modules.`,
 	Run: func(cmd *cobra.Command, args []string) {
 		// Validate module configuration
 		if len(enabledModules) > 0 && len(disabledModules) > 0 {

codebench-mcp

@@ -15,15 +15,14 @@ func TestBuildToolDescription(t *testing.T) {
 	}{
 		{
 			name:           "All modules enabled",
-			enabledModules: []string{"console", "fs", "http", "fetch", "timers", "process", "require"},
+			enabledModules: []string{"console", "fs", "http", "timers", "process", "require"},
 			expectedContent: []string{
 				"simplified JavaScript VM (goja)",
 				"NOT a full Node.js environment",
 				"📦 Available modules:",
 				"• console: Console logging",
 				"• fs: File system operations",
 				"• http: HTTP server creation",
-				"• fetch: HTTP client requests",
 				"• timers: Timer functions",
 				"• process: Process information",
 				"• require: Module loading system",
@@ -44,7 +43,6 @@ func TestBuildToolDescription(t *testing.T) {
 			notExpected: []string{
 				"• fs:",
 				"• http:",
-				"• fetch:",
 				"• process:",
 				"• require:",
 			},
@@ -64,17 +62,16 @@ func TestBuildToolDescription(t *testing.T) {
 			},
 		},
 		{
-			name:           "Only fetch enabled",
-			enabledModules: []string{"fetch"},
+			name:           "Only timers enabled",
+			enabledModules: []string{"timers"},
 			expectedContent: []string{
 				"📦 Available modules:",
-				"• fetch: HTTP client requests (fetch API with Promise support for GET/POST/etc)",
+				"• timers: Timer functions",
 			},
 			notExpected: []string{
 				"• console:",
 				"• fs:",
 				"• http:",
-				"• timers:",
 				"• process:",
 				"• require:",
 			},
@@ -101,7 +98,7 @@ func TestBuildToolDescription(t *testing.T) {
 func TestToolDescriptionDynamicUpdate(t *testing.T) {
 	// Test that different configurations produce different descriptions
 	config1 := ModuleConfig{EnabledModules: []string{"console", "fs"}}
-	config2 := ModuleConfig{EnabledModules: []string{"fetch", "timers"}}
+	config2 := ModuleConfig{EnabledModules: []string{"timers"}}
 
 	server1, err := NewJSServerWithConfig(config1)
 	assert.NoError(t, err)
@@ -120,11 +117,9 @@ func TestToolDescriptionDynamicUpdate(t *testing.T) {
 	// Config1 should mention console and fs
 	assert.Contains(t, desc1, "• console:")
 	assert.Contains(t, desc1, "• fs:")
-	assert.NotContains(t, desc1, "• fetch:")
 	assert.NotContains(t, desc1, "• timers:")
 
-	// Config2 should mention fetch and timers
-	assert.Contains(t, desc2, "• fetch:")
+	// Config2 should mention timers
 	assert.Contains(t, desc2, "• timers:")
 	assert.NotContains(t, desc2, "• console:")
 	assert.NotContains(t, desc2, "• fs:")

jsserver/description_test.go

@@ -27,12 +27,10 @@ func TestModuleConfiguration_EnabledModules(t *testing.T) {
 			// These should not be available
 			const fsAvailable = typeof fs !== 'undefined';
 			const httpAvailable = typeof http !== 'undefined';
-			const fetchAvailable = typeof fetch !== 'undefined';
 			const processAvailable = typeof process !== 'undefined';
 			
 			console.log("fs available:", fsAvailable);
 			console.log("http available:", httpAvailable);
-			console.log("fetch available:", fetchAvailable);
 			console.log("process available:", processAvailable);
 			
 			"module test completed";
@@ -51,15 +49,14 @@ func TestModuleConfiguration_EnabledModules(t *testing.T) {
 	// Should not have other modules
 	assert.Contains(t, text, "fs available: false")
 	assert.Contains(t, text, "http available: false")
-	assert.Contains(t, text, "fetch available: false")
 	assert.Contains(t, text, "process available: false")
 	assert.Contains(t, text, "Result: module test completed")
 }
 
 func TestModuleConfiguration_DisabledModules(t *testing.T) {
 	// Test with all modules except fs and http
 	config := ModuleConfig{
-		EnabledModules: []string{"console", "fetch", "timers", "process"},
+		EnabledModules: []string{"console", "timers", "process"},
 	}
 	handler := NewJSHandlerWithConfig(config)
 
@@ -69,7 +66,6 @@ func TestModuleConfiguration_DisabledModules(t *testing.T) {
 		"code": `
 			// These should work
 			console.log("Console works");
-			console.log("fetch available:", typeof fetch !== 'undefined');
 			console.log("process available:", typeof process !== 'undefined');
 			
 			// These should not be available
@@ -88,7 +84,6 @@ func TestModuleConfiguration_DisabledModules(t *testing.T) {
 
 	// Should have enabled modules
 	assert.Contains(t, text, "Console works")
-	assert.Contains(t, text, "fetch available: true")
 	assert.Contains(t, text, "process available: true")
 
 	// Should not have disabled modules