mirror of
https://github.com/openai/codex.git
synced 2026-05-23 12:34:25 +00:00
0d344aca9b
Addresses #22833, #22245, #23067 ## Why `/goal` can keep synthesizing turns even when the next turn cannot make meaningful progress. Hard usage exhaustion can replay failing turns, and repeated permission or external-resource blockers can keep burning tokens while waiting for user or system intervention. ## What changed - Add resumable `blocked` and `usageLimited` goal states. As with `paused`, goal continuation stops with these states. - Move to `usageLimited` after usage-limit failures. - Allow the built-in `update_goal` tool to set `blocked` only under explicit repeated-impasse guidance. Updated goal continuation prompt to specify that agent should use `blocked` only when it has made at least three attempts to get past an impasse. Most of the files touched by this PR are because of the small app server protocol update. ## Validation I manually reproduced a number of situations where an agent can run into a true impasse and verified that it properly enters `blocked` state. I then resumed and verified that it once again entered `blocked` state several turns later if the impasse still exists. I also manually reproduced the usage-limit condition by creating a simulated responses API endpoint that returns 429 errors with the appropriate error message. Verified that the goal runtime properly moves the goal into `usageLimited` state and TUI UI updates appropriately. Verified that `/goal resume` resumes (and immediately goes back into `ussageLImited` state if appropriate). ## Follow-up PRs Small changes will be needed to the GUI clients to properly handle the two new states.
49 lines
1.0 KiB
SQL
49 lines
1.0 KiB
SQL
PRAGMA foreign_keys=OFF;
|
|
|
|
CREATE TABLE thread_goals_new (
|
|
thread_id TEXT PRIMARY KEY NOT NULL REFERENCES threads(id) ON DELETE CASCADE,
|
|
goal_id TEXT NOT NULL,
|
|
objective TEXT NOT NULL,
|
|
status TEXT NOT NULL CHECK(status IN (
|
|
'active',
|
|
'paused',
|
|
'blocked',
|
|
'usage_limited',
|
|
'budget_limited',
|
|
'complete'
|
|
)),
|
|
token_budget INTEGER,
|
|
tokens_used INTEGER NOT NULL DEFAULT 0,
|
|
time_used_seconds INTEGER NOT NULL DEFAULT 0,
|
|
created_at_ms INTEGER NOT NULL,
|
|
updated_at_ms INTEGER NOT NULL
|
|
);
|
|
|
|
INSERT INTO thread_goals_new (
|
|
thread_id,
|
|
goal_id,
|
|
objective,
|
|
status,
|
|
token_budget,
|
|
tokens_used,
|
|
time_used_seconds,
|
|
created_at_ms,
|
|
updated_at_ms
|
|
)
|
|
SELECT
|
|
thread_id,
|
|
goal_id,
|
|
objective,
|
|
status,
|
|
token_budget,
|
|
tokens_used,
|
|
time_used_seconds,
|
|
created_at_ms,
|
|
updated_at_ms
|
|
FROM thread_goals;
|
|
|
|
DROP TABLE thread_goals;
|
|
ALTER TABLE thread_goals_new RENAME TO thread_goals;
|
|
|
|
PRAGMA foreign_keys=ON;
|