Repository navigation
fix(debug): keep allocation progress visible - #255
Conversation
Co-authored-by: OpenAI Codex <codex@openai.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 227e095b16
ℹ️ 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 (@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 (@codex) address that feedback".
| elif now - last_progress_at >= _STATUS_HEARTBEAT_SECS: | ||
| elapsed_secs = int(now - started_at) | ||
| print( | ||
| f"Debug worker: {status} (still waiting; {elapsed_secs}s elapsed)", | ||
| flush=True, |
There was a problem hiding this comment.
Document the new allocation heartbeat
The 30-second status heartbeat changes user-visible macrodata debug create output, but this change set does not update docs/running-pipelines/cloud-debug-sessions.md to explain the periodic progress message or when users should expect it. Add the corresponding user-first documentation alongside this behavior change.
AGENTS.md reference: AGENTS.md:L49-L52
Useful? React with 👍 / 👎.
Summary
Verification
.venv/bin/python -m pytest -q tests/cli/test_debug_commands.py(18 passed)ty checkpassed