update usage of deprecated str downcase thingy in nushell

[TheBearodactyl]

Summary by CodeRabbit

• Bug Fixes
  • Updated Windows command completion to use the supported lowercase operation for case-insensitive prefix matching.
  • Preserved case-sensitive command matching on non-Windows platforms.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Central YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro Plus

Run ID: 6c71619d-e0f1-40de-a9b6-2e6b2b5ab7e6

📥 Commits

Reviewing files that changed from the base of the PR and between f05c715 and 5ffe52f.

📒 Files selected for processing (2)

• argv/src/script.rs
• go/argv/script.go

Included review availability: Your plan provides up to 4 included reviews per hour; 3 remain after this review.

---

📝 Walkthrough

Walkthrough

Nu shell command completion now uses str lowercase for Windows-insensitive prefix matching in both generated scripts. Non-Windows matching remains case-sensitive.

Changes

Nu command matching

Layer / File(s)	Summary
Windows prefix normalization argv/src/script.rs, go/argv/script.go	Windows command-prefix comparisons now use str lowercase instead of str downcase. Non-Windows matching is unchanged.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 5ffe5

This is a small, localized syntax update in both generated completers. Older Nushell compatibility should be confirmed, but the supplied evidence does not establish a contractual break, so no actionable merge-blocking risk remains beyond normal checks.

Suggested reviewers: jdx

Poem

A rabbit checks the command line bright,
Lowercase keeps Windows matching right.
The other paths stay case-aware,
Two scripts share the fix with care.
Hop, hop—completion works tonight!

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately identifies the Nushell change from deprecated str downcase to the supported command.
Docstring Coverage	✅ Passed	Docstring check was indeterminate for this PR — some files could not be analyzed in time. Not blocking.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[github-actions]

This PR currently has failing checks. If this continues for 7 days, it will be closed automatically.

This is warning day 1 of 7.

Please update the PR when you have a chance. Feel free to reopen or create a new PR if it is closed and you'd like to continue working on it.

This comment was generated by an automated workflow.