maintainability-agent 2.4.1__tar.gz → 2.6.0__tar.gz

{maintainability_agent-2.4.1/src/maintainability_agent.egg-info → maintainability_agent-2.6.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: maintainability-agent
-Version: 2.4.1
+Version: 2.6.0
 Summary: Maintainability gate and AI remediation prompt generator for CI.
 Author: Marshall Guillory
 License: MIT
@@ -44,7 +44,7 @@ Dynamic: license-file
 **A deterministic, offline maintainability audit whose output is a _bounded
 work order_ for an AI coding agent** — a copy-paste prompt, per finding, that
 says *fix exactly these and refactor nothing else*. Chat-primary; CLI for CI.
-Version **2.4.1**.
+Version **2.6.0**.
 
 **Languages parsed:** Python, Java, C, C++, C#, Swift, Fortran (free-form
 *and* fixed-form), and the JS/TS family — each by a scanner written for it, and
@@ -152,9 +152,8 @@ registered verdict stands at **INCONCLUSIVE**. Method, limits and raw data:
 every claim the project could not stand behind was removed. A scoring engine
 rebuilt after it was found to grade repo *size*; a headline claim about AI
 authorship retracted against a matched control; a score that is withheld rather
-than guessed. That arc is the credibility, and it is recorded in full — with the
-figures quoted from their approved summaries — in
-[the track record](docs/track-record.md).
+than guessed. That arc is the credibility, and it is recorded in full, with the
+figures quoted from their approved summaries, in [the track record](docs/track-record.md).
 
 ## Install
 
@@ -223,8 +222,8 @@ withheld and the scope is named as the reason.
 ### Checking that the agent did the work
 
 The work order tells an agent to fix exactly these findings and refactor nothing
-else. These flags check that the diff that came back obeyed it — read the
-diff's _shape_, not its correctness:
+else. These flags check that the diff obeyed it — they read its _shape_, not its
+correctness:
 
 ```bash
 maintainability-agent \
@@ -251,6 +250,11 @@ maintainability-agent \
   work order, and what moved. It is reproducible and **not signed** — nothing
   here holds a key, and the document says so in its own text. A check nobody ran
   renders as _not asked_, never as passed.
+- **`--transformation NAME`** names the class of work a scan followed and
+  reports how this run of it compares with earlier ones. **A report, never a
+  gate.** It says one run "moved further" than another, never that it was
+  _better_ — two runs of one codemod land on different code — and it measures
+  the interval, not the transformation.
 
 The [roadmap](docs/roadmap.md#the-remediation-hole-closed-in-210-through-230)
 has the history of why this was the hole under the product's central claim.

{maintainability_agent-2.4.1 → maintainability_agent-2.6.0}/README.md

@@ -5,7 +5,7 @@
 **A deterministic, offline maintainability audit whose output is a _bounded
 work order_ for an AI coding agent** — a copy-paste prompt, per finding, that
 says *fix exactly these and refactor nothing else*. Chat-primary; CLI for CI.
-Version **2.4.1**.
+Version **2.6.0**.
 
 **Languages parsed:** Python, Java, C, C++, C#, Swift, Fortran (free-form
 *and* fixed-form), and the JS/TS family — each by a scanner written for it, and
@@ -113,9 +113,8 @@ registered verdict stands at **INCONCLUSIVE**. Method, limits and raw data:
 every claim the project could not stand behind was removed. A scoring engine
 rebuilt after it was found to grade repo *size*; a headline claim about AI
 authorship retracted against a matched control; a score that is withheld rather
-than guessed. That arc is the credibility, and it is recorded in full — with the
-figures quoted from their approved summaries — in
-[the track record](docs/track-record.md).
+than guessed. That arc is the credibility, and it is recorded in full, with the
+figures quoted from their approved summaries, in [the track record](docs/track-record.md).
 
 ## Install
 
@@ -184,8 +183,8 @@ withheld and the scope is named as the reason.
 ### Checking that the agent did the work
 
 The work order tells an agent to fix exactly these findings and refactor nothing
-else. These flags check that the diff that came back obeyed it — read the
-diff's _shape_, not its correctness:
+else. These flags check that the diff obeyed it — they read its _shape_, not its
+correctness:
 
 ```bash
 maintainability-agent \
@@ -212,6 +211,11 @@ maintainability-agent \
   work order, and what moved. It is reproducible and **not signed** — nothing
   here holds a key, and the document says so in its own text. A check nobody ran
   renders as _not asked_, never as passed.
+- **`--transformation NAME`** names the class of work a scan followed and
+  reports how this run of it compares with earlier ones. **A report, never a
+  gate.** It says one run "moved further" than another, never that it was
+  _better_ — two runs of one codemod land on different code — and it measures
+  the interval, not the transformation.
 
 The [roadmap](docs/roadmap.md#the-remediation-hole-closed-in-210-through-230)
 has the history of why this was the hole under the product's central claim.

{maintainability_agent-2.4.1 → maintainability_agent-2.6.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "maintainability-agent"
-version = "2.4.1"
+version = "2.6.0"
 description = "Maintainability gate and AI remediation prompt generator for CI."
 readme = "README.md"
 # 3.11, not 3.10. `_discovery`, `_pillars` and `_runner` all import

{maintainability_agent-2.4.1 → maintainability_agent-2.6.0/src/maintainability_agent.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: maintainability-agent
-Version: 2.4.1
+Version: 2.6.0
 Summary: Maintainability gate and AI remediation prompt generator for CI.
 Author: Marshall Guillory
 License: MIT
@@ -44,7 +44,7 @@ Dynamic: license-file
 **A deterministic, offline maintainability audit whose output is a _bounded
 work order_ for an AI coding agent** — a copy-paste prompt, per finding, that
 says *fix exactly these and refactor nothing else*. Chat-primary; CLI for CI.
-Version **2.4.1**.
+Version **2.6.0**.
 
 **Languages parsed:** Python, Java, C, C++, C#, Swift, Fortran (free-form
 *and* fixed-form), and the JS/TS family — each by a scanner written for it, and
@@ -152,9 +152,8 @@ registered verdict stands at **INCONCLUSIVE**. Method, limits and raw data:
 every claim the project could not stand behind was removed. A scoring engine
 rebuilt after it was found to grade repo *size*; a headline claim about AI
 authorship retracted against a matched control; a score that is withheld rather
-than guessed. That arc is the credibility, and it is recorded in full — with the
-figures quoted from their approved summaries — in
-[the track record](docs/track-record.md).
+than guessed. That arc is the credibility, and it is recorded in full, with the
+figures quoted from their approved summaries, in [the track record](docs/track-record.md).
 
 ## Install
 
@@ -223,8 +222,8 @@ withheld and the scope is named as the reason.
 ### Checking that the agent did the work
 
 The work order tells an agent to fix exactly these findings and refactor nothing
-else. These flags check that the diff that came back obeyed it — read the
-diff's _shape_, not its correctness:
+else. These flags check that the diff obeyed it — they read its _shape_, not its
+correctness:
 
 ```bash
 maintainability-agent \
@@ -251,6 +250,11 @@ maintainability-agent \
   work order, and what moved. It is reproducible and **not signed** — nothing
   here holds a key, and the document says so in its own text. A check nobody ran
   renders as _not asked_, never as passed.
+- **`--transformation NAME`** names the class of work a scan followed and
+  reports how this run of it compares with earlier ones. **A report, never a
+  gate.** It says one run "moved further" than another, never that it was
+  _better_ — two runs of one codemod land on different code — and it measures
+  the interval, not the transformation.
 
 The [roadmap](docs/roadmap.md#the-remediation-hole-closed-in-210-through-230)
 has the history of why this was the hole under the product's central claim.

{maintainability_agent-2.4.1 → maintainability_agent-2.6.0}/src/maintainability_agent.egg-info/SOURCES.txt

@@ -11,6 +11,7 @@ src/maintainability_audit/__init__.py
 src/maintainability_audit/__main__.py
 src/maintainability_audit/_adapters.py
 src/maintainability_audit/_analysis.py
+src/maintainability_audit/_arguments.py
 src/maintainability_audit/_aspects.py
 src/maintainability_audit/_attestation.py
 src/maintainability_audit/_backfill.py
@@ -67,6 +68,7 @@ src/maintainability_audit/_ranges_js.py
 src/maintainability_audit/_ranges_swift.py
 src/maintainability_audit/_ratchet.py
 src/maintainability_audit/_recurrence.py
+src/maintainability_audit/_run_comparison.py
 src/maintainability_audit/_runner.py
 src/maintainability_audit/_safe_write.py
 src/maintainability_audit/_scan_history.py
@@ -82,6 +84,7 @@ src/maintainability_audit/_setup_persist.py
 src/maintainability_audit/_skill_install.py
 src/maintainability_audit/_stored_grants.py
 src/maintainability_audit/_tdd_view.py
+src/maintainability_audit/_test_commands.py
 src/maintainability_audit/_test_execution.py
 src/maintainability_audit/_test_pairing.py
 src/maintainability_audit/_tokens.py
@@ -273,6 +276,7 @@ tests/test_report_remedy_copy.py
 tests/test_roles_recorded.py
 tests/test_root_grants.py
 tests/test_rubric_ignores_project_lint_config.py
+tests/test_run_comparison.py
 tests/test_runner.py
 tests/test_sarif.py
 tests/test_scan_history.py
@@ -293,6 +297,7 @@ tests/test_stage8_contract.py
 tests/test_suite_isolation.py
 tests/test_swift_declarations.py
 tests/test_tdd_structure.py
+tests/test_test_commands.py
 tests/test_test_execution_class.py
 tests/test_three_presentations.py
 tests/test_trends.py

{maintainability_agent-2.4.1 → maintainability_agent-2.6.0}/src/maintainability_audit/__init__.py

@@ -1,3 +1,3 @@
 """Maintainability Audit CI."""
 
-__version__ = "2.4.1"
+__version__ = "2.6.0"

maintainability_agent-2.6.0/src/maintainability_audit/_arguments.py

@@ -0,0 +1,186 @@
+"""What the command line accepts, separate from what the command does.
+
+Split out of `cli` when it crossed this project's own `max_file_lines`
+gate — the second time that gate has shaped this module, after
+`_add_gate_arguments` was split from `add_arguments` for the function
+budget. Both splits were forced by the tool on a change to the tool, which
+is the gate doing the job it exists for rather than an inconvenience.
+
+The boundary is deliberate and not merely a line count: everything here
+declares a flag and nothing here runs an audit. `cli` imports it, never
+the reverse.
+"""
+from __future__ import annotations
+
+import argparse
+
+from .config import VERSION
+
+
+def _add_gate_arguments(parser: argparse.ArgumentParser) -> None:
+    """What can make a run fail, and the conformance record behind one.
+
+    Split from `add_arguments` when the conformance flags took it past this
+    project's own function-length gate — which is the gate doing its job on
+    the change that added a gate.
+    """
+    parser.add_argument("--fail-on-gate", action="store_true", help="Exit 1 when hard gates fail.")
+    parser.add_argument(
+        "--conformance",
+        metavar="REVSPEC",
+        help="Report how a diff relates to the work order, e.g. `main...HEAD`. "
+             "The bounded work order says 'fix exactly these and refactor nothing "
+             "else'; without this the bound is an instruction nothing checks. "
+             "Reports only unless --fail-on-out-of-scope is passed.",
+    )
+    parser.add_argument(
+        "--fail-on-regression", action="store_true",
+        help="Exit 1 when any scoring category is lower than the previous "
+             "comparable scan. Reads the recorded history, and refuses to "
+             "compare across an instrument change rather than reporting a "
+             "false regression: two scans taken under different calibration "
+             "cannot be subtracted.",
+    )
+    parser.add_argument(
+        "--fail-on-out-of-scope", action="store_true",
+        help="With --conformance, exit 1 when the diff touched files the work "
+             "order did not name and that do not pair to one as its test, or "
+             "when it added a suppression to a file the work order flagged.",
+    )
+
+
+def _add_artifact_arguments(parser: argparse.ArgumentParser) -> None:
+    """The documents a run can write beside its report, and the run's label.
+
+    Split out for the same reason `_add_gate_arguments` was, and by the same
+    gate: `--transformation` took `add_arguments` past this project's own
+    function-length threshold, so the gate failed the change that added a
+    reporting flag. Grouped by what they produce rather than alphabetically
+    — every one of these is an artifact for somebody other than the person
+    running the command.
+    """
+    parser.add_argument("--prompt-output", help="Optional Markdown prompt for AI-assisted remediation.")
+    parser.add_argument("--comment-output", help="Optional Markdown body suitable for a PR comment.")
+    parser.add_argument("--agent-instructions-output", help="Optional reusable instructions for AI coding agents.")
+    parser.add_argument(
+        "--attestation-output",
+        help="Write the attestation: an independent, reproducible record of "
+             "what was measured, what the change was told to do, what it did, "
+             "and what moved. Populated by --conformance and "
+             "--fail-on-regression; a generator cannot produce this about its "
+             "own output.",
+    )
+    parser.add_argument(
+        "--transformation",
+        metavar="NAME",
+        help="Name the class of work this scan followed, e.g. `react-18`. "
+             "Records the name on the scan and reports how this run of it "
+             "compares with earlier ones. Nothing in a tree says which "
+             "transformation produced it, so this is your claim and the tool "
+             "records it without verifying it. A report, never a gate.",
+    )
+    parser.add_argument("--hostile-prompt-output",
+                        help="Optional adversarial audit brief seeded from this run (ADR 013). "
+                             "Text only: it does not gate, score, or send anything.")
+
+
+def add_arguments(parser: argparse.ArgumentParser) -> None:
+    parser.add_argument("--version", action="version", version=f"maintainability-agent {VERSION}")
+    parser.add_argument("--root", default=".", help="Repository root to scan.")
+    parser.add_argument("--config", help="Path to JSON config.")
+    # Default None, not "markdown": the TTY question (8.4) only fires
+    # when the user stated nothing, and argparse cannot distinguish a
+    # stated default from an omitted flag.
+    parser.add_argument("--format", choices=["json", "markdown", "html"], default=None)
+    parser.add_argument("--html-output", help="Write the single-file HTML report here (ADR 011).")
+    parser.add_argument("--output", help="Output file. Defaults to stdout.")
+    _add_artifact_arguments(parser)
+    parser.add_argument("--sarif-output", help="Optional SARIF output path for GitHub code scanning.")
+    parser.add_argument("--sarif-input", action="append", help="Optional external SARIF file to summarize in reports. Repeatable.")
+    parser.add_argument("--changed-only", help="Audit only files changed in a git revspec, for example main...HEAD.")
+    parser.add_argument("--baseline", help="Existing baseline JSON. With --fail-on-new, only new findings fail.")
+    parser.add_argument("--write-baseline", help="Write current findings to a baseline JSON file.")
+    parser.add_argument("--fail-on-new", action="store_true", help="Fail only when findings are not in --baseline.")
+    parser.add_argument(
+        "--analyzers", action="store_true",
+        help="Force the external analyzer pool on for this run. The pool is "
+             "the primary evidence source (ADR 006, docs/analyzer-pool.md); a "
+             "repository with a config file runs it by default, so this flag "
+             "matters only for unconfigured trees or to override "
+             "analyzers.run=false. Measurements move the point estimate where "
+             "the full concept set was measured; otherwise the built-in "
+             "fallback stands and the range widens around the two.",
+    )
+    parser.add_argument(
+        "--no-analyzers", action="store_true",
+        help="Force the analyzer pool off for this run, overriding the "
+             "config. The report will say the built-in fallback supplied the "
+             "evidence (P8); it is a faster scan, not the full audit.",
+    )
+    parser.add_argument(
+        "--work", action="append", metavar="AXIS=VALUE",
+        help="Narrow the work order, for example --work band=quick-win or "
+             "--work path=src/. Repeatable; every criterion must match. "
+             "Axes: band, finding_class, path, verification. Narrowing "
+             "changes what is shown and never what anything scored.",
+    )
+    parser.add_argument(
+        "--record-history", action="store_true",
+        help="Append this scan to the history at paths.history (default "
+             ".maintainability/history.jsonl). Opt-in, like every other write "
+             "this tool performs. Once the file exists, every later successful "
+             "scan appends to it and reads the resulting history.",
+    )
+    parser.add_argument(
+        "--backfill", metavar="REVSPEC",
+        help="Scan each commit in a range into the history and exit, for "
+             "example --backfill HEAD~50..HEAD. Each commit is checked out in "
+             "a temporary worktree, so the working tree is never touched. "
+             "Expensive and therefore explicit: it never runs as part of a "
+             "normal scan. Records are marked as reconstructed.",
+    )
+    parser.add_argument(
+        "--backfill-interval", type=int, default=1, metavar="N",
+        help="With --backfill, scan every Nth commit instead of all of them. "
+             "The shape of a series is what a trend reads, and a thousand "
+             "commits is hours of work nobody asked for.",
+    )
+    _add_gate_arguments(parser)
+    _add_setup_actions(parser)
+
+
+def _add_setup_actions(parser: argparse.ArgumentParser) -> None:
+    """Flags that perform a setup action and exit instead of auditing."""
+    parser.add_argument("--init-agent-standards", action="store_true", help="Write model/tool-specific instruction files and exit.")
+    parser.add_argument(
+        "--install-skill", action="store_true",
+        help="Copy the packaged agent skill into the skills directory and "
+             "exit. Re-run after every upgrade: an installed skill that "
+             "drifts from the shipped one teaches agents a dead workflow.",
+    )
+    parser.add_argument(
+        "--skills-dir", default=None, metavar="DIR",
+        help="Where --install-skill writes (default: ~/.claude/skills).",
+    )
+    parser.add_argument(
+        "--force-skill", action="store_true",
+        help="With --install-skill: overwrite a differing installed copy "
+             "and remove files the package no longer ships. Without it, a "
+             "differing copy is refused with the list of differences.",
+    )
+    parser.add_argument(
+        "--target-path",
+        action="append",
+        metavar="TARGET=PATH",
+        help="Where a target's standing instructions go, e.g. "
+             "`--target-path bob=.bob/instructions.md`. Required for any agent "
+             "this tool has no built-in convention for; overrides the built-in "
+             "path for one it does. The target list will always trail the "
+             "market, and guessing a path writes a file the agent never opens.",
+    )
+    parser.add_argument(
+        "--target",
+        action="append",
+        help="Instruction target. Repeatable. Used with --init-agent-standards.",
+    )
+    parser.add_argument("--instructions-output-dir", default=".", help="Directory for generated instruction files.")

{maintainability_agent-2.4.1 → maintainability_agent-2.6.0}/src/maintainability_audit/_first_run.py

@@ -22,6 +22,7 @@ sat in ``DEFAULTS``, documented as reserved, read by nothing, from
 """
 from __future__ import annotations
 
+import contextlib
 import json
 import sys
 from pathlib import Path
@@ -190,11 +191,53 @@ def maybe_prompt_economics(root: Path, config: dict) -> None:
     print(f"Wrote {written}")
 
 
+def _input_with_default(prompt: str, default: str) -> str:
+    """`input()` with the line pre-filled, where the terminal allows it.
+
+    `readline` is what makes the default *editable* rather than merely
+    described: the text is inserted into the line buffer, so Enter submits
+    it, backspace edits it, and clearing it returns the empty string that
+    has always meant "cancel". Without that, a pre-filled default and a
+    blank-means-cancel rule cannot both be true on one keystroke.
+
+    Absent or unusable `readline` — Windows without pyreadline, a terminal
+    that refuses the hook — falls back to a plain `input()`. The default is
+    still named in the prompt text, so the operator loses the keystroke and
+    not the information. Never let this be the thing that fails a setup.
+    """
+    if not default:
+        return input(prompt)
+    try:
+        import readline
+    except ImportError:
+        return input(prompt)
+
+    def _prefill() -> None:
+        readline.insert_text(default)
+        readline.redisplay()
+
+    try:
+        readline.set_startup_hook(_prefill)
+        return input(prompt)
+    except Exception:  # noqa: BLE001 - a cosmetic hook must not fail setup
+        return input(prompt)
+    finally:
+        # Global state: a hook left installed would pre-type this command
+        # into whatever the session asks next.
+        with contextlib.suppress(Exception):
+            readline.set_startup_hook()
+
+
 def maybe_prompt_test_command(root: Path, config: dict) -> None:
     """Ask the opted-in operator for the test command, once -- the CLI half
     of the Class 5 second stage, mirroring the labor-rate ask. Fires only at
     a TTY, only when the suite was opted into and no command is stored, and
     a blank answer cancels the opt-in. Nothing runs until a command exists.
+
+    Where the tree's manifests name a runner, the line is **pre-filled**
+    with that command rather than merely mentioning it, so this surface and
+    the MCP one carry the same default in the same field: Enter accepts,
+    editing replaces, and clearing the line cancels exactly as before.
     """
     import shlex
 
@@ -209,10 +252,19 @@ def maybe_prompt_test_command(root: Path, config: dict) -> None:
     if (config.get("expected_commands") or {}).get("test"):
         return
 
-    answer = input(
+    from ._test_commands import suggested_test_command
+
+    suggested = suggested_test_command(root)
+    detected = " ".join(suggested.command) if suggested is not None else ""
+    prompt = (
+        f"Detected `{detected}` from {suggested.evidence}. The command that "
+        "runs this repository's test suite (Enter to accept, clear the line "
+        "to cancel running it): "
+        if suggested is not None else
         "The command that runs this repository's test suite, e.g. `pytest` "
         "(Enter to cancel running it): "
-    ).strip()
+    )
+    answer = _input_with_default(prompt, detected).strip()
 
     from ._safe_write import write_bounded
 

{maintainability_agent-2.4.1 → maintainability_agent-2.6.0}/src/maintainability_audit/_gates.py

@@ -51,18 +51,42 @@ def attach_ratchet(report: dict, history_path: Path) -> None:
     report["dimension_ratchet"] = dimension_ratchet(read_history(history_path))
 
 
+def attach_run_comparison(
+    args: argparse.Namespace, report: dict, history_path: Path
+) -> None:
+    """Compare this run of a named transformation with earlier ones.
+
+    Only when the operator named one: an unnamed run belongs to no series,
+    and inventing a series for it would be the tool answering a question
+    nobody asked.
+    """
+    label = getattr(args, "transformation", None)
+    if not label or not history_path.exists():
+        return
+    from ._run_comparison import compare_runs
+    from ._scan_history import read_history
+
+    report["run_comparison"] = compare_runs(read_history(history_path), label)
+
+
 def _attach_post_audit_records(
     args: argparse.Namespace, report: dict, history_path: Path
 ) -> None:
     """Records composed from the finished report, before it is rendered.
 
-    Both read the report rather than the tree, and neither reaches scoring:
-    whether a diff obeyed its work order, and whether a dimension slipped
-    since the last comparable scan, are facts about a change and its
-    history — not evidence about the code's condition.
+    All three read the report or the history rather than the tree, and none
+    reaches scoring: whether a diff obeyed its work order, whether a
+    dimension slipped since the last comparable scan, and how this run of a
+    transformation compares with earlier ones are facts about a change and
+    its history — not evidence about the code's condition.
+
+    The third is the only one that cannot fail a build. It is a report, by
+    design: "run seven moved further than run six" is worth knowing and is
+    not a thing to block a merge on.
     """
     attach_conformance(args, report)
     attach_ratchet(report, history_path)
+    attach_run_comparison(args, report, history_path)
 
 
 def _conformance_exit(args: argparse.Namespace, report: dict) -> int:

{maintainability_agent-2.4.1 → maintainability_agent-2.6.0}/src/maintainability_audit/_history_view.py

@@ -120,3 +120,49 @@ def escalations_markdown(escalated: list[dict[str, Any]] | None) -> list[str]:
         ])
     lines.append("")
     return lines
+
+
+def run_comparison_markdown(comparison: dict[str, Any] | None) -> list[str]:
+    """The markdown section. Presentation only — it computes no score.
+
+    Empty for a run nobody named, which is most of them.
+    """
+    if not comparison:
+        return []
+    label = comparison["label"]
+    lines = [f"## Runs of `{label}`", ""]
+    runs = comparison["runs"]
+    if not runs:
+        return [*lines, comparison["trend"].get("reason", "No runs recorded."), ""]
+
+    lines += ["| Recorded | Commit | Estimate | Moved | Files | Declarations |",
+              "|---|---|---|---|---|---|"]
+    for run in runs:
+        estimate = "withheld" if run["estimate"] is None else f"{run['estimate']:.2f}"
+        moved = "—" if run["moved"] is None else f"{run['moved']:+.2f}"
+        populations = run["populations"]
+        lines.append(
+            f"| {run['recorded_at']} | `{(run['commit'] or '')[:8]}` | {estimate} | "
+            f"{moved} | {populations.get('files_scanned', '—')} | "
+            f"{populations.get('declarations_scanned', '—')} |"
+        )
+    lines.append("")
+
+    trend = comparison["trend"]
+    lines.append(
+        f"**{trend['summary']}.**" if trend.get("comparable")
+        else f"**Not established:** {trend['reason']}."
+    )
+    if comparison.get("excluded_earlier_runs"):
+        lines += ["", (
+            f"{comparison['excluded_earlier_runs']} earlier run(s) under this "
+            f"name are excluded: {comparison['exclusion_reason']}."
+        )]
+    lines += ["", (
+        "*`Moved` is the change in estimate between the previous recorded "
+        "scan and this one — the movement across that interval, not the "
+        "effect of the transformation. Anything else that happened in the "
+        "same interval is inside the number, and two runs of one "
+        "transformation land on different code.*"
+    ), ""]
+    return lines

{maintainability_agent-2.4.1 → maintainability_agent-2.6.0}/src/maintainability_audit/_mcp_setup.py

@@ -226,21 +226,35 @@ def run_tests_pending(root: Path) -> bool:
             and "test_execution" not in stored)
 
 
-def test_command_questions() -> list[dict[str, Any]]:
+def test_command_questions(root: Path | None = None) -> list[dict[str, Any]]:
     """The one command, asked only of someone who opted into running it.
 
     A free-text answer (empty options), the second stage of the opt-in the
     way the labor rates are the second stage of the economic scenario.
+
+    When the tree's manifests name a runner, that command is the question's
+    `default` and the prompt says which file proposed it. It is a default,
+    not a decision: the operator's answer is what gets stored, and blank
+    still cancels the opt-in — a pre-filled field the operator clears is
+    the same cancellation it always was.
     """
+    from ._test_commands import suggested_test_command
+
+    suggested = suggested_test_command(root) if root is not None else None
+    detected = " ".join(suggested.command) if suggested is not None else ""
+    because = (
+        f"Detected `{detected}` from {suggested.evidence}. "
+        if suggested is not None else ""
+    )
     return [{
         "name": "test_command",
         "prompt": (
-            "The command that runs this repository's test suite "
+            f"{because}The command that runs this repository's test suite "
             "(e.g. `pytest`, `npm test`). It runs in the repository root, "
             "and leaving it blank cancels the opt-in."
         ),
         "options": [],
-        "default": "",
+        "default": detected,
     }]
 
 
@@ -334,7 +348,7 @@ def setup_schema(root: Path | None = None):
     if root is not None and economics_bounds_pending(root):
         return _schema_for(economics_bound_questions())
     if root is not None and test_command_pending(root):
-        return _schema_for(test_command_questions())
+        return _schema_for(test_command_questions(root))
     return _schema_for(setup_questions(load_config(None)))