perf(client): pass pagination offset and limit as GraphQL variables

[fatih-acar]

Summary

Paginated queries generated by the SDK (all(), filters(), get(), and resource pool allocation lookups) now pass offset and limit as GraphQL variables instead of inlining them into the query text. The query document stays byte-identical across pages, so repeated pages of the same fetch can hit Infrahub's server-side GraphQL query cache (opsmill/infrahub#8102) — the same optimization the frontend adopted in opsmill/infrahub#8245.

Key Changes

• client.filters() (async and sync) renders the query once per call with $offset/$limit placeholders and sends the per-page values in the GraphQL variables payload — previously the query was re-generated and re-rendered for every page with literal values baked in.
• get_pool_allocated_resources() (async and sync) does the same for its pagination loop, extending the $pool_id/$resource_id variables it already used.
• generate_query_data() / generate_query_data_init() accept variable placeholder strings (e.g. "$offset") for offset and limit; passing literal ints keeps working unchanged.
• count() needed no change: it takes no pagination arguments, so its query text was already stable per filter set.

Related Context

• fix(frontend): use variables for graphql pagination infrahub#8245 — frontend counterpart of this change
• fix(backend): add graphql parse/validate caching infrahub#8102 — the backend GraphQL query caching this leverages

Documentation Updates

• Regenerated SDK reference (docs/docs/python-sdk/sdk_ref/infrahub_sdk/node/node.mdx) for the widened signatures.
• Changelog fragment changelog/+graphql-pagination-variables.changed.md.

Test Plan

• uv run pytest tests/unit/ — updated the six operation-name assertions to expect the ($offset: Int!, $limit: Int!) signature; added test_method_all_pagination_uses_graphql_variables, which fetches two pages and asserts the query text is identical across pages while only the variables differ ({"offset": 0, "limit": 3} → {"offset": 3, "limit": 3}).
• uv run invoke lint-code (ruff, ty, mypy) passes; uv run invoke docs-generate output is committed.

🤖 Generated with Claude Code

---

Summary by cubic

SDK paginated queries now send offset and limit as GraphQL variables, keeping the query text identical across pages and enabling server-side cache hits. Queries are rendered once per call, and generate_query_data* now accept variable placeholders.

• Refactors
  • Use GraphQL variables for offset/limit in all(), filters(), get(), and pool allocation lookups. One rendered document per call; per-page values go in variables.
  • generate_query_data() and generate_query_data_init() accept int or variable placeholders (e.g. "$offset", "$limit"). Backward compatible.
  • count() unchanged.

^{Written for commit ac9e0f7. Summary will update on new commits.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

@@                 Coverage Diff                  @@
##           infrahub-develop    #1207      +/-   ##
====================================================
- Coverage             83.98%   83.17%   -0.81%     
====================================================
  Files                   139      139              
  Lines                 14390    12441    -1949     
  Branches               2381     1852     -529     
====================================================
- Hits                  12085    10348    -1737     
+ Misses                 1654     1528     -126     
+ Partials                651      565      -86     

Flag	Coverage Δ
integration-tests	40.30% <60.00%> (-2.38%)	⬇️
python-3.10	56.94% <100.00%> (-1.71%)	⬇️
python-3.11	56.93% <100.00%> (-1.72%)	⬇️
python-3.12	56.94% <100.00%> (-1.69%)	⬇️
python-3.13	56.94% <100.00%> (-1.71%)	⬇️
python-3.14	56.94% <100.00%> (-1.71%)	⬇️
python-filler-3.12	22.50% <0.00%> (-0.22%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
infrahub_sdk/client.py	79.62% <100.00%> (-1.28%)	⬇️
infrahub_sdk/node/node.py	87.40% <100.00%> (-1.27%)	⬇️

... and 13 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[cubic-dev-ai]

No issues found across 5 files

_{Re-trigger cubic}

[dgarros]

Looks good, ideally I think we should target infrahub-develop directly because there are a lot of changes in this branch right now compared to develop

[cloudflare-workers-and-pages]

Deploying infrahub-sdk-python with    Cloudflare Pages

Latest commit:	ac9e0f7
Status:	✅  Deploy successful!
Preview URL:	https://86441eb7.infrahub-sdk-python.pages.dev
Branch Preview URL:	https://fac-check-sdk-variables-2h50.infrahub-sdk-python.pages.dev

View logs