Skip to content

Add Migrator director for reversible data migrations - #963

Merged
TheGreatAxios merged 5 commits into
mainfrom
cl-7671-migrator-director
Sep 13, 2026
Merged

Add Migrator director for reversible data migrations#963
TheGreatAxios merged 5 commits into
mainfrom
cl-7671-migrator-director

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Collaborator

Summary

  • Adds a Migrator leaf director that plans reversible settings, config, and session-state data changes with forward migration, rollback, dry-run evidence, and in-flight session impact
  • Registers Migrator in the closed director set and the Skywalker spawn allowlist

Verification

  • bun run typecheck passes
  • bun run check passes except one flaky timing test in src/tui/tool-execution-watchdog.test.ts that passes in isolation
  • bun test ./src/agent/directors/ ./src/agent/prompt-sizes.test.ts ./src/agent/directors-prompt.test.ts passes (307 pass, 0 fail)

Fixes CL-7671

@linear-code

linear-code Bot commented Sep 13, 2026

Copy link
Copy Markdown

CL-7671

@TheGreatAxios

Copy link
Copy Markdown
Collaborator Author

Audit (Migrator director): Approve contingent on rename nit.

Findings:

run_shell is dry-run/scratch-copy ONLY: never execute the forward migration against live state, no background shells (shell_collect unmounted; foreground with timeouts only). Scratch copies live under tmp/, are cleaned up afterwards, and the scratch path is reported in the delivery. Rename the registry leaf test to dry-run-scoped and raise the migrator prompt budgets to 12800 chars / 13900 bytes (measured-max + 2000/+3000, ceiling 100).
@TheGreatAxios
TheGreatAxios force-pushed the cl-7671-migrator-director branch from 1df5bfa to 8749842 Compare September 13, 2026 05:44
@TheGreatAxios
TheGreatAxios merged commit f3659d6 into main Sep 13, 2026
9 checks passed
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.

1 participant