Skip to content

fix: correct capability presentation examples - #846

Open
sungmpar wants to merge 1 commit into
SmartThingsCommunity:mainfrom
sungmpar:fix/capabilities-presentation-examples
Open

fix: correct capability presentation examples#846
sungmpar wants to merge 1 commit into
SmartThingsCommunity:mainfrom
sungmpar:fix/capabilities-presentation-examples

Conversation

@sungmpar

Copy link
Copy Markdown

Summary

  • correct the four capabilities:presentation help examples that used the
    non-existent capabilities:presentations command
  • strengthen regression coverage to verify the complete example array
  • add a patch changeset for @smartthings/cli

Root cause

The command is registered as capabilities:presentation, but four independently
hard-coded help examples used capabilities:presentations. The existing builder
test only verified that the example builder was called once, so incorrect
example arguments were not detected.

Verification

  • npm run build
  • npm run test -- --runTestsByPath src/__tests__/commands/capabilities/presentation.test.ts --no-watchman
  • npm run lint
  • npm run test -- --no-watchman
  • git diff --check upstream/main...HEAD
  • node dist/src/run.js capabilities:presentation --help

The built help output contains all four corrected singular examples and zero
capabilities:presentations examples. The README generator ran as part of
npm run build and produced no tracked README change.

Fixes #842

Checklist

  • I have read the CONTRIBUTING document
  • Any required documentation has been added
  • My code follows the code style of this project (npm run lint produces no warnings/errors)
  • I have added tests to cover my changes

@changeset-bot

changeset-bot Bot commented Aug 30, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 20ce290

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@smartthings/cli Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another 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.

Examples for smartthings capabilities:presentation refer to capabilities:presentations.

1 participant