[codex] namespace sleep under clock

[rka-oai]

Summary

• expose the interruptible sleep tool as clock.sleep instead of top-level sleep
• keep clock.curr_time and clock.sleep in the same model-visible namespace when both features are enabled
• update existing core and app-server integration coverage to issue namespaced sleep calls

Why

Sleep is a clock operation. Grouping it with clock.curr_time gives the model a more coherent tool surface without changing the sleep feature gate or runtime behavior.

Validation

• just test -p codex-core sleep_tool_follows_feature_gate
• just test -p codex-core any_new_input_interrupts_sleep
• just test -p codex-app-server sleep_emits_started_and_completed_items

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: b512044faf

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

Preserve legacy sleep hook matching

When hooks are enabled and a user already has a PreToolUse/PostToolUse matcher for sleep, this rename changes the hook-facing name from sleep to clocksleep: the default function hook payload uses flat_tool_name(&invocation.tool_name), and the hook engine matches only that canonical name plus explicit aliases. Because SleepHandler does not add a sleep matcher alias, existing hooks that used to audit or block the sleep tool stop running as soon as the model calls the new clock.sleep; please preserve sleep as a matcher alias or override the hook payload while keeping the model-visible namespace.

Useful? React with 👍 / 👎.