Skip to content

chore(deps): upgrade gql to 4.0.0#295

Merged
adriannoes merged 4 commits into
mainfrom
chore/upgrade-gql-4
Jun 12, 2026
Merged

chore(deps): upgrade gql to 4.0.0#295
adriannoes merged 4 commits into
mainfrom
chore/upgrade-gql-4

Conversation

@adriannoes

Copy link
Copy Markdown
Collaborator

Summary

  • Bump gql[httpx]>=4.0.0,<5 in packages/sdk and refresh uv.lock.
  • Adapt SDK tests for gql 4: gql() constants are GraphQLRequest objects (use .document for AST introspection); HTTP timeouts surface as TransportConnectionFailed (subclass of TransportError, so existing CLI catches remain valid).
  • session.execute(query, variable_values=...) preserved; verify=True unchanged from PR1.
  • CHANGELOG entry under [Unreleased].

Stacked on pydantic upgrade branch — merge after chore/upgrade-pydantic-2.13.

Test plan

  • uv run pytest -m "not integration" — 2947 passed
  • uv run ruff check .
  • uv run pytest -m integration — 46 skipped (no PIPEFY_* creds in CI agent env)

Manual smoke (post-merge)

  • Cursor MCP: invoke a read-only tool (e.g. get_me) against a configured server.
  • Optional: npx @modelcontextprotocol/inspector uv --directory . run pipefy-mcp-server for transport debugging.

Comment thread packages/sdk/src/pipefy_sdk/base_client.py Outdated
Comment thread packages/sdk/src/pipefy_sdk/base_client.py
@adriannoes adriannoes force-pushed the chore/upgrade-pydantic-2.13 branch from cec96ed to dcbffc2 Compare June 12, 2026 13:27
@adriannoes adriannoes changed the base branch from chore/upgrade-pydantic-2.13 to main June 12, 2026 13:30
gql 4 deprecates variable_values kwargs on session.execute(); build a
fresh GraphQLRequest per call so shared query constants stay immutable.
Document empty-dict omission, add regression test, and replace
stale DocumentNode references with GraphQLRequest.
@adriannoes adriannoes force-pushed the chore/upgrade-gql-4 branch from 1939a70 to 54d4c99 Compare June 12, 2026 13:32
@adriannoes adriannoes merged commit 107bbb2 into main Jun 12, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants