fix(codex): priority-steer mid-turn user messages + friendly placeholder #71

Merged
devops-engineer merged 2 commits from fix/codex-chat-priority-steer into main 2026-06-01 08:17:22 +00:00
Member

What

Approach B for the codex 'My Chat' priority problem: a message arriving while a turn is in flight (e.g. during a continuous autonomous tick) was steered into the running turn and the agent often kept doing self-directed work without surfacing a reply — the canvas user saw the cryptic '[steered into in-flight turn ...]' placeholder and no answer.

Change (executor.py turn/steer path)

  1. Prepend an explicit priority directive to the steered input: if the arrival is from the user, reply promptly via send_message_to_user before continuing; if it is a delegated/peer request, address it appropriately.
  2. Replace the internal placeholder with a human-readable line: 'Got your message — I'm mid-task right now and will reply here shortly.'

The directive is phrased to be correct for both user and peer arrivals (the platform does not yet reliably tag canvas_user on inbound — RFC#637 is blocked).

Test

tests/test_executor.py::test_steer_injects_priority_directive_and_friendly_placeholder — holds the turn lock to simulate an in-flight turn, asserts the steered turn/steer input carries the directive + the user text, and that the friendly placeholder (not the old jargon) reaches the canvas.

Follow-up: will verify live that a reply lands during an active autonomous tick before closing the task.

## What Approach B for the codex 'My Chat' priority problem: a message arriving while a turn is in flight (e.g. during a continuous autonomous tick) was steered into the running turn and the agent often kept doing self-directed work without surfacing a reply — the canvas user saw the cryptic '[steered into in-flight turn ...]' placeholder and no answer. ## Change (executor.py turn/steer path) 1. Prepend an explicit priority directive to the steered input: if the arrival is from the user, reply promptly via send_message_to_user before continuing; if it is a delegated/peer request, address it appropriately. 2. Replace the internal placeholder with a human-readable line: 'Got your message — I'm mid-task right now and will reply here shortly.' The directive is phrased to be correct for both user and peer arrivals (the platform does not yet reliably tag canvas_user on inbound — RFC#637 is blocked). ## Test tests/test_executor.py::test_steer_injects_priority_directive_and_friendly_placeholder — holds the turn lock to simulate an in-flight turn, asserts the steered turn/steer input carries the directive + the user text, and that the friendly placeholder (not the old jargon) reaches the canvas. Follow-up: will verify live that a reply lands during an active autonomous tick before closing the task.
devops-engineer added 2 commits 2026-06-01 04:02:38 +00:00
fix(codex): priority-steer mid-turn messages + friendly placeholder
CI / Template validation (static) (push) Waiting to run
CI / Template validation (runtime) (push) Blocked by required conditions
CI / T4 tier-4 conformance (live) (push) Blocked by required conditions
CI / validate (push) Blocked by required conditions
CI / Adapter unit tests (push) Waiting to run
9519c0e49d
fix(codex): priority-steer mid-turn messages + friendly placeholder
CI / Template validation (static) (pull_request) Successful in 32s
CI / Adapter unit tests (pull_request) Successful in 22s
verify-providers-projection / Regenerate projection, fail on drift, assert registry ⊆ template (pull_request) Successful in 1m40s
CI / Template validation (runtime) (push) Successful in 2m15s
CI / Adapter unit tests (push) Successful in 22s
CI / Template validation (static) (push) Successful in 32s
CI / T4 tier-4 conformance (live) (push) Successful in 1m22s
CI / validate (push) Successful in 3s
CI / validate (pull_request) Has been cancelled
CI / T4 tier-4 conformance (live) (pull_request) Has been cancelled
CI / Template validation (runtime) (pull_request) Has been cancelled
eb6d5c1c2e
devops-engineer added 1 commit 2026-06-01 04:02:39 +00:00
fix(codex): priority-steer mid-turn messages + friendly placeholder
CI / Template validation (static) (push) Waiting to run
CI / Template validation (runtime) (push) Blocked by required conditions
CI / T4 tier-4 conformance (live) (push) Blocked by required conditions
CI / validate (push) Blocked by required conditions
CI / Adapter unit tests (push) Waiting to run
9519c0e49d
devops-engineer added 1 commit 2026-06-01 04:02:40 +00:00
fix(codex): priority-steer mid-turn messages + friendly placeholder
CI / Template validation (static) (pull_request) Successful in 32s
CI / Adapter unit tests (pull_request) Successful in 22s
verify-providers-projection / Regenerate projection, fail on drift, assert registry ⊆ template (pull_request) Successful in 1m40s
CI / Template validation (runtime) (push) Successful in 2m15s
CI / Adapter unit tests (push) Successful in 22s
CI / Template validation (static) (push) Successful in 32s
CI / T4 tier-4 conformance (live) (push) Successful in 1m22s
CI / validate (push) Successful in 3s
CI / validate (pull_request) Has been cancelled
CI / T4 tier-4 conformance (live) (pull_request) Has been cancelled
CI / Template validation (runtime) (pull_request) Has been cancelled
eb6d5c1c2e
devops-engineer closed this pull request 2026-06-01 05:04:53 +00:00
devops-engineer reopened this pull request 2026-06-01 05:04:56 +00:00
core-lead approved these changes 2026-06-01 08:17:17 +00:00
core-lead left a comment
Member

Approve: chat-priority steer + friendly placeholder; adapter unit test + all substantive CI passed on eb6d5c1c (push-event); orphaned pull_request-event duplicates reconciled.

Approve: chat-priority steer + friendly placeholder; adapter unit test + all substantive CI passed on eb6d5c1c (push-event); orphaned pull_request-event duplicates reconciled.
core-be approved these changes 2026-06-01 08:17:18 +00:00
core-be left a comment
Member

Approve: chat-priority steer + friendly placeholder; adapter unit test + all substantive CI passed on eb6d5c1c (push-event); orphaned pull_request-event duplicates reconciled.

Approve: chat-priority steer + friendly placeholder; adapter unit test + all substantive CI passed on eb6d5c1c (push-event); orphaned pull_request-event duplicates reconciled.
devops-engineer merged commit f384a15af2 into main 2026-06-01 08:17:22 +00:00
Sign in to join this conversation.
No Reviewers
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: molecule-ai/molecule-ai-workspace-template-codex#71