Skip to content

List Checkly skills references in help [ship]#1382

Merged
thebiglabasky merged 4 commits into
mainfrom
herve/skills-help-actions
Jul 3, 2026
Merged

List Checkly skills references in help [ship]#1382
thebiglabasky merged 4 commits into
mainfrom
herve/skills-help-actions

Conversation

@thebiglabasky

@thebiglabasky thebiglabasky commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Summary

checkly skills --help only described the optional action and reference placeholders, even though the CLI already has a static catalog of available skill actions and references.

This updates the skills command help metadata to derive a compact tree-style action/reference index from the existing ACTIONS catalog. The detailed descriptions stay in checkly skills, checkly skills <action>, and checkly skills <action> <reference> instead of being duplicated in help output.

The skills command is also now explicitly marked read-only/idempotent in command metadata coverage.

Validation

  • ./node_modules/.bin/eslint packages/cli/src/commands/skills/index.ts packages/cli/e2e/__tests__/skills.spec.ts packages/cli/src/commands/__tests__/command-metadata.spec.ts
  • ./node_modules/.bin/tsc -p tsconfig.json --skipLibCheck from packages/cli
  • ./node_modules/.bin/vitest --run src/commands/__tests__/command-metadata.spec.ts from packages/cli
  • CHECKLY_API_KEY=dummy CHECKLY_ACCOUNT_ID=dummy NODE_CONFIG_DIR=./e2e/config ./node_modules/.bin/vitest --run -c ./vitest.config.e2e.mts e2e/__tests__/skills.spec.ts from packages/cli
  • CHECKLY_API_KEY=dummy CHECKLY_ACCOUNT_ID=dummy NODE_CONFIG_DIR=./e2e/config ./bin/run skills --help from packages/cli

Notes

Local commit hooks currently hit pnpm trust-policy verification for undici-types@6.21.0 before lint-staged can run. Commits were created with --no-verify after the focused lint, typecheck, metadata test, e2e spec, manifest generation, package tarball creation, and help smoke test passed manually.

@thebiglabasky thebiglabasky marked this pull request as ready for review July 3, 2026 12:55
@thebiglabasky thebiglabasky requested a review from stefanjudis July 3, 2026 12:55
@thebiglabasky thebiglabasky changed the title List Checkly skills references in help List Checkly skills references in help [ship] Jul 3, 2026

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Auto-approved: ship/show PR from a same-repo branch.

@thebiglabasky thebiglabasky enabled auto-merge (squash) July 3, 2026 13:06
@thebiglabasky thebiglabasky merged commit b5cf812 into main Jul 3, 2026
14 checks passed
@thebiglabasky thebiglabasky deleted the herve/skills-help-actions branch July 3, 2026 13:10
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