feat: use defer processing error (#98)

songzhibin97 · web-flow · commit 62cdf7131a59 · 2025-04-10T20:31:21.000+03:00
diff --git a/client/sse.go b/client/sse.go
@@ -391,7 +391,7 @@ func (c *SSEMCPClient) Initialize(
 			err,
 		)
 	}
-	resp.Body.Close()
+	defer resp.Body.Close()
 
 	c.initialized = true
 	return &result, nil
diff --git a/server/internal/gen/main.go b/server/internal/gen/main.go
@@ -28,21 +28,19 @@ func RenderTemplateToFile(templateContent, destPath, fileName string, data any)
 	}
 	tempFilePath := tempFile.Name()
 	defer os.Remove(tempFilePath) // Clean up temp file when done
+	defer tempFile.Close()
 
 	// Parse and execute template to temp file
 	tmpl, err := template.New(fileName).Funcs(template.FuncMap{
 		"toLower": strings.ToLower,
 	}).Parse(templateContent)
 	if err != nil {
-		tempFile.Close()
 		return err
 	}
 
 	if err := tmpl.Execute(tempFile, data); err != nil {
-		tempFile.Close()
 		return err
 	}
-	tempFile.Close()
 
 	// Run goimports on the temp file
 	cmd := exec.Command("go", "run", "golang.org/x/tools/cmd/goimports@latest", "-w", tempFilePath)

Original file line number	Diff line number	Diff line change
`@@ -391,7 +391,7 @@ func (c *SSEMCPClient) Initialize(`
`391`	`391`	`err,`
`392`	`392`	`)`
`393`	`393`	`}`
`394`		`- resp.Body.Close()`
	`394`	`+ defer resp.Body.Close()`
`395`	`395`
`396`	`396`	`c.initialized = true`
`397`	`397`	`return &result, nil`