Fix some linting errors (#89)

Author: peteski22

.github/ISSUE_TEMPLATE/bug_report.yaml

@@ -1,7 +1,6 @@
 name: Bug Report
 description: Report a bug or an unexpected behavior
 labels: ["bug", "needs-triage"]
-assignees: []
 body:
   - type: markdown
     attributes:

.github/ISSUE_TEMPLATE/feature_request.yaml

@@ -1,7 +1,6 @@
 name: Feature Request
 description: Suggest an idea or enhancement for the project.
 labels: ["enhancement", "needs-triage"]
-assignees: []
 body:
   - type: markdown
     attributes:

.gitignore

@@ -47,4 +47,6 @@ secrets.dev.toml
 docs/commands/
 
 # Any Python virtual environments
-.venv/
+.venv/
+# Added by goreleaser init:
+dist/

CONTRIBUTING.md

@@ -30,7 +30,7 @@ We encourage contributions via GitHub pull requests. Before you start, please re
     git checkout -b your-feature-or-bugfix-branch
     ```
 1. **Make your changes.**
-1. **Format and Lint:** Ensure your code is formatted using [gofumpt](https://github.com/mvdan/gofumpt).
+1. **Format and Lint:** Ensure your code is formatted using [gofumpt](https://github.com/mvdan/gofumpt) and [golangci-lint run ./...](https://golangci-lint.run/welcome/install/).
 1. **Add Unit Tests:** All new features and bug fixes should be accompanied by relevant unit tests.
 1. **Commit your changes** with a clear and descriptive message.
 1. **Push your branch** to your forked repository.

SECURITY.md

@@ -11,7 +11,7 @@ This document outlines the process for reporting vulnerabilities in `mcpd`.
 The following versions are currently supported for security updates:
 
 | Version | Supported          |
-| ------- | ------------------ |
+|---------|--------------------|
 | 0.0.x   | :white_check_mark: |
 
 **Please ensure you are using a supported version when reporting a vulnerability.**

cmd/add_test.go

@@ -31,7 +31,7 @@ func (f *fakeConfig) AddServer(entry config.ServerEntry) error {
 	return nil
 }
 
-func (f *fakeConfig) RemoveServer(name string) error {
+func (f *fakeConfig) RemoveServer(_ string) error {
 	return nil
 }
 
@@ -44,7 +44,7 @@ type fakeLoader struct {
 	err error
 }
 
-func (f *fakeLoader) Load(path string) (config.Modifier, error) {
+func (f *fakeLoader) Load(_ string) (config.Modifier, error) {
 	return f.cfg, f.err
 }
 

cmd/config/args/clear.go

@@ -75,7 +75,12 @@ func (c *ClearCmd) run(cmd *cobra.Command, args []string) error {
 		return fmt.Errorf("error clearing arguments for server '%s': %w", serverName, err)
 	}
 
-	fmt.Fprintf(cmd.OutOrStdout(), "✓ Arguments cleared for server '%s' (operation: %s)\n", serverName, string(res))
+	if _, err := fmt.Fprintf(
+		cmd.OutOrStdout(),
+		"✓ Arguments cleared for server '%s' (operation: %s)\n", serverName, string(res),
+	); err != nil {
+		return err
+	}
 
 	return nil
 }

cmd/config/args/list.go

@@ -40,7 +40,7 @@ func NewListCmd(baseCmd *cmd.BaseCmd, opt ...options.CmdOption) (*cobra.Command,
 	return cobraCmd, nil
 }
 
-func (c *ListCmd) run(cmd *cobra.Command, args []string) error {
+func (c *ListCmd) run(_ *cobra.Command, args []string) error {
 	serverName := strings.TrimSpace(args[0])
 	if serverName == "" {
 		return fmt.Errorf("server-name is required")

cmd/config/args/remove.go

@@ -78,7 +78,12 @@ func (c *RemoveCmd) run(cmd *cobra.Command, args []string) error {
 		return fmt.Errorf("error removing arguments for server '%s': %w", serverName, err)
 	}
 
-	fmt.Fprintf(cmd.OutOrStdout(), "✓ Arguments removed for server '%s' (operation: %s): %v\n", serverName, string(res), slices.Collect(maps.Keys(argMap)))
+	if _, err := fmt.Fprintf(
+		cmd.OutOrStdout(),
+		"✓ Arguments removed for server '%s' (operation: %s): %v\n", serverName, string(res), slices.Collect(maps.Keys(argMap)),
+	); err != nil {
+		return err
+	}
 
 	return nil
 }

cmd/config/args/set.go

@@ -80,7 +80,12 @@ func (c *SetCmd) run(cmd *cobra.Command, args []string) error {
 		return fmt.Errorf("error setting arguments for server '%s': %w", serverName, err)
 	}
 
-	fmt.Fprintf(cmd.OutOrStdout(), "✓ Startup arguments set for server '%s' (operation: %s): %v\n", serverName, string(res), normalizedArgs)
+	if _, err := fmt.Fprintf(
+		cmd.OutOrStdout(),
+		"✓ Startup arguments set for server '%s' (operation: %s): %v\n", serverName, string(res), normalizedArgs,
+	); err != nil {
+		return err
+	}
 
 	return nil
 }