Skip to content

docs(ums-cli): Update README.md based on current source code#125

Open
Copilot wants to merge 3 commits intodevelopfrom
copilot/update-readme-for-ums-cli
Open

docs(ums-cli): Update README.md based on current source code#125
Copilot wants to merge 3 commits intodevelopfrom
copilot/update-readme-for-ums-cli

Conversation

Copy link

Copilot AI commented Dec 15, 2025

README.md was out of sync with the actual CLI implementation. Updated to reflect current behavior.

Key Changes

  • validate command: Now documents runtime validation via SDK's validateAll() instead of outdated "use tsc --noEmit" guidance
  • build command: Added missing --emit-declarations option and output file documentation (.build.json, declarations/)
  • inspect command: Added missing --sources option
  • MCP commands: Marked as stubs (not yet implemented)
  • Dependencies: Corrected from ums-lib to ums-sdk

Minor Updates

  • Cognitive level filters now document enum name support (e.g., UNIVERSAL_PATTERNS)
  • Fixed YAML indentation in config example
  • Removed outdated "v1 → v2 migration" status note
  • Simplified limitations section
Original prompt

Update the README.md for package ums-cli based only on the current state of the source code of that package.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: synthable <374893+synthable@users.noreply.github.com>
Copilot AI changed the title [WIP] Update README.md for package ums-cli docs(ums-cli): Update README.md based on current source code Dec 15, 2025
Copilot AI requested a review from synthable December 15, 2025 13:25
@synthable synthable marked this pull request as ready for review December 15, 2025 14:00
Copilot AI review requested due to automatic review settings December 15, 2025 14:00
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the ums-cli README.md to accurately reflect the current CLI implementation, correcting outdated guidance and adding missing documentation for various command options and features.

Key Changes:

  • Updated validate command documentation to reflect runtime validation via SDK's validateAll() instead of outdated "use tsc --noEmit" guidance
  • Added missing command options: --emit-declarations for build, --sources for inspect, and documented support for cognitive level enum names
  • Corrected dependency reference from ums-lib to ums-sdk and marked MCP commands as stubs (not yet implemented)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.


# Verbose mode (adds extra tips)
# Validate a specific path
copilot-instructions validate ./instructions-modules
Copy link

Copilot AI Dec 15, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The example path ./instructions-modules is inconsistent with the configuration example and project structure section which use ./instruct-modules-v2. Consider updating this example to use ./instruct-modules-v2 for consistency throughout the documentation.

Suggested change
copilot-instructions validate ./instructions-modules
copilot-instructions validate ./instruct-modules-v2

Copilot uses AI. Check for mistakes.
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.

2 participants