Skip to content

feat: session-configurable reasoning effort + media provider aliases - #459

Merged
rita-aga merged 1 commit into
mainfrom
claude/mirror-genesis-os-apps
Aug 12, 2026
Merged

feat: session-configurable reasoning effort + media provider aliases#459
rita-aga merged 1 commit into
mainfrom
claude/mirror-genesis-os-apps

Conversation

@rita-aga

@rita-aga rita-aga commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator

Two clean features on top of current main (the earlier blind Genesis→GitHub mirror was reverted — see the divergence note below).

1. Session-configurable reasoning effort (provider_caller): the codex/OpenAI Responses call reads reasoning_effort from the session's provider_options_json ({"reasoning_effort":"minimal|low|medium|high"}); absent/invalid keeps the prior hardcoded medium. katagami synthesis will set high — the hardcoded medium is the prime suspect for identical/safe design output across independent runs of a brief.

2. Media provider aliases + hint (paw-media): openai (and obvious spellings) → openai_codex, the sole v1 provider. Both observed hero-image failures (Mullion, Madori) burned a first attempt on the openai guess. The Generate action hint names valid values so the Code-Mode typed reference surfaces the contract.

Both wasm rebuilt; module + paw_fs_hot_path/session_turn_architecture/datadog_observability_contract suites green.


⚠️ Divergence discovered (needs a decision, not part of this PR): production paw-agent installs from a long-lived Genesis branch codex/paw-agent-main-session-storage-20260621 that never merged back to GitHub main. main and that branch have diverged bidirectionally — main is ahead on some modules (e.g. artifact_batch_apply bounded_reads), the deployment branch is ahead on others (workspace_fs resolver fix, monty_repl files channel, typed-session Fail steering — my earlier work, currently only on Genesis). A blind mirror regresses main; a blind reverse regresses production. Reconciling the two lineages (or picking one canonical) is an architectural call (ARN-69 territory). Filed for follow-up.

🤖 Generated with Claude Code

Greptile Summary

This PR makes OpenAI reasoning effort session-configurable and expands accepted media-provider aliases.

  • Reads an allowlisted reasoning_effort from provider_options_json, retaining medium as the default.
  • Advertises media-provider values and normalizes additional OpenAI Codex aliases.

Confidence Score: 5/5

The PR appears safe to merge because no blocking failure remains within the eligible follow-up scope.

No blocking failure remains.

Important Files Changed

Filename Overview
os-apps/paw-agent/wasm/provider_caller/src/lib.rs Adds allowlisted reasoning-effort selection to OpenAI Responses requests and unit coverage for defaults and supported values.
os-apps/paw-media/specs/media_generation.ioa.toml Expands the Generate action hint with supported provider and operation guidance.
os-apps/paw-media/wasm/openai_codex_image_generate/src/lib.rs Adds common OpenAI Codex provider spellings to image-generation request normalization.

Reviews (2): Last reviewed commit: "feat(paw-agent,paw-media): session-confi..." | Re-trigger Greptile

Comment thread os-apps/paw-agent/wasm/sandbox_provisioner/src/lib.rs Outdated
…ia provider aliases

- provider_caller: the OpenAI/codex Responses call reads reasoning effort
  from the session's provider_options_json ({"reasoning_effort":
  "minimal|low|medium|high"}); absent/invalid keeps the prior hardcoded
  medium. Unit-tested.
- paw-media: normalize_provider accepts 'openai' (and obvious spellings) as
  aliases for the sole v1 provider openai_codex — both observed hero-image
  failures burned a first attempt on the 'openai' guess. Generate action
  hint names the valid values so generated typed references surface it.
- Both wasm modules rebuilt.

Applied on top of current main (NOT a Genesis mirror — see PR notes on the
main/deployment-branch divergence).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Vw8tCkfnUW8p149yUNfSEE
@rita-aga
rita-aga force-pushed the claude/mirror-genesis-os-apps branch from 2042ded to 9e76725 Compare August 12, 2026 16:42
@rita-aga rita-aga changed the title Mirror deployed Genesis os-app state; configurable reasoning effort; media provider aliases feat: session-configurable reasoning effort + media provider aliases Aug 12, 2026
@rita-aga
rita-aga merged commit 3778951 into main Aug 12, 2026
2 checks passed
@rita-aga
rita-aga deleted the claude/mirror-genesis-os-apps branch August 12, 2026 17:01
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