sourcecode 1.33.17__tar.gz → 1.33.19__tar.gz

{sourcecode-1.33.17 → sourcecode-1.33.19}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: sourcecode
-Version: 1.33.17
+Version: 1.33.19
 Summary: Persistent structural context and ultra-fast repeated analysis for AI coding agents
 License-File: LICENSE
 Keywords: agents,ai,codebase,context,developer-tools,llm
@@ -39,7 +39,7 @@ Description-Content-Type: text/markdown
 
 **Persistent structural context and ultra-fast repeated analysis for AI coding agents.**
 
-![Version](https://img.shields.io/badge/version-1.33.17-blue)
+![Version](https://img.shields.io/badge/version-1.33.19-blue)
 ![Python](https://img.shields.io/badge/python-3.10%2B-green)
 
 ---

{sourcecode-1.33.17 → sourcecode-1.33.19}/README.md

@@ -2,7 +2,7 @@
 
 **Persistent structural context and ultra-fast repeated analysis for AI coding agents.**
 
-![Version](https://img.shields.io/badge/version-1.33.17-blue)
+![Version](https://img.shields.io/badge/version-1.33.19-blue)
 ![Python](https://img.shields.io/badge/python-3.10%2B-green)
 
 ---

{sourcecode-1.33.17 → sourcecode-1.33.19}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "sourcecode"
-version = "1.33.17"
+version = "1.33.19"
 description = "Persistent structural context and ultra-fast repeated analysis for AI coding agents"
 readme = "README.md"
 requires-python = ">=3.9"

{sourcecode-1.33.17 → sourcecode-1.33.19}/src/sourcecode/__init__.py

@@ -1,3 +1,3 @@
 """sourcecode — Deterministic codebase context maps for AI coding agents."""
 
-__version__ = "1.33.17"
+__version__ = "1.33.19"

{sourcecode-1.33.17 → sourcecode-1.33.19}/src/sourcecode/cli.py

@@ -887,6 +887,14 @@ def main(
         )
         raise typer.Exit(code=1)
 
+    # MEJORA-1: --compact is silently ignored when --agent is used.
+    # Always warn (not TTY-gated): user explicitly set both flags, one is being ignored.
+    if compact and agent:
+        typer.echo(
+            "[warning] --compact ignored when --agent is used. --agent takes precedence.",
+            err=True,
+        )
+
     # P0-2 FIX: --full without --compact or --agent has no effect in contract/raw mode.
     # Warn so the user knows the flag is not doing anything.
     if full and not compact and not agent:
@@ -1266,7 +1274,7 @@ def main(
             _view_key = ""
             _core_hash = ""
 
-    if _cache_hit_content is not None:
+    if _cache_hit_content is not None and not changed_only:
         from sourcecode.serializer import write_output
         if format == "json":
             try:
@@ -1948,7 +1956,10 @@ def main(
                 if _uc:
                     # Include untracked (new files not yet staged) so new source files
                     # are analyzed under --changed-only, not silently treated as "clean".
-                    _allowed_changed_files = set(_uc.staged) | set(_uc.unstaged) | set(_uc.untracked)
+                    # Exclude directory entries (trailing "/") — e.g. untracked tool
+                    # cache dirs like ".sourcecode-cache/" are dirs not source files.
+                    _uc_files = {p for p in _uc.untracked if not p.endswith("/")}
+                    _allowed_changed_files = set(_uc.staged) | set(_uc.unstaged) | _uc_files
                 if not _allowed_changed_files:
                     # Git is available and confirms no uncommitted changes.
                     # Do NOT fall back to a full scan — that would silently produce
@@ -1967,9 +1978,11 @@ def main(
             changed_only = False
         if _git_confirmed_clean:
             _nc_payload = json.dumps({
+                "changed_files_count": 0,
                 "changed_files": [],
                 "message": "no uncommitted changes detected",
                 "analysis_scope": "empty",
+                "note": "No uncommitted changes detected. No output produced — use without --changed-only for full context.",
                 "_meta": {"changed_only": True},
             }, ensure_ascii=False)
             write_output(_nc_payload, output=output)
@@ -2075,8 +2088,9 @@ def main(
         if not no_redact:
             data = redact_dict(data)
         # P0-1: Apply output budget — safety net for large repos.
+        # Skip budget when writing to a file (no size constraint); warn on stdout.
         from sourcecode.output_budget import trim_to_budget as _trim, BUDGET_AGENT
-        data = _trim(data, BUDGET_AGENT, label="agent")
+        data = _trim(data, BUDGET_AGENT, label="agent", skip=(output is not None), warn_stderr=(output is None))
         # FIX-P0-2: agent mode must honour --format yaml (previously always emitted JSON).
         if format == "yaml":
             from io import StringIO
@@ -2119,8 +2133,9 @@ def main(
         if not no_redact:
             data = redact_dict(data)
         # P0-1: Apply output budget — safety net for large repos.
+        # Skip budget when writing to a file (no size constraint); warn on stdout.
         from sourcecode.output_budget import trim_to_budget as _trim_c, BUDGET_COMPACT
-        data = _trim_c(data, BUDGET_COMPACT, label="compact")
+        data = _trim_c(data, BUDGET_COMPACT, label="compact", skip=(output is not None), warn_stderr=(output is None))
         if format == "yaml":
             from io import StringIO
             from ruamel.yaml import YAML as _YAML
@@ -2498,8 +2513,11 @@ def prepare_context_cmd(
     # Pro gate: generate-tests and delta require an active Pro license.
     _PRO_TASKS: frozenset[str] = frozenset({"generate-tests", "delta"})
     if task in _PRO_TASKS:
-        from sourcecode.license import require_pro as _require_pro
-        _require_pro(task)
+        from sourcecode.license import require_feature as _require_feature
+        _extra: dict = {}
+        if task == "delta":
+            _extra["free_tier_alternative"] = "sourcecode prepare-context review-pr --since <ref>"
+        _require_feature(task, extra_fields=_extra if _extra else None)
 
     # Validate --format: only "json" and "github-comment" are valid for prepare-context.
     # "yaml" is intentionally NOT supported here (use main command for yaml output).
@@ -3297,6 +3315,10 @@ def impact_cmd(
       - transactional_boundaries_touched — @Transactional classes in the call chain
       - risk_score / risk_level — quantified change risk
 
+    \b
+    NOTE: This feature requires a Pro license. Run 'sourcecode license' for details.
+    Upgrade: sourcecode activate <license_key>
+
     \b
     Examples:
       sourcecode impact UserService
@@ -4323,10 +4345,19 @@ def mcp_status() -> None:
             typer.echo(f"    Fix: sourcecode mcp init --target {client.slug}")
     typer.echo("")
 
+    # Build config state map for cross-check in Stage 3.
+    _configured_clients: set[str] = set()
+    for _c in clients:
+        if _c.app_installed:
+            _cfg = applier.read_config(_c.config_path)
+            if applier.is_installed(_cfg):
+                _configured_clients.add(_c.slug)
+
     # Stage 3: Process liveness — is the client app currently running?
     # This is independent from config: a running app may still need restart to pick up config.
     typer.echo("Runtime (client app process running?)")
     any_installed = any(c.app_installed for c in clients)
+    _action_required: list[str] = []
     if not any_installed:
         typer.echo("  (no client apps found — nothing to check)")
     else:
@@ -4335,11 +4366,18 @@ def mcp_status() -> None:
                 continue
             if is_client_running(client):
                 typer.echo(f"  {client.name:<20} ✓ running")
+                if client.slug not in _configured_clients:
+                    _action_required.append(client.name)
             else:
                 typer.echo(f"  {client.name:<20} ✗ not running")
                 typer.echo(f"    Fix: open {client.name}, then run sourcecode mcp status")
 
     typer.echo(sep)
+    if _action_required:
+        for _name in _action_required:
+            typer.echo(f"  ⚠ ACTION REQUIRED: {_name} is running but sourcecode is not configured.")
+        typer.echo("    Run: sourcecode mcp init")
+        typer.echo("")
     typer.echo("  Note:    'configured' and 'running' are checked independently.")
     typer.echo("           A running app still needs restart after first-time config.")
     typer.echo("  Path:    repo_path must use forward slashes: C:/Users/... or /unix/path")

{sourcecode-1.33.17 → sourcecode-1.33.19}/src/sourcecode/license.py

@@ -206,7 +206,10 @@ def can_use(feature_name: str) -> bool:
     return is_pro
 
 
-def require_feature(feature_name: str) -> None:
+def require_feature(
+    feature_name: str,
+    extra_fields: Optional[dict] = None,
+) -> None:
     """Exit with a clean upgrade prompt when feature_name requires Pro.
 
     Re-validates stale cached license before gating (once per 24 h, online).
@@ -214,6 +217,10 @@ def require_feature(feature_name: str) -> None:
     Writes human-readable context to stderr (terminal UX) and a JSON error
     to stdout (backward-compatible machine-readable format).
 
+    Args:
+        extra_fields: Optional extra keys merged into the JSON error payload
+                      (e.g. ``{"free_tier_alternative": "..."}``)
+
     Example:
         from sourcecode.license import require_feature
         require_feature("impact")
@@ -241,7 +248,7 @@ def require_feature(feature_name: str) -> None:
     sys.stderr.flush()
 
     # JSON on stdout — backward-compatible for CI / MCP consumers
-    payload = {
+    payload: dict = {
         "error": "pro_required",
         "feature": feature_name,
         "message": (
@@ -250,9 +257,11 @@ def require_feature(feature_name: str) -> None:
         ),
         "upgrade_hint": "sourcecode activate <license_key>",
     }
+    if extra_fields:
+        payload.update(extra_fields)
     sys.stdout.write(json.dumps(payload, ensure_ascii=False) + "\n")
     sys.stdout.flush()
-    sys.exit(1)
+    sys.exit(2)  # exit 2 = Pro feature required (0=ok, 1=runtime error, 2=license required)
 
 
 def require_pro(feature_name: str) -> None:

{sourcecode-1.33.17 → sourcecode-1.33.19}/src/sourcecode/output_budget.py

@@ -18,6 +18,7 @@ Always preserved: project_type, project_summary, architecture_summary, task,
 from __future__ import annotations
 
 import json
+import sys
 from typing import Any
 
 
@@ -78,20 +79,36 @@ def _serialized_size(data: Any) -> int:
     return len(json.dumps(data, ensure_ascii=False).encode("utf-8"))
 
 
-def trim_to_budget(data: dict, budget_bytes: int, *, label: str = "") -> dict:
+def trim_to_budget(
+    data: dict,
+    budget_bytes: int,
+    *,
+    label: str = "",
+    skip: bool = False,
+    warn_stderr: bool = False,
+) -> dict:
     """Progressively trim *data* to fit within *budget_bytes*.
 
     Preserves the highest-value sections. Adds ``_budget_note`` when trimming
     occurs so callers can surface it to users.
 
-    Returns data unchanged if already within budget.
+    Args:
+        skip: When True (e.g. writing to a file), skip all trimming and return
+              data unchanged with no ``_budget_note``.
+        warn_stderr: When True, emit a WARNING line to stderr before returning
+                     if trimming was applied. Used for stdout output so users
+                     see the warning before the JSON payload.
+
+    Returns data unchanged if already within budget or ``skip=True``.
     """
-    if _serialized_size(data) <= budget_bytes:
+    if skip or _serialized_size(data) <= budget_bytes:
         return data
 
     result: dict = dict(data)
     original_size = _serialized_size(result)
     trimmed_sections: list[str] = []
+    # Track original counts for total_omitted_items calculation.
+    _original_counts: dict[str, int] = {}
 
     for top_key, inner_key, max_items in _TRIM_SCHEDULE:
         if _serialized_size(result) <= budget_bytes:
@@ -107,9 +124,12 @@ def trim_to_budget(data: dict, budget_bytes: int, *, label: str = "") -> dict:
             if max_items == 0:
                 if top_key in _ALWAYS_KEEP:
                     continue
+                if isinstance(section_val, list):
+                    _original_counts[top_key] = _original_counts.get(top_key, len(section_val))
                 del result[top_key]
                 trimmed_sections.append(f"{top_key}:dropped")
             elif isinstance(section_val, list) and len(section_val) > max_items:
+                _original_counts[top_key] = _original_counts.get(top_key, len(section_val))
                 result[top_key] = section_val[:max_items]
                 trimmed_sections.append(f"{top_key}≤{max_items}")
         else:
@@ -119,19 +139,29 @@ def trim_to_budget(data: dict, budget_bytes: int, *, label: str = "") -> dict:
             if inner_key not in section_val:
                 continue
             inner_val = section_val[inner_key]
+            _inner_key = f"{top_key}.{inner_key}"
             if max_items == 0:
+                if isinstance(inner_val, list):
+                    _original_counts[_inner_key] = _original_counts.get(_inner_key, len(inner_val))
                 new_sec = dict(section_val)
                 del new_sec[inner_key]
                 result[top_key] = new_sec
-                trimmed_sections.append(f"{top_key}.{inner_key}:dropped")
+                trimmed_sections.append(f"{_inner_key}:dropped")
             elif isinstance(inner_val, list) and len(inner_val) > max_items:
+                _original_counts[_inner_key] = _original_counts.get(_inner_key, len(inner_val))
                 new_sec = dict(section_val)
                 new_sec[inner_key] = inner_val[:max_items]
                 result[top_key] = new_sec
-                trimmed_sections.append(f"{top_key}.{inner_key}≤{max_items}")
+                trimmed_sections.append(f"{_inner_key}≤{max_items}")
 
     final_size = _serialized_size(result)
     if trimmed_sections:
+        # Build human-readable section summary for note/warning.
+        _section_summary_parts: list[str] = []
+        for _sk, _orig in _original_counts.items():
+            _cur_key = _sk.replace(".", "/")  # normalize for display
+            _section_summary_parts.append(f"{_sk} ({_orig} total)")
+
         note = (
             f"Output trimmed {original_size // 1024}KB → {final_size // 1024}KB "
             f"(budget {budget_bytes // 1024}KB). "
@@ -142,6 +172,51 @@ def trim_to_budget(data: dict, budget_bytes: int, *, label: str = "") -> dict:
             note = f"[{label}] {note}"
         result["_budget_note"] = note
 
+        # Compute total omitted items across all truncated lists.
+        total_omitted = 0
+        for _sk, _orig in _original_counts.items():
+            if ":dropped" in "".join(s for s in trimmed_sections if _sk in s):
+                total_omitted += _orig
+            else:
+                # Find the last max_items cap applied to this key.
+                _caps = [
+                    int(s.split("≤")[1])
+                    for s in trimmed_sections
+                    if s.startswith(_sk + "≤")
+                ]
+                _cap = min(_caps) if _caps else 0
+                total_omitted += max(0, _orig - _cap)
+
+        result["_truncation_summary"] = {
+            "total_omitted_items": total_omitted,
+            "original_size_kb": original_size // 1024,
+            "final_size_kb": final_size // 1024,
+            "budget_kb": budget_bytes // 1024,
+        }
+
+        if warn_stderr:
+            # Build per-section counts for the warning line.
+            _warn_sections: list[str] = []
+            for _sk, _orig in _original_counts.items():
+                _caps = [
+                    int(s.split("≤")[1])
+                    for s in trimmed_sections
+                    if s.startswith(_sk + "≤")
+                ]
+                _shown = min(_caps) if _caps else 0
+                _warn_sections.append(f"{_sk} ({_shown}/{_orig})")
+            _warn_line = (
+                f"WARNING: Output will be trimmed "
+                f"({original_size // 1024}KB → {final_size // 1024}KB, "
+                f"budget {budget_bytes // 1024}KB). "
+                f"Affected: {', '.join(_warn_sections)}. "
+                "Use --output <file> to capture full output.\n"
+            )
+            if label:
+                _warn_line = f"[{label}] {_warn_line}"
+            sys.stderr.write(_warn_line)
+            sys.stderr.flush()
+
     return result
 
 

{sourcecode-1.33.17 → sourcecode-1.33.19}/src/sourcecode/ris.py

@@ -350,14 +350,15 @@ def _current_git_head(repo_root: Path) -> str:
 
 
 def _has_uncommitted_changes(repo_root: Path) -> bool:
-    """Return True if working tree has staged or unstaged changes.
+    """Return True if working tree has staged or unstaged changes to tracked files.
 
-    Uses ``git status --porcelain`` — any non-empty output means the working
-    tree diverges from HEAD.  Returns False on any error (non-git dirs, etc.).
+    Uses ``git status --porcelain --untracked-files=no`` so that untracked
+    files (e.g. legacy .sourcecode-cache/ directories) do not produce false
+    positives.  Returns False on any error (non-git dirs, etc.).
     """
     try:
         result = subprocess.run(
-            ["git", "-C", str(repo_root), "status", "--porcelain"],
+            ["git", "-C", str(repo_root), "status", "--porcelain", "--untracked-files=no"],
             capture_output=True,
             text=True,
             timeout=2,