Remove team shell aliases and delete .devcontainer/config/#9
Merged
Conversation
The shell customization mechanism (sourced *.shared.sh / *.local.sh files) was purely an interactive convenience — no script, compose file, or CI job ever invoked the aliases. After the stacks-colocation refactor, config/shell/ was the last remaining occupant of .devcontainer/config/. - Delete .devcontainer/config/ (aliases.shared.sh, shell README). - Drop setup_shell_customization() and its call from post-create.sh so new containers no longer bake a sourcing block into ~/.zshrc. - Replace the MOTD "Quick Reference" alias list with the real full commands (docker compose ..., git status/log/diff, task, claude); drop the Tips lines pointing at config/shell/. - Remove the now-dead *.local.sh rule from .gitignore. - Remove shell-customization references from CONFIGURATION.md (decision tree, Quick Reference table, subsection, blockquote, directory map). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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.
Summary
Removes the team shell-alias mechanism and deletes the now-empty
.devcontainer/config/folder. The*.shared.sh/*.local.shsourcing convention was purely an interactive convenience — nothing in the repo (scripts, compose, CI) ever invoked the aliases. After the stacks-colocation refactor,config/shell/was the last occupant of.devcontainer/config/.Changes
.devcontainer/config/—shell/aliases.shared.shandshell/README.md.post-create.sh— removesetup_shell_customization()and its call; new containers no longer bake a sourcing block into~/.zshrc.motd.sh— rename_motd_aliases()→_motd_quickref()and replace the alias list with the real full commands (docker compose -f .devcontainer/compose.yaml up -d / down / logs -f,git status / log / diff,task,claude); drop the two Tips lines pointing atconfig/shell/..gitignore— remove the now-dead*.local.shrule.CONFIGURATION.md— remove shell-customization references (decision tree, Quick Reference table row, "Shell Customization" subsection, "not a stack" blockquote, directory-map entries).Behavior note
Existing containers that already have the sourcing block in
~/.zshrcwon't error — the zsh(N)null-glob makes the loops silent no-ops now that the folder is gone. The mechanism fully disappears on a rebuild.Verification
grepforconfig/shell/aliases.shared/setup_shell_customization/_motd_aliases→ no residual references.-x --source-path=SCRIPTDIR) → clean.config/shell.🤖 Generated with Claude Code