[5ym]

Infisical(#535)・yosegaki(danything/blog#226)・todoroku(doa/todoroku#430)を smtp-relay に切り替え、Infisical の Pod から info@doany.io 宛のテストが Graph で送れた(Email sent successfully!)。ROADMAP から項目を外し、理由を docs/decisions.md に残す。pulumi/dns.ts の SPF のコメントも直す(コメントだけ)。

ついでに ROADMAP の PT3 の項に、siderolabs/talos#14724 を実機で確かめた結果を足す(別件)。

🤖 Generated with Claude Code

https://claude.ai/code/session_01NN4TmSXHrMP8dkTbtkjfz1

[claude]

PR #536 was updated after the trigger (c3ecb3d -> eafb468). Push again or mention Claude to run on the latest version.

[claude]

Claude finished @5ym's task in 14s —— View job

---

レビュー結果

docs のみの変更で、大きな問題はありません。apps/smtp-relay/(network-policy.yaml を含む)は実在し、decisions.md の記述と合っています。

軽微な指摘(任意)

• pulumi/dns.ts:39 のコメントに「アプリも smtp.office365.com 経由」とありますが、今回の変更で古くなりました。SPF の -all 自体は、Graph も Exchange から送るので問題ありません。コメントだけ「中継 → Graph 経由」に直すと、あとで読む人が迷いません。このファイルは今回の差分に含まれていないので、別の PR でも構いません。
• ROADMAP.md の PT3 の項は SMTP とは別件で、modprobe helper を実機で確かめた内容です。PR タイトルと本文には出てこないので、混ぜたことを本文に一言あると追いやすくなります。内容自体は問題ありません。
• 要確認: decisions.md の「Kiosk は SMTP の OAuth に対応していないとされる」は、根拠の URL がありません。決定の理由になっているので、Microsoft のドキュメントへのリンクを足すと後から検証できます。

テストの対象になるコードはありません。

[claude]

Claude finished @5ym's task in 12s —— View job

---

レビュー結果

docs とコメントだけの変更で、問題はありません。

• apps/smtp-relay/ と network-policy.yaml は実在します。ingress は infisical・blog・todoroku の 3 namespace に絞られていて、decisions.md の「届く namespace は NetworkPolicy で絞る」と合っています。
• 前回指摘した pulumi/dns.ts:39 の SPF コメントは、今回の更新で直っています。
• クラスタ内に smtp.office365.com を指す設定は残っていません。

軽微な指摘(任意、要確認)

• docs/decisions.md の「Kiosk は SMTP の OAuth に対応していないとされる」には根拠の URL がありません。決定の理由なので、Microsoft のドキュメントへのリンクを足すと後から確かめられます。この点は私が調べておらず、事実かどうかは未確認です。
• ROADMAP.md の PT3(modprobe helper)の項は SMTP とは別件です。PR 本文に一言あると追いやすくなります。

テスト対象のコードはありません。

[github-actions]

🍹 preview on doany-cloudflare/prod

Pulumi report

   Previewing update (prod):

@ previewing update.....
~  azuread:index:Application smtp-relay update [diff: ~owners]
~  azuread:index:ServicePrincipal smtp-relay update [diff: ~owners]
   pulumi:pulumi:Stack doany-cloudflare-prod  
Resources:
   ~ 2 to update
   57 unchanged