alissa-tools-github-revloop 0.31.4__tar.gz → 0.31.5__tar.gz

{alissa_tools_github_revloop-0.31.4/src/main/alissa_tools_github_revloop.egg-info → alissa_tools_github_revloop-0.31.5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: alissa-tools-github-revloop
-Version: 0.31.4
+Version: 0.31.5
 Summary: ALISSA-TOOLS-GITHUB-REVLOOP
 Home-page: https://alissa.app
 Author: Fahera

{alissa_tools_github_revloop-0.31.4 → alissa_tools_github_revloop-0.31.5}/src/main/alissa/tools/github/revloop/alissa.py

@@ -647,17 +647,20 @@ class VerdictEnvelope:
     envelope has no parseable `Merge-Readiness` line -- the case the native
     post fails closed on. `readiness_reason` is already one bounded,
     backtick-free line (see clean_readiness_reason); empty when the envelope
-    gave none. `readiness_class` is the READINESS_CLASSES token an operator
-    reason leads with (issue #142), or None: for `auto`, for an unclassed
-    operator reason, and for a missing line alike -- the reason keeps the
-    prefix either way, so the class is never lost between here and the
-    native trailer.
+    gave none, and it keeps the class prefix an operator reason leads with
+    (issue #142) -- so the class is never lost between here and the native
+    trailer.
+
+    The class is deliberately NOT a field. It is a function of the reason
+    (classify_readiness_reason), the emitter copies the reason whole, and
+    the narration reads the class back off the emitted line -- so the fact
+    has one source, and a second copy here would be a reader-less field
+    that could only ever drift from it (PR #143 round 1).
     """
 
     verdict: str
     readiness: "str | None" = None
     readiness_reason: str = ""
-    readiness_class: "str | None" = None
 
 
 @dataclass(frozen=True)
@@ -1158,9 +1161,12 @@ class Alissa:
                     continue
                 match = _VERDICT_RE.search(blob)
                 if match:
-                    readiness, reason, klass = parse_readiness(content)
+                    # The class the triple carries is not stored: the
+                    # reason keeps its prefix, and the envelope's readers
+                    # re-derive the class from the line they emit.
+                    readiness, reason, _ = parse_readiness(content)
                     if readiness is None:
-                        readiness, reason, klass = parse_readiness(title)
+                        readiness, reason, _ = parse_readiness(title)
                     found.append(
                         (Alissa._created_key(item.get("createdAt")),
                          index,
@@ -1168,7 +1174,6 @@ class Alissa:
                              verdict=match.group(1).lower(),
                              readiness=readiness,
                              readiness_reason=reason,
-                             readiness_class=klass,
                          ))
                     )
                     break

{alissa_tools_github_revloop-0.31.4 → alissa_tools_github_revloop-0.31.5}/src/main/alissa/tools/github/revloop/loop.py

@@ -713,6 +713,16 @@ _RECORD_THE_CAP = (
 # reviewer needs to classify honestly: an unverified-* hold is validation
 # work, not merge risk, and saying so lets the policy merge it. The table's
 # rows are the README's, condensed; the README is the long form.
+#
+# The table is closed, so the directive must also say what to do when NO
+# row fits (PR #143 round 1): the skill's checklist has operator triggers
+# the enum does not name (dependencies, a deleted public surface, a waived
+# `[major]`, a judgment residual, an operator gate that is a human action).
+# A reviewer told the class is mandatory would shoehorn such a reason into
+# the nearest row, and the nearest row for a vague reason is one of the
+# `unverified-*` pair -- exactly what the policy may merge unattended. So
+# the directive names the way out: no row, no class, and unclassed is the
+# correct hard hold, never a failure.
 _MERGE_READINESS_LINE = (
     "Your verdict envelope MUST carry the skill's `- **Merge-Readiness:** auto | "
     "operator — <class>: <reason>` line, AND your OWN native review (every `gh pr "
@@ -742,7 +752,13 @@ _MERGE_READINESS_LINE = (
     "`release-act` (the merge itself is a release — VERSION bump → publish, tag, "
     "npm). "
     "When more than one row applies, name the MOST SEVERE one — the row listed "
-    "first — and only it. Classify honestly, not defensively: an unverified-* "
+    "first — and only it. If NO row fits — a dependency change, a deleted or "
+    "renamed public surface, a waived `[major]`, a judgment residual, an "
+    "operator gate that is a human ACTION rather than a look or a smoke run — "
+    "write the reason with NO class: an unclassed operator line is the correct "
+    "hard hold, never a failure, and shoehorning such a reason into the nearest "
+    "row is the one misclassification the merge policy cannot recover from. "
+    "Classify honestly, not defensively: an unverified-* "
     "hold is validation work the human does at gate 2 anyway, not merge risk, "
     "and the merge policy may merge it while it keeps holding every row above; "
     "an operator line whose reason leads with no recognised token still parses "

alissa_tools_github_revloop-0.31.5/src/main/alissa/tools/github/revloop/version

@@ -0,0 +1 @@
+0.31.5

{alissa_tools_github_revloop-0.31.4 → alissa_tools_github_revloop-0.31.5/src/main/alissa_tools_github_revloop.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: alissa-tools-github-revloop
-Version: 0.31.4
+Version: 0.31.5
 Summary: ALISSA-TOOLS-GITHUB-REVLOOP
 Home-page: https://alissa.app
 Author: Fahera

alissa_tools_github_revloop-0.31.4/src/main/alissa/tools/github/revloop/version

@@ -1 +0,0 @@
-0.31.4