Skip to content

Add update access setup flows - #11

Merged
Pluglug merged 4 commits into
mainfrom
website/update-access-setup-flows
Aug 25, 2026
Merged

Add update access setup flows#11
Pluglug merged 4 commits into
mainfrom
website/update-access-setup-flows

Conversation

@Pluglug

@Pluglug Pluglug commented Aug 24, 2026

Copy link
Copy Markdown
Member

概要 setup専用クライアントへ、既存の初回claimに加えて次の操作を追加します。 - 1年分のUpdate Extension適用 - server-side update accessの確認 - Recovery secretによるRepository accessの置換 初回offerは購入ページ側の固定linkから公開offer codeを渡し、購入者が期間やpolicyを入力するUIは設けません。 ## Security / privacy境界 - purchase key、Recovery secret、Repository tokenをbrowser storageやURLへ保存しない - claim/renewal/reissueの再試行IDだけを8日間保持する - statusはread-onlyで再試行IDを保持しない - reissue後のone-time credential deliveryと明示acknowledgementは既存機構を再利用する - CSP、SRI、exact-origin、no-store、noindexを維持する - service responseはbounded parserを通し、statusとupdates-through以外の内部情報を表示しない ## 検証 - npm test (19 tests) - node --check dist/setup/app.mjs - git diff --check - local browserで初期画面、4操作selector、responsive layoutを目視確認 ## Landing note このrepositoryはmain mergeでCloudflare Pagesへ自動deployされます。service側C5Aがproductionへcutoverされる前にこのPRをmergeしません。Security Review CLEANとrequired checksを取得した後、C6のcutover順序に従って待機させます。

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Aug 24, 2026

Copy link
Copy Markdown

Deploying pie-menu-editor-website with  Cloudflare Pages  Cloudflare Pages

Latest commit: 55d1f9f
Status: ✅  Deploy successful!
Preview URL: https://4bf55fce.pie-menu-editor-website.pages.dev
Branch Preview URL: https://website-update-access-setup.pie-menu-editor-website.pages.dev

View logs

@Pluglug

Pluglug commented Aug 24, 2026

Copy link
Copy Markdown
Member Author

@codex security review

@Pluglug

Pluglug commented Aug 24, 2026

Copy link
Copy Markdown
Member Author

@codex security review

@Pluglug

Pluglug commented Aug 24, 2026

Copy link
Copy Markdown
Member Author

@codex security review

@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: 61c6b53dd2

ℹ️ 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 credential-setup/app.source.js.txt
@Pluglug

Pluglug commented Aug 24, 2026

Copy link
Copy Markdown
Member Author

@codex security review

@chatgpt-codex-connector

Copy link
Copy Markdown

Security review completed. No security issues were found in this pull request.

Reviewed commit: 55d1f9f59e

View security finding report

Only the user who started this review can view the report in Codex.

ℹ️ About Codex security reviews in GitHub

This is an experimental Codex feature. Security reviews are triggered when:

  • You comment "@codex security review"
  • A regular code review gets triggered (for example, "@codex review" or when a PR is opened), and you’re opted in so security review runs alongside code review

Once complete, Codex will leave suggestions, or a comment if no findings are found.

@Pluglug
Pluglug merged commit f3431cd into main Aug 25, 2026
2 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