Skip to content

fix: use opaque Comms IDs in help examples - #40

Merged
jacobo-doist merged 1 commit into
mainfrom
jacobo/add-cli-completion
Jul 14, 2026
Merged

fix: use opaque Comms IDs in help examples#40
jacobo-doist merged 1 commit into
mainfrom
jacobo/add-cli-completion

Conversation

@jacobo-doist

Copy link
Copy Markdown
Contributor

Summary

  • replace numeric-looking Comms entity IDs in command help examples with opaque id: refs
  • update reference docs to clarify numeric IDs are for users/workspaces while most Comms entities use opaque strings
  • regenerate the bundled comms-cli skill file

Testing

  • npm run sync:skill
  • npm run type-check
  • npm run lint:check
  • npm run check:skill-sync
  • npm test
  • built help smoke checks for conversation/thread/view

@doistbot
doistbot requested a review from henningmu July 10, 2026 09:55
@jacobo-doist jacobo-doist changed the title Fix opaque Comms IDs in help examples fix: use opaque Comms IDs in help examples Jul 10, 2026

@doistbot doistbot left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This PR updates Fix opaque Comms IDs in help examples. No issues were flagged in the reviewed diff.

Share FeedbackReview Logs

@henningmu henningmu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Good stuff, thank you 👏

@jacobo-doist
jacobo-doist merged commit f8ad8e4 into main Jul 14, 2026
6 of 7 checks passed
@jacobo-doist
jacobo-doist deleted the jacobo/add-cli-completion branch July 14, 2026 09:31
doist-release-bot Bot added a commit that referenced this pull request Jul 14, 2026
## [1.9.3](v1.9.2...v1.9.3) (2026-07-14)

### Bug Fixes

* forward conversation and thread date filters ([#42](#42)) ([f69968e](f69968e))
* use opaque Comms IDs in help examples ([#40](#40)) ([f8ad8e4](f8ad8e4))
@doist-release-bot

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 1.9.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants