cleanup: retire the legacy /faber command bridge - #168
Merged
Conversation
Deploying ystack with
|
| Latest commit: |
32b553a
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://9ff7a7f9.fabrica-6yx.pages.dev |
| Branch Preview URL: | https://codex-retire-faber-command-b.fabrica-6yx.pages.dev |
Owner
Author
Codex reviewer (cross-vendor, read-only)Reviewed-head: 32b553a Posted verbatim by The installer, doctor, documentation, and regression tests consistently retire the legacy command while preserving existing files and symlinks. No blocking functional regressions were found. |
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.
Retire only the legacy command bridge
This completes the repository half of the accepted rename Ops 4 command-entry cleanup. It does not retire
.fabrica/**orFABRICA_*target compatibility and does not claim the full external Ops 4 checklist is complete.What changed
scripts/install.shcreates and updates only/yshifu./faberfilesystem entry still exists, including as a dangling symlink, installer warns loudly but never reads, backs up, overwrites, or deletes it.scripts/doctor.shrequires/yshifu; it no longer accepts the old command as fallback and fails while the retired entry remains./yshifu, inspect the old file, move it outside command discovery to a unique owner-controlled backup, run doctor/full smoke, and use no-clobber rollback.Safety and scope
~/.claude/commands/faber.mdand never changes~/.claude/settings.json..fabrica, legacy environment keys/markers, old domain redirect, settings cleanup, and historical artifacts are outside scope.Proof
Verified in a clean detached worktree at
32b553a285de48cb1579f19720a5076c39bb3ab9:git diff --check: passed.Closes #167