Skip to content

[feat] デプロイ時にコミットSHAを焼き込み /healthz で識別できるようにする - #9

Merged
haruto-kamijo merged 1 commit into
developfrom
feature/healthz-git-sha
Sep 3, 2026
Merged

[feat] デプロイ時にコミットSHAを焼き込み /healthz で識別できるようにする#9
haruto-kamijo merged 1 commit into
developfrom
feature/healthz-git-sha

Conversation

@haruto-kamijo

Copy link
Copy Markdown
Collaborator

/healthz は version に GIT_SHA を返すが、これを設定する仕組みが無く、
本番が常に {"ok":true,"version":"dev"} を返していた。障害対応中に
どのコミットが動いているか判別できない状態だった。

deploy スクリプトを scripts/deploy.mjs 経由に変え、wrangler deploy に --var GIT_SHA: を渡す。未コミットの変更があるときは -dirty を 付け、稼働中のコードがそのコミットと同一でないことを示す。

package.json に $(git rev-parse --short HEAD) を直接書かないのは、pnpm が Windows では cmd.exe でスクリプトを実行し、コマンド置換が働かないため。

README のデプロイ方法が「main への push で自動デプロイ」となっていたが、
実際はローカルからの手動デプロイなので実態に合わせ、ヘルスチェックの
エンドポイント一覧と監視先の注意点もあわせて記載する。

What & why

How this was tested

  • pnpm lint
  • pnpm typecheck
  • pnpm build
  • Manually verified in the browser / via wrangler dev

Screenshots (if UI change)

/healthz は version に GIT_SHA を返すが、これを設定する仕組みが無く、
本番が常に {"ok":true,"version":"dev"} を返していた。障害対応中に
どのコミットが動いているか判別できない状態だった。

deploy スクリプトを scripts/deploy.mjs 経由に変え、wrangler deploy に
--var GIT_SHA:<short-sha> を渡す。未コミットの変更があるときは -dirty を
付け、稼働中のコードがそのコミットと同一でないことを示す。

package.json に $(git rev-parse --short HEAD) を直接書かないのは、pnpm が
Windows では cmd.exe でスクリプトを実行し、コマンド置換が働かないため。

README のデプロイ方法が「main への push で自動デプロイ」となっていたが、
実際はローカルからの手動デプロイなので実態に合わせ、ヘルスチェックの
エンドポイント一覧と監視先の注意点もあわせて記載する。

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@Aquarius0715

Copy link
Copy Markdown

ありがとうございます!

これ、healthzとreadyzはどちらもボットに呼ばせる感じですか?

@Aquarius0715 Aquarius0715 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.

LGTM!

@haruto-kamijo
haruto-kamijo merged commit 5a2d3fa into develop Sep 3, 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.

2 participants