diffimpactscout 0.2.0__tar.gz → 0.3.0__tar.gz

{diffimpactscout-0.2.0/src/diffimpactscout.egg-info → diffimpactscout-0.3.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: diffimpactscout
-Version: 0.2.0
+Version: 0.3.0
 Summary: Incremental pre-push guard and AST blast-radius impact analyzer for git repositories.
 Author: DiffImpactScout contributors
 License: MIT

{diffimpactscout-0.2.0 → diffimpactscout-0.3.0}/setup.py

@@ -7,7 +7,7 @@ with io.open("README.md", encoding="utf-8") as fh:
 
 setup(
     name="diffimpactscout",
-    version="0.2.0",
+    version="0.3.0",
     description="Incremental pre-push guard and AST blast-radius impact analyzer for git repositories.",
     long_description=long_description,
     long_description_content_type="text/markdown",

{diffimpactscout-0.2.0 → diffimpactscout-0.3.0}/src/diffimpactscout/__init__.py

@@ -1,3 +1,3 @@
 """diffimpactscout: incremental pre-push guard and AST blast-radius impact analyzer."""
 
-__version__ = "0.2.0"
+__version__ = "0.3.0"

{diffimpactscout-0.2.0 → diffimpactscout-0.3.0}/src/diffimpactscout/base.py

@@ -2,13 +2,54 @@
 
 Example: when the repo has an upstream/sprint/12 ref, diffs are measured
 against it instead of origin/master.
+
+================================================================================
+ARCHITECTURAL OVERVIEW: 2-DOT TREE DIFF FOR INCREMENTAL CHECKS
+================================================================================
+
+Why a 2-dot tree diff (`git diff <anchor> HEAD`):
+
+In standard Git workflows a 3-dot diff (`git diff <anchor>...HEAD`) computes
+changes relative to the common merge-base ancestor of the two tips. Whenever a
+developer syncs their branch with upstream (merge, rebase, or squash), the
+merge-base moves and the 3-dot range silently grows to include every commit
+introduced by upstream since the branch diverged. Every pre-existing upstream
+violation (a stale trailing space, a long line, an old lint error) then shows up
+as a "new" issue in the pre-push guard, producing false positives the developer
+did not author and cannot reasonably fix in that push.
+
+A 2-dot tree diff (`git diff <anchor> HEAD`, no `...`) compares the exact
+filesystem snapshot ("tree") of the resolved upstream anchor directly against
+the `HEAD` tree. Consequences:
+
+  1. Zero Upstream Blame: any file or line that exists byte-identically in the
+     anchor tree contributes no diff entries. Only files and lines actually
+     added or changed by the developer are passed to the checks.
+  2. Sync Agnostic: behaves identically whether the branch was synced with
+     `git merge`, `git rebase`, or `git merge --squash`, because it never walks
+     commit ancestry, only the two tip trees.
+  3. Shallow-Clone Safe: computing the name-only diff needs just the two tip
+     trees (plus any blobs referenced in the range), so it works on shallow
+     clones that lack full commit history.
+
+Sprint branch resolution:
+
+In sprint-based development models (e.g. Contify) work is branched from active
+sprint branches such as `refs/remotes/upstream/sprint/33.1` rather than from
+`master`. The resolver inspects `refs/remotes/upstream/master` plus the top
+`DEFAULT_MAX_SPRINT_CANDIDATES` (5) most recent sprint branches sorted by
+committer date, and picks the candidate whose tree is closest to `HEAD` (the
+fewest changed files on a `--diff-filter=ACMRT` name-only diff). The closest
+tree means the smallest honest change-set, which is what incremental checks
+want to target.
+================================================================================
 """
 
 import diffimpactscout.gitrun as gitrun
 
 MASTER_REF = "refs/remotes/upstream/master"
 SPRINT_PREFIX = "sprint/"
-DEFAULT_MAX_SPRINT_CANDIDATES = 10
+DEFAULT_MAX_SPRINT_CANDIDATES = 5
 REMOTE_FALLBACKS = ("main", "master", "develop")
 
 
@@ -16,7 +57,7 @@ def resolve_change_base(root, limit=DEFAULT_MAX_SPRINT_CANDIDATES):
     """Return the remote ref closest to HEAD, or None.
 
     `limit` caps how many `sprint/*` refs are considered: only the
-    `limit` (default 10) most-recent ones by committer date, unlike the
+    `limit` (default 5) most-recent ones by committer date, unlike the
     bash original dev_base.sh which considered every sprint branch.
     """
     for remote in _remote_order(root):

{diffimpactscout-0.2.0 → diffimpactscout-0.3.0}/src/diffimpactscout/checks/base.py

@@ -97,7 +97,9 @@ class ExternalCheck(Check):
                 output = (out or err).decode("utf-8", errors="replace").strip()
                 if not output:
                     output = "exit code %s" % proc.returncode
-                issues.append(CheckIssue(".", 0, 0, self.id, output))
+                else:
+                    output = "%s (exit code %s)" % (output, proc.returncode)
+                issues.append(CheckIssue(".", None, None, self.id, output))
             return CheckResult(issues=issues, fixed=fixed, skipped=skipped, warned=warned)
         for path in files or []:
             argv = self._argv_for(path)
@@ -116,7 +118,9 @@ class ExternalCheck(Check):
                 output = (out or err).decode("utf-8", errors="replace").strip()
                 if not output:
                     output = "exit code %s" % proc.returncode
-                issues.append(CheckIssue(path, 0, 0, self.id, output))
+                else:
+                    output = "%s (exit code %s)" % (output, proc.returncode)
+                issues.append(CheckIssue(path, None, None, self.id, output))
         return CheckResult(issues=issues, fixed=fixed, skipped=skipped, warned=warned)
 
 
@@ -151,7 +155,7 @@ class CheckContext(object):
 
 
 class CheckIssue(object):
-    def __init__(self, path, line, column, code, message):
+    def __init__(self, path, line=None, column=None, code=None, message=None):
         self.path = path
         self.line = line
         self.column = column
@@ -159,13 +163,13 @@ class CheckIssue(object):
         self.message = message
 
     def format(self):
-        return "%s:%s:%s: %s %s" % (
-            self.path,
-            self.line,
-            self.column,
-            self.code,
-            self.message,
-        )
+        if self.line is None and self.column is None:
+            location = self.path
+        else:
+            line = "-" if self.line is None else str(self.line)
+            column = "-" if self.column is None else str(self.column)
+            location = "%s:%s:%s" % (self.path, line, column)
+        return "%s: %s %s" % (location, self.code, self.message)
 
 
 class CheckResult(object):

{diffimpactscout-0.2.0 → diffimpactscout-0.3.0}/src/diffimpactscout/checks/eslint.py

@@ -155,8 +155,8 @@ class EslintCheck(Check):
                     issues.append(
                         CheckIssue(
                             path,
-                            line or 0,
-                            column or 0,
+                            line,
+                            column,
                             rule_id or self.id,
                             message.get("message") or "",
                         )

{diffimpactscout-0.2.0 → diffimpactscout-0.3.0}/src/diffimpactscout/checks/prettier.py

@@ -93,8 +93,8 @@ class PrettierCheck(Check):
             issues.append(
                 CheckIssue(
                     path,
-                    0,
-                    0,
+                    None,
+                    None,
                     self.id,
                     "%s is not formatted. Fix with: npx prettier --write %s"
                     % (path, path),

{diffimpactscout-0.2.0 → diffimpactscout-0.3.0}/src/diffimpactscout/checks/repo_checks.py

@@ -1,11 +1,14 @@
-"""Repo-wide checks for oversized files and leaked private keys.
+"""Repo-wide checks for oversized files, leaked private keys, and case-colliding file paths.
 
-Example: a 300 MB binary is flagged with a git-lfs hint, and any file
-containing a private key block always blocks the push.
+Example: a 300 MB binary is flagged with a git-lfs hint, any file
+containing a private key block always blocks the push, and a pushed
+`FILE.TXT` colliding with a tracked `file.txt` is flagged because the two
+names clobber each other on case-insensitive filesystems (macOS, Windows).
 """
 
 import os
 
+import diffimpactscout.gitrun as gitrun
 from diffimpactscout.checks.base import (
     Check,
     CheckIssue,
@@ -23,6 +26,33 @@ PRIVATE_KEY_MARKERS = (
 )
 
 
+@register
+class CaseConflictCheck(Check):
+    id = "repo/case-conflict"
+    scoped = "files"
+
+    def run(self, context, files):
+        tracked = gitrun.git_nul(["ls-files", "-z"], context.root)
+        by_lower = {}
+        for path in tracked:
+            by_lower.setdefault(path.lower(), []).append(path)
+        issues = []
+        for path in files or []:
+            for other in by_lower.get(path.lower(), ()):
+                if other != path:
+                    issues.append(
+                        CheckIssue(
+                            path,
+                            None,
+                            None,
+                            self.id,
+                            "case conflict detected with %s" % other,
+                        )
+                    )
+                    break
+        return CheckResult(issues=issues)
+
+
 @register
 class LargeFilesCheck(Check):
     id = "repo/large-files"
@@ -61,8 +91,8 @@ class LargeFilesCheck(Check):
                 issues.append(
                     CheckIssue(
                         path,
-                        0,
-                        0,
+                        None,
+                        None,
                         self.id,
                         "file size %d kB exceeds limit of %d kB; consider git lfs"
                         % (size_kb, self.max_kb),
@@ -89,7 +119,7 @@ class PrivateKeyCheck(Check):
             for marker in PRIVATE_KEY_MARKERS:
                 if marker in data:
                     issues.append(
-                        CheckIssue(path, 0, 0, self.id, "private key detected")
+                        CheckIssue(path, None, None, self.id, "private key detected")
                     )
                     break
         return CheckResult(issues=issues)

{diffimpactscout-0.2.0 → diffimpactscout-0.3.0}/src/diffimpactscout/checks/ruff.py

@@ -157,8 +157,8 @@ class RuffCheck(Check):
                 issues.append(
                     CheckIssue(
                         path,
-                        row or 0,
-                        location.get("column") or 0,
+                        row,
+                        location.get("column"),
                         violation.get("code") or self.id,
                         violation.get("message") or "",
                     )
@@ -215,8 +215,8 @@ class RuffFormatCheck(Check):
             issues.append(
                 CheckIssue(
                     path,
-                    0,
-                    0,
+                    None,
+                    None,
                     self.id,
                     "formatting issue in changed lines:\n%s\nFix with: ruff format %s"
                     % (_truncate(kept), path),

{diffimpactscout-0.2.0 → diffimpactscout-0.3.0}/src/diffimpactscout/checks/syntax.py

@@ -38,8 +38,8 @@ class JsonSyntaxCheck(Check):
                 issues.append(
                     CheckIssue(
                         path,
-                        getattr(exc, "lineno", 0) or 0,
-                        getattr(exc, "colno", 0) or 0,
+                        getattr(exc, "lineno", None),
+                        getattr(exc, "colno", None),
                         self.id,
                         "invalid JSON: %s" % exc,
                     )
@@ -69,8 +69,8 @@ class AstSyntaxCheck(Check):
                 issues.append(
                     CheckIssue(
                         path,
-                        exc.lineno or 0,
-                        exc.offset or 0,
+                        exc.lineno,
+                        exc.offset,
                         self.id,
                         "invalid python syntax: %s" % (exc.msg or "syntax error"),
                     )
@@ -78,7 +78,11 @@ class AstSyntaxCheck(Check):
             except ValueError as exc:
                 issues.append(
                     CheckIssue(
-                        path, 0, 0, self.id, "invalid python source: %s" % exc
+                        path,
+                        None,
+                        None,
+                        self.id,
+                        "invalid python source: %s" % exc,
                     )
                 )
         return CheckResult(issues=issues)
@@ -111,7 +115,7 @@ class MergeConflictCheck(Check):
                 line = lines[lineno - 1]
                 if self._is_marker(line):
                     issues.append(
-                        CheckIssue(path, lineno, 0, self.id, "conflict marker found")
+                        CheckIssue(path, lineno, None, self.id, "conflict marker found")
                     )
         return CheckResult(issues=issues)
 

{diffimpactscout-0.2.0 → diffimpactscout-0.3.0}/src/diffimpactscout/cli.py

@@ -46,6 +46,11 @@ def _parser():
     p_impact.add_argument("--staged", action="store_true")
     p_impact.add_argument("--fast", action="store_true")
     p_impact.add_argument("--json", action="store_true")
+    p_impact.add_argument(
+        "--markdown",
+        action="store_true",
+        help="emit a Markdown table instead of the plain ASCII table",
+    )
     p_impact.set_defaults(func=_cmd_impact)
 
     p_check = sub.add_parser(
@@ -148,6 +153,7 @@ def _cmd_impact(args):
         staged=args.staged,
         fast=args.fast,
         json_out=args.json,
+        markdown=args.markdown,
     )
 
 
@@ -243,6 +249,20 @@ def _prompt_choice(text, choices, default):
     return default
 
 
+def _prompt_blocking(default):
+    sys.stdout.write(
+        "Guard blocking mode: strict (block the push) or warn (report only)? "
+        "[strict/Warn] (default: %s): " % default
+    )
+    sys.stdout.flush()
+    answer = _read_answer()
+    if answer in ("strict", "s"):
+        return "strict"
+    if answer in ("warn", "w"):
+        return "warn"
+    return default
+
+
 def _check_id(entry):
     if isinstance(entry, dict):
         return entry.get("id", "?")
@@ -334,10 +354,7 @@ def _ask_overrides(data, args):
     data["guard"] = merged["guard"]
     data["impact"] = merged["impact"]
     if not args.blocking:
-        strict = _prompt_yes_default(
-            "Block the push when checks report issues? (strict mode) [y/N]", False
-        )
-        data["guard"]["blocking"] = "strict" if strict else "warn"
+        data["guard"]["blocking"] = _prompt_blocking("strict")
     hint = _LINT_HINT.get(profile)
     if hint:
         keep = _prompt_yes_default(
@@ -385,11 +402,13 @@ def _cmd_install_hooks(args):
         detected = detect.detect_stack(root)
         profile = _resolve_profile(args, detected)
         data = _build_config_data(profile, args)
+        interactive = not args.yes and _is_tty()
+        if interactive and not args.blocking:
+            data["guard"]["blocking"] = "strict"
         sys.stdout.write(_render_preview(detected, data))
         note = _other_stack_note(root, profile)
         if note:
             sys.stderr.write("diffimpactscout: %s\n" % note)
-        interactive = not args.yes and _is_tty()
         if interactive:
             proceed = _prompt_yes_default("Proceed? [Y/n]", True)
             if not proceed:

{diffimpactscout-0.2.0 → diffimpactscout-0.3.0}/src/diffimpactscout/config.py

@@ -21,6 +21,7 @@ DEFAULT_IGNORE_PATHS = [
     "**/dist/**",
     "**/build/**",
     "**/.git/**",
+    "**/.impact_analysis_cache.json",
 ]
 
 DEFAULT_GUARD_CHECKS = [
@@ -32,6 +33,7 @@ DEFAULT_GUARD_CHECKS = [
     {"id": "syntax/merge-conflict"},
     {"id": "repo/large-files", "args": ["--maxkb=250000"]},
     {"id": "repo/private-key"},
+    {"id": "repo/case-conflict"},
 ]
 
 

{diffimpactscout-0.2.0 → diffimpactscout-0.3.0}/src/diffimpactscout/guard.py

@@ -10,6 +10,7 @@ import diffimpactscout.env as env
 import diffimpactscout.gitrun as gitrun
 import diffimpactscout.scope as scope
 from diffimpactscout.checks.base import CheckContext, make_check
+from diffimpactscout.config import is_excluded
 
 DIFF_FILTER = "ACMRT"
 
@@ -48,6 +49,77 @@ def _warn(text):
     sys.stderr.write("diffimpactscout: warning: %s\n" % text)
 
 
+_FIX_HINTS = {
+    "ruff": "ruff check --fix {path}",
+    "ruff-format": "ruff format {path}",
+    "eslint": "eslint --fix {path}",
+    "prettier": "prettier --write {path}",
+}
+
+
+def _fix_hint(check_id, path):
+    template = _FIX_HINTS.get(check_id)
+    if not template or not path:
+        return None
+    return template.format(path=path)
+
+
+def _print_header(anchor, num_files, mode):
+    print("=" * 70)
+    print("  DiffImpactScout Guard")
+    print(
+        "  Diff Base: %s | %d changed file(s) | %s mode"
+        % (anchor or "none", num_files, mode)
+    )
+    print("=" * 70)
+
+
+def _print_check_result(check, result):
+    if result.has_issues():
+        print("  [FAIL]   %s" % check.id)
+        for issue in result.issues:
+            print("      %s" % issue.format())
+        hint = _fix_hint(check.id, result.issues[0].path)
+        if hint:
+            print("      --> Fix with: %s" % hint)
+    elif result.fixed:
+        print("  [FIXED]  %s" % check.id)
+        for path in result.fixed:
+            print("      [fixed] %s" % path)
+    elif result.warned:
+        print("  [SKIP]   %s" % check.id)
+    else:
+        print("  [PASS]   %s" % check.id)
+
+
+def _print_summary(ran, total_issues, total_fixed):
+    print("-" * 70)
+    print(
+        "  diffimpactscout: %d check(s), %d issue(s), %d fixed"
+        % (ran, total_issues, total_fixed)
+    )
+
+
+def _print_verdict(blocked, total_issues, mode, always_blocked):
+    if blocked:
+        if always_blocked:
+            verdict = (
+                "[BLOCKED] push rejected (%s mode; hard-block check: %d issue(s))"
+                % (mode, total_issues)
+            )
+        else:
+            verdict = "[BLOCKED] push rejected (strict mode: %d issue(s))" % total_issues
+    elif total_issues:
+        verdict = "[ALLOWED] push allowed (%s mode: %d issue(s) reported)" % (
+            mode,
+            total_issues,
+        )
+    else:
+        verdict = "[ALLOWED] push allowed (%s mode: no issues)" % mode
+    print("  Verdict: %s" % verdict)
+    print("=" * 70)
+
+
 def run_guard(root, cfg, staged=False, all_files=False, files=None):
     if _skip_requested():
         return 0
@@ -67,6 +139,8 @@ def run_guard(root, cfg, staged=False, all_files=False, files=None):
         anchor = dev_scope.scope_base(from_ref, to_ref)
         file_set = dev_scope.dev_files(anchor, from_ref, to_ref)
 
+    file_set = [p for p in file_set if not is_excluded(p, cfg, root)]
+
     mode = _guard_mode(cfg)
     ctx = CheckContext(
         root=root,
@@ -78,9 +152,13 @@ def run_guard(root, cfg, staged=False, all_files=False, files=None):
         echo=True,
     )
 
+    _print_header(anchor, len(file_set), mode)
+
     ran = 0
     total_issues = 0
+    total_fixed = 0
     blocked = False
+    always_blocked = False
 
     for entry in cfg.get("guard", {}).get("checks") or []:
         check = None
@@ -96,21 +174,20 @@ def run_guard(root, cfg, staged=False, all_files=False, files=None):
         check_files = [] if check.scoped == "repo" else file_set
         try:
             result = check.run(ctx, check_files)
-            for issue in result.issues:
-                print(issue.format())
-                total_issues += 1
-            for path in result.fixed:
-                print("[fixed] %s" % path)
+            total_issues += len(result.issues)
+            total_fixed += len(result.fixed)
             for text in result.warned:
                 _warn(text)
             should_block = check.always_block or (check.blocking and mode == "strict")
             if should_block and result.has_issues():
                 blocked = True
+                if check.always_block:
+                    always_blocked = True
+            _print_check_result(check, result)
         except Exception as exc:
             _warn("check %r failed: %s" % (check.id, exc))
             continue
 
-    print(
-        "diffimpactscout: %d check(s), %d issue(s)" % (ran, total_issues)
-    )
+    _print_summary(ran, total_issues, total_fixed)
+    _print_verdict(blocked, total_issues, mode, always_blocked)
     return 1 if blocked else 0

{diffimpactscout-0.2.0 → diffimpactscout-0.3.0}/src/diffimpactscout/impact/impact.py

@@ -30,7 +30,7 @@ _ENTITY_KINDS = {
 _DEFAULT_KINDS = ("name",)
 
 
-def run_impact(root, cfg, staged=False, fast=False, json_out=False):
+def run_impact(root, cfg, staged=False, fast=False, json_out=False, markdown=False):
     if _skip_requested():
         return 0
     impact_cfg = cfg.get("impact") or {}
@@ -82,7 +82,9 @@ def run_impact(root, cfg, staged=False, fast=False, json_out=False):
             + "\n"
         )
     else:
-        sys.stdout.write(reporter.render_report(rows, unresolved, changed_count))
+        sys.stdout.write(
+            reporter.render_report(rows, unresolved, changed_count, markdown=markdown)
+        )
     strict_env = os.environ.get("IMPACT_CHECK_STRICT")
     tty = reporter.interactive_tty()
     block, reason = reporter.should_block(strict_env, tty)
@@ -174,6 +176,9 @@ def _compose_rows(root, impact_cfg, profile, entities, analyses, changed_paths,
         row["severity"] = reporter.classify_severity(
             layers.get(name) or (), deleted, changed_paths, row.get("path")
         )
+        row["reason"] = reporter.severity_reason(
+            layers.get(name) or (), deleted, changed_paths, row.get("path")
+        )
         row["action"] = _action(row["severity"], deleted)
     return rows, unresolved
 

diffimpactscout-0.3.0/src/diffimpactscout/impact/reporter.py

@@ -0,0 +1,191 @@
+"""Render the impact report, classify severity, and decide whether a push should be blocked.
+
+The terminal/stdout default is a plain aligned ASCII table that matches the
+guard report's visual style; a Markdown table is available via an explicit
+`--markdown` output mode for piping into PR comments or docs pages.
+
+Example: a changed view referenced from a template and a frontend service
+gets High severity, while a reference inside the changed file gets Low.
+"""
+
+import sys
+
+import diffimpactscout.env as env
+
+_HEADER = "| # | Impacted File Path | Module / Subsystem | Category | Detected Reference / Usage | Severity | Action Required |"
+_SEPARATOR = "| --- | --- | --- | --- | --- | --- | --- |"
+_COLUMNS = ("#", "Impacted File Path", "Module / Subsystem", "Category", "Detected Reference / Usage", "Severity", "Action Required")
+
+
+def classify_severity(layers, deleted_renamed, changed_paths, ref_path):
+    if deleted_renamed:
+        return "High"
+    if len(set(layers or ())) >= 2:
+        return "High"
+    if ref_path in (changed_paths or ()):
+        return "Low"
+    if not ref_path or not changed_paths:
+        return "Low"
+    return "Medium"
+
+
+def severity_reason(layers, deleted_renamed, changed_paths, ref_path):
+    if deleted_renamed:
+        return "symbol was deleted or renamed in this change-set"
+    layers = set(layers or ())
+    if len(layers) >= 2:
+        return "referenced from multiple layers (%s)" % " + ".join(sorted(layers))
+    if ref_path in (changed_paths or ()):
+        return "reference is inside the changed file itself"
+    if not ref_path or not changed_paths:
+        return "no external reference beyond the change-set"
+    return "referenced from a file outside the change-set"
+
+
+def _cell(row, key):
+    if isinstance(row, dict):
+        value = row.get(key)
+    else:
+        value = getattr(row, key, None)
+    if value is None:
+        return ""
+    return str(value)
+
+
+def _row_values(row, index):
+    return [str(index)] + [_cell(row, k) for k in ("path", "module", "category", "ref", "severity", "action")]
+
+
+def _counts(rows):
+    counts = {"High": 0, "Medium": 0, "Low": 0}
+    for row in rows:
+        sev = _cell(row, "severity")
+        if sev in counts:
+            counts[sev] += 1
+    return counts
+
+
+def _summary_line(rows, changed_count):
+    counts = _counts(rows or [])
+    return "Summary: High: %d, Medium: %d, Low: %d (%d changed file(s))" % (
+        counts["High"],
+        counts["Medium"],
+        counts["Low"],
+        changed_count,
+    )
+
+
+def _rationale_lines(rows):
+    lines = []
+    for i, row in enumerate(rows or [], start=1):
+        reason = _cell(row, "reason")
+        if not reason:
+            continue
+        label = "%s -> %s" % (_cell(row, "path") or "-", _cell(row, "ref") or "-")
+        lines.append("    %d. %s - %s (%s)" % (i, _cell(row, "severity") or "?", label, reason))
+    return lines
+
+
+def _ascii_table(rows):
+    values = [_row_values(row, i) for i, row in enumerate(rows or [], start=1)]
+    widths = []
+    for i in range(len(_COLUMNS)):
+        width = len(_COLUMNS[i])
+        for vals in values:
+            width = max(width, len(vals[i]))
+        widths.append(width)
+    header = "  " + "  ".join(c.ljust(widths[i]) for i, c in enumerate(_COLUMNS))
+    body = []
+    for vals in values:
+        body.append("  " + "  ".join(v.ljust(widths[i]) for i, v in enumerate(vals)))
+    return header, body
+
+
+def _render_ascii(rows, unresolved, changed_count):
+    lines = []
+    lines.append("=" * 70)
+    lines.append("  Impact Analysis Report")
+    lines.append("  Blast radius: %d changed file(s)" % changed_count)
+    lines.append("=" * 70)
+    lines.append("")
+    header, body = _ascii_table(rows)
+    lines.append(header)
+    for line in body:
+        lines.append(line)
+    if unresolved:
+        lines.append("")
+        lines.append("  Unresolved references (manual check required):")
+        for ref in unresolved:
+            lines.append("    - %s" % str(ref))
+    rationale = _rationale_lines(rows)
+    if rationale:
+        lines.append("")
+        lines.append("  Severity rationale:")
+        lines.extend(rationale)
+    lines.append("")
+    lines.append("-" * 70)
+    lines.append("  " + _summary_line(rows, changed_count))
+    lines.append("=" * 70)
+    return "\n".join(lines) + "\n"
+
+
+def _render_markdown(rows, unresolved, changed_count):
+    lines = []
+    lines.append("# Impact Analysis Report")
+    lines.append("")
+    lines.append(_HEADER)
+    lines.append(_SEPARATOR)
+    for i, row in enumerate(rows or [], start=1):
+        cells = [str(i)] + [_cell(row, k).replace("|", "\\|") for k in ("path", "module", "category", "ref", "severity", "action")]
+        lines.append("| " + " | ".join(cells) + " |")
+    if unresolved:
+        lines.append("")
+        lines.append("## Unresolved references (manual check required)")
+        for ref in unresolved:
+            lines.append("- %s" % str(ref))
+    rationale = _rationale_lines(rows)
+    if rationale:
+        lines.append("")
+        lines.append("## Severity rationale")
+        for line in rationale:
+            lines.append("- " + line.lstrip())
+    lines.append("")
+    lines.append(_summary_line(rows, changed_count))
+    return "\n".join(lines) + "\n"
+
+
+def render_report(rows, unresolved, changed_count, markdown=False):
+    if markdown:
+        return _render_markdown(rows, unresolved, changed_count)
+    return _render_ascii(rows, unresolved, changed_count)
+
+
+def interactive_tty():
+    return sys.stdin.isatty()
+
+
+def confirm(prompt_text):
+    sys.stdout.write(prompt_text + " ")
+    sys.stdout.flush()
+    answer = (sys.stdin.readline() or "").strip().lower()
+    if answer in ("", "y", "yes"):
+        return True
+    return False
+
+
+def _is_true(value):
+    return env.is_true(value)
+
+
+def _skip_requested():
+    return env.skip_requested()
+
+
+def should_block(strict_env, tty):
+    if _skip_requested():
+        return (False, "skip env var set; impact check skipped")
+    if tty:
+        return (True, "interactive; prompt user to proceed")
+    if _is_true(strict_env):
+        return (True, "strict mode; blocking in non-interactive context")
+    return (False, "non-interactive; report shown as warning, commit not blocked")

{diffimpactscout-0.2.0 → diffimpactscout-0.3.0/src/diffimpactscout.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: diffimpactscout
-Version: 0.2.0
+Version: 0.3.0
 Summary: Incremental pre-push guard and AST blast-radius impact analyzer for git repositories.
 Author: DiffImpactScout contributors
 License: MIT

{diffimpactscout-0.2.0 → diffimpactscout-0.3.0}/tests/test_fixture_smoke.py

@@ -89,11 +89,13 @@ CHECKERS = {
 
 
 def test_gitrepo_scenarios(gitrepo):
+    """Verifies that the gitrepo fixture is valid and matches its expected scenario state."""
     assert gitrepo.git_ok("rev-parse", "--verify", "HEAD")
     CHECKERS[gitrepo.scenario](gitrepo)
 
 
 def test_repo_helpers_rm_mv_merge(tmp_path):
+    """Verifies that the GitRepo helper methods for merge, rm, mv, and commit behave correctly."""
     repo = GitRepo.init(str(tmp_path / "repo"))
     repo.write("a.txt", "a\n")
     repo.commit("A")

diffimpactscout-0.2.0/src/diffimpactscout/impact/reporter.py

@@ -1,107 +0,0 @@
-"""Render the impact report table, classify severity, and decide whether a push should be blocked.
-
-Example: a changed view referenced from a template and a frontend service
-gets High severity, while a reference inside the changed file gets Low.
-"""
-
-import sys
-
-import diffimpactscout.env as env
-
-_HEADER = "| # | Impacted File Path | Module / Subsystem | Category | Detected Reference / Usage | Severity | Action Required |"
-_SEPARATOR = "| --- | --- | --- | --- | --- | --- | --- |"
-
-
-def classify_severity(layers, deleted_renamed, changed_paths, ref_path):
-    if deleted_renamed:
-        return "High"
-    if len(set(layers or ())) >= 2:
-        return "High"
-    if ref_path in (changed_paths or ()):
-        return "Low"
-    if not ref_path or not changed_paths:
-        return "Low"
-    return "Medium"
-
-
-def _cell(row, key):
-    if isinstance(row, dict):
-        value = row.get(key)
-    else:
-        value = getattr(row, key, None)
-    if value is None:
-        return ""
-    return str(value).replace("|", "\\|")
-
-
-def _counts(rows):
-    counts = {"High": 0, "Medium": 0, "Low": 0}
-    for row in rows:
-        sev = _cell(row, "severity")
-        if sev in counts:
-            counts[sev] += 1
-    return counts
-
-
-def render_report(rows, unresolved, changed_count):
-    lines = []
-    lines.append("# Impact Analysis Report")
-    lines.append("")
-    lines.append(_HEADER)
-    lines.append(_SEPARATOR)
-    for i, row in enumerate(rows or [], start=1):
-        lines.append(
-            "| %d | %s | %s | %s | %s | %s | %s |"
-            % (
-                i,
-                _cell(row, "path"),
-                _cell(row, "module"),
-                _cell(row, "category"),
-                _cell(row, "ref"),
-                _cell(row, "severity"),
-                _cell(row, "action"),
-            )
-        )
-    if unresolved:
-        lines.append("")
-        lines.append("## Unresolved references (manual check required)")
-        for ref in unresolved:
-            lines.append("- %s" % str(ref))
-    lines.append("")
-    counts = _counts(rows or [])
-    lines.append(
-        "Summary: %d changed file(s); High: %d, Medium: %d, Low: %d"
-        % (changed_count, counts["High"], counts["Medium"], counts["Low"])
-    )
-    return "\n".join(lines) + "\n"
-
-
-def interactive_tty():
-    return sys.stdin.isatty()
-
-
-def confirm(prompt_text):
-    sys.stdout.write(prompt_text + " ")
-    sys.stdout.flush()
-    answer = (sys.stdin.readline() or "").strip().lower()
-    if answer in ("", "y", "yes"):
-        return True
-    return False
-
-
-def _is_true(value):
-    return env.is_true(value)
-
-
-def _skip_requested():
-    return env.skip_requested()
-
-
-def should_block(strict_env, tty):
-    if _skip_requested():
-        return (False, "skip env var set; impact check skipped")
-    if tty:
-        return (True, "interactive; prompt user to proceed")
-    if _is_true(strict_env):
-        return (True, "strict mode; blocking in non-interactive context")
-    return (False, "non-interactive; report shown as warning, commit not blocked")