Skip to content

Fix active no-signal drain budget continuations#829

Merged
chubes4 merged 1 commit into
mainfrom
issue-823-safe-drain
Jun 27, 2026
Merged

Fix active no-signal drain budget continuations#829
chubes4 merged 1 commit into
mainfrom
issue-823-safe-drain

Conversation

@chubes4

@chubes4 chubes4 commented Jun 27, 2026

Copy link
Copy Markdown
Member

Summary

  • Add explicit active/no-signal drain continuations when the budget expires before the next safe stage or after a bounded cleanup pass.
  • Return a clear continuation reason, hint, stage, offset, and next command instead of falling through to finalization.
  • Cover budget-boundary continuation behavior in the abandoned cleanup orchestrator smoke test.

Closes #823

Tests

  • php -l inc/Workspace/WorkspaceAbandonedCleanupOrchestrator.php
  • php -l tests/smoke-abandoned-cleanup-orchestrator.php
  • php tests/smoke-abandoned-cleanup-orchestrator.php
  • php tests/workspace-compact-output.php
  • php tests/smoke-cleanup-eligible-drain.php
  • git diff --check

AI assistance

  • AI assistance: Yes
  • Tool(s): openai/gpt-5.5 and OpenCode
  • Used for: Implementing the active/no-signal drain budget-continuation fix, updating focused smoke coverage, and running verification.

@chubes4 chubes4 merged commit af0f74d into main Jun 27, 2026
4 of 5 checks passed
@chubes4 chubes4 deleted the issue-823-safe-drain branch June 27, 2026 02:16
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.

active-no-signal drain should make continuation feel like a true drain

1 participant