Skip to content

西武バスGTFSを常時有効化しENABLE_EXPERIMENTAL_BUS_FEATUREフラグを削除 - #1594

Merged
TinyKitten merged 3 commits into
devfrom
claude/seibu-bus-gtfs-flag-removal-almjie
Jul 5, 2026
Merged

西武バスGTFSを常時有効化しENABLE_EXPERIMENTAL_BUS_FEATUREフラグを削除#1594
TinyKitten merged 3 commits into
devfrom
claude/seibu-bus-gtfs-flag-removal-almjie

Conversation

@TinyKitten

@TinyKitten TinyKitten commented Jul 5, 2026

Copy link
Copy Markdown
Member

概要

西武バスのGTFSフィードを ENABLE_EXPERIMENTAL_BUS_FEATURE 環境変数なしで常時インポートするようにし、同フラグの記述をコード・設定・ドキュメントの全ファイルから削除しました。

変更の種類

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

変更内容

  • stationapi/src/import.rs: GtfsFeedexperimental フィールド、ENABLE_EXPERIMENTAL_BUS_FEATURE_ENV 定数、is_experimental_bus_feature_enabled() / gtfs_feeds_for_experimental_enabled() / enabled_gtfs_feeds() を削除し、取り込み処理が GTFS_FEEDS(都営バス・西武バス)を直接参照するように変更。テストを test_gtfs_feeds(toei と seibu の両方が含まれることを検証)に置き換え
  • .env: ENABLE_EXPERIMENTAL_BUS_FEATURE=false の行を削除
  • AGENTS.md / docs/architecture.md: フラグの説明を削除し、全GTFSフィードが常にインポートされる旨に更新
  • README.md: 西武バスのデータソース表記から「(experimental feed)」を削除

レビュー時の注意: 西武バスフィードは ODPT の consumer key を要求するため、本変更以降は ODPT_ACCESS_TOKEN 未設定の環境ではGTFSインポートがエラーになります(従来はフラグ無効時にスキップ)。バス機能全体を止める DISABLE_BUS_FEATURE は従来どおり利用できます。

テスト

  • cargo fmt --all -- --check が通ること
  • cargo clippy -- -D warnings が通ること
  • cargo test(SQLX_OFFLINE=true)が通ること(396件成功)

関連Issue

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


Generated by Claude Code

Summary by CodeRabbit

  • 変更
    • バスデータの取り込み対象が、設定済みの全フィードに統一されました。
    • 西武バスを含むバスデータの案内表記を更新し、「実験的」扱いを हटめました。
    • バス関連の説明文書を最新の動作に合わせて整理しました。

- GtfsFeedのexperimentalフィールドと関連の環境変数判定を削除し、
  設定済みの全GTFSフィード(都営バス・西武バス)を常にインポートするように変更
- .env / AGENTS.md / docs/architecture.md / README.md から
  ENABLE_EXPERIMENTAL_BUS_FEATUREおよびexperimental表記を削除

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FVJjKGTcRodoo6TEeiy6he
@coderabbitai

coderabbitai Bot commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

実験的バスフィード機能フラグ(ENABLE_EXPERIMENTAL_BUS_FEATURE)に関するロジックがimport.rsから削除され、GTFSフィードは常にToei BusとSeibu Busを含む全フィードを取り込むようになった。関連する.env、AGENTS.md、README.md、docs/architecture.mdの記述もこれに合わせて更新された。

Changes

実験フラグ撤去とフィード常時有効化

Layer / File(s) Summary
実験フラグ分岐の削除とフィード有効化ロジックの簡素化
stationapi/src/import.rs
GtfsFeedexperimentalフィールド、ENABLE_EXPERIMENTAL_BUS_FEATURE_ENV定数、is_experimental_bus_feature_enabled相当の判定関数を削除し、enabled_gtfs_feeds()が常にGTFS_FEEDS(toei/seibu)を返すよう変更、対応するテストも更新。
ドキュメント・設定の記述更新
.env, AGENTS.md, README.md, docs/architecture.md
ENABLE_EXPERIMENTAL_BUS_FEATURE=falseの環境変数定義を削除し、各ドキュメントの記述を「既定はToeiのみ、フラグ有効時に全フィード」から「起動時に設定済み全フィード(Seibu含む)を常時取り込み」へ更新。

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

  • TrainLCD/StationAPI#1530: 同一の実験系フィード分岐(gtfs_feeds_for_experimental_enabled/is_experimental_bus_feature_enabled等)を追加していたPRで、本PRはその機構を削除する逆方向の変更。

Suggested labels: feature

Poem

ぴょんぴょん跳ねて フラグ消えた
セイブもトエイも みんな一緒に
🐰 GTFSの道 まとめて走る
フラグの分岐 もういらない
ぱたり眠る 実験フィード、おやすみ〜

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed PRの主変更点である西武バスGTFSの常時有効化と実験フラグ削除を簡潔に表しています。
Description check ✅ Passed テンプレートの主要セクション(概要・変更の種類・変更内容・テスト)を満たし、関連Issueと任意項目も含まれています。
✨ 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 claude/seibu-bus-gtfs-flag-removal-almjie

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

@TinyKitten TinyKitten self-assigned this Jul 5, 2026

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@stationapi/src/import.rs`:
- Around line 317-319: `enabled_gtfs_feeds()` は実質 `GTFS_FEEDS.to_vec()`
の単なるラッパーで、名前にある「enable」判定を行っていません。`enabled_gtfs_feeds` の意図が分かるように、呼び出し側で直接
`GTFS_FEEDS.to_vec()` を使うか、この関数を将来の拡張用エイリアスであることが分かるように明確なコメントを追加して、`GtfsFeed`
の返却意図を誤解しない形にしてください。
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: fb64d443-fbb5-4f6d-b220-d1675e7af9d9

📥 Commits

Reviewing files that changed from the base of the PR and between 1ccbfae and a43959c.

📒 Files selected for processing (5)
  • .env
  • AGENTS.md
  • README.md
  • docs/architecture.md
  • stationapi/src/import.rs
💤 Files with no reviewable changes (1)
  • .env

Comment thread stationapi/src/import.rs Outdated
フィルタリングを行わない薄いラッパーになっていたため、
呼び出し側で GTFS_FEEDS.to_vec() を直接使用する形に変更

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FVJjKGTcRodoo6TEeiy6he
@TinyKitten
TinyKitten merged commit cd5ec16 into dev Jul 5, 2026
11 checks passed
@TinyKitten
TinyKitten deleted the claude/seibu-bus-gtfs-flag-removal-almjie branch July 5, 2026 11:29
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.

2 participants