[mykeelium]

Add a PydanticValidator for EnterpriseCredentialInventory

Summary by CodeRabbit

• New Features
  • Added support for representing enterprise credential inventory exports, including export details, column information, and raw credential rows.
  • Export data can retain unrecognized credential types, malformed credential identifiers, and additional columns for later processing.
  • Inventory exports now validate that rows are provided as a list.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Organization UI
• Review profile: CHILL
• Plan: Essentials
• Run ID: 38032132-76c7-4f13-806c-8a1884e3dd0c

📥 Commits

Reviewing files that changed from the base of the PR and between 2d9c526 and 622f063.

📒 Files selected for processing: 4

• src/openhound_github/models/__init__.py
• src/openhound_github/models/enterprise_credential_inventory.py
• src/openhound_github/resources/enterprise.py
• tests/test_classic_personal_access_tokens.py

Included review availability: This review used your included allowance. 2 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.

Walkthrough

The change adds and exports an EnterpriseCredentialInventory model. The credential inventory transformer uses it as its schema, and tests check validation of raw rows and parser behavior.

Changes

Credential inventory schema

Layer / File(s)	Summary
Define and export the inventory model src/openhound_github/models/enterprise_credential_inventory.py, src/openhound_github/models/__init__.py	Adds and exports a Pydantic model for export metadata, columns, and raw rows. as_of is optional and defaults to None; other fields are required.
Use and validate the inventory model src/openhound_github/resources/enterprise.py, tests/test_classic_personal_access_tokens.py	The transformer uses the model as its schema. Tests check that validation preserves raw rows, rejects a non-list rows value, and lets the parser emit only the row with numeric credential ID 42.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Feature

Suggested reviewers: jaredcatkinson

Merge Risk: ⚪ Minimal · up to 622f0

The inventory model preserves the raw credential data and parser behavior covered by the tests. No actionable merge risk remains beyond normal checks.

Pre-merge checks: 4 · 1

Failed checks: 1 warning

Check name	Status	Explanation	Resolution
Docstring Coverage		Docstring coverage is 16.67% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 6 functions across 4 files.	Write docstrings for the functions missing them to satisfy the coverage threshold.

Passed checks: 4 passed

Check name	Status	Explanation
Description Check		Check skipped - CodeRabbit’s high-level summary is enabled.
Title check		The title clearly describes the addition of Pydantic validation for EnterpriseCredentialInventory, which matches the main change.
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

• Fix all pre-merge checks with AI

✨ Finishing Touches: 💡 1

📝 Generate docstrings: 💡

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

_{A rabbit checks each row with care
Raw fields stay just as they were
A model names the columns clear
One numeric token passes here
The inventory’s shape is near
Hop, the tests confirm the gear!}

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

[jaredcatkinson]

Looks good to me