Skip to content

fix: exclude DiskSage from active-use probes - #322

Draft
seonghobae wants to merge 10 commits into
feat/generated-cache-symlink-manifest-v1from
fix/active-use-self-pid-v1
Draft

fix: exclude DiskSage from active-use probes#322
seonghobae wants to merge 10 commits into
feat/generated-cache-symlink-manifest-v1from
fix/active-use-self-pid-v1

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

Outcome

  • exclude the current DiskSage PID and bounded probe PIDs from active-use results
  • preserve every external handle and command-path blocker
  • document the live self-blocking cache-audit gap and decision boundary

Live proof

Before the fix, each generated-cache audit reported its own transient CLI PID as process-active. After the fix, the inactive Homebrew API/bootsnap and Playwright caches produced complete blocker-free fresh plans; active uv remained protected by external processes and its lock.

Validation

  • cargo test --manifest-path src-tauri/Cargo.toml active_use_pid_filter_excludes_probe_and_disksage_processes --lib
  • fresh plan and immediate re-plan through disksage-generated-cache-reclaim
  • git diff --check

Stacked on #320.


Devin Review

Protected prerequisite status

This PR remains Draft while the generated-cache owner PR #295 and its protected safety prerequisites remain Proposed and unmerged. Parent-handle checks reduce a reproduced gap; they do not prove exclusion of arbitrary late writers. Atomic no-replace restoration and current-head validation remain acceptance work. Unique changes are preserved through normal non-force parent integration; this status change is not a close or a shipped-safety claim.

@coderabbitai

coderabbitai Bot commented Aug 30, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

devin-ai-integration[bot]

This comment was marked as resolved.

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Devin Review found 1 new potential issue.

Devin Review

Comment thread src-tauri/tests/active_use_self_pid.rs
@seonghobae
seonghobae marked this pull request as draft September 6, 2026 15:55
@seonghobae seonghobae added bug Something isn't working priority: medium Normal-priority or P2 work status: draft Draft pull request type: bug Defect or incorrect behavior labels Sep 6, 2026 — with ChatGPT Codex Connector
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working priority: medium Normal-priority or P2 work status: draft Draft pull request type: bug Defect or incorrect behavior

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant