fix: add labels to NodeNotFoundError for clearer messaging

[DharmaBytesX]

The error message now shows Branch:, Kind:, and Identifier: labels
instead of raw pipe-separated values, helping users understand
which branch the lookup failed on.

Before:

        Unable to find the node in the database.
        main | InfraDevice | {'name__value': 'bad-device404'}

After:

        Unable to find the node in the database.
        Branch: main | Kind: InfraDevice | Identifier: {'name__value': 'bad-device404'}

Fixes opsmill/infrahub#5514

---

Summary by cubic

Add labels to NodeNotFoundError and require keyword-only branch_name, node_type, and identifier, producing consistent messages like "Branch: ... | Kind: ... | Identifier: ...".
Updated file download paths to pass branch and node ID, and store/CTL lookups to include branch (falling back to the client’s default), so all NodeNotFoundError instances carry full context.

^{Written for commit 1ab16ae. Summary will update on new commits. Review in cubic}

[cubic-dev-ai]

No issues found across 1 file

_{Re-trigger cubic}

[DharmaBytesX]

Hi! A small fix here to improve the NodeNotFoundError message — could someone give it a quick review when you get a chance? Thanks!

[ogenstad]

I don't think we should have if-statements within the __str__() method of classes in general. I think the problem here is really that the parameters to the __init__() method are optional instead of being required. We should probably look at the places where we raise this error and check if we are always sending in all fields (or if we have enough information to do so) and then update the init method to require there parameters.

[DharmaBytesX]

Thanks for the review. I've reworked the change along the lines you suggested: branch_name, node_type and identifier are now required keyword-only parameters of NodeNotFoundError.__init__, and __str__ no longer has any conditional logic.

I audited the call sites and updated them to always pass the three fields:

• infrahub_sdk/client.py was already passing all three.
• infrahub_sdk/ctl/object/utils.py now resolves the branch to client.default_branch when the caller did not provide one.
• infrahub_sdk/file_handler.py: handle_response and handle_error_response now take branch and node_id, plumbed from FileHandler.download / _stream_to_file in both the async and sync paths.
• infrahub_sdk/store.py: passes self.branch_name, and falls back to "unknown" only on the internal lookup paths where the caller genuinely never specified a kind.

Also added tests/unit/sdk/test_exceptions.py covering the new contract (keyword-only, all three required, rendered format).