Record t1.22.1 task evidence

This commit is contained in:
ben
2026-03-24 17:26:00 -04:00
parent 867275c67a
commit eb3959ae0f

View File

@@ -1106,6 +1106,26 @@ remove transitional detritus from the repo and make the final folder/script layo
** notes
- Final layout decision: `catalog.csv` now lives under `data/review/`, while `purchases.csv` and the chart-ready analysis outputs live under the step-4 `data/analysis/` folder.
- Removed obsolete top-level pipeline files and their dead tests so the active entrypoints are now the collect, normalize, review/combine, status, and analysis scripts only.
* [X] t1.22.1: remove unneeded python deps
** acceptance criteria
1. update requirements.txt to add/remove necessary python libs
2. keep only direct runtime deps in requirements.txt; transitive deps should not be pinned unless imported directly
** evidence
- commit: `867275c` `Trim requirements to direct runtime deps`
- tests: `./venv/bin/python -m unittest discover -s tests`; `./venv/bin/python collect_giant_web.py --help`; `./venv/bin/python collect_costco_web.py --help`; `./venv/bin/python normalize_giant_web.py --help`; `./venv/bin/python normalize_costco_web.py --help`; `./venv/bin/python build_purchases.py --help`; `./venv/bin/python review_products.py --help`; `./venv/bin/python report_pipeline_status.py --help`; `./venv/bin/python analyze_purchases.py --help`
- date: 2026-03-24 17:25:39 EDT
** notes
- `requirements.txt` now keeps only direct runtime deps imported by the active pipeline: `browser-cookie3`, `click`, `curl_cffi`, and `python-dotenv`.
- Low-level support packages such as `cffi`, `jeepney`, `lz4`, `pycryptodomex`, and `certifi` are left to transitive installation instead of being pinned directly.
#+END_
*
* [ ] t1.10: add optional llm-assisted suggestion workflow for unresolved normalized retailer items (2-4 commits)