Skip to content

fix: unblock client release workflows#56

Merged
linfangw merged 1 commit into
mainfrom
fix/release-workflow-tags
May 20, 2026
Merged

fix: unblock client release workflows#56
linfangw merged 1 commit into
mainfrom
fix/release-workflow-tags

Conversation

@linfangw
Copy link
Copy Markdown
Contributor

Summary

  • Let the MCP OpenAPI generation script use a pinned npx invocation so package.json stays aligned with the existing lockfile for npm ci
  • Allow explicit Python SDK QverisConfig constructor values to override environment aliases
  • Add a regression test for constructor values taking precedence over QVERIS_* environment variables

Validation

  • git diff --check
  • npm --prefix packages/mcp ci
  • npm --prefix packages/mcp run typecheck
  • npm --prefix packages/mcp test
  • npm --prefix packages/mcp run build
  • env QVERIS_API_KEY=sk-env QVERIS_BASE_URL=https://env.example/api/v1 uv run --extra dev pytest tests/test_client_contracts.py -q
  • env QVERIS_API_KEY=sk-env QVERIS_BASE_URL=https://env.example/api/v1 uv run --extra dev pytest
  • env QVERIS_API_KEY=sk-env QVERIS_BASE_URL=https://env.example/api/v1 uv run python -m compileall qveris examples

Release Follow-up

After this merges, move tags mcp-v0.7.0 and python-sdk-v0.2.0 to the merge commit and force-push those tags to rerun the failed release workflows.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request modifies the build scripts in packages/mcp/package.json by switching to npx for openapi-typescript and removing it from devDependencies. It also updates the Python SDK's QverisConfig to support population by name and adds a test verifying that constructor arguments take precedence over environment variables. Feedback was provided regarding the removal of the openapi-typescript dependency, recommending it be kept to ensure build determinism and avoid unnecessary network requests.

Comment thread packages/mcp/package.json Outdated
Comment thread packages/mcp/package.json
@linfangw linfangw force-pushed the fix/release-workflow-tags branch from 59bd9e5 to 77663c8 Compare May 20, 2026 01:50
@linfangw linfangw merged commit 5c98af3 into main May 20, 2026
2 checks passed
@linfangw linfangw deleted the fix/release-workflow-tags branch May 20, 2026 01:53
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