fix: remediate audited dependency vulnerabilities - #256
Conversation
|
|
Hi @l7wei — could you approve the fork workflow runs for this PR when convenient? The upstream This is intentionally an independent, reviewable dependency-security PR: remediation of known dependency advisories, FastMCP v3 compatibility cleanup, MCP The PR description records the CPython 3.14.7 test run (680 passed), a If the project prefers a different dependency-upgrade or review workflow, a brief direction is enough. Thanks. |
There was a problem hiding this comment.
Pull request overview
This pull request remediates audited dependency vulnerabilities, upgrades FastMCP, adds MCP initialization coverage, and tightens Docker build exclusions.
Changes:
- Updates dependency pins.
- Removes obsolete FastMCP configuration.
- Adds mounted
/mcpinitialization regression coverage. - Expands
.dockerignoreexclusions.
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated no comments.
| File | Summary |
|---|---|
tests/test_mcp_tools.py |
Adds mounted MCP initialization testing. |
src/data_api/core/settings.py |
Removes obsolete FastMCP configuration. |
requirements.txt |
Updates dependency pins. |
.dockerignore |
Excludes environments, secrets, and development artifacts. |
Suppressed comments (1)
.dockerignore:13
- The documented test command writes its HTML report to
coverage/and its XML report tocoverage.xml(README.md:84-87 and both CI workflows), but neither path is ignored here. Because the Dockerfile usesCOPY . ., running the tests beforedocker buildwill send these artifacts in the build context, so the stated context reduction is not reliable; add both coverage paths.
htmlcov/
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.



Summary
pip-auditadvisories.initializeregression test so the mounted/mcptransport is verified in addition to individual tool functions..env/.env.*files from the Docker build context.Verification — CPython 3.14.7
requirements-tests.txtandrequirements-dev.txt;pip checkpassed.python -m pip_audit -r requirements.txt --timeout 5 --progress-spinner off— no known vulnerabilities.python -m compileall -q src tests.python -m pytest -q— 680 passed on CPython 3.14.7.pre-commit run --all-files— Black and isort passed.docker build --pullusingpython:3.14-slim— final build context: 18.23 kB.initializerequest to/mcpreturned HTTP 200 and the expectedNTHU Campus Assistantidentity; the temporary container was stopped and auto-removed.Security review
git diff --check.8a363b9..f9e451c— complete coverage, 0 findings.CI note
GitHub's upstream
TestandLint with pre-commitworkflows are currentlyaction_requiredfor this fork PR and have not run. A maintainer must approve their execution; this PR does not represent them as passing CI.Scope
This is intentionally an independent dependency-security PR. It does not change the academic-calendar design in #184, PR #255, or the separate CORS test-first PR.