Fix .NET 10 test runner and finalize ragsharp-graph rename / storage path changes#3
Conversation
There was a problem hiding this comment.
Pull request overview
This PR completes the rename from ragsharp-codegraph to ragsharp-graph, updates storage paths from .codegraph/ to .ragsharp/graph/, and fixes .NET 10 test runner compatibility issues by adopting the Microsoft Testing Platform.
- Renamed the CLI binary and updated all references from
ragsharp-codegraphtoragsharp-graph - Migrated storage paths from
.codegraph/*to.ragsharp/graph/*across code, tests, docs, and templates - Fixed .NET 10 test execution by enabling Microsoft Testing Platform in test projects and adding a Directory.Build.targets workaround for VSTest
Reviewed changes
Copilot reviewed 21 out of 22 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| tests/RagSharp.SkillInstaller.Tests/RagSharp.SkillInstaller.Tests.csproj | Added Microsoft Testing Platform properties to enable .NET 10 test execution |
| tests/RagSharp.CodeGraph.Tests/RagSharp.CodeGraph.Tests.csproj | Added Microsoft Testing Platform properties to enable .NET 10 test execution |
| tests/RagSharp.CodeGraph.Tests/CodeGraphIntegrationTests.cs | Updated test temporary paths and database locations to use new .ragsharp/graph/ structure |
| src/RagSharp.SkillInstaller/RagSharp.SkillInstaller.csproj | Added explicit PackageId to resolve project name collision |
| src/RagSharp.Packaging/RagSharp.Packaging.csproj | Added ProjectName and updated packaging path from ragsharp-codegraph to ragsharp-graph |
| src/RagSharp.CodeGraph.Cli/RagSharp.CodeGraph.Cli.csproj | Changed AssemblyName from ragsharp-codegraph to ragsharp-graph |
| src/RagSharp.CodeGraph.Cli/Program.cs | Updated all default paths from .codegraph/ to .ragsharp/graph/ and renamed CLI in help text |
| global.json | Moved testRunner configuration out of sdk block to correct JSON structure |
| docs/Development/Troubleshooting.md | Updated CLI name and paths in troubleshooting documentation |
| docs/Development/QueryContract.md | Updated CLI name in query contract documentation |
| docs/Development/Performance.md | Updated CLI name in performance documentation |
| docs/Development/CodeGraph.md | Updated CLI name, paths, and file locations throughout CLI documentation |
| assets/skill-templates/query-code-graph/scripts/run.sh | Updated CLI name and database path in query skill template |
| assets/skill-templates/query-code-graph/scripts/run.ps1 | Updated CLI name and database path in query skill template |
| assets/skill-templates/query-code-graph/SKILL.md | Updated CLI name and database path in skill documentation |
| assets/skill-templates/build-code-graph/scripts/run.sh | Updated CLI name and database/state paths in build skill template |
| assets/skill-templates/build-code-graph/scripts/run.ps1 | Updated CLI name and database/state paths in build skill template |
| assets/skill-templates/build-code-graph/references/README.md | Updated CLI name in skill reference documentation |
| assets/skill-templates/build-code-graph/SKILL.md | Updated CLI name, description, and all paths in skill documentation |
| README.md | Comprehensive update of CLI name, paths, examples, and expanded documentation with new sections |
| Directory.Build.targets | Added VSTest workaround to disable global.json testRunner guard and ensure builds before tests |
| .gitignore | Updated ignored path from .codegraph/ to .ragsharp/graph/ |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| - `.codegraph/` contains the index and state files (not committed). | ||
| - Ensure `.codegraph/` and `state.json` remain in `.gitignore`. | ||
| - `.ragsharp/graph/` contains the index and state files (not committed). | ||
| - Ensure `.ragsharp/graph/` and `state.json` remain in `.gitignore`. |
There was a problem hiding this comment.
The documentation states "Ensure .ragsharp/graph/ and state.json remain in .gitignore", but the .gitignore pattern .ragsharp/graph/ will already ignore state.json since it's located within that directory. The phrasing suggests they are separate items to ignore, which could be confusing. Consider updating to "Ensure .ragsharp/ remains in .gitignore" or "Ensure .ragsharp/graph/ (which contains index.db and state.json) remains in .gitignore."
| - Ensure `.ragsharp/graph/` and `state.json` remain in `.gitignore`. | |
| - Ensure `.ragsharp/graph/` (which contains `index.db` and `state.json`) remains in `.gitignore`. |
|
|
||
| - `.ragsharp/graph/index.db` — SQLite graph database. | ||
| - `.ragsharp/graph/state.json` — incremental index state. | ||
| - `.ragsharp/graph/schema_version` — schema version guard. |
There was a problem hiding this comment.
The markdown formatting is missing a blank line between the "Files" section and the "Exit codes" heading. This can cause the "Exit codes" heading to not render properly in some markdown parsers. Add a blank line before line 16.
| - `.ragsharp/graph/schema_version` — schema version guard. | |
| - `.ragsharp/graph/schema_version` — schema version guard. |
Motivation
dotnet testfrom running.Description
ragsharp-graphand changed default index/state locations from.codegraph/*to.ragsharp/graph/*inProgram.cs, docs, and skill templates.ProjectName/PackageId) and updating the solution entries to includeNameattributes so restore no longer reports an ambiguous project name.EnableTestingPlatform,TestingPlatformDotnetTestSupport, andUseMSBuildTestInfrastructureproperties and added aDirectory.Build.targetsoverride to avoid the VSTest guard while ensuring builds occur before test invocation.RagSharp.Packaging.csproj, adjustedglobal.jsonlayout, and refreshed README/docs/examples to useragsharp-graphand.ragsharp/graphpaths.Testing
10.0.100) via the installed runtime during the run.dotnet test ragsharp.slnxand confirmed it completes successfully after the fixes.RagSharp.CodeGraph.Testspassed (103 tests) andRagSharp.SkillInstaller.Testspassed (2 tests), with no failing tests.projectNameentries so project restore now succeeds.Codex Task