Skip to content

ci: restore deterministic green builds - #29

Merged
arthurpanhku merged 2 commits into
mainfrom
agent/fix-main-ci
Aug 18, 2026
Merged

ci: restore deterministic green builds#29
arthurpanhku merged 2 commits into
mainfrom
agent/fix-main-ci

Conversation

@arthurpanhku

@arthurpanhku arthurpanhku commented Aug 18, 2026

Copy link
Copy Markdown
Owner

Summary

  • pin the MCP SDK version that provides the FastMCP import path used by the backend
  • add the missing ReportLab dependency for the prompt-injection corpus tests
  • upgrade React Router to 7.18.2 to clear the production audit

Root cause

The backend dependency manifest could resolve an MCP package without mcp.server.fastmcp; the prompt-injection corpus also required ReportLab but it was absent from development dependencies. The frontend lockfile still selected React Router 7.18.1, which has two high-severity advisories.

A first CI attempt also showed that the existing macOS-generated hash lock omits Linux-only CUDA transitive hashes, so CI continues to install the primary manifest while the MCP compatibility boundary is pinned explicitly.

Validation

  • ruff check app tests evals
  • backend: 121 tests passed, 71% coverage
  • evals: 5 tests passed
  • OpenAPI contract check passed
  • frontend: generated API types unchanged
  • frontend: 6 tests passed
  • npm audit --omit=dev: 0 vulnerabilities
  • production build passed

@socket-security

socket-security Bot commented Aug 18, 2026

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatednpm/​react-router-dom@​7.18.1 ⏵ 7.18.21001006594 -1100

View full report

@arthurpanhku
arthurpanhku marked this pull request as ready for review August 18, 2026 04:00
@arthurpanhku
arthurpanhku merged commit c58dd08 into main Aug 18, 2026
4 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.

1 participant