Merge develop into infrahub-develop#1174
Merged
Merged
Conversation
The hand-maintained async/sync client twins had silently diverged. Fix the
behavioral and annotation drift and lock the two behavioral fixes with tests:
- branch: sync merge now applies the 120s minimum timeout floor like async
- node: sync create(allow_upsert=True) excludes the hfid to avoid upsert overhead
- node: async artifact fetch uses the FETCH (not GENERATE) unsupported message
- node: fix sync resource-pool error text typo ("Allocate" -> "Allocated")
- client: fix async filters docstring return type (InfrahubNode, not the Sync)
- schema/node: converge private-helper annotations that drifted between twins
Add regression tests covering the upsert-hfid exclusion and the merge timeout
floor for both the async and sync clients.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Regenerated docs/docs/python-sdk/sdk_ref/infrahub_sdk/client.mdx to match the corrected InfrahubClient.filters docstring (list[InfrahubNode] instead of list[InfrahubNodeSync]) from the async/sync twin drift fix. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
fix: correct drifted async/sync twins and add parity regression tests
Merge stable into develop
github-actions
Bot
added
the
type/documentation
Deploying infrahub-sdk-python with
|
| Latest commit: |
bfedde9
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://1bf666ab.infrahub-sdk-python.pages.dev |
| Branch Preview URL: | https://develop.infrahub-sdk-python.pages.dev |
Codecov Report❌ Patch coverage is
@@ Coverage Diff @@
## infrahub-develop #1174 +/- ##
====================================================
+ Coverage 82.96% 83.02% +0.06%
====================================================
Files 139 139
Lines 12703 12703
Branches 1906 1906
====================================================
+ Hits 10539 10547 +8
+ Misses 1597 1587 -10
- Partials 567 569 +2
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Merging develop into infrahub-develop after merging pull request #1173.
Summary by cubic
Aligns the async and sync Python SDK clients and fixes small drifts. Branch merges now enforce a 120s minimum timeout, upserts omit
hfidfor better performance, and messages/docs are corrected.branch.mergeappliesmax(120, default_timeout)to match async.create(allow_upsert=True)omitshfidin mutation payload in both clients to avoid server-side overhead.artifact_fetchunsupported message and a resource-pool error text typo.client.filtersreturn type tolist[InfrahubNode]and aligned internal annotations to keep async/sync twins consistent.Written for commit bfedde9. Summary will update on new commits.