Skip to content

Test selected capabilities across unavailable resume#30215

Merged
jif-oai merged 4 commits into
mainfrom
jif/test-selected-capabilities-resume-unavailable
Jun 26, 2026
Merged

Test selected capabilities across unavailable resume#30215
jif-oai merged 4 commits into
mainfrom
jif/test-selected-capabilities-resume-unavailable

Conversation

@jif-oai

@jif-oai jif-oai commented Jun 26, 2026

Copy link
Copy Markdown
Collaborator

Why

The selected-capability integration test already covers initial attachment and cold resume, but it resumes while the selected executor is still reachable.

That leaves an important World State transition untested: a thread remembers its selected capability root, resumes while that environment is unavailable, and later sees the same stable environment return.

What this tests

This extends the existing end-to-end scenario:

selected executor available
        ↓
app-server stops and the executor goes away
        ↓
thread resumes with the executor unavailable
        ↓
skills, selected MCP tools, and connector attribution are absent
        ↓
the same environment ID is attached again
        ↓
skills, MCP tools, and connector attribution return

The test also checks that the unavailable snapshot explicitly tells the model that no selected-environment skills are currently available. After reattachment, it invokes the selected skill again and verifies that a new executor-owned MCP process starts.

Scope

This is test-only. It keeps the existing assumption that an environment ID refers to stable capability contents. It does not add package-file invalidation or live transport reconnect behavior.

@jif-oai jif-oai merged commit 3c03bb4 into main Jun 26, 2026
31 checks passed
@jif-oai jif-oai deleted the jif/test-selected-capabilities-resume-unavailable branch June 26, 2026 10:02
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 26, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant