Skip to content

fix: report active jobs from explicit result lookups#504

Open
NgoQuocViet2001 wants to merge 1 commit into
openai:mainfrom
NgoQuocViet2001:fix-498-active-result-message
Open

fix: report active jobs from explicit result lookups#504
NgoQuocViet2001 wants to merge 1 commit into
openai:mainfrom
NgoQuocViet2001:fix-498-active-result-message

Conversation

@NgoQuocViet2001

Copy link
Copy Markdown

Fixes #498

Summary

  • Resolve an explicit job reference before filtering by terminal status.
  • Report queued or running jobs as active instead of incorrectly saying they do not exist.
  • Add regression coverage for result <job-id> against a running job.

Testing

  • node --test --test-name-pattern="result" tests/runtime.test.mjs (7 passed)
  • codex app-server generate-ts --out plugins/codex/.generated/app-server-types
  • tsc -p tsconfig.app-server.json

@NgoQuocViet2001
NgoQuocViet2001 requested a review from a team July 15, 2026 07:30
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.

result <id> on a queued/running job reports "No job found" — the still-running message is unreachable when a reference is given

1 participant