Skip to content

Adds nitro skills to Nitro Commandline - #10303

Open
PascalSenn wants to merge 3 commits into
mainfrom
pse/adds-skills-to-nitro
Open

Adds nitro skills to Nitro Commandline#10303
PascalSenn wants to merge 3 commits into
mainfrom
pse/adds-skills-to-nitro

Conversation

@PascalSenn

Copy link
Copy Markdown
Member

No description provided.

Copilot AI lite review requested due to automatic review settings August 27, 2026 10:24
@github-actions

Copy link
Copy Markdown
Contributor

Nitro CLI E2E snapshots: no flow recordings changed vs main.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR integrates the Skills feature into the Nitro CLI by wiring a new nitro skills command into the command tree and registering the required services, along with a smoke test to validate the command’s help output.

Changes:

  • Register Skills DI services in the Nitro CLI startup.
  • Add a skills subcommand to the root command.
  • Add a smoke test verifying nitro skills --help output.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
src/Nitro/CommandLine/test/CommandLine.Smoke.Tests/SmokeTests.cs Adds a smoke test covering skills --help output.
src/Nitro/CommandLine/src/CommandLine/Program.cs Registers Skills services and adds the SkillsCommand to the CLI.
src/Nitro/CommandLine/src/CommandLine/Nitro.CommandLine.csproj Adds the Skills.Library package reference to the CLI project.
src/Directory.Packages.props Adds the centrally-managed version for Skills.Library.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/Nitro/CommandLine/test/CommandLine.Smoke.Tests/SmokeTests.cs Outdated
PascalSenn and others added 2 commits August 27, 2026 15:55
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
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