superlocalmemory 3.8.0 → 3.8.1

package/src/superlocalmemory/hooks/portable_kit.py

@@ -240,11 +240,12 @@ def connect_ide(
     here: bool = False,
     profile: str | None = None,
     agents_md_source: Callable[[], str] | None = None,
+    dry_run: bool = False,
 ) -> dict[str, Any]:
     """Wire SLM into the target IDE config via merge-not-clobber.
 
     Returns a result dict:
-        {ide, mcp_config: wrote|merged|unchanged|skipped|error,
+        {ide, mcp_config: wrote|merged|unchanged|would_write|skipped|error,
          mcp_path, agents_md: wrote|skipped(...)|unchanged|error,
          servers_preserved: int, error: str|None}
     """
@@ -265,6 +266,13 @@ def connect_ide(
         )
         return result
 
+    if ide_id == "vscode-copilot" and not here:
+        result["error"] = (
+            "VS Code / Copilot integration is project-scoped. "
+            "Run `slm connect vscode-copilot --here` from the project root."
+        )
+        return result
+
     # Step 1a — claude-code short-circuit (AC6)
     if desc.fmt == "":
         print(CLAUDE_CODE_PLUGIN_POINTER)
@@ -325,7 +333,28 @@ def connect_ide(
         result["servers_preserved"] = max(0, pre_count - (0 if pre_slm is None else 1))
         result["mcp_config"] = mcp_status
 
-    # Step 6 — atomic write
+    # Step 6 — atomic write. A dry run deliberately exercises the merge and
+    # packaged-asset lookup path without touching a user-owned IDE config.
+    if dry_run:
+        result["mcp_config"] = "would_write"
+        if desc.agents_md_path is None:
+            result["agents_md"] = "skipped(unsupported)"
+        elif agents_md_source is None:
+            result["agents_md"] = "skipped(no-source)"
+        else:
+            try:
+                source_content = agents_md_source()
+            except Exception as exc:
+                result["agents_md"] = "error(source-unavailable)"
+                result["error"] = f"AGENTS.md asset lookup failed: {exc}"
+                return result
+            if not isinstance(source_content, str) or not source_content.strip():
+                result["agents_md"] = "error(empty-source)"
+                result["error"] = "AGENTS.md asset is empty"
+                return result
+            result["agents_md"] = "would_write"
+        return result
+
     try:
         _atomic_write(config_path, data, desc.fmt)
     except Exception as exc:
@@ -528,6 +557,9 @@ def _handle_agents_md(
     except Exception as exc:
         logger.warning("agents_md_source() failed: %s — skipping AGENTS.md write", exc)
         return "skipped(source-error)"
+    if not isinstance(source_content, str) or not source_content.strip():
+        logger.warning("agents_md_source() returned no content — skipping AGENTS.md write")
+        return "skipped(no-source-content)"
 
     # Read existing content
     existing = ""

package/src/superlocalmemory/learning/model_rollback.py

@@ -34,6 +34,8 @@ import sqlite3
 from datetime import datetime, timedelta, timezone
 from typing import Final
 
+from superlocalmemory.learning.model_cache import invalidate as invalidate_model_cache
+
 logger = logging.getLogger(__name__)
 
 
@@ -235,6 +237,7 @@ class ModelRollback:
                 )
                 return False
 
+        invalidate_model_cache(self._profile_id)
         logger.warning(
             "AUTO-ROLLBACK profile=%s reason=%s observations=%d "
             "baseline_ndcg=%.4f current_ndcg=%.4f",

package/src/superlocalmemory/learning/ranker_retrain_online.py

@@ -33,6 +33,7 @@ from datetime import datetime, timezone
 from pathlib import Path
 from typing import Final
 
+from superlocalmemory.learning.model_cache import invalidate as invalidate_model_cache
 from superlocalmemory.learning.ranker_common import (
     _build_training_matrix,
     _compute_eval_metrics,
@@ -389,6 +390,7 @@ def _promote_candidate(
                 (json.dumps(meta), candidate_id),
             )
             conn.commit()
+            invalidate_model_cache(profile_id)
             return True
         except sqlite3.Error as exc:
             conn.rollback()

package/src/superlocalmemory/learning/reward.py

@@ -73,6 +73,24 @@ _DWELL_MAX_MS: Final[int] = 3_600_000  # 1 h
 _BUSY_TIMEOUT_MS: Final[int] = 50
 
 
+def _feed_source_quality(
+    memory_db_path: Path,
+    rewards: list[tuple[str, str, list[str], float]],
+) -> None:
+    """Fail-soft provenance bridge kept outside the reward transaction."""
+    try:
+        from superlocalmemory.learning.source_quality import (
+            update_source_quality_for_reward_batch,
+        )
+        update_source_quality_for_reward_batch(
+            memory_db_path=memory_db_path,
+            learning_db_path=memory_db_path.parent / "learning.db",
+            rewards=rewards,
+        )
+    except Exception as exc:  # noqa: BLE001 - reward finalization must survive
+        logger.debug("source-quality reward feed failed (non-fatal): %s", exc)
+
+
 # ---------------------------------------------------------------------------
 # Signal contract — names match the manifest A.1 label formula.
 # ---------------------------------------------------------------------------
@@ -604,6 +622,7 @@ class EngagementRewardModel:
                 # lock-scope — SQLite rows are tied to the connection.
                 _pid = pending["profile_id"]
                 _qid = pending["recall_query_id"]
+                _facts_json = str(pending["fact_ids_json"] or "[]")
         except sqlite3.Error as exc:
             logger.debug("finalize_outcome SQLite error: %s", exc)
             return _FALLBACK_REWARD
@@ -633,6 +652,21 @@ class EngagementRewardModel:
             except Exception as exc:  # noqa: BLE001 — defence in depth
                 logger.debug("feed_recall_settled failed (non-fatal): %s", exc)
 
+        try:
+            _fact_ids = json.loads(_facts_json)
+            if not isinstance(_fact_ids, list):
+                _fact_ids = []
+        except (TypeError, ValueError, json.JSONDecodeError):
+            _fact_ids = []
+        _feed_source_quality(
+            self._db,
+            [(
+                str(_pid),
+                str(outcome_id),
+                [str(fid) for fid in _fact_ids if fid],
+                float(reward),
+            )],
+        )
         return reward
 
     # ------------------------------------------------------------------
@@ -749,6 +783,22 @@ class EngagementRewardModel:
                     except sqlite3.Error:
                         conn.execute("ROLLBACK")
                         raise
+                reward_sources = []
+                for item in i_chunk:
+                    try:
+                        fact_ids = json.loads(str(item[2] or "[]"))
+                    except (TypeError, ValueError, json.JSONDecodeError):
+                        fact_ids = []
+                    reward_sources.append((
+                        str(item[1]),
+                        str(item[0]),
+                        (
+                            [str(fid) for fid in fact_ids if fid]
+                            if isinstance(fact_ids, list) else []
+                        ),
+                        float(item[4]),
+                    ))
+                _feed_source_quality(self._db, reward_sources)
         except sqlite3.Error as exc:  # pragma: no cover — defensive
             logger.debug("reap_stale SQLite error: %s", exc)
             return written