feat: align CLI/UI workflow and reusable analysis payloads#11
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3b2ea2ebd5
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| # 3) 테스트 | ||
| ollama run <bitnet-model-tag> "샘플 매출 데이터를 요약해줘" | ||
| # 3) CSV 분석 payload 생성 | ||
| bitnet-analyze analyze sample.csv --question "샘플 매출 데이터를 요약해줘" --out payload.json |
There was a problem hiding this comment.
Add sample CSV creation before analyze command
The new quick-start command block now calls bitnet-analyze analyze sample.csv ... without creating or pointing to an existing sample.csv, so users following section 6 verbatim will hit FileNotFoundError instead of completing the advertised “minimum commands” flow. This regresses the onboarding path introduced by this commit; please add a prior step that creates a sample CSV (or reference a concrete existing file path).
Useful? React with 👍 / 👎.
Motivation
bitnet-analyze.bitnet_tools.Description
analyzeanduisubcommands inbitnet_tools.cli, provide_build_parserandmain(argv: ...), and auto-route legacy bare-CSV invocations into theanalyzesubcommand.build_prompt(summary, question)and addbuild_analysis_payload_from_csv_texttobitnet_tools.analysisto support inline CSV analysis for the UI.bitnet_tools.__init__to export the actual public functions (build_analysis_payload,build_analysis_payload_from_csv_text,build_prompt,summarize_rows).bitnet-analyze analyze ...andbitnet-analyze ui ...commands and add tests intests/test_cli.pywhile extendingtests/test_analysis.py.Testing
pytest -qinitially failed during collection because the package was not importable in the environment without settingPYTHONPATHandpip install -e .failed due to blocked external index access when resolving build dependencies.PYTHONPATH=. pytest -qand all tests passed (5 passed).PYTHONPATH=. python -m bitnet_tools.cli --help, which succeeded.Codex Task