Skip to content

Use actual output directory in CLI completion message#181

Draft
Copilot wants to merge 2 commits into
mainfrom
copilot/update-cli-message-output-dir
Draft

Use actual output directory in CLI completion message#181
Copilot wants to merge 2 commits into
mainfrom
copilot/update-cli-message-output-dir

Conversation

Copy link
Copy Markdown

Copilot AI commented Apr 29, 2026

The CLI completion message after report generation always showed hardcoded paths (streamlit_report/sections, quarto_report) regardless of the --output_directory flag, making the message misleading when a custom output path was specified.

Changes

  • src/vuegen/utils/__init__.py — Added output_dir parameter to get_completion_message. When provided, all displayed paths are derived from it; falls back to previous defaults when None (backward compatible).
  • src/vuegen/__main__.py — Captures report_dir from get_report() (previously discarded as _, _) and passes it as output_dir to get_completion_message.

Example

With --output_directory /custom/path, the message now reflects the real location:

📂 All scripts to build the Streamlit app are available at:
    /custom/path/streamlit_report/sections

▶️ To run the Streamlit app, use the following command:
    streamlit run /custom/path/streamlit_report/sections/report_manager.py

Copilot AI linked an issue Apr 29, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Update CLI message based on output directory flag Use actual output directory in CLI completion message Apr 29, 2026
Copilot AI requested a review from enryH April 29, 2026 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update cli message if output dir is provided

2 participants