@rubix0270/arboris 1.0.2 → 1.0.4

package/prisma/skills/continuous-learning-v2/scripts/test_parse_instinct.py

@@ -1,1421 +0,0 @@
-"""Tests for continuous-learning-v2 instinct-cli.py
-
-Covers:
-  - parse_instinct_file() — content preservation, edge cases
-  - _validate_file_path() — path traversal blocking
-  - detect_project() — project detection with mocked git/env
-  - load_all_instincts() — loading from project + global dirs, dedup
-  - _load_instincts_from_dir() — directory scanning
-  - cmd_projects() — listing projects from registry
-  - cmd_status() — status display
-  - _promote_specific() — single instinct promotion
-  - _promote_auto() — auto-promotion across projects
-"""
-
-import importlib.util
-import io
-import json
-import os
-import sys
-from pathlib import Path
-from types import SimpleNamespace
-from unittest import mock
-
-import pytest
-
-# Load instinct-cli.py (hyphenated filename requires importlib)
-_spec = importlib.util.spec_from_file_location(
-    "instinct_cli",
-    os.path.join(os.path.dirname(__file__), "instinct-cli.py"),
-)
-_mod = importlib.util.module_from_spec(_spec)
-_spec.loader.exec_module(_mod)
-
-parse_instinct_file = _mod.parse_instinct_file
-_validate_file_path = _mod._validate_file_path
-detect_project = _mod.detect_project
-load_all_instincts = _mod.load_all_instincts
-load_project_only_instincts = _mod.load_project_only_instincts
-_load_instincts_from_dir = _mod._load_instincts_from_dir
-cmd_status = _mod.cmd_status
-cmd_projects = _mod.cmd_projects
-_promote_specific = _mod._promote_specific
-_promote_auto = _mod._promote_auto
-_find_cross_project_instincts = _mod._find_cross_project_instincts
-load_registry = _mod.load_registry
-_validate_instinct_id = _mod._validate_instinct_id
-_validate_import_url = _mod._validate_import_url
-_update_registry = _mod._update_registry
-_write_registry = _mod._write_registry
-_remove_project_storage = _mod._remove_project_storage
-_confidence_bar = _mod._confidence_bar
-
-
-# ─────────────────────────────────────────────
-# Fixtures
-# ─────────────────────────────────────────────
-
-SAMPLE_INSTINCT_YAML = """\
----
-id: test-instinct
-trigger: "when writing tests"
-confidence: 0.8
-domain: testing
-scope: project
----
-
-## Action
-Always write tests first.
-
-## Evidence
-TDD leads to better design.
-"""
-
-SAMPLE_GLOBAL_INSTINCT_YAML = """\
----
-id: global-instinct
-trigger: "always"
-confidence: 0.9
-domain: security
-scope: global
----
-
-## Action
-Validate all user input.
-"""
-
-
-@pytest.fixture
-def project_tree(tmp_path):
-    """Create a realistic project directory tree for testing."""
-    homunculus = tmp_path / ".claude" / "homunculus"
-    projects_dir = homunculus / "projects"
-    global_personal = homunculus / "instincts" / "personal"
-    global_inherited = homunculus / "instincts" / "inherited"
-    global_evolved = homunculus / "evolved"
-
-    for d in [
-        global_personal, global_inherited,
-        global_evolved / "skills", global_evolved / "commands", global_evolved / "agents",
-        projects_dir,
-    ]:
-        d.mkdir(parents=True, exist_ok=True)
-
-    return {
-        "root": tmp_path,
-        "homunculus": homunculus,
-        "projects_dir": projects_dir,
-        "global_personal": global_personal,
-        "global_inherited": global_inherited,
-        "global_evolved": global_evolved,
-        "registry_file": homunculus / "projects.json",
-    }
-
-
-@pytest.fixture
-def patch_globals(project_tree, monkeypatch):
-    """Patch module-level globals to use tmp_path-based directories."""
-    monkeypatch.setattr(_mod, "HOMUNCULUS_DIR", project_tree["homunculus"])
-    monkeypatch.setattr(_mod, "PROJECTS_DIR", project_tree["projects_dir"])
-    monkeypatch.setattr(_mod, "REGISTRY_FILE", project_tree["registry_file"])
-    monkeypatch.setattr(_mod, "GLOBAL_PERSONAL_DIR", project_tree["global_personal"])
-    monkeypatch.setattr(_mod, "GLOBAL_INHERITED_DIR", project_tree["global_inherited"])
-    monkeypatch.setattr(_mod, "GLOBAL_EVOLVED_DIR", project_tree["global_evolved"])
-    monkeypatch.setattr(_mod, "GLOBAL_OBSERVATIONS_FILE", project_tree["homunculus"] / "observations.jsonl")
-    return project_tree
-
-
-def _make_project(tree, pid="abc123", pname="test-project"):
-    """Create project directory structure and return a project dict."""
-    project_dir = tree["projects_dir"] / pid
-    personal_dir = project_dir / "instincts" / "personal"
-    inherited_dir = project_dir / "instincts" / "inherited"
-    for d in [personal_dir, inherited_dir,
-              project_dir / "evolved" / "skills",
-              project_dir / "evolved" / "commands",
-              project_dir / "evolved" / "agents",
-              project_dir / "observations.archive"]:
-        d.mkdir(parents=True, exist_ok=True)
-
-    return {
-        "id": pid,
-        "name": pname,
-        "root": str(tree["root"] / "fake-repo"),
-        "remote": "https://github.com/test/test-project.git",
-        "project_dir": project_dir,
-        "instincts_personal": personal_dir,
-        "instincts_inherited": inherited_dir,
-        "evolved_dir": project_dir / "evolved",
-        "observations_file": project_dir / "observations.jsonl",
-    }
-
-
-# ─────────────────────────────────────────────
-# parse_instinct_file tests
-# ─────────────────────────────────────────────
-
-MULTI_SECTION = """\
----
-id: instinct-a
-trigger: "when coding"
-confidence: 0.9
-domain: general
----
-
-## Action
-Do thing A.
-
-## Examples
-- Example A1
-
----
-id: instinct-b
-trigger: "when testing"
-confidence: 0.7
-domain: testing
----
-
-## Action
-Do thing B.
-"""
-
-
-def test_multiple_instincts_preserve_content():
-    result = parse_instinct_file(MULTI_SECTION)
-    assert len(result) == 2
-    assert "Do thing A." in result[0]["content"]
-    assert "Example A1" in result[0]["content"]
-    assert "Do thing B." in result[1]["content"]
-
-
-def test_single_instinct_preserves_content():
-    content = """\
----
-id: solo
-trigger: "when reviewing"
-confidence: 0.8
-domain: review
----
-
-## Action
-Check for security issues.
-
-## Evidence
-Prevents vulnerabilities.
-"""
-    result = parse_instinct_file(content)
-    assert len(result) == 1
-    assert "Check for security issues." in result[0]["content"]
-    assert "Prevents vulnerabilities." in result[0]["content"]
-
-
-def test_empty_content_no_error():
-    content = """\
----
-id: empty
-trigger: "placeholder"
-confidence: 0.5
-domain: general
----
-"""
-    result = parse_instinct_file(content)
-    assert len(result) == 1
-    assert result[0]["content"] == ""
-
-
-def test_parse_no_id_skipped():
-    """Instincts without an 'id' field should be silently dropped."""
-    content = """\
----
-trigger: "when doing nothing"
-confidence: 0.5
----
-
-No id here.
-"""
-    result = parse_instinct_file(content)
-    assert len(result) == 0
-
-
-def test_parse_confidence_is_float():
-    content = """\
----
-id: float-check
-trigger: "when parsing"
-confidence: 0.42
-domain: general
----
-
-Body.
-"""
-    result = parse_instinct_file(content)
-    assert isinstance(result[0]["confidence"], float)
-    assert result[0]["confidence"] == pytest.approx(0.42)
-
-
-def test_parse_trigger_strips_quotes():
-    content = """\
----
-id: quote-check
-trigger: "when quoting"
-confidence: 0.5
-domain: general
----
-
-Body.
-"""
-    result = parse_instinct_file(content)
-    assert result[0]["trigger"] == "when quoting"
-
-
-def test_parse_empty_string():
-    result = parse_instinct_file("")
-    assert result == []
-
-
-def test_parse_garbage_input():
-    result = parse_instinct_file("this is not yaml at all\nno frontmatter here")
-    assert result == []
-
-
-# ─────────────────────────────────────────────
-# _validate_file_path tests
-# ─────────────────────────────────────────────
-
-def test_validate_normal_path(tmp_path):
-    test_file = tmp_path / "test.yaml"
-    test_file.write_text("hello")
-    result = _validate_file_path(str(test_file), must_exist=True)
-    assert result == test_file.resolve()
-
-
-def test_validate_rejects_etc():
-    with pytest.raises(ValueError, match="system directory"):
-        _validate_file_path("/etc/passwd")
-
-
-def test_validate_rejects_var_log():
-    with pytest.raises(ValueError, match="system directory"):
-        _validate_file_path("/var/log/syslog")
-
-
-def test_validate_rejects_usr():
-    with pytest.raises(ValueError, match="system directory"):
-        _validate_file_path("/usr/local/bin/foo")
-
-
-def test_validate_rejects_proc():
-    with pytest.raises(ValueError, match="system directory"):
-        _validate_file_path("/proc/self/status")
-
-
-def test_validate_must_exist_fails(tmp_path):
-    with pytest.raises(ValueError, match="does not exist"):
-        _validate_file_path(str(tmp_path / "nonexistent.yaml"), must_exist=True)
-
-
-def test_validate_home_expansion(tmp_path):
-    """Tilde expansion should work."""
-    result = _validate_file_path("~/test.yaml")
-    assert str(result).startswith(str(Path.home()))
-
-
-def test_validate_relative_path(tmp_path, monkeypatch):
-    """Relative paths should be resolved."""
-    monkeypatch.chdir(tmp_path)
-    test_file = tmp_path / "rel.yaml"
-    test_file.write_text("content")
-    result = _validate_file_path("rel.yaml", must_exist=True)
-    assert result == test_file.resolve()
-
-
-def test_validate_import_url_rejects_http():
-    """Remote imports should not downgrade to plaintext HTTP."""
-    with pytest.raises(ValueError, match="require https"):
-        _validate_import_url("http://example.com/instincts.yaml")
-
-
-def test_validate_import_url_rejects_private_hosts(monkeypatch):
-    """Remote imports should not resolve to private or loopback addresses."""
-    monkeypatch.setattr(
-        _mod.socket,
-        "getaddrinfo",
-        lambda *args, **kwargs: [(None, None, None, None, ("127.0.0.1", 443))],
-    )
-    with pytest.raises(ValueError, match="non-public address"):
-        _validate_import_url("https://example.com/instincts.yaml")
-
-
-def test_validate_import_url_allows_public_https(monkeypatch):
-    monkeypatch.setattr(
-        _mod.socket,
-        "getaddrinfo",
-        lambda *args, **kwargs: [(None, None, None, None, ("93.184.216.34", 443))],
-    )
-    assert _validate_import_url("https://example.com/instincts.yaml") == "https://example.com/instincts.yaml"
-
-
-# ─────────────────────────────────────────────
-# detect_project tests
-# ─────────────────────────────────────────────
-
-def test_detect_project_global_fallback(patch_globals, monkeypatch):
-    """When no git and no env var, should return global project."""
-    monkeypatch.delenv("CLAUDE_PROJECT_DIR", raising=False)
-
-    # Mock subprocess.run to simulate git not available
-    def mock_run(*args, **kwargs):
-        raise FileNotFoundError("git not found")
-
-    monkeypatch.setattr("subprocess.run", mock_run)
-
-    project = detect_project()
-    assert project["id"] == "global"
-    assert project["name"] == "global"
-
-
-def test_detect_project_from_env(patch_globals, monkeypatch, tmp_path):
-    """CLAUDE_PROJECT_DIR env var should be used as project root."""
-    fake_repo = tmp_path / "my-repo"
-    fake_repo.mkdir()
-    monkeypatch.setenv("CLAUDE_PROJECT_DIR", str(fake_repo))
-
-    # Mock git remote to return a URL
-    def mock_run(cmd, **kwargs):
-        if "rev-parse" in cmd:
-            return SimpleNamespace(returncode=0, stdout=str(fake_repo) + "\n", stderr="")
-        if "get-url" in cmd:
-            return SimpleNamespace(returncode=0, stdout="https://github.com/test/my-repo.git\n", stderr="")
-        return SimpleNamespace(returncode=1, stdout="", stderr="")
-
-    monkeypatch.setattr("subprocess.run", mock_run)
-
-    project = detect_project()
-    assert project["id"] != "global"
-    assert project["name"] == "my-repo"
-
-
-def test_detect_project_git_timeout(patch_globals, monkeypatch):
-    """Git timeout should fall through to global."""
-    monkeypatch.delenv("CLAUDE_PROJECT_DIR", raising=False)
-    import subprocess as sp
-
-    def mock_run(cmd, **kwargs):
-        raise sp.TimeoutExpired(cmd, 5)
-
-    monkeypatch.setattr("subprocess.run", mock_run)
-
-    project = detect_project()
-    assert project["id"] == "global"
-
-
-def test_detect_project_creates_directories(patch_globals, monkeypatch, tmp_path):
-    """detect_project should create the project dir structure."""
-    fake_repo = tmp_path / "structured-repo"
-    fake_repo.mkdir()
-    monkeypatch.setenv("CLAUDE_PROJECT_DIR", str(fake_repo))
-
-    def mock_run(cmd, **kwargs):
-        if "rev-parse" in cmd:
-            return SimpleNamespace(returncode=0, stdout=str(fake_repo) + "\n", stderr="")
-        if "get-url" in cmd:
-            return SimpleNamespace(returncode=1, stdout="", stderr="no remote")
-        return SimpleNamespace(returncode=1, stdout="", stderr="")
-
-    monkeypatch.setattr("subprocess.run", mock_run)
-
-    project = detect_project()
-    assert project["instincts_personal"].exists()
-    assert project["instincts_inherited"].exists()
-    assert (project["evolved_dir"] / "skills").exists()
-
-
-# ─────────────────────────────────────────────
-# _load_instincts_from_dir tests
-# ─────────────────────────────────────────────
-
-def test_load_from_empty_dir(tmp_path):
-    result = _load_instincts_from_dir(tmp_path, "personal", "project")
-    assert result == []
-
-
-def test_load_from_nonexistent_dir(tmp_path):
-    result = _load_instincts_from_dir(tmp_path / "does-not-exist", "personal", "project")
-    assert result == []
-
-
-def test_load_annotates_metadata(tmp_path):
-    """Loaded instincts should have _source_file, _source_type, _scope_label."""
-    yaml_file = tmp_path / "test.yaml"
-    yaml_file.write_text(SAMPLE_INSTINCT_YAML)
-
-    result = _load_instincts_from_dir(tmp_path, "personal", "project")
-    assert len(result) == 1
-    assert result[0]["_source_file"] == str(yaml_file)
-    assert result[0]["_source_type"] == "personal"
-    assert result[0]["_scope_label"] == "project"
-
-
-def test_load_defaults_scope_from_label(tmp_path):
-    """If an instinct has no 'scope' in frontmatter, it should default to scope_label."""
-    no_scope_yaml = """\
----
-id: no-scope
-trigger: "test"
-confidence: 0.5
-domain: general
----
-
-Body.
-"""
-    (tmp_path / "no-scope.yaml").write_text(no_scope_yaml)
-    result = _load_instincts_from_dir(tmp_path, "inherited", "global")
-    assert result[0]["scope"] == "global"
-
-
-def test_load_preserves_explicit_scope(tmp_path):
-    """If frontmatter has explicit scope, it should be preserved."""
-    yaml_file = tmp_path / "test.yaml"
-    yaml_file.write_text(SAMPLE_INSTINCT_YAML)
-
-    result = _load_instincts_from_dir(tmp_path, "personal", "global")
-    # Frontmatter says scope: project, scope_label is global
-    # The explicit scope should be preserved (not overwritten)
-    assert result[0]["scope"] == "project"
-
-
-def test_load_handles_corrupt_file(tmp_path, capsys):
-    """Corrupt YAML files should be warned about but not crash."""
-    # A file that will cause parse_instinct_file to return empty
-    (tmp_path / "good.yaml").write_text(SAMPLE_INSTINCT_YAML)
-    (tmp_path / "bad.yaml").write_text("not yaml\nno frontmatter")
-
-    result = _load_instincts_from_dir(tmp_path, "personal", "project")
-    # bad.yaml has no valid instincts (no id), so only good.yaml contributes
-    assert len(result) == 1
-    assert result[0]["id"] == "test-instinct"
-
-
-def test_load_supports_yml_extension(tmp_path):
-    yml_file = tmp_path / "test.yml"
-    yml_file.write_text(SAMPLE_INSTINCT_YAML)
-
-    result = _load_instincts_from_dir(tmp_path, "personal", "project")
-    ids = {i["id"] for i in result}
-    assert "test-instinct" in ids
-
-
-def test_load_supports_md_extension(tmp_path):
-    md_file = tmp_path / "legacy-instinct.md"
-    md_file.write_text(SAMPLE_INSTINCT_YAML)
-
-    result = _load_instincts_from_dir(tmp_path, "personal", "project")
-    ids = {i["id"] for i in result}
-    assert "test-instinct" in ids
-
-
-def test_load_instincts_from_dir_uses_utf8_encoding(tmp_path, monkeypatch):
-    yaml_file = tmp_path / "test.yaml"
-    yaml_file.write_text("placeholder")
-    calls = []
-
-    def fake_read_text(self, *args, **kwargs):
-        calls.append(kwargs.get("encoding"))
-        return SAMPLE_INSTINCT_YAML
-
-    monkeypatch.setattr(Path, "read_text", fake_read_text)
-    result = _load_instincts_from_dir(tmp_path, "personal", "project")
-    assert result[0]["id"] == "test-instinct"
-    assert calls == ["utf-8"]
-
-
-# ─────────────────────────────────────────────
-# load_all_instincts tests
-# ─────────────────────────────────────────────
-
-def test_load_all_project_and_global(patch_globals):
-    """Should load from both project and global directories."""
-    tree = patch_globals
-    project = _make_project(tree)
-
-    # Write a project instinct
-    (project["instincts_personal"] / "proj.yaml").write_text(SAMPLE_INSTINCT_YAML)
-    # Write a global instinct
-    (tree["global_personal"] / "glob.yaml").write_text(SAMPLE_GLOBAL_INSTINCT_YAML)
-
-    result = load_all_instincts(project)
-    ids = {i["id"] for i in result}
-    assert "test-instinct" in ids
-    assert "global-instinct" in ids
-
-
-def test_load_all_project_overrides_global(patch_globals):
-    """When project and global have same ID, project wins."""
-    tree = patch_globals
-    project = _make_project(tree)
-
-    # Same ID but different confidence
-    proj_yaml = SAMPLE_INSTINCT_YAML.replace("id: test-instinct", "id: shared-id")
-    proj_yaml = proj_yaml.replace("confidence: 0.8", "confidence: 0.9")
-    glob_yaml = SAMPLE_GLOBAL_INSTINCT_YAML.replace("id: global-instinct", "id: shared-id")
-    glob_yaml = glob_yaml.replace("confidence: 0.9", "confidence: 0.3")
-
-    (project["instincts_personal"] / "shared.yaml").write_text(proj_yaml)
-    (tree["global_personal"] / "shared.yaml").write_text(glob_yaml)
-
-    result = load_all_instincts(project)
-    shared = [i for i in result if i["id"] == "shared-id"]
-    assert len(shared) == 1
-    assert shared[0]["_scope_label"] == "project"
-    assert shared[0]["confidence"] == 0.9
-
-
-def test_load_all_global_only(patch_globals):
-    """Global project should only load global instincts."""
-    tree = patch_globals
-    (tree["global_personal"] / "glob.yaml").write_text(SAMPLE_GLOBAL_INSTINCT_YAML)
-
-    global_project = {
-        "id": "global",
-        "name": "global",
-        "root": "",
-        "project_dir": tree["homunculus"],
-        "instincts_personal": tree["global_personal"],
-        "instincts_inherited": tree["global_inherited"],
-        "evolved_dir": tree["global_evolved"],
-        "observations_file": tree["homunculus"] / "observations.jsonl",
-    }
-
-    result = load_all_instincts(global_project)
-    assert len(result) == 1
-    assert result[0]["id"] == "global-instinct"
-
-
-def test_load_project_only_excludes_global(patch_globals):
-    """load_project_only_instincts should NOT include global instincts."""
-    tree = patch_globals
-    project = _make_project(tree)
-
-    (project["instincts_personal"] / "proj.yaml").write_text(SAMPLE_INSTINCT_YAML)
-    (tree["global_personal"] / "glob.yaml").write_text(SAMPLE_GLOBAL_INSTINCT_YAML)
-
-    result = load_project_only_instincts(project)
-    ids = {i["id"] for i in result}
-    assert "test-instinct" in ids
-    assert "global-instinct" not in ids
-
-
-def test_load_project_only_global_fallback_loads_global(patch_globals):
-    """Global fallback should return global instincts for project-only queries."""
-    tree = patch_globals
-    (tree["global_personal"] / "glob.yaml").write_text(SAMPLE_GLOBAL_INSTINCT_YAML)
-
-    global_project = {
-        "id": "global",
-        "name": "global",
-        "root": "",
-        "project_dir": tree["homunculus"],
-        "instincts_personal": tree["global_personal"],
-        "instincts_inherited": tree["global_inherited"],
-        "evolved_dir": tree["global_evolved"],
-        "observations_file": tree["homunculus"] / "observations.jsonl",
-    }
-
-    result = load_project_only_instincts(global_project)
-    assert len(result) == 1
-    assert result[0]["id"] == "global-instinct"
-
-
-def test_load_all_empty(patch_globals):
-    """No instincts at all should return empty list."""
-    tree = patch_globals
-    project = _make_project(tree)
-
-    result = load_all_instincts(project)
-    assert result == []
-
-
-# ─────────────────────────────────────────────
-# cmd_status tests
-# ─────────────────────────────────────────────
-
-def test_cmd_status_no_instincts(patch_globals, monkeypatch, capsys):
-    """Status with no instincts should print fallback message."""
-    tree = patch_globals
-    project = _make_project(tree)
-    monkeypatch.setattr(_mod, "detect_project", lambda: project)
-
-    args = SimpleNamespace()
-    ret = cmd_status(args)
-    assert ret == 0
-    out = capsys.readouterr().out
-    assert "No instincts found." in out
-
-
-def test_cmd_status_with_instincts(patch_globals, monkeypatch, capsys):
-    """Status should show project and global instinct counts."""
-    tree = patch_globals
-    project = _make_project(tree)
-    monkeypatch.setattr(_mod, "detect_project", lambda: project)
-
-    (project["instincts_personal"] / "proj.yaml").write_text(SAMPLE_INSTINCT_YAML)
-    (tree["global_personal"] / "glob.yaml").write_text(SAMPLE_GLOBAL_INSTINCT_YAML)
-
-    args = SimpleNamespace()
-    ret = cmd_status(args)
-    assert ret == 0
-    out = capsys.readouterr().out
-    assert "INSTINCT STATUS" in out
-    assert "Project instincts: 1" in out
-    assert "Global instincts:  1" in out
-    assert "PROJECT-SCOPED" in out
-    assert "GLOBAL" in out
-
-
-def test_confidence_bar_uses_unicode_when_supported():
-    """Confidence bars should retain block glyphs on UTF-8 streams."""
-    stream = SimpleNamespace(encoding="utf-8")
-    assert _confidence_bar(0.8, stream=stream) == "\u2588" * 8 + "\u2591" * 2
-
-
-def test_confidence_bar_uses_ascii_when_stream_rejects_block_glyphs():
-    """Windows cp1252 streams cannot encode block glyphs."""
-    stream = SimpleNamespace(encoding="cp1252")
-    assert _confidence_bar(0.8, stream=stream) == "########.."
-
-
-def test_print_instincts_by_domain_is_cp1252_safe(monkeypatch):
-    """Status rendering should not crash on Windows cp1252 stdout."""
-    raw = io.BytesIO()
-    stream = io.TextIOWrapper(raw, encoding="cp1252")
-    monkeypatch.setattr(_mod.sys, "stdout", stream)
-
-    _mod._print_instincts_by_domain([{
-        "id": "windows-safe",
-        "trigger": "when stdout uses cp1252",
-        "confidence": 0.8,
-        "domain": "platform",
-        "scope": "project",
-    }])
-
-    stream.flush()
-    out = raw.getvalue().decode("cp1252")
-    assert "########.." in out
-    assert "\u2588" not in out
-    assert "\u2591" not in out
-
-
-def test_cmd_status_returns_int(patch_globals, monkeypatch):
-    """cmd_status should always return an int."""
-    tree = patch_globals
-    project = _make_project(tree)
-    monkeypatch.setattr(_mod, "detect_project", lambda: project)
-
-    args = SimpleNamespace()
-    ret = cmd_status(args)
-    assert isinstance(ret, int)
-
-
-# ─────────────────────────────────────────────
-# cmd_projects tests
-# ─────────────────────────────────────────────
-
-def test_cmd_projects_empty_registry(patch_globals, capsys):
-    """No projects should print helpful message."""
-    args = SimpleNamespace()
-    ret = cmd_projects(args)
-    assert ret == 0
-    out = capsys.readouterr().out
-    assert "No projects registered yet." in out
-
-
-def test_cmd_projects_with_registry(patch_globals, capsys):
-    """Should list projects from registry."""
-    tree = patch_globals
-
-    # Create a project dir with instincts
-    pid = "test123abc"
-    project = _make_project(tree, pid=pid, pname="my-app")
-    (project["instincts_personal"] / "inst.yaml").write_text(SAMPLE_INSTINCT_YAML)
-
-    # Write registry
-    registry = {
-        pid: {
-            "name": "my-app",
-            "root": "/home/user/my-app",
-            "remote": "https://github.com/user/my-app.git",
-            "last_seen": "2025-01-15T12:00:00Z",
-        }
-    }
-    tree["registry_file"].write_text(json.dumps(registry))
-
-    args = SimpleNamespace()
-    ret = cmd_projects(args)
-    assert ret == 0
-    out = capsys.readouterr().out
-    assert "my-app" in out
-    assert pid in out
-    assert "1 personal" in out
-
-
-# ─────────────────────────────────────────────
-# _promote_specific tests
-# ─────────────────────────────────────────────
-
-def test_promote_specific_not_found(patch_globals, capsys):
-    """Promoting nonexistent instinct should fail."""
-    tree = patch_globals
-    project = _make_project(tree)
-
-    ret = _promote_specific(project, "nonexistent", force=True)
-    assert ret == 1
-    out = capsys.readouterr().out
-    assert "not found" in out
-
-
-def test_promote_specific_rejects_invalid_id(patch_globals, capsys):
-    """Path-like instinct IDs should be rejected before file writes."""
-    tree = patch_globals
-    project = _make_project(tree)
-
-    ret = _promote_specific(project, "../escape", force=True)
-    assert ret == 1
-    err = capsys.readouterr().err
-    assert "Invalid instinct ID" in err
-
-
-def test_promote_specific_already_global(patch_globals, capsys):
-    """Promoting an instinct that already exists globally should fail."""
-    tree = patch_globals
-    project = _make_project(tree)
-
-    # Write same-id instinct in both project and global
-    (project["instincts_personal"] / "shared.yaml").write_text(SAMPLE_INSTINCT_YAML)
-    global_yaml = SAMPLE_INSTINCT_YAML  # same id: test-instinct
-    (tree["global_personal"] / "shared.yaml").write_text(global_yaml)
-
-    ret = _promote_specific(project, "test-instinct", force=True)
-    assert ret == 1
-    out = capsys.readouterr().out
-    assert "already exists in global" in out
-
-
-def test_promote_specific_success(patch_globals, capsys):
-    """Promote a project instinct to global with --force."""
-    tree = patch_globals
-    project = _make_project(tree)
-
-    (project["instincts_personal"] / "inst.yaml").write_text(SAMPLE_INSTINCT_YAML)
-
-    ret = _promote_specific(project, "test-instinct", force=True)
-    assert ret == 0
-    out = capsys.readouterr().out
-    assert "Promoted" in out
-
-    # Verify file was created in global dir
-    promoted_file = tree["global_personal"] / "test-instinct.yaml"
-    assert promoted_file.exists()
-    content = promoted_file.read_text()
-    assert "scope: global" in content
-    assert "promoted_from: abc123" in content
-
-
-# ─────────────────────────────────────────────
-# _promote_auto tests
-# ─────────────────────────────────────────────
-
-def test_promote_auto_no_candidates(patch_globals, capsys):
-    """Auto-promote with no cross-project instincts should say so."""
-    tree = patch_globals
-    project = _make_project(tree)
-
-    # Empty registry
-    tree["registry_file"].write_text("{}")
-
-    ret = _promote_auto(project, force=True, dry_run=False)
-    assert ret == 0
-    out = capsys.readouterr().out
-    assert "No instincts qualify" in out
-
-
-def test_promote_auto_dry_run(patch_globals, capsys):
-    """Dry run should list candidates but not write files."""
-    tree = patch_globals
-
-    # Create two projects with the same high-confidence instinct
-    p1 = _make_project(tree, pid="proj1", pname="project-one")
-    p2 = _make_project(tree, pid="proj2", pname="project-two")
-
-    high_conf_yaml = """\
----
-id: cross-project-instinct
-trigger: "when reviewing"
-confidence: 0.95
-domain: security
-scope: project
----
-
-## Action
-Always review for injection.
-"""
-    (p1["instincts_personal"] / "cross.yaml").write_text(high_conf_yaml)
-    (p2["instincts_personal"] / "cross.yaml").write_text(high_conf_yaml)
-
-    # Write registry
-    registry = {
-        "proj1": {"name": "project-one", "root": "/a", "remote": "", "last_seen": "2025-01-01T00:00:00Z"},
-        "proj2": {"name": "project-two", "root": "/b", "remote": "", "last_seen": "2025-01-01T00:00:00Z"},
-    }
-    tree["registry_file"].write_text(json.dumps(registry))
-
-    project = p1
-    ret = _promote_auto(project, force=True, dry_run=True)
-    assert ret == 0
-    out = capsys.readouterr().out
-    assert "DRY RUN" in out
-    assert "cross-project-instinct" in out
-
-    # Verify no file was created
-    assert not (tree["global_personal"] / "cross-project-instinct.yaml").exists()
-
-
-def test_promote_auto_writes_file(patch_globals, capsys):
-    """Auto-promote with force should write global instinct file."""
-    tree = patch_globals
-
-    p1 = _make_project(tree, pid="proj1", pname="project-one")
-    p2 = _make_project(tree, pid="proj2", pname="project-two")
-
-    high_conf_yaml = """\
----
-id: universal-pattern
-trigger: "when coding"
-confidence: 0.85
-domain: general
-scope: project
----
-
-## Action
-Use descriptive variable names.
-"""
-    (p1["instincts_personal"] / "uni.yaml").write_text(high_conf_yaml)
-    (p2["instincts_personal"] / "uni.yaml").write_text(high_conf_yaml)
-
-    registry = {
-        "proj1": {"name": "project-one", "root": "/a", "remote": "", "last_seen": "2025-01-01T00:00:00Z"},
-        "proj2": {"name": "project-two", "root": "/b", "remote": "", "last_seen": "2025-01-01T00:00:00Z"},
-    }
-    tree["registry_file"].write_text(json.dumps(registry))
-
-    ret = _promote_auto(p1, force=True, dry_run=False)
-    assert ret == 0
-
-    promoted = tree["global_personal"] / "universal-pattern.yaml"
-    assert promoted.exists()
-    content = promoted.read_text()
-    assert "scope: global" in content
-    assert "auto-promoted" in content
-
-
-def test_promote_auto_skips_invalid_id(patch_globals, capsys):
-    tree = patch_globals
-
-    p1 = _make_project(tree, pid="proj1", pname="project-one")
-    p2 = _make_project(tree, pid="proj2", pname="project-two")
-
-    bad_id_yaml = """\
----
-id: ../escape
-trigger: "when coding"
-confidence: 0.9
-domain: general
-scope: project
----
-
-## Action
-Invalid id should be skipped.
-"""
-    (p1["instincts_personal"] / "bad.yaml").write_text(bad_id_yaml)
-    (p2["instincts_personal"] / "bad.yaml").write_text(bad_id_yaml)
-
-    registry = {
-        "proj1": {"name": "project-one", "root": "/a", "remote": "", "last_seen": "2025-01-01T00:00:00Z"},
-        "proj2": {"name": "project-two", "root": "/b", "remote": "", "last_seen": "2025-01-01T00:00:00Z"},
-    }
-    tree["registry_file"].write_text(json.dumps(registry))
-
-    ret = _promote_auto(p1, force=True, dry_run=False)
-    assert ret == 0
-    err = capsys.readouterr().err
-    assert "Skipping invalid instinct ID" in err
-    assert not (tree["global_personal"] / "../escape.yaml").exists()
-
-
-# ─────────────────────────────────────────────
-# _find_cross_project_instincts tests
-# ─────────────────────────────────────────────
-
-def test_find_cross_project_empty_registry(patch_globals):
-    tree = patch_globals
-    tree["registry_file"].write_text("{}")
-    result = _find_cross_project_instincts()
-    assert result == {}
-
-
-def test_find_cross_project_single_project(patch_globals):
-    """Single project should return nothing (need 2+)."""
-    tree = patch_globals
-    p1 = _make_project(tree, pid="proj1", pname="project-one")
-    (p1["instincts_personal"] / "inst.yaml").write_text(SAMPLE_INSTINCT_YAML)
-
-    registry = {"proj1": {"name": "project-one", "root": "/a", "remote": "", "last_seen": "2025-01-01T00:00:00Z"}}
-    tree["registry_file"].write_text(json.dumps(registry))
-
-    result = _find_cross_project_instincts()
-    assert result == {}
-
-
-def test_find_cross_project_shared_instinct(patch_globals):
-    """Same instinct ID in 2 projects should be found."""
-    tree = patch_globals
-    p1 = _make_project(tree, pid="proj1", pname="project-one")
-    p2 = _make_project(tree, pid="proj2", pname="project-two")
-
-    (p1["instincts_personal"] / "shared.yaml").write_text(SAMPLE_INSTINCT_YAML)
-    (p2["instincts_personal"] / "shared.yaml").write_text(SAMPLE_INSTINCT_YAML)
-
-    registry = {
-        "proj1": {"name": "project-one", "root": "/a", "remote": "", "last_seen": "2025-01-01T00:00:00Z"},
-        "proj2": {"name": "project-two", "root": "/b", "remote": "", "last_seen": "2025-01-01T00:00:00Z"},
-    }
-    tree["registry_file"].write_text(json.dumps(registry))
-
-    result = _find_cross_project_instincts()
-    assert "test-instinct" in result
-    assert len(result["test-instinct"]) == 2
-
-
-# ─────────────────────────────────────────────
-# load_registry tests
-# ─────────────────────────────────────────────
-
-def test_load_registry_missing_file(patch_globals):
-    result = load_registry()
-    assert result == {}
-
-
-def test_load_registry_corrupt_json(patch_globals):
-    tree = patch_globals
-    tree["registry_file"].write_text("not json at all {{{")
-    result = load_registry()
-    assert result == {}
-
-
-def test_load_registry_valid(patch_globals):
-    tree = patch_globals
-    data = {"abc": {"name": "test", "root": "/test"}}
-    tree["registry_file"].write_text(json.dumps(data))
-    result = load_registry()
-    assert result == data
-
-
-def test_load_registry_uses_utf8_encoding(monkeypatch):
-    calls = []
-
-    def fake_open(path, mode="r", *args, **kwargs):
-        calls.append(kwargs.get("encoding"))
-        return io.StringIO("{}")
-
-    monkeypatch.setattr(_mod, "open", fake_open, raising=False)
-    assert load_registry() == {}
-    assert calls == ["utf-8"]
-
-
-def test_validate_instinct_id():
-    assert _validate_instinct_id("good-id_1.0")
-    assert not _validate_instinct_id("../bad")
-    assert not _validate_instinct_id("bad/name")
-    assert not _validate_instinct_id(".hidden")
-
-
-def test_update_registry_atomic_replaces_file(patch_globals):
-    tree = patch_globals
-    _update_registry("abc123", "demo", "/repo", "https://example.com/repo.git")
-    data = json.loads(tree["registry_file"].read_text())
-    assert "abc123" in data
-    leftovers = list(tree["registry_file"].parent.glob(".projects.json.tmp.*"))
-    assert leftovers == []
-
-
-def test_update_registry_matches_shell_schema(patch_globals):
-    # Issue #2299: the Python writer must emit the same field set as the shell
-    # counterpart in detect-project.sh (id, name, root, remote, created_at,
-    # last_seen) so a projects.json entry has a consistent shape regardless of
-    # which path wrote it.
-    tree = patch_globals
-    _update_registry("abc123", "demo", "/repo", "https://example.com/repo.git")
-    entry = json.loads(tree["registry_file"].read_text())["abc123"]
-    assert set(entry) == {"id", "name", "root", "remote", "created_at", "last_seen"}
-    assert entry["id"] == "abc123"
-    assert entry["name"] == "demo"
-    assert entry["root"] == "/repo"
-    assert entry["remote"] == "https://example.com/repo.git"
-    # On the initial write both timestamps come from the same `now`, so the
-    # first-write contract is created_at == last_seen.
-    assert entry["created_at"]
-    assert entry["created_at"] == entry["last_seen"]
-
-
-def test_update_registry_preserves_created_at(patch_globals):
-    # created_at is stamped on first write and preserved on subsequent updates,
-    # while last_seen advances — matching entry.get("created_at", now) in the
-    # shell counterpart.
-    tree = patch_globals
-    _update_registry("abc123", "demo", "/repo", "https://example.com/repo.git")
-    first = json.loads(tree["registry_file"].read_text())["abc123"]
-
-    _update_registry("abc123", "demo-renamed", "/repo", "https://example.com/repo.git")
-    second = json.loads(tree["registry_file"].read_text())["abc123"]
-
-    assert second["created_at"] == first["created_at"]
-    assert second["name"] == "demo-renamed"
-    assert second["last_seen"] >= first["last_seen"]
-
-
-def test_update_registry_heals_malformed_entry(patch_globals):
-    # Issue #2299 follow-up: a non-dict value for the project id (e.g. a
-    # corrupt registry) must not crash _update_registry. The entry is healed by
-    # the rewrite, preserving the old unconditional-overwrite behavior.
-    tree = patch_globals
-    tree["registry_file"].write_text(json.dumps({"abc123": None}), encoding="utf-8")
-    _update_registry("abc123", "demo", "/repo", "https://example.com/repo.git")
-    entry = json.loads(tree["registry_file"].read_text())["abc123"]
-    assert isinstance(entry, dict)
-    assert entry["id"] == "abc123"
-    assert entry["created_at"]
-    assert entry["created_at"] == entry["last_seen"]
-
-
-def test_update_registry_heals_non_dict_registry(patch_globals):
-    # Issue #2299 follow-up: a top-level registry that is valid JSON but not a
-    # mapping (e.g. a list or string from a corrupt projects.json) must not
-    # crash _update_registry before the per-entry guard runs. The whole file is
-    # healed by the rewrite, preserving the old unconditional-overwrite behavior.
-    tree = patch_globals
-    tree["registry_file"].write_text(json.dumps(["oops"]), encoding="utf-8")
-    _update_registry("abc123", "demo", "/repo", "https://example.com/repo.git")
-    registry = json.loads(tree["registry_file"].read_text())
-    assert isinstance(registry, dict)
-    entry = registry["abc123"]
-    assert entry["id"] == "abc123"
-    assert entry["created_at"] == entry["last_seen"]
-
-
-def test_write_registry_atomic_no_tmp_leftovers(patch_globals):
-    # Issue #2294: _write_registry now holds the registry lock like
-    # _update_registry. It must still write atomically with no stray tmp files.
-    tree = patch_globals
-    _write_registry({"keep": {"name": "demo", "root": "/repo", "remote": ""}})
-    data = json.loads(tree["registry_file"].read_text())
-    assert data == {"keep": {"name": "demo", "root": "/repo", "remote": ""}}
-    leftovers = list(tree["registry_file"].parent.glob(".projects.json.tmp.*"))
-    assert leftovers == []
-
-
-def test_remove_project_storage_deletes_contained_dir(patch_globals):
-    tree = patch_globals
-    target = tree["projects_dir"] / "proj-1"
-    (target / "instincts").mkdir(parents=True)
-    (target / "instincts" / "x.md").write_text("hi", encoding="utf-8")
-    _remove_project_storage("proj-1")
-    assert not target.exists()
-
-
-def test_remove_project_storage_missing_dir_is_noop(patch_globals):
-    # No raise when the contained dir simply does not exist.
-    _remove_project_storage("never-created")
-
-
-def test_remove_project_storage_blocks_traversal(patch_globals):
-    # Issue #2297: defense-in-depth — a traversal id must be refused even when a
-    # caller skips _validate_project_id, so this can never delete outside
-    # PROJECTS_DIR.
-    with pytest.raises(ValueError):
-        _remove_project_storage("../../etc")
-
-
-def test_remove_project_storage_blocks_root_itself(patch_globals):
-    with pytest.raises(ValueError):
-        _remove_project_storage(".")
-
-
-# ─────────────────────────────────────────────
-# Issue #2302 coverage:
-#   _normalize_remote_url, _promote_specific dry-run,
-#   projects delete/gc/merge, cmd_prune
-# ─────────────────────────────────────────────
-
-_normalize_remote_url = _mod._normalize_remote_url
-_cmd_projects_delete = _mod._cmd_projects_delete
-_cmd_projects_gc = _mod._cmd_projects_gc
-_cmd_projects_merge = _mod._cmd_projects_merge
-cmd_prune = _mod.cmd_prune
-
-
-# ── _normalize_remote_url ────────────────────
-
-def test_normalize_remote_url_empty_returns_empty():
-    assert _normalize_remote_url("") == ""
-    assert _normalize_remote_url(None) == ""
-
-
-def test_normalize_remote_url_scp_form():
-    # scp-style host:path -> host/path, credentials/.git stripped, lowercased
-    assert _normalize_remote_url("git@github.com:Test/Repo.git") == "github.com/test/repo"
-
-
-def test_normalize_remote_url_https_strips_credentials_and_scheme():
-    assert (
-        _normalize_remote_url("https://user:token@github.com/test/repo.git")
-        == "github.com/test/repo"
-    )
-
-
-def test_normalize_remote_url_network_is_lowercased():
-    assert _normalize_remote_url("https://GitHub.com/Owner/Project") == "github.com/owner/project"
-
-
-def test_normalize_remote_url_trailing_slash_and_dotgit_stripped():
-    assert _normalize_remote_url("https://github.com/a/b.git/") == "github.com/a/b"
-
-
-def test_normalize_remote_url_file_scheme_preserves_case():
-    # Local file paths are not network URLs: scheme is stripped but case is preserved.
-    assert _normalize_remote_url("file:///srv/Repos/My-Repo/") == "/srv/Repos/My-Repo"
-
-
-def test_normalize_remote_url_idempotent():
-    once = _normalize_remote_url("https://user@github.com/Test/Repo.git")
-    assert _normalize_remote_url(once) == once
-
-
-# ── _promote_specific dry-run ────────────────
-
-def test_promote_specific_dry_run_writes_nothing(patch_globals, capsys):
-    """dry_run returns 0, prints [DRY RUN], and writes no global file."""
-    tree = patch_globals
-    project = _make_project(tree)
-    (project["instincts_personal"] / "inst.yaml").write_text(SAMPLE_INSTINCT_YAML)
-
-    ret = _promote_specific(project, "test-instinct", force=True, dry_run=True)
-    assert ret == 0
-    out = capsys.readouterr().out
-    assert "[DRY RUN]" in out
-    assert not (tree["global_personal"] / "test-instinct.yaml").exists()
-    assert list(tree["global_personal"].iterdir()) == []
-
-
-# ── projects delete ──────────────────────────
-
-def test_projects_delete_rejects_invalid_id(patch_globals, capsys):
-    args = SimpleNamespace(project_id="../escape", dry_run=False, force=True)
-    assert _cmd_projects_delete(args) == 1
-    assert "Invalid project ID" in capsys.readouterr().err
-
-
-def test_projects_delete_not_found(patch_globals, capsys):
-    args = SimpleNamespace(project_id="ghost123", dry_run=False, force=True)
-    assert _cmd_projects_delete(args) == 1
-    assert "not found" in capsys.readouterr().err
-
-
-def test_projects_delete_dry_run_keeps_registry_and_storage(patch_globals, capsys):
-    tree = patch_globals
-    _make_project(tree, pid="proj1", pname="p1")
-    tree["registry_file"].write_text(json.dumps({"proj1": {"name": "p1"}}))
-
-    args = SimpleNamespace(project_id="proj1", dry_run=True, force=False)
-    assert _cmd_projects_delete(args) == 0
-    assert "[DRY RUN]" in capsys.readouterr().out
-    assert (tree["projects_dir"] / "proj1").exists()
-    assert "proj1" in json.loads(tree["registry_file"].read_text())
-
-
-def test_projects_delete_force_removes_registry_and_storage(patch_globals, capsys):
-    tree = patch_globals
-    _make_project(tree, pid="proj1", pname="p1")
-    tree["registry_file"].write_text(json.dumps({"proj1": {"name": "p1"}}))
-
-    args = SimpleNamespace(project_id="proj1", dry_run=False, force=True)
-    assert _cmd_projects_delete(args) == 0
-    assert "Deleted project" in capsys.readouterr().out
-    assert not (tree["projects_dir"] / "proj1").exists()
-    assert "proj1" not in json.loads(tree["registry_file"].read_text())
-
-
-# ── projects gc ──────────────────────────────
-
-def test_projects_gc_no_candidates(patch_globals, capsys):
-    tree = patch_globals
-    tree["registry_file"].write_text("{}")
-    args = SimpleNamespace(dry_run=False, force=True)
-    assert _cmd_projects_gc(args) == 0
-    assert "No zero-value project entries" in capsys.readouterr().out
-
-
-def test_projects_gc_dry_run_keeps_entry(patch_globals, capsys):
-    tree = patch_globals
-    _make_project(tree, pid="empty1", pname="e1")  # zero instincts/observations
-    tree["registry_file"].write_text(json.dumps({"empty1": {"name": "e1"}}))
-
-    args = SimpleNamespace(dry_run=True, force=False)
-    assert _cmd_projects_gc(args) == 0
-    assert "[DRY RUN]" in capsys.readouterr().out
-    assert "empty1" in json.loads(tree["registry_file"].read_text())
-    # dry-run must not touch storage on disk
-    assert (tree["projects_dir"] / "empty1").exists()
-
-
-def test_projects_gc_force_removes_only_zero_value(patch_globals, capsys):
-    tree = patch_globals
-    _make_project(tree, pid="empty1", pname="e1")
-    full = _make_project(tree, pid="full1", pname="f1")
-    (full["instincts_personal"] / "inst.yaml").write_text(SAMPLE_INSTINCT_YAML)
-    tree["registry_file"].write_text(
-        json.dumps({"empty1": {"name": "e1"}, "full1": {"name": "f1"}})
-    )
-
-    args = SimpleNamespace(dry_run=False, force=True)
-    assert _cmd_projects_gc(args) == 0
-    reg = json.loads(tree["registry_file"].read_text())
-    assert "empty1" not in reg
-    assert "full1" in reg
-    assert not (tree["projects_dir"] / "empty1").exists()
-    assert (tree["projects_dir"] / "full1").exists()
-
-
-# ── projects merge ───────────────────────────
-
-def test_projects_merge_rejects_same_id(patch_globals, capsys):
-    args = SimpleNamespace(from_id="dup", into_id="dup", dry_run=False, force=True)
-    assert _cmd_projects_merge(args) == 1
-    assert "into itself" in capsys.readouterr().err
-
-
-def test_projects_merge_missing_source(patch_globals, capsys):
-    tree = patch_globals
-    tree["registry_file"].write_text(json.dumps({"dest": {"name": "d"}}))
-    args = SimpleNamespace(from_id="src", into_id="dest", dry_run=False, force=True)
-    assert _cmd_projects_merge(args) == 1
-    assert "Source project" in capsys.readouterr().err
-
-
-def test_projects_merge_missing_destination(patch_globals, capsys):
-    tree = patch_globals
-    # Source present, destination absent — exercises the symmetric error branch.
-    tree["registry_file"].write_text(json.dumps({"src": {"name": "s"}}))
-    args = SimpleNamespace(from_id="src", into_id="dest", dry_run=False, force=True)
-    assert _cmd_projects_merge(args) == 1
-    assert "Destination project" in capsys.readouterr().err
-
-
-def test_projects_merge_dry_run_no_changes(patch_globals, capsys):
-    tree = patch_globals
-    src = _make_project(tree, pid="src", pname="s")
-    _make_project(tree, pid="dest", pname="d")
-    (src["instincts_personal"] / "i.yaml").write_text(SAMPLE_INSTINCT_YAML)
-    tree["registry_file"].write_text(json.dumps({"src": {"name": "s"}, "dest": {"name": "d"}}))
-
-    args = SimpleNamespace(from_id="src", into_id="dest", dry_run=True, force=False)
-    assert _cmd_projects_merge(args) == 0
-    assert "[DRY RUN]" in capsys.readouterr().out
-    reg = json.loads(tree["registry_file"].read_text())
-    assert "src" in reg and "dest" in reg
-    assert (tree["projects_dir"] / "src").exists()
-    # dry-run must not copy any instinct into the destination storage
-    assert not list((tree["projects_dir"] / "dest" / "instincts" / "personal").glob("*.yaml"))
-
-
-def test_projects_merge_force_moves_and_removes_source(patch_globals, capsys):
-    tree = patch_globals
-    src = _make_project(tree, pid="src", pname="s")
-    _make_project(tree, pid="dest", pname="d")
-    (src["instincts_personal"] / "i.yaml").write_text(SAMPLE_INSTINCT_YAML)
-    tree["registry_file"].write_text(json.dumps({"src": {"name": "s"}, "dest": {"name": "d"}}))
-
-    args = SimpleNamespace(from_id="src", into_id="dest", dry_run=False, force=True)
-    assert _cmd_projects_merge(args) == 0
-    reg = json.loads(tree["registry_file"].read_text())
-    assert "src" not in reg
-    assert "dest" in reg
-    assert not (tree["projects_dir"] / "src").exists()
-    moved = list((tree["projects_dir"] / "dest" / "instincts" / "personal").glob("*.yaml"))
-    assert len(moved) >= 1
-
-
-# ── cmd_prune ────────────────────────────────
-
-def _pending_item(path, age_days):
-    return {
-        "path": path,
-        "created": None,
-        "age_days": age_days,
-        "name": path.stem,
-        "parent_dir": str(path.parent),
-    }
-
-
-def test_cmd_prune_dry_run_keeps_files(monkeypatch, tmp_path, capsys):
-    f_old = tmp_path / "old.yaml"
-    f_old.write_text("x", encoding="utf-8")
-    f_new = tmp_path / "new.yaml"
-    f_new.write_text("y", encoding="utf-8")
-    items = [_pending_item(f_old, 40), _pending_item(f_new, 5)]
-    monkeypatch.setattr(_mod, "_collect_pending_instincts", lambda: items)
-
-    args = SimpleNamespace(max_age=30, dry_run=True, quiet=False)
-    assert cmd_prune(args) == 0
-    assert "[DRY RUN]" in capsys.readouterr().out
-    assert f_old.exists()
-    assert f_new.exists()
-
-
-def test_cmd_prune_deletes_only_expired(monkeypatch, tmp_path, capsys):
-    f_old = tmp_path / "old.yaml"
-    f_old.write_text("x", encoding="utf-8")
-    f_new = tmp_path / "new.yaml"
-    f_new.write_text("y", encoding="utf-8")
-    items = [_pending_item(f_old, 40), _pending_item(f_new, 5)]
-    monkeypatch.setattr(_mod, "_collect_pending_instincts", lambda: items)
-
-    args = SimpleNamespace(max_age=30, dry_run=False, quiet=False)
-    assert cmd_prune(args) == 0
-    assert not f_old.exists()
-    assert f_new.exists()
-    assert "Pruned 1" in capsys.readouterr().out
-
-
-def test_cmd_prune_quiet_suppresses_output(monkeypatch, tmp_path, capsys):
-    f_old = tmp_path / "old.yaml"
-    f_old.write_text("x", encoding="utf-8")
-    items = [_pending_item(f_old, 99)]
-    monkeypatch.setattr(_mod, "_collect_pending_instincts", lambda: items)
-
-    args = SimpleNamespace(max_age=30, dry_run=False, quiet=True)
-    assert cmd_prune(args) == 0
-    assert not f_old.exists()
-    captured = capsys.readouterr()
-    assert captured.out == ""
-    assert captured.err == ""
-
-
-def test_cmd_prune_empty_pending_nothing_to_do(monkeypatch, capsys):
-    # Nothing pending at all: the non-dry-run, non-quiet branch must report
-    # "nothing to do" (not "[DRY RUN]"), return 0, and not crash.
-    monkeypatch.setattr(_mod, "_collect_pending_instincts", lambda: [])
-
-    args = SimpleNamespace(max_age=30, dry_run=False, quiet=False)
-    assert cmd_prune(args) == 0
-    out = capsys.readouterr().out
-    assert "No pending instincts older than 30 days." in out
-    assert "[DRY RUN]" not in out