Give every published package its own DESCRIPTION.md - #361
Merged
Conversation
ktsu.Sdk reads DESCRIPTION.md from the project directory and falls back to the solution root, so the six packages without one were all shipping the whole-suite description to NuGet: ForceDirectedLayout, ImGui.Color, ImGui.Markdown, ImGui.NodeEditor, NodeGraph and SyntaxHighlighting each advertised the ImGui suite rather than themselves. Each now says what its own package does and what it does not depend on, since the layering is the reason several of them exist separately — NodeGraph and ForceDirectedLayout reference no UI, SyntaxHighlighting no graphics API, and ImGui.Color and ImGui.Markdown no application host. ForceDirectedLayout.Native is IsPackable=false and needs none. Verified by packing SyntaxHighlighting and reading the description back out of the generated nuspec. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015qVmjxGLzVE2mZctNNjSJC
This was referenced Sep 8, 2026
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.
ktsu.SdkresolvesDESCRIPTION.mdfrom the project directory and falls back to the solution root:Seven packages had their own file; six did not, so every one of those shipped the root description — the whole ImGui suite's elevator pitch — as its NuGet description. Someone landing on
ktsu.NodeGraph, a package with no dependencies and no UI, was told about PID frame limiting and 50+ themes.What each one now says
ktsu.ForceDirectedLayoutktsu.ImGui.Colorktsu.Semantics.Colorand Dear ImGui — conversions at the seam, contrast and accessibility analysis, noImGui.Appdependencyktsu.ImGui.Markdownktsu.ImGui.NodeEditorktsu.NodeGraphktsu.SyntaxHighlightingEach leads with what the package does and names what it deliberately does not depend on, since that layering is the reason several of these are separate packages at all. Every claim was checked against the project's own
.csprojand source — the "no dependencies at all" inNodeGraph, theIsAotCompatible/IsTrimmableandSolveinForceDirectedLayout, the zoom range inNodeEditorRenderer, the library folders behindNodeGraph's node list.ForceDirectedLayout.NativeisIsPackable=falseand needs none, so it is untouched.Verification
Packed
SyntaxHighlightingand read the description back out of the generated.nuspecto confirm the per-project file wins over the root fallback.Two things noticed while checking, both left alone as out of scope:
ImGui.NodeEditor/README.mddoes not mention the zoom added in #359, andTAGS.mdexists only at the root, so every package still shares one tag set.🤖 Generated with Claude Code
https://claude.ai/code/session_015qVmjxGLzVE2mZctNNjSJC
Generated by Claude Code