Skip to content

fix: document optional command parameters correctly - #109

Open
vycdev2 wants to merge 1 commit into
vycdev:developfrom
vycdev2:fix/command-doc-optional-parameters
Open

fix: document optional command parameters correctly#109
vycdev2 wants to merge 1 commit into
vycdev:developfrom
vycdev2:fix/command-doc-optional-parameters

Conversation

@vycdev2

@vycdev2 vycdev2 commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • fix the command-documentation generator to report optional parameter names and types instead of default values
  • preserve the repository's generated COMMANDS.md path formatting across platforms to keep the generated diff focused
  • add Python regression coverage and contributor instructions for the generator tests

Verification

  • dotnet restore Morpheus.sln — passed; existing NU1903 SQLitePCLRaw vulnerability warning and SDK locale warnings remain
  • dotnet build Morpheus.sln --no-restore — passed with 0 errors; existing warnings remain
  • focused .NET test HelpCommandRegistrationTests — passed (1/1)
  • full .NET test suite — 297 passed, 2 unrelated tr-TR globalization failures in MiscModuleTests because this runner is in invariant globalization mode
  • full .NET suite excluding NormalizeTimeUntilEventName_NormalizesCase — passed (297/297)
  • python3 -m unittest tools.test_generate_commands_md -v — passed (3/3)
  • python3 tools/generate_commands_md.py — passed; generated COMMANDS.md is 1,421 lines
  • generated documentation diff audit — passed; 82 optional-parameter entries changed and no unrelated source-path churn remains
  • git diff --check — passed

Risk

  • Low: this changes documentation parsing and generated command reference output only; command runtime behavior is unchanged.

This was generated by an AI agent (vycdev2). Please verify any changes before merging or applying.

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.

1 participant