fix: resolve final linting issues

- Fix shellcheck warning in stop.sh by using underscore for unused loop variable
- Fix YAML line length issues in release workflow by breaking go build commands further
- Remove extra blank line in release workflow YAML

Author: maximilien

.github/workflows/release.yml

@@ -64,23 +64,28 @@ jobs:
           GOOS=linux GOARCH=amd64 go build \
             -ldflags \
             "-X github.com/maximilien/maestro-mcp/src/pkg/config.Version=$VERSION" \
-            -o release/maestro-mcp-linux-amd64 ./src/main.go
+            -o release/maestro-mcp-linux-amd64 \
+            ./src/main.go
           GOOS=linux GOARCH=arm64 go build \
             -ldflags \
             "-X github.com/maximilien/maestro-mcp/src/pkg/config.Version=$VERSION" \
-            -o release/maestro-mcp-linux-arm64 ./src/main.go
+            -o release/maestro-mcp-linux-arm64 \
+            ./src/main.go
           GOOS=darwin GOARCH=amd64 go build \
             -ldflags \
             "-X github.com/maximilien/maestro-mcp/src/pkg/config.Version=$VERSION" \
-            -o release/maestro-mcp-darwin-amd64 ./src/main.go
+            -o release/maestro-mcp-darwin-amd64 \
+            ./src/main.go
           GOOS=darwin GOARCH=arm64 go build \
             -ldflags \
             "-X github.com/maximilien/maestro-mcp/src/pkg/config.Version=$VERSION" \
-            -o release/maestro-mcp-darwin-arm64 ./src/main.go
+            -o release/maestro-mcp-darwin-arm64 \
+            ./src/main.go
           GOOS=windows GOARCH=amd64 go build \
             -ldflags \
             "-X github.com/maximilien/maestro-mcp/src/pkg/config.Version=$VERSION" \
-            -o release/maestro-mcp-windows-amd64.exe ./src/main.go
+            -o release/maestro-mcp-windows-amd64.exe \
+            ./src/main.go
 
           # Make binaries executable
           chmod +x release/maestro-mcp-linux-amd64
@@ -158,7 +163,6 @@ jobs:
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 
-      - name: Upload build artifacts
         uses: actions/upload-artifact@v4
         with:
           name: release-binaries-${{ steps.version.outputs.version }}

stop.sh

@@ -100,7 +100,7 @@ stop_server() {
                 print_status "Sent SIGTERM signal to server..."
 
                 # Wait for graceful shutdown
-                for i in {1..30}; do
+                for _ in {1..30}; do
                     if ! ps -p "$PID" > /dev/null 2>&1; then
                         print_success "Server stopped gracefully"
                         rm -f maestro-mcp.pid