Skip to content

Unhide shopify app security commands - #8766

Open
nickwesselman wants to merge 2 commits into
mainfrom
unhide-app-security-commands
Open

nickwesselman wants to merge 2 commits into
mainfrom
unhide-app-security-commands

Conversation

@nickwesselman

Copy link
Copy Markdown
Contributor

Replaces #8742, which GitHub closed when its stacked base branch was deleted after the App Security stack merged. This is the same change, rebased onto main. It touches the same generated files and command tests as #8765 (generic "app security check" wording), so whichever merges second needs a quick rebase.

WHY are these changes introduced?

The shopify app security commands (check, clean, instructions, record, review) were hidden while in development. They're ready to be discoverable.

WHAT is this pull request doing?

Removes hidden = true from the app security commands so they appear in help output, the CLI README, and shopify.dev docs. Regenerates the manifest, README, dev docs, and commands snapshot.

How to manually test your changes?

  • pnpm shopify app --help — app security commands are listed
  • pnpm shopify app security check --help

Checklist

  • I've considered possible cross-platform impacts (Mac, Linux, Windows)
  • I've considered possible documentation changes
  • I've considered analytics changes to measure impact
  • The change is user-facing — I've identified the correct bump type (patch for bug fixes · minor for new features · major for breaking changes) and added a changeset with pnpm changeset add

🤖 Generated with Claude Code

@nickwesselman
nickwesselman requested review from a team as code owners October 5, 2026 21:42
@nickwesselman nickwesselman mentioned this pull request Oct 5, 2026
1 of 4 tasks
@github-actions github-actions Bot added shopify.dev preview Area: @shopify/cli @shopify/cli package issues labels Oct 5, 2026
@nickwesselman
nickwesselman force-pushed the unhide-app-security-commands branch from 8a59a65 to a627752 Compare October 5, 2026 21:44
@nickwesselman
nickwesselman force-pushed the unhide-app-security-commands branch from a627752 to 74b3d61 Compare October 5, 2026 22:28
@nickwesselman

Copy link
Copy Markdown
Contributor Author

/snapit

@github-actions

github-actions Bot commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

🫰✨ Thanks @nickwesselman! Your snapshot has been published to npm.

Built from 74b3d6187c4a0a075cd59578f4a8812b0adc1c7b. Workflow run.

Test the snapshot by installing your package globally:

pnpm i -g --@shopify:registry=https://registry.npmjs.org @shopify/cli@0.0.0-snapshot-20261005222923

Caution

After installing, validate the version by running shopify version in your terminal.
If the versions don't match, you might have multiple global instances installed.
Use which shopify to find out which one you are running and uninstall it.

nickwesselman and others added 2 commits October 5, 2026 18:39
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Visibility is already covered by the generated oclif manifest and the e2e
command tree snapshot, so the unit tests no longer assert on `hidden`.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@nickwesselman
nickwesselman force-pushed the unhide-app-security-commands branch from 74b3d61 to 8eb5385 Compare October 5, 2026 22:39
@nickwesselman

Copy link
Copy Markdown
Contributor Author

/snapit

@github-actions

github-actions Bot commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

🫰✨ Thanks @nickwesselman! Your snapshot has been published to npm.

Built from 8eb5385437819fedfed1d5798a56a096268e63e8. Workflow run.

Test the snapshot by installing your package globally:

pnpm i -g --@shopify:registry=https://registry.npmjs.org @shopify/cli@0.0.0-snapshot-20261005224042

Caution

After installing, validate the version by running shopify version in your terminal.
If the versions don't match, you might have multiple global instances installed.
Use which shopify to find out which one you are running and uninstall it.

This branch has not been deployed

No deployments
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.

2 participants