feat(agent): add PowerShell command interface - #6238
Open
Avallix wants to merge 1 commit into
Open
Conversation
Add a native PowerShell executor, schema and parser wiring, routing and permission gates, UI metadata, and regression tests.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds PowerShell as a first-class command interface so Windows users can run PowerShell commands through the agent tool loop instead of having PowerShell requests route as unavailable shell/tool actions. The change adds the executor, parser/schema wiring, routing and permission gates, UI labels, and regression coverage for the new tool path.
Target branch
dev, notmain. All PRs land indev;mainis curated by the maintainer at each release. If your PR is onmainby accident, click "Edit" on this PR and change the base.Linked Issue
Fixes #6237
Type of Change
Checklist
devdocker compose uporuvicorn app:app) and verified the change works end-to-end. Type-checks and unit tests are not enough.How to Test
Start the app from a clean temporary local data directory with auth disabled for validation:
The app started successfully at
http://127.0.0.1:7701/. ChromaDB-backed RAG/memory reported degraded because ChromaDB was not running locally; the app remained usable and that service is unrelated to this PowerShell command-interface change.Open Settings → Agent Tools, expand the Code section, and verify
PowerShellappears with the descriptionExecute PowerShell commands.Run the focused validation used for this branch:
Result from the local run:
235 passed, 1 warning.Run the manual PowerShell smoke checks:
The direct tool smoke resolved
i.exe, and the dispatcher smoke returneddispatcher-okwith exit code0.Visual / UI changes — REQUIRED if you touched anything that renders
Anything that changes what the UI looks like — buttons, icons, padding, colors, fonts, spacing, layout, CSS, HTML, SVG, or any
static/jsmodule that draws to the DOM — needs all of the following. PRs that change rendering without these WILL be closed.--red,--fg,--bg,--card,--border, etc.) — do not introduce new color values, font sizes, or spacing units.static/index.html) or plain text.Fira Code) for primary UI text. Don't override.Screenshots / clips