Skip to content

canary - #6988

Merged
TinyKitten merged 2 commits into
canaryfrom
dev
Sep 15, 2026
Merged

canary#6988
TinyKitten merged 2 commits into
canaryfrom
dev

Conversation

@TinyKitten

Copy link
Copy Markdown
Member

概要

dev の内容を canary へ反映するリリース PR。含まれる変更は #6987 の 1 件。

変更の種類

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

変更内容

平滑化判定の診断情報 (#6987)

#6983 で入れた DevOverlay の診断情報コピーが、地下鉄分岐(skipSmoothing)に入っているかを説明できない状態だった。持ち出していた accuracyHistory は DevOverlay のチャート用(1 秒ごとにサンプリングし、無効値は NaN を積む)で、平滑化の要否を決めている判定用の accuracyHistoryAtom(測位を受理するたびに積み、無効値は捨てる)とは別物だったため、どちらの経路を通ったのかが JSON から追えなかった。

スナップショットに filter を追加し、判定用の精度履歴・判定結果・判定に使った路線種別を出すようにした。

"filter": {
  "skipSmoothing": true,
  "lineType": "Subway",
  "accuracyHistory": [ ... ]
}

setLocation が判定した結果と入力を smoothingDecisionAtom へ組で書き込み、DevOverlay はその組をそのまま出す。同じ条件を DevOverlay 側で組み直すと、判定と表示が別々に育って食い違う。判定内容が前回と変わらないときは書き込まないため、購読側の再レンダーは判定が変化した瞬間だけに限られる。

測位の判定そのもの(isAccuracyStableskipSmoothing の条件)は変更していないので、位置の挙動は変わらない。DevOverlay は src/screens/Main.tsxisDevApp && devOverlayEnabled のときだけマウントされるため、本番ビルドの画面には出ない。

ネイティブ依存の増減はなし(expo-clipboard#6983 で追加済みで、その canary ビルドは #6985 で通っている)。

テスト

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

origin/dev (799d347) をローカルで検証した結果:

  • npm run lintbiome check ./src、Checked 767 files, No fixes applied
  • npm run typechecktsc --noEmit、エラーなし
  • npm test → 287 suites / 3,142 tests すべて pass

検証は #6987 の head (67f7a59) で実行したもので、git diff --stat 67f7a59 origin/dev が空であることを確認済み(ツリーが一致するため origin/dev の内容に対する結果として扱える)。#6987 の CI(quality / typecheck / test)も同コミットで success。

canary で確認したいこと

関連Issue

参考: #6981(この診断が必要になった地下鉄ワープの調査。Draft のまま実走行データ待ち)

スクリーンショット(任意)

UI 変更なし: src/components/DevOverlay.tsx の差分は購読先の atom の差し替えと、クリップボードへ載せる JSON への値の追加だけで、画面に描画される要素は変わりません(stationAtom の購読はむしろ 1 つ減っています)。製品画面の変更もありません。#6987 に記載した理由と同じです。

🤖 Generated with Claude Code

https://claude.ai/code/session_01Sfi8S4Yvob2sEzib4VUUBs


Generated by Claude Code

@coderabbitai

coderabbitai Bot commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Essentials

Run ID: 97930d2a-89f8-45f6-a7d8-8f5fa4c348e2

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.

@TinyKitten
TinyKitten merged commit 63cb326 into canary Sep 15, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant