Skip to content

feat: align CLI/UI workflow and reusable analysis payloads#11

Merged
rad1092 merged 1 commit into
mainfrom
codex/provide-step-by-step-execution-guide
Feb 13, 2026
Merged

feat: align CLI/UI workflow and reusable analysis payloads#11
rad1092 merged 1 commit into
mainfrom
codex/provide-step-by-step-execution-guide

Conversation

@rad1092
Copy link
Copy Markdown
Owner

@rad1092 rad1092 commented Feb 13, 2026

Motivation

  • Align the CLI behavior with the beginner README so newcomers can follow the guide end-to-end using bitnet-analyze.
  • Make prompt and payload generation reusable between CLI and the web UI to avoid duplication and support inline CSV from the UI.
  • Fix package exports to prevent import-time errors when tests or consumers import bitnet_tools.

Description

  • Add analyze and ui subcommands in bitnet_tools.cli, provide _build_parser and main(argv: ...), and auto-route legacy bare-CSV invocations into the analyze subcommand.
  • Factor prompt construction into build_prompt(summary, question) and add build_analysis_payload_from_csv_text to bitnet_tools.analysis to support inline CSV analysis for the UI.
  • Update bitnet_tools.__init__ to export the actual public functions (build_analysis_payload, build_analysis_payload_from_csv_text, build_prompt, summarize_rows).
  • Update the README quick-run block to include project install, bitnet-analyze analyze ... and bitnet-analyze ui ... commands and add tests in tests/test_cli.py while extending tests/test_analysis.py.

Testing

  • pytest -q initially failed during collection because the package was not importable in the environment without setting PYTHONPATH and pip install -e . failed due to blocked external index access when resolving build dependencies.
  • Ran PYTHONPATH=. pytest -q and all tests passed (5 passed).
  • Verified the CLI help with PYTHONPATH=. python -m bitnet_tools.cli --help, which succeeded.

Codex Task

@rad1092 rad1092 merged commit 3c89641 into main Feb 13, 2026
0 of 4 checks passed
@rad1092 rad1092 deleted the codex/provide-step-by-step-execution-guide branch February 13, 2026 14:32
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 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".

Comment thread README.md
# 3) 테스트
ollama run <bitnet-model-tag> "샘플 매출 데이터를 요약해줘"
# 3) CSV 분석 payload 생성
bitnet-analyze analyze sample.csv --question "샘플 매출 데이터를 요약해줘" --out payload.json
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge 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 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant