cmd/sketch: move a function

josharian · josharian · commit 0d04c13e27e0 · 2025-07-24T12:20:54.000-07:00
Moves the runIn* functions closer together,
for when humans need to step in and un-****
this code.
diff --git a/cmd/sketch/main.go b/cmd/sketch/main.go
@@ -435,6 +435,28 @@ func parseCLIFlags() CLIFlags {
 	return flags
 }
 
+func skabandMcpConfiguration(flags CLIFlags) string {
+	skabandaddr, err := skabandclient.LocalhostToDockerInternal(flags.skabandAddr)
+	if err != nil {
+		skabandaddr = flags.skabandAddr
+	}
+	config := mcp.ServerConfig{
+		Name: "sketchdev",
+		Type: "http",
+		URL:  skabandaddr + "/api/mcp",
+		Headers: map[string]string{
+			"Session-Id":     flags.sessionID,
+			"Public-Key":     "env:SKETCH_PUB_KEY",
+			"Session-Secret": "env:SKETCH_MODEL_API_KEY",
+		},
+	}
+	out, err := json.Marshal(&config)
+	if err != nil {
+		panic("programming error" + err.Error())
+	}
+	return string(out)
+}
+
 // runInHostMode handles execution on the host machine, which typically involves
 // checking host requirements and launching a Docker container.
 func runInHostMode(ctx context.Context, flags CLIFlags) error {
@@ -534,28 +556,6 @@ func runInHostMode(ctx context.Context, flags CLIFlags) error {
 	return nil
 }
 
-func skabandMcpConfiguration(flags CLIFlags) string {
-	skabandaddr, err := skabandclient.LocalhostToDockerInternal(flags.skabandAddr)
-	if err != nil {
-		skabandaddr = flags.skabandAddr
-	}
-	config := mcp.ServerConfig{
-		Name: "sketchdev",
-		Type: "http",
-		URL:  skabandaddr + "/api/mcp",
-		Headers: map[string]string{
-			"Session-Id":     flags.sessionID,
-			"Public-Key":     "env:SKETCH_PUB_KEY",
-			"Session-Secret": "env:SKETCH_MODEL_API_KEY",
-		},
-	}
-	out, err := json.Marshal(&config)
-	if err != nil {
-		panic("programming error" + err.Error())
-	}
-	return string(out)
-}
-
 // runInContainerMode handles execution inside the Docker container.
 // The inInsideSketch parameter indicates whether we're inside the sketch container
 // with access to outside environment variables.
