Skip to content

fix(cleanup): scope active artifact process probes - #1005

Merged
chubes4 merged 1 commit into
mainfrom
fix-1004-active-artifact-scope
Aug 5, 2026
Merged

fix(cleanup): scope active artifact process probes#1005
chubes4 merged 1 commit into
mainfrom
fix-1004-active-artifact-scope

Conversation

@chubes4

@chubes4 chubes4 commented Aug 5, 2026

Copy link
Copy Markdown
Member

Summary

  • retry an incomplete global process scan against the candidate worktree and artifact roots
  • preserve fail-closed skips when scoped evidence is still incomplete
  • record candidate path, PID, match method, and confidence for active-process skips

Closes #1004

Tests

  • php tests/artifact-cleanup-live-process-guards.php
  • php tests/artifact-cleanup-reviewed-run.php
  • php tests/artifact-cleanup-plan-output-contract.php
  • php -l inc/Support/ProcessPathProbe.php
  • php -l inc/Workspace/WorkspaceArtifactCleanup.php
  • php -l tests/artifact-cleanup-live-process-guards.php

AI assistance

OpenAI openai/gpt-5.6-sol via OpenCode implemented the scoped probe, evidence fields, and focused tests. Chris Huber reviewed and owns the change.

Retry incomplete host-wide process evidence against each candidate's roots so sibling builds do not block safe artifact reclamation. Preserve fail-closed behavior when scoped evidence remains ambiguous.

AI assistance: OpenAI openai/gpt-5.6-sol via OpenCode implemented the scoped probe, evidence fields, and focused tests. Chris Huber reviewed and owns the change.
@chubes4
chubes4 merged commit 23b080d into main Aug 5, 2026
@chubes4
chubes4 deleted the fix-1004-active-artifact-scope branch August 5, 2026 22:45
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.

fix(cleanup): scope active-build evidence to the artifact worktree

1 participant