feat: ship reversible identity v2 migration - #13
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
AGENTS.mdthe new-install identity surface, including the SOUL v2 schema slotsrules/orchestration.md§11 (R1–R5) and route it fromrules/INDEX.mdmigrate-identitypreview/apply/rollback path for existingAGENTS.mdand legacySOUL.mdinstallsMigration safety
AGENTS.md.v2candidate, and SHA-bound receiptSOUL.mduntil an operator-reviewed cutoverVerification
npm test— 252/252 passinit --non-interactive/doctor --non-interactive/ clean-diff smoke — passnpm pack --dry-run --json— pass, 300 packaged filesgit diff --check— passWindows CI remediation
The first PR run exposed failures already present on base
master(baseline run 31664945809): a POSIX-only mode-bit assertion, locale-dependent JSON I/O, and an unnecessary rewrite of an already-active matching flow that could race on Windows rename semantics. Commits7322b30,e937601, anda37e1a4preserve the POSIX contract off Windows, enforce UTF-8 input plus ASCII-safe JSON output, add Korean cp1252 regression coverage, and make matching-flow start read-only/idempotent while retaining new-flow writes and different-flow denial.Independent review
3d2c6e4AI_Second_Brain/100-project/2026-08-16-bot-rules-docs-redesign/30-execution-log.mdCoordinates
e9e37e39e478c00250e95b4f63b8eb8cb10c1f66a37e1a46a5cd74c915cd2b7152438d29e65f9286c58338961e17e0bcecca67ffdec3c47d8e35853c7dcc7e1b3ec334e86bdac921Draft only. Do not merge until the BRD final review and operator decision.