[thomaspinder]

Summary

CBS returns explicit null for missing metadata fields. .get(key, "") only applies its default when the key is absent, so it passed None through and pydantic rejected it. As a result, get_metadata("86103NED") crashed.

 _parse_column(prop)
-  description = prop.get("Description", "")    # None when CBS sends null
+  description = prop.get("Description") or ""

The same change applies to every string field of Column and TableMetadata.

Evidence

• Before: Client().get_metadata("86103NED") raises ValidationError: description — Input should be a valid string [input_value=None].
  After: it returns 40 properties.
• New test TestGetMetadata::test_null_fields_become_empty_strings: fails on main with the same ValidationError and passes on this branch.
• All 54 tests pass. ruff check, ruff format --check and ty check are clean.

Merge Danger

Door: two-way

Blast Radius: small

Only the null-handling changes. A field that used to crash is now "". id falls back to the requested table ID when Identifier is null.

🤖 Generated with Claude Code

[codecov-commenter]

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

ℹ️ You can also turn on project coverage checks and project coverage reporting on Pull Request comment

Thanks for integrating Codecov - We've got you covered ☂️