Bump patch version (#16236)

Bump patch version

.github/workflows/friendly-config-reminder.yml

@@ -0,0 +1,25 @@
+name: Comment on changes to the config
+on:
+  pull_request_target:
+    paths:
+      - 'crates/nu-protocol/src/config/**'
+jobs:
+  comment:
+    runs-on: ubuntu-latest
+    steps:
+    - name: Check if there is already a bot comment
+      uses: peter-evans/find-comment@v3
+      id: fc
+      with:
+        issue-number: ${{ github.event.pull_request.number }}
+        comment-author: 'github-actions[bot]'
+        body-includes: Hey, just a bot checking in!
+    - name: Create comment if there is not
+      if: steps.fc.outputs.comment-id == ''
+      uses: peter-evans/create-or-update-comment@v4
+      with:
+        issue-number: ${{ github.event.pull_request.number }}
+        body: |
+          Hey, just a bot checking in! You edited files related to the configuration.
+          If you changed any of the default values or added a new config option, don't forget to update the [`doc_config.nu`](https://github.com/nushell/nushell/blob/main/crates/nu-utils/src/default_files/doc_config.nu) which documents the options for our users including the defaults provided by the Rust implementation.
+          If you didn't make a change here, you can just ignore me.

.github/workflows/nightly-build.yml

@@ -46,7 +46,7 @@ jobs:
         uses: hustcer/setup-nu@v3
         if: github.repository == 'nushell/nightly'
         with:
-          version: 0.103.0
+          version: 0.105.1
 
       # Synchronize the main branch of nightly repo with the main branch of Nushell official repo
       - name: Prepare for Nightly Release
@@ -127,6 +127,7 @@ jobs:
         - armv7-unknown-linux-musleabihf
         - riscv64gc-unknown-linux-gnu
         - loongarch64-unknown-linux-gnu
+        - loongarch64-unknown-linux-musl
         include:
         - target: aarch64-apple-darwin
           os: macos-latest
@@ -152,6 +153,8 @@ jobs:
           os: ubuntu-22.04
         - target: loongarch64-unknown-linux-gnu
           os: ubuntu-22.04
+        - target: loongarch64-unknown-linux-musl
+          os: ubuntu-22.04
 
     runs-on: ${{matrix.os}}
     steps:
@@ -179,36 +182,22 @@ jobs:
       uses: actions-rust-lang/setup-rust-toolchain@v1
       # WARN: Keep the rustflags to prevent from the winget submission error: `CAQuietExec: Error 0xc0000135`
       with:
+        cache: false
         rustflags: ''
 
     - name: Setup Nushell
       uses: hustcer/setup-nu@v3
-      if: ${{ matrix.os != 'windows-11-arm' }}
       with:
-        version: 0.103.0
+        version: 0.105.1
 
     - name: Release Nu Binary
       id: nu
-      if: ${{ matrix.os != 'windows-11-arm' }}
       run: nu .github/workflows/release-pkg.nu
       env:
         OS: ${{ matrix.os }}
         REF: ${{ github.ref }}
         TARGET: ${{ matrix.target }}
 
-    - name: Build Nu for Windows ARM64
-      id: nu0
-      shell: pwsh
-      if: ${{ matrix.os == 'windows-11-arm' }}
-      run: |
-        $env:OS = 'windows'
-        $env:REF = '${{ github.ref }}'
-        $env:TARGET = '${{ matrix.target }}'
-        cargo build --release --all --target aarch64-pc-windows-msvc
-        cp ./target/${{ matrix.target }}/release/nu.exe .
-        ./nu.exe -c 'version'
-        ./nu.exe ${{github.workspace}}/.github/workflows/release-pkg.nu
-
     - name: Create an Issue for Release Failure
       if: ${{ failure() }}
       uses: JasonEtco/create-an-issue@v2
@@ -228,9 +217,7 @@ jobs:
         prerelease: true
         files: |
           ${{ steps.nu.outputs.msi }}
-          ${{ steps.nu0.outputs.msi }}
           ${{ steps.nu.outputs.archive }}
-          ${{ steps.nu0.outputs.archive }}
         tag_name: ${{ needs.prepare.outputs.nightly_tag }}
         name: ${{ needs.prepare.outputs.build_date }}-${{ needs.prepare.outputs.nightly_tag }}
       env:
@@ -276,7 +263,7 @@ jobs:
       - name: Setup Nushell
         uses: hustcer/setup-nu@v3
         with:
-          version: 0.103.0
+          version: 0.105.1
 
       # Keep the last a few releases
       - name: Delete Older Releases

.github/workflows/release-msi.yml

@@ -58,7 +58,7 @@ jobs:
     - name: Setup Nushell
       uses: hustcer/setup-nu@v3
       with:
-        version: nightly
+        version: 0.105.1
 
     - name: Release MSI Packages
       id: nu

.github/workflows/release-pkg.nu

@@ -99,6 +99,14 @@ if $os in ['macos-latest'] or $USE_UBUNTU {
             $env.CARGO_TARGET_LOONGARCH64_UNKNOWN_LINUX_GNU_LINKER = 'loongarch64-unknown-linux-gnu-gcc'
             cargo-build-nu
         }
+        'loongarch64-unknown-linux-musl' => {
+            print $"(ansi g)Downloading LoongArch64 musl cross-compilation toolchain...(ansi reset)"
+            aria2c -q https://github.com/LoongsonLab/oscomp-toolchains-for-oskernel/releases/download/loongarch64-linux-musl-cross-gcc-13.2.0/loongarch64-linux-musl-cross.tgz
+            tar -xf loongarch64-linux-musl-cross.tgz
+            $env.PATH = ($env.PATH | split row (char esep) | prepend $'($env.PWD)/loongarch64-linux-musl-cross/bin')
+            $env.CARGO_TARGET_LOONGARCH64_UNKNOWN_LINUX_MUSL_LINKER = "loongarch64-linux-musl-gcc"
+            cargo-build-nu
+        }
         _ => {
             # musl-tools to fix 'Failed to find tool. Is `musl-gcc` installed?'
             # Actually just for x86_64-unknown-linux-musl target

.github/workflows/release.yml

@@ -35,6 +35,7 @@ jobs:
         - armv7-unknown-linux-musleabihf
         - riscv64gc-unknown-linux-gnu
         - loongarch64-unknown-linux-gnu
+        - loongarch64-unknown-linux-musl
         include:
         - target: aarch64-apple-darwin
           os: macos-latest
@@ -60,6 +61,8 @@ jobs:
           os: ubuntu-22.04
         - target: loongarch64-unknown-linux-gnu
           os: ubuntu-22.04
+        - target: loongarch64-unknown-linux-musl
+          os: ubuntu-22.04
 
     runs-on: ${{matrix.os}}
 
@@ -90,32 +93,17 @@ jobs:
 
     - name: Setup Nushell
       uses: hustcer/setup-nu@v3
-      if: ${{ matrix.os != 'windows-11-arm' }}
       with:
-        version: 0.103.0
+        version: 0.105.1
 
     - name: Release Nu Binary
       id: nu
-      if: ${{ matrix.os != 'windows-11-arm' }}
       run: nu .github/workflows/release-pkg.nu
       env:
         OS: ${{ matrix.os }}
         REF: ${{ github.ref }}
         TARGET: ${{ matrix.target }}
 
-    - name: Build Nu for Windows ARM64
-      id: nu0
-      shell: pwsh
-      if: ${{ matrix.os == 'windows-11-arm' }}
-      run: |
-        $env:OS = 'windows'
-        $env:REF = '${{ github.ref }}'
-        $env:TARGET = '${{ matrix.target }}'
-        cargo build --release --all --target aarch64-pc-windows-msvc
-        cp ./target/${{ matrix.target }}/release/nu.exe .
-        ./nu.exe -c 'version'
-        ./nu.exe ${{github.workspace}}/.github/workflows/release-pkg.nu
-
     # WARN: Don't upgrade this action due to the release per asset issue.
     # See: https://github.com/softprops/action-gh-release/issues/445
     - name: Publish Archive
@@ -125,9 +113,7 @@ jobs:
         draft: true
         files: |
           ${{ steps.nu.outputs.msi }}
-          ${{ steps.nu0.outputs.msi }}
           ${{ steps.nu.outputs.archive }}
-          ${{ steps.nu0.outputs.archive }}
       env:
         GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 

.github/workflows/typos.yml

@@ -10,4 +10,4 @@ jobs:
         uses: actions/checkout@v4.1.7
 
       - name: Check spelling
-        uses: crate-ci/typos@v1.33.1
+        uses: crate-ci/typos@v1.34.0

.github/workflows/winget-submission.yml

@@ -10,6 +10,11 @@ on:
         required: true
         type: string
 
+permissions:
+  contents: write
+  packages: write
+  pull-requests: write
+
 jobs:
 
   winget:
@@ -25,5 +30,5 @@ jobs:
           installers-regex: 'msvc\.msi$'
           version: ${{ inputs.tag_name || github.event.release.tag_name }}
           release-tag: ${{ inputs.tag_name || github.event.release.tag_name }}
-          token: ${{ secrets.GITHUB_TOKEN }}
+          token: ${{ secrets.NUSHELL_PAT }}
           fork-user: nushell

.gitignore

@@ -32,11 +32,17 @@ unstable_cargo_features.txt
 # Helix configuration folder
 .helix/*
 .helix
+wix/bin/
+wix/obj/
+wix/nu/
 
 # Coverage tools
 lcov.info
 tarpaulin-report.html
 
+# benchmarking
+/tango
+
 # Visual Studio
 .vs/*
 *.rsproj

Cargo.toml

@@ -10,8 +10,8 @@ homepage = "https://www.nushell.sh"
 license = "MIT"
 name = "nu"
 repository = "https://github.com/nushell/nushell"
-rust-version = "1.85.1"
+rust-version = "1.86.0"
-version = "0.105.1"
+version = "0.106.1"
 
 # See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
 
@@ -24,36 +24,37 @@ pkg-fmt = "zip"
 
 [workspace]
 members = [
+  "crates/nu_plugin_custom_values",
+  "crates/nu_plugin_example",
+  "crates/nu_plugin_formats",
+  "crates/nu_plugin_gstat",
+  "crates/nu_plugin_inc",
+  "crates/nu_plugin_polars",
+  "crates/nu_plugin_query",
+  "crates/nu_plugin_stress_internals",
   "crates/nu-cli",
-  "crates/nu-engine",
-  "crates/nu-parser",
-  "crates/nu-system",
   "crates/nu-cmd-base",
   "crates/nu-cmd-extra",
   "crates/nu-cmd-lang",
   "crates/nu-cmd-plugin",
-  "crates/nu-command",
   "crates/nu-color-config",
+  "crates/nu-command",
+  "crates/nu-derive-value",
+  "crates/nu-engine",
+  "crates/nu-experimental",
   "crates/nu-explore",
   "crates/nu-json",
   "crates/nu-lsp",
-  "crates/nu-pretty-hex",
+  "crates/nu-parser",
-  "crates/nu-protocol",
-  "crates/nu-derive-value",
-  "crates/nu-plugin",
   "crates/nu-plugin-core",
   "crates/nu-plugin-engine",
   "crates/nu-plugin-protocol",
   "crates/nu-plugin-test-support",
-  "crates/nu_plugin_inc",
+  "crates/nu-plugin",
-  "crates/nu_plugin_gstat",
+  "crates/nu-pretty-hex",
-  "crates/nu_plugin_example",
+  "crates/nu-protocol",
-  "crates/nu_plugin_query",
-  "crates/nu_plugin_custom_values",
-  "crates/nu_plugin_formats",
-  "crates/nu_plugin_polars",
-  "crates/nu_plugin_stress_internals",
   "crates/nu-std",
+  "crates/nu-system",
   "crates/nu-table",
   "crates/nu-term-grid",
   "crates/nu-test-support",
@@ -63,7 +64,7 @@ members = [
 
 [workspace.dependencies]
 alphanumeric-sort = "1.5"
-ansi-str = "0.8"
+ansi-str = "0.9"
 anyhow = "1.0.82"
 base64 = "0.22.1"
 bracoxide = "0.1.6"
@@ -71,7 +72,7 @@ brotli = "7.0"
 byteorder = "1.5"
 bytes = "1"
 bytesize = "1.3.3"
-calamine = "0.26"
+calamine = "0.28"
 chardetng = "0.1.17"
 chrono = { default-features = false, version = "0.4.34" }
 chrono-humanize = "0.2.3"
@@ -92,7 +93,7 @@ filesize = "0.2"
 filetime = "0.2"
 heck = "0.5.0"
 human-date-parser = "0.3.0"
-indexmap = "2.9"
+indexmap = "2.10"
 indicatif = "0.17"
 interprocess = "2.2.0"
 is_executable = "1.0"
@@ -131,7 +132,7 @@ proc-macro-error2 = "2.0"
 proc-macro2 = "1.0"
 procfs = "0.17.0"
 pwd = "1.3"
-quick-xml = "0.37.0"
+quick-xml = "0.37.5"
 quickcheck = "1.0"
 quickcheck_macros = "1.1"
 quote = "1.0"
@@ -140,7 +141,7 @@ getrandom = "0.2" # pick same version that rand requires
 rand_chacha = "0.9"
 ratatui = "0.29"
 rayon = "1.10"
-reedline = "0.40.0"
+reedline = "0.41.0"
 rmp = "0.8"
 rmp-serde = "1.3"
 roxmltree = "0.20"
@@ -156,17 +157,18 @@ serde_json = "1.0.97"
 serde_urlencoded = "0.7.1"
 serde_yaml = "0.9.33"
 sha2 = "0.10"
-strip-ansi-escapes = "0.2.0"
+strip-ansi-escapes = "0.2.1"
 strum = "0.26"
 strum_macros = "0.26"
 syn = "2.0"
-sysinfo = "0.33"
+sysinfo = "0.36"
 tabled = { version = "0.20", default-features = false }
 tempfile = "3.20"
-titlecase = "3.5"
+thiserror = "2.0.12"
+titlecase = "3.6"
 toml = "0.8"
 trash = "5.2"
-update-informer = { version = "1.2.0", default-features = false, features = ["github", "ureq"] }
+update-informer = { version = "1.3.0", default-features = false, features = ["github", "ureq"] }
 umask = "2.1"
 unicode-segmentation = "1.12"
 unicode-width = "0.2"
@@ -184,7 +186,7 @@ uuid = "1.16.0"
 v_htmlescape = "0.15.0"
 wax = "0.6"
 web-time = "1.1.0"
-which = "7.0.0"
+which = "8.0.0"
 windows = "0.56"
 windows-sys = "0.48"
 winreg = "0.52"
@@ -195,27 +197,29 @@ webpki-roots = "1.0"
 # Warning: workspace lints affect library code as well as tests, so don't enable lints that would be too noisy in tests like that.
 # todo = "warn"
 unchecked_duration_subtraction = "warn"
+used_underscore_binding = "warn"
 
 [lints]
 workspace = true
 
 [dependencies]
-nu-cli = { path = "./crates/nu-cli", version = "0.105.1" }
+nu-cli = { path = "./crates/nu-cli", version = "0.106.1" }
-nu-cmd-base = { path = "./crates/nu-cmd-base", version = "0.105.1" }
+nu-cmd-base = { path = "./crates/nu-cmd-base", version = "0.106.1" }
-nu-cmd-lang = { path = "./crates/nu-cmd-lang", version = "0.105.1" }
+nu-cmd-extra = { path = "./crates/nu-cmd-extra", version = "0.106.1" }
-nu-cmd-plugin = { path = "./crates/nu-cmd-plugin", version = "0.105.1", optional = true }
+nu-cmd-lang = { path = "./crates/nu-cmd-lang", version = "0.106.1" }
-nu-cmd-extra = { path = "./crates/nu-cmd-extra", version = "0.105.1" }
+nu-cmd-plugin = { path = "./crates/nu-cmd-plugin", version = "0.106.1", optional = true }
-nu-command = { path = "./crates/nu-command", version = "0.105.1", default-features = false, features = ["os"] }
+nu-command = { path = "./crates/nu-command", version = "0.106.1", default-features = false, features = ["os"] }
-nu-engine = { path = "./crates/nu-engine", version = "0.105.1" }
+nu-engine = { path = "./crates/nu-engine", version = "0.106.1" }
-nu-explore = { path = "./crates/nu-explore", version = "0.105.1" }
+nu-experimental = { path = "./crates/nu-experimental", version = "0.106.1" }
-nu-lsp = { path = "./crates/nu-lsp/", version = "0.105.1" }
+nu-explore = { path = "./crates/nu-explore", version = "0.106.1" }
-nu-parser = { path = "./crates/nu-parser", version = "0.105.1" }
+nu-lsp = { path = "./crates/nu-lsp/", version = "0.106.1" }
-nu-path = { path = "./crates/nu-path", version = "0.105.1" }
+nu-parser = { path = "./crates/nu-parser", version = "0.106.1" }
-nu-plugin-engine = { path = "./crates/nu-plugin-engine", optional = true, version = "0.105.1" }
+nu-path = { path = "./crates/nu-path", version = "0.106.1" }
-nu-protocol = { path = "./crates/nu-protocol", version = "0.105.1" }
+nu-plugin-engine = { path = "./crates/nu-plugin-engine", optional = true, version = "0.106.1" }
-nu-std = { path = "./crates/nu-std", version = "0.105.1" }
+nu-protocol = { path = "./crates/nu-protocol", version = "0.106.1" }
-nu-system = { path = "./crates/nu-system", version = "0.105.1" }
+nu-std = { path = "./crates/nu-std", version = "0.106.1" }
-nu-utils = { path = "./crates/nu-utils", version = "0.105.1" }
+nu-system = { path = "./crates/nu-system", version = "0.106.1" }
+nu-utils = { path = "./crates/nu-utils", version = "0.106.1" }
 reedline = { workspace = true, features = ["bashisms", "sqlite"] }
 
 crossterm = { workspace = true }
@@ -244,9 +248,9 @@ nix = { workspace = true, default-features = false, features = [
 ] }
 
 [dev-dependencies]
-nu-test-support = { path = "./crates/nu-test-support", version = "0.105.1" }
+nu-test-support = { path = "./crates/nu-test-support", version = "0.106.1" }
-nu-plugin-protocol = { path = "./crates/nu-plugin-protocol", version = "0.105.1" }
+nu-plugin-protocol = { path = "./crates/nu-plugin-protocol", version = "0.106.1" }
-nu-plugin-core = { path = "./crates/nu-plugin-core", version = "0.105.1" }
+nu-plugin-core = { path = "./crates/nu-plugin-core", version = "0.106.1" }
 assert_cmd = "2.0"
 dirs = { workspace = true }
 tango-bench = "0.6"
@@ -257,10 +261,14 @@ serial_test = "3.2"
 tempfile = { workspace = true }
 
 [features]
+# Enable all features while still avoiding mutually exclusive features.
+# Use this if `--all-features` fails.
+full = ["plugin", "rustls-tls", "system-clipboard", "trash-support", "sqlite"]
+
 plugin = [
   # crates
-  "nu-cmd-plugin",
+  "dep:nu-cmd-plugin",
-  "nu-plugin-engine",
+  "dep:nu-plugin-engine",
 
   # features
   "nu-cli/plugin",
@@ -286,21 +294,20 @@ stable = ["default"]
 
 # Enable to statically link OpenSSL (perl is required, to build OpenSSL https://docs.rs/openssl/latest/openssl/);
 # otherwise the system version will be used. Not enabled by default because it takes a while to build
-static-link-openssl = ["dep:openssl", "nu-cmd-lang/static-link-openssl"]
+static-link-openssl = ["dep:openssl"]
 
 # Optional system clipboard support in `reedline`, this behavior has problematic compatibility with some systems.
 # Missing X server/ Wayland can cause issues
 system-clipboard = [
   "reedline/system_clipboard",
   "nu-cli/system-clipboard",
-  "nu-cmd-lang/system-clipboard",
 ]
 
 # Stable (Default)
-trash-support = ["nu-command/trash-support", "nu-cmd-lang/trash-support"]
+trash-support = ["nu-command/trash-support"]
 
 # SQLite commands for nushell
-sqlite = ["nu-command/sqlite", "nu-cmd-lang/sqlite", "nu-std/sqlite"]
+sqlite = ["nu-command/sqlite", "nu-std/sqlite"]
 
 [profile.release]
 opt-level = "s"     # Optimize for size

benches/benchmarks.rs

@@ -199,7 +199,7 @@ fn bench_record_nested_access(n: usize) -> impl IntoBenchmarks {
     let nested_access = ".col".repeat(n);
     bench_command(
         format!("record_nested_access_{n}"),
-        format!("$record{} | ignore", nested_access),
+        format!("$record{nested_access} | ignore"),
         stack,
         engine,
     )
@@ -319,7 +319,7 @@ fn bench_eval_par_each(n: usize) -> impl IntoBenchmarks {
     let stack = Stack::new();
     bench_command(
         format!("eval_par_each_{n}"),
-        format!("(1..{}) | par-each -t 2 {{|_| 1 }} | ignore", n),
+        format!("(1..{n}) | par-each -t 2 {{|_| 1 }} | ignore"),
         stack,
         engine,
     )
@@ -357,7 +357,7 @@ fn encode_json(row_cnt: usize, col_cnt: usize) -> impl IntoBenchmarks {
     let encoder = Rc::new(EncodingType::try_from_bytes(b"json").unwrap());
 
     [benchmark_fn(
-        format!("encode_json_{}_{}", row_cnt, col_cnt),
+        format!("encode_json_{row_cnt}_{col_cnt}"),
         move |b| {
             let encoder = encoder.clone();
             let test_data = test_data.clone();
@@ -377,7 +377,7 @@ fn encode_msgpack(row_cnt: usize, col_cnt: usize) -> impl IntoBenchmarks {
     let encoder = Rc::new(EncodingType::try_from_bytes(b"msgpack").unwrap());
 
     [benchmark_fn(
-        format!("encode_msgpack_{}_{}", row_cnt, col_cnt),
+        format!("encode_msgpack_{row_cnt}_{col_cnt}"),
         move |b| {
             let encoder = encoder.clone();
             let test_data = test_data.clone();
@@ -399,7 +399,7 @@ fn decode_json(row_cnt: usize, col_cnt: usize) -> impl IntoBenchmarks {
     encoder.encode(&test_data, &mut res).unwrap();
 
     [benchmark_fn(
-        format!("decode_json_{}_{}", row_cnt, col_cnt),
+        format!("decode_json_{row_cnt}_{col_cnt}"),
         move |b| {
             let res = res.clone();
             b.iter(move || {
@@ -422,7 +422,7 @@ fn decode_msgpack(row_cnt: usize, col_cnt: usize) -> impl IntoBenchmarks {
     encoder.encode(&test_data, &mut res).unwrap();
 
     [benchmark_fn(
-        format!("decode_msgpack_{}_{}", row_cnt, col_cnt),
+        format!("decode_msgpack_{row_cnt}_{col_cnt}"),
         move |b| {
             let res = res.clone();
             b.iter(move || {

crates/nu-cli/Cargo.toml

@@ -5,29 +5,29 @@ repository = "https://github.com/nushell/nushell/tree/main/crates/nu-cli"
 edition = "2024"
 license = "MIT"
 name = "nu-cli"
-version = "0.105.1"
+version = "0.106.1"
 
 [lib]
 bench = false
 
 [dev-dependencies]
-nu-cmd-lang = { path = "../nu-cmd-lang", version = "0.105.1" }
+nu-cmd-lang = { path = "../nu-cmd-lang", version = "0.106.1" }
-nu-command = { path = "../nu-command", version = "0.105.1" }
+nu-command = { path = "../nu-command", version = "0.106.1" }
-nu-std = { path = "../nu-std", version = "0.105.1" }
+nu-std = { path = "../nu-std", version = "0.106.1" }
-nu-test-support = { path = "../nu-test-support", version = "0.105.1" }
+nu-test-support = { path = "../nu-test-support", version = "0.106.1" }
 rstest = { workspace = true, default-features = false }
 tempfile = { workspace = true }
 
 [dependencies]
-nu-cmd-base = { path = "../nu-cmd-base", version = "0.105.1" }
+nu-cmd-base = { path = "../nu-cmd-base", version = "0.106.1" }
-nu-engine = { path = "../nu-engine", version = "0.105.1", features = ["os"] }
+nu-engine = { path = "../nu-engine", version = "0.106.1", features = ["os"] }
-nu-glob = { path = "../nu-glob", version = "0.105.1" }
+nu-glob = { path = "../nu-glob", version = "0.106.1" }
-nu-path = { path = "../nu-path", version = "0.105.1" }
+nu-path = { path = "../nu-path", version = "0.106.1" }
-nu-parser = { path = "../nu-parser", version = "0.105.1" }
+nu-parser = { path = "../nu-parser", version = "0.106.1" }
-nu-plugin-engine = { path = "../nu-plugin-engine", version = "0.105.1", optional = true }
+nu-plugin-engine = { path = "../nu-plugin-engine", version = "0.106.1", optional = true }
-nu-protocol = { path = "../nu-protocol", version = "0.105.1", features = ["os"] }
+nu-protocol = { path = "../nu-protocol", version = "0.106.1", features = ["os"] }
-nu-utils = { path = "../nu-utils", version = "0.105.1" }
+nu-utils = { path = "../nu-utils", version = "0.106.1" }
-nu-color-config = { path = "../nu-color-config", version = "0.105.1" }
+nu-color-config = { path = "../nu-color-config", version = "0.106.1" }
 nu-ansi-term = { workspace = true }
 reedline = { workspace = true, features = ["bashisms", "sqlite"] }
 

crates/nu-cli/src/commands/history/history_import.rs

@@ -57,7 +57,7 @@ Note that history item IDs are ignored when importing from file."#
                 result: None,
             },
             Example {
-                example: "[[ command_line cwd ]; [ foo /home ]] | history import",
+                example: "[[ command cwd ]; [ foo /home ]] | history import",
                 description: "Append `foo` ran from `/home` to the current history",
                 result: None,
             },

crates/nu-cli/src/completions/cell_path_completions.rs

@@ -118,7 +118,7 @@ fn get_suggestions_by_value(
             || s.chars()
                 .any(|c: char| !(c.is_ascii_alphabetic() || ['_', '-'].contains(&c)))
         {
-            format!("{:?}", s)
+            format!("{s:?}")
         } else {
             s
         };

crates/nu-cli/src/completions/command_completions.rs

@@ -52,7 +52,7 @@ impl CommandCompletion {
                                 continue;
                             };
                             let value = if matched_internal(&name) {
-                                format!("^{}", name)
+                                format!("^{name}")
                             } else {
                                 name.clone()
                             };

crates/nu-cli/src/completions/completer.rs

@@ -176,7 +176,7 @@ impl NuCompleter {
             &mut working_set,
             Some("completer"),
             // Add a placeholder `a` to the end
-            format!("{}a", line).as_bytes(),
+            format!("{line}a").as_bytes(),
             false,
         );
         self.fetch_completions_by_block(block, &working_set, pos, offset, line, true)
@@ -370,7 +370,8 @@ impl NuCompleter {
                                 FileCompletion,
                             );
 
-                            suggestions.extend(self.process_completion(&mut completer, &ctx));
+                            // Prioritize argument completions over (sub)commands
+                            suggestions.splice(0..0, self.process_completion(&mut completer, &ctx));
                             break;
                         }
 
@@ -384,33 +385,39 @@ impl NuCompleter {
                             };
                             self.process_completion(&mut flag_completions, &ctx)
                         };
-                        suggestions.extend(match arg {
+                        // Prioritize argument completions over (sub)commands
-                            // flags
+                        suggestions.splice(
-                            Argument::Named(_) | Argument::Unknown(_)
+                            0..0,
-                                if prefix.starts_with(b"-") =>
+                            match arg {
-                            {
+                                // flags
-                                flag_completion_helper()
+                                Argument::Named(_) | Argument::Unknown(_)
-                            }
+                                    if prefix.starts_with(b"-") =>
-                            // only when `strip` == false
+                                {
-                            Argument::Positional(_) if prefix == b"-" => flag_completion_helper(),
+                                    flag_completion_helper()
-                            // complete according to expression type and command head
+                                }
-                            Argument::Positional(expr) => {
+                                // only when `strip` == false
-                                let command_head = working_set.get_decl(call.decl_id).name();
+                                Argument::Positional(_) if prefix == b"-" => {
-                                positional_arg_indices.push(arg_idx);
+                                    flag_completion_helper()
-                                self.argument_completion_helper(
+                                }
-                                    PositionalArguments {
+                                // complete according to expression type and command head
-                                        command_head,
+                                Argument::Positional(expr) => {
-                                        positional_arg_indices,
+                                    let command_head = working_set.get_decl(call.decl_id).name();
-                                        arguments: &call.arguments,
+                                    positional_arg_indices.push(arg_idx);
-                                        expr,
+                                    self.argument_completion_helper(
-                                    },
+                                        PositionalArguments {
-                                    pos,
+                                            command_head,
-                                    &ctx,
+                                            positional_arg_indices,
-                                    suggestions.is_empty(),
+                                            arguments: &call.arguments,
-                                )
+                                            expr,
-                            }
+                                        },
-                            _ => vec![],
+                                        pos,
-                        });
+                                        &ctx,
+                                        suggestions.is_empty(),
+                                    )
+                                }
+                                _ => vec![],
+                            },
+                        );
                         break;
                     } else if !matches!(arg, Argument::Named(_)) {
                         positional_arg_indices.push(arg_idx);
@@ -462,10 +469,18 @@ impl NuCompleter {
                             if let Some(external_result) =
                                 self.external_completion(closure, &text_spans, offset, new_span)
                             {
-                                suggestions.extend(external_result);
+                                // Prioritize external results over (sub)commands
+                                suggestions.splice(0..0, external_result);
                                 return suggestions;
                             }
                         }
+                        // for external path arguments with spaces, please check issue #15790
+                        if suggestions.is_empty() {
+                            let (new_span, prefix) =
+                                strip_placeholder_if_any(working_set, &span, strip);
+                            let ctx = Context::new(working_set, new_span, prefix, offset);
+                            return self.process_completion(&mut FileCompletion, &ctx);
+                        }
                         break;
                     }
                 }
@@ -842,7 +857,7 @@ mod completer_tests {
         for (line, has_result, begins_with, expected_values) in dataset {
             let result = completer.fetch_completions_at(line, line.len());
             // Test whether the result is empty or not
-            assert_eq!(!result.is_empty(), has_result, "line: {}", line);
+            assert_eq!(!result.is_empty(), has_result, "line: {line}");
 
             // Test whether the result begins with the expected value
             result
@@ -857,8 +872,7 @@ mod completer_tests {
                     .filter(|x| *x)
                     .count(),
                 expected_values.len(),
-                "line: {}",
+                "line: {line}"
-                line
             );
         }
     }

crates/nu-cli/src/completions/completion_common.rs

@@ -314,7 +314,7 @@ pub fn escape_path(path: String) -> String {
         if path.contains('\'') {
             // decide to use double quotes
             // Path as Debug will do the escaping for `"`, `\`
-            format!("{:?}", path)
+            format!("{path:?}")
         } else {
             format!("'{path}'")
         }

crates/nu-cli/src/completions/completion_options.rs

@@ -102,7 +102,11 @@ impl<T> NuMatcher<'_, T> {
                     options,
                     needle: needle.to_owned(),
                     state: State::Fuzzy {
-                        matcher: Matcher::new(Config::DEFAULT),
+                        matcher: Matcher::new({
+                            let mut cfg = Config::DEFAULT;
+                            cfg.prefer_prefix = true;
+                            cfg
+                        }),
                         atom,
                         items: Vec::new(),
                     },

crates/nu-cli/src/completions/dotnu_completions.rs

@@ -129,7 +129,7 @@ impl Completer for DotNuCompletion {
                     .take_while(|c| "`'\"".contains(*c))
                     .collect::<String>();
                 for path in ["std", "std-rfc"] {
-                    let path = format!("{}{}", surround_prefix, path);
+                    let path = format!("{surround_prefix}{path}");
                     matcher.add(
                         path.clone(),
                         FileSuggestion {
@@ -146,7 +146,7 @@ impl Completer for DotNuCompletion {
                 for sub_vp_id in sub_paths {
                     let (path, sub_vp) = working_set.get_virtual_path(*sub_vp_id);
                     let path = path
-                        .strip_prefix(&format!("{}/", base_dir))
+                        .strip_prefix(&format!("{base_dir}/"))
                         .unwrap_or(path)
                         .to_string();
                     matcher.add(

crates/nu-cli/src/eval_cmds.rs

@@ -3,9 +3,9 @@ use nu_engine::eval_block;
 use nu_parser::parse;
 use nu_protocol::{
     PipelineData, ShellError, Spanned, Value,
-    cli_error::report_compile_error,
     debugger::WithoutDebug,
     engine::{EngineState, Stack, StateWorkingSet},
+    report_error::report_compile_error,
     report_parse_error, report_parse_warning,
 };
 use std::sync::Arc;

crates/nu-cli/src/eval_file.rs

@@ -5,9 +5,9 @@ use nu_parser::parse;
 use nu_path::canonicalize_with;
 use nu_protocol::{
     PipelineData, ShellError, Span, Value,
-    cli_error::report_compile_error,
     debugger::WithoutDebug,
     engine::{EngineState, Stack, StateWorkingSet},
+    report_error::report_compile_error,
     report_parse_error, report_parse_warning,
     shell_error::io::*,
 };

crates/nu-cli/src/menus/help_completions.rs

@@ -1,4 +1,4 @@
-use nu_engine::documentation::{HelpStyle, get_flags_section};
+use nu_engine::documentation::{FormatterValue, HelpStyle, get_flags_section};
 use nu_protocol::{Config, engine::EngineState, levenshtein_distance};
 use nu_utils::IgnoreCaseExt;
 use reedline::{Completer, Suggestion};
@@ -66,8 +66,11 @@ impl NuHelpCompleter {
                 let _ = write!(long_desc, "Usage:\r\n  > {}\r\n", sig.call_signature());
 
                 if !sig.named.is_empty() {
-                    long_desc.push_str(&get_flags_section(&sig, &help_style, |v| {
+                    long_desc.push_str(&get_flags_section(&sig, &help_style, |v| match v {
-                        v.to_parsable_string(", ", &self.config)
+                        FormatterValue::DefaultValue(value) => {
+                            value.to_parsable_string(", ", &self.config)
+                        }
+                        FormatterValue::CodeString(text) => text.to_string(),
                     }))
                 }
 

crates/nu-cli/src/nu_highlight.rs

@@ -3,6 +3,8 @@ use std::sync::Arc;
 use nu_engine::command_prelude::*;
 use reedline::{Highlighter, StyledText};
 
+use crate::syntax_highlight::highlight_syntax;
+
 #[derive(Clone)]
 pub struct NuHighlight;
 
@@ -14,6 +16,11 @@ impl Command for NuHighlight {
     fn signature(&self) -> Signature {
         Signature::build("nu-highlight")
             .category(Category::Strings)
+            .switch(
+                "reject-garbage",
+                "Return an error if invalid syntax (garbage) was encountered",
+                Some('r'),
+            )
             .input_output_types(vec![(Type::String, Type::String)])
     }
 
@@ -32,19 +39,33 @@ impl Command for NuHighlight {
         call: &Call,
         input: PipelineData,
     ) -> Result<PipelineData, ShellError> {
+        let reject_garbage = call.has_flag(engine_state, stack, "reject-garbage")?;
         let head = call.head;
 
         let signals = engine_state.signals();
 
-        let highlighter = crate::NuHighlighter {
+        let engine_state = Arc::new(engine_state.clone());
-            engine_state: Arc::new(engine_state.clone()),
+        let stack = Arc::new(stack.clone());
-            stack: Arc::new(stack.clone()),
-        };
 
         input.map(
             move |x| match x.coerce_into_string() {
                 Ok(line) => {
-                    let highlights = highlighter.highlight(&line, line.len());
+                    let result = highlight_syntax(&engine_state, &stack, &line, line.len());
+
+                    let highlights = match (reject_garbage, result.found_garbage) {
+                        (false, _) => result.text,
+                        (true, None) => result.text,
+                        (true, Some(span)) => {
+                            let error = ShellError::OutsideSpannedLabeledError {
+                                src: line,
+                                error: "encountered invalid syntax while highlighting".into(),
+                                msg: "invalid syntax".into(),
+                                span,
+                            };
+                            return Value::error(error, head);
+                        }
+                    };
+
                     Value::string(highlights.render_simple(), head)
                 }
                 Err(err) => Value::error(err, head),

crates/nu-cli/src/reedline_config.rs

@@ -1172,6 +1172,7 @@ fn edit_from_record(
         "cutfromlinestart" => EditCommand::CutFromLineStart,
         "cuttoend" => EditCommand::CutToEnd,
         "cuttolineend" => EditCommand::CutToLineEnd,
+        "killline" => EditCommand::KillLine,
         "cutwordleft" => EditCommand::CutWordLeft,
         "cutbigwordleft" => EditCommand::CutBigWordLeft,
         "cutwordright" => EditCommand::CutWordRight,

crates/nu-cli/src/repl.rs

@@ -22,8 +22,8 @@ use nu_color_config::StyleComputer;
 use nu_engine::env_to_strings;
 use nu_engine::exit::cleanup_exit;
 use nu_parser::{lex, parse, trim_quotes_str};
-use nu_protocol::shell_error;
 use nu_protocol::shell_error::io::IoError;
+use nu_protocol::{BannerKind, shell_error};
 use nu_protocol::{
     HistoryConfig, HistoryFileFormat, PipelineData, ShellError, Span, Spanned, Value,
     config::NuCursorShape,
@@ -145,8 +145,8 @@ pub fn evaluate_repl(
 
     if load_std_lib.is_none() {
         match engine_state.get_config().show_banner {
-            Value::Bool { val: false, .. } => {}
+            BannerKind::None => {}
-            Value::String { ref val, .. } if val == "short" => {
+            BannerKind::Short => {
                 eval_source(
                     engine_state,
                     &mut unique_stack,
@@ -156,7 +156,7 @@ pub fn evaluate_repl(
                     false,
                 );
             }
-            _ => {
+            BannerKind::Full => {
                 eval_source(
                     engine_state,
                     &mut unique_stack,
@@ -239,7 +239,7 @@ fn escape_special_vscode_bytes(input: &str) -> Result<String, ShellError> {
 
                 match byte {
                     // Escape bytes below 0x20
-                    b if b < 0x20 => format!("\\x{:02X}", byte).into_bytes(),
+                    b if b < 0x20 => format!("\\x{byte:02X}").into_bytes(),
                     // Escape semicolon as \x3B
                     b';' => "\\x3B".to_string().into_bytes(),
                     // Escape backslash as \\
@@ -1097,8 +1097,7 @@ fn run_shell_integration_osc633(
             // If we're in vscode, run their specific ansi escape sequence.
             // This is helpful for ctrl+g to change directories in the terminal.
             run_ansi_sequence(&format!(
-                "{}{}{}",
+                "{VSCODE_CWD_PROPERTY_MARKER_PREFIX}{path}{VSCODE_CWD_PROPERTY_MARKER_SUFFIX}"
-                VSCODE_CWD_PROPERTY_MARKER_PREFIX, path, VSCODE_CWD_PROPERTY_MARKER_SUFFIX
             ));
 
             perf!(
@@ -1114,10 +1113,7 @@ fn run_shell_integration_osc633(
 
             //OSC 633 ; E ; <commandline> [; <nonce] ST - Explicitly set the command line with an optional nonce.
             run_ansi_sequence(&format!(
-                "{}{}{}",
+                "{VSCODE_COMMANDLINE_MARKER_PREFIX}{replaced_cmd_text}{VSCODE_COMMANDLINE_MARKER_SUFFIX}"
-                VSCODE_COMMANDLINE_MARKER_PREFIX,
-                replaced_cmd_text,
-                VSCODE_COMMANDLINE_MARKER_SUFFIX
             ));
         }
     }
@@ -1493,7 +1489,7 @@ mod test_auto_cd {
         // Parse the input. It must be an auto-cd operation.
         let op = parse_operation(input.to_string(), &engine_state, &stack).unwrap();
         let ReplOperation::AutoCd { cwd, target, span } = op else {
-            panic!("'{}' was not parsed into an auto-cd operation", input)
+            panic!("'{input}' was not parsed into an auto-cd operation")
         };
 
         // Perform the auto-cd operation.

crates/nu-cli/src/syntax_highlight.rs

@@ -17,147 +17,173 @@ pub struct NuHighlighter {
 }
 
 impl Highlighter for NuHighlighter {
-    fn highlight(&self, line: &str, _cursor: usize) -> StyledText {
+    fn highlight(&self, line: &str, cursor: usize) -> StyledText {
-        trace!("highlighting: {}", line);
+        let result = highlight_syntax(&self.engine_state, &self.stack, line, cursor);
+        result.text
+    }
+}
 
-        let config = self.stack.get_config(&self.engine_state);
+/// Result of a syntax highlight operation
-        let highlight_resolved_externals = config.highlight_resolved_externals;
+#[derive(Default)]
-        let mut working_set = StateWorkingSet::new(&self.engine_state);
+pub(crate) struct HighlightResult {
-        let block = parse(&mut working_set, None, line.as_bytes(), false);
+    /// The highlighted text
-        let (shapes, global_span_offset) = {
+    pub(crate) text: StyledText,
-            let mut shapes = flatten_block(&working_set, &block);
+    /// The span of any garbage that was highlighted
-            // Highlighting externals has a config point because of concerns that using which to resolve
+    pub(crate) found_garbage: Option<Span>,
-            // externals may slow down things too much.
+}
-            if highlight_resolved_externals {
-                for (span, shape) in shapes.iter_mut() {
-                    if *shape == FlatShape::External {
-                        let str_contents =
-                            working_set.get_span_contents(Span::new(span.start, span.end));
 
-                        let str_word = String::from_utf8_lossy(str_contents).to_string();
+pub(crate) fn highlight_syntax(
-                        let paths = env::path_str(&self.engine_state, &self.stack, *span).ok();
+    engine_state: &EngineState,
-                        #[allow(deprecated)]
+    stack: &Stack,
-                        let res = if let Ok(cwd) =
+    line: &str,
-                            env::current_dir_str(&self.engine_state, &self.stack)
+    cursor: usize,
-                        {
+) -> HighlightResult {
-                            which::which_in(str_word, paths.as_ref(), cwd).ok()
+    trace!("highlighting: {}", line);
-                        } else {
+
-                            which::which_in_global(str_word, paths.as_ref())
+    let config = stack.get_config(engine_state);
-                                .ok()
+    let highlight_resolved_externals = config.highlight_resolved_externals;
-                                .and_then(|mut i| i.next())
+    let mut working_set = StateWorkingSet::new(engine_state);
-                        };
+    let block = parse(&mut working_set, None, line.as_bytes(), false);
-                        if res.is_some() {
+    let (shapes, global_span_offset) = {
-                            *shape = FlatShape::ExternalResolved;
+        let mut shapes = flatten_block(&working_set, &block);
-                        }
+        // Highlighting externals has a config point because of concerns that using which to resolve
+        // externals may slow down things too much.
+        if highlight_resolved_externals {
+            for (span, shape) in shapes.iter_mut() {
+                if *shape == FlatShape::External {
+                    let str_contents =
+                        working_set.get_span_contents(Span::new(span.start, span.end));
+
+                    let str_word = String::from_utf8_lossy(str_contents).to_string();
+                    let paths = env::path_str(engine_state, stack, *span).ok();
+                    let res = if let Ok(cwd) = engine_state.cwd(Some(stack)) {
+                        which::which_in(str_word, paths.as_ref(), cwd).ok()
+                    } else {
+                        which::which_in_global(str_word, paths.as_ref())
+                            .ok()
+                            .and_then(|mut i| i.next())
+                    };
+                    if res.is_some() {
+                        *shape = FlatShape::ExternalResolved;
                     }
                 }
             }
-            (shapes, self.engine_state.next_span_start())
+        }
+        (shapes, engine_state.next_span_start())
+    };
+
+    let mut result = HighlightResult::default();
+    let mut last_seen_span = global_span_offset;
+
+    let global_cursor_offset = cursor + global_span_offset;
+    let matching_brackets_pos = find_matching_brackets(
+        line,
+        &working_set,
+        &block,
+        global_span_offset,
+        global_cursor_offset,
+    );
+
+    for shape in &shapes {
+        if shape.0.end <= last_seen_span
+            || last_seen_span < global_span_offset
+            || shape.0.start < global_span_offset
+        {
+            // We've already output something for this span
+            // so just skip this one
+            continue;
+        }
+        if shape.0.start > last_seen_span {
+            let gap = line
+                [(last_seen_span - global_span_offset)..(shape.0.start - global_span_offset)]
+                .to_string();
+            result.text.push((Style::new(), gap));
+        }
+        let next_token = line
+            [(shape.0.start - global_span_offset)..(shape.0.end - global_span_offset)]
+            .to_string();
+
+        let mut add_colored_token = |shape: &FlatShape, text: String| {
+            result
+                .text
+                .push((get_shape_color(shape.as_str(), &config), text));
         };
 
-        let mut output = StyledText::default();
+        match shape.1 {
-        let mut last_seen_span = global_span_offset;
+            FlatShape::Garbage => {
-
+                result.found_garbage.get_or_insert_with(|| {
-        let global_cursor_offset = _cursor + global_span_offset;
+                    Span::new(
-        let matching_brackets_pos = find_matching_brackets(
+                        shape.0.start - global_span_offset,
-            line,
+                        shape.0.end - global_span_offset,
-            &working_set,
+                    )
-            &block,
+                });
-            global_span_offset,
+                add_colored_token(&shape.1, next_token)
-            global_cursor_offset,
-        );
-
-        for shape in &shapes {
-            if shape.0.end <= last_seen_span
-                || last_seen_span < global_span_offset
-                || shape.0.start < global_span_offset
-            {
-                // We've already output something for this span
-                // so just skip this one
-                continue;
             }
-            if shape.0.start > last_seen_span {
+            FlatShape::Nothing => add_colored_token(&shape.1, next_token),
-                let gap = line
+            FlatShape::Binary => add_colored_token(&shape.1, next_token),
-                    [(last_seen_span - global_span_offset)..(shape.0.start - global_span_offset)]
+            FlatShape::Bool => add_colored_token(&shape.1, next_token),
-                    .to_string();
+            FlatShape::Int => add_colored_token(&shape.1, next_token),
-                output.push((Style::new(), gap));
+            FlatShape::Float => add_colored_token(&shape.1, next_token),
-            }
+            FlatShape::Range => add_colored_token(&shape.1, next_token),
-            let next_token = line
+            FlatShape::InternalCall(_) => add_colored_token(&shape.1, next_token),
-                [(shape.0.start - global_span_offset)..(shape.0.end - global_span_offset)]
+            FlatShape::External => add_colored_token(&shape.1, next_token),
-                .to_string();
+            FlatShape::ExternalArg => add_colored_token(&shape.1, next_token),
-
+            FlatShape::ExternalResolved => add_colored_token(&shape.1, next_token),
-            let mut add_colored_token = |shape: &FlatShape, text: String| {
+            FlatShape::Keyword => add_colored_token(&shape.1, next_token),
-                output.push((get_shape_color(shape.as_str(), &config), text));
+            FlatShape::Literal => add_colored_token(&shape.1, next_token),
-            };
+            FlatShape::Operator => add_colored_token(&shape.1, next_token),
-
+            FlatShape::Signature => add_colored_token(&shape.1, next_token),
-            match shape.1 {
+            FlatShape::String => add_colored_token(&shape.1, next_token),
-                FlatShape::Garbage => add_colored_token(&shape.1, next_token),
+            FlatShape::RawString => add_colored_token(&shape.1, next_token),
-                FlatShape::Nothing => add_colored_token(&shape.1, next_token),
+            FlatShape::StringInterpolation => add_colored_token(&shape.1, next_token),
-                FlatShape::Binary => add_colored_token(&shape.1, next_token),
+            FlatShape::DateTime => add_colored_token(&shape.1, next_token),
-                FlatShape::Bool => add_colored_token(&shape.1, next_token),
+            FlatShape::List
-                FlatShape::Int => add_colored_token(&shape.1, next_token),
+            | FlatShape::Table
-                FlatShape::Float => add_colored_token(&shape.1, next_token),
+            | FlatShape::Record
-                FlatShape::Range => add_colored_token(&shape.1, next_token),
+            | FlatShape::Block
-                FlatShape::InternalCall(_) => add_colored_token(&shape.1, next_token),
+            | FlatShape::Closure => {
-                FlatShape::External => add_colored_token(&shape.1, next_token),
+                let span = shape.0;
-                FlatShape::ExternalArg => add_colored_token(&shape.1, next_token),
+                let shape = &shape.1;
-                FlatShape::ExternalResolved => add_colored_token(&shape.1, next_token),
+                let spans = split_span_by_highlight_positions(
-                FlatShape::Keyword => add_colored_token(&shape.1, next_token),
+                    line,
-                FlatShape::Literal => add_colored_token(&shape.1, next_token),
+                    span,
-                FlatShape::Operator => add_colored_token(&shape.1, next_token),
+                    &matching_brackets_pos,
-                FlatShape::Signature => add_colored_token(&shape.1, next_token),
+                    global_span_offset,
-                FlatShape::String => add_colored_token(&shape.1, next_token),
+                );
-                FlatShape::RawString => add_colored_token(&shape.1, next_token),
+                for (part, highlight) in spans {
-                FlatShape::StringInterpolation => add_colored_token(&shape.1, next_token),
+                    let start = part.start - span.start;
-                FlatShape::DateTime => add_colored_token(&shape.1, next_token),
+                    let end = part.end - span.start;
-                FlatShape::List
+                    let text = next_token[start..end].to_string();
-                | FlatShape::Table
+                    let mut style = get_shape_color(shape.as_str(), &config);
-                | FlatShape::Record
+                    if highlight {
-                | FlatShape::Block
+                        style = get_matching_brackets_style(style, &config);
-                | FlatShape::Closure => {
-                    let span = shape.0;
-                    let shape = &shape.1;
-                    let spans = split_span_by_highlight_positions(
-                        line,
-                        span,
-                        &matching_brackets_pos,
-                        global_span_offset,
-                    );
-                    for (part, highlight) in spans {
-                        let start = part.start - span.start;
-                        let end = part.end - span.start;
-                        let text = next_token[start..end].to_string();
-                        let mut style = get_shape_color(shape.as_str(), &config);
-                        if highlight {
-                            style = get_matching_brackets_style(style, &config);
-                        }
-                        output.push((style, text));
                     }
+                    result.text.push((style, text));
                 }
-
-                FlatShape::Filepath => add_colored_token(&shape.1, next_token),
-                FlatShape::Directory => add_colored_token(&shape.1, next_token),
-                FlatShape::GlobInterpolation => add_colored_token(&shape.1, next_token),
-                FlatShape::GlobPattern => add_colored_token(&shape.1, next_token),
-                FlatShape::Variable(_) | FlatShape::VarDecl(_) => {
-                    add_colored_token(&shape.1, next_token)
-                }
-                FlatShape::Flag => add_colored_token(&shape.1, next_token),
-                FlatShape::Pipe => add_colored_token(&shape.1, next_token),
-                FlatShape::Redirection => add_colored_token(&shape.1, next_token),
-                FlatShape::Custom(..) => add_colored_token(&shape.1, next_token),
-                FlatShape::MatchPattern => add_colored_token(&shape.1, next_token),
             }
-            last_seen_span = shape.0.end;
-        }
 
-        let remainder = line[(last_seen_span - global_span_offset)..].to_string();
+            FlatShape::Filepath => add_colored_token(&shape.1, next_token),
-        if !remainder.is_empty() {
+            FlatShape::Directory => add_colored_token(&shape.1, next_token),
-            output.push((Style::new(), remainder));
+            FlatShape::GlobInterpolation => add_colored_token(&shape.1, next_token),
+            FlatShape::GlobPattern => add_colored_token(&shape.1, next_token),
+            FlatShape::Variable(_) | FlatShape::VarDecl(_) => {
+                add_colored_token(&shape.1, next_token)
+            }
+            FlatShape::Flag => add_colored_token(&shape.1, next_token),
+            FlatShape::Pipe => add_colored_token(&shape.1, next_token),
+            FlatShape::Redirection => add_colored_token(&shape.1, next_token),
+            FlatShape::Custom(..) => add_colored_token(&shape.1, next_token),
+            FlatShape::MatchPattern => add_colored_token(&shape.1, next_token),
         }
-
+        last_seen_span = shape.0.end;
-        output
     }
+
+    let remainder = line[(last_seen_span - global_span_offset)..].to_string();
+    if !remainder.is_empty() {
+        result.text.push((Style::new(), remainder));
+    }
+
+    result
 }
 
 fn split_span_by_highlight_positions(

crates/nu-cli/src/util.rs

@@ -5,9 +5,9 @@ use nu_engine::{eval_block, eval_block_with_early_return};
 use nu_parser::{Token, TokenContents, lex, parse, unescape_unquote_string};
 use nu_protocol::{
     PipelineData, ShellError, Span, Value,
-    cli_error::report_compile_error,
     debugger::WithoutDebug,
     engine::{EngineState, Stack, StateWorkingSet},
+    report_error::report_compile_error,
     report_parse_error, report_parse_warning, report_shell_error,
 };
 #[cfg(windows)]

crates/nu-cli/tests/commands/nu_highlight.rs

@@ -5,3 +5,9 @@ fn nu_highlight_not_expr() {
     let actual = nu!("'not false' | nu-highlight | ansi strip");
     assert_eq!(actual.out, "not false");
 }
+
+#[test]
+fn nu_highlight_where_row_condition() {
+    let actual = nu!("'ls | where a b 12345(' | nu-highlight | ansi strip");
+    assert_eq!(actual.out, "ls | where a b 12345(");
+}

crates/nu-cli/tests/completions/mod.rs

@@ -9,14 +9,16 @@ use std::{
 use nu_cli::NuCompleter;
 use nu_engine::eval_block;
 use nu_parser::parse;
-use nu_path::expand_tilde;
+use nu_path::{AbsolutePathBuf, expand_tilde};
 use nu_protocol::{Config, PipelineData, debugger::WithoutDebug, engine::StateWorkingSet};
 use nu_std::load_standard_library;
+use nu_test_support::fs;
 use reedline::{Completer, Suggestion};
 use rstest::{fixture, rstest};
 use support::{
     completions_helpers::{
-        new_dotnu_engine, new_external_engine, new_partial_engine, new_quote_engine,
+        new_dotnu_engine, new_engine_helper, new_external_engine, new_partial_engine,
+        new_quote_engine,
     },
     file, folder, match_suggestions, match_suggestions_by_string, new_engine,
 };
@@ -123,7 +125,7 @@ fn custom_completer_with_options(
         global_opts,
         completions
             .iter()
-            .map(|comp| format!("'{}'", comp))
+            .map(|comp| format!("'{comp}'"))
             .collect::<Vec<_>>()
             .join(", "),
         completer_opts,
@@ -307,10 +309,10 @@ fn custom_arguments_and_subcommands() {
     let suggestions = completer.complete(completion_str, completion_str.len());
     // including both subcommand and directory completions
     let expected = [
-        "foo test bar".into(),
         folder("test_a"),
         file("test_a_symlink"),
         folder("test_b"),
+        "foo test bar".into(),
     ];
     match_suggestions_by_string(&expected, &suggestions);
 }
@@ -328,7 +330,7 @@ fn custom_flags_and_subcommands() {
     let completion_str = "foo --test";
     let suggestions = completer.complete(completion_str, completion_str.len());
     // including both flag and directory completions
-    let expected: Vec<_> = vec!["foo --test bar", "--test"];
+    let expected: Vec<_> = vec!["--test", "foo --test bar"];
     match_suggestions(&expected, &suggestions);
 }
 
@@ -716,6 +718,16 @@ fn external_completer_fallback() {
     let expected = [folder("test_a"), file("test_a_symlink"), folder("test_b")];
     let suggestions = run_external_completion(block, input);
     match_suggestions_by_string(&expected, &suggestions);
+
+    // issue #15790
+    let input = "foo `dir with space/`";
+    let expected = vec!["`dir with space/bar baz`", "`dir with space/foo`"];
+    let suggestions = run_external_completion_within_pwd(
+        block,
+        input,
+        fs::fixtures().join("external_completions"),
+    );
+    match_suggestions(&expected, &suggestions);
 }
 
 /// Fallback to external completions for flags of `sudo`
@@ -1468,11 +1480,12 @@ fn command_watch_with_filecompletion() {
     match_suggestions(&expected_paths, &suggestions)
 }
 
-#[rstest]
+#[test]
-fn subcommand_completions() {
+fn subcommand_vs_external_completer() {
     let (_, _, mut engine, mut stack) = new_engine();
     let commands = r#"
             $env.config.completions.algorithm = "fuzzy"
+            $env.config.completions.external.completer = {|spans| ["external"]}
             def foo-test-command [] {}
             def "foo-test-command bar" [] {}
             def "foo-test-command aagap bcr" [] {}
@@ -1485,6 +1498,7 @@ fn subcommand_completions() {
     let suggestions = subcommand_completer.complete(prefix, prefix.len());
     match_suggestions(
         &vec![
+            "external",
             "food bar",
             "foo-test-command bar",
             "foo-test-command aagap bcr",
@@ -1494,7 +1508,7 @@ fn subcommand_completions() {
 
     let prefix = "foot bar";
     let suggestions = subcommand_completer.complete(prefix, prefix.len());
-    match_suggestions(&vec!["foo-test-command bar"], &suggestions);
+    match_suggestions(&vec!["external", "foo-test-command bar"], &suggestions);
 }
 
 #[test]
@@ -2103,11 +2117,15 @@ fn alias_of_another_alias() {
     match_suggestions(&expected_paths, &suggestions)
 }
 
-fn run_external_completion(completer: &str, input: &str) -> Vec<Suggestion> {
+fn run_external_completion_within_pwd(
+    completer: &str,
+    input: &str,
+    pwd: AbsolutePathBuf,
+) -> Vec<Suggestion> {
     let completer = format!("$env.config.completions.external.completer = {completer}");
 
     // Create a new engine
-    let (_, _, mut engine_state, mut stack) = new_engine();
+    let (_, _, mut engine_state, mut stack) = new_engine_helper(pwd);
     let (block, delta) = {
         let mut working_set = StateWorkingSet::new(&engine_state);
         let block = parse(&mut working_set, None, completer.as_bytes(), false);
@@ -2131,6 +2149,10 @@ fn run_external_completion(completer: &str, input: &str) -> Vec<Suggestion> {
     completer.complete(input, input.len())
 }
 
+fn run_external_completion(completer: &str, input: &str) -> Vec<Suggestion> {
+    run_external_completion_within_pwd(completer, input, fs::fixtures().join("completions"))
+}
+
 #[test]
 fn unknown_command_completion() {
     let (_, _, engine, stack) = new_engine();

crates/nu-cmd-base/Cargo.toml

@@ -5,7 +5,7 @@ edition = "2024"
 license = "MIT"
 name = "nu-cmd-base"
 repository = "https://github.com/nushell/nushell/tree/main/crates/nu-cmd-base"
-version = "0.105.1"
+version = "0.106.1"
 
 # See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
 
@@ -13,10 +13,10 @@ version = "0.105.1"
 workspace = true
 
 [dependencies]
-nu-engine = { path = "../nu-engine", version = "0.105.1", default-features = false }
+nu-engine = { path = "../nu-engine", version = "0.106.1", default-features = false }
-nu-parser = { path = "../nu-parser", version = "0.105.1" }
+nu-parser = { path = "../nu-parser", version = "0.106.1" }
-nu-path = { path = "../nu-path", version = "0.105.1" }
+nu-path = { path = "../nu-path", version = "0.106.1" }
-nu-protocol = { path = "../nu-protocol", version = "0.105.1", default-features = false }
+nu-protocol = { path = "../nu-protocol", version = "0.106.1", default-features = false }
 
 indexmap = { workspace = true }
 miette = { workspace = true }

crates/nu-cmd-base/src/hook.rs

@@ -1,11 +1,11 @@
 use miette::Result;
-use nu_engine::{eval_block, eval_block_with_early_return};
+use nu_engine::{eval_block, eval_block_with_early_return, redirect_env};
 use nu_parser::parse;
 use nu_protocol::{
     PipelineData, PositionalArg, ShellError, Span, Type, Value, VarId,
-    cli_error::{report_parse_error, report_shell_error},
     debugger::WithoutDebug,
     engine::{Closure, EngineState, Stack, StateWorkingSet},
+    report_error::{report_parse_error, report_shell_error},
 };
 use std::{collections::HashMap, sync::Arc};
 
@@ -325,19 +325,7 @@ fn run_hook(
     }
 
     // If all went fine, preserve the environment of the called block
-    let caller_env_vars = stack.get_env_var_names(engine_state);
+    redirect_env(engine_state, stack, &callee_stack);
 
-    // remove env vars that are present in the caller but not in the callee
-    // (the callee hid them)
-    for var in caller_env_vars.iter() {
-        if !callee_stack.has_env_var(engine_state, var) {
-            stack.remove_env_var(engine_state, var);
-        }
-    }
-
-    // add new env vars from callee to caller
-    for (var, value) in callee_stack.get_stack_env_vars() {
-        stack.add_env_var(var, value);
-    }
     Ok(pipeline_data)
 }

crates/nu-cmd-extra/Cargo.toml

@@ -5,7 +5,7 @@ edition = "2024"
 license = "MIT"
 name = "nu-cmd-extra"
 repository = "https://github.com/nushell/nushell/tree/main/crates/nu-cmd-extra"
-version = "0.105.1"
+version = "0.106.1"
 
 # See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
 
@@ -16,13 +16,13 @@ bench = false
 workspace = true
 
 [dependencies]
-nu-cmd-base = { path = "../nu-cmd-base", version = "0.105.1" }
+nu-cmd-base = { path = "../nu-cmd-base", version = "0.106.1" }
-nu-engine = { path = "../nu-engine", version = "0.105.1", default-features = false }
+nu-engine = { path = "../nu-engine", version = "0.106.1", default-features = false }
-nu-json = { version = "0.105.1", path = "../nu-json" }
+nu-json = { version = "0.106.1", path = "../nu-json" }
-nu-parser = { path = "../nu-parser", version = "0.105.1" }
+nu-parser = { path = "../nu-parser", version = "0.106.1" }
-nu-pretty-hex = { version = "0.105.1", path = "../nu-pretty-hex" }
+nu-pretty-hex = { version = "0.106.1", path = "../nu-pretty-hex" }
-nu-protocol = { path = "../nu-protocol", version = "0.105.1", default-features = false }
+nu-protocol = { path = "../nu-protocol", version = "0.106.1", default-features = false }
-nu-utils = { path = "../nu-utils", version = "0.105.1", default-features = false }
+nu-utils = { path = "../nu-utils", version = "0.106.1", default-features = false }
 
 # Potential dependencies for extras
 heck = { workspace = true }
@@ -37,6 +37,6 @@ itertools = { workspace = true }
 mime = { workspace = true }
 
 [dev-dependencies]
-nu-cmd-lang = { path = "../nu-cmd-lang", version = "0.105.1" }
+nu-cmd-lang = { path = "../nu-cmd-lang", version = "0.106.1" }
-nu-command = { path = "../nu-command", version = "0.105.1" }
+nu-command = { path = "../nu-command", version = "0.106.1" }
-nu-test-support = { path = "../nu-test-support", version = "0.105.1" }
+nu-test-support = { path = "../nu-test-support", version = "0.106.1" }

crates/nu-cmd-extra/src/extra/filters/update_cells.rs

@@ -12,7 +12,10 @@ impl Command for UpdateCells {
 
     fn signature(&self) -> Signature {
         Signature::build("update cells")
-            .input_output_types(vec![(Type::table(), Type::table())])
+            .input_output_types(vec![
+                (Type::table(), Type::table()),
+                (Type::record(), Type::record()),
+            ])
             .required(
                 "closure",
                 SyntaxShape::Closure(Some(vec![SyntaxShape::Any])),
@@ -77,6 +80,15 @@ impl Command for UpdateCells {
                     "2021-11-18" => Value::test_string(""),
                 })])),
             },
+            Example {
+                example: r#"{a: 1, b: 2, c: 3} | update cells { $in + 10 }"#,
+                description: "Update each value in a record.",
+                result: Some(Value::test_record(record! {
+                    "a" => Value::test_int(11),
+                    "b" => Value::test_int(12),
+                    "c" => Value::test_int(13),
+                })),
+            },
         ]
     }
 
@@ -85,7 +97,7 @@ impl Command for UpdateCells {
         engine_state: &EngineState,
         stack: &mut Stack,
         call: &Call,
-        input: PipelineData,
+        mut input: PipelineData,
     ) -> Result<PipelineData, ShellError> {
         let head = call.head;
         let closure: Closure = call.req(engine_state, stack, 0)?;
@@ -102,14 +114,51 @@ impl Command for UpdateCells {
 
         let metadata = input.metadata();
 
-        Ok(UpdateCellIterator {
+        match input {
-            iter: input.into_iter(),
+            PipelineData::Value(
-            closure: ClosureEval::new(engine_state, stack, closure),
+                Value::Record {
-            columns,
+                    ref mut val,
-            span: head,
+                    internal_span,
+                },
+                ..,
+            ) => {
+                let val = val.to_mut();
+                update_record(
+                    val,
+                    &mut ClosureEval::new(engine_state, stack, closure),
+                    internal_span,
+                    columns.as_ref(),
+                );
+                Ok(input)
+            }
+            _ => Ok(UpdateCellIterator {
+                iter: input.into_iter(),
+                closure: ClosureEval::new(engine_state, stack, closure),
+                columns,
+                span: head,
+            }
+            .into_pipeline_data(head, engine_state.signals().clone())
+            .set_metadata(metadata)),
+        }
+    }
+}
+
+fn update_record(
+    record: &mut Record,
+    closure: &mut ClosureEval,
+    span: Span,
+    cols: Option<&HashSet<String>>,
+) {
+    if let Some(columns) = cols {
+        for (col, val) in record.iter_mut() {
+            if columns.contains(col) {
+                *val = eval_value(closure, span, std::mem::take(val));
+            }
+        }
+    } else {
+        for (_, val) in record.iter_mut() {
+            *val = eval_value(closure, span, std::mem::take(val))
         }
-        .into_pipeline_data(head, engine_state.signals().clone())
-        .set_metadata(metadata))
     }
 }
 
@@ -128,18 +177,7 @@ impl Iterator for UpdateCellIterator {
 
         let value = if let Value::Record { val, .. } = &mut value {
             let val = val.to_mut();
-            if let Some(columns) = &self.columns {
+            update_record(val, &mut self.closure, self.span, self.columns.as_ref());
-                for (col, val) in val.iter_mut() {
-                    if columns.contains(col) {
-                        *val = eval_value(&mut self.closure, self.span, std::mem::take(val));
-                    }
-                }
-            } else {
-                for (_, val) in val.iter_mut() {
-                    *val = eval_value(&mut self.closure, self.span, std::mem::take(val))
-                }
-            }
-
             value
         } else {
             eval_value(&mut self.closure, self.span, value)

crates/nu-cmd-extra/src/extra/formats/to/html.rs

@@ -188,7 +188,7 @@ fn get_theme_from_asset_file(
         Some(t) => t,
         None => {
             return Err(ShellError::TypeMismatch {
-                err_message: format!("Unknown HTML theme '{}'", theme_name),
+                err_message: format!("Unknown HTML theme '{theme_name}'"),
                 span: theme_span,
             });
         }
@@ -774,8 +774,7 @@ mod tests {
         for key in required_keys {
             assert!(
                 theme_map.contains_key(key),
-                "Expected theme to contain key '{}'",
+                "Expected theme to contain key '{key}'"
-                key
             );
         }
     }
@@ -792,15 +791,13 @@ mod tests {
         if let Err(err) = result {
             assert!(
                 matches!(err, ShellError::TypeMismatch { .. }),
-                "Expected TypeMismatch error, got: {:?}",
+                "Expected TypeMismatch error, got: {err:?}"
-                err
             );
 
             if let ShellError::TypeMismatch { err_message, span } = err {
                 assert!(
                     err_message.contains("doesnt-exist"),
-                    "Error message should mention theme name, got: {}",
+                    "Error message should mention theme name, got: {err_message}"
-                    err_message
                 );
                 assert_eq!(span.start, 0);
                 assert_eq!(span.end, 13);

crates/nu-cmd-extra/src/extra/strings/format/bits.rs

@@ -161,28 +161,28 @@ fn convert_to_smallest_number_type(num: i64, span: Span) -> Value {
         let bytes = v.to_ne_bytes();
         let mut raw_string = "".to_string();
         for ch in bytes {
-            raw_string.push_str(&format!("{:08b} ", ch));
+            raw_string.push_str(&format!("{ch:08b} "));
         }
         Value::string(raw_string.trim(), span)
     } else if let Some(v) = num.to_i16() {
         let bytes = v.to_ne_bytes();
         let mut raw_string = "".to_string();
         for ch in bytes {
-            raw_string.push_str(&format!("{:08b} ", ch));
+            raw_string.push_str(&format!("{ch:08b} "));
         }
         Value::string(raw_string.trim(), span)
     } else if let Some(v) = num.to_i32() {
         let bytes = v.to_ne_bytes();
         let mut raw_string = "".to_string();
         for ch in bytes {
-            raw_string.push_str(&format!("{:08b} ", ch));
+            raw_string.push_str(&format!("{ch:08b} "));
         }
         Value::string(raw_string.trim(), span)
     } else {
         let bytes = num.to_ne_bytes();
         let mut raw_string = "".to_string();
         for ch in bytes {
-            raw_string.push_str(&format!("{:08b} ", ch));
+            raw_string.push_str(&format!("{ch:08b} "));
         }
         Value::string(raw_string.trim(), span)
     }
@@ -193,7 +193,7 @@ fn action(input: &Value, _args: &Arguments, span: Span) -> Value {
         Value::Binary { val, .. } => {
             let mut raw_string = "".to_string();
             for ch in val {
-                raw_string.push_str(&format!("{:08b} ", ch));
+                raw_string.push_str(&format!("{ch:08b} "));
             }
             Value::string(raw_string.trim(), span)
         }
@@ -204,7 +204,7 @@ fn action(input: &Value, _args: &Arguments, span: Span) -> Value {
             let raw_bytes = val.as_bytes();
             let mut raw_string = "".to_string();
             for ch in raw_bytes {
-                raw_string.push_str(&format!("{:08b} ", ch));
+                raw_string.push_str(&format!("{ch:08b} "));
             }
             Value::string(raw_string.trim(), span)
         }

crates/nu-cmd-extra/src/extra/strings/format/number.rs

@@ -16,6 +16,11 @@ impl Command for FormatNumber {
     fn signature(&self) -> nu_protocol::Signature {
         Signature::build("format number")
             .input_output_types(vec![(Type::Number, Type::record())])
+            .switch(
+                "no-prefix",
+                "don't include the binary, hex or octal prefixes",
+                Some('n'),
+            )
             .category(Category::Conversions)
     }
 
@@ -24,20 +29,36 @@ impl Command for FormatNumber {
     }
 
     fn examples(&self) -> Vec<Example> {
-        vec![Example {
+        vec![
-            description: "Get a record containing multiple formats for the number 42",
+            Example {
-            example: "42 | format number",
+                description: "Get a record containing multiple formats for the number 42",
-            result: Some(Value::test_record(record! {
+                example: "42 | format number",
-                    "binary" =>   Value::test_string("0b101010"),
+                result: Some(Value::test_record(record! {
-                    "debug" =>    Value::test_string("42"),
+                        "debug" =>    Value::test_string("42"),
-                    "display" =>  Value::test_string("42"),
+                        "display" =>  Value::test_string("42"),
-                    "lowerexp" => Value::test_string("4.2e1"),
+                        "binary" =>   Value::test_string("0b101010"),
-                    "lowerhex" => Value::test_string("0x2a"),
+                        "lowerexp" => Value::test_string("4.2e1"),
-                    "octal" =>    Value::test_string("0o52"),
+                        "upperexp" => Value::test_string("4.2E1"),
-                    "upperexp" => Value::test_string("4.2E1"),
+                        "lowerhex" => Value::test_string("0x2a"),
-                    "upperhex" => Value::test_string("0x2A"),
+                        "upperhex" => Value::test_string("0x2A"),
-            })),
+                        "octal" =>    Value::test_string("0o52"),
-        }]
+                })),
+            },
+            Example {
+                description: "Format float without prefixes",
+                example: "3.14 | format number --no-prefix",
+                result: Some(Value::test_record(record! {
+                        "debug" =>    Value::test_string("3.14"),
+                        "display" =>  Value::test_string("3.14"),
+                        "binary" =>   Value::test_string("100000000001001000111101011100001010001111010111000010100011111"),
+                        "lowerexp" => Value::test_string("3.14e0"),
+                        "upperexp" => Value::test_string("3.14E0"),
+                        "lowerhex" => Value::test_string("40091eb851eb851f"),
+                        "upperhex" => Value::test_string("40091EB851EB851F"),
+                        "octal" =>    Value::test_string("400110753412172702437"),
+                })),
+            },
+        ]
     }
 
     fn run(
@@ -59,14 +80,24 @@ pub(crate) fn format_number(
 ) -> Result<PipelineData, ShellError> {
     let cell_paths: Vec<CellPath> = call.rest(engine_state, stack, 0)?;
     let args = CellPathOnlyArgs::from(cell_paths);
-    operate(action, args, input, call.head, engine_state.signals())
+    if call.has_flag(engine_state, stack, "no-prefix")? {
+        operate(
+            action_no_prefix,
+            args,
+            input,
+            call.head,
+            engine_state.signals(),
+        )
+    } else {
+        operate(action, args, input, call.head, engine_state.signals())
+    }
 }
 
 fn action(input: &Value, _args: &CellPathOnlyArgs, span: Span) -> Value {
     match input {
-        Value::Float { val, .. } => format_f64(*val, span),
+        Value::Float { val, .. } => format_f64(*val, false, span),
-        Value::Int { val, .. } => format_i64(*val, span),
+        Value::Int { val, .. } => format_i64(*val, false, span),
-        Value::Filesize { val, .. } => format_i64(val.get(), span),
+        Value::Filesize { val, .. } => format_i64(val.get(), false, span),
         // Propagate errors by explicitly matching them before the final case.
         Value::Error { .. } => input.clone(),
         other => Value::error(
@@ -81,33 +112,80 @@ fn action(input: &Value, _args: &CellPathOnlyArgs, span: Span) -> Value {
     }
 }
 
-fn format_i64(num: i64, span: Span) -> Value {
+fn action_no_prefix(input: &Value, _args: &CellPathOnlyArgs, span: Span) -> Value {
+    match input {
+        Value::Float { val, .. } => format_f64(*val, true, span),
+        Value::Int { val, .. } => format_i64(*val, true, span),
+        Value::Filesize { val, .. } => format_i64(val.get(), true, span),
+        // Propagate errors by explicitly matching them before the final case.
+        Value::Error { .. } => input.clone(),
+        other => Value::error(
+            ShellError::OnlySupportsThisInputType {
+                exp_input_type: "float, int, or filesize".into(),
+                wrong_type: other.get_type().to_string(),
+                dst_span: span,
+                src_span: other.span(),
+            },
+            span,
+        ),
+    }
+}
+
+fn format_i64(num: i64, no_prefix: bool, span: Span) -> Value {
     Value::record(
         record! {
-            "binary" => Value::string(format!("{num:#b}"), span),
             "debug" => Value::string(format!("{num:#?}"), span),
             "display" => Value::string(format!("{num}"), span),
+            "binary" => Value::string(
+                if no_prefix { format!("{num:b}") } else { format!("{num:#b}") },
+                span,
+            ),
             "lowerexp" => Value::string(format!("{num:#e}"), span),
-            "lowerhex" => Value::string(format!("{num:#x}"), span),
-            "octal" => Value::string(format!("{num:#o}"), span),
             "upperexp" => Value::string(format!("{num:#E}"), span),
-            "upperhex" => Value::string(format!("{num:#X}"), span),
+            "lowerhex" => Value::string(
+                if no_prefix { format!("{num:x}") } else { format!("{num:#x}") },
+                span,
+            ),
+            "upperhex" => Value::string(
+                if no_prefix { format!("{num:X}") } else { format!("{num:#X}") },
+                span,
+            ),
+            "octal" => Value::string(
+                if no_prefix { format!("{num:o}") } else { format!("{num:#o}") },
+                span,
+            )
         },
         span,
     )
 }
 
-fn format_f64(num: f64, span: Span) -> Value {
+fn format_f64(num: f64, no_prefix: bool, span: Span) -> Value {
     Value::record(
         record! {
-            "binary" => Value::string(format!("{:b}", num.to_bits()), span),
             "debug" => Value::string(format!("{num:#?}"), span),
             "display" => Value::string(format!("{num}"), span),
+            "binary" => Value::string(
+                if no_prefix {
+                    format!("{:b}", num.to_bits())
+                } else {
+                    format!("{:#b}", num.to_bits())
+                },
+                span,
+            ),
             "lowerexp" => Value::string(format!("{num:#e}"), span),
-            "lowerhex" => Value::string(format!("{:0x}", num.to_bits()), span),
-            "octal" => Value::string(format!("{:0o}", num.to_bits()), span),
             "upperexp" => Value::string(format!("{num:#E}"), span),
-            "upperhex" => Value::string(format!("{:0X}", num.to_bits()), span),
+            "lowerhex" => Value::string(
+                if no_prefix { format!("{:x}", num.to_bits()) } else { format!("{:#x}", num.to_bits()) },
+                span,
+            ),
+            "upperhex" => Value::string(
+                if no_prefix { format!("{:X}", num.to_bits()) } else { format!("{:#X}", num.to_bits()) },
+                span,
+            ),
+            "octal" => Value::string(
+                if no_prefix { format!("{:o}", num.to_bits()) } else { format!("{:#o}", num.to_bits()) },
+                span,
+            )
         },
         span,
     )

crates/nu-cmd-lang/Cargo.toml

@@ -6,7 +6,7 @@ repository = "https://github.com/nushell/nushell/tree/main/crates/nu-cmd-lang"
 edition = "2024"
 license = "MIT"
 name = "nu-cmd-lang"
-version = "0.105.1"
+version = "0.106.1"
 
 [lib]
 bench = false
@@ -15,17 +15,18 @@ bench = false
 workspace = true
 
 [dependencies]
-nu-engine = { path = "../nu-engine", version = "0.105.1", default-features = false }
+nu-engine = { path = "../nu-engine", version = "0.106.1", default-features = false }
-nu-parser = { path = "../nu-parser", version = "0.105.1" }
+nu-experimental = { path = "../nu-experimental", version = "0.106.1" }
-nu-protocol = { path = "../nu-protocol", version = "0.105.1", default-features = false }
+nu-parser = { path = "../nu-parser", version = "0.106.1" }
-nu-utils = { path = "../nu-utils", version = "0.105.1", default-features = false }
+nu-protocol = { path = "../nu-protocol", version = "0.106.1", default-features = false }
-nu-cmd-base = { path = "../nu-cmd-base", version = "0.105.1" }
+nu-utils = { path = "../nu-utils", version = "0.106.1", default-features = false }
+nu-cmd-base = { path = "../nu-cmd-base", version = "0.106.1" }
 
 itertools = { workspace = true }
-shadow-rs = { version = "1.1", default-features = false }
+shadow-rs = { version = "1.2", default-features = false }
 
 [build-dependencies]
-shadow-rs = { version = "1.1", default-features = false, features = ["build"] }
+shadow-rs = { version = "1.2", default-features = false, features = ["build"] }
 
 [dev-dependencies]
 quickcheck = { workspace = true }
@@ -43,8 +44,3 @@ plugin = [
   "nu-protocol/plugin",
   "os",
 ]
-
-trash-support = []
-sqlite = []
-static-link-openssl = []
-system-clipboard = []

crates/nu-cmd-lang/src/core_commands/describe.rs

@@ -296,7 +296,7 @@ fn run(
             } else {
                 let value = stream.into_value();
                 let base_description = value.get_type().to_string();
-                Value::string(format!("{} (stream)", base_description), head)
+                Value::string(format!("{base_description} (stream)"), head)
             }
         }
         PipelineData::Value(value, ..) => {

crates/nu-cmd-lang/src/core_commands/do_.rs

@@ -264,7 +264,7 @@ fn bind_args_to(
             .expect("internal error: all custom parameters must have var_ids");
         if let Some(result) = val_iter.next() {
             let param_type = param.shape.to_type();
-            if required && !result.is_subtype_of(&param_type) {
+            if !result.is_subtype_of(&param_type) {
                 return Err(ShellError::CantConvert {
                     to_type: param.shape.to_type().to_string(),
                     from_type: result.get_type().to_string(),

crates/nu-cmd-lang/src/core_commands/error_make.rs

@@ -229,7 +229,7 @@ fn make_other_error(value: &Value, throw_span: Option<Span>) -> ShellError {
             error: "invalid error format.".into(),
             msg: "`$.label.start` should be smaller than `$.label.end`".into(),
             span: Some(label_span),
-            help: Some(format!("{} > {}", span_start, span_end)),
+            help: Some(format!("{span_start} > {span_end}")),
             inner: vec![],
         };
     }

crates/nu-cmd-lang/src/core_commands/if_.rs

@@ -60,11 +60,13 @@ impl Command for If {
     ) -> Result<PipelineData, ShellError> {
         let call = call.assert_ast_call()?;
         let cond = call.positional_nth(0).expect("checked through parser");
-        let then_block = call
+        let then_expr = call.positional_nth(1).expect("checked through parser");
-            .positional_nth(1)
+        let then_block = then_expr
-            .expect("checked through parser")
             .as_block()
-            .expect("internal error: missing block");
+            .ok_or_else(|| ShellError::TypeMismatch {
+                err_message: "expected block".into(),
+                span: then_expr.span,
+            })?;
         let else_case = call.positional_nth(2);
 
         if eval_constant(working_set, cond)?.as_bool()? {

crates/nu-cmd-lang/src/core_commands/mod.rs

@@ -69,5 +69,5 @@ pub use return_::Return;
 pub use scope::*;
 pub use try_::Try;
 pub use use_::Use;
-pub use version::Version;
+pub use version::{VERSION_NU_FEATURES, Version};
 pub use while_::While;

crates/nu-cmd-lang/src/core_commands/overlay/hide.rs

@@ -86,12 +86,17 @@ impl Command for OverlayHide {
             vec![]
         };
 
+        // also restore env vars which has been hidden
+        let env_vars_to_restore = stack.get_hidden_env_vars(&overlay_name.item, engine_state);
         stack.remove_overlay(&overlay_name.item);
+        for (name, val) in env_vars_to_restore {
+            stack.add_env_var(name, val);
+        }
 
         for (name, val) in env_vars_to_keep {
             stack.add_env_var(name, val);
         }
-
+        stack.update_config(engine_state)?;
         Ok(PipelineData::empty())
     }
 

crates/nu-cmd-lang/src/core_commands/overlay/use_.rs

@@ -158,7 +158,7 @@ impl Command for OverlayUse {
                 }
 
                 let eval_block = get_eval_block(engine_state);
-                let _ = eval_block(engine_state, &mut callee_stack, block, input);
+                let _ = eval_block(engine_state, &mut callee_stack, block, input)?;
 
                 // The export-env block should see the env vars *before* activating this overlay
                 caller_stack.add_overlay(overlay_name);
@@ -178,6 +178,7 @@ impl Command for OverlayUse {
             }
         } else {
             caller_stack.add_overlay(overlay_name);
+            caller_stack.update_config(engine_state)?;
         }
 
         Ok(PipelineData::empty())

crates/nu-cmd-lang/src/core_commands/version.rs

@@ -1,11 +1,48 @@
-use std::sync::OnceLock;
+use std::{borrow::Cow, sync::OnceLock};
 
+use itertools::Itertools;
 use nu_engine::command_prelude::*;
 use nu_protocol::engine::StateWorkingSet;
 use shadow_rs::shadow;
 
 shadow!(build);
 
+/// Static container for the cargo features used by the `version` command.
+///
+/// This `OnceLock` holds the features from `nu`.
+/// When you build `nu_cmd_lang`, Cargo doesn't pass along the same features that `nu` itself uses.
+/// By setting this static before calling `version`, you make it show `nu`'s features instead
+/// of `nu_cmd_lang`'s.
+///
+/// Embedders can set this to any feature list they need, but in most cases you'll probably want to
+/// pass the cargo features of your host binary.
+///
+/// # How to get cargo features in your build script
+///
+/// In your binary's build script:
+/// ```rust,ignore
+/// // Re-export CARGO_CFG_FEATURE to the main binary.
+/// // It holds all the features that cargo sets for your binary as a comma-separated list.
+/// println!(
+///     "cargo:rustc-env=NU_FEATURES={}",
+///     std::env::var("CARGO_CFG_FEATURE").expect("set by cargo")
+/// );
+/// ```
+///
+/// Then, before you call `version`:
+/// ```rust,ignore
+/// // This uses static strings, but since we're using `Cow`, you can also pass owned strings.
+/// let features = env!("NU_FEATURES")
+///     .split(',')
+///     .map(Cow::Borrowed)
+///     .collect();
+///
+/// nu_cmd_lang::VERSION_NU_FEATURES
+///     .set(features)
+///     .expect("couldn't set VERSION_NU_FEATURES");
+/// ```
+pub static VERSION_NU_FEATURES: OnceLock<Vec<Cow<'static, str>>> = OnceLock::new();
+
 #[derive(Clone)]
 pub struct Version;
 
@@ -113,7 +150,17 @@ pub fn version(engine_state: &EngineState, span: Span) -> Result<PipelineData, S
 
     record.push(
         "features",
-        Value::string(features_enabled().join(", "), span),
+        Value::string(
+            VERSION_NU_FEATURES
+                .get()
+                .as_ref()
+                .map(|v| v.as_slice())
+                .unwrap_or_default()
+                .iter()
+                .filter(|f| !f.starts_with("dep:"))
+                .join(", "),
+            span,
+        ),
     );
 
     #[cfg(not(feature = "plugin"))]
@@ -141,6 +188,17 @@ pub fn version(engine_state: &EngineState, span: Span) -> Result<PipelineData, S
         );
     }
 
+    record.push(
+        "experimental_options",
+        Value::string(
+            nu_experimental::ALL
+                .iter()
+                .map(|option| format!("{}={}", option.identifier(), option.get()))
+                .join(", "),
+            span,
+        ),
+    );
+
     Ok(Value::record(record, span).into_pipeline_data())
 }
 
@@ -164,42 +222,12 @@ fn global_allocator() -> &'static str {
     "standard"
 }
 
-fn features_enabled() -> Vec<String> {
-    let mut names = vec!["default".to_string()];
-
-    // NOTE: There should be another way to know features on.
-
-    #[cfg(feature = "trash-support")]
-    {
-        names.push("trash".to_string());
-    }
-
-    #[cfg(feature = "sqlite")]
-    {
-        names.push("sqlite".to_string());
-    }
-
-    #[cfg(feature = "static-link-openssl")]
-    {
-        names.push("static-link-openssl".to_string());
-    }
-
-    #[cfg(feature = "system-clipboard")]
-    {
-        names.push("system-clipboard".to_string());
-    }
-
-    names.sort();
-
-    names
-}
-
 #[cfg(test)]
 mod test {
     #[test]
     fn test_examples() {
         use super::Version;
         use crate::test_examples;
-        test_examples(Version {})
+        test_examples(Version)
     }
 }

crates/nu-cmd-lang/src/example_support.rs

@@ -221,23 +221,23 @@ impl std::fmt::Debug for DebuggableValue<'_> {
     fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
         match self.0 {
             Value::Bool { val, .. } => {
-                write!(f, "{:?}", val)
+                write!(f, "{val:?}")
             }
             Value::Int { val, .. } => {
-                write!(f, "{:?}", val)
+                write!(f, "{val:?}")
             }
             Value::Float { val, .. } => {
-                write!(f, "{:?}f", val)
+                write!(f, "{val:?}f")
             }
             Value::Filesize { val, .. } => {
-                write!(f, "Filesize({:?})", val)
+                write!(f, "Filesize({val:?})")
             }
             Value::Duration { val, .. } => {
                 let duration = std::time::Duration::from_nanos(*val as u64);
-                write!(f, "Duration({:?})", duration)
+                write!(f, "Duration({duration:?})")
             }
             Value::Date { val, .. } => {
-                write!(f, "Date({:?})", val)
+                write!(f, "Date({val:?})")
             }
             Value::Range { val, .. } => match **val {
                 Range::IntRange(range) => match range.end() {
@@ -280,7 +280,7 @@ impl std::fmt::Debug for DebuggableValue<'_> {
                 },
             },
             Value::String { val, .. } | Value::Glob { val, .. } => {
-                write!(f, "{:?}", val)
+                write!(f, "{val:?}")
             }
             Value::Record { val, .. } => {
                 write!(f, "{{")?;
@@ -305,22 +305,22 @@ impl std::fmt::Debug for DebuggableValue<'_> {
                 write!(f, "]")
             }
             Value::Closure { val, .. } => {
-                write!(f, "Closure({:?})", val)
+                write!(f, "Closure({val:?})")
             }
             Value::Nothing { .. } => {
                 write!(f, "Nothing")
             }
             Value::Error { error, .. } => {
-                write!(f, "Error({:?})", error)
+                write!(f, "Error({error:?})")
             }
             Value::Binary { val, .. } => {
-                write!(f, "Binary({:?})", val)
+                write!(f, "Binary({val:?})")
             }
             Value::CellPath { val, .. } => {
                 write!(f, "CellPath({:?})", val.to_string())
             }
             Value::Custom { val, .. } => {
-                write!(f, "CustomValue({:?})", val)
+                write!(f, "CustomValue({val:?})")
             }
         }
     }

crates/nu-cmd-plugin/Cargo.toml

@@ -5,7 +5,7 @@ edition = "2024"
 license = "MIT"
 name = "nu-cmd-plugin"
 repository = "https://github.com/nushell/nushell/tree/main/crates/nu-cmd-plugin"
-version = "0.105.1"
+version = "0.106.1"
 
 # See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
 
@@ -13,10 +13,10 @@ version = "0.105.1"
 workspace = true
 
 [dependencies]
-nu-engine = { path = "../nu-engine", version = "0.105.1" }
+nu-engine = { path = "../nu-engine", version = "0.106.1" }
-nu-path = { path = "../nu-path", version = "0.105.1" }
+nu-path = { path = "../nu-path", version = "0.106.1" }
-nu-protocol = { path = "../nu-protocol", version = "0.105.1", features = ["plugin"] }
+nu-protocol = { path = "../nu-protocol", version = "0.106.1", features = ["plugin"] }
-nu-plugin-engine = { path = "../nu-plugin-engine", version = "0.105.1" }
+nu-plugin-engine = { path = "../nu-plugin-engine", version = "0.106.1" }
 
 itertools = { workspace = true }
 

crates/nu-color-config/Cargo.toml

@@ -5,7 +5,7 @@ repository = "https://github.com/nushell/nushell/tree/main/crates/nu-color-confi
 edition = "2024"
 license = "MIT"
 name = "nu-color-config"
-version = "0.105.1"
+version = "0.106.1"
 
 [lib]
 bench = false
@@ -14,12 +14,12 @@ bench = false
 workspace = true
 
 [dependencies]
-nu-protocol = { path = "../nu-protocol", version = "0.105.1", default-features = false }
+nu-protocol = { path = "../nu-protocol", version = "0.106.1", default-features = false }
-nu-engine = { path = "../nu-engine", version = "0.105.1", default-features = false }
+nu-engine = { path = "../nu-engine", version = "0.106.1", default-features = false }
-nu-json = { path = "../nu-json", version = "0.105.1" }
+nu-json = { path = "../nu-json", version = "0.106.1" }
 nu-ansi-term = { workspace = true }
 
 serde = { workspace = true, features = ["derive"] }
 
 [dev-dependencies]
-nu-test-support = { path = "../nu-test-support", version = "0.105.1" }
+nu-test-support = { path = "../nu-test-support", version = "0.106.1" }

crates/nu-command/Cargo.toml

@@ -5,7 +5,7 @@ edition = "2024"
 license = "MIT"
 name = "nu-command"
 repository = "https://github.com/nushell/nushell/tree/main/crates/nu-command"
-version = "0.105.1"
+version = "0.106.1"
 
 # See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html
 
@@ -16,21 +16,22 @@ bench = false
 workspace = true
 
 [dependencies]
-nu-cmd-base = { path = "../nu-cmd-base", version = "0.105.1" }
-nu-color-config = { path = "../nu-color-config", version = "0.105.1" }
-nu-engine = { path = "../nu-engine", version = "0.105.1", default-features = false }
-nu-glob = { path = "../nu-glob", version = "0.105.1" }
-nu-json = { path = "../nu-json", version = "0.105.1" }
-nu-parser = { path = "../nu-parser", version = "0.105.1" }
-nu-path = { path = "../nu-path", version = "0.105.1" }
-nu-pretty-hex = { path = "../nu-pretty-hex", version = "0.105.1" }
-nu-protocol = { path = "../nu-protocol", version = "0.105.1", default-features = false }
-nu-system = { path = "../nu-system", version = "0.105.1" }
-nu-table = { path = "../nu-table", version = "0.105.1" }
-nu-term-grid = { path = "../nu-term-grid", version = "0.105.1" }
-nu-utils = { path = "../nu-utils", version = "0.105.1", default-features = false }
 nu-ansi-term = { workspace = true }
-nuon = { path = "../nuon", version = "0.105.1" }
+nu-cmd-base = { path = "../nu-cmd-base", version = "0.106.1" }
+nu-color-config = { path = "../nu-color-config", version = "0.106.1" }
+nu-engine = { path = "../nu-engine", version = "0.106.1", default-features = false }
+nu-experimental = { path = "../nu-experimental", version = "0.106.1" }
+nu-glob = { path = "../nu-glob", version = "0.106.1" }
+nu-json = { path = "../nu-json", version = "0.106.1" }
+nu-parser = { path = "../nu-parser", version = "0.106.1" }
+nu-path = { path = "../nu-path", version = "0.106.1" }
+nu-pretty-hex = { path = "../nu-pretty-hex", version = "0.106.1" }
+nu-protocol = { path = "../nu-protocol", version = "0.106.1", default-features = false }
+nu-system = { path = "../nu-system", version = "0.106.1" }
+nu-table = { path = "../nu-table", version = "0.106.1" }
+nu-term-grid = { path = "../nu-term-grid", version = "0.106.1" }
+nu-utils = { path = "../nu-utils", version = "0.106.1", default-features = false }
+nuon = { path = "../nuon", version = "0.106.1" }
 
 alphanumeric-sort = { workspace = true }
 base64 = { workspace = true }
@@ -226,8 +227,8 @@ sqlite = ["rusqlite"]
 trash-support = ["trash"]
 
 [dev-dependencies]
-nu-cmd-lang = { path = "../nu-cmd-lang", version = "0.105.1" }
+nu-cmd-lang = { path = "../nu-cmd-lang", version = "0.106.1" }
-nu-test-support = { path = "../nu-test-support", version = "0.105.1" }
+nu-test-support = { path = "../nu-test-support", version = "0.106.1" }
 
 dirs = { workspace = true }
 mockito = { workspace = true, default-features = false }

crates/nu-command/src/charting/histogram.rs

@@ -89,7 +89,7 @@ impl Command for Histogram {
                             "frequency-column-name can't be {}",
                             forbidden_column_names
                                 .iter()
-                                .map(|val| format!("'{}'", val))
+                                .map(|val| format!("'{val}'"))
                                 .collect::<Vec<_>>()
                                 .join(", ")
                         ),

crates/nu-command/src/conversions/into/binary.rs

@@ -142,7 +142,7 @@ fn into_binary(
     }
 }
 
-fn action(input: &Value, _args: &Arguments, span: Span) -> Value {
+fn action(input: &Value, args: &Arguments, span: Span) -> Value {
     let value = match input {
         Value::Binary { .. } => input.clone(),
         Value::Int { val, .. } => Value::binary(val.to_ne_bytes().to_vec(), span),
@@ -168,7 +168,7 @@ fn action(input: &Value, _args: &Arguments, span: Span) -> Value {
         ),
     };
 
-    if _args.compact {
+    if args.compact {
         let val_span = value.span();
         if let Value::Binary { val, .. } = value {
             let val = if cfg!(target_endian = "little") {

crates/nu-command/src/conversions/into/datetime.rs

@@ -678,7 +678,7 @@ fn parse_value_from_record_as_u32(
         Value::Int { val, .. } => {
             if *val < 0 || *val > u32::MAX as i64 {
                 return Err(ShellError::IncorrectValue {
-                    msg: format!("incorrect value for {}", col),
+                    msg: format!("incorrect value for {col}"),
                     val_span: *head,
                     call_span: *span,
                 });

crates/nu-command/src/conversions/into/duration.rs

@@ -368,8 +368,7 @@ fn merge_record(record: &Record, head: Span, span: Span) -> Result<Value, ShellE
                 if !ALLOWED_SIGNS.contains(&val.as_str()) {
                     let allowed_signs = ALLOWED_SIGNS.join(", ");
                     return Err(ShellError::IncorrectValue {
-                        msg: format!("Invalid sign. Allowed signs are {}", allowed_signs)
+                        msg: format!("Invalid sign. Allowed signs are {allowed_signs}").to_string(),
-                            .to_string(),
                         val_span: sign.span(),
                         call_span: head,
                     });

crates/nu-command/src/database/commands/into_sqlite.rs

@@ -122,8 +122,8 @@ impl Table {
             .conn
             .query_row(&table_exists_query, [], |row| row.get(0))
             .map_err(|err| ShellError::GenericError {
-                error: format!("{:#?}", err),
+                error: format!("{err:#?}"),
-                msg: format!("{:#?}", err),
+                msg: format!("{err:#?}"),
                 span: None,
                 help: None,
                 inner: Vec::new(),
@@ -241,7 +241,7 @@ fn insert_in_transaction(
     let tx = table.try_init(&first_val)?;
 
     for stream_value in stream {
-        if let Err(err) = signals.check(span) {
+        if let Err(err) = signals.check(&span) {
             tx.rollback().map_err(|e| ShellError::GenericError {
                 error: "Failed to rollback SQLite transaction".into(),
                 msg: e.to_string(),
@@ -257,7 +257,7 @@ fn insert_in_transaction(
         let insert_statement = format!(
             "INSERT INTO [{}] ({}) VALUES ({})",
             table_name,
-            Itertools::intersperse(val.columns().map(|c| format!("`{}`", c)), ", ".to_string())
+            Itertools::intersperse(val.columns().map(|c| format!("`{c}`")), ", ".to_string())
                 .collect::<String>(),
             Itertools::intersperse(itertools::repeat_n("?", val.len()), ", ").collect::<String>(),
         );
@@ -353,6 +353,7 @@ fn nu_value_to_sqlite_type(val: &Value) -> Result<&'static str, ShellError> {
 
         // intentionally enumerated so that any future types get handled
         Type::Any
+        | Type::Block
         | Type::CellPath
         | Type::Closure
         | Type::Custom(_)
@@ -381,7 +382,7 @@ fn get_columns_with_sqlite_types(
             .map(|name| (format!("`{}`", name.0), name.1))
             .any(|(name, _)| name == *c)
         {
-            columns.push((format!("`{}`", c), nu_value_to_sqlite_type(v)?));
+            columns.push((format!("`{c}`"), nu_value_to_sqlite_type(v)?));
         }
     }
 

crates/nu-command/src/database/values/sqlite.rs

@@ -112,16 +112,31 @@ impl SQLiteDatabase {
         if self.path == PathBuf::from(MEMORY_DB) {
             open_connection_in_memory_custom()
         } else {
-            Connection::open(&self.path).map_err(|e| ShellError::GenericError {
+            let conn = Connection::open(&self.path).map_err(|e| ShellError::GenericError {
                 error: "Failed to open SQLite database from open_connection".into(),
                 msg: e.to_string(),
                 span: None,
                 help: None,
                 inner: vec![],
-            })
+            })?;
+            conn.busy_handler(Some(SQLiteDatabase::sleeper))
+                .map_err(|e| ShellError::GenericError {
+                    error: "Failed to set busy handler for SQLite database".into(),
+                    msg: e.to_string(),
+                    span: None,
+                    help: None,
+                    inner: vec![],
+                })?;
+            Ok(conn)
         }
     }
 
+    fn sleeper(attempts: i32) -> bool {
+        log::warn!("SQLITE_BUSY, retrying after 250ms (attempt {})", attempts);
+        std::thread::sleep(std::time::Duration::from_millis(250));
+        true
+    }
+
     pub fn get_tables(&self, conn: &Connection) -> Result<Vec<DbTable>, SqliteError> {
         let mut table_names =
             conn.prepare("SELECT name FROM sqlite_master WHERE type = 'table'")?;
@@ -158,7 +173,7 @@ impl SQLiteDatabase {
         filename: String,
     ) -> Result<(), SqliteError> {
         //vacuum main into 'c:\\temp\\foo.db'
-        conn.execute(&format!("vacuum main into '{}'", filename), [])?;
+        conn.execute(&format!("vacuum main into '{filename}'"), [])?;
 
         Ok(())
     }
@@ -573,7 +588,7 @@ fn prepared_statement_to_nu_list(
             let mut row_values = vec![];
 
             for row_result in row_results {
-                signals.check(call_span)?;
+                signals.check(&call_span)?;
                 if let Ok(row_value) = row_result {
                     row_values.push(row_value);
                 }
@@ -599,7 +614,7 @@ fn prepared_statement_to_nu_list(
             let mut row_values = vec![];
 
             for row_result in row_results {
-                signals.check(call_span)?;
+                signals.check(&call_span)?;
                 if let Ok(row_value) = row_result {
                     row_values.push(row_value);
                 }
@@ -668,13 +683,23 @@ pub fn convert_sqlite_value_to_nu_value(value: ValueRef, span: Span) -> Value {
 
 pub fn open_connection_in_memory_custom() -> Result<Connection, ShellError> {
     let flags = OpenFlags::default();
-    Connection::open_with_flags(MEMORY_DB, flags).map_err(|e| ShellError::GenericError {
+    let conn =
-        error: "Failed to open SQLite custom connection in memory".into(),
+        Connection::open_with_flags(MEMORY_DB, flags).map_err(|e| ShellError::GenericError {
-        msg: e.to_string(),
+            error: "Failed to open SQLite custom connection in memory".into(),
-        span: Some(Span::test_data()),
+            msg: e.to_string(),
-        help: None,
+            span: Some(Span::test_data()),
-        inner: vec![],
+            help: None,
-    })
+            inner: vec![],
+        })?;
+    conn.busy_handler(Some(SQLiteDatabase::sleeper))
+        .map_err(|e| ShellError::GenericError {
+            error: "Failed to set busy handler for SQLite custom connection in memory".into(),
+            msg: e.to_string(),
+            span: Some(Span::test_data()),
+            help: None,
+            inner: vec![],
+        })?;
+    Ok(conn)
 }
 
 pub fn open_connection_in_memory() -> Result<Connection, ShellError> {

crates/nu-command/src/debug/experimental_options.rs

@@ -0,0 +1,64 @@
+use nu_engine::command_prelude::*;
+use nu_experimental::Status;
+
+#[derive(Clone)]
+pub struct DebugExperimentalOptions;
+
+impl Command for DebugExperimentalOptions {
+    fn name(&self) -> &str {
+        "debug experimental-options"
+    }
+
+    fn signature(&self) -> Signature {
+        Signature::new(self.name())
+            .input_output_type(
+                Type::Nothing,
+                Type::Table(Box::from([
+                    (String::from("identifier"), Type::String),
+                    (String::from("enabled"), Type::Bool),
+                    (String::from("status"), Type::String),
+                    (String::from("description"), Type::String),
+                ])),
+            )
+            .add_help()
+            .category(Category::Debug)
+    }
+
+    fn description(&self) -> &str {
+        "Show all experimental options."
+    }
+
+    fn run(
+        &self,
+        _engine_state: &EngineState,
+        _stack: &mut Stack,
+        call: &Call,
+        _input: PipelineData,
+    ) -> Result<PipelineData, ShellError> {
+        Ok(PipelineData::Value(
+            Value::list(
+                nu_experimental::ALL
+                    .iter()
+                    .map(|option| {
+                        Value::record(
+                            nu_protocol::record! {
+                                "identifier" => Value::string(option.identifier(), call.head),
+                                "enabled" => Value::bool(option.get(), call.head),
+                                "status" => Value::string(match option.status() {
+                                    Status::OptIn => "opt-in",
+                                    Status::OptOut => "opt-out",
+                                    Status::DeprecatedDiscard => "deprecated-discard",
+                                    Status::DeprecatedDefault => "deprecated-default"
+                                }, call.head),
+                                "description" => Value::string(option.description(), call.head),
+                            },
+                            call.head,
+                        )
+                    })
+                    .collect(),
+                call.head,
+            ),
+            None,
+        ))
+    }
+}

crates/nu-command/src/debug/metadata.rs

@@ -43,6 +43,17 @@ impl Command for Metadata {
         let arg = call.positional_nth(stack, 0);
         let head = call.head;
 
+        if !matches!(input, PipelineData::Empty) {
+            if let Some(arg_expr) = arg {
+                return Err(ShellError::IncompatibleParameters {
+                    left_message: "pipeline input was provided".into(),
+                    left_span: head,
+                    right_message: "but a positional metadata expression was also given".into(),
+                    right_span: arg_expr.span,
+                });
+            }
+        }
+
         match arg {
             Some(Expression {
                 expr: Expr::FullCellPath(full_cell_path),
@@ -56,7 +67,6 @@ impl Command for Metadata {
                             ..
                         } => {
                             let origin = stack.get_var_with_origin(*var_id, *span)?;
-
                             Ok(build_metadata_record_value(
                                 &origin,
                                 input.metadata().as_ref(),
@@ -87,10 +97,9 @@ impl Command for Metadata {
                         .into_pipeline_data(),
                 )
             }
-            None => Ok(
+            None => {
-                Value::record(build_metadata_record(input.metadata().as_ref(), head), head)
+                Ok(Value::record(build_metadata_record(&input, head), head).into_pipeline_data())
-                    .into_pipeline_data(),
+            }
-            ),
         }
     }
 
@@ -116,19 +125,7 @@ fn build_metadata_record_value(
     head: Span,
 ) -> Value {
     let mut record = Record::new();
-
+    record.push("span", arg.span().into_value(head));
-    let span = arg.span();
-    record.push(
-        "span",
-        Value::record(
-            record! {
-                "start" => Value::int(span.start as i64,span),
-                "end" => Value::int(span.end as i64, span),
-            },
-            head,
-        ),
-    );
-
     Value::record(extend_record_with_metadata(record, metadata, head), head)
 }
 

crates/nu-command/src/debug/metadata_access.rs

@@ -42,10 +42,7 @@ impl Command for MetadataAccess {
         // `ClosureEvalOnce` is not used as it uses `Stack::captures_to_stack` rather than
         // `Stack::captures_to_stack_preserve_out_dest`. This command shouldn't collect streams
         let mut callee_stack = caller_stack.captures_to_stack_preserve_out_dest(closure.captures);
-        let metadata_record = Value::record(
+        let metadata_record = Value::record(build_metadata_record(&input, call.head), call.head);
-            build_metadata_record(input.metadata().as_ref(), call.head),
-            call.head,
-        );
 
         if let Some(var_id) = block.signature.get_positional(0).and_then(|var| var.var_id) {
             callee_stack.add_var(var_id, metadata_record)
@@ -58,12 +55,10 @@ impl Command for MetadataAccess {
     fn examples(&self) -> Vec<Example> {
         vec![Example {
             description: "Access metadata and data from a stream together",
-            example: r#"{foo: bar} | to json --raw | metadata access {|meta| {in: $in, meta: $meta}}"#,
+            example: r#"{foo: bar} | to json --raw | metadata access {|meta| {in: $in, content: $meta.content_type}}"#,
             result: Some(Value::test_record(record! {
                 "in" => Value::test_string(r#"{"foo":"bar"}"#),
-                "meta" => Value::test_record(record! {
+                "content" => Value::test_string(r#"application/json"#)
-                    "content_type" => Value::test_string(r#"application/json"#)
-                })
             })),
         }]
     }

crates/nu-command/src/debug/metadata_set.rs

@@ -63,7 +63,18 @@ impl Command for MetadataSet {
         match (ds_fp, ds_ls) {
             (Some(path), false) => metadata.data_source = DataSource::FilePath(path.into()),
             (None, true) => metadata.data_source = DataSource::Ls,
-            (Some(_), true) => (), // TODO: error here
+            (Some(_), true) => {
+                return Err(ShellError::IncompatibleParameters {
+                    left_message: "cannot use `--datasource-filepath`".into(),
+                    left_span: call
+                        .get_flag_span(stack, "datasource-filepath")
+                        .expect("has flag"),
+                    right_message: "with `--datasource-ls`".into(),
+                    right_span: call
+                        .get_flag_span(stack, "datasource-ls")
+                        .expect("has flag"),
+                });
+            }
             (None, false) => (),
         }
 
@@ -79,15 +90,13 @@ impl Command for MetadataSet {
             },
             Example {
                 description: "Set the metadata of a file path",
-                example: "'crates' | metadata set --datasource-filepath $'(pwd)/crates' | metadata",
+                example: "'crates' | metadata set --datasource-filepath $'(pwd)/crates'",
                 result: None,
             },
             Example {
                 description: "Set the metadata of a file path",
-                example: "'crates' | metadata set --content-type text/plain | metadata",
+                example: "'crates' | metadata set --content-type text/plain | metadata | get content_type",
-                result: Some(Value::test_record(record! {
+                result: Some(Value::test_string("text/plain")),
-                    "content_type" => Value::test_string("text/plain"),
-                })),
             },
         ]
     }

crates/nu-command/src/debug/mod.rs

@@ -1,6 +1,7 @@
 mod ast;
 mod debug_;
 mod env;
+mod experimental_options;
 mod explain;
 mod info;
 mod inspect;
@@ -21,6 +22,7 @@ mod view_span;
 pub use ast::Ast;
 pub use debug_::Debug;
 pub use env::DebugEnv;
+pub use experimental_options::DebugExperimentalOptions;
 pub use explain::Explain;
 pub use info::DebugInfo;
 pub use inspect::Inspect;

crates/nu-command/src/debug/util.rs

@@ -1,4 +1,4 @@
-use nu_protocol::{DataSource, PipelineMetadata, Record, Span, Value};
+use nu_protocol::{DataSource, IntoValue, PipelineData, PipelineMetadata, Record, Span, Value};
 
 pub fn extend_record_with_metadata(
     mut record: Record,
@@ -29,6 +29,10 @@ pub fn extend_record_with_metadata(
     record
 }
 
-pub fn build_metadata_record(metadata: Option<&PipelineMetadata>, head: Span) -> Record {
+pub fn build_metadata_record(pipeline: &PipelineData, head: Span) -> Record {
-    extend_record_with_metadata(Record::new(), metadata, head)
+    let mut record = Record::new();
+    if let Some(span) = pipeline.span() {
+        record.insert("span", span.into_value(head));
+    }
+    extend_record_with_metadata(record, pipeline.metadata().as_ref(), head)
 }

crates/nu-command/src/debug/view_source.rs

@@ -126,10 +126,10 @@ impl Command for ViewSource {
                                 }
                                 let _ = write!(&mut final_contents, "--{}", n.long);
                                 if let Some(short) = n.short {
-                                    let _ = write!(&mut final_contents, "(-{})", short);
+                                    let _ = write!(&mut final_contents, "(-{short})");
                                 }
                                 if let Some(arg) = &n.arg {
-                                    let _ = write!(&mut final_contents, ": {}", arg);
+                                    let _ = write!(&mut final_contents, ": {arg}");
                                 }
                                 final_contents.push(' ');
                             }
@@ -146,7 +146,7 @@ impl Command for ViewSource {
                                 let mut c = 0;
                                 for (insig, outsig) in type_signatures {
                                     c += 1;
-                                    let s = format!("{} -> {}", insig, outsig);
+                                    let s = format!("{insig} -> {outsig}");
                                     final_contents.push_str(&s);
                                     if c != len {
                                         final_contents.push_str(", ")

crates/nu-command/src/default_context.rs

@@ -153,6 +153,7 @@ pub fn add_shell_command_context(mut engine_state: EngineState) -> EngineState {
             Ast,
             Debug,
             DebugEnv,
+            DebugExperimentalOptions,
             DebugInfo,
             DebugProfile,
             Explain,
@@ -188,6 +189,9 @@ pub fn add_shell_command_context(mut engine_state: EngineState) -> EngineState {
 
         // Strings
         bind_command! {
+            Ansi,
+            AnsiLink,
+            AnsiStrip,
             Char,
             Decode,
             Encode,
@@ -250,9 +254,6 @@ pub fn add_shell_command_context(mut engine_state: EngineState) -> EngineState {
         // Platform
         #[cfg(feature = "os")]
         bind_command! {
-            Ansi,
-            AnsiLink,
-            AnsiStrip,
             Clear,
             Du,
             Input,

crates/nu-command/src/filesystem/glob.rs

@@ -85,7 +85,7 @@ impl Command for Glob {
                 result: None,
             },
             Example {
-                description: "Search for files for folders that do not begin with c, C, b, M, or s",
+                description: "Search for files or folders that do not begin with c, C, b, M, or s",
                 example: r#"glob "[!cCbMs]*""#,
                 result: None,
             },
@@ -329,7 +329,7 @@ fn glob_to_value(
 ) -> ListStream {
     let map_signals = signals.clone();
     let result = glob_results.filter_map(move |entry| {
-        if let Err(err) = map_signals.check(span) {
+        if let Err(err) = map_signals.check(&span) {
             return Some(Value::error(err, span));
         };
         let file_type = entry.file_type();

crates/nu-command/src/filesystem/ls.rs

@@ -341,7 +341,7 @@ fn ls_for_one_pattern(
 
     let mut paths_peek = paths.peekable();
     let no_matches = paths_peek.peek().is_none();
-    signals.check(call_span)?;
+    signals.check(&call_span)?;
     if no_matches {
         return Err(ShellError::GenericError {
             error: format!("No matches found for {:?}", path.item),
@@ -979,14 +979,14 @@ fn read_dir(
         .read_dir()
         .map_err(|err| IoError::new(err, span, f.clone()))?
         .map(move |d| {
-            signals_clone.check(span)?;
+            signals_clone.check(&span)?;
             d.map(|r| r.path())
                 .map_err(|err| IoError::new(err, span, f.clone()))
                 .map_err(ShellError::from)
         });
     if !use_threads {
         let mut collected = items.collect::<Vec<_>>();
-        signals.check(span)?;
+        signals.check(&span)?;
         collected.sort_by(|a, b| match (a, b) {
             (Ok(a), Ok(b)) => a.cmp(b),
             (Ok(_), Err(_)) => Ordering::Greater,

crates/nu-command/src/filesystem/mktemp.rs

@@ -112,8 +112,8 @@ impl Command for Mktemp {
                 .map_err(|_| ShellError::NonUtf8 { span })?,
             Err(e) => {
                 return Err(ShellError::GenericError {
-                    error: format!("{}", e),
+                    error: format!("{e}"),
-                    msg: format!("{}", e),
+                    msg: format!("{e}"),
                     span: None,
                     help: None,
                     inner: vec![],

crates/nu-command/src/filesystem/open.rs

@@ -198,7 +198,7 @@ impl Command for Open {
                     let converter = exts_opt.and_then(|exts| {
                         exts.iter().find_map(|ext| {
                             engine_state
-                                .find_decl(format!("from {}", ext).as_bytes(), &[])
+                                .find_decl(format!("from {ext}").as_bytes(), &[])
                                 .map(|id| (id, ext.to_string()))
                         })
                     });
@@ -314,7 +314,7 @@ fn extract_extensions(filename: &str) -> Vec<String> {
         if current_extension.is_empty() {
             current_extension.push_str(part);
         } else {
-            current_extension = format!("{}.{}", part, current_extension);
+            current_extension = format!("{part}.{current_extension}");
         }
         extensions.push(current_extension.clone());
     }

crates/nu-command/src/filesystem/rm.rs

@@ -454,7 +454,7 @@ fn rm(
     });
 
     for result in iter {
-        engine_state.signals().check(call.head)?;
+        engine_state.signals().check(&call.head)?;
         match result {
             Ok(None) => {}
             Ok(Some(msg)) => eprintln!("{msg}"),

crates/nu-command/src/filesystem/save.rs

@@ -91,7 +91,8 @@ impl Command for Save {
             PipelineData::ByteStream(stream, metadata) => {
                 check_saving_to_source_file(metadata.as_ref(), &path, stderr_path.as_ref())?;
 
-                let (file, stderr_file) = get_files(&path, stderr_path.as_ref(), append, force)?;
+                let (file, stderr_file) =
+                    get_files(engine_state, &path, stderr_path.as_ref(), append, force)?;
 
                 let size = stream.known_size();
                 let signals = engine_state.signals();
@@ -201,7 +202,8 @@ impl Command for Save {
                     stderr_path.as_ref(),
                 )?;
 
-                let (mut file, _) = get_files(&path, stderr_path.as_ref(), append, force)?;
+                let (mut file, _) =
+                    get_files(engine_state, &path, stderr_path.as_ref(), append, force)?;
                 for val in ls {
                     file.write_all(&value_to_bytes(val)?)
                         .map_err(&from_io_error)?;
@@ -226,7 +228,8 @@ impl Command for Save {
                     input_to_bytes(input, Path::new(&path.item), raw, engine_state, stack, span)?;
 
                 // Only open file after successful conversion
-                let (mut file, _) = get_files(&path, stderr_path.as_ref(), append, force)?;
+                let (mut file, _) =
+                    get_files(engine_state, &path, stderr_path.as_ref(), append, force)?;
 
                 file.write_all(&bytes).map_err(&from_io_error)?;
                 file.flush().map_err(&from_io_error)?;
@@ -422,13 +425,14 @@ fn prepare_path(
     }
 }
 
-fn open_file(path: &Path, span: Span, append: bool) -> Result<File, ShellError> {
+fn open_file(
-    let file: Result<File, nu_protocol::shell_error::io::ErrorKind> = match (append, path.exists())
+    engine_state: &EngineState,
-    {
+    path: &Path,
-        (true, true) => std::fs::OpenOptions::new()
+    span: Span,
-            .append(true)
+    append: bool,
-            .open(path)
+) -> Result<File, ShellError> {
-            .map_err(|err| err.into()),
+    let file: std::io::Result<File> = match (append, path.exists()) {
+        (true, true) => std::fs::OpenOptions::new().append(true).open(path),
         _ => {
             // This is a temporary solution until `std::fs::File::create` is fixed on Windows (rust-lang/rust#134893)
             // A TOCTOU problem exists here, which may cause wrong error message to be shown
@@ -438,22 +442,51 @@ fn open_file(path: &Path, span: Span, append: bool) -> Result<File, ShellError>
                     deprecated,
                     reason = "we don't get a IsADirectory error, so we need to provide it"
                 )]
-                Err(nu_protocol::shell_error::io::ErrorKind::from_std(
+                Err(std::io::ErrorKind::IsADirectory.into())
-                    std::io::ErrorKind::IsADirectory,
-                ))
             } else {
-                std::fs::File::create(path).map_err(|err| err.into())
+                std::fs::File::create(path)
             }
             #[cfg(not(target_os = "windows"))]
-            std::fs::File::create(path).map_err(|err| err.into())
+            std::fs::File::create(path)
         }
     };
 
-    file.map_err(|err_kind| ShellError::Io(IoError::new(err_kind, span, PathBuf::from(path))))
+    match file {
+        Ok(file) => Ok(file),
+        Err(err) => {
+            // In caase of NotFound, search for the missing parent directory.
+            // This also presents a TOCTOU (or TOUTOC, technically?)
+            if err.kind() == std::io::ErrorKind::NotFound {
+                if let Some(missing_component) =
+                    path.ancestors().skip(1).filter(|dir| !dir.exists()).last()
+                {
+                    // By looking at the postfix to remove, rather than the prefix
+                    // to keep, we are able to handle relative paths too.
+                    let components_to_remove = path
+                        .strip_prefix(missing_component)
+                        .expect("Stripping ancestor from a path should never fail")
+                        .as_os_str()
+                        .as_encoded_bytes();
+
+                    return Err(ShellError::Io(IoError::new(
+                        ErrorKind::DirectoryNotFound,
+                        engine_state
+                            .span_match_postfix(span, components_to_remove)
+                            .map(|(pre, _post)| pre)
+                            .unwrap_or(span),
+                        PathBuf::from(missing_component),
+                    )));
+                }
+            }
+
+            Err(ShellError::Io(IoError::new(err, span, PathBuf::from(path))))
+        }
+    }
 }
 
 /// Get output file and optional stderr file
 fn get_files(
+    engine_state: &EngineState,
     path: &Spanned<PathBuf>,
     stderr_path: Option<&Spanned<PathBuf>>,
     append: bool,
@@ -467,7 +500,7 @@ fn get_files(
         .transpose()?;
 
     // Only if both files can be used open and possibly truncate them
-    let file = open_file(path, path_span, append)?;
+    let file = open_file(engine_state, path, path_span, append)?;
 
     let stderr_file = stderr_path_and_span
         .map(|(stderr_path, stderr_path_span)| {
@@ -480,7 +513,7 @@ fn get_files(
                     inner: vec![],
                 })
             } else {
-                open_file(stderr_path, stderr_path_span, append)
+                open_file(engine_state, stderr_path, stderr_path_span, append)
             }
         })
         .transpose()?;
@@ -510,7 +543,7 @@ fn stream_to_file(
         let mut reader = BufReader::new(source);
 
         let res = loop {
-            if let Err(err) = signals.check(span) {
+            if let Err(err) = signals.check(&span) {
                 bar.abandoned_msg("# Cancelled #".to_owned());
                 return Err(err);
             }

crates/nu-command/src/filesystem/start.rs

@@ -49,7 +49,8 @@ impl Command for Start {
         }
         // If it's not a URL, treat it as a file path
         let cwd = engine_state.cwd(Some(stack))?;
-        let full_path = cwd.join(path_no_whitespace);
+        let full_path = nu_path::expand_path_with(path_no_whitespace, &cwd, true);
+
         // Check if the path exists or if it's a valid file/directory
         if full_path.exists() {
             open_path(full_path, engine_state, stack, path.span)?;

crates/nu-command/src/filesystem/ucp.rs

@@ -272,8 +272,8 @@ impl Command for UCp {
                 uu_cp::Error::NotAllFilesCopied => {}
                 _ => {
                     return Err(ShellError::GenericError {
-                        error: format!("{}", error),
+                        error: format!("{error}"),
-                        msg: format!("{}", error),
+                        msg: format!("{error}"),
                         span: None,
                         help: None,
                         inner: vec![],
@@ -373,7 +373,7 @@ fn parse_and_set_attribute(
                 "xattr" => &mut attribute.xattr,
                 _ => {
                     return Err(ShellError::IncompatibleParametersSingle {
-                        msg: format!("--preserve flag got an unexpected attribute \"{}\"", val),
+                        msg: format!("--preserve flag got an unexpected attribute \"{val}\""),
                         span: value.span(),
                     });
                 }

crates/nu-command/src/filesystem/umkdir.rs

@@ -77,8 +77,8 @@ impl Command for UMkdir {
         for dir in directories {
             if let Err(error) = mkdir(&dir, IS_RECURSIVE, get_mode(), is_verbose) {
                 return Err(ShellError::GenericError {
-                    error: format!("{}", error),
+                    error: format!("{error}"),
-                    msg: format!("{}", error),
+                    msg: format!("{error}"),
                     span: None,
                     help: None,
                     inner: vec![],

crates/nu-command/src/filesystem/umv.rs

@@ -195,8 +195,8 @@ impl Command for UMv {
         };
         if let Err(error) = uu_mv::mv(&files, &options) {
             return Err(ShellError::GenericError {
-                error: format!("{}", error),
+                error: format!("{error}"),
-                msg: format!("{}", error),
+                msg: format!("{error}"),
                 span: None,
                 help: None,
                 inner: Vec::new(),

crates/nu-command/src/filesystem/utouch.rs

@@ -220,7 +220,7 @@ impl Command for UTouch {
                     inner: Vec::new(),
                 },
                 TouchError::InvalidDateFormat(date) => ShellError::IncorrectValue {
-                    msg: format!("Invalid date: {}", date),
+                    msg: format!("Invalid date: {date}"),
                     val_span: date_span.expect("touch should've been given a date"),
                     call_span: call.head,
                 },

crates/nu-command/src/filesystem/watch.rs

@@ -6,11 +6,7 @@ use notify_debouncer_full::{
     },
 };
 use nu_engine::{ClosureEval, command_prelude::*};
-use nu_protocol::{
+use nu_protocol::{engine::Closure, report_shell_error, shell_error::io::IoError};
-    engine::{Closure, StateWorkingSet},
-    format_shell_error,
-    shell_error::io::IoError,
-};
 use std::{
     path::PathBuf,
     sync::mpsc::{RecvTimeoutError, channel},
@@ -203,14 +199,9 @@ impl Command for Watch {
                     .run_with_input(PipelineData::Empty);
 
                 match result {
-                    Ok(val) => {
+                    Ok(val) => val.print_table(engine_state, stack, false, false)?,
-                        val.print_table(engine_state, stack, false, false)?;
+                    Err(err) => report_shell_error(engine_state, &err),
-                    }
+                };
-                    Err(err) => {
-                        let working_set = StateWorkingSet::new(engine_state);
-                        eprintln!("{}", format_shell_error(&working_set, &err));
-                    }
-                }
             }
 
             Ok(())

crates/nu-command/src/filters/default.rs

@@ -2,7 +2,7 @@ use std::{borrow::Cow, ops::Deref};
 
 use nu_engine::{ClosureEval, command_prelude::*};
 use nu_protocol::{
-    ListStream, Signals,
+    ListStream, ReportMode, ShellWarning, Signals,
     ast::{Expr, Expression},
     report_shell_warning,
 };
@@ -82,7 +82,7 @@ impl Command for Default {
             },
             Example {
                 description: "Get the env value of `MY_ENV` with a default value 'abc' if not present",
-                example: "$env | get --ignore-errors MY_ENV | default 'abc'",
+                example: "$env | get --optional MY_ENV | default 'abc'",
                 result: Some(Value::test_string("abc")),
             },
             Example {
@@ -213,7 +213,7 @@ fn default(
         || (default_when_empty
             && matches!(input, PipelineData::Value(ref value, _) if value.is_empty()))
     {
-        default_value.pipeline_data()
+        default_value.single_run_pipeline_data()
     } else if default_when_empty && matches!(input, PipelineData::ListStream(..)) {
         let PipelineData::ListStream(ls, metadata) = input else {
             unreachable!()
@@ -221,7 +221,7 @@ fn default(
         let span = ls.span();
         let mut stream = ls.into_inner().peekable();
         if stream.peek().is_none() {
-            return default_value.pipeline_data();
+            return default_value.single_run_pipeline_data();
         }
 
         // stream's internal state already preserves the original signals config, so if this
@@ -278,8 +278,14 @@ impl DefaultValue {
         }
     }
 
-    fn pipeline_data(&mut self) -> Result<PipelineData, ShellError> {
+    /// Used when we know the value won't need to be cached to allow streaming.
-        self.value().map(|x| x.into_pipeline_data())
+    fn single_run_pipeline_data(self) -> Result<PipelineData, ShellError> {
+        match self {
+            DefaultValue::Uncalculated(mut closure) => {
+                closure.item.run_with_input(PipelineData::Empty)
+            }
+            DefaultValue::Calculated(val) => Ok(val.into_pipeline_data()),
+        }
     }
 }
 
@@ -323,24 +329,25 @@ fn closure_variable_warning(
         (Value::Closure { .. }, true) => {
             let span_contents = String::from_utf8_lossy(engine_state.get_span_contents(span));
             let carapace_suggestion = "re-run carapace init with version v1.3.3 or later\nor, change this to `{ $carapace_completer }`";
-            let suggestion = match span_contents {
+            let label = match span_contents {
                 Cow::Borrowed("$carapace_completer") => carapace_suggestion.to_string(),
                 Cow::Owned(s) if s.deref() == "$carapace_completer" => {
                     carapace_suggestion.to_string()
                 }
-                _ => format!("change this to {{ {} }}", span_contents).to_string(),
+                _ => format!("change this to {{ {span_contents} }}").to_string(),
             };
 
             report_shell_warning(
                 engine_state,
-                &ShellError::DeprecationWarning {
+                &ShellWarning::Deprecated {
-                    deprecation_type: "Behavior",
+                    dep_type: "Behavior".to_string(),
-                    suggestion,
+                    label,
                     span,
                     help: Some(
                         r"Since 0.105.0, closure literals passed to default are lazily evaluated, rather than returned as a value.
-In a future release, closures passed by variable will also be lazily evaluated.",
+In a future release, closures passed by variable will also be lazily evaluated.".to_string(),
                     ),
+                    report_mode: ReportMode::FirstUse,
                 },
             );
 

crates/nu-command/src/filters/drop/nth.rs

@@ -1,6 +1,6 @@
-use itertools::Either;
 use nu_engine::command_prelude::*;
 use nu_protocol::{PipelineIterator, Range};
+use std::collections::VecDeque;
 use std::ops::Bound;
 
 #[derive(Clone)]
@@ -18,13 +18,11 @@ impl Command for DropNth {
                 (Type::list(Type::Any), Type::list(Type::Any)),
             ])
             .allow_variants_without_examples(true)
-            .required(
+            .rest(
-                "row number or row range",
+                "rest",
-                // FIXME: we can make this accept either Int or Range when we can compose SyntaxShapes
                 SyntaxShape::Any,
-                "The number of the row to drop or a range to drop consecutive rows.",
+                "The row numbers or ranges to drop.",
             )
-            .rest("rest", SyntaxShape::Any, "The number of the row to drop.")
             .category(Category::Filters)
     }
 
@@ -103,110 +101,125 @@ impl Command for DropNth {
     ) -> Result<PipelineData, ShellError> {
         let head = call.head;
         let metadata = input.metadata();
-        let number_or_range = extract_int_or_range(engine_state, stack, call)?;
 
-        let rows = match number_or_range.item {
+        let args: Vec<Value> = call.rest(engine_state, stack, 0)?;
-            Either::Left(row_number) => {
+        if args.is_empty() {
-                let and_rows: Vec<Spanned<i64>> = call.rest(engine_state, stack, 1)?;
+            return Ok(input);
-                let mut rows: Vec<_> = and_rows.into_iter().map(|x| x.item as usize).collect();
+        }
-                rows.push(row_number as usize);
-                rows.sort_unstable();
-                rows
-            }
-            Either::Right(Range::FloatRange(_)) => {
-                return Err(ShellError::UnsupportedInput {
-                    msg: "float range".into(),
-                    input: "value originates from here".into(),
-                    msg_span: head,
-                    input_span: number_or_range.span,
-                });
-            }
-            Either::Right(Range::IntRange(range)) => {
-                // check for negative range inputs, e.g., (2..-5)
-                let end_negative = match range.end() {
-                    Bound::Included(end) | Bound::Excluded(end) => end < 0,
-                    Bound::Unbounded => false,
-                };
-                if range.start().is_negative() || end_negative {
-                    return Err(ShellError::UnsupportedInput {
-                        msg: "drop nth accepts only positive ints".into(),
-                        input: "value originates from here".into(),
-                        msg_span: head,
-                        input_span: number_or_range.span,
-                    });
-                }
-                // check if the upper bound is smaller than the lower bound, e.g., do not accept 4..2
-                if range.step() < 0 {
-                    return Err(ShellError::UnsupportedInput {
-                        msg: "The upper bound needs to be equal or larger to the lower bound"
-                            .into(),
-                        input: "value originates from here".into(),
-                        msg_span: head,
-                        input_span: number_or_range.span,
-                    });
-                }
 
-                let start = range.start() as usize;
+        let (rows_to_drop, min_unbounded_start) = get_rows_to_drop(&args, head)?;
 
-                let end = match range.end() {
+        let input = if let Some(cutoff) = min_unbounded_start {
-                    Bound::Included(end) => end as usize,
+            input
-                    Bound::Excluded(end) => (end - 1) as usize,
+                .into_iter()
-                    Bound::Unbounded => {
+                .take(cutoff)
-                        return Ok(input
+                .into_pipeline_data_with_metadata(
-                            .into_iter()
+                    head,
-                            .take(start)
+                    engine_state.signals().clone(),
-                            .into_pipeline_data_with_metadata(
+                    metadata.clone(),
-                                head,
+                )
-                                engine_state.signals().clone(),
+        } else {
-                                metadata,
+            input
-                            ));
-                    }
-                };
-
-                let end = if let PipelineData::Value(Value::List { vals, .. }, _) = &input {
-                    end.min(vals.len() - 1)
-                } else {
-                    end
-                };
-
-                (start..=end).collect()
-            }
         };
 
         Ok(DropNthIterator {
             input: input.into_iter(),
-            rows,
+            rows: rows_to_drop,
             current: 0,
         }
         .into_pipeline_data_with_metadata(head, engine_state.signals().clone(), metadata))
     }
 }
 
-fn extract_int_or_range(
+fn get_rows_to_drop(
-    engine_state: &EngineState,
+    args: &[Value],
-    stack: &mut Stack,
+    head: Span,
-    call: &Call,
+) -> Result<(VecDeque<usize>, Option<usize>), ShellError> {
-) -> Result<Spanned<Either<i64, Range>>, ShellError> {
+    let mut rows_to_drop = Vec::new();
-    let value: Value = call.req(engine_state, stack, 0)?;
+    let mut min_unbounded_start: Option<usize> = None;
 
-    let int_opt = value.as_int().map(Either::Left).ok();
+    for value in args {
-    let range_opt = value.as_range().map(Either::Right).ok();
+        if let Ok(i) = value.as_int() {
+            if i < 0 {
+                return Err(ShellError::UnsupportedInput {
+                    msg: "drop nth accepts only positive ints".into(),
+                    input: "value originates from here".into(),
+                    msg_span: head,
+                    input_span: value.span(),
+                });
+            }
+            rows_to_drop.push(i as usize);
+        } else if let Ok(range) = value.as_range() {
+            match range {
+                Range::IntRange(range) => {
+                    let start = range.start();
+                    if start < 0 {
+                        return Err(ShellError::UnsupportedInput {
+                            msg: "drop nth accepts only positive ints".into(),
+                            input: "value originates from here".into(),
+                            msg_span: head,
+                            input_span: value.span(),
+                        });
+                    }
 
-    int_opt
+                    match range.end() {
-        .or(range_opt)
+                        Bound::Included(end) => {
-        .ok_or_else(|| ShellError::TypeMismatch {
+                            if end < start {
-            err_message: "int or range".into(),
+                                return Err(ShellError::UnsupportedInput {
-            span: value.span(),
+                                    msg: "The upper bound must be greater than or equal to the lower bound".into(),
-        })
+                                    input: "value originates from here".into(),
-        .map(|either| Spanned {
+                                    msg_span: head,
-            item: either,
+                                    input_span: value.span(),
-            span: value.span(),
+                                });
-        })
+                            }
+                            rows_to_drop.extend((start as usize)..=(end as usize));
+                        }
+                        Bound::Excluded(end) => {
+                            if end <= start {
+                                return Err(ShellError::UnsupportedInput {
+                                    msg: "The upper bound must be greater than the lower bound"
+                                        .into(),
+                                    input: "value originates from here".into(),
+                                    msg_span: head,
+                                    input_span: value.span(),
+                                });
+                            }
+                            rows_to_drop.extend((start as usize)..(end as usize));
+                        }
+                        Bound::Unbounded => {
+                            let start_usize = start as usize;
+                            min_unbounded_start = Some(
+                                min_unbounded_start.map_or(start_usize, |s| s.min(start_usize)),
+                            );
+                        }
+                    }
+                }
+                Range::FloatRange(_) => {
+                    return Err(ShellError::UnsupportedInput {
+                        msg: "float range not supported".into(),
+                        input: "value originates from here".into(),
+                        msg_span: head,
+                        input_span: value.span(),
+                    });
+                }
+            }
+        } else {
+            return Err(ShellError::TypeMismatch {
+                err_message: "Expected int or range".into(),
+                span: value.span(),
+            });
+        }
+    }
+
+    rows_to_drop.sort_unstable();
+    rows_to_drop.dedup();
+
+    Ok((VecDeque::from(rows_to_drop), min_unbounded_start))
 }
 
 struct DropNthIterator {
     input: PipelineIterator,
-    rows: Vec<usize>,
+    rows: VecDeque<usize>,
     current: usize,
 }
 
@@ -215,9 +228,9 @@ impl Iterator for DropNthIterator {
 
     fn next(&mut self) -> Option<Self::Item> {
         loop {
-            if let Some(row) = self.rows.first() {
+            if let Some(row) = self.rows.front() {
                 if self.current == *row {
-                    self.rows.remove(0);
+                    self.rows.pop_front();
                     self.current += 1;
                     let _ = self.input.next();
                     continue;

crates/nu-command/src/filters/find.rs

@@ -47,7 +47,7 @@ impl Command for Find {
             .named(
                 "columns",
                 SyntaxShape::List(Box::new(SyntaxShape::String)),
-                "column names to be searched (with rest parameter, not regex yet)",
+                "column names to be searched",
                 Some('c'),
             )
             .switch(
@@ -163,7 +163,12 @@ impl Command for Find {
                 example: r#"[["Larry", "Moe"], ["Victor", "Marina"]] | find --regex "rr""#,
                 result: Some(Value::list(
                     vec![Value::list(
-                        vec![Value::test_string("Larry"), Value::test_string("Moe")],
+                        vec![
+                            Value::test_string(
+                                "\u{1b}[37mLa\u{1b}[0m\u{1b}[41;37mrr\u{1b}[0m\u{1b}[37my\u{1b}[0m",
+                            ),
+                            Value::test_string("Moe"),
+                        ],
                         Span::test_data(),
                     )],
                     Span::test_data(),
@@ -344,7 +349,10 @@ fn get_match_pattern_from_arguments(
 // map functions
 
 fn highlight_matches_in_string(pattern: &MatchPattern, val: String) -> String {
-    // strip haystack to remove existing ansi style
+    if !pattern.regex.is_match(&val).unwrap_or(false) {
+        return val;
+    }
+
     let stripped_val = nu_utils::strip_ansi_string_unlikely(val);
     let mut last_match_end = 0;
     let mut highlighted = String::new();
@@ -390,7 +398,7 @@ fn highlight_matches_in_string(pattern: &MatchPattern, val: String) -> String {
     highlighted
 }
 
-fn highlight_matches_in_record_or_value(
+fn highlight_matches_in_value(
     pattern: &MatchPattern,
     value: Value,
     columns_to_search: &[String],
@@ -412,16 +420,16 @@ fn highlight_matches_in_record_or_value(
                     continue;
                 }
 
-                if let Value::String { val: val_str, .. } = val {
+                *val = highlight_matches_in_value(pattern, std::mem::take(val), &[]);
-                    if pattern.regex.is_match(val_str).unwrap_or(false) {
-                        let val_str = std::mem::take(val_str);
-                        *val = highlight_matches_in_string(pattern, val_str).into_value(span)
-                    }
-                }
             }
 
             Value::record(record, span)
         }
+        Value::List { vals, .. } => vals
+            .into_iter()
+            .map(|item| highlight_matches_in_value(pattern, item, &[]))
+            .collect::<Vec<Value>>()
+            .into_value(span),
         Value::String { val, .. } => highlight_matches_in_string(pattern, val).into_value(span),
         _ => value,
     }
@@ -444,24 +452,22 @@ fn find_in_pipelinedata(
         PipelineData::Value(_, _) => input
             .filter(
                 move |value| {
-                    record_or_value_should_be_printed(&pattern, value, &columns_to_search, &config)
+                    value_should_be_printed(&pattern, value, &columns_to_search, &config)
+                        != pattern.invert
                 },
                 engine_state.signals(),
             )?
             .map(
-                move |x| {
+                move |x| highlight_matches_in_value(&map_pattern, x, &map_columns_to_search),
-                    highlight_matches_in_record_or_value(&map_pattern, x, &map_columns_to_search)
-                },
                 engine_state.signals(),
             ),
         PipelineData::ListStream(stream, metadata) => {
             let stream = stream.modify(|iter| {
                 iter.filter(move |value| {
-                    record_or_value_should_be_printed(&pattern, value, &columns_to_search, &config)
+                    value_should_be_printed(&pattern, value, &columns_to_search, &config)
-                })
+                        != pattern.invert
-                .map(move |x| {
-                    highlight_matches_in_record_or_value(&map_pattern, x, &map_columns_to_search)
                 })
+                .map(move |x| highlight_matches_in_value(&map_pattern, x, &map_columns_to_search))
             });
 
             Ok(PipelineData::ListStream(stream, metadata))
@@ -495,7 +501,12 @@ fn string_should_be_printed(pattern: &MatchPattern, value: &str) -> bool {
     pattern.regex.is_match(value).unwrap_or(false)
 }
 
-fn value_should_be_printed(pattern: &MatchPattern, value: &Value, config: &Config) -> bool {
+fn value_should_be_printed(
+    pattern: &MatchPattern,
+    value: &Value,
+    columns_to_search: &[String],
+    config: &Config,
+) -> bool {
     let lower_value = value.to_expanded_string("", config).to_lowercase();
 
     match value {
@@ -507,8 +518,7 @@ fn value_should_be_printed(pattern: &MatchPattern, value: &Value, config: &Confi
         | Value::Range { .. }
         | Value::Float { .. }
         | Value::Closure { .. }
-        | Value::Nothing { .. }
+        | Value::Nothing { .. } => {
-        | Value::Error { .. } => {
             if !pattern.lower_terms.is_empty() {
                 // look for exact match when searching with terms
                 pattern
@@ -519,37 +529,25 @@ fn value_should_be_printed(pattern: &MatchPattern, value: &Value, config: &Confi
                 string_should_be_printed(pattern, &lower_value)
             }
         }
-        Value::Glob { .. }
+        Value::Glob { .. } | Value::CellPath { .. } | Value::Custom { .. } => {
-        | Value::List { .. }
+            string_should_be_printed(pattern, &lower_value)
-        | Value::CellPath { .. }
+        }
-        | Value::Record { .. }
-        | Value::Custom { .. } => string_should_be_printed(pattern, &lower_value),
         Value::String { val, .. } => string_should_be_printed(pattern, val),
-        Value::Binary { .. } => false,
+        Value::List { vals, .. } => vals
-    }
+            .iter()
-}
+            .any(|item| value_should_be_printed(pattern, item, &[], config)),
-
-fn record_or_value_should_be_printed(
-    pattern: &MatchPattern,
-    value: &Value,
-    columns_to_search: &[String],
-    config: &Config,
-) -> bool {
-    let match_found = match value {
         Value::Record { val: record, .. } => {
-            // Only perform column selection if given columns.
             let col_select = !columns_to_search.is_empty();
             record.iter().any(|(col, val)| {
                 if col_select && !columns_to_search.contains(col) {
                     return false;
                 }
-                value_should_be_printed(pattern, val, config)
+                value_should_be_printed(pattern, val, &[], config)
             })
         }
-        _ => value_should_be_printed(pattern, value, config),
+        Value::Binary { .. } => false,
-    };
+        Value::Error { .. } => true,
-
+    }
-    match_found != pattern.invert
 }
 
 // utility
@@ -574,6 +572,46 @@ fn split_string_if_multiline(input: PipelineData, head_span: Span) -> PipelineDa
     }
 }
 
+/// function for using find from other commands
+pub fn find_internal(
+    input: PipelineData,
+    engine_state: &EngineState,
+    stack: &mut Stack,
+    search_term: &str,
+    columns_to_search: &[&str],
+    highlight: bool,
+) -> Result<PipelineData, ShellError> {
+    let span = input.span().unwrap_or(Span::unknown());
+
+    let style_computer = StyleComputer::from_config(engine_state, stack);
+    let string_style = style_computer.compute("string", &Value::string("search result", span));
+    let highlight_style =
+        style_computer.compute("search_result", &Value::string("search result", span));
+
+    let regex_str = format!("(?i){}", escape(search_term));
+
+    let regex = Regex::new(regex_str.as_str()).map_err(|e| ShellError::TypeMismatch {
+        err_message: format!("invalid regex: {e}"),
+        span: Span::unknown(),
+    })?;
+
+    let pattern = MatchPattern {
+        regex,
+        lower_terms: vec![search_term.to_lowercase()],
+        highlight,
+        invert: false,
+        string_style,
+        highlight_style,
+    };
+
+    let columns_to_search = columns_to_search
+        .iter()
+        .map(|str| String::from(*str))
+        .collect();
+
+    find_in_pipelinedata(pattern, columns_to_search, engine_state, stack, input)
+}
+
 #[cfg(test)]
 mod tests {
     use super::*;

crates/nu-command/src/filters/get.rs

@@ -40,9 +40,14 @@ If multiple cell paths are given, this will produce a list of values."#
                 "The cell path to the data.",
             )
             .rest("rest", SyntaxShape::CellPath, "Additional cell paths.")
+            .switch(
+                "optional",
+                "make all cell path members optional (returns `null` for missing values)",
+                Some('o'),
+            )
             .switch(
                 "ignore-errors",
-                "ignore missing data (make all cell path members optional)",
+                "ignore missing data (make all cell path members optional) (deprecated)",
                 Some('i'),
             )
             .switch(
@@ -109,13 +114,14 @@ If multiple cell paths are given, this will produce a list of values."#
     ) -> Result<PipelineData, ShellError> {
         let cell_path: CellPath = call.req_const(working_set, 0)?;
         let rest: Vec<CellPath> = call.rest_const(working_set, 1)?;
-        let ignore_errors = call.has_flag_const(working_set, "ignore-errors")?;
+        let optional = call.has_flag_const(working_set, "optional")?
+            || call.has_flag_const(working_set, "ignore-errors")?;
         let metadata = input.metadata();
         action(
             input,
             cell_path,
             rest,
-            ignore_errors,
+            optional,
             working_set.permanent().signals().clone(),
             call.head,
         )
@@ -131,13 +137,14 @@ If multiple cell paths are given, this will produce a list of values."#
     ) -> Result<PipelineData, ShellError> {
         let cell_path: CellPath = call.req(engine_state, stack, 0)?;
         let rest: Vec<CellPath> = call.rest(engine_state, stack, 1)?;
-        let ignore_errors = call.has_flag(engine_state, stack, "ignore-errors")?;
+        let optional = call.has_flag(engine_state, stack, "optional")?
+            || call.has_flag(engine_state, stack, "ignore-errors")?;
         let metadata = input.metadata();
         action(
             input,
             cell_path,
             rest,
-            ignore_errors,
+            optional,
             engine_state.signals().clone(),
             call.head,
         )
@@ -152,6 +159,13 @@ If multiple cell paths are given, this will produce a list of values."#
                 since: Some("0.105.0".into()),
                 expected_removal: None,
                 help: Some("Cell-paths are now case-sensitive by default.\nTo access fields case-insensitively, add `!` after the relevant path member.".into())
+            },
+            DeprecationEntry {
+                ty: DeprecationType::Flag("ignore-errors".into()),
+                report_mode: ReportMode::FirstUse,
+                since: Some("0.106.0".into()),
+                expected_removal: None,
+                help: Some("This flag has been renamed to `--optional (-o)` to better reflect its behavior.".into())
             }
         ]
     }
@@ -161,29 +175,19 @@ fn action(
     input: PipelineData,
     mut cell_path: CellPath,
     mut rest: Vec<CellPath>,
-    ignore_errors: bool,
+    optional: bool,
     signals: Signals,
     span: Span,
 ) -> Result<PipelineData, ShellError> {
-    if ignore_errors {
+    if optional {
         cell_path.make_optional();
         for path in &mut rest {
             path.make_optional();
         }
     }
 
-    match input {
+    if let PipelineData::Empty = input {
-        PipelineData::Empty => return Err(ShellError::PipelineEmpty { dst_span: span }),
+        return Err(ShellError::PipelineEmpty { dst_span: span });
-        // Allow chaining of get -i
-        PipelineData::Value(val @ Value::Nothing { .. }, ..) if !ignore_errors => {
-            return Err(ShellError::OnlySupportsThisInputType {
-                exp_input_type: "table or record".into(),
-                wrong_type: "nothing".into(),
-                dst_span: span,
-                src_span: val.span(),
-            });
-        }
-        _ => (),
     }
 
     if rest.is_empty() {

crates/nu-command/src/filters/group_by.rs

@@ -231,7 +231,12 @@ pub fn group_by(
 
     let values: Vec<Value> = input.into_iter().collect();
     if values.is_empty() {
-        return Ok(Value::record(Record::new(), head).into_pipeline_data());
+        let val = if to_table {
+            Value::list(Vec::new(), head)
+        } else {
+            Value::record(Record::new(), head)
+        };
+        return Ok(val.into_pipeline_data());
     }
 
     let grouped = match &groupers[..] {

crates/nu-command/src/filters/join.rs

@@ -379,10 +379,7 @@ fn merge_records(left: &Record, right: &Record, shared_key: Option<&str>) -> Rec
         let k_shared = shared_key == Some(k.as_str());
         // Do not output shared join key twice
         if !(k_seen && k_shared) {
-            record.push(
+            record.push(if k_seen { format!("{k}_") } else { k.clone() }, v.clone());
-                if k_seen { format!("{}_", k) } else { k.clone() },
-                v.clone(),
-            );
         }
     }
     record

crates/nu-command/src/filters/last.rs

@@ -100,7 +100,7 @@ impl Command for Last {
                 let mut buf = VecDeque::new();
 
                 for row in iterator {
-                    engine_state.signals().check(head)?;
+                    engine_state.signals().check(&head)?;
                     if buf.len() == rows {
                         buf.pop_front();
                     }

crates/nu-command/src/filters/mod.rs

@@ -70,7 +70,7 @@ pub use empty::empty;
 pub use enumerate::Enumerate;
 pub use every::Every;
 pub use filter::Filter;
-pub use find::Find;
+pub use find::{Find, find_internal};
 pub use first::First;
 pub use flatten::Flatten;
 pub use get::Get;

crates/nu-command/src/filters/reduce.rs

@@ -119,7 +119,7 @@ impl Command for Reduce {
         let mut closure = ClosureEval::new(engine_state, stack, closure);
 
         for value in iter {
-            engine_state.signals().check(head)?;
+            engine_state.signals().check(&head)?;
             acc = closure
                 .add_arg(value)
                 .add_arg(acc.clone())

crates/nu-command/src/filters/reject.rs

@@ -1,5 +1,5 @@
 use nu_engine::command_prelude::*;
-use nu_protocol::{ast::PathMember, casing::Casing};
+use nu_protocol::{DeprecationEntry, DeprecationType, ReportMode, ast::PathMember, casing::Casing};
 use std::{cmp::Reverse, collections::HashSet};
 
 #[derive(Clone)]
@@ -17,9 +17,10 @@ impl Command for Reject {
                 (Type::table(), Type::table()),
                 (Type::list(Type::Any), Type::list(Type::Any)),
             ])
+            .switch("optional", "make all cell path members optional", Some('o'))
             .switch(
                 "ignore-errors",
-                "ignore missing data (make all cell path members optional)",
+                "ignore missing data (make all cell path members optional) (deprecated)",
                 Some('i'),
             )
             .rest(
@@ -90,8 +91,9 @@ impl Command for Reject {
         }
         let span = call.head;
 
-        let ignore_errors = call.has_flag(engine_state, stack, "ignore-errors")?;
+        let optional = call.has_flag(engine_state, stack, "optional")?
-        if ignore_errors {
+            || call.has_flag(engine_state, stack, "ignore-errors")?;
+        if optional {
             for cell_path in &mut new_columns {
                 cell_path.make_optional();
             }
@@ -100,6 +102,19 @@ impl Command for Reject {
         reject(engine_state, span, input, new_columns)
     }
 
+    fn deprecation_info(&self) -> Vec<DeprecationEntry> {
+        vec![DeprecationEntry {
+            ty: DeprecationType::Flag("ignore-errors".into()),
+            report_mode: ReportMode::FirstUse,
+            since: Some("0.106.0".into()),
+            expected_removal: None,
+            help: Some(
+                "This flag has been renamed to `--optional (-o)` to better reflect its behavior."
+                    .into(),
+            ),
+        }]
+    }
+
     fn examples(&self) -> Vec<Example> {
         vec![
             Example {

crates/nu-command/src/filters/select.rs

@@ -1,5 +1,8 @@
 use nu_engine::command_prelude::*;
-use nu_protocol::{PipelineIterator, ast::PathMember, casing::Casing};
+use nu_protocol::{
+    DeprecationEntry, DeprecationType, PipelineIterator, ReportMode, ast::PathMember,
+    casing::Casing,
+};
 use std::collections::BTreeSet;
 
 #[derive(Clone)]
@@ -18,9 +21,14 @@ impl Command for Select {
                 (Type::table(), Type::table()),
                 (Type::List(Box::new(Type::Any)), Type::Any),
             ])
+            .switch(
+                "optional",
+                "make all cell path members optional (returns `null` for missing values)",
+                Some('o'),
+            )
             .switch(
                 "ignore-errors",
-                "ignore missing data (make all cell path members optional)",
+                "ignore missing data (make all cell path members optional) (deprecated)",
                 Some('i'),
             )
             .rest(
@@ -100,10 +108,11 @@ produce a table, a list will produce a list, and a record will produce a record.
                 }
             }
         }
-        let ignore_errors = call.has_flag(engine_state, stack, "ignore-errors")?;
+        let optional = call.has_flag(engine_state, stack, "optional")?
+            || call.has_flag(engine_state, stack, "ignore-errors")?;
         let span = call.head;
 
-        if ignore_errors {
+        if optional {
             for cell_path in &mut new_columns {
                 cell_path.make_optional();
             }
@@ -112,6 +121,19 @@ produce a table, a list will produce a list, and a record will produce a record.
         select(engine_state, span, new_columns, input)
     }
 
+    fn deprecation_info(&self) -> Vec<DeprecationEntry> {
+        vec![DeprecationEntry {
+            ty: DeprecationType::Flag("ignore-errors".into()),
+            report_mode: ReportMode::FirstUse,
+            since: Some("0.106.0".into()),
+            expected_removal: None,
+            help: Some(
+                "This flag has been renamed to `--optional (-o)` to better reflect its behavior."
+                    .into(),
+            ),
+        }]
+    }
+
     fn examples(&self) -> Vec<Example> {
         vec![
             Example {

crates/nu-command/src/filters/utils.rs

@@ -31,7 +31,7 @@ pub fn boolean_fold(
     let mut closure = ClosureEval::new(engine_state, stack, closure);
 
     for value in input {
-        engine_state.signals().check(head)?;
+        engine_state.signals().check(&head)?;
         let pred = closure.run_with_value(value)?.into_value(head)?.is_true();
 
         if pred == accumulator {

crates/nu-command/src/filters/where_.rs

@@ -43,10 +43,7 @@ Row conditions cannot be stored in a variable. To pass a condition with a variab
             ])
             .required(
                 "condition",
-                SyntaxShape::OneOf(vec![
+                SyntaxShape::RowCondition,
-                    SyntaxShape::RowCondition,
-                    SyntaxShape::Closure(Some(vec![SyntaxShape::Any])),
-                ]),
                 "Filter row condition or closure.",
             )
             .allow_variants_without_examples(true)

crates/nu-command/src/formats/from/csv.rs

@@ -203,6 +203,7 @@ mod test {
 
     use super::*;
 
+    use crate::Reject;
     use crate::{Metadata, MetadataSet};
 
     #[test]
@@ -221,6 +222,7 @@ mod test {
             working_set.add_decl(Box::new(FromCsv {}));
             working_set.add_decl(Box::new(Metadata {}));
             working_set.add_decl(Box::new(MetadataSet {}));
+            working_set.add_decl(Box::new(Reject {}));
 
             working_set.render()
         };
@@ -229,7 +231,7 @@ mod test {
             .merge_delta(delta)
             .expect("Error merging delta");
 
-        let cmd = r#""a,b\n1,2" | metadata set --content-type 'text/csv' --datasource-ls | from csv | metadata | $in"#;
+        let cmd = r#""a,b\n1,2" | metadata set --content-type 'text/csv' --datasource-ls | from csv | metadata | reject span | $in"#;
         let result = eval_pipeline_without_terminal_expression(
             cmd,
             std::env::temp_dir().as_ref(),

crates/nu-command/src/formats/from/json.rs

@@ -248,6 +248,7 @@ fn convert_string_to_value_strict(string_input: &str, span: Span) -> Result<Valu
 mod test {
     use nu_cmd_lang::eval_pipeline_without_terminal_expression;
 
+    use crate::Reject;
     use crate::{Metadata, MetadataSet};
 
     use super::*;
@@ -268,6 +269,7 @@ mod test {
             working_set.add_decl(Box::new(FromJson {}));
             working_set.add_decl(Box::new(Metadata {}));
             working_set.add_decl(Box::new(MetadataSet {}));
+            working_set.add_decl(Box::new(Reject {}));
 
             working_set.render()
         };
@@ -276,7 +278,7 @@ mod test {
             .merge_delta(delta)
             .expect("Error merging delta");
 
-        let cmd = r#"'{"a":1,"b":2}' | metadata set --content-type 'application/json' --datasource-ls | from json | metadata | $in"#;
+        let cmd = r#"'{"a":1,"b":2}' | metadata set --content-type 'application/json' --datasource-ls | from json | metadata | reject span | $in"#;
         let result = eval_pipeline_without_terminal_expression(
             cmd,
             std::env::temp_dir().as_ref(),

crates/nu-command/src/formats/from/msgpack.rs

@@ -212,7 +212,7 @@ impl From<ReadError> for ShellError {
             },
             ReadError::TypeMismatch(marker, span) => ShellError::GenericError {
                 error: "Invalid marker while reading MessagePack data".into(),
-                msg: format!("unexpected {:?} in data", marker),
+                msg: format!("unexpected {marker:?} in data"),
                 span: Some(span),
                 help: None,
                 inner: vec![],
@@ -514,6 +514,7 @@ fn assert_eof(input: &mut impl io::Read, span: Span) -> Result<(), ShellError> {
 mod test {
     use nu_cmd_lang::eval_pipeline_without_terminal_expression;
 
+    use crate::Reject;
     use crate::{Metadata, MetadataSet, ToMsgpack};
 
     use super::*;
@@ -535,6 +536,7 @@ mod test {
             working_set.add_decl(Box::new(FromMsgpack {}));
             working_set.add_decl(Box::new(Metadata {}));
             working_set.add_decl(Box::new(MetadataSet {}));
+            working_set.add_decl(Box::new(Reject {}));
 
             working_set.render()
         };
@@ -543,7 +545,7 @@ mod test {
             .merge_delta(delta)
             .expect("Error merging delta");
 
-        let cmd = r#"{a: 1 b: 2} | to msgpack | metadata set --datasource-ls | from msgpack | metadata | $in"#;
+        let cmd = r#"{a: 1 b: 2} | to msgpack | metadata set --datasource-ls | from msgpack | metadata | reject span | $in"#;
         let result = eval_pipeline_without_terminal_expression(
             cmd,
             std::env::temp_dir().as_ref(),

crates/nu-command/src/formats/from/nuon.rs

@@ -74,6 +74,7 @@ impl Command for FromNuon {
 mod test {
     use nu_cmd_lang::eval_pipeline_without_terminal_expression;
 
+    use crate::Reject;
     use crate::{Metadata, MetadataSet};
 
     use super::*;
@@ -94,6 +95,7 @@ mod test {
             working_set.add_decl(Box::new(FromNuon {}));
             working_set.add_decl(Box::new(Metadata {}));
             working_set.add_decl(Box::new(MetadataSet {}));
+            working_set.add_decl(Box::new(Reject {}));
 
             working_set.render()
         };
@@ -102,7 +104,7 @@ mod test {
             .merge_delta(delta)
             .expect("Error merging delta");
 
-        let cmd = r#"'[[a, b]; [1, 2]]' | metadata set --content-type 'application/x-nuon' --datasource-ls | from nuon | metadata | $in"#;
+        let cmd = r#"'[[a, b]; [1, 2]]' | metadata set --content-type 'application/x-nuon' --datasource-ls | from nuon | metadata | reject span | $in"#;
         let result = eval_pipeline_without_terminal_expression(
             cmd,
             std::env::temp_dir().as_ref(),

crates/nu-command/src/formats/from/ssv.rs

@@ -167,7 +167,7 @@ fn parse_aligned_columns<'a>(
         let headers: Vec<(String, usize)> = indices
             .iter()
             .enumerate()
-            .map(|(i, position)| (format!("column{}", i), *position))
+            .map(|(i, position)| (format!("column{i}"), *position))
             .collect();
 
         construct(ls.iter().map(|s| s.to_owned()), headers)

crates/nu-command/src/formats/from/toml.rs

@@ -145,6 +145,7 @@ pub fn convert_string_to_value(string_input: String, span: Span) -> Result<Value
 
 #[cfg(test)]
 mod tests {
+    use crate::Reject;
     use crate::{Metadata, MetadataSet};
 
     use super::*;
@@ -345,6 +346,7 @@ mod tests {
             working_set.add_decl(Box::new(FromToml {}));
             working_set.add_decl(Box::new(Metadata {}));
             working_set.add_decl(Box::new(MetadataSet {}));
+            working_set.add_decl(Box::new(Reject {}));
 
             working_set.render()
         };
@@ -353,7 +355,7 @@ mod tests {
             .merge_delta(delta)
             .expect("Error merging delta");
 
-        let cmd = r#""[a]\nb = 1\nc = 1" | metadata set --content-type 'text/x-toml' --datasource-ls | from toml | metadata | $in"#;
+        let cmd = r#""[a]\nb = 1\nc = 1" | metadata set --content-type 'text/x-toml' --datasource-ls | from toml | metadata | reject span | $in"#;
         let result = eval_pipeline_without_terminal_expression(
             cmd,
             std::env::temp_dir().as_ref(),

crates/nu-command/src/formats/from/tsv.rs

@@ -160,6 +160,7 @@ fn from_tsv(
 mod test {
     use nu_cmd_lang::eval_pipeline_without_terminal_expression;
 
+    use crate::Reject;
     use crate::{Metadata, MetadataSet};
 
     use super::*;
@@ -180,6 +181,7 @@ mod test {
             working_set.add_decl(Box::new(FromTsv {}));
             working_set.add_decl(Box::new(Metadata {}));
             working_set.add_decl(Box::new(MetadataSet {}));
+            working_set.add_decl(Box::new(Reject {}));
 
             working_set.render()
         };
@@ -188,7 +190,7 @@ mod test {
             .merge_delta(delta)
             .expect("Error merging delta");
 
-        let cmd = r#""a\tb\n1\t2" | metadata set --content-type 'text/tab-separated-values' --datasource-ls | from tsv | metadata | $in"#;
+        let cmd = r#""a\tb\n1\t2" | metadata set --content-type 'text/tab-separated-values' --datasource-ls | from tsv | metadata | reject span | $in"#;
         let result = eval_pipeline_without_terminal_expression(
             cmd,
             std::env::temp_dir().as_ref(),

crates/nu-command/src/formats/from/xml.rs

@@ -252,7 +252,7 @@ fn process_xml_parse_error(source: String, err: roxmltree::Error, span: Span) ->
             pos,
         ),
         roxmltree::Error::UnknownNamespace(prefix, pos) => {
-            make_xml_error_spanned(format!("Unknown prefix {}", prefix), source, pos)
+            make_xml_error_spanned(format!("Unknown prefix {prefix}"), source, pos)
         }
         roxmltree::Error::UnexpectedCloseTag(expected, actual, pos) => make_xml_error_spanned(
             format!("Unexpected close tag {actual}, expected {expected}"),
@@ -370,6 +370,7 @@ fn make_xml_error_spanned(msg: impl Into<String>, src: String, pos: TextPos) ->
 mod tests {
     use crate::Metadata;
     use crate::MetadataSet;
+    use crate::Reject;
 
     use super::*;
 
@@ -541,6 +542,7 @@ mod tests {
             working_set.add_decl(Box::new(FromXml {}));
             working_set.add_decl(Box::new(Metadata {}));
             working_set.add_decl(Box::new(MetadataSet {}));
+            working_set.add_decl(Box::new(Reject {}));
 
             working_set.render()
         };
@@ -552,7 +554,7 @@ mod tests {
         let cmd = r#"'<?xml version="1.0" encoding="UTF-8"?>
 <note>
   <remember>Event</remember>
-</note>' | metadata set --content-type 'application/xml' --datasource-ls | from xml | metadata | $in"#;
+</note>' | metadata set --content-type 'application/xml' --datasource-ls | from xml | metadata | reject span | $in"#;
         let result = eval_pipeline_without_terminal_expression(
             cmd,
             std::env::temp_dir().as_ref(),

crates/nu-command/src/formats/from/yaml.rs

@@ -158,27 +158,26 @@ fn convert_yaml_value_to_nu_value(
         }
         serde_yaml::Value::Tagged(t) => {
             let tag = &t.tag;
-            let value = match &t.value {
+
+            match &t.value {
                 serde_yaml::Value::String(s) => {
-                    let val = format!("{} {}", tag, s).trim().to_string();
+                    let val = format!("{tag} {s}").trim().to_string();
                     Value::string(val, span)
                 }
                 serde_yaml::Value::Number(n) => {
-                    let val = format!("{} {}", tag, n).trim().to_string();
+                    let val = format!("{tag} {n}").trim().to_string();
                     Value::string(val, span)
                 }
                 serde_yaml::Value::Bool(b) => {
-                    let val = format!("{} {}", tag, b).trim().to_string();
+                    let val = format!("{tag} {b}").trim().to_string();
                     Value::string(val, span)
                 }
                 serde_yaml::Value::Null => {
-                    let val = format!("{}", tag).trim().to_string();
+                    let val = format!("{tag}").trim().to_string();
                     Value::string(val, span)
                 }
                 v => convert_yaml_value_to_nu_value(v, span, val_span)?,
-            };
+            }
-
-            value
         }
         serde_yaml::Value::Null => Value::nothing(span),
         x => unimplemented!("Unsupported YAML case: {:?}", x),
@@ -244,6 +243,7 @@ fn from_yaml(input: PipelineData, head: Span) -> Result<PipelineData, ShellError
 
 #[cfg(test)]
 mod test {
+    use crate::Reject;
     use crate::{Metadata, MetadataSet};
 
     use super::*;
@@ -410,6 +410,7 @@ mod test {
             working_set.add_decl(Box::new(FromYaml {}));
             working_set.add_decl(Box::new(Metadata {}));
             working_set.add_decl(Box::new(MetadataSet {}));
+            working_set.add_decl(Box::new(Reject {}));
 
             working_set.render()
         };
@@ -418,7 +419,7 @@ mod test {
             .merge_delta(delta)
             .expect("Error merging delta");
 
-        let cmd = r#""a: 1\nb: 2" | metadata set --content-type 'application/yaml' --datasource-ls | from yaml | metadata | $in"#;
+        let cmd = r#""a: 1\nb: 2" | metadata set --content-type 'application/yaml' --datasource-ls | from yaml | metadata | reject span | $in"#;
         let result = eval_pipeline_without_terminal_expression(
             cmd,
             std::env::temp_dir().as_ref(),

crates/nu-command/src/formats/to/csv.rs

@@ -166,14 +166,14 @@ mod test {
             .merge_delta(delta)
             .expect("Error merging delta");
 
-        let cmd = "{a: 1 b: 2} | to csv  | metadata | get content_type";
+        let cmd = "{a: 1 b: 2} | to csv  | metadata | get content_type | $in";
         let result = eval_pipeline_without_terminal_expression(
             cmd,
             std::env::temp_dir().as_ref(),
             &mut engine_state,
         );
         assert_eq!(
-            Value::test_record(record!("content_type" => Value::test_string("text/csv"))),
+            Value::test_string("text/csv"),
             result.expect("There should be a result")
         );
     }

crates/nu-command/src/formats/to/delimited.rs

@@ -50,7 +50,7 @@ fn make_unsupported_input_error(
 ) -> ShellError {
     ShellError::UnsupportedInput {
         msg: "expected table or record".to_string(),
-        input: format!("input type: {}", r#type),
+        input: format!("input type: {type}"),
         msg_span: head,
         input_span: span,
     }

crates/nu-command/src/formats/to/json.rs

@@ -229,14 +229,14 @@ mod test {
             .merge_delta(delta)
             .expect("Error merging delta");
 
-        let cmd = "{a: 1 b: 2} | to json  | metadata | get content_type";
+        let cmd = "{a: 1 b: 2} | to json  | metadata | get content_type | $in";
         let result = eval_pipeline_without_terminal_expression(
             cmd,
             std::env::temp_dir().as_ref(),
             &mut engine_state,
         );
         assert_eq!(
-            Value::test_record(record!("content_type" => Value::test_string("application/json"))),
+            Value::test_string("application/json"),
             result.expect("There should be a result")
         );
     }