[rycerzes]

Env apps pass reset_observation_cls only when the installed create_app accepts it, so images built from their own lockfiles (openenv 0.3.1/0.4.1) start again.

Tested: echo_env and finqa_env built from their Dockerfiles start and return 200 on /health; new tests import each app against both create_app signatures.

Fixes #1339

---

Note

Low Risk
Compatibility-only startup logic with regression tests; no change to env behavior when a current openenv-core is installed.

Overview
Fixes env server startup when Docker images pin older openenv-core (e.g. 0.3.1/0.4.1) that do not accept reset_observation_cls or state_cls on create_app.

Each affected env server/app.py now builds optional kwargs with inspect.signature(create_app) and only passes reset_observation_cls=Observation and the env-specific state_cls when those parameters exist. On newer core, behavior is unchanged; on older core, import no longer fails with unexpected keyword arguments.

Adds tests/envs/test_reset_observation_compat.py to re-import all five env apps under mocked “released” and “current” create_app signatures, asserting apps load and still forward Observation / state models when supported.

^{Reviewed by Cursor Bugbot for commit e4c40be. Bugbot is set up for automated code reviews on this repo. Configure here.}

[burtenshaw]

Reviewed the compatibility guard and preserved current-main removal of Pi/OpenCode during refresh. All 10 import tests pass against old and current create_app signatures; touched lint, format, and import sorting pass.

[bot-ci-comment]

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

[burtenshaw]

Refreshed against main after #1198. The compatibility wrapper now checks support for state_cls and reset_observation_cls independently, preserving each environment's concrete State model on current core while keeping older core signatures working.

Verified at e4c40be: 39 compatibility/State-wiring tests pass; touched-file lint, formatting, and import ordering pass. Independent review is checking the mixed-signature cases as well.

[burtenshaw]

Reviewed the #1198 integration. Reset and State keyword support are gated independently; the strict old-core stub remains unchanged. Independent validation passed all 39 compatibility/State tests plus 8 reset-only and State-only signature cases, with touched-file lint clean.