Repository navigation
Add the evidence-backed 2027-01 tax webhook warning - #52
Merged
efegokdemir merged 1 commit intoOct 22, 2026
Merged
Conversation
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.
Summary
Adds the evidence-backed UG-TAX-001 warning for Shopify's 2027-01 tax summary webhook identifier change. AI assistance: Codex prepared the changes and ran the local checks listed below.
Motivation
The documented change can affect handlers for apps subscribed to
tax_summaries/create; a bounded static warning helps app developers review the integration before upgrading.Changes
Detects the exact topic in declared
[[webhooks.subscriptions]]tables inshopify.app*.toml, only for target 2027-01 or later. Adds version-boundary and negative-case tests, migration evidence, changelog text, and a rebuilt Action bundle. Also applies the portfolio README/community and workflow updates.Tests performed
Node 22 and 24 local validation passed: clean install, tests, coverage, lint, build, distribution integrity and package/Action smoke checks. The new tests fail against the pre-change source and pass with this rule.
npm auditreported no vulnerabilities.Compatibility
Requires Node.js >=22. The rule is not in the published v0.4.0 package or Action; no existing release tag or package is changed.
Security and privacy
The rule is offline and emits a medium-confidence warning, not a security or migration guarantee. It does not inspect merchant data or call Shopify APIs.
Shopify source/evidence
https://shopify.dev/changelog/posts/tax-webhook-summary-and-calculation-requests-now-use-global-ids
https://shopify.dev/docs/apps/build/webhooks/subscribe
https://shopify.dev/changelog/release-notes/2027-01
Release impact
This is a consumer-facing source change. Review a separate SemVer minor v0.5.0 release only after the implementation merges and hosted checks pass; no package publication is part of this PR.
Checklist