-
-
Notifications
You must be signed in to change notification settings - Fork 0
feat(setup): enhance loader with tool installer and config isolation #127
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Changes from 15 commits
3363ec5
d84b267
64f59da
8869455
bb00c23
2f45c38
374029f
11908e6
aeeceb5
e607213
dd590fe
3ac96d5
df50da0
78b5dd2
035b251
ce87615
e6152e0
099d4a9
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -75,7 +75,8 @@ src/ | |
| gc.rs — `numan gc [--dry-run]`: delete orphaned payload directories (Phase 5) | ||
| nupm.rs — `numan nupm status|inspect|import|diff`: nupm discovery + import + drift (Phase 6.1–6.3) | ||
| completions.rs — `numan completions <shell>`: install by default (mkdir+write); `--print` for stdout (Phase 7.3) | ||
| setup.rs — `numan setup nu [VERSION]|remove|path|use <path>` + `setup loader`: Nushell bootstrap + nushell-loader install | ||
| setup.rs — `numan setup nu [VERSION]|remove|path|use <path>` + `setup loader [--status|--detect|--add|--remove|--clean|--install]`: Nushell bootstrap + nushell-loader install with loader-config.nu isolation | ||
| setup_tools.rs — CLI shell tool presets + GitHub release binary installer (starship, zoxide, carapace, atuin, mise, direnv, oh-my-posh) | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win Also record the new loader actions in the structure map. The new As per coding guidelines: "update 🤖 Prompt for AI AgentsSource: Coding guidelines |
||
| try_cmd.rs — `numan try <owner/name[@version]> [--no-activate]`: attempt a package for current Nu; explain compatible managed Nu versions if incompatible | ||
| use_cmd.rs — `numan use <version>|latest|list`: activates a previously installed managed Nu version (no auto-download); cross-minor leave/teardown (modules then plugins) + restore (plugins then modules) via activation profiles; same-target is restore-only; writes the active-version marker after a PreMutation snapshot under the root mutation lock | ||
| activation_switch.rs — shared leave/restore orchestration for `numan use` (lower-level lifecycle, no profile-sync wrappers) | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
P2: This synopsis makes
--addand--removelook valueless and makes--installlook mutually exclusive, so contributors following it will form invalid commands or miss supported options. Document the required tool values and the combinable modifiers.Prompt for AI agents