Skip to content

feat(whaleflow): add model role policy registry#2830

Merged
Hmbown merged 1 commit into
codex/v0.9.0-stewardshipfrom
codex/v090-whaleflow-model-policy
Jun 6, 2026
Merged

feat(whaleflow): add model role policy registry#2830
Hmbown merged 1 commit into
codex/v0.9.0-stewardshipfrom
codex/v090-whaleflow-model-policy

Conversation

@Hmbown
Copy link
Copy Markdown
Owner

@Hmbown Hmbown commented Jun 6, 2026

Summary

  • Adds ModelRole, ModelCapabilities, and a crate-local ProviderRegistry for role/capability-based WhaleFlow model selection.
  • Adds ModelProvider, CompletionRequest, CompletionResponse, and a MockModelProvider foundation without live provider calls.
  • Adds fail-closed parse_json_with_repair / repair_json_text_once for the WhaleFlow: model-provider registry and role-based model policy #2672 JSON fallback shape.
  • Covers role defaults, provider capability fallback, agent-role mapping, mock provider response, and JSON repair/fail-closed tests.

Stewardship Notes

Verification

  • cargo test -p codewhale-whaleflow --locked
  • cargo clippy -p codewhale-whaleflow --all-targets --locked -- -D warnings
  • cargo fmt --all -- --check
  • ./scripts/release/check-versions.sh
  • ./scripts/release/check-ohos-deps.sh
  • git diff --check

@gemini-code-assist
Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmbown has reached the 50-review limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@Hmbown Hmbown merged commit 5044a29 into codex/v0.9.0-stewardship Jun 6, 2026
2 checks passed
@Hmbown Hmbown deleted the codex/v090-whaleflow-model-policy branch June 6, 2026 05:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant