Skip to content

fix(news-aggregator): parse limits strictly#12

Merged
ralyodio merged 1 commit into
profullstack:masterfrom
rissrice2105-agent:fix/news-aggregator-limit-parsing
Jul 13, 2026
Merged

fix(news-aggregator): parse limits strictly#12
ralyodio merged 1 commit into
profullstack:masterfrom
rissrice2105-agent:fix/news-aggregator-limit-parsing

Conversation

@rissrice2105-agent

Copy link
Copy Markdown
Contributor

Summary

  • add a shared strict positive-integer limit parser for news-aggregator
  • use it in both the HTTP search endpoint and MCP tool endpoint
  • add regression coverage so malformed values like 2abc no longer get treated as 2

Tests

  • corepack pnpm install --frozen-lockfile
  • corepack pnpm install (inside mcp_modules/news-aggregator)
  • corepack pnpm exec mocha --require=./test/setup.js mcp_modules/news-aggregator/test/controller.test.js
  • git diff --check

Note: root eslint on the touched news-aggregator files still reports pre-existing indentation style errors across the module, so I did not reformat the module in this focused bug fix.

@ralyodio
ralyodio force-pushed the fix/news-aggregator-limit-parsing branch from d453c00 to 35bc20b Compare July 13, 2026 13:48
@ralyodio
ralyodio merged commit b0bd1c1 into profullstack:master Jul 13, 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