Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

packages/app/src/context/global-sync/child-store.ts (1)

133-164: ⚠️ Potential issue | 🟠 Major

Roll back the earlier persisted caches when later setup fails.

If project or icon creation throws, the earlier createPersistedChildCache(...) calls have already bound their persistence work to input.owner, but this path has no rollback/dispose step for those partial successes. The cache maps stay empty now, but retries can still accumulate orphaned persisted listeners/writers for the same workspace:* keys. Please create the three persisted caches under a disposable per-directory root and tear that root down on failure before rethrowing.

#!/bin/bash
set -euo pipefail

echo "== child-store setup path =="
fd '^child-store\.ts$' packages/app/src/context/global-sync -x sed -n '130,230p' {}

echo
echo "== persisted factory return contract =="
fd '^persist\.ts$' packages/app/src -x sed -n '1,80p' {}
fd '^persist\.ts$' packages/app/src -x sed -n '485,620p' {}

Expected result: ensureChild() creates vcs, project, and icon sequentially before any directory-scoped disposer is registered, while persisted(...) returns only the 4-tuple and does not give this call site a way to tear down already-created partial successes. As per coding guidelines "Always prefer createStore over multiple createSignal calls in SolidJS".

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@packages/app/src/context/global-sync/child-store.ts` around lines 133 - 164,
The createPersistedChildCache calls in ensureChild() (creating vcs, meta, icon)
can partially succeed and leave persisted listeners bound to input.owner; wrap
these three creates in a directory-scoped disposable root (e.g. create a
per-directory disposer or CompositeDisposable) so that if any
createPersistedChildCache (vcs, project, icon) throws you call that disposer to
tear down previously-created caches (unbind persistence for Persist.workspace
entries) before rethrowing; update ensureChild() to register the successful
directory-scoped disposer with the parent only after all three caches are
created, and reference the existing symbols createPersistedChildCache,
Persist.workspace, input.owner, and createStore to locate the changes.