Merged
Conversation
Added a warning to inform users that they must restart their bash shell after installation to apply the changes.
- Revise naming conventions, activation commands, and installation guidelines- Modify shim templates and contributor documentation for POSIX compliance - Adjust architecture docs to reflect approved POSIX rewrite decisions - Remove Bash-specific references and symlink install mode from documentation
Simplify install to single-root layout: - Remove symlink install mode - Remove exported Shimmy path variables - Derive all paths (shim, images, lib) from install root Rewrite scripts in POSIX shell for improved portability. Activation no longer exports Shimmy path variables (SHIMMY_*); only updates PATH.
…ation workflow Rewrite the compact restart brief to provide clear step-by-step instructions for implementing approved decisions. Add structured design decisions section and clarify checklist management process.
prompts. New questions focus on code acceptance, design decision approval, and documenting implementation findings for follow-up.
…naging shell startup files. Integrate startup file management into `install` and `update` commands. The `install` command now updates the user's shell startup file by default, with options to override the shell, specify a custom startup file, or skip the update. The `update` command gains a `--repair-startup` option to rewrite the managed startup block. Update documentation and tests to reflect the new behavior and remove references to the old onboarding helper.
Delete the tracked Bash-only helpers under lib/repo/ Update CONTRIBUTING.md and README.md to reflect the new shimmy-startup.sh helper and clarify startup file behavior. Refactor dash parser checks in test-shimmy.sh to dynamically discover tracked shell files instead of maintaining a hardcoded list. Mark completed design decisions and removal task in the session checklist.
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
1 participant
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.
Conversion of BASH based implementation to Posix for macOS compatibility