Refine build pipe CLI output with colors and conditional inputs#655
Open
lchoquel wants to merge 4 commits intofeature/tweaks2from
Open
Refine build pipe CLI output with colors and conditional inputs#655lchoquel wants to merge 4 commits intofeature/tweaks2from
lchoquel wants to merge 4 commits intofeature/tweaks2from
Conversation
Replace noisy per-file `✅` messages with `log.verbose()` and a single consolidated recap matching the `pipelex run` output pattern. Individual file saves are now hidden at default verbosity, and a clean summary lists output directory contents with format names. The "how to run" hint now shows `pipelex run <dir>` instead of verbose flag-based invocation. Also adds a `quiet` parameter to `generate_structures_from_blueprints()` so callers can suppress per-file progress output when embedding structure generation in a larger workflow. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Add --bundle-view/--no-bundle-view flag (disabled by default) to make bundle view HTML/SVG generation opt-in - Show domain and main pipe name in the bundle.plx recap line - Only generate inputs.json when the pipe actually requires inputs - Add a note prompting users to fill inputs.json before running - Promote several log.debug/log.dev calls to log.verbose for better observability in graph_factory, library_manager, and telemetry Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Style the "Output saved to" path in bold magenta for both build and run commands, use red for the main pipe name, and green for structure names in the build pipe recap. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
| saved_formats.append("reactflow") | ||
|
|
||
| return saved_count | ||
| return saved_formats |
There was a problem hiding this comment.
Graph format tracking omits non-HTML saved files
Low Severity
_save_graph_outputs_to_dir only appends to saved_formats when mermaidflow_html or reactflow_html are generated, but not for graphspec_json, mermaidflow_mmd, or reactflow_viewspec. If a user configures graphs to generate only non-HTML formats, files get saved to disk but the recap shows no graph output — and saved_graph_sections stays empty because the caller checks truthiness of the returned list.
The `no_extras` path already returns early, so the subsequent check is always true. Dedent the extras generation block to reduce nesting. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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
build pipeCLI output: replace per-file✅lines with a consolidated recap--bundle-view/--no-bundle-viewflag (disabled by default) to make bundle view generation opt-ininputs.jsonwhen the pipe actually requires inputs, with a note to fill it before runningbundle.plxrecap line with domain and main pipe namequietparameter togenerate_structures_from_blueprints()for suppressed output when called from builderlog.debug/log.devcalls tolog.verbosefor better observabilityTest plan
make agent-checkpasses (ruff, pyright, mypy)make agent-testpassespipelex build pipe "Given a theme, write a Haiku" --no-logoshows clean recap with colored outputpipelex build pipe "Given a theme, write a Haiku" --no-extras --no-logoshows PLX-only recappipelex build pipe "Given a theme, write a Haiku" --bundle-view --no-logoincludes bundle_view linepipelex runshows bold magenta output path in recap🤖 Generated with Claude Code
Note
Medium Risk
Moderate risk due to behavior changes in
pipelex build pipeoutputs (new flags, conditional file generation, graph saving semantics), though changes are largely CLI/observability-focused and don’t alter core pipeline execution logic.Overview
pipelex build pipeoutput is reworked to be less noisy: per-file successsecholines are replaced withlog.verbosemessages and a single colored recap showing what was generated (bundle, optional bundle view, structures, optionalinputs.json, runner, and graph formats). Generation ofinputs.jsonis now conditional on the pipe having inputs, and a new--bundle-view/--no-bundle-viewflag makes bundle view HTML/SVG generation opt-in.Graph saving helper
_save_graph_outputs_to_dirnow returns the list of generated graph formats (instead of a count) so the recap can reportmermaidflow/reactflowper section, and structure generation gains aquietmode for suppressed CLI output. Minor UX polish includes styled output paths inpipelex runrecaps, and multiple internal logs are promoted fromdebug/devtoverbose(graph/reactflow, library loading, telemetry). Dependency lockfile bumps several packages (e.g.,anthropic,openai,posthog,coverage).Written by Cursor Bugbot for commit 3dbe575. This will update automatically on new commits. Configure here.