fix: URI collision, retry off-by-one, scheduler interval validation

- backend_memory: use uuid4 hex suffix instead of time() for duplicate URIs
- async_jobs: get_retryable_jobs uses < max_retries (was <=, ran one extra)
- scheduler: clamp interval to min 1h, prevent 0/negative crash

Author: ponsde

curator/async_jobs.py

@@ -127,4 +127,4 @@ def is_transient_error(error: str) -> bool:
 def get_retryable_jobs(max_retries: int = 3) -> list[dict]:
     """Return failed jobs that are retryable (transient error + under retry limit)."""
     failed = list_failed()
-    return [j for j in failed if is_transient_error(j.get("error", "")) and j.get("retries", 0) <= max_retries]
+    return [j for j in failed if is_transient_error(j.get("error", "")) and j.get("retries", 0) < max_retries]

curator/backend_memory.py

@@ -142,9 +142,9 @@ def ingest(self, content: str, title: str = "", metadata: dict | None = None) ->
         """
         safe = title.replace(" ", "_").replace("/", "_") if title else str(uuid.uuid4())[:8]
         uri = f"mem://{safe}"
-        # Handle duplicate URIs by appending suffix
+        # Handle duplicate URIs by appending random suffix
         if uri in self._store:
-            uri = f"{uri}_{int(time.time())}"
+            uri = f"{uri}_{uuid.uuid4().hex[:8]}"
         self._store[uri] = {
             "content": content,
             "title": title,

curator/scheduler.py

@@ -269,8 +269,8 @@ def start_scheduler() -> bool:
         try:
             from apscheduler.schedulers.background import BackgroundScheduler
 
-            freshness_h = float(env("CURATOR_FRESHNESS_INTERVAL_HOURS", "24"))
-            strengthen_h = float(env("CURATOR_STRENGTHEN_INTERVAL_HOURS", "168"))
+            freshness_h = max(1.0, float(env("CURATOR_FRESHNESS_INTERVAL_HOURS", "24") or "24"))
+            strengthen_h = max(1.0, float(env("CURATOR_STRENGTHEN_INTERVAL_HOURS", "168") or "168"))
 
             governance_enabled = env("CURATOR_GOVERNANCE_ENABLED", "").lower() in _ENABLED_VALUES
             governance_replaces_strengthen = (
@@ -298,7 +298,7 @@ def start_scheduler() -> bool:
                 )
 
             if governance_enabled:
-                governance_h = float(env("CURATOR_GOVERNANCE_INTERVAL_HOURS", "168"))
+                governance_h = max(1.0, float(env("CURATOR_GOVERNANCE_INTERVAL_HOURS", "168") or "168"))
                 _scheduler.add_job(
                     _run_governance,
                     "interval",