Skip to content

fix: allow SessionStart hook more time to restore state - #672

Open
Hughhhhcoder wants to merge 1 commit into
openai:mainfrom
Hughhhhcoder:fix/session-start-hook-timeout-670
Open

fix: allow SessionStart hook more time to restore state#672
Hughhhhcoder wants to merge 1 commit into
openai:mainfrom
Hughhhhcoder:fix/session-start-hook-timeout-670

Conversation

@Hughhhhcoder

Copy link
Copy Markdown

Summary

  • increase the Codex SessionStart hook timeout from 5 seconds to 60 seconds
  • add a manifest-level regression test so the timeout does not regress

Fixes #670.

Validation

  • npm test (92 passed)
  • npx tsc -p tsconfig.app-server.json --noEmit
  • python3 -m json.tool plugins/codex/hooks/hooks.json
  • git diff --check

npm run build could not run because the local prebuild command resolves a missing global Codex binary (codex app-server generate-ts, ENOENT).

@Hughhhhcoder
Hughhhhcoder requested a review from a team August 23, 2026 06:14
@Hughhhhcoder

Copy link
Copy Markdown
Author

Status update: the current head remains 592b7ac7f1a93545922cad04edbe1bd49f8da9df; local npm test passed (91 tests) and tsc -p tsconfig.app-server.json --noEmit passed. GitHub Pull Request CI is currently action_required with zero jobs, so maintainer approval is needed before CI can run. The full build remains locally blocked only by the prebuild dependency on a global codex app-server generate-ts binary.

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.

SessionStart hook is killed by its own 5s timeout, silently dropping CODEX_COMPANION_SESSION_ID

1 participant