chore(config): externalize entity schema

[bzp2010]

Summary by CodeRabbit

• Refactor
  • Extracted JSON schema definitions for API key and model configurations into external schema files, decoupling them from application source.
  • Schema loading now uses embedded schema documents at build time; runtime validation and behavior remain unchanged.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 3496bcca-a1cc-4da6-ac48-51788a2f05ff

📥 Commits

Reviewing files that changed from the base of the PR and between 78a3c59 and 0c5be35.

📒 Files selected for processing (2)

• src/config/entities/apikeys.rs
• src/config/entities/models.rs

🚧 Files skipped from review as they are similar to previous changes (2)

• src/config/entities/apikeys.rs
• src/config/entities/models.rs

---

📝 Walkthrough

Walkthrough

Two JSON Schema files were added for API key and model configurations, and two Rust modules were updated to parse those schemas at compile time via include_str! instead of embedding JSON literals.

Changes

Cohort / File(s)	Summary
Schema files src/config/entities/apikeys-schema.json, src/config/entities/models-schema.json	Added JSON Schema documents: apikeys-schema.json defines required key, allowed_models, optional rate_limit; models-schema.json (draft 2020-12) enforces name, model, provider_config, provider-specific allOf rules for OpenAI-compatible vs Bedrock configs and disallows additional properties.
Rust schema loading src/config/entities/apikeys.rs, src/config/entities/models.rs	Replaced inline json!({ ... }) schema literals with compile-time embedded JSON parsed from the new schema files using include_str! and serde_json::from_str. Removed unused serde_json::json imports; validator initialization remains based on parsed schema.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

🚥 Pre-merge checks | ✅ 6

✅ Passed checks (6 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'chore(config): externalize entity schema' accurately describes the main change: moving JSON schemas from inline Rust code to external JSON files, affecting both apikeys and models schemas across the config/entities directory.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
E2e Test Quality Review	✅ Passed	PR includes robust test coverage across multiple layers: Rust unit tests comprehensively verify schema loading from external JSON files with boundary cases; TypeScript E2E tests exercise full business flow implicitly validating externalized schemas function identically to original implementations.
Security Check	✅ Passed	Pure refactoring externalizing JSON schema definitions to separate files with compile-time embedding via include_str!(). Validation behavior, error handling, and security posture remain identical.

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

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch bzp/chore-externalize-entity-schema

---

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

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/config/entities/models.rs`:
- Around line 20-21: The SCHEMA static currently wraps the file text as a JSON
string via json!(include_str!("models-schema.json")), which yields a
Value::String and breaks jsonschema::validator_for(); change SCHEMA's
initializer to parse the file text with
serde_json::from_str(include_str!("models-schema.json")) and unwrap/expect on
parse errors so SCHEMA is a proper serde_json::Value (object/boolean) usable by
the validator (refer to SCHEMA and LazyLock in models.rs and the
models-schema.json asset).

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 0d5de95f-bf11-42ad-b8a2-aa3cefa6255b

📥 Commits

Reviewing files that changed from the base of the PR and between bcef7e1 and 78a3c59.

📒 Files selected for processing (4)

• src/config/entities/apikeys-schema.json
• src/config/entities/apikeys.rs
• src/config/entities/models-schema.json
• src/config/entities/models.rs