Skip to content

chore(store): screenshot pipeline + captures from the new design - #14

Merged
saagpatel merged 1 commit into
mainfrom
chore/store-screenshots
Aug 11, 2026
Merged

chore(store): screenshot pipeline + captures from the new design#14
saagpatel merged 1 commit into
mainfrom
chore/store-screenshots

Conversation

@saagpatel

Copy link
Copy Markdown
Owner

One-command screenshot pipeline (bash scripts/capture-screenshots.sh): builds, installs on the iPhone 17 Pro Max simulator, and captures the three demo scenes to screenshots/iphone-69/ at native 1320×2868 — the current 6.9" App Store size.

APPSTORE-METADATA.md's screenshot section is reconciled to the app that shipped: the old plan referenced views that were never built (MatchResultView, a gallery grid) and outdated device sizes. The submission checklist now flags explicitly that the comparison capture uses the synthetic debug fixture and must be recaptured with a real match on device before upload; the city selector and camera captures are shippable as-is.

Verification: pipeline run end-to-end; all three PNGs confirmed at 1320×2868 and visually inspected (design renders correctly at 6.9"). No app code changed.

scripts/capture-screenshots.sh builds, installs, and captures the
three demo scenes on the iPhone 17 Pro Max simulator at native
1320x2868. APPSTORE-METADATA.md screenshot plan reconciled to the
screens that actually shipped (the old plan referenced unbuilt
MatchResult/Gallery views and outdated sizes); submission checklist
now flags that the comparison capture must be redone with a real
match on device before upload.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 38aa5ff038

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".


capture "01-comparison" --afterimage-demo-comparison
capture "02-city-selector" --afterimage-demo-cities
capture "03-camera-enable"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Reset camera permission before capturing the enable state

When this simulator has previously granted or denied camera access for com.afterimage.app, this launch renders the live-preview or denied branch instead of the advertised “Tap to Enable Camera” screen because the script never resets privacy state. xcrun simctl help privacy documents the privacy <device> <action> <service> [<bundle identifier>] command and its reset action; reset camera access for this bundle (or erase/uninstall it) before this capture so repeated runs remain deterministic.

Useful? React with 👍 / 👎.

@saagpatel
saagpatel merged commit 488e271 into main Aug 11, 2026
0 of 2 checks passed
@saagpatel
saagpatel deleted the chore/store-screenshots branch August 11, 2026 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant