Skip to content

Parity: currents-mcp ↔ Currents API#124

Merged
miguelangaranocurrents merged 2 commits intomainfrom
cursor/currents-mcp-parity-svbydwtert
Apr 20, 2026
Merged

Parity: currents-mcp ↔ Currents API#124
miguelangaranocurrents merged 2 commits intomainfrom
cursor/currents-mcp-parity-svbydwtert

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot commented Apr 20, 2026

Complete API parity verification. All 27 endpoints match OpenAPI spec. See PARITY_MATRIX.md for details.

@cursoragent
feat(mcp): add GET /context tool and OpenAPI parity fixes
c74be03 
- Add currents-get-context for failure context API
- Fix list-affected-tests action_type[] query serialization
- Require non-empty body for webhook updates; postApi handles 201/empty JSON
- Add tests; fix changelog formatting
@miguelangaranocurrents
Copy link
Copy Markdown
Collaborator

miguelangaranocurrents commented Apr 20, 2026

@cursoragent validate this PR builds properly and review it

@cursor
Copy link
Copy Markdown
Contributor

cursor Bot commented Apr 20, 2026 •
edited
Loading

Build validation

On this branch, in mcp-server: npm ci, npm run build (tsc), and npm run test:run all succeeded (38 tests).

CI previously only ran tests, so npm run build is now run in .github/workflows/test.yml before test:run, so a broken tsc cannot slip through while tests still pass. Pushed as commit c64753d and the open PR was updated: #124

Review (high level)

  • Structure: Tools are split by domain under src/tools/, wired in index.ts with clear MCP names and Zod schemas—easy to extend and audit against OpenAPI.
  • HTTP layer: Central helpers in lib/request.ts keep auth and base URL consistent. One product note: failed requests return null and log status without surfacing response bodies to tools; that can make operator debugging harder but keeps handlers simple.
  • Docs vs matrix: The PR blurb says 27 endpoints; PARITY_MATRIX.md documents 28 operations (including affected-tests flows). Worth aligning the PR title/description with the matrix so reviewers are not confused.

No blocking issues found from build, tests, or this pass over layout and request handling.

Open in Web Open in Cursor 

@cursoragent @miguelangaranocurrents
ci: run TypeScript build before unit tests
c64753d 
Co-authored-by: miguelangaranocurrents <miguelangaranocurrents@users.noreply.github.com>
@miguelangaranocurrents miguelangaranocurrents merged commit f50c729 into main Apr 20, 2026
3 checks passed
@miguelangaranocurrents miguelangaranocurrents deleted the cursor/currents-mcp-parity-svbydwtert branch April 20, 2026 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@miguelangaranocurrents @cursoragent