Skip to content

fix(resources): disable unsafe citizen workspace eviction - #3921

Merged
joelteply merged 1 commit into
canaryfrom
af44428c/disable-unsafe-citizen-workspace-evictio
Sep 8, 2026
Merged

fix(resources): disable unsafe citizen workspace eviction#3921
joelteply merged 1 commit into
canaryfrom
af44428c/disable-unsafe-citizen-workspace-evictio

Conversation

@joelteply

@joelteply joelteply commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

The citizen workspace pool added in #3920 can delete the only copy of clean, unpushed Persona autosaves: preservation checks git status, skips clean repositories, and does not preserve local commit objects or refs. Its synchronous snapshot also recursively scans citizen trees during broker status reads.

Remove the pool's production registration until preservation and accounting are repaired. The generic disk-pressure monitor only reports pressure; it cannot delete files. Existing pool implementation and its repair work remain available; this change stops the broker from invoking it.

Validation: independent source review audited all CitizenWorkspacePool references and the non-deleting generic disk-pressure monitor; the removed registration was its only production construction. Diff checks pass. This containment does not claim to repair archival or recover already-deleted data. No live processes were changed.

AIRC card af44428c-fda2-4405-878a-dbaa85af311a. Blocking review: #3920 (comment)

@joelteply
joelteply enabled auto-merge (squash) September 8, 2026 19:42
@joelteply
joelteply merged commit 3972123 into canary Sep 8, 2026
5 checks passed
@joelteply
joelteply deleted the af44428c/disable-unsafe-citizen-workspace-evictio branch September 8, 2026 19:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant