use CoreNodeBase in get_schema_name

[ajtmccarty]

Summary by CodeRabbit

• Bug Fixes

  • Improved schema name handling: returns None for absent inputs, normalizes names for specialized node types, and trims trailing "Sync" suffix where applicable.

• Tests

  • Added unit tests covering the updated schema name resolution behavior.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

The get_schema_name function in infrahub_sdk/store.py was reworked to explicitly handle CoreNodeBase subclasses: it returns None for None input, detects whether the class save method is a coroutine, and strips a trailing Sync suffix from class names. A prior runtime-protocol attribute check was removed. New imports for inspect and CoreNodeBase were added. Unit tests were added to verify the function returns the expected names for BuiltinIPPrefix and BuiltinIPAddressSync.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title directly describes the main change: refactoring get_schema_name to use CoreNodeBase for type checking instead of previous runtime-protocol attribute check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[cloudflare-workers-and-pages]

Deploying infrahub-sdk-python with    Cloudflare Pages

Latest commit:	42ac2d4
Status:	✅  Deploy successful!
Preview URL:	https://e5a1b99c.infrahub-sdk-python.pages.dev
Branch Preview URL:	https://ajtm-01212026-get-schema-nam.infrahub-sdk-python.pages.dev

View logs

[codecov]

Codecov Report

❌ Patch coverage is 70.00000% with 3 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
infrahub_sdk/store.py	70.00%	1 Missing and 2 partials ⚠️

@@             Coverage Diff             @@
##           develop     #756      +/-   ##
===========================================
- Coverage    80.38%   80.38%   -0.01%     
===========================================
  Files          115      115              
  Lines         9886     9894       +8     
  Branches      1514     1517       +3     
===========================================
+ Hits          7947     7953       +6     
- Misses        1417     1418       +1     
- Partials       522      523       +1     

Flag	Coverage Δ
integration-tests	41.35% <0.00%> (-0.04%)	⬇️
python-3.10	51.41% <50.00%> (-0.03%)	⬇️
python-3.11	51.43% <50.00%> (-0.01%)	⬇️
python-3.12	51.41% <50.00%> (-0.03%)	⬇️
python-3.13	51.43% <50.00%> (-0.01%)	⬇️
python-3.14	53.07% <50.00%> (-0.01%)	⬇️
python-filler-3.12	24.04% <20.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/store.py	77.47% <70.00%> (-0.12%)	⬇️

🚀 New features to boost your workflow:

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

[solababs]

Closing this PR as the work here has been transferred here #763