rename to skills#10
Open
PascalSenn wants to merge 2 commits into
Open
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Renames the CLI and primary namespaces from Skillz to Skills, updating the tool packaging, tests, and CI/release/e2e infrastructure to match the new command name and output.
Changes:
- Renamed namespaces/usings and user-facing strings from
Skillz/skillztoSkills/skillsacross src and tests. - Updated CLI entrypoints/tool wrapper to support the new command name via
CliExecutionContext.CommandName. - Updated smoke/e2e tests plus CI/release workflows to build, run, and validate the
skillsbinary.
Reviewed changes
Copilot reviewed 197 out of 199 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| test/Skills.Tests/Utils/VimNavConsoleTests.cs | Update usings/namespace for rename |
| test/Skills.Tests/Utils/FileLockTests.cs | Update usings/namespace and temp path prefix |
| test/Skills.Tests/Utils/CommandSnapshot.cs | Update command/root command naming in snapshots |
| test/Skills.Tests/Utils/CliTestHelperTests.cs | Update usings/namespace for rename |
| test/Skills.Tests/Utils/CliTestHelper.cs | Update DI registrations/root command type |
| test/Skills.Tests/TestServices/TestSourceParser.cs | Update usings/namespace for rename |
| test/Skills.Tests/TestServices/TestSkillDiscovery.cs | Update usings/namespace for rename |
| test/Skills.Tests/TestServices/TestRemoveCommandPrompter.cs | Update usings/namespace for rename |
| test/Skills.Tests/TestServices/TestProjectLockFile.cs | Update lock filename and namespaces |
| test/Skills.Tests/TestServices/TestInteractionService.cs | Update usings/namespace for rename |
| test/Skills.Tests/TestServices/TestInstaller.cs | Update usings/namespace for rename |
| test/Skills.Tests/TestServices/TestGlobalLockFile.cs | Update usings/namespace for rename |
| test/Skills.Tests/TestServices/TestGitHubTokenProvider.cs | Update usings/namespace for rename |
| test/Skills.Tests/TestServices/TestGitClient.cs | Update usings/namespace for rename |
| test/Skills.Tests/TestServices/TestConsoleEnvironment.cs | Update namespace for rename |
| test/Skills.Tests/TestServices/TestBlobClient.cs | Update usings/namespace for rename |
| test/Skills.Tests/TestServices/TestAddCommandPrompter.cs | Update usings/namespace for rename |
| test/Skills.Tests/TestServices/FakeTimeProvider.cs | Update namespace for rename |
| test/Skills.Tests/TestServices/FakeSystemEnvironment.cs | Update usings/namespace for rename |
| test/Skills.Tests/TestServices/FakeFileStoreTests.cs | Update namespace for rename |
| test/Skills.Tests/TestServices/FakeFileStore.cs | Update usings/namespace and error hint text |
| test/Skills.Tests/Sources/SourceParserTests.cs | Update usings/namespace for rename |
| test/Skills.Tests/Sources/Providers/WellKnownProviderTests.cs | Update usings/namespace for rename |
| test/Skills.Tests/Sources/Providers/ProviderRegistryTests.cs | Update usings/namespace for rename |
| test/Skills.Tests/SmokeTest.cs | Update usings/namespace for rename |
| test/Skills.Tests/Skills/SkillNameSanitizerTests.cs | Update usings/namespace for rename |
| test/Skills.Tests/Skills/SkillDiscoveryTests.cs | Update usings/namespace and temp path prefixes |
| test/Skills.Tests/Skills/FrontmatterParserTests.cs | Update usings/namespace for rename |
| test/Skills.Tests/Skills.Tests.csproj | Reference renamed src project |
| test/Skills.Tests/ProgramTests.cs | Update usings/namespace for rename |
| test/Skills.Tests/Plugins/PluginManifestTests.cs | Update usings/namespace and temp path prefixes |
| test/Skills.Tests/Plugins/PluginGroupingTests.cs | Update usings/namespace and temp path prefixes |
| test/Skills.Tests/Paths/SafeTreeWalkerTests.cs | Update usings/namespace and temp path prefix |
| test/Skills.Tests/Paths/SafePathTraversalTests.cs | Update usings/namespace and temp path prefix |
| test/Skills.Tests/Paths/SafePathTests.cs | Update usings/namespace and doc comment strings |
| test/Skills.Tests/Paths/SafePathDisplayTests.cs | Update usings/namespace for rename |
| test/Skills.Tests/Paths/SafePathContainmentTests.cs | Update namespace and temp path prefixes |
| test/Skills.Tests/Net/StubHttpMessageHandler.cs | Update namespace for rename |
| test/Skills.Tests/Net/MaxBytesStreamTests.cs | Update usings/namespace for rename |
| test/Skills.Tests/Net/HttpClientDefaultsTests.cs | Update namespace for rename |
| test/Skills.Tests/Net/FakeHttpClientFactory.cs | Update namespace for rename |
| test/Skills.Tests/Net/FakeGitHubTokenProvider.cs | Update usings/namespace for rename |
| test/Skills.Tests/Net/BlobClientTests.cs | Update usings/namespace for rename |
| test/Skills.Tests/Lock/ProjectLockFileTests.cs | Update usings/namespace and temp path prefix |
| test/Skills.Tests/Lock/GlobalLockFileTests.cs | Update usings/namespace and temp path prefix |
| test/Skills.Tests/Interaction/SearchableMultiSelectionPromptTests.cs | Update usings/namespace for rename |
| test/Skills.Tests/Interaction/ConsoleInteractionServiceTests.cs | Update usings/namespace for rename |
| test/Skills.Tests/Install/XdgPathsTests.cs | Update namespaces and expected XDG paths |
| test/Skills.Tests/Install/InstallerTests.cs | Update namespaces and temp path prefix |
| test/Skills.Tests/Install/AgentRegistryTests.cs | Update usings/namespace for rename |
| test/Skills.Tests/Install/AgentEnvironmentTests.cs | Update usings/namespace for rename |
| test/Skills.Tests/Git/GitUrlTests.cs | Update usings/namespace for rename |
| test/Skills.Tests/Git/GitClientTests.cs | Update namespaces, env vars, and temp path prefixes |
| test/Skills.Tests/Git/GitArgumentsTests.cs | Update usings/namespace for rename |
| test/Skills.Tests/ContainsControlCharacterTests.cs | Update usings/namespace for rename |
| test/Skills.Tests/Commands/UpdateCommandTests.cs | Update namespaces and expected output strings |
| test/Skills.Tests/Commands/UpdateCommandSnapshotTests.cs | Update snapshots to skills command |
| test/Skills.Tests/Commands/UpdateCommandProgressTests.cs | Update usings/namespace for rename |
| test/Skills.Tests/Commands/RemoveCommandTests.cs | Update namespaces and temp path prefix |
| test/Skills.Tests/Commands/RemoveCommandSnapshotTests.cs | Update snapshots to skills command |
| test/Skills.Tests/Commands/ListCommandTests.cs | Update namespaces and temp path prefix |
| test/Skills.Tests/Commands/ListCommandSnapshotTests.cs | Update snapshots to skills command |
| test/Skills.Tests/Commands/InitCommandTests.cs | Update namespaces and temp path prefix |
| test/Skills.Tests/Commands/InitCommandSnapshotTests.cs | Update snapshots to skills command |
| test/Skills.Tests/Commands/CommandTestCollection.cs | Update namespace for rename |
| test/Skills.Tests/Commands/AddCommandTests.cs | Update namespaces and canonical path prefixes |
| test/Skills.Tests/Commands/AddCommandSnapshotTests.cs | Update snapshots/paths to skills |
| test/Skills.Tests/Commands/AddCommandPrompterTests.cs | Update usings/namespace for rename |
| test/Skills.SmokeTests/SmokeTests.cs | Update smoke tests to run skills |
| test/Skills.SmokeTests/Skills.SmokeTests.csproj | New/updated smoke test project file |
| test/e2e/update-flow.tape | Update recorded commands to skills |
| test/e2e/update-flow.golden.txt | Update golden output to skills |
| test/e2e/run.sh | Update publish/run script to skills |
| test/e2e/remove-flow.tape | Update recorded commands to skills |
| test/e2e/remove-flow.golden.txt | Update golden output to skills |
| test/e2e/README.md | Update e2e docs and artifact naming |
| test/e2e/list-flow.tape | Update recorded commands to skills |
| test/e2e/list-flow.golden.txt | Update golden output to skills |
| test/e2e/init-flow.tape | Update recorded commands to skills |
| test/e2e/init-flow.golden.txt | Update golden output to skills |
| test/e2e/global-flow.tape | Update recorded commands to skills |
| test/e2e/global-flow.golden.txt | Update golden output to skills |
| test/e2e/error-flow.tape | Update error flow command/markers to skills |
| test/e2e/error-flow.golden.txt | Update golden output/markers to skills |
| test/e2e/copy-flow.tape | Update recorded commands to skills |
| test/e2e/copy-flow.golden.txt | Update golden output to skills |
| test/e2e/add-flow.tape | Update recorded commands to skills |
| test/e2e/add-flow.golden.txt | Update golden output to skills |
| src/Skillz.Tool/Program.cs | Remove old wrapper entrypoint |
| src/Skills/Utils/VimNavConsole.cs | Update namespace for rename |
| src/Skills/Utils/SystemFileStore.cs | Update namespace/usings for rename |
| src/Skills/Utils/IFileStore.cs | Update namespace/usings for rename |
| src/Skills/Utils/FileLock.cs | Update namespace for rename |
| src/Skills/StringExtensions.cs | Update namespace for rename |
| src/Skills/Sources/SourceParser.cs | Update namespace/usings for rename |
| src/Skills/Sources/SkillSource.cs | Update namespace for rename |
| src/Skills/Sources/Providers/WellKnownProvider.cs | Update namespaces/constants/temp prefixes |
| src/Skills/Sources/Providers/TempDirCleanup.cs | Update namespace for rename |
| src/Skills/Sources/Providers/ProviderRegistry.cs | Update namespace for rename |
| src/Skills/Sources/Providers/ProviderConversions.cs | Update namespace/usings for rename |
| src/Skills/Sources/Providers/Models/WellKnownIndexEntry.cs | Update namespace for rename |
| src/Skills/Sources/Providers/Models/WellKnownIndex.cs | Update namespace for rename |
| src/Skills/Sources/Providers/LocalProvider.cs | Update namespace/usings for rename |
| src/Skills/Sources/Providers/IProvider.cs | Update namespace/usings for rename |
| src/Skills/Sources/Providers/GitProvider.cs | Update namespace/usings/temp prefix |
| src/Skills/Sources/Providers/GitLabProvider.cs | Update namespace/usings/temp prefix |
| src/Skills/Sources/Providers/GitHubProvider.cs | Update namespace/usings/temp prefix |
| src/Skills/Sources/OwnerRepoParser.cs | Update namespace for rename |
| src/Skills/Skills/TerminalSanitizer.cs | Update namespace for rename |
| src/Skills/Skills/SkillNameSanitizer.cs | Update namespace for rename |
| src/Skills/Skills/SkillDiscovery.cs | Update namespace/usings for rename |
| src/Skills/Skills/ResolvedSkill.cs | Update namespace for rename |
| src/Skills/Skills/Models/Skill.cs | Update namespace for rename |
| src/Skills/Skills/ISkillDiscovery.cs | Update namespace for rename |
| src/Skills/Skills/FrontmatterParser.cs | Update namespace for rename |
| src/Skills/Skills.csproj | Rename assembly/root namespace and IVT |
| src/Skills/Program.cs | Add RunAsync + wire command name context |
| src/Skills/Plugins/PluginManifest.cs | Update namespace/usings for rename |
| src/Skills/Plugins/PluginGrouping.cs | Update namespace/usings for rename |
| src/Skills/Plugins/Models/SinglePluginManifest.cs | Update namespace for rename |
| src/Skills/Plugins/Models/PluginManifestEntry.cs | Update namespace for rename |
| src/Skills/Plugins/Models/MarketplaceMetadata.cs | Update namespace for rename |
| src/Skills/Plugins/Models/MarketplaceManifest.cs | Update namespace for rename |
| src/Skills/Paths/WalkOptions.cs | Update namespace for rename |
| src/Skills/Paths/WalkEntryKind.cs | Update namespace for rename |
| src/Skills/Paths/WalkEntry.cs | Update namespace for rename |
| src/Skills/Paths/SafeTreeWalker.cs | Update namespace for rename |
| src/Skills/Paths/SafeTreeHash.cs | Update namespace for rename |
| src/Skills/Paths/SafePath.cs | Update namespace and user-facing hint strings |
| src/Skills/Paths/OnSymlink.cs | Update namespace for rename |
| src/Skills/Paths/LeafPolicy.cs | Update namespace and comment strings |
| src/Skills/Net/Models/TreeEntry.cs | Update namespace for rename |
| src/Skills/Net/Models/RepoTree.cs | Update namespace for rename |
| src/Skills/Net/Models/GitHubTreeResponse.cs | Update namespace for rename |
| src/Skills/Net/MaxBytesStream.cs | Update namespace for rename |
| src/Skills/Net/IGitHubTokenProvider.cs | Update namespace for rename |
| src/Skills/Net/IBlobClient.cs | Update namespace for rename |
| src/Skills/Net/GitHubTokenProvider.cs | Update namespace for rename |
| src/Skills/Net/BlobFetchTimeoutException.cs | Update namespace for rename |
| src/Skills/Net/BlobClient.cs | Update client names/user-agent strings |
| src/Skills/Lock/ProjectLockFile.cs | Update namespace/usings for rename |
| src/Skills/Lock/Models/SkillLockFile.cs | Update namespace for rename |
| src/Skills/Lock/Models/SkillLockEntry.cs | Update namespace for rename |
| src/Skills/Lock/Models/LocalSkillLockFile.cs | Update namespace for rename |
| src/Skills/Lock/Models/LocalSkillLockEntry.cs | Update namespace for rename |
| src/Skills/Lock/JsonLockFile.cs | Update warning/error strings to skills |
| src/Skills/Lock/IProjectLockFile.cs | Update namespace for rename |
| src/Skills/Lock/IGlobalLockFile.cs | Update namespace for rename |
| src/Skills/Lock/GlobalLockFile.cs | Update namespace/usings for rename |
| src/Skills/KnownConfigNames.cs | Update namespace and lock filename constant |
| src/Skills/JsonSourceGenerationContext.cs | Update namespace/usings for rename |
| src/Skills/Interaction/SearchableMultiSelectionPrompt.cs | Update namespace for rename |
| src/Skills/Interaction/SearchableItem.cs | Update namespace for rename |
| src/Skills/Interaction/IInteractionService.cs | Update namespace for rename |
| src/Skills/Interaction/ConsoleInteractionService.cs | Update namespace/usings for rename |
| src/Skills/Interaction/BannerService.cs | Use dynamic command name in banner/help |
| src/Skills/Install/XdgPaths.cs | Update namespaces and XDG root directory |
| src/Skills/Install/SystemEnvironment.cs | Update namespace for rename |
| src/Skills/Install/SkillInstaller.cs | Update namespace/usings and comment strings |
| src/Skills/Install/ISystemEnvironment.cs | Update namespace for rename |
| src/Skills/Install/ISkillInstaller.cs | Update namespace/usings for rename |
| src/Skills/Install/InstallResult.cs | Update namespace for rename |
| src/Skills/Install/AgentRegistry.cs | Update namespace for rename |
| src/Skills/Install/AgentEnvironment.cs | Update namespace for rename |
| src/Skills/Install/AgentConfig.cs | Update namespace for rename |
| src/Skills/Git/IGitClient.cs | Update namespace for rename |
| src/Skills/Git/GitUrl.cs | Update namespace for rename |
| src/Skills/Git/GitCloneException.cs | Update namespace for rename |
| src/Skills/Git/GitClient.cs | Rename clone timeout env var |
| src/Skills/Git/GitArguments.cs | Update namespace for rename |
| src/Skills/ExitCodeConstants.cs | Update namespace for rename |
| src/Skills/ConsoleEnvironment.cs | Update namespace for rename |
| src/Skills/CommonOptionNames.cs | Update namespace for rename |
| src/Skills/Commands/UpdateCommand.cs | Use dynamic command name in suggestions |
| src/Skills/Commands/SkillsRootCommand.cs | Rename root command type/description |
| src/Skills/Commands/RemoveCommandPrompter.cs | Update namespace/usings for rename |
| src/Skills/Commands/RemoveCommand.cs | Update namespace/usings for rename |
| src/Skills/Commands/ListCommand.cs | Update namespace/usings for rename |
| src/Skills/Commands/IRemoveCommandPrompter.cs | Update namespace for rename |
| src/Skills/Commands/InitCommand.cs | Use dynamic command name in guidance text |
| src/Skills/Commands/IAddCommandPrompter.cs | Update namespace/usings for rename |
| src/Skills/Commands/CommandResult.cs | Update namespace for rename |
| src/Skills/Commands/BaseCommand.cs | Update namespace for rename |
| src/Skills/Commands/AddCommandPrompter.cs | Update namespace and comment string |
| src/Skills/Commands/AddCommandOptions.cs | Update namespace for rename |
| src/Skills/Commands/AddCommandExecutor.cs | Update namespace/usings for rename |
| src/Skills/Commands/AddCommand.cs | Use dynamic command name in usage text |
| src/Skills/CliExecutionContext.cs | Add CommandName to execution context |
| src/Skills/CliException.cs | Update namespace for rename |
| src/Skills.Tool/Skills.Tool.csproj | New dual-package tool packaging setup |
| src/Skills.Tool/README.md | Update tool docs for skills |
| src/Skills.Tool/Program.cs | Wrapper passes tool command name to CLI |
| Skills.sln | Rename projects/paths in solution |
| README.md | Update main README to Skills/skills |
| Directory.Build.props | Update repository URL to skills |
| .github/workflows/release.yml | Publish/package/push skills + dual nupkgs |
| .github/workflows/e2e-demo.yml | Update paths/artifact names to skills |
| .github/workflows/ci.yml | Update AOT publish target path/name |
Comments suppressed due to low confidence (1)
src/Skills.Tool/README.md:13
- The README says “The
skillspackage is published as an equivalent package alias…”, but the alias package isskillz(seeSkills.Tool.csprojwherePackageIdis set viaSkillsToolPackageIdand CI packs bothskillsandskillz). As written, this is misleading for users looking for the legacy package name.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
✅ E2E snapshots — no flow recordings changed vs |
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.
No description provided.