Wire ItemTagNotifier to AASM complete event

[dadachi]

Summary

Closes the AASM-trigger half of issue #58 PR #2 scope.

ItemTag now fires ItemTagNotifier on the complete AASM event via after_commit. Recipients are all shopkeepers in the shop's account, excluding the completer (completed_by). idle! does not fire.

event :complete do
  transitions from: [:idled], to: :completed
  after_commit :notify_completed
end

def notify_completed
  recipients = shop.account.shopkeepers
  recipients = recipients.where.not(id: completed_by_id) if completed_by_id.present?
  return if recipients.empty?
  ItemTagNotifier.with(record: self).deliver(recipients)
end

Out of scope

• APNs .p8 + FCM service account JSON: still need provisioning (bin/rails credentials:edit). ApplicationPushNotification.enabled defaults to !Rails.env.test?, so the trigger is a no-op in tests and a runtime no-op in dev/prod until credentials are configured.
• Free iOS/Android push registration: intentionally absent (per issue Add push notifications via noticed v2 (Action Push Native, paid-clients-only) #58 — the gate is "you have to write the client to use it"). Paid clients (PRs [render preview]test5 #3, Welcome to nativeapptemplateapi Discussions! #4 in the issue) are the next step.

Test plan

• 4 new test cases in item_tag_test.rb:
  • complete! fires notifier; recipient is the non-completer shopkeeper
  • complete! fires to all shopkeepers when completed_by is unset
  • complete! is a no-op when account has only the completer
  • idle! does not fire the notifier
• bin/rails test — 423 runs, 884 assertions, 0 failures
• bin/rubocop — 243 files, 0 offenses
• bin/brakeman — 0 warnings

🤖 Generated with Claude Code