Repository navigation
Refer to app security checks generically instead of "App Security" - #8765
Merged
Merged
Conversation
dmerand
approved these changes
Oct 5, 2026
nickwesselman
enabled auto-merge
October 5, 2026 21:36
1 of 4 tasks
github-merge-queue
Bot
removed this pull request from the merge queue due to a conflict with the base branch
Oct 5, 2026
nickwesselman
enabled auto-merge
October 5, 2026 22:06
Replace the title-cased "App Security" product name in command help, messages, agent instructions, check prompts, comments, and tests with generic wording: "app security check" and "app security check results". Regenerates the embedded checks, oclif manifest, and docs. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
nickwesselman
force-pushed
the
app-security-generic-wording
branch
from
October 5, 2026 22:08
639712e to
ebd7430
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
WHY are these changes introduced?
The
shopify app securitycommands, their messages and the agent instructions call the feature "App Security", in title case, as if it were a product name. It should be described generically as an app security check.WHAT is this pull request doing?
Replaces every title-cased "App Security" outside CHANGELOGs with generic wording, mostly "app security check", "app security check results" and "app security check instructions". The replacements were written phrase by phrase so sentences still read naturally. For example:
Covered:
check,clean,instructions,record,review)INSTRUCTIONS.md) and theEOL_API_VERSIONcheck promptRegenerated:
checks/embedded.ts(viaembed-checks.mjs) andoclif.manifest.json. The README and docs are unchanged because the commands are still hidden onmain."App Store", the product name, is unchanged. "App Security" still appears in CHANGELOGs, which record history.
How to manually test your changes?
shopify app security check --yeson an app. The summary box and the printed agent instructions should say "app security check", never "App Security".shopify app security reviewbefore runningcheck. It should say "No app security check results to review."Checklist
patch) and added a changeset.🤖 Generated with Claude Code