Skip to content

feat: add argument forwarding to winutil commands#206

Open
vyas-devgna wants to merge 1 commit into
ChrisTitusTech:mainfrom
vyas-devgna:feat/argument-forwarding
Open

feat: add argument forwarding to winutil commands#206
vyas-devgna wants to merge 1 commit into
ChrisTitusTech:mainfrom
vyas-devgna:feat/argument-forwarding

Conversation

@vyas-devgna
Copy link
Copy Markdown

@vyas-devgna vyas-devgna commented May 27, 2026

Description

Updates the existing winutil and winutildev profile functions to support proper argument forwarding (e.g., winutil -Preset Standard -Noui).

Key Improvements:

  1. Zero Bloat One-Liners: Uses direct, clean, and inline compilation/execution to avoid polluting the global namespace with helper functions.
  2. Argument Splatting: Employs native @args splatting to seamlessly unpack and bind named parameters/switches (like -Config or -Noui) perfectly, fully supported since PowerShell 2.0.

Build & Release Note:

  • Local Verification: Fully verified locally using the latest compiled WinUtil script containing the new automation parameters.
  • Online Builds: Note that since live online builds (christitus.com/win and windev) are cached from commits prior to the automation changes, they currently ignore parameters and launch the GUI. Once a new release is pushed online, this profile update will work seamlessly.

Usage Example:

winutil -Preset Standard -Noui
winutildev -Config "C:\path\to\config.json"

@GabiNun2
Copy link
Copy Markdown
Contributor

original pr was better

@vyas-devgna
Copy link
Copy Markdown
Author

original pr was better

in what sense?

@GabiNun2
Copy link
Copy Markdown
Contributor

GabiNun2 commented May 27, 2026

original pr was better

in what sense?

it was not bloat

@vyas-devgna
Copy link
Copy Markdown
Author

original pr was better

in what sense?

it was not bloat

you mean #205 ?

@GabiNun2
Copy link
Copy Markdown
Contributor

original pr was better

in what sense?

it was not bloat

you mean #205 ?

yes

@vyas-devgna vyas-devgna force-pushed the feat/argument-forwarding branch from 641e472 to c50fa01 Compare May 27, 2026 19:06
@GabiNun2
Copy link
Copy Markdown
Contributor

did you test this tho?

@vyas-devgna
Copy link
Copy Markdown
Author

did you test this tho?

I just tested it locally with a populated config JSON and it parsed the settings and executed the tweaks and winget installs flawlessly

@GabiNun2
Copy link
Copy Markdown
Contributor

GabiNun2 commented May 27, 2026

did you test this tho?

I just tested it locally with a populated config JSON and it parsed the settings and executed the tweaks and winget installs flawlessly

congrats your first ever actually good pr

@vyas-devgna
Copy link
Copy Markdown
Author

{F8461C90-F6B1-4B85-95A9-0350F68694D8} {86322C4A-9B37-4152-870F-EB7D324FFC69}

@GabiNun2
Copy link
Copy Markdown
Contributor

GabiNun2 commented May 27, 2026

{F8461C90-F6B1-4B85-95A9-0350F68694D8} {86322C4A-9B37-4152-870F-EB7D324FFC69}

i know it works i tested it myself, after i asked you if it works

@vyas-devgna
Copy link
Copy Markdown
Author

This is actually my first PR overall, thanks so much for all your guidance and patience in helping me make it perfect!

@GabiNun2
Copy link
Copy Markdown
Contributor

This is actually my first PR overall, thanks so much for all your guidance and patience in helping me make it perfect!

not your first pr you made some prs in winutil, i did bullly you into closing them tho

@GabiNun2
Copy link
Copy Markdown
Contributor

you did add an unneccary space in line 104 please remove it
{CFF02815-25DA-4880-9564-979C7637653A}

@vyas-devgna vyas-devgna force-pushed the feat/argument-forwarding branch from c50fa01 to 2cf3e16 Compare May 27, 2026 19:21
@vyas-devgna
Copy link
Copy Markdown
Author

This is actually my first PR overall, thanks so much for all your guidance and patience in helping me make it perfect!

not your first pr you made some prs in winutil, i did bullly you into closing them tho

Technically my first successful one without bloat and crappy code then. Still really appreciate all the guidance and reviews(bullying)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants