Add AWS Spot Placement Scores support (#5)

* Add AWS Spot Placement Scores core functionality

- Add score.go with AWS EC2 spot placement score integration
- Extend types.go with score-related data structures
- Add comprehensive types_test.go with score functionality tests
- Implements region and AZ-level score fetching with timeout handling

* Integrate spot placement scores into client layer

- Update client.go with score enrichment capability
- Add comprehensive client_test.go coverage for score features
- Regenerate mocks_test.go for new scoring interfaces
- Enables score filtering, sorting, and regional/AZ-level fetching

* Add CLI flags and UI for AWS Spot Placement Scores

- Add --with-score, --min-score, --az, --score-timeout flags
- Implement color-coded score display (🟢/🔴 indicators)
- Add score sorting and filtering in CLI interface
- Update tests for new scoring command-line functionality

* Extend MCP server with spot placement score capabilities

- Update MCP tools to support score parameters and filtering
- Add score enrichment to MCP tool responses
- Refactor and streamline MCP test coverage
- Enable score features in Model Context Protocol interface

* Add comprehensive documentation for spot placement scores

- Add aws-spot-placement-scores.md with detailed API documentation
- Update usage.md with score filtering examples and CLI options
- Extend examples.md with score-based use cases and scenarios
- Update mcp-server.md with score parameter documentation
- Enhance data-sources.md and api-reference.md for score features

* Update dependencies and tooling configuration

- Update go.mod/go.sum with AWS SDK dependencies for score API
- Enhance .golangci.yaml with additional linting rules
- Update .mockery.yaml for new scoring interfaces
- Ensure compatibility with score enrichment dependencies

* Update README with AWS Spot Placement Scores documentation

- Add score feature overview and benefits section
- Include score CLI examples and usage scenarios
- Document score filtering, sorting, and visualization features
- Streamline README structure with focus on core functionality

Author: alexei-led

.golangci.yaml

@@ -6,14 +6,11 @@ linters:
       enable:
         - shadow
         - fieldalignment
-    revive:
-      min-confidence: 0
     gocyclo:
       min-complexity: 15
     cyclop:
       max-complexity: 15
       package-average: 10.0
-      skip-tests: true
     dupl:
       threshold: 100
     goconst:
@@ -67,13 +64,20 @@ linters:
             - $all
           allow:
             - $gostd
-            - spotinfo/pkg/spot
+            - spotinfo/internal/spot
             - spotinfo/internal
+            - spotinfo/internal/mcp
             - github.com/jedib0t/go-pretty/v6
             - github.com/urfave/cli/v2
             - github.com/stretchr/testify
             - github.com/mark3labs/mcp-go
             - github.com/spf13/cast
+            - github.com/bluele/gcache
+            - golang.org/x/time
+            - github.com/aws/aws-sdk-go-v2
+            - github.com/aws/smithy-go
+            - github.com/jedib0t/go-pretty/v6/table
+            - github.com/jedib0t/go-pretty/v6/text
     testifylint:
       enable-all: true
     tagalign:
@@ -85,53 +89,53 @@ linters:
   default: none
   enable:
     # Essential bug-finding linters
-    - errcheck           # Check for unchecked errors
-    - gosec             # Security issues
-    - govet             # Go vet built-in checks
-    - ineffassign       # Ineffective assignments
-    - staticcheck       # Comprehensive static analysis
-    - unused            # Unused code
-    - errorlint         # Error wrapping issues
-    
+    - errcheck # Check for unchecked errors
+    - gosec # Security issues
+    - govet # Go vet built-in checks
+    - ineffassign # Ineffective assignments
+    - staticcheck # Comprehensive static analysis
+    - unused # Unused code
+    - errorlint # Error wrapping issues
+
     # Code quality and best practices
-    - bodyclose         # HTTP response body close check
-    - contextcheck      # Context usage patterns
-    - copyloopvar       # Loop variable copying issues
-    - cyclop            # Cyclomatic complexity
-    - depguard          # Import restrictions
-    - dupl              # Code duplication
-    - exhaustive        # Switch statement exhaustiveness
-    - funlen            # Function length
-    - goconst           # Repeated strings that could be constants
-    - gocritic          # Comprehensive go code critic
-    - gocyclo           # Cyclomatic complexity
-    - makezero          # Slice initialization
-    - nakedret          # Naked returns
-    - nestif            # Nested if statements
-    - nilerr            # Nil error returns
-    - nilnil            # Nil error and value returns
-    - noctx             # HTTP without context
-    - predeclared       # Predeclared identifier shadowing
-    - rowserrcheck      # SQL rows.Err() check
-    - sqlclosecheck     # SQL close checks
-    - unconvert         # Unnecessary conversions
-    - unparam           # Unused parameters
-    - wastedassign      # Wasted assignments
-    
+    - bodyclose # HTTP response body close check
+    - contextcheck # Context usage patterns
+    - copyloopvar # Loop variable copying issues
+    - cyclop # Cyclomatic complexity
+    - depguard # Import restrictions
+    - dupl # Code duplication
+    - exhaustive # Switch statement exhaustiveness
+    - funlen # Function length
+    - goconst # Repeated strings that could be constants
+    - gocritic # Comprehensive go code critic
+    - gocyclo # Cyclomatic complexity
+    - makezero # Slice initialization
+    - nakedret # Naked returns
+    - nestif # Nested if statements
+    - nilerr # Nil error returns
+    - nilnil # Nil error and value returns
+    # - noctx             # HTTP without context - disabled: too verbose for initialization and error handling code
+    - predeclared # Predeclared identifier shadowing
+    - rowserrcheck # SQL rows.Err() check
+    - sqlclosecheck # SQL close checks
+    - unconvert # Unnecessary conversions
+    - unparam # Unused parameters
+    - wastedassign # Wasted assignments
+
     # Performance-related
-    - prealloc          # Slice preallocation
-    
+    - prealloc # Slice preallocation
+
     # Additional best practices (keep important ones)
-    - durationcheck     # Duration multiplication
-    - errname           # Error naming conventions
-    - interfacebloat    # Interface size
-    - maintidx          # Maintainability index
-    - mirror            # Wrong mirror patterns
-    - mnd               # Magic numbers
-    - nilnesserr        # Nil check patterns
+    - durationcheck # Duration multiplication
+    - errname # Error naming conventions
+    - interfacebloat # Interface size
+    - maintidx # Maintainability index
+    - mirror # Wrong mirror patterns
+    - mnd # Magic numbers
+    - nilnesserr # Nil check patterns
     - nosprintfhostport # Sprintf host:port misuse
-    - reassign          # Package variable reassignment
-    - forcetypeassert   # Type assertion without check
+    - reassign # Package variable reassignment
+    - forcetypeassert # Type assertion without check
 
   # Explicitly disabled linters with reasons
   # - gochecknoglobals  # Too restrictive for CLI apps that need global flags/config
@@ -196,4 +200,6 @@ formatters:
         - default
         - prefix(spotinfo)
     goimports:
-      local-prefixes: spotinfo
+      local-prefixes:
+        - spotinfo
+

.mockery.yaml

@@ -7,9 +7,11 @@ packages:
     interfaces:
       advisorProvider:
       pricingProvider:
+      scoreProvider:
+      awsAPIProvider:
   spotinfo/cmd/spotinfo:
     interfaces:
-      SpotClient:
+      spotClient:
   spotinfo/internal/mcp:
     interfaces:
       spotClient: