Skip to content

Support multiple command strings#1723

Open
bennycode wants to merge 1 commit intomainfrom
bennycode-multiple-cmds
Open

Support multiple command strings#1723
bennycode wants to merge 1 commit intomainfrom
bennycode-multiple-cmds

Conversation

@bennycode
Copy link
Member

@bennycode bennycode commented Feb 25, 2026

Support multiple command strings by updating CommandRouter.command in [ LOLE ] module

Update CommandRouter.command to accept an array of command aliases, validate each alias starts with /, register a single handler under all lowercased aliases, and surface all aliases in commandList. Add tests for alias registration, listing, descriptions, and validation. Update the demo to register /v and /version for the version reply.

📍Where to Start

Start with the command method in CommandRouter.ts, then review the new tests in CommandRouter.test.ts, and finally the demo change in main.ts.

Macroscope summarized 1fa3a6d.

@bennycode bennycode requested a review from a team as a code owner February 25, 2026 00:23
@vercel
Copy link

vercel bot commented Feb 25, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

2 Skipped Deployments
Project Deployment Actions Updated (UTC)
xmtp-chat-api-service Skipped Skipped Feb 25, 2026 0:29am
xmtp-js-xmtp-chat Skipped Skipped Feb 25, 2026 0:29am

Request Review

@changeset-bot
Copy link

changeset-bot bot commented Feb 25, 2026

⚠️ No Changeset found

Latest commit: 1fa3a6d

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

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