specfact-cli 0.15.2__tar.gz → 0.15.3__tar.gz

{specfact_cli-0.15.2 → specfact_cli-0.15.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: specfact-cli
-Version: 0.15.2
+Version: 0.15.3
 Summary: Brownfield-first CLI: Reverse engineer legacy Python → specs → enforced contracts. Automate legacy code documentation and prevent modernization regressions.
 Project-URL: Homepage, https://github.com/nold-ai/specfact-cli
 Project-URL: Repository, https://github.com/nold-ai/specfact-cli.git

{specfact_cli-0.15.2 → specfact_cli-0.15.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "specfact-cli"
-version = "0.15.2"
+version = "0.15.3"
 description = "Brownfield-first CLI: Reverse engineer legacy Python → specs → enforced contracts. Automate legacy code documentation and prevent modernization regressions."
 readme = "README.md"
 requires-python = ">=3.11"

{specfact_cli-0.15.2 → specfact_cli-0.15.3}/resources/prompts/specfact.03-review.md

@@ -101,6 +101,7 @@ The recommendation helps less-experienced users make informed decisions.
 - `--list-questions` - Output questions in JSON format. Default: False
 - `--output-questions PATH` - Save questions directly to file (JSON format). Use with `--list-questions` to save instead of stdout. Default: None
 - `--list-findings` - Output all findings in structured format. Default: False
+- `--output-findings PATH` - Save findings directly to file (JSON/YAML format). Use with `--list-findings` to save instead of stdout. Default: None
 - `--findings-format FORMAT` - Output format: json, yaml, or table. Default: json for non-interactive, table for interactive
 
 ### Behavior/Options
@@ -150,7 +151,11 @@ specfact plan review [<bundle-name>] --list-questions --output-questions /tmp/qu
 ```bash
 # Get findings (saves to stdout - can redirect to /tmp/)
 # Use /tmp/ to avoid polluting the codebase
+# Option 1: Redirect output (includes CLI banner - not recommended)
 specfact plan review [<bundle-name>] --list-findings --findings-format json --no-interactive > /tmp/findings.json
+
+# Option 2: Save directly to file (recommended - clean JSON only)
+specfact plan review [<bundle-name>] --list-findings --output-findings /tmp/findings.json --no-interactive
 ```
 
 **Note**: The `--output-questions` option saves questions directly to a file, avoiding the need for complex JSON parsing. The ambiguity scanner now recognizes the simplified format (e.g., "Must verify X works correctly (see contract examples)") as valid and will not flag it as vague.
@@ -362,7 +367,8 @@ When in copilot mode, follow this three-phase workflow:
 
 ```bash
 # Option 1: Get findings (redirect to /tmp/ to avoid polluting codebase)
-specfact plan review [<bundle-name>] --list-findings --findings-format json --no-interactive > /tmp/findings.json
+# Option 1: Save findings directly to file (recommended - clean JSON only)
+specfact plan review [<bundle-name>] --list-findings --output-findings /tmp/findings.json --no-interactive
 
 # Option 2: Get questions and save directly to /tmp/ (recommended - avoids JSON parsing)
 specfact plan review [<bundle-name>] --list-questions --output-questions /tmp/questions.json --no-interactive
@@ -513,6 +519,7 @@ Create one with: specfact plan init legacy-api
 # Get findings first
 /specfact.03-review --list-findings                    # List all findings
 /specfact.03-review --list-findings --findings-format json  # JSON format for enrichment
+/specfact.03-review --list-findings --output-findings /tmp/findings.json  # Save findings to file (clean JSON)
 
 # Interactive review
 /specfact.03-review                                    # Uses active plan (default: 5 questions per session)
@@ -557,7 +564,7 @@ Create one with: specfact plan init legacy-api
 2. **Get findings** (optional, for comprehensive analysis - use `/tmp/`):
 
    ```bash
-   specfact plan review [<bundle-name>] --list-findings --findings-format json --no-interactive > /tmp/findings.json
+   specfact plan review [<bundle-name>] --list-findings --output-findings /tmp/findings.json --no-interactive
    ```
 
 3. **LLM reasoning and user selection** (REQUIRED for partial findings):

{specfact_cli-0.15.2 → specfact_cli-0.15.3}/src/__init__.py

@@ -3,4 +3,4 @@ SpecFact CLI - Spec→Contract→Sentinel tool for contract-driven development.
 """
 
 # Define the package version (kept in sync with pyproject.toml and setup.py)
-__version__ = "0.14.2"
+__version__ = "0.15.3"

{specfact_cli-0.15.2 → specfact_cli-0.15.3}/src/specfact_cli/__init__.py

@@ -9,6 +9,6 @@ This package provides command-line tools for:
 - Validating reproducibility
 """
 
-__version__ = "0.15.2"
+__version__ = "0.15.3"
 
 __all__ = ["__version__"]

{specfact_cli-0.15.2 → specfact_cli-0.15.3}/src/specfact_cli/commands/plan.py

@@ -2985,6 +2985,7 @@ def _output_findings(
     report: Any,  # AmbiguityReport (imported locally to avoid circular dependency)
     findings_format: str | None,
     is_non_interactive: bool,
+    output_path: Path | None = None,
 ) -> None:
     """
     Output findings in structured format or table.
@@ -2993,9 +2994,15 @@ def _output_findings(
         report: Ambiguity report
         findings_format: Output format (json, yaml, table)
         is_non_interactive: Whether in non-interactive mode
+        output_path: Optional file path to save findings. If None, outputs to stdout.
     """
+    from rich.console import Console
+    from rich.table import Table
+
     from specfact_cli.analyzers.ambiguity_scanner import AmbiguityStatus
 
+    console = Console()
+
     # Determine output format
     output_format_str = findings_format
     if not output_format_str:
@@ -3069,7 +3076,7 @@ def _output_findings(
         import sys
 
         if output_format_str == "json":
-            sys.stdout.write(json.dumps(findings_data, indent=2))
+            formatted_output = json.dumps(findings_data, indent=2) + "\n"
         else:  # yaml
             from ruamel.yaml import YAML
 
@@ -3080,9 +3087,20 @@ def _output_findings(
 
             output = StringIO()
             yaml.dump(findings_data, output)
-            sys.stdout.write(output.getvalue())
-        sys.stdout.write("\n")
-        sys.stdout.flush()
+            formatted_output = output.getvalue()
+
+        if output_path:
+            # Save to file
+            output_path.parent.mkdir(parents=True, exist_ok=True)
+            output_path.write_text(formatted_output, encoding="utf-8")
+            from rich.console import Console
+
+            console = Console()
+            console.print(f"[green]✓[/green] Findings saved to: {output_path}")
+        else:
+            # Output to stdout
+            sys.stdout.write(formatted_output)
+            sys.stdout.flush()
     else:
         print_error(f"Invalid findings format: {findings_format}. Must be 'json', 'yaml', or 'table'")
         raise typer.Exit(1)
@@ -3987,6 +4005,11 @@ def review(
         case_sensitive=False,
         hidden=True,  # Hidden by default, shown with --help-advanced
     ),
+    output_findings: Path | None = typer.Option(
+        None,
+        "--output-findings",
+        help="Save findings to file (JSON/YAML format). If --list-findings is also set, findings are saved to file instead of stdout. Default: None",
+    ),
     # Behavior/Options
     no_interactive: bool = typer.Option(
         False,
@@ -4031,7 +4054,9 @@ def review(
         specfact plan review legacy-api
         specfact plan review auth-module --max-questions 3 --category "Functional Scope"
         specfact plan review legacy-api --list-questions  # Output questions as JSON
+        specfact plan review legacy-api --list-questions --output-questions /tmp/questions.json  # Save questions to file
         specfact plan review legacy-api --list-findings --findings-format json  # Output all findings as JSON
+        specfact plan review legacy-api --list-findings --output-findings /tmp/findings.json  # Save findings to file
         specfact plan review legacy-api --answers '{"Q001": "answer1", "Q002": "answer2"}'  # Non-interactive
     """
     from rich.console import Console
@@ -4095,7 +4120,7 @@ def review(
 
             # Handle --list-findings mode
             if list_findings:
-                _output_findings(report, findings_format, is_non_interactive)
+                _output_findings(report, findings_format, is_non_interactive, output_findings)
                 raise typer.Exit(0)
 
             # Show initial coverage summary BEFORE questions (so user knows what's missing)