Repair builds, complete dev CLI, and satisfy ganda repo audit#4
Merged
Conversation
The repo failed ganda repo audit on 11 checks and, independently, neither the library nor the dev CLI compiled: IDE0005 requires GenerateDocumentationFile, the runfile tripped IDE0211 (top-level statements are mandatory in runfiles), and the slnx listed dev.cs as a Project, which the MSBuild solution parser rejects. - Apply ganda audit fixes: directory scaffold, memsearch hooks, VS Code window icon, TimeWarp.Nuru 3.0.0-beta.71, CPM cleanup, and TimeWarp.Build.Tasks wired at the root (removed the source/ copy that caused a duplicate-reference NU1504) - Enable GenerateDocumentationFile repo-wide so IDE0005 runs on build; the package now ships XML docs - Add check-version and verify-samples endpoints required by the dev-cli-capabilities check; workflow release mode delegates its version-check step to the endpoint instead of a private copy - List dev.cs as a File solution item; runfiles have no project type - Keep dev clean from deleting the installed bin/dev binary and let dev test skip when tests/ holds no test files - Add #region Purpose annotations to all dev-cli files - Bump version to 1.0.0-beta.3; beta.2 is already published 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Fable 5 <[email protected]>
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
1 participant
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
Brings the repository into full
ganda repo auditcompliance (was 11 failing checks, now 0) and fixes pre-existing build breakage discovered along the way.Audit compliance
TimeWarp.Build.Taskswired in the rootDirectory.Build.props#region Purposeannotations to all dev-cli filescheck-versionandverify-samplesendpoints required by thedev-cli-capabilitiescheck;workflowrelease mode now delegates tocheck-versioninstead of keeping a private copy of the logicBuild fixes (pre-existing, confirmed on the clean tree)
GenerateDocumentationFile=true(now set repo-wide; the package ships XML docs), and the runfile tripped IDE0211 since runfiles must use top-level statements (now suppressed for the dev-cli only)source/TimeWarp.Build.Tasksreference that duplicated the root one (NU1504)timewarp-builder.slnxlisteddev.csas a<Project>, which the MSBuild solution parser rejects — it is now a<File>solution itemGlobalSuppressions.cs: its only suppression (IDE0290) is already in the globalNoWarn, and its target string was invalid (IDE0076)Dev CLI robustness
dev cleanno longer deletes the installedbin/devbinary out from under itselfdev testskips whentests/contains no test files (only.gitkeep)Release readiness
dev check-versionconfirms it is safe to releaseTest plan
ganda repo audit— 21 passed, 0 faileddev workflow --mode pr(the exact CI entry point) — clean → build → test, exit 0dev format,dotnet build timewarp-builder.slnx,dev check-version,dev verify-samplesall pass locally🤖 Generated with Claude Code