specfuse-loop 0.20.1__tar.gz → 0.22.0__tar.gz

{specfuse_loop-0.20.1/specfuse_loop.egg-info → specfuse_loop-0.22.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: specfuse-loop
-Version: 0.20.1
+Version: 0.22.0
 Summary: Local-first executor for the Specfuse Plan + Work Unit gate-cycle methodology.
 Author: Specfuse contributors
 License: Apache-2.0

{specfuse_loop-0.20.1 → specfuse_loop-0.22.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "specfuse-loop"
-version = "0.20.1"
+version = "0.22.0"
 description = "Local-first executor for the Specfuse Plan + Work Unit gate-cycle methodology."
 readme = "README.md"
 requires-python = ">=3.10"
@@ -76,6 +76,7 @@ specfuse-monitor-lint = "specfuse.loop.lint_monitoring:main"
 specfuse-monitor = "specfuse.monitor.cli:main"
 specfuse-stats = "specfuse.loop.events_stats:main"
 specfuse-agent = "specfuse.agent.run:main"
+specfuse-backfill-severity = "specfuse.agent.severity_backfill:main"
 
 [tool.setuptools.packages.find]
 # Ship ONLY the specfuse namespace package. Without this scope, setuptools'

{specfuse_loop-0.20.1 → specfuse_loop-0.22.0}/specfuse/agent/drafting_invoke.py

@@ -27,10 +27,24 @@ from specfuse.agent.drafting_answers import AnswerGateResult, OUTCOME_FALLBACK
 from specfuse.agent.invoke import build_claude_argv
 from specfuse.loop.loop import parse_result_block
 
-__all__ = ("DraftingInvokeError", "build_invocation", "read_result")
+__all__ = (
+    "DraftingInvokeError",
+    "DEFAULT_COMMAND",
+    "build_invocation",
+    "read_result",
+)
 
 _DEFAULT_WORKING_DIR = "."
 
+#: The slash command this lane dispatches. The skill ships in the
+#: `specfuse@specfuse` plugin, where it is addressed `/specfuse:draft-feature`;
+#: the bare `/draft-feature` resolves only in a repository carrying its own
+#: project-level `.claude/skills/draft-feature/`. Same defect, same lane shape
+#: as `autofix_invoke.DEFAULT_COMMAND` -- see #3342, where the bare spelling
+#: made every dispatched item fail with `Unknown command`. A caller whose
+#: project-level copy is the one that resolves passes `command=`.
+DEFAULT_COMMAND = "/specfuse:draft-feature"
+
 
 class DraftingInvokeError(Exception):
     """Raised when a gate result cannot be turned into a drafting session,
@@ -44,9 +58,10 @@ def build_invocation(
     working_dir: str = _DEFAULT_WORKING_DIR,
     model: str = "sonnet",
     effort: str = "medium",
+    command: str = DEFAULT_COMMAND,
 ):
-    """Build argv and prompt text for a headless `/draft-feature` session
-    against one `draft_ready` answer gate result.
+    """Build argv and prompt text for a headless `/specfuse:draft-feature`
+    session against one `draft_ready` answer gate result.
 
     Returns a `(argv, prompt)` tuple; runs nothing. Raises
     `DraftingInvokeError` if `gate_result.outcome` is `OUTCOME_FALLBACK` --
@@ -76,7 +91,7 @@ def build_invocation(
         assumption_lines = "(none -- every question was answered)"
 
     prompt = (
-        f"/draft-feature {feature_id}\n\n"
+        f"{command} {feature_id}\n\n"
         f"Working directory: {working_dir}\n\n"
         "The interview's answer gate has already run and returned "
         "draft_ready. Every question's effective answer -- given or "

{specfuse_loop-0.20.1 → specfuse_loop-0.22.0}/specfuse/agent/providers/bugs.py

@@ -27,7 +27,13 @@ from specfuse.agent.run import (
     _default_runner,
 )
 from specfuse.agent.state import AgentSnapshot
-from specfuse.loop.agent_policy import bug_lane_ci_wait_seconds
+from specfuse.loop.agent_policy import (
+    bug_lane_ci_wait_seconds,
+    meets_severity_floor,
+    read_severity_label,
+    resolve_min_severity,
+    resolve_severity_aliases,
+)
 from specfuse.loop.bug_lane import REASON_CI_PENDING
 from specfuse.loop.bug_lane_run import (
     OUTCOME_AUTOMERGE_OFF,
@@ -85,7 +91,10 @@ def _has_open_pr(snapshot: AgentSnapshot, issue_number: int) -> bool:
     lane's "which PR fixes this issue" are the same question, and two copies
     of the answer is how they drift.
     """
-    return any(pr_closes_issue(pr.body or "", issue_number) for pr in snapshot.prs)
+    return any(
+        pr_closes_issue(pr.body or "", issue_number, title=pr.title or "")
+        for pr in snapshot.prs
+    )
 
 
 def _pr_ref(pr_number: Optional[int]) -> str:
@@ -275,8 +284,46 @@ def _rationale_block(rationale: str) -> str:
 
 
 def _fix_bug_stopped_payload(
-    issue_number: int, outcome: str, rationale: str = ""
+    issue_number: int, outcome: str, rationale: str = "",
+    *, outcome_named: bool = True,
 ) -> EscalationPayload:
+    """The halt brief for a `/fix-bug` session that stopped without a PR.
+
+    `outcome_named` is whether the session actually said one of the outcome
+    words (#3343). When it did not, `classify_outcome` failed closed to
+    `could_not_proceed` and **nothing about a refusal is established** — the
+    session may never have started. Defaults True so a caller predating this
+    keeps today's text rather than silently getting the weaker claim.
+    """
+    if outcome_named:
+        why_not_auto = (
+            "`/fix-bug`'s own refusal or precondition check stopped the run "
+            "before a PR existed; the bug lane never reached a guardrail or "
+            "merge decision on this path."
+        )
+        recommendation = (
+            "Read `/fix-bug`'s own reasoning first -- a `refused` outcome "
+            "usually means the fix is feature-scoped, which points at "
+            "promoting rather than forcing a bug-sized fix. Re-running the "
+            "lane unchanged will reach the same outcome."
+        )
+    else:
+        why_not_auto = (
+            "The session produced **no outcome marker at all**, so nothing "
+            "about this issue is established -- `classify_outcome` fails "
+            "closed to `could_not_proceed` for output naming no outcome, "
+            "which covers both a real refusal and a session that never "
+            "started. Its output above is the evidence for which happened. "
+            "Do not read this as `/fix-bug` having judged the issue."
+        )
+        recommendation = (
+            "Check the runner's own setup before touching this issue -- a "
+            "session that named no outcome usually did not run at all (a "
+            "command that does not resolve, a missing binary, an auth "
+            "failure), and that cause is identical for every issue in the "
+            "queue. Promoting this to a feature on the strength of this "
+            "escalation would be acting on a judgement nobody made."
+        )
     return EscalationPayload(
         target_issue=issue_number,
         done_so_far=(
@@ -292,11 +339,7 @@ def _fix_bug_stopped_payload(
             "Whether a human should fix this bug directly, promote it to a "
             "feature, or close it."
         ),
-        why_not_auto=(
-            "`/fix-bug`'s own refusal or precondition check stopped the run "
-            "before a PR existed; the bug lane never reached a guardrail or "
-            "merge decision on this path."
-        ),
+        why_not_auto=why_not_auto,
         options=[
             ("Fix it by hand", "unblocks the issue directly", "costs a human's time"),
             (
@@ -310,12 +353,7 @@ def _fix_bug_stopped_payload(
                 "loses whatever the report was pointing at",
             ),
         ],
-        recommendation=(
-            "Read `/fix-bug`'s own reasoning first -- a `refused` outcome "
-            "usually means the fix is feature-scoped, which points at "
-            "promoting rather than forcing a bug-sized fix. Re-running the "
-            "lane unchanged will reach the same outcome."
-        ),
+        recommendation=recommendation,
         category="blocked-wu",
     )
 
@@ -615,15 +653,30 @@ class BugsProvider:
         working_dir: str = ".",
         policy_path: Any = None,
         now: Optional[float] = None,
+        report: Optional[Callable[[str], None]] = None,
     ):
         self._repo = repo
         self._runner = runner
         self._working_dir = working_dir
         self._policy_path = policy_path
         self._now = now
+        # Optional so every existing construction site keeps working; a
+        # severity skip with nowhere to report is dropped rather than raising
+        # (#3339). run() passes its own reporter.
+        self._report = report or (lambda _message: None)
 
     def advertise(self, snapshot: AgentSnapshot) -> Sequence[ActionItem]:
         items = []
+        # #3339: `rules.bugs.min_severity` was validated and read by nothing, so
+        # a declared floor did not stop `/fix-bug` dispatching on minor issues.
+        # Resolved once per advertise rather than per issue -- the policy does
+        # not change mid-pass, and re-reading it per issue would make an empty
+        # lane cost one file read per open issue.
+        floor = resolve_min_severity(self._policy_path)
+        # #3349: what this repo's own `severity:*` labels mean, when they are
+        # not spelled the way `SEVERITY_ORDER` spells them. Resolved beside the
+        # floor and for the same reason -- the policy does not change mid-pass.
+        aliases = resolve_severity_aliases(self._policy_path)
         for issue in snapshot.issues:
             if issue.triage_category != "bug":
                 continue
@@ -631,6 +684,14 @@ class BugsProvider:
                 continue
             if _has_open_pr(snapshot, issue.number):
                 continue
+            severity, via = read_severity_label(issue.labels, aliases)
+            ok, why = meets_severity_floor(severity, floor, via=via)
+            if not ok:
+                # Reported, not silent: with a floor set and nothing labelled,
+                # the lane advertises nothing, and an operator needs to see why
+                # rather than conclude the lane is broken.
+                self._report(f"bug #{issue.number} skipped — {why}")
+                continue
             items.append(
                 ActionItem(
                     item_id=f"{_ITEM_ID_PREFIX}{issue.number}",
@@ -702,7 +763,8 @@ class BugsProvider:
                     detail = f"{detail} — {commits} committed on `{wip_ref}`, unpushed"
                 else:
                     escalation = _fix_bug_stopped_payload(
-                        issue_number, result.outcome, result.stop_rationale
+                        issue_number, result.outcome, result.stop_rationale,
+                        outcome_named=result.outcome_named,
                     )
         elif result.outcome == OUTCOME_AUTOMERGE_OFF:
             escalation = _automerge_off_payload(issue_number, result.pr_number)

{specfuse_loop-0.20.1 → specfuse_loop-0.22.0}/specfuse/agent/providers/triage.py

@@ -38,12 +38,18 @@ from specfuse.agent.run import (
 )
 from specfuse.agent.invoke import run_claude, usage_spend
 from specfuse.agent.state import AgentSnapshot
-from specfuse.agent.triage_invoke import build_invocation, classify_result
+from specfuse.agent.triage_invoke import build_invocation, classify_result, classify_severity
 from specfuse.loop.escalation import CATEGORY_LABELS
+from specfuse.loop.labels import read_severity_rubric
 from specfuse.loop.triage import CATEGORIES, apply_triage, list_untriaged
 
 _ITEM_ID_PREFIX = "triage-"
 
+#: Sentinel distinguishing "not fetched yet this run" from "fetched, empty
+#: rubric" -- `read_severity_rubric` legitimately returns `{}` on the
+#: degradation path (FEAT-2026-0113/T06), so `None`/`{}` can't do double duty.
+_RUBRIC_UNSET = object()
+
 
 def _triage_options() -> list:
     """The two routes open on any triage halt: hand-triage, or leave it."""
@@ -181,9 +187,22 @@ class TriageProvider:
         self._policy_path = policy_path
         self._rows: dict = {}
         self._auto = False
+        self._rubric: Any = _RUBRIC_UNSET
+
+    def _rubric_for_run(self) -> dict:
+        """The repository's severity rubric, read at most once per run
+        (FEAT-2026-0113/T06 criterion 3) -- fetched lazily on the first
+        `execute()` call after an `advertise()`, so a run that classifies
+        nothing issues no `gh label list` at all."""
+        if self._rubric is _RUBRIC_UNSET:
+            self._rubric = read_severity_rubric(
+                self._working_dir, runner=self._runner, repo=self._repo
+            )
+        return self._rubric
 
     def advertise(self, snapshot: AgentSnapshot) -> Sequence[ActionItem]:
         self._auto = snapshot.triage_auto
+        self._rubric = _RUBRIC_UNSET
         rows = list_untriaged(self._runner, self._repo)
 
         self._rows = {}
@@ -249,7 +268,10 @@ class TriageProvider:
                 ),
             )
 
-        argv, prompt = build_invocation(number, title, body, self._repo, self._working_dir)
+        rubric = self._rubric_for_run()
+        argv, prompt = build_invocation(
+            number, title, body, self._repo, self._working_dir, rubric=rubric
+        )
         invoked = run_claude(argv, prompt, runner=self._runner)
         spend = usage_spend(invoked.usage)
         classification = classify_result(invoked.text)
@@ -305,12 +327,14 @@ class TriageProvider:
                 ),
             )
 
+        severity = classify_severity(invoked.text, rubric)
         decisions = [
             {
                 "number": number,
                 "body": body,
                 "category": category,
                 "confidence": confidence,
+                "severity": severity,
             }
         ]
         results = apply_triage(self._runner, self._repo, decisions, auto=self._auto)