mapify-cli 1.6.0__tar.gz → 1.6.1__tar.gz

{mapify_cli-1.6.0 → mapify_cli-1.6.1}/.gitignore

@@ -37,8 +37,10 @@ htmlcov/
 
 # ACE Playbook - user-specific data
 .claude/embeddings_cache/
+# LEGACY: playbook.json entries for migration support (ignore old format files)
 .claude/playbook.json
 .claude/playbook.json.backup.*
+# Current format: SQLite database
 .claude/playbook.db
 .claude/playbook.db-shm
 .claude/playbook.db-wal

{mapify_cli-1.6.0 → mapify_cli-1.6.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mapify-cli
-Version: 1.6.0
+Version: 1.6.1
 Summary: MAP Framework installer - Modular Agentic Planner for Claude Code
 Project-URL: Homepage, https://github.com/azalio/map-framework
 Project-URL: Repository, https://github.com/azalio/map-framework.git

{mapify_cli-1.6.0 → mapify_cli-1.6.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "mapify-cli"
-version = "1.6.0"
+version = "1.6.1"
 description = "MAP Framework installer - Modular Agentic Planner for Claude Code"
 authors = [{ name = "MAP Framework Contributors" }]
 readme = "README.md"

{mapify_cli-1.6.0 → mapify_cli-1.6.1}/src/mapify_cli/__init__.py

@@ -23,7 +23,7 @@ Or install globally:
     mapify check
 """
 
-__version__ = "1.6.0"
+__version__ = "1.6.1"
 
 import os
 import subprocess
@@ -2046,6 +2046,48 @@ def init(
         from mapify_cli.playbook_manager import PlaybookManager
 
         playbook_db_path = project_path / ".claude" / "playbook.db"
+        playbook_json_path = project_path / ".claude" / "playbook.json"
+
+        # When --force is used and playbook.json exists, handle migration scenarios
+        if force and playbook_json_path.exists() and playbook_db_path.exists():
+            # Check if the existing DB has valid schema (requires both bullets and metadata tables)
+            db_is_valid = False
+            try:
+                test_conn = sqlite3.connect(str(playbook_db_path))
+                cursor = test_conn.cursor()
+                # Check for required tables
+                cursor.execute(
+                    "SELECT COUNT(*) FROM sqlite_master WHERE type='table' AND name IN ('bullets', 'metadata')"
+                )
+                table_count = cursor.fetchone()[0]
+                test_conn.close()
+                db_is_valid = table_count == 2  # Both tables must exist
+            except sqlite3.Error:
+                db_is_valid = False
+
+            if not db_is_valid:
+                # DB is missing required schema or corrupted - remove it to allow migration
+                try:
+                    playbook_db_path.unlink()
+                    console.print(
+                        "[yellow]Removing incomplete playbook.db to migrate from playbook.json[/yellow]"
+                    )
+                except OSError:
+                    pass  # If we can't remove it, let PlaybookManager handle the error
+            else:
+                # DB is valid but playbook.json also exists - remove stale JSON to avoid confusion
+                # Create backup first in case user needs it
+                backup_path = str(playbook_json_path) + ".stale"
+                try:
+                    import shutil
+
+                    shutil.move(str(playbook_json_path), backup_path)
+                    console.print(
+                        f"[yellow]Moved stale playbook.json to {backup_path} (valid playbook.db already exists)[/yellow]"
+                    )
+                except OSError:
+                    pass  # If we can't move it, it's not critical
+
         manager = PlaybookManager(
             db_path=str(playbook_db_path), use_semantic_search=False
         )

{mapify_cli-1.6.0 → mapify_cli-1.6.1}/src/mapify_cli/playbook_manager.py

@@ -81,9 +81,11 @@ class PlaybookManager:
                 self.db_path = Path(".claude/playbook.db")
             else:
                 self.db_path = Path(db_path)
-            self.playbook_path = (
-                self.db_path.parent / "playbook.json"
-            )  # For migration check only
+            # LEGACY: playbook.json path for migration support only.
+            # This allows users upgrading from older versions to have their
+            # playbook.json automatically migrated to playbook.db.
+            # DO NOT use playbook_path for new functionality.
+            self.playbook_path = self.db_path.parent / "playbook.json"
 
         # Check if DB exists, if not but JSON exists → migrate
         if not self.db_path.exists() and self.playbook_path.exists():
@@ -449,10 +451,17 @@ class PlaybookManager:
 
     def _migrate_json_to_sqlite(self) -> None:
         """
-        Migrate existing playbook.json to SQLite database.
+        LEGACY MIGRATION: Migrate existing playbook.json to SQLite database.
+
+        This method is intentionally preserved for backward compatibility.
+        It allows users upgrading from MAP Framework versions < 2.2 to have
+        their playbook.json automatically migrated to the new playbook.db format.
+
+        The references to playbook.json in this method are intentional and
+        should NOT be removed - they are part of the migration logic.
 
         Steps:
-        1. Load playbook.json
+        1. Load legacy playbook.json
         2. Create SQLite schema
         3. Insert all bullets into bullets table
         4. Insert metadata
@@ -565,7 +574,7 @@ class PlaybookManager:
 
         conn.close()
 
-        # Create backup of JSON
+        # Create backup of JSON and remove original
         backup_path = (
             str(self.playbook_path)
             + f".backup.{datetime.now(timezone.utc).strftime('%Y%m%d_%H%M%S')}"
@@ -578,6 +587,20 @@ class PlaybookManager:
         )
         print(f"✅ JSON backup saved to {backup_path}", file=sys.stderr)
 
+        # Remove original playbook.json to avoid confusion
+        # The backup is always available if needed
+        try:
+            self.playbook_path.unlink()
+            print(
+                f"✅ Removed {self.playbook_path} (backup preserved)",
+                file=sys.stderr,
+            )
+        except OSError as e:
+            print(
+                f"⚠️ Could not remove {self.playbook_path}: {e}",
+                file=sys.stderr,
+            )
+
     def _load_playbook_from_db(self) -> Dict:
         """Load playbook structure from SQLite database for backward compatibility."""
         cursor = self.db_conn.cursor()

{mapify_cli-1.6.0 → mapify_cli-1.6.1}/src/mapify_cli/recitation_manager.py

@@ -586,6 +586,8 @@ class RecitationManager:
             from mapify_cli.playbook_manager import PlaybookManager
 
             playbook_db_path = self.project_root / ".claude" / "playbook.db"
+            # LEGACY: playbook_json_path is only used for backward compatibility
+            # to support migration from older versions. DO NOT use for new functionality.
             playbook_json_path = self.project_root / ".claude" / "playbook.json"
 
             # Check if playbook exists (prefer .db, fall back to .json for backward compatibility)

{mapify_cli-1.6.0 → mapify_cli-1.6.1}/src/mapify_cli/relationship_detector.py

@@ -29,7 +29,7 @@ class RelationshipType(Enum):
     USES = "USES"  # A uses B (pytest USES Python)
     DEPENDS_ON = "DEPENDS_ON"  # A depends on B (MAP-workflow DEPENDS_ON playbook.db)
     CONTRADICTS = "CONTRADICTS"  # A contradicts B (generic-exception CONTRADICTS specific-exceptions)
-    SUPERSEDES = "SUPERSEDES"  # A replaces B (playbook.db SUPERSEDES playbook.json)
+    SUPERSEDES = "SUPERSEDES"  # A replaces B (SQLite SUPERSEDES JSON-storage)
     RELATED_TO = "RELATED_TO"  # Generic relationship (fallback)
 
     # Bonus 4 types (for comprehensive graph)

{mapify_cli-1.6.0 → mapify_cli-1.6.1}/src/mapify_cli/schemas.py

@@ -98,7 +98,7 @@ CREATE TABLE IF NOT EXISTS relationships (
         'USES',          -- Entity A uses Entity B (e.g., 'pytest' USES 'Python')
         'DEPENDS_ON',    -- A depends on B (e.g., 'MAP-workflow' DEPENDS_ON 'playbook.db')
         'CONTRADICTS',   -- A contradicts B (e.g., 'generic-exception' CONTRADICTS 'specific-exceptions')
-        'SUPERSEDES',    -- A replaces B (e.g., 'playbook.db' SUPERSEDES 'playbook.json')
+        'SUPERSEDES',    -- A replaces B (e.g., 'SQLite' SUPERSEDES 'JSON-storage')
         'RELATED_TO',    -- Generic relationship (fallback)
         'IMPLEMENTS',    -- A implements pattern B (e.g., 'retry-logic' IMPLEMENTS 'resilience-pattern')
         'CAUSES',        -- A causes B (e.g., 'race-condition' CAUSES 'data-corruption')

{mapify_cli-1.6.0 → mapify_cli-1.6.1}/src/mapify_cli/templates/agents/curator.md

@@ -339,7 +339,7 @@ mapify playbook apply-delta operations.json --dry-run
 - ❌ `sqlite3 .claude/playbook.db "UPDATE bullets SET..."` → Direct SQL bypasses validation
 - ❌ `Edit(.claude/playbook.db, ...)` → Cannot edit binary database
 - ❌ Using "op" field → ✅ Correct field name is "type"
-- ❌ Reading/writing playbook.json → ✅ Migrated to playbook.db (SQLite)
+- ❌ Using legacy JSON format → ✅ Use playbook.db (SQLite)
 
 **Why apply-delta is mandatory**:
 - Validates operations before applying

{mapify_cli-1.6.0 → mapify_cli-1.6.1}/src/mapify_cli/templates/skills/map-cli-reference/SKILL.md

@@ -124,9 +124,9 @@ mapify upgrade
 ✅ **CORRECT**: `mapify playbook apply-delta operations.json`
 📝 **Explanation**: Cannot edit binary SQLite database. Generate delta operations JSON and apply via CLI.
 
-❌ **WRONG**: Reading/writing `playbook.json`
+❌ **WRONG**: Using legacy JSON format for playbook
 ✅ **CORRECT**: `mapify playbook query "..."`
-📝 **Explanation**: `playbook.json` is deprecated (migrated to `playbook.db`). Use CLI commands to interact with playbook.
+📝 **Explanation**: Playbook uses SQLite database (`playbook.db`). Use CLI commands to interact with playbook.
 
 ---