design_handoff の API ホストを signalarm.app に揃え、#56 の完了条件を確認する

[bannzai]

概要

#56 の完了条件のうち「api.alarmify.app の記述がリポジトリに残っていない」が、PR #57 の後に main へ入った design_handoff/ (#60) で再び満たされなくなっていたため、デザイン成果物の URL を signalarm.app に揃える。あわせて、#57 のマージ後に残っていた確認項目 (配信 workflow の成功・公開 URL の応答) を実体で確認し、その結果をここに記録する。

Closes #56

変更点

• design_handoff/canvas/build.mjs の curl 例 2 箇所を https://api.signalarm.app/v1/alarms に変更し、node design_handoff/canvas/build.mjs で *.dc.html を再生成した (変更前の状態で実行して差分が出ないことを確認済み。再生成で変わったのは URL を含む 6 アートボードだけ)
• 同じ 6 アートボード (OnboardingToken / Tokens / RecipeDetail の Dark・Light) の PNG を shoot-artboards.sh と同じ手順 (agent-browser、390×844 の 2x) で撮り直した。旧 PNG と見比べて、URL の文字列以外のレイアウトが変わっていないことを目視で確認した
• design_handoff/screens/recipes.md の API リファレンスのリンク先を https://bannzai.github.io/Alarmify/api から https://signalarm.app/api に変更した (アプリの RecipesView のリンク先と同じ)
• documents/cost.md の Cloudflare Pages の行を「移行予定」から「移行済み」にした

検証 (2026-09-15)

#56 の完了条件	確認結果
https://signalarm.app/ で LP が配信され、verify-lp.sh --app-store-support --has-account が pass	curl で HTTP 200。bash ~/.agents/skills/landing-page-builder/scripts/verify-lp.sh --app-store-support --has-account docs/index.html が OK
https://signalarm.app/api で API リファレンスと連携レシピが読める	/api が HTTP 200
curl -X POST https://api.signalarm.app/v1/alarms が alarmsApi に到達し未認証で 401	HTTP 401、本文 {"error":{"code":"unauthenticated",...}}
main へのマージで Pages が自動配信される workflow があり、1 回成功	pages-deploy.yml の push run 34693223301 (PR #57 のマージ)・34700497403・34747117074 で deploy ジョブが success (34693223301 と 34747117074 はステップ「Cloudflare Pages に配信」の success も確認)。api-proxy-deploy.yml の push run 34693223309 でも Worker の配信が success
api.alarmify.app の記述がリポジトリに残っていない	本 PR の変更後に git grep -n -e api.alarmify.app -e bannzai.github.io が 0 件

アプリ・Functions・docs/ のコードは変更していないため、make build-ios / make test / サイトのテストは実行していない。

issue の「やること」の残り (2026-09-16 に完了)

App Store Connect の URL 差し替えと旧 GitHub Pages の停止は、agent の実行が auto mode の判定 (Modify Shared Resources) で拒否されたため bannzai が実行し、結果を agent が実体で確認した。

• App Store Connect: バージョン 1.0 (未提出) の 35 ロケールのサポート URL とプライバシーポリシー URL を signalarm.app に差し替えた (70 件)。差し替え後に同じスクリプトを dry run し、変更 0 件・既に signalarm.app 70 件であることを確認した。リポジトリの fastlane/metadata の値に揃えているため、fastlane metadata_upload を実行しても同じ値になる
• 旧 GitHub Pages: gh api -X DELETE repos/bannzai/Alarmify/pages で停止した。gh api repos/bannzai/Alarmify/pages が 404、https://bannzai.github.io/Alarmify/ も 404。差し替え後の https://signalarm.app/ ・ https://signalarm.app/PrivacyPolicy-en ・ https://signalarm.app/PrivacyPolicy-ja はいずれも HTTP 200
• 実行順は GitHub Pages の停止が先になったが、アプリが未提出で旧 URL が公開されていないため影響はない
• 記録: ユーザー作業の一覧 (Secrets・Pub/Sub・初回デプロイ) #95 (comment)

本 PR のスコープ外

• 公開済みのデザインカンバス (design_handoff/README.md のカンバス URL) は再公開していない。リポジトリ内の生成元と PNG だけを更新した

Codex ローカルレビュー

• 対象: origin/main...f0b50ed (PR design_handoff の API ホストを signalarm.app に揃え、#56 の完了条件を確認する #100)
• モデル / effort: gpt-6-astra / low
• ラウンド: 1 (指摘 0 件)
• 修正した指摘: 0 件
• reject した指摘: 0 件

人間が確認

なし

セッション再開

cd /Users/bannzai/worktrees/bannzai/Alarmify/issue-56
claude --resume 8843a08c-bbed-4a71-9ca4-ad28ee95b572

概要

design_handoff/ 内の API URL と API リファレンス URL を signalarm.app 系へ更新しました。

変更内容

• API 例の送信先を https://api.signalarm.app/v1/alarms に更新
  • オンボーディング画面
  • トークン画面
  • レシピ画面
  • build.mjs の生成テンプレート
• API リファレンスのリンクを https://signalarm.app/api に更新
• documents/cost.md の Cloudflare Pages の状態を「移行済み」に更新

対象外

• App Store Connect の URL 更新
• 旧 GitHub Pages の停止
• 公開済みデザインキャンバスの再公開

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 3ce2621d-d135-49e1-89d4-c5fb7bf7fcf5

📥 Commits

Reviewing files that changed from the base of the PR and between e7bc237 and f0b50ed.

⛔ Files ignored due to path filters (6)

• design_handoff/png/OnboardingTokenDark.png is excluded by !**/*.png
• design_handoff/png/OnboardingTokenLight.png is excluded by !**/*.png
• design_handoff/png/RecipeDetailDark.png is excluded by !**/*.png
• design_handoff/png/RecipeDetailLight.png is excluded by !**/*.png
• design_handoff/png/TokensDark.png is excluded by !**/*.png
• design_handoff/png/TokensLight.png is excluded by !**/*.png

📒 Files selected for processing (9)

• design_handoff/canvas/OnboardingTokenDark.dc.html
• design_handoff/canvas/OnboardingTokenLight.dc.html
• design_handoff/canvas/RecipeDetailDark.dc.html
• design_handoff/canvas/RecipeDetailLight.dc.html
• design_handoff/canvas/TokensDark.dc.html
• design_handoff/canvas/TokensLight.dc.html
• design_handoff/canvas/build.mjs
• design_handoff/screens/recipes.md
• documents/cost.md

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

---

📝 Walkthrough

Walkthrough

Changes

SignalArm URL migration

Layer / File(s)	Summary
API example endpoints design_handoff/canvas/*, design_handoff/canvas/build.mjs	Updated displayed and generated API examples to use https://api.signalarm.app/v1/alarms.
Site reference and migration status design_handoff/screens/recipes.md, documents/cost.md	Updated the API reference link and marked the Cloudflare Pages migration as completed.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to f0b50

The SignalArm API examples and reference links are aligned with the published endpoint, with no concrete merge-blocking risk identified.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Linked Issues check	❓ Inconclusive	Issue #56 requires the LP at signalarm.app, API reference and recipes at /api, api.signalarm.app/v1/alarms routing to alarmsApi with HTTP 401 when unauthenticated, an automatic Pages deploymen…	Provide reviewable evidence for each unresolved #56 completion condition, including the LP verification result, /api publication, unauthenticated API response and alarmsApi routing, a successful Pages workflow run, and the repository-wi…

✅ Passed checks (4 passed)

Check name	Status	Explanation
Out of Scope Changes check	✅ Passed	The changed design_handoff/ API examples, API reference link, and Cloudflare Pages migration status support the domain migration and documentation objectives in issue #56. The supplied summary shows…
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
Title check	✅ Passed	タイトルは、design_handoff 内の API ホストを signalarm.app に統一する主要変更と、Issue #56 の完了条件確認を明確に示しています。
Description check	✅ Passed	説明にはテンプレートの「概要」「変更点」「人間が確認」セクションがあり、変更内容、検証結果、未達項目、対象外の作業を具体的に記載しています。

Full details: Linked Issues check

Explanation

Issue #56 requires the LP at signalarm.app, API reference and recipes at /api, api.signalarm.app/v1/alarms routing to alarmsApi with HTTP 401 when unauthenticated, an automatic Pages deployment workflow with one successful run, and removal of api.alarmify.app. The reviewed changes update API URLs in design_handoff/, update one API reference link, and mark Cloudflare Pages as migrated. The summary does not provide reviewable evidence for the LP verification, /api publication, API routing and response, workflow success, or repository-wide old-domain removal. Existing implementation at the reviewed head cannot be confirmed from the supplied evidence.

Resolution

Provide reviewable evidence for each unresolved #56 completion condition, including the LP verification result, /api publication, unauthenticated API response and alarmsApi routing, a successful Pages workflow run, and the repository-wide search result for api.alarmify.app.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch issue-56-finalize

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Old endpoints fade from view
SignalArm URLs now shine through
Examples point the proper way
Pages migration marks the day
Clear paths guide each alarm away}

_{Comment @coderabbitai help to get the list of available commands.}

[bannzai]

自動マージ判定 (auto-merge-pr skill による判定)

pr-triage-merge-recent-repos の毎時定期実行 (非対話) から、auto-merge-pr skill「判定の設計」に基づき審査し、条件充足と判断してマージする。

• 判定 Tier: Tier 2 (classify-changes.sh summary=needs_llm、attention_flags なし。変更は design_handoff のカンバス HTML / PNG / build.mjs とドキュメント (screens/recipes.md・documents/cost.md) のみで、アプリ・Functions・docs/ のランタイムコードを含まない)
• クリーンコンテキストのレビュー: Codex ローカルレビューマーカー head=f0b50ed8 が審査時 head f0b50ed8442108574d82a1498474a434acff3276 と一致、status=clean / scope=base:origin/main、PR author は gh 認証ユーザーと同一 (bannzai)
• 指摘の全解消: レビュースレッド 0 件、changes_requested なし
• CI: 実 CI (build-and-test / functions) すべて SUCCESS
• エビデンス: PR body「検証」に LP を signalarm.app (Cloudflare Pages) で配信し、API を api.signalarm.app に揃える #56 完了条件の実測記録 (curl の HTTP 200 / 401、verify-lp.sh OK、workflow run ID、git grep 0 件)。PNG は撮り直し後に旧版と目視比較済みと記載
• トレーサビリティ: Refs LP を signalarm.app (Cloudflare Pages) で配信し、API を api.signalarm.app に揃える #56 (意図的に Closes にしない理由が body に明記)、セッション再開コマンドあり
• 未チェックのチェックリスト: 0 件 (「人間が確認: なし」)。未達の完了条件 (ASC の URL 差し替え・旧 GitHub Pages の停止) はユーザー作業の一覧 issue ユーザー作業の一覧 (Secrets・Pub/Sub・初回デプロイ) #95 へ移管済みで、本 PR のマージはそれらに影響しない
• マージ方式: merge-pr.sh --match-head-commit f0b50ed8442108574d82a1498474a434acff3276 で審査時 head に固定。非対話定期実行のため head ブランチの削除は行わない

セッション再開

cd /Users/bannzai/.homesick/repos/castle
claude --resume 4a92f58e-ad7e-46c7-abd0-47fbc84f2f60