fix(lifeos): print the current 'lifeos' command name instead of the old 'k' - #1745
Open
elhoim wants to merge 1 commit into
Open
fix(lifeos): print the current 'lifeos' command name instead of the old 'k'#1745elhoim wants to merge 1 commit into
elhoim wants to merge 1 commit into
Conversation
…comments The launcher was renamed from 'k' to 'lifeos', but the printed usage strings and several code comments still told users to run 'k'. Every install path (install.sh, INSTALL.md, Setup.md) wires the alias as 'lifeos', so the printed advice was unrunnable as shown. Renames the stale command name in user-facing output and comments only; no behaviour change. The destructured variable 'k' in getMcpProfiles() is untouched.
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.
What
install/LIFEOS/TOOLS/lifeos.tsstill prints and documents the launcher's old command name,k. Every install path wires the alias aslifeos(install/install.sh:316,INSTALL.md:103,Workflows/Setup.md:56), so the usage lines the CLI prints are not runnable as shown on a current install.Change (one file)
install/LIFEOS/TOOLS/lifeos.ts— renamesk→lifeosin 15 places: the wallpaper usage/example lines, the update-available hint, themcp setusage lines (2), the profiles header, the threeExamples:lines inmcp list, themcp list | mcp setandprompterror strings, thecmdDoctordocblock (3 mentions), the bun---swallowing comment, and the-rsession-ID comment.Strings only — no behaviour change, no control flow touched. The three
Examples:lines got their inline#comments re-aligned since the command grew five characters.Deliberately not renamed:
.map(([k]) => k)ingetMcpProfiles()— thatkis a destructured entry key, unrelated to the command name.Verification
bun build --target=bun --no-bundle install/LIFEOS/TOOLS/lifeos.ts— parses clean.lifeos mcp list; output now readsProfiles (use with 'lifeos mcp set')and the three examples print runnable commands.rg '\bk\b' install/LIFEOS/TOOLS/lifeos.ts→ only the destructured-variable line remains.