alissa-tools-github-orcloop 0.4.2__tar.gz → 0.4.3__tar.gz

{alissa_tools_github_orcloop-0.4.2/src/main/alissa_tools_github_orcloop.egg-info → alissa_tools_github_orcloop-0.4.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: alissa-tools-github-orcloop
-Version: 0.4.2
+Version: 0.4.3
 Summary: ALISSA-TOOLS-GITHUB-ORCLOOP
 Home-page: https://alissa.app
 Author: Fahera

{alissa_tools_github_orcloop-0.4.2 → alissa_tools_github_orcloop-0.4.3}/src/main/alissa/tools/github/orcloop/loop.py

@@ -583,6 +583,19 @@ ORIGIN_MISSING_MAX_ATTEMPTS = 3
 # is written by the loop itself, before anything can refuse it -- a counter
 # whose advance depends on a remote write is not a counter.
 #
+# THE FAMILY PREFIX, TRAILING COLON INCLUDED -- not the bare stem, which is what
+# the `_PREFIX` in the name is there to say. Its siblings a few hundred lines up
+# (`ESCALATION_CLOSELOOP_FAILED`, `ESCALATION_CLOSELOOP_TERMINAL`) are bare stems
+# whose call sites append the separator; this one carries it, and the name is the
+# only thing that tells the next author which of the two shapes they are holding.
+# Every reader and the clear pass this one string, so the two sides are the same
+# expression
+# rather than two literals that happen to agree: with the colon left to the call
+# sites, the reads scanned `marker:origin-missing:%` while `CLEARABLE_KINDS`
+# forced the clear to `marker:origin-missing%`, and a future sibling kind --
+# `marker:origin-missing-since`, exactly the clock shape dedupe and context both
+# grew -- would have been swept by a clear meant only for the attempts.
+#
 # `marker:` for the reason the other markers carry it: `escalations_matching` is
 # a prefix reader, and these rows must stay out of every prefix read of a page
 # kind.
@@ -600,7 +613,7 @@ ORIGIN_MISSING_MAX_ATTEMPTS = 3
 # the same argument applies to `marker:dedupe-unreadable`, which is cleared for
 # exactly this reason (a blip that clears must not make the next outage look
 # days old).
-ORIGIN_MISSING_ATTEMPT = "marker:origin-missing"
+ORIGIN_MISSING_ATTEMPT_PREFIX = "marker:origin-missing:"
 
 # The close edge could not write its evidence AT ALL -- not the enriched
 # delivery record, not the bare confirmation line -- for longer than the
@@ -788,7 +801,7 @@ def origin_missing_attempt_kind(attempt: int) -> str:
     `closeloop_failed_kind`, so the row count IS the attempt number under the
     ledger's idempotent (task, kind) insert -- two writes of the same attempt
     can never inflate it, and a restart cannot reset it."""
-    return f"{ORIGIN_MISSING_ATTEMPT}:a{attempt}"
+    return f"{ORIGIN_MISSING_ATTEMPT_PREFIX}a{attempt}"
 
 
 # --- escalation copy -------------------------------------------------------
@@ -4289,10 +4302,11 @@ class Orchestrator:
         `_origin_missing` at all (`_close_loop` returns before the read), so
         there are no rows for it to clear and no side effect to suppress. The
         guard below means it does not even query."""
-        if not self.state.escalations_matching(task_number, ORIGIN_MISSING_ATTEMPT + ":"):
+        if not self.state.escalations_matching(task_number, ORIGIN_MISSING_ATTEMPT_PREFIX):
             return
+        # The SAME string the read above scans with -- see the constant.
         cleared = self.state.clear_escalations_matching(
-            task_number, ORIGIN_MISSING_ATTEMPT
+            task_number, ORIGIN_MISSING_ATTEMPT_PREFIX
         )
         log.info(
             "%s: the origin task reads again — clearing %d unreadable-attempt "
@@ -4368,7 +4382,7 @@ class Orchestrator:
         What is NOT done here: the trigger label is not touched (parking stays
         the operator's call, the rule the park path holds), the issue is not
         closed or re-opened, and no Alissa write of any kind is attempted."""
-        prior = self.state.escalations_matching(task_number, ORIGIN_MISSING_ATTEMPT + ":")
+        prior = self.state.escalations_matching(task_number, ORIGIN_MISSING_ATTEMPT_PREFIX)
         if len(prior) >= ORIGIN_MISSING_MAX_ATTEMPTS:
             last_at = max(int(row["escalated_at"]) for row in prior)
             waited = self._now() - last_at

{alissa_tools_github_orcloop-0.4.2 → alissa_tools_github_orcloop-0.4.3}/src/main/alissa/tools/github/orcloop/state.py

@@ -74,14 +74,20 @@ CREATE TABLE IF NOT EXISTS meta (
 # imported from `loop`, which imports this module; `test_loop` pins that the
 # loop's constants and this set stay the same five strings.
 #
-# `marker:origin-missing` is the one entry that names a PREFIX rather than a
+# `marker:origin-missing:` is the one entry that names a PREFIX rather than a
 # whole kind: its rows are `marker:origin-missing:a<j>`, one per poll, and they
 # are cleared as a family by `clear_escalations_matching` -- a counter of
 # CONSECUTIVE failures is only consecutive if a success resets it.
+#
+# Its TRAILING COLON is load-bearing, and is the entry's whole job: the allowlist
+# is what forces the prefix the clear may use, so the bare stem would have
+# licensed a DELETE of `marker:origin-missing%` -- sweeping any future sibling
+# kind (`marker:origin-missing-since`, say) along with the attempts. Narrower
+# here means narrower everywhere, since the loop reads with this same string.
 CLEARABLE_KINDS = frozenset({
     "marker:dedupe-unreadable", "dedupe-unreadable",
     "marker:context-unreadable", "context-unreadable",
-    "marker:origin-missing",
+    "marker:origin-missing:",
 })
 
 

alissa_tools_github_orcloop-0.4.3/src/main/alissa/tools/github/orcloop/version

@@ -0,0 +1 @@
+0.4.3

{alissa_tools_github_orcloop-0.4.2 → alissa_tools_github_orcloop-0.4.3/src/main/alissa_tools_github_orcloop.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: alissa-tools-github-orcloop
-Version: 0.4.2
+Version: 0.4.3
 Summary: ALISSA-TOOLS-GITHUB-ORCLOOP
 Home-page: https://alissa.app
 Author: Fahera

alissa_tools_github_orcloop-0.4.2/src/main/alissa/tools/github/orcloop/version

@@ -1 +0,0 @@
-0.4.2