Skip to content

Add Inworld cloud provider metadata - #112

Open
oleksa-source wants to merge 1 commit into
tinyhumansai:mainfrom
oleksa-source:codex/inworld-provider-metadata
Open

Add Inworld cloud provider metadata#112
oleksa-source wants to merge 1 commit into
tinyhumansai:mainfrom
oleksa-source:codex/inworld-provider-metadata

Conversation

@oleksa-source

Copy link
Copy Markdown

Summary

Add the host metadata required to represent Inworld as a built-in OpenAI-compatible provider. The catalog now exposes the canonical Inworld endpoint with explicit HTTP Basic authentication and classifies its host as chat-completions-only.

Related issue

None.

API or behavior changes

Adds the additive AuthStyle::Basic public variant with wire value basic and adds the Inworld built-in provider preset. Existing auth styles and provider metadata remain unchanged.

Validation

Commands actually run, with their outcome:

  • cargo fmt --all -- --check
  • cargo clippy --all-targets --all-features -- -D warnings
  • cargo build --all-targets --all-features
  • cargo test --all-features

Tests

Extended catalog, legacy migration, auth-style serialization, built-in host classification, and Responses API fallback coverage. No deliberately untested edge cases.

Documentation

Added docs/specs/inworld-cloud-provider.md and its linked implementation plan.

Checklist

  • The change is focused on one logical change
  • No new allow attributes, ignored tests, or relaxed lints
  • No secrets, tokens, or environment contents in the diff or description

@coderabbitai

coderabbitai Bot commented Aug 27, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: bb575605-7c42-45f9-a001-c135083fb7e7

Warning

Your free Security trial is over. An organization admin can activate Security or dismiss this notice.


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

@tinysweeper tinysweeper Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

tinysweeper found nothing blocking. Approving.

$0.0000 · 0 in / 0 out

@tinysweeper

tinysweeper Bot commented Aug 27, 2026

Copy link
Copy Markdown

How this change flows

5 changed behaviours across 9 relationships. 6 surrounding behaviours are shown (60 graph nodes walked). 26 further behaviours left out to keep the diagram readable.

flowchart LR
  n0["AuthStyle<br/>changed"]:::changed
  n1["BuiltinCloudProvider<br/>changed"]:::changed
  n2["builtin_cloud_provider_slugs_are_unique<br/>changed"]:::changed
  n3["...cloud_provider_recognises_inference_hosts<br/>changed"]:::changed
  n4["..._chat_completions_only_regardless_of_slug<br/>changed"]:::changed
  n5["assert"]:::impacted
  n6["CloudProviderCreds"]:::impacted
  n7["builtin_cloud_provider"]:::impacted
  n8["migrate_legacy_fields"]:::impacted
  n9["endpoint_host_is_chat_completions_only"]:::impacted
  n10["as_str"]:::impacted
  n1 -->|uses| n0
  n2 -->|calls| n5
  n3 -->|calls| n5
  n4 -->|calls| n5
  n6 -->|uses| n0
  n7 -->|uses| n1
  n8 -->|uses| n6
  n8 -->|calls| n7
  n9 -->|calls| n10
  classDef changed fill:#0d4429,stroke:#238636,color:#e6edf3
  classDef impacted fill:#161b22,stroke:#6e7681,color:#c9d1d9
  classDef flagged fill:#5a1e02,stroke:#d93f0b,color:#ffffff
  classDef blocking fill:#67060c,stroke:#f85149,color:#ffffff
Loading

Green: changed behaviour. Grey: surrounding behaviour. Arrows name the call, use, implementation, or test relationship. Orange: has findings. Red: has a finding that blocks the merge.

tinysweeper 0.1.0

@tinysweeper tinysweeper Bot added the priority: p3 Whenever. Cosmetic, a nicety, or a cleanup with no user visible effect. label Aug 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority: p3 Whenever. Cosmetic, a nicety, or a cleanup with no user visible effect.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant