**WIP** fix: Support attribute assignment of cardinality-many relationships

[PhillSimonds]

What

Fixes #1152.

Assigning to a cardinality-many relationship attribute (e.g. node.members = [peer]) previously fell through InfrahubNode.__setattr__ to super().__setattr__, storing the raw value as a normal attribute and not correctly wrapping the object in a RelationshipManager. The node then failed in save() with the confusing 'list' object has no attribute 'initialized' error. This behavior is different than instantiating an object via .create() with a list passed into data=, which correctly used .setattr to construct a RelationshipManager and doesn't fall through to super().__setattr__.

Change

__setattr__ now handles cardinality-many relationships (both InfrahubNode and InfrahubNodeSync):

• a list rebuilds the RelationshipManager from the value and marks it _has_update so save() actually persists it;
• a non-list value raises the existing "expects a list of nodes" error from the RelationshipManager constructor, at assignment time rather than deferred to save().

Tests

Added to tests/unit/sdk/test_node.py (async + sync):

• test_cardinality_many_assignment_requires_list — single node/dict assignment raises "expects a list of nodes".
• test_cardinality_many_assignment_accepts_list — list assignment populates peers and sets has_update.

uv run invoke format lint-code clean; uv run pytest tests/unit/sdk/test_node.py → 226 passed.

Related

Same relationship-editing ergonomics area as #1031 and the sibling PR for #1153.

---

Summary by cubic

Fixes assignment to cardinality-many relationships so list assignments populate the manager and non-list values fail fast, eliminating the confusing 'InfrahubNode' object has no attribute 'initialized' save-time error. Applies to both async and sync nodes.

• Bug Fixes
  • Handle cardinality-many attrs in __setattr__ for InfrahubNode and InfrahubNodeSync.
  • List assignment rebuilds the relationship manager, marks _has_update, and is included in mutation payloads.
  • Non-list assignment raises a clear “expects a list of nodes” error at assignment time.
  • Added tests for list vs non-list assignment, input payload generation, and the save() path for both clients.

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

[cloudflare-workers-and-pages]

Deploying infrahub-sdk-python with    Cloudflare Pages

Latest commit:	9beed45
Status:	✅  Deploy successful!
Preview URL:	https://388d8111.infrahub-sdk-python.pages.dev
Branch Preview URL:	https://sts-fix-1152-many-rel-assign.infrahub-sdk-python.pages.dev

View logs

[codecov]

Codecov Report

❌ Patch coverage is 77.77778% with 4 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
infrahub_sdk/node/node.py	77.77%	2 Missing and 2 partials ⚠️

❗ There is a different number of reports uploaded between BASE (17483a0) and HEAD (9beed45). Click for more details.

HEAD has 1 upload less than BASE

Flag	BASE (17483a0)	HEAD (9beed45)
integration-tests	1	0

@@             Coverage Diff             @@
##           develop    #1154      +/-   ##
===========================================
- Coverage    82.35%   75.51%   -6.84%     
===========================================
  Files          138      138              
  Lines        12065    12060       -5     
  Branches      1805     1809       +4     
===========================================
- Hits          9936     9107     -829     
- Misses        1573     2401     +828     
+ Partials       556      552       -4     

Flag	Coverage Δ
integration-tests	?
python-3.10	55.76% <77.77%> (-0.05%)	⬇️
python-3.11	55.77% <77.77%> (-0.05%)	⬇️
python-3.12	55.76% <77.77%> (-0.06%)	⬇️
python-3.13	55.76% <77.77%> (-0.06%)	⬇️
python-3.14	55.77% <77.77%> (-0.03%)	⬇️
python-filler-3.12	22.44% <0.00%> (+<0.01%)	⬆️

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

Files with missing lines	Coverage Δ
infrahub_sdk/node/node.py	80.42% <77.77%> (-7.15%)	⬇️

... and 34 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 3 files

_{Re-trigger cubic}