Skip to content

Promote live SDK recovery and guarded review-media readiness to main - #3201

Merged
twoimo merged 23 commits into
mainfrom
data
Oct 13, 2026
Merged

twoimo merged 23 commits into
mainfrom
data

Conversation

@twoimo

@twoimo twoimo commented Oct 13, 2026

Copy link
Copy Markdown
Owner

Live Naver SDK loading errors previously switched the map into an offline test stub. This promotion preserves an explicit failure state and synchronizes one retry across public and administrator map consumers, including configured SDK URLs.

The promotion also carries the existing review-media successor and operating-admission tooling. Its launch policy remains held; hosted SQL, verification-photo transfer and production deployment require fresh source, rollback, writer-fence and rehearsal receipts.

Validation: exact develop HEAD CI passed, including the fresh GitGuardian rescan after two synthetic test incidents were classified. The SDK changes passed 59 related tests and four production-build recovery scenarios using actual remote Naver SDK/tiles with synthetic REST735/Auth401 data. These are desktop Chrome viewport scenarios; physical Galaxy and field gains are unverified. The separate swipe snapshot performance candidate is excluded.

Promotion source: data d608d9f, following develop 4ba9c6d through PR #3199. The remaining unpatched braces development-dependency advisory is retained; the full audit is not reported as clean.

twoimo added 17 commits October 12, 2026 19:59
Preserve review-media expiry and terminal outcomes
…0261013

Preserve Naver SDK errors and recover shared public/admin maps on retry
Bind the credential-free SQL fixture and preserve proof index history
Promote shared SDK recovery and held review-media tooling to data
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-13T21:18:46.535855Z 8fbf12a New commits
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@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: d608d9f999

ℹ️ 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".

Comment thread apps/web/scripts/review-media-sql-successor.mjs Outdated
Comment thread apps/web/scripts/review-media-sql-successor.mjs Outdated
Reconcile data promotion history without changing source bytes
Promote reviewed review-media execution and evidence repairs to data
@twoimo
twoimo merged commit 3d7919d into main Oct 13, 2026
29 checks passed
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