Skip to content

Introduce local store auth summary API#7709

Draft
dmerand wants to merge 1 commit into
donald/store-list-business-platformfrom
donald/store-list-local-auth-seam
Draft

Introduce local store auth summary API#7709
dmerand wants to merge 1 commit into
donald/store-list-business-platformfrom
donald/store-list-local-auth-seam

Conversation

@dmerand
Copy link
Copy Markdown
Contributor

@dmerand dmerand commented Jun 3, 2026

Summary

Introduce a local auth summary API without changing shopify store list behavior yet.

Scope

  • add StoredStoreAuthSummary
  • add listStoredStoreAuthSummaries() under services/store/auth
  • keep token-bearing session buckets as the current persistence source of truth
  • keep conf traversal and bucket cleanup in session-store.ts
  • re-export the summary API from services/store/auth/index.ts
  • fix the stale store list command test assertion after the BP-only trim

Why this split exists

The next PR needs one narrow question answered first:

  • what locally stored store auth exists on this machine?

This PR answers that without taking on product behavior:

  • no --from store-auth
  • no organization→store-auth fallback
  • no persistence migration

That keeps the behavior PR focused on command policy and output, not on storage traversal.

Boundary this PR is trying to establish

  • persistence details stay in session-store.ts
  • callers see StoredStoreAuthSummary[]
  • callers do not need token-bearing session buckets or conf layout knowledge

@github-actions github-actions Bot added the no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users. label Jun 3, 2026
@dmerand dmerand changed the title Introduce local store auth summary seam Introduce local store auth summary API Jun 4, 2026
@dmerand dmerand force-pushed the donald/store-list-business-platform branch from 92fffc1 to c7e152d Compare June 5, 2026 13:41
@dmerand dmerand force-pushed the donald/store-list-local-auth-seam branch 2 times, most recently from 619636c to 7c049c1 Compare June 5, 2026 14:10
@dmerand dmerand force-pushed the donald/store-list-business-platform branch from 3ac3560 to 6fbe857 Compare June 5, 2026 14:24
@dmerand dmerand force-pushed the donald/store-list-local-auth-seam branch from 7c049c1 to 52daf1c Compare June 5, 2026 14:24
Copy link
Copy Markdown
Contributor Author

dmerand commented Jun 5, 2026

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@dmerand dmerand force-pushed the donald/store-list-business-platform branch from 6fbe857 to 87919a9 Compare June 5, 2026 18:12
@dmerand dmerand force-pushed the donald/store-list-local-auth-seam branch 2 times, most recently from 00dce2a to 22e4f4a Compare June 5, 2026 18:31
@dmerand dmerand force-pushed the donald/store-list-business-platform branch from 87919a9 to f516bbe Compare June 5, 2026 18:31
Adds a read-only API that projects the current session for every store with
locally stored `store auth`, for use by later store list sources.
@dmerand dmerand force-pushed the donald/store-list-business-platform branch from f516bbe to 79dbfa7 Compare June 5, 2026 19:06
@dmerand dmerand force-pushed the donald/store-list-local-auth-seam branch from 22e4f4a to 31a07f1 Compare June 5, 2026 19:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog This PR doesn't include a changeset entry. Is an internal only change not relevant to end users.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant