Remove Places API and migrate to new CSV - #1
Merged
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
Google Places API 依存を完全に削除し、Google マップ保存リスト由来の data/places.csv のみをソースとして前処理→ベクトルDB構築→検索/生成を行う構成へ移行する PR です。uv によるプロジェクトローカル環境と pytest を追加し、CSVベース運用の手順をドキュメント化しています。
Changes:
- Places API の前処理/実行時参照を削除し、検索コンテキストは CSV/Chroma のみで構築
- 新 CSV 形式(タイトル/メモ/URL/タグ/コメント/緯度/経度/住所)を正規化して
places_data.jsonを生成・DB化 - uv セットアップ、
.env.example、pytest を追加し、移行の回帰テストを追加
Reviewed changes
Copilot reviewed 15 out of 18 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| 操作説明書.md | uv 前提の実行手順と、CSV/ローカルDBのみで動作する旨へ更新 |
| README.md | セットアップ・実行・生成物の説明をCSV運用に合わせて刷新 |
| pyproject.toml | uv/パッケージング/依存関係/pytest を追加 |
| .python-version | Python 3.12 を固定 |
| .gitignore | .venv/.env/pytest cache/egg-info を追加し生成物を除外 |
| .env.example | GROQ_API_KEY の雛形を追加 |
| fetch_places_data.py | Places API を廃止し、新CSV→JSON変換・正規化ロジックへ変更 |
| make_vector_db.py | CSV由来フィールドのみで Document/metadata を構築するよう変更 |
| nodes/search_node.py | live lookup を削除し、Chroma 検索結果のみで context を構築 |
| nodes/generate_node.py | 営業時間に関する生成ルールを削除(根拠データが無いため) |
| nodes/self_eval_node.py | 営業時間整合性の評価観点を削除(根拠データが無いため) |
| places_client.py | Places API クライアントを削除 |
| add_address_to_csv.py | 住所付与用の Places API スクリプトを削除 |
| tests/test_fetch_places_data.py | 新CSVマッピング・座標バリデーション・実CSVの整合をテスト追加 |
| tests/test_make_vector_db.py | ページコンテンツ/メタデータがCSV由来のみであることをテスト追加 |
| tests/test_search_node.py | 検索ノードが live lookup なしで context を構築することをテスト追加 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
data/places.csv.Changes
Validation
uv lock --checkuv sync --locked --offlineuv run pytest -q(6 passed)Interactive Groq plan generation was not run because no API key was available.