-
Notifications
You must be signed in to change notification settings - Fork 222
Add app execute command for GraphQL operations
#6686
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
app execute command for GraphQL operations
Coverage report
Show new covered files 🐣
Show files with reduced coverage 🔻
Test suite run success3484 tests passing in 1406 suites. Report generated by 🧪jest coverage report action from 24bff60 |
This comment has been minimized.
This comment has been minimized.
|
/snapit |
|
🫰✨ Thanks @nickwesselman! Your snapshot has been published to npm. Test the snapshot by installing your package globally: npm i -g --@shopify:registry=https://registry.npmjs.org @shopify/cli@0.0.0-snapshot-20251204205735Caution After installing, validate the version by running just |
|
/snapit |
|
🫰✨ Thanks @nickwesselman! Your snapshot has been published to npm. Test the snapshot by installing your package globally: npm i -g --@shopify:registry=https://registry.npmjs.org @shopify/cli@0.0.0-snapshot-20251204211708Caution After installing, validate the version by running just |
|
/snapit |
|
🫰✨ Thanks @nickwesselman! Your snapshot has been published to npm. Test the snapshot by installing your package globally: npm i -g --@shopify:registry=https://registry.npmjs.org @shopify/cli@0.0.0-snapshot-20251204214618Caution After installing, validate the version by running just |
cc46276 to
7ecb877
Compare
|
/snapit |
|
🫰✨ Thanks @nickwesselman! Your snapshot has been published to npm. Test the snapshot by installing your package globally: npm i -g --@shopify:registry=https://registry.npmjs.org @shopify/cli@0.0.0-snapshot-20251204220615Caution After installing, validate the version by running just |
|
Thanks for the helper functions. The code seems much cleaner now! |
packages/app/src/cli/services/bulk-operations/execute-bulk-operation.test.ts
Show resolved
Hide resolved
packages/app/src/cli/services/bulk-operations/execute-bulk-operation.test.ts
Show resolved
Hide resolved
jordanverasamy
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not a super thorough look yet, will have to come back to this tomorrow, but from what i see so far, this is really helpful! thanks for going the extra mile on extracting the helpers :) love it!
gonzaloriestra
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice idea and refactor! It's working well, I just left a few suggestions
jordanverasamy
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
left some nits/comments but nothing blocking, this looks fantastic, thanks for putting it together :)
…` and `app bulk status`
bece5d5 to
5568d31
Compare
|
/snapit |
|
🫰✨ Thanks @nickwesselman! Your snapshot has been published to npm. Test the snapshot by installing your package globally: npm i -g --@shopify:registry=https://registry.npmjs.org @shopify/cli@0.0.0-snapshot-20251205201055Caution After installing, validate the version by running just |
jordanverasamy
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
man, really nice stuff, thanks for accommodating my nits 😂
looks great, excited for this!!
|
I did some extensive e2e testing after these last changes, I think we are in good shape! |

WHY are these changes introduced?
To provide developers with a convenient way to execute GraphQL queries and mutations directly from the CLI, making it easier to interact with Shopify's Admin API.
WHAT is this pull request doing?
Adds a new
shopify app executecommand that allows developers to:This PR also:
executecommand and existingbulk executecommandHow to test your changes?
echo "query { shop { name } }" | shopify app execute 2>/dev/null | jq .shop.nameMeasuring impact
How do we know this change was effective? Please choose one:
Checklist