Skip to content

fix(weather): validate coordinates at API boundaries#14

Merged
ralyodio merged 1 commit into
profullstack:masterfrom
rissrice2105-agent:fix/weather-coordinate-validation
Jul 16, 2026
Merged

fix(weather): validate coordinates at API boundaries#14
ralyodio merged 1 commit into
profullstack:masterfrom
rissrice2105-agent:fix/weather-coordinate-validation

Conversation

@rissrice2105-agent

Copy link
Copy Markdown
Contributor

Summary

  • parse REST latitude and longitude strictly so partial values such as 40abc are rejected
  • reject non-finite and out-of-range coordinates before weather services are called
  • apply the same validation to the MCP weather tool endpoint
  • add focused REST, MCP, and utility regression coverage

Tests

  • pnpm --dir mcp_modules/weather test (32 passing)
  • targeted ESLint for Weather controller, utilities, and tests
  • git diff --check

The root suite reaches 53 passing and 5 pending tests; two existing Module Loader assertions fail in the Windows environment and are unrelated to Weather.

@ralyodio
ralyodio merged commit 283945a into profullstack:master Jul 16, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants