Skip to content

dev<-master - #6968

Merged
TinyKitten merged 4 commits into
devfrom
chore/dev-from-master
Sep 13, 2026
Merged

dev<-master#6968
TinyKitten merged 4 commits into
devfrom
chore/dev-from-master

Conversation

@claude

@claude claude Bot commented Sep 13, 2026

Copy link
Copy Markdown

概要

本番リリース v10.15.1 後に master ブランチへ積まれた差分を dev ブランチへ同期するマージPRです。master と dev の履歴を揃えることが目的で、コードの新規変更は含みません。

変更の種類

  • バグ修正
  • 新機能
  • リファクタリング
  • ドキュメント
  • CI/CD
  • その他

変更内容

master に積まれていて dev に未反映の以下 581 件のコミットを dev に取り込みます。

コミット件名が多いため、先頭 50 件のみ記載し残り 531 件は省略しています。

版数ファイルのコンフリクト解決

dev 側の canary bump と master 側のリリース版数がねじれていたため、chore/dev-from-master に origin/dev をマージして解決コミットを 1 つだけ積んでいます（アプリコードの衝突は 0 件）。採用値は sync-dev-from-master の方針「semver はリリース版、ビルド番号は最大値」に従っています。

項目 master dev 採用
semver（version / versionName / MARKETING_VERSION） 10.15.1 10.15.0 10.15.1
:app versionCode 100000755 100000757 100000757
:wearable versionCode 100000756 100000758 100000758
buildNumber / CURRENT_PROJECT_VERSION 2907 2908 2908
app.config.ts versionCode 100000755 100000757 100000757

android/wearable/build.gradle.kts は :wearable の versionCode を必ず :app + 1 に保つ必要があります（逆転すると Wear バンドルが配信されずリリースを公開できない）。100000757 + 1 = 100000758 となり、最大値ルールと当該不変条件は同じ値を指します。

解決後の dev との差分は semver のみで、ビルド番号は dev 側の値を維持しています。

テスト

  • npm run lint が通ること
  • npm test が通ること
  • npm run typecheck が通ること

本PRはアプリコードの変更を含まないマージPRのため、当ブランチ上で lint / test / typecheck は実行していません。取り込まれる各コミットは元の PR 時点および master への取り込み時点で検証済みです。解決コミットが触れているのは版数ファイルの数値のみです。

関連Issue

なし（リリース後の同期PR）。

スクリーンショット（任意）

UI変更なし: 版数ファイルの数値以外に dev との差分が無いマージPRのため。

Generated by Claude Code

@claude
claude Bot requested a review from TinyKitten as a code owner September 13, 2026 14:25
@coderabbitai

coderabbitai Bot commented Sep 13, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: bdb3117e-dcec-48a9-8a5b-277fb6eb97db

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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

@claude

claude Bot commented Sep 13, 2026

Copy link
Copy Markdown
Author

Warning

このPRは必ず「Create a merge commit」でマージしてください。Squash / Rebase は禁止です。
Squash すると master 側の merge commit 構造が潰れ、dev と master の履歴分岐が壊れます（PR #5838 で同じミスが起き #5840 で再掲した実績あり）。

現在マージできない理由

finalize-release から自動マージを試みましたが、実行環境の制約で停止しました。

relax http=403  {"message":"Write access to this GitHub API path is not permitted through this proxy."}
merge http=405  Merge commits are not allowed on this repository.

dev の Ruleset (#9386058) が allowed_merge_methods: ["squash"] に絞られており、本来はこれを一時緩和してから merge commit でマージしますが、当実行環境では Ruleset への書き込みがプロキシに拒否されるため緩和できませんでした。squash での回避は上記の理由により行っていません。

Ruleset は変更されていません（現在値 ["squash"] のまま。緩和 PUT 自体が 403 で弾かれたため、実際の変更は発生していません）。

手当ての方法

CI は全て green（quality / test / typecheck / scripts / triage、bump-version は skipped）、mergeable_state は clean です。gh 認証済みの端末から、いずれかで仕上げてください。

  1. finalize-release スキルの手順 6-3 をローカル実行（RS_ID=9386058 / PR 6968）
  2. 手動: Settings → Rules → dev ruleset で一時的に Allow merge commits を有効化 → 「Create a merge commit」 でマージ → squash-only に戻す

マージ後の検証:

git fetch origin dev master
git rev-list --parents -n 1 origin/dev   # 3 つの SHA（= 2 親の merge commit）になること
git rev-list --count origin/dev..7ccd30d0d9b127f38200e239700782c6aec6d9c5   # 0 になること

あわせて chore/dev-from-master の削除もお願いします（--delete-branch 相当が未実行です）。

Generated by Claude Code

@TinyKitten
TinyKitten merged commit d3cf1ed into dev Sep 13, 2026
9 checks passed
@TinyKitten
TinyKitten deleted the chore/dev-from-master branch September 13, 2026 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@TinyKitten TinyKitten Awaiting requested review from TinyKitten TinyKitten is a code owner

Assignees

@TinyKitten TinyKitten

Labels

android ios

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@TinyKitten @claude