Merge pull request #764 from Pipelex/release/v0.20.13

Release v0.20.13

Author: lchoquel

.badges/tests.json

@@ -1,7 +1,7 @@
 {
   "schemaVersion": 1,
   "label": "tests",
-  "message": "4032",
+  "message": "4001",
   "color": "blue",
   "cacheSeconds": 300
 }

.pipelex/pipelex.toml

@@ -48,7 +48,6 @@ error_stack_traces = true
 graphspec_json = true
 mermaidflow_mmd = true
 mermaidflow_html = true
-reactflow_viewspec = true
 reactflow_html = true
 
 [pipelex.pipeline_execution_config.graph_config.reactflow_config]

CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## [v0.20.13] - 2026-03-16
+
+### Removed
+
+- Remove unused `ViewSpec` class and related code from graph/reactflow — dead code that was no longer referenced.
+
 ## [v0.20.12] - 2026-03-16
 
 ### Fixed

docs/under-the-hood/execution-graph-tracing.md

@@ -84,7 +84,6 @@ graph_spec = await dry_run_pipe_with_graph(pipe)
 | `graphspec_json` | `_graphspec.json` | Canonical graph representation |
 | `mermaidflow_mmd` | `_mermaid.mmd` | Mermaid flowchart code |
 | `mermaidflow_html` | `_mermaid.html` | Standalone Mermaid viewer |
-| `reactflow_viewspec` | `_viewspec.json` | ViewSpec for ReactFlow |
 | `reactflow_html` | `_reactflow.html` | Interactive ReactFlow viewer |
 
 ---
@@ -132,11 +131,9 @@ flowchart TB
     subgraph RENDER["Renderers"]
         direction TB
         MF["MermaidflowFactory"]
-        RF["ViewSpec Transformer"]
-        HTML1["Mermaid HTML"]
-        HTML2["ReactFlow HTML"]
-        MF --> HTML1
-        RF --> HTML2
+        RF["ReactFlow HTML Generator"]
+        MF --> HTML1["Mermaid HTML"]
+        RF --> HTML2["ReactFlow HTML"]
     end
 
     JOB --> CTX
@@ -363,27 +360,9 @@ print(mermaidflow.mermaid_code)
 - Stuff nodes (data items) rendered as stadium shapes
 - DATA edges connect producers → stuff → consumers
 
-### ViewSpec (ReactFlow)
+### ReactFlow HTML
 
-Transforms GraphSpec into a viewer-oriented model:
-
-```python
-from pipelex.graph.reactflow.viewspec_transformer import graphspec_to_viewspec
-
-analysis = GraphAnalysis.from_graphspec(graph_spec)
-viewspec = graphspec_to_viewspec(
-    graph_spec,
-    analysis,
-    options={"show_data_edges": True},
-)
-```
-
-ViewSpec provides:
-
-- `ViewNode` with UI metadata (badges, classes, inspector data)
-- `ViewEdge` with ReactFlow properties (animated, hidden)
-- `ViewIndex` for fast client-side lookups
-- Layout configuration for Dagre
+ReactFlow HTML is generated directly from GraphSpec — no intermediate ViewSpec layer. The HTML generator embeds GraphSpec as JSON and the client-side JavaScript handles dataflow analysis, layout, and rendering.
 
 ---
 
@@ -405,7 +384,6 @@ error_stack_traces = true       # Include full stack traces
 graphspec_json = true           # Generate GraphSpec JSON
 mermaidflow_mmd = true          # Generate Mermaid code
 mermaidflow_html = true         # Generate Mermaid HTML
-reactflow_viewspec = true       # Generate ViewSpec JSON
 reactflow_html = true           # Generate ReactFlow HTML
 ```
 
@@ -480,7 +458,7 @@ validate_graphspec(graph_spec)
 | `GraphSpec.to_json()` | Serialize to JSON string |
 | `GraphAnalysis.from_graphspec(g)` | Pre-compute analysis |
 | `MermaidflowFactory.make_from_graphspec(...)` | Generate Mermaid |
-| `graphspec_to_viewspec(g, analysis)` | Generate ViewSpec |
+| `generate_reactflow_html(graphspec, config)` | Generate ReactFlow HTML |
 | `generate_graph_outputs(g, config, pipe_code)` | Generate all outputs |
 
 ---

pipelex/cli/agent_cli/commands/validate/bundle_cmd.py

@@ -49,7 +49,7 @@ def validate_bundle_cmd(
     ] = None,
     view: Annotated[
         bool,
-        typer.Option("--view", help="On successful bundle validation, include a ViewSpec (structured JSON for graph rendering) in the output"),
+        typer.Option("--view", help="On successful bundle validation, include a GraphSpec (structured JSON for graph rendering) in the output"),
     ] = False,
     library_dir: Annotated[
         list[str] | None,
@@ -147,7 +147,7 @@ def validate_bundle_cmd(
             except Exception as exc:
                 agent_error(f"Graph generation failed: {exc}", type(exc).__name__, cause=exc)
 
-        # Generate view (ViewSpec JSON) if requested and validation succeeded
+        # Generate view (GraphSpec JSON) if requested and validation succeeded
         if view:
             try:
                 view_result = asyncio.run(

pipelex/graph/graph_config.py

@@ -18,7 +18,6 @@ class GraphsInclusionConfig(ConfigModel):
     graphspec_json: bool
     mermaidflow_mmd: bool
     mermaidflow_html: bool
-    reactflow_viewspec: bool
     reactflow_html: bool
 
 

pipelex/graph/graph_factory.py

@@ -11,13 +11,10 @@
 from pydantic import BaseModel
 
 from pipelex import log
-from pipelex.graph.graph_analysis import GraphAnalysis
 from pipelex.graph.mermaidflow.mermaid_html import render_mermaid_html_async, render_mermaid_html_with_data_async
 from pipelex.graph.mermaidflow.mermaidflow_factory import MermaidflowFactory
 from pipelex.graph.mermaidflow.stuff_collector import collect_stuff_data_html, collect_stuff_data_text
 from pipelex.graph.reactflow.reactflow_html import generate_reactflow_html_async
-from pipelex.graph.reactflow.viewspec import LayoutSpec
-from pipelex.graph.reactflow.viewspec_transformer import graphspec_to_viewspec
 from pipelex.tools.misc.string_utils import snake_to_title_case
 
 if TYPE_CHECKING:
@@ -37,14 +34,12 @@ class GraphOutputs(BaseModel):
         graphspec_json: The GraphSpec serialized as JSON.
         mermaidflow_mmd: Mermaidflow view as Mermaid flowchart code.
         mermaidflow_html: Mermaidflow view as standalone HTML page.
-        reactflow_viewspec: The ViewSpec serialized as JSON for ReactFlow rendering.
         reactflow_html: ReactFlow interactive graph as standalone HTML page.
     """
 
     graphspec_json: str | None = None
     mermaidflow_mmd: str | None = None
     mermaidflow_html: str | None = None
-    reactflow_viewspec: str | None = None
     reactflow_html: str | None = None
 
 
@@ -64,15 +59,14 @@ async def generate_graph_outputs(
     This can generate:
     - GraphSpec JSON: The canonical graph representation
     - Mermaidflow view: Data flow with controller subgraphs (Mermaid)
-    - ReactFlow ViewSpec: JSON for ReactFlow rendering
     - ReactFlow HTML: Interactive graph viewer
 
     Args:
         graph_spec: The GraphSpec to render.
         graph_config: Configuration controlling which outputs to generate and data inclusion.
         pipe_code: The pipe code, used to derive the HTML page title when title is not provided.
         title: Explicit HTML page title. When provided, overrides the auto-derived title from pipe_code.
-        direction: Flowchart direction for Mermaid diagrams. When None, uses graph_config.mermaid_config.direction.
+        direction: Flowchart direction override for both Mermaid and ReactFlow outputs. When None, each renderer uses its own config default.
         include_subgraphs: Whether to render controller hierarchy as subgraphs in Mermaid output.
 
     Returns:
@@ -84,7 +78,6 @@ async def generate_graph_outputs(
     graphspec_json: str | None = None
     mermaidflow_mmd: str | None = None
     mermaidflow_html: str | None = None
-    reactflow_viewspec: str | None = None
     reactflow_html: str | None = None
 
     # Generate GraphSpec JSON
@@ -121,47 +114,34 @@ async def generate_graph_outputs(
             else:
                 mermaidflow_html = await render_mermaid_html_async(mermaidflow.mermaid_code, title=page_title, theme=mermaid_theme)
 
-    # Generate ReactFlow outputs
-    if inclusion.reactflow_viewspec or inclusion.reactflow_html:
-        analysis = GraphAnalysis.from_graphspec(graph_spec)
-        rf_config = graph_config.reactflow_config
-        effective_rf_direction = direction or rf_config.layout_direction
-        layout = LayoutSpec(
-            direction=effective_rf_direction,
-            nodesep=rf_config.nodesep,
-            ranksep=rf_config.ranksep,
+    # Generate ReactFlow HTML
+    if inclusion.reactflow_html:
+        # Collect stuff data in alternate formats if configured
+        rf_stuff_data_text: dict[str, str] | None = None
+        rf_stuff_data_html: dict[str, str] | None = None
+        if graph_config.data_inclusion.stuff_text_content:
+            log.verbose("Collecting stuff data text for graph_spec")
+            rf_stuff_data_text = collect_stuff_data_text(graph_spec)
+        else:
+            log.verbose("No stuff data text to collect for graph_spec")
+        if graph_config.data_inclusion.stuff_html_content:
+            rf_stuff_data_html = collect_stuff_data_html(graph_spec)
+
+        effective_rf_config = graph_config.reactflow_config
+        if direction is not None:
+            effective_rf_config = effective_rf_config.model_copy(update={"layout_direction": direction})
+        reactflow_html = await generate_reactflow_html_async(
+            graph_spec,
+            effective_rf_config,
+            stuff_data_text=rf_stuff_data_text,
+            stuff_data_html=rf_stuff_data_html,
+            title=page_title,
         )
-        viewspec = graphspec_to_viewspec(graph_spec, analysis, layout=layout)
-
-        if inclusion.reactflow_viewspec:
-            reactflow_viewspec = viewspec.model_dump_json(indent=2)
-
-        if inclusion.reactflow_html:
-            # Collect stuff data in alternate formats if configured
-            rf_stuff_data_text: dict[str, str] | None = None
-            rf_stuff_data_html: dict[str, str] | None = None
-            if graph_config.data_inclusion.stuff_text_content:
-                log.verbose("Collecting stuff data text for graph_spec")
-                rf_stuff_data_text = collect_stuff_data_text(graph_spec)
-            else:
-                log.verbose("No stuff data text to collect for graph_spec")
-            if graph_config.data_inclusion.stuff_html_content:
-                rf_stuff_data_html = collect_stuff_data_html(graph_spec)
-
-            reactflow_html = await generate_reactflow_html_async(
-                viewspec,
-                graph_config.reactflow_config,
-                graphspec=graph_spec,
-                stuff_data_text=rf_stuff_data_text,
-                stuff_data_html=rf_stuff_data_html,
-                title=page_title,
-            )
 
     return GraphOutputs(
         graphspec_json=graphspec_json,
         mermaidflow_mmd=mermaidflow_mmd,
         mermaidflow_html=mermaidflow_html,
-        reactflow_viewspec=reactflow_viewspec,
         reactflow_html=reactflow_html,
     )
 
@@ -205,12 +185,6 @@ def save_graph_outputs_to_dir(
         saved_files["mermaidflow_html"] = file_path
         log.verbose(f"Mermaidflow HTML saved to: {file_path}")
 
-    if graph_outputs.reactflow_viewspec is not None:
-        file_path = output_dir / "viewspec.json"
-        file_path.write_text(graph_outputs.reactflow_viewspec, encoding="utf-8")
-        saved_files["reactflow_viewspec"] = file_path
-        log.verbose(f"ReactFlow ViewSpec saved to: {file_path}")
-
     if graph_outputs.reactflow_html is not None:
         file_path = output_dir / "reactflow.html"
         file_path.write_text(graph_outputs.reactflow_html, encoding="utf-8")

pipelex/graph/graph_rendering.py

@@ -11,12 +11,9 @@
 from pipelex.config import get_config
 from pipelex.core.interpreter.exceptions import PipelexInterpreterError
 from pipelex.core.interpreter.interpreter import PipelexInterpreter
-from pipelex.graph.graph_analysis import GraphAnalysis
 from pipelex.graph.graph_config import GraphConfig
 from pipelex.graph.graph_factory import GraphOutputs, generate_graph_outputs, save_graph_outputs_to_dir
 from pipelex.graph.graphspec import GraphSpec
-from pipelex.graph.reactflow.viewspec import LayoutSpec
-from pipelex.graph.reactflow.viewspec_transformer import graphspec_to_viewspec
 from pipelex.pipe_run.pipe_run_mode import PipeRunMode
 from pipelex.pipeline.runner import PipelexRunner
 from pipelex.tools.misc.chart_utils import FlowchartDirection
@@ -89,7 +86,6 @@ async def render_graph_from_spec(
                     "graphspec_json": False,
                     "mermaidflow_mmd": False,
                     "mermaidflow_html": include_mermaidflow,
-                    "reactflow_viewspec": False,
                     "reactflow_html": include_reactflow,
                 }
             ),
@@ -246,15 +242,14 @@ async def generate_graph_for_bundle(
     }
 
 
-# TODO: refactor to do only one dry run and generate both graph and ViewSpec
 async def generate_view_for_bundle(
     bundle_path: Path,
     library_dirs: list[str] | None = None,
     direction: FlowchartDirection | None = None,
 ) -> dict[str, Any]:
-    """Generate a ViewSpec for a bundle via dry-run pipeline execution.
+    """Generate a GraphSpec for a bundle via dry-run pipeline execution.
 
-    Returns structured JSON data (ViewSpec) suitable for client-side rendering,
+    Returns structured JSON data (GraphSpec) suitable for client-side rendering,
     without writing any files to disk.
 
     Args:
@@ -263,7 +258,7 @@ async def generate_view_for_bundle(
         direction: Flowchart layout direction (default: None, uses config default).
 
     Returns:
-        Dictionary with viewspec (JSON-serializable dict), pipe_code, and direction.
+        Dictionary with graphspec (JSON-serializable dict), pipe_code, and direction.
 
     Raises:
         PipelexInterpreterError: If bundle parsing fails or main_pipe is missing.
@@ -279,16 +274,7 @@ async def generate_view_for_bundle(
     rf_config = execution_config.graph_config.reactflow_config
     effective_direction = direction or rf_config.layout_direction
 
-    analysis = GraphAnalysis.from_graphspec(graph_spec)
-    layout = LayoutSpec(
-        direction=effective_direction,
-        nodesep=rf_config.nodesep,
-        ranksep=rf_config.ranksep,
-    )
-    viewspec = graphspec_to_viewspec(graph_spec, analysis, layout=layout)
-
     return {
-        "viewspec": viewspec.model_dump(mode="json"),
         "graphspec": graph_spec.model_dump(mode="json", by_alias=True),
         "pipe_code": pipe_code,
         "direction": str(effective_direction) if effective_direction else None,