refactor(shell): enhance shell detection and add `VP_SHELL` override by nekomoyi · Pull Request #1658 · voidzero-dev/vite-plus

nekomoyi · 2026-05-21T18:55:50Z

What changed

Replaced VP_SHELL_NU and VP_SHELL_PWSH with one env var: VP_SHELL
Added shell parsing that accepts bash, zsh, fish, nu, pwsh, and cmd

Why

The old approach depended on a few shell-specific flags and inherited env vars. It could mis-detect nested shells and is harder to maintain.

The new shell detection stages:

VP_SHELL if it is set
Fallback to posix or cmd

Replace VP_SHELL_NU and VP_SHELL_PWSH with a single VP_SHELL env var that accepts shell names (bash, fish, nu, pwsh, cmd). Detection now follows: VP_SHELL → process tree inference → platform default. - Add Shell::from_str for case-insensitive name parsing - Add process tree walking to infer shell from parent processes - Add sysinfo dep for Windows process enumeration - Update shell wrapper scripts to remove shell-specific env vars

These tests assert on `detect_shell()` default fallback, but process tree inference varies across environments (CI uses `shell: bash` on Windows, local machines may have Git Bash), making the result non-deterministic.

netlify · 2026-05-21T18:55:57Z

✅ Deploy Preview for viteplus-preview canceled.

Name	Link
🔨 Latest commit	`4212485`
🔍 Latest deploy log	https://app.netlify.com/projects/viteplus-preview/deploys/6a12f94ddd37fe0008e16fde

Copilot

Pull request overview

This PR refactors shell detection for the global CLI by consolidating multiple shell-specific env flags into a single VP_SHELL override, and adding process-tree inference to better detect the active shell for output formatting (notably vp env use eval output).

Changes:

Replaced VP_SHELL_NU / VP_SHELL_PWSH with a single VP_SHELL env var (case-insensitive parsing + aliases).
Added process-tree based shell inference (ps on Unix, sysinfo on Windows) with a platform-default fallback.
Added global snap tests validating VP_SHELL-driven formatting across supported shells.

Reviewed changes

Copilot reviewed 10 out of 11 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
packages/cli/snap-tests-global/command-env-use-shells/steps.json	Adds snap-test commands covering `VP_SHELL` parsing and formatting.
packages/cli/snap-tests-global/command-env-use-shells/snap.txt	Snapshot output for the new `VP_SHELL` shell-formatting coverage.
packages/cli/snap-tests-global/command-env-use-shells/package.json	Fixture metadata for the new snap-test case.
crates/vite_shared/src/env_vars.rs	Replaces shell marker constants with `VP_SHELL`.
crates/vite_shared/src/env_config.rs	Updates `EnvConfig` to read `VP_SHELL` (and removes old marker fields).
crates/vite_global_cli/src/commands/shell.rs	Implements `Shell::FromStr`, `VP_SHELL` override, and process-tree inference.
crates/vite_global_cli/src/commands/env/use.rs	Updates tests to use `vp_shell: Some(...)` instead of old marker booleans.
crates/vite_global_cli/src/commands/env/setup.rs	Removes wrapper-set env markers (`VP_SHELL_NU`, `VP_SHELL_PWSH`) from generated shell setup scripts.
crates/vite_global_cli/Cargo.toml	Adds Windows-only `sysinfo` dependency for process-tree inference.
Cargo.toml	Adds `sysinfo` to workspace dependencies.
Cargo.lock	Lockfile updates due to dependency graph changes.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7a3ed1ceb9

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

fengmk2 · 2026-05-22T02:54:57Z

@codex review

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 271fbd296e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

…oidzero-dev#1658) ## What changed - Replaced `VP_SHELL_NU` and `VP_SHELL_PWSH` with one env var: `VP_SHELL` - Added shell parsing that accepts `bash`, `zsh`, `fish`, `nu`, `pwsh`, and `cmd` ## Why The old approach depended on a few shell-specific flags and inherited env vars. It could mis-detect nested shells and is harder to maintain. The new shell detection stages: 1. `VP_SHELL` if it is set 2. Fallback to posix or cmd --------- Co-authored-by: MK (fengmk2) <fengmk2@gmail.com>

@fengmk2

Release vite-plus v0.1.23. Enterprise-ready HTTP (proxy + custom CA), task command shorthands in `vite.config.ts`, a smoother `vp create`/`vp migrate`, and the oxc/vite/rolldown bundled stack moves forward. ### Highlights - **Proxy and custom-CA aware HTTP**: a new process-wide `vite_shared::shared_http_client()` honors `HTTPS_PROXY` / `HTTP_PROXY` / `NO_PROXY`, picks up macOS System Settings / Windows registry proxies, loads custom CAs from `SSL_CERT_FILE` and `NODE_EXTRA_CA_CERTS`, and exposes a `VP_INSECURE_TLS` diagnostic switch; makes `vp` work through Socket Firewall Free and other TLS-intercepting proxies ([#1686](#1686)), by @fengmk2 - **Task command shorthands**: `run.tasks` entries in `vite.config.ts` now accept a bare string (`"build": "cmd"`) or array (`"build": ["cmd1", "cmd2"]`) instead of always requiring `{ command: ... }`; arrays reuse the existing `&&` planning path so cache, `dependsOn`, and task options stay consistent ([vite-task#391](voidzero-dev/vite-task#391)), by @jong-kyung - **Managed `vp outdated -g`**: routes through Vite+'s managed global package metadata instead of delegating to the underlying `npm outdated -g` store, so all installed global packages are reported consistently ([#1659](#1659)), by @liangmiQwQ ### Features - `vp pm approve-builds`: new unified subcommand that mirrors `pnpm approve-builds` one-to-one, adapts to `bun pm trust`, and warns-and-noops on npm/yarn ([#1662](#1662)), by @fengmk2 - `vp create`: opt-in GitHub Copilot setup; selecting `--agent copilot` now generates a `.github/workflows/copilot-setup-steps.yml` so the Copilot Coding Agent can set up Vite+ and run `vp` in the new project out of the box ([#1683](#1683)), by @jong-kyung - `vp migrate`: prompt to remove `baseUrl` from `tsconfig.json` before applying type-aware lint defaults (runs `@andrewbranch/ts5to6 --fixBaseUrl .` under the hood; auto-applied in non-interactive mode) ([#1692](#1692)), by @TheAlexLichter - Respect `packageManager` in package-manager shims (`npm`/`npx`, `pnpm`/`pnpx`, `yarn`/`yarnpkg`, `bun`/`bunx`); add non-mutating `packageManager` resolution metadata for `vp env current` and `vp env which` ([#1654](#1654)), by @fengmk2 - vite-task: `--filter` no-match now exits `0` by default; add `--fail-if-no-match` to opt back in ([vite-task#393](voidzero-dev/vite-task#393)), by @kazupon ### Fixes & Enhancements - `vp create`: keep generated `.vscode/settings.json` trackable when the VS Code editor option is selected (avoid templates' `.vscode/*` `.gitignore` masking it) ([#1700](#1700)), by @jong-kyung - `vp create vite:monorepo`: normalize sub-package `vite-plus` to `catalog:` even when only `vite-plus` (not `vite`/`vitest`/...) is present, and drop the `vite`/`vitest` aliases generated by the upstream library template ([#1697](#1697)), by @fengmk2 - `vp add/install -g <path>`: resolve the real package name from `package.json` instead of using the path string, so local-path installs don't create broken directories ([#1685](#1685)), by @liangmiQwQ - `vp test --coverage` and other direct built-in commands now expose the workspace's `package.json#packageManager` to child processes so tools like Vitest coverage can spawn the configured PM ([#1696](#1696)), by @jong-kyung - `vp migrate`: clean up the whole ESLint ecosystem (plugins, configs, parser/resolver, type-utils) rather than just `eslint`; skip the migration entirely when `@nuxt/eslint` is detected ([#1682](#1682)), by @fengmk2 - `vp create`: write `fmt.configPath` (not `configPath`) for Zed oxfmt settings to match the official Zed OXC extension layout ([#1687](#1687)), by @chungweileong94 - `vp migrate`: parse `tsconfig.json` as JSONC so files with comments don't break `baseUrl` detection/removal ([#1688](#1688)), by @TheAlexLichter - `vp env setup`: Unix env shims now point at the active `vp` executable instead of always assuming `VP_HOME/current/bin/vp`, so Homebrew-style installs work ([#1631](#1631)), by @leohara - `vp outdated -g` / `vp why -g`: don't require a local `package.json`; global commands run regardless of cwd ([#1622](#1622)), by @liangmiQwQ - `vp create`: default the "Initialize a git repository?" prompt to yes ([#1650](#1650)), by @fengmk2 - `vp` hooks: include the managed Node bin in `PATH` so `./node_modules/.bin/vp` can find `node` from a VS Code commit on macOS ([#1647](#1647)), by @TheAlexLichter - `vpx` on Windows now invokes the package's `.cmd` shim instead of the Unix binary ([#1652](#1652)), by @tobynguyen27 - vite-task: bump cache database schema to version 13 (forces a one-time rebuild of the local task cache) ([vite-task#402](voidzero-dev/vite-task#402)), by @branchseer - Bump vite-task to `d02b257` and `5833b374`; also bumps the repo's Rust nightly toolchain to `nightly-2026-05-24` and ships the regenerated `run` config types and docs for the new task command shorthand ([#1689](#1689), [#1695](#1695)), by @branchseer ### Refactor - Replace `VP_SHELL_NU`/`VP_SHELL_PWSH` with a single `VP_SHELL` override; add explicit shell parsing for `bash`, `zsh`, `fish`, `nu`, `pwsh`, and `cmd`, and harden auto-detection against nested shells ([#1658](#1658)), by @nekomoyi - vite-task: replace `allocator-api2` with `bumpalo` collections ([vite-task#400](voidzero-dev/vite-task#400)), by @branchseer - vite-task: drop the unused `and_item_index` field from `ExecutionItemDisplay` ([vite-task#394](voidzero-dev/vite-task#394)), by @branchseer ### Docs - Add the root `AGENTS.md` as the primary AI-agent guide for the vite-plus repository; convert `CLAUDE.md` into a compatibility pointer ([#1670](#1670)), by @jong-kyung - Align the agent validation table to match `AGENTS.md` ([#1673](#1673)), by @jong-kyung - Update the task output caching guide so the documented behavior matches what vite-task actually does ([#1639](#1639)), by @ericclemmons - Correct the bundled-source location in `packages/core/BUNDLING.md` ([#1660](#1660)), by @shulaoda ### Chore - Clarify `--help` text for `vp env default`/`pin`/`use`/`exec` with `Examples:` blocks ([#1664](#1664)), by @Boshen - Refresh trusted stack stats on the docs homepage ([#1680](#1680)), by @voidzero-guard[bot] - Drop the standalone `pnpm --filter @rolldown/pluginutils build` step now that `@rolldown/pluginutils` is published from its own package ([#1655](#1655)), by @shulaoda - Preserve single-quote style when `sync-remote` rewrites `pnpm-workspace.yaml` ([#1672](#1672)), by @lyzno1 - Enable `vite_pm_cli` lib tests by removing a stale `test = false` flag ([#1661](#1661)), by @shulaoda - CI: switch macOS runners back to `namespace-profile-mac-default` ([#1701](#1701)), by @fengmk2 - CI: fix release-day flakes in the upgrade test and snap test when the dev `package.json` version equals npm latest ([#1645](#1645)), by @fengmk2 - CI: replace `zizmor` and `cargo-deny` workflows with `oxc-project/security-action` ([#1635](#1635)), by @Boshen - CI: warm-up monorepo cache test under npm ([#1649](#1649)), by @fengmk2 - CI: attach per-target `vp` binary archives (`.tar.gz`/`.zip`) to GitHub Releases alongside the existing `vp-setup-*.exe` installers ([#1665](#1665)), by @Boshen - CI: declare Playwright via `repo.json` in ecosystem tests and bump consumers to `>=1.60` to dodge the Node 24.16.0 hang ([#1668](#1668)), by @fengmk2 - Update GitHub Actions ([#1640](#1640), [#1675](#1675), [#1678](#1678), [#1679](#1679), [#1691](#1691)), by @renovate[bot] - Upgrade upstream dependencies: vite `8.0.11 → 8.0.14`, rolldown `1.0.0 → 1.0.3`, vitest `4.1.6 → 4.1.7`, oxlint `1.63.0 → 1.67.0`, oxfmt `0.48.0 → 0.52.0`, oxlint-tsgolint `0.22.1 → 0.23.0`, `@oxc-project/*` and oxc Rust crates `0.129.0 → 0.133.0` ([#1646](#1646), [#1653](#1653), [#1693](#1693), [#1699](#1699)), by @voidzero-guard[bot] ### Bundled Versions | Tool | Version | Source | | --- | --- | --- | | vite | `8.0.14` | [`c917f1e`](vitejs/vite@c917f1e) | | rolldown | `1.0.3` | [`a287faa`](rolldown/rolldown@a287faa) | | tsdown | `0.22.0` | [npm](https://npmx.dev/package/tsdown/v/0.22.0) | | vitest | `4.1.7` | [npm](https://npmx.dev/package/vitest/v/4.1.7) | | oxlint | `1.67.0` | [npm](https://npmx.dev/package/oxlint/v/1.67.0) | | oxlint-tsgolint | `0.23.0` | [npm](https://npmx.dev/package/oxlint-tsgolint/v/0.23.0) | | oxfmt | `0.52.0` | [npm](https://npmx.dev/package/oxfmt/v/0.52.0) | ### New Contributors Welcome to all new contributors! 🎉 @ericclemmons, @tobynguyen27, @shulaoda, @leohara, @chungweileong94 **Full Changelog**: v0.1.22...v0.1.23 Merging this PR will trigger the release workflow. --------- Co-authored-by: voidzero-guard[bot] <278573678+voidzero-guard[bot]@users.noreply.github.com> Co-authored-by: MK <fengmk2@gmail.com>

_Requested by **Brooklyn** · [project thread](https://claude.ai/code/project/chan_018s9RxRdtH424j2EhXdzV3N?thread=cmsg_018s9RxRdtH424j2EhXdzV3NQxbeR8Pt7qaqSQwoq9rEFo)_  Before: `vp_global_cli` and `vp_pm_cli` carried Rust unit tests that could not fail when the product broke: an `assert!(true)` in `create.rs`, four `env use` shell-detection tests that repeat `commands::shell` tests for code `env use` only imports, and three tests of test-only helpers. `upgrade_check.rs` had 20 hand-unrolled lookup tests. After: those 8 tests are gone and the 20 lookup tests are 2 table-driven tests that keep every row and comment, with the failing row named in the assertion message. Only test code changes: every hunk is inside a `#[cfg(test)]` module or the `#[cfg(test)]`-only `resolution::test_utils` module. How: each candidate was defended by mutating product code with the most realistic regression only it would catch, running the crate's tests, and reverting. No deleted test failed alone on any mutation. ### Totals | | Before (origin/main c04bcf6) | After | |---|---|---| | Files touched | – | 5 | | `vp_global_cli` cases collected (Linux) | 502 (+2 ignored) | 479 (+2 ignored) | | `vp_pm_cli` cases collected | 923 (+2 ignored) | 920 (+2 ignored) | - Cases deleted: **8** of 1,425 collected in the two touched crates. Repo-wide: about 2,432 `#[test]`/`#[tokio::test]` attributes in tracked Rust (static count) and 1,492 static `it`/`test` calls in 101 JS spec files (not collected; Vitest needs a full `packages/core` build). - Cases folded: **20 into 2**, every row and comment kept. Runner-collected total: −26. - Net lines: **−159** (+42 / −201). - Runtime saved: about 0.03 s (the 8 deleted cases each report 0.000 s; the 20 folded cases summed 0.028 s against 0.011 s for the 2 replacements). - Test-binary time on this shared machine: `vp_global_cli` 3.13 s → 3.32 s, `vp_pm_cli` 27.81 s → 29.06 s (noise). ### Per file 1. **`crates/vp_global_cli/src/commands/create.rs` › `test_create_command_module_exists`** - Class: tautology (`assert!(true)`). - Best defense: M1 (delegate `"new"` instead of `"create"`) passes all 502 tests, so this test cannot fail. The whole `mod tests` goes. - Coverage gap (pre-existing, unchanged by this PR): no unit test covers `vp create` delegation. It may be covered by the PTY snapshot suite, which was not run. 2. **`crates/vp_global_cli/src/commands/env/use.rs` › `test_detect_shell_vp_shell_{powershell,fish,nu}`, `test_detect_shell_posix_default`** - Class: duplicates of stronger tests. `detect_shell` / `Shell::from_str` live in `commands/shell.rs`; `env use` only imports them. - Still covered: `fish` is byte-identical to `commands::shell::tests::test_detect_shell_vp_shell_fish`; `nu` to `test_detect_shell_vp_shell_explicit`; lowercase `powershell` by `test_shell_from_str` and `test_detect_shell_vp_shell_case_insensitive`; `posix_default` by `test_detect_shell_defaults_without_vp_shell`. - Best defenses M2–M5: each failed the nominee together with at least one `commands::shell` test. - The `use vp_shared::env_vars;` import is now `#[cfg(windows)]`, since only the three Windows-only tests in the module still use it. 3. **`crates/vp_pm_cli/src/resolution/test_utils.rs` › `parses_args_without_a_program_name`, `parses_subcommands_without_a_program_name`** - Class: test tooling testing test tooling. - Best defense: M7 fails 122 tests and M8 fails 17; every resolution parser test exercises these helpers. 4. **`crates/vp_pm_cli/src/resolution/commands/list.rs` › `list_args_helper_keeps_pattern`** (and its test-only `list_args` helper, which had no other caller) - Class: test tooling testing test tooling; it only checked struct construction. - Best defenses: M9 fails only `test_list_with_pattern_and_pass_through_args`; M10 fails `test_parser_accepts_short_dev_and_global_flags` and `cli::tests::projects_managed_global_commands`. The nominee passes both. ### Folds (`crates/vp_global_cli/src/upgrade_check.rs`) - The 9 `is_newer_version_*` tests → `is_newer_version_compares_semver` (12 rows, original comments kept, one comment added for the `0.0.0` dev-build row). - The 11 `should_run_for_*` / `should_not_run_for_*` tests → `should_run_for_command_skips_excluded_and_machine_readable_commands` (11 rows). - M11, M12 and M13 still fail the folded tables. ### Kept after defense — do not re-nominate None: no nominee survived its defense. ### Case counts | file | before | after | delta | expected delta | |---|---|---|---|---| | vp_global_cli `commands/create.rs` | 1 | 0 | −1 | −1 | | vp_global_cli `commands/env/use.rs` (Linux) | 16 | 12 | −4 | −4 | | vp_global_cli `upgrade_check.rs` | 36 | 18 | −18 | −18 (20 folded into 2) | | vp_pm_cli `resolution/commands/list.rs` | 45 | 44 | −1 | −1 | | vp_pm_cli `resolution/test_utils.rs` | 2 | 0 | −2 | −2 | | **Crates total** | 1,425 | 1,399 | −26 | −26 | **0 new failures** (0 failed and 4 ignored, before and after). ### Mutations Each was temporary and reverted (`git status` clean afterwards). 1. M1 `create.rs:15`: delegate `"new"` instead of `"create"`. Failed: none. 2. M2 `shell.rs:30`: `"fish"` → `"fishshell"`. Failed: `use::…_fish`, `shell::test_shell_from_str`, `shell::…_fish`. 3. M3 `shell.rs:31`: `"nu" | "nushell"` → `"nushell"`. Failed: `use::…_nu`, `shell::…_explicit`, `shell::test_shell_from_str`. 4. M4 `shell.rs:32`: drop `"powershell"`. Failed: `use::…_powershell`, `shell::…_case_insensitive`, `shell::test_shell_from_str`. 5. M5 `shell.rs:54`: default always `Cmd`. Failed: `use::…_posix_default`, `shell::…_defaults_without_vp_shell`, `shell::…_invalid_vp_shell_falls_back_to_default`. 6. M6 `shell.rs:49`: upper-case the value. Failed: none (equivalent mutation; `from_str` lowercases). 7. M7 `test_utils.rs:70`: `test_argv` no longer prepends a program name. Failed: 122 tests, including both nominees. 8. M8 `test_utils.rs:57`: `parse_subcommand` gets raw args. Failed: 17 tests (`config`, `dist_tag`, `owner`, `stage`, `token`, plus the nominee). 9. M9 `list.rs:78-80`: pnpm drops the pattern. Failed: `test_list_with_pattern_and_pass_through_args` only. 10. M10 `list.rs:11`: `pattern` becomes `#[arg(long)]`. Failed: `test_parser_accepts_short_dev_and_global_flags`, `cli::tests::projects_managed_global_commands`. 11. M11 `upgrade_check.rs:170` (after the fold): `>` → `>=`. Failed: `is_newer_version_compares_semver`. 12. M12 `upgrade_check.rs:338` (after the fold): `Upgrade` removed from the excluded commands. Failed: `should_run_for_command_skips_excluded_and_machine_readable_commands`. 13. M13 `upgrade_check.rs:165` (after the fold): the `0.0.0` early return removed. Failed: `is_newer_version_compares_semver`. ### What ran and what did not - Ran, at origin/main and on this branch: `cargo test -p vp_global_cli` and `cargo test -p vp_pm_cli` with `--report-time`; `cargo clippy -p vp_global_cli -p vp_pm_cli --all-targets --all-features -- --deny warnings` with the `just lint` allow-list; `cargo fmt --all -- --check`; `git diff --check`. All pass. - Not run: the full workspace `just test` / `just lint` / `just check`; Windows targets (none installed here, so the `#[cfg(windows)]` import gate was reviewed but not compiled; Windows CI will compile it); the JS/Vitest suite; the PTY snapshot suite. ### Excluded pull requests 55 open PRs touch 737 files between them; none of the 5 files here is among them. #3055 #3052 #3051 #3049 #3047 #3046 #3044 #3040 #3038 #3037 #3020 #3018 #3014 #3012 #3009 #3008 #3000 #2986 #2981 #2966 #2936 #2922 #2900 #2863 #2862 #2810 #2792 #2740 #2721 #2699 #2695 #2574 #2557 #2552 #2549 #2543 #2542 #2541 #2483 #2475 #2463 #2462 #2461 #2456 #2428 #2418 #2357 #2267 #2125 #1958 #1783 #1782 #1614 #1517 #1180. ### Age check (cutoff: 30 days, 2026-09-14) First-added dates from `git log -S`, followed across the `vite_*` → `vp_*` crate rename. | Test | First added | |---|---| | `test_create_command_module_exists` | 2026-02-10 (a29ac83, #570) | | `test_detect_shell_posix_default` | 2026-02-07 (194c0dc, #524) | | `test_detect_shell_vp_shell_{powershell,fish,nu}` | 2026-05-24 (53ae721, #1658) | | `test_utils` tests and `list_args_helper_keeps_pattern` | 2026-08-03 (c1574dd, #2140) | | The 20 folded `upgrade_check` tests | 2026-03-30 (db75675) | None was added for a bug or incident, and none is in a permission, security or entitlement suite. ### Checks | Check | Before | After | |---|---|---| | Tests `vp_global_cli` | 502 pass | 479 pass | | Tests `vp_pm_cli` | 923 pass | 920 pass | | Clippy (deny warnings) | pass | pass | | rustfmt | – | pass | | `git diff --check` | – | clean | To revert: `git revert 687935d`. 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_01PxVsmvB9xfZKfVnUonY8kf --- _Generated by [Claude Code](https://claude.ai/code/session_01PxVsmvB9xfZKfVnUonY8kf)_ Co-authored-by: Claude <noreply@anthropic.com>

nekomoyi added 4 commits May 22, 2026 01:50

test(cli): add snap test for shell environment detection

00720a4

chore: remove blank lines

fd034f3

lint

7a3ed1c

nekomoyi force-pushed the feat/vp-shell-env branch from 5cdc2bc to 7a3ed1c Compare May 21, 2026 19:06

nekomoyi requested a review from Copilot May 21, 2026 19:14

Copilot started reviewing on behalf of nekomoyi May 21, 2026 19:15

View session

Copilot AI reviewed May 21, 2026

View reviewed changes

chatgpt-codex-connector Bot reviewed May 21, 2026

View reviewed changes

Comment thread crates/vite_global_cli/src/commands/shell.rs Outdated

nekomoyi added 3 commits May 22, 2026 03:54

fix: pass shell hints from eval wrappers

5ca9797

fix: remove unstable shell fallback snapshot

9ca020f

fix: ignore failed ps shell probes

271fbd2

nekomoyi marked this pull request as ready for review May 21, 2026 20:12

chatgpt-codex-connector Bot reviewed May 22, 2026

View reviewed changes

Comment thread crates/vite_global_cli/src/commands/shell.rs

nekomoyi added 2 commits May 22, 2026 11:38

Merge remote-tracking branch 'origin/main' into feat/vp-shell-env

2d6931e

fix: force POSIX shell for env wrapper

59c2921

fengmk2 added test: e2e Auto run e2e tests test: create-e2e Run `vp create` e2e tests labels May 22, 2026

fengmk2 approved these changes May 22, 2026

View reviewed changes

fengmk2 reviewed May 22, 2026

View reviewed changes

Comment thread crates/vite_global_cli/src/commands/shell.rs Outdated

fix: remove unused shell infer

e1ab2e0

fengmk2 requested a review from naokihaba May 22, 2026 07:07

Merge branch 'main' into feat/vp-shell-env

9c9aec4

fengmk2 self-assigned this May 22, 2026

naokihaba approved these changes May 22, 2026

View reviewed changes

fengmk2 added 2 commits May 24, 2026 20:08

Merge branch 'main' into feat/vp-shell-env

9c6fbb8

Merge branch 'main' into feat/vp-shell-env

c8eac63

fengmk2 added 2 commits May 24, 2026 21:05

Merge branch 'main' into feat/vp-shell-env

58027ef

Merge branch 'main' into feat/vp-shell-env

4212485

fengmk2 merged commit 53ae721 into voidzero-dev:main May 24, 2026
89 of 91 checks passed

fengmk2 mentioned this pull request May 28, 2026

release: v0.1.23 #1703

Merged

BrewTestBot mentioned this pull request May 29, 2026

vite-plus 0.1.23 Homebrew/homebrew-core#285211

Closed

Brooooooklyn mentioned this pull request Oct 14, 2026

test: remove tautological and duplicate Rust unit tests #3057

Merged

Navigation Menu

Repository navigation

refactor(shell): enhance shell detection and add `VP_SHELL` override - #1658

refactor(shell): enhance shell detection and add `VP_SHELL` override#1658
fengmk2 merged 16 commits into
voidzero-dev:mainfrom
nekomoyi:feat/vp-shell-env

nekomoyi commented May 21, 2026 •

edited

Loading

Uh oh!

netlify Bot commented May 21, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

chatgpt-codex-connector Bot left a comment

Uh oh!

Uh oh!

fengmk2 commented May 22, 2026

Uh oh!

chatgpt-codex-connector Bot left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

nekomoyi commented May 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

What changed

Why

Uh oh!

netlify Bot commented May 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Deploy Preview for viteplus-preview canceled.

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

Uh oh!

fengmk2 commented May 22, 2026

Uh oh!

chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

nekomoyi commented May 21, 2026 •

edited

Loading

netlify Bot commented May 21, 2026 •

edited

Loading