devsquad 4.5.17__tar.gz → 4.5.18__tar.gz

{devsquad-4.5.17 → devsquad-4.5.18}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: devsquad
-Version: 4.5.17
+Version: 4.5.18
 Summary: Production-Ready Multi-Role AI Task Orchestrator
 Author-email: DevSquad Team <devsquad@example.com>
 License: MIT
@@ -106,7 +106,7 @@ Dynamic: license-file
   <img alt="Python" src="https://img.shields.io/badge/Python-3.10+-blue?logo=python&logoColor=white" />
   <img alt="License" src="https://img.shields.io/badge/License-MIT-green" />
   <img alt="Tests" src="https://img.shields.io/badge/Tests-9400%2B%20passing-brightgreen" />
-  <img alt="Version" src="https://img.shields.io/badge/V4.5.17-success" />
+  <img alt="Version" src="https://img.shields.io/badge/V4.5.18-success" />
   <img alt="CI" src="https://img.shields.io/badge/CI-GitHub_Actions-blue?logo=githubactions" />
   <img alt="Quality" src="https://img.shields.io/badge/Code%20Quality-4.3%2F5%20%E2%98%85%E2%98%85%E2%98%85%E2%98%85%E2%98%86-blue" />
   <img alt="Security" src="https://img.shields.io/badge/Security-5%2F5%20%E2%98%85%E2%98%85%E2%98%85%E2%98%85%E2%98%85-success" />
@@ -638,7 +638,7 @@ devsquad dispatch -t "Design user authentication system"
 ```bash
 # Check version
 devsquad --version
-# Expected: devsquad 4.5.17
+# Expected: devsquad 4.5.18
 
 # Run tests
 pytest tests/ -v --tb=short
@@ -693,7 +693,7 @@ export DEVSQUAD_OPENAI_API_KEY=sk-...
 
 ### Quick Smoke Test (< 30 seconds)
 ```bash
-python3 scripts/cli.py --version       # Expected: DevSquad 4.5.17
+python3 scripts/cli.py --version       # Expected: DevSquad 4.5.18
 python3 scripts/cli.py status          # Expected: System ready
 python3 scripts/cli.py roles           # Expected: 7 core roles listed
 ```

{devsquad-4.5.17 → devsquad-4.5.18}/README.md

@@ -10,7 +10,7 @@
   <img alt="Python" src="https://img.shields.io/badge/Python-3.10+-blue?logo=python&logoColor=white" />
   <img alt="License" src="https://img.shields.io/badge/License-MIT-green" />
   <img alt="Tests" src="https://img.shields.io/badge/Tests-9400%2B%20passing-brightgreen" />
-  <img alt="Version" src="https://img.shields.io/badge/V4.5.17-success" />
+  <img alt="Version" src="https://img.shields.io/badge/V4.5.18-success" />
   <img alt="CI" src="https://img.shields.io/badge/CI-GitHub_Actions-blue?logo=githubactions" />
   <img alt="Quality" src="https://img.shields.io/badge/Code%20Quality-4.3%2F5%20%E2%98%85%E2%98%85%E2%98%85%E2%98%85%E2%98%86-blue" />
   <img alt="Security" src="https://img.shields.io/badge/Security-5%2F5%20%E2%98%85%E2%98%85%E2%98%85%E2%98%85%E2%98%85-success" />
@@ -542,7 +542,7 @@ devsquad dispatch -t "Design user authentication system"
 ```bash
 # Check version
 devsquad --version
-# Expected: devsquad 4.5.17
+# Expected: devsquad 4.5.18
 
 # Run tests
 pytest tests/ -v --tb=short
@@ -597,7 +597,7 @@ export DEVSQUAD_OPENAI_API_KEY=sk-...
 
 ### Quick Smoke Test (< 30 seconds)
 ```bash
-python3 scripts/cli.py --version       # Expected: DevSquad 4.5.17
+python3 scripts/cli.py --version       # Expected: DevSquad 4.5.18
 python3 scripts/cli.py status          # Expected: System ready
 python3 scripts/cli.py roles           # Expected: 7 core roles listed
 ```

{devsquad-4.5.17 → devsquad-4.5.18}/devsquad.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: devsquad
-Version: 4.5.17
+Version: 4.5.18
 Summary: Production-Ready Multi-Role AI Task Orchestrator
 Author-email: DevSquad Team <devsquad@example.com>
 License: MIT
@@ -106,7 +106,7 @@ Dynamic: license-file
   <img alt="Python" src="https://img.shields.io/badge/Python-3.10+-blue?logo=python&logoColor=white" />
   <img alt="License" src="https://img.shields.io/badge/License-MIT-green" />
   <img alt="Tests" src="https://img.shields.io/badge/Tests-9400%2B%20passing-brightgreen" />
-  <img alt="Version" src="https://img.shields.io/badge/V4.5.17-success" />
+  <img alt="Version" src="https://img.shields.io/badge/V4.5.18-success" />
   <img alt="CI" src="https://img.shields.io/badge/CI-GitHub_Actions-blue?logo=githubactions" />
   <img alt="Quality" src="https://img.shields.io/badge/Code%20Quality-4.3%2F5%20%E2%98%85%E2%98%85%E2%98%85%E2%98%85%E2%98%86-blue" />
   <img alt="Security" src="https://img.shields.io/badge/Security-5%2F5%20%E2%98%85%E2%98%85%E2%98%85%E2%98%85%E2%98%85-success" />
@@ -638,7 +638,7 @@ devsquad dispatch -t "Design user authentication system"
 ```bash
 # Check version
 devsquad --version
-# Expected: devsquad 4.5.17
+# Expected: devsquad 4.5.18
 
 # Run tests
 pytest tests/ -v --tb=short
@@ -693,7 +693,7 @@ export DEVSQUAD_OPENAI_API_KEY=sk-...
 
 ### Quick Smoke Test (< 30 seconds)
 ```bash
-python3 scripts/cli.py --version       # Expected: DevSquad 4.5.17
+python3 scripts/cli.py --version       # Expected: DevSquad 4.5.18
 python3 scripts/cli.py status          # Expected: System ready
 python3 scripts/cli.py roles           # Expected: 7 core roles listed
 ```

{devsquad-4.5.17 → devsquad-4.5.18}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "devsquad"
-version = "4.5.17"
+version = "4.5.18"
 description = "Production-Ready Multi-Role AI Task Orchestrator"
 readme = "README.md"
 license = {text = "MIT"}

devsquad-4.5.18/scripts/collaboration/_version.py

@@ -0,0 +1,2 @@
+__version__ = "4.5.18"
+__version_info__ = (4, 5, 18)

{devsquad-4.5.17 → devsquad-4.5.18}/scripts/collaboration/ccr_store.py

@@ -66,7 +66,15 @@ class CCRStore:
             str(self._db_path),
             check_same_thread=False,
         )
+        # V4.5.18: extend the V4.5.17 PRAGMA profile (originally added for
+        # code_graph_storage) to CCR's content-cache store. WAL keeps
+        # journal append-only, NORMAL defers fsync to checkpoint (still
+        # durable on power-loss for committed transactions), and
+        # busy_timeout=5000 makes concurrent store/retrieve calls queue
+        # instead of raising immediately when the writer holds the lock.
         conn.execute("PRAGMA journal_mode=WAL")
+        conn.execute("PRAGMA synchronous=NORMAL")
+        conn.execute("PRAGMA busy_timeout=5000")
         return conn
 
     def _init_schema(self) -> None:

{devsquad-4.5.17 → devsquad-4.5.18}/scripts/collaboration/skill_registry.py

@@ -2,6 +2,7 @@
 import hashlib
 import json
 import logging
+import threading
 from collections.abc import Callable
 from dataclasses import dataclass, field
 from datetime import datetime
@@ -14,6 +15,15 @@ if TYPE_CHECKING:
 logger = logging.getLogger(__name__)
 
 
+# V4.5.18: reentrant lock protecting the in-memory `skills` / `handlers` dicts
+# and the `_save` iteration. Without it, concurrent register()/unregister()
+# callers can race on `for s in self.skills.values()` inside `_save`,
+# raising `RuntimeError: dictionary changed size during iteration` and
+# causing the integration test
+# `tests/integration/test_skill_registry_integration.py::test_06_concurrent_register_is_thread_safe`
+# to fail intermittently on fast CI runners.
+
+
 @dataclass
 class SkillEntry:
     skill_id: str = field(
@@ -89,6 +99,8 @@ class SkillRegistry:
         self.storage_path.mkdir(parents=True, exist_ok=True)
         self.skills: dict[str, SkillEntry] = {}
         self.handlers: dict[str, Callable] = {}
+        # V4.5.18: reentrant lock guarding skills/handlers mutation + _save iteration.
+        self._lock = threading.RLock()
         self._load()
         # V4.5.0: protocol-native skill provider. Default = BuiltinSkillProvider,
         # which wraps the existing import-based registry (100% backward compatible).
@@ -154,10 +166,11 @@ class SkillRegistry:
         """
         if ".." in skill.skill_id or "/" in skill.skill_id or "\\" in skill.skill_id:
             raise ValueError(f"Invalid skill_id: {skill.skill_id}")
-        self.skills[skill.skill_id] = skill
-        if handler:
-            self.handlers[skill.skill_id] = handler
-        self._save()
+        with self._lock:
+            self.skills[skill.skill_id] = skill
+            if handler:
+                self.handlers[skill.skill_id] = handler
+            self._save()
         logger.info("Skill registered: %s (%s)", skill.name, skill.skill_id)
         return skill.skill_id
 
@@ -170,12 +183,13 @@ class SkillRegistry:
         Returns:
             True when the skill existed and was removed, False otherwise.
         """
-        if skill_id in self.skills:
-            del self.skills[skill_id]
-            self.handlers.pop(skill_id, None)
-            self._save()
-            return True
-        return False
+        with self._lock:
+            if skill_id in self.skills:
+                del self.skills[skill_id]
+                self.handlers.pop(skill_id, None)
+                self._save()
+                return True
+            return False
 
     def get(self, skill_id: str) -> SkillEntry | None:
         """Retrieve a skill entry by id.
@@ -186,7 +200,8 @@ class SkillRegistry:
         Returns:
             The matching SkillEntry, or None when not found.
         """
-        return self.skills.get(skill_id)
+        with self._lock:
+            return self.skills.get(skill_id)
 
     def execute(self, skill_id: str, **kwargs: Any) -> Any:
         """Execute a registered skill by id.
@@ -201,17 +216,18 @@ class SkillRegistry:
         Raises:
             ValueError: When the skill or its handler is not found.
         """
-        skill = self.skills.get(skill_id)
-        if not skill:
-            raise ValueError(f"Skill not found: {skill_id}")
+        with self._lock:
+            skill = self.skills.get(skill_id)
+            if not skill:
+                raise ValueError(f"Skill not found: {skill_id}")
 
-        handler = self.handlers.get(skill_id)
-        if not handler:
-            raise ValueError(f"No handler for skill: {skill_id}")
+            handler = self.handlers.get(skill_id)
+            if not handler:
+                raise ValueError(f"No handler for skill: {skill_id}")
 
-        skill.usage_count += 1
-        skill.last_used = datetime.now().isoformat()
-        self._save()
+            skill.usage_count += 1
+            skill.last_used = datetime.now().isoformat()
+            self._save()
 
         return handler(**kwargs)
 
@@ -227,16 +243,17 @@ class SkillRegistry:
         Returns:
             List of matching SkillEntry objects sorted by confidence desc.
         """
-        results = list(self.skills.values())
-        if category:
-            results = [s for s in results if s.category == category]
-        if tags:
-            results = [s for s in results if any(t in s.tags for t in tags)]
-        if query:
-            q = query.lower()
-            results = [s for s in results if q in s.name.lower() or q in s.description.lower()]
-        results.sort(key=lambda s: s.confidence, reverse=True)
-        return results
+        with self._lock:
+            results = list(self.skills.values())
+            if category:
+                results = [s for s in results if s.category == category]
+            if tags:
+                results = [s for s in results if any(t in s.tags for t in tags)]
+            if query:
+                q = query.lower()
+                results = [s for s in results if q in s.name.lower() or q in s.description.lower()]
+            results.sort(key=lambda s: s.confidence, reverse=True)
+            return results
 
     def propose_from_result(
         self, name: str, description: str, category: str = "", confidence: float = 0.0, tags: list[str] | None = None
@@ -272,10 +289,11 @@ class SkillRegistry:
         Returns:
             List of skill dictionaries produced by :meth:`SkillEntry.to_dict`.
         """
-        skills = list(self.skills.values())
-        if category:
-            skills = [s for s in skills if s.category == category]
-        return [s.to_dict() for s in skills]
+        with self._lock:
+            skills = list(self.skills.values())
+            if category:
+                skills = [s for s in skills if s.category == category]
+            return [s.to_dict() for s in skills]
 
     def get_stats(self) -> dict[str, Any]:
         """Return summary statistics for the registry.
@@ -284,14 +302,15 @@ class SkillRegistry:
             Dictionary with total skill count, per-category counts, and
             the number of skills with registered handlers.
         """
-        categories: dict[str, int] = {}
-        for s in self.skills.values():
-            categories[s.category] = categories.get(s.category, 0) + 1
-        return {
-            "total_skills": len(self.skills),
-            "categories": categories,
-            "with_handlers": len(self.handlers),
-        }
+        with self._lock:
+            categories: dict[str, int] = {}
+            for s in self.skills.values():
+                categories[s.category] = categories.get(s.category, 0) + 1
+            return {
+                "total_skills": len(self.skills),
+                "categories": categories,
+                "with_handlers": len(self.handlers),
+            }
 
     def _load(self) -> None:
         registry_file = self.storage_path / "registry.json"
@@ -308,7 +327,8 @@ class SkillRegistry:
     def _save(self) -> None:
         registry_file = self.storage_path / "registry.json"
         try:
-            data = {"skills": [s.to_dict() for s in self.skills.values()]}
+            with self._lock:
+                data = {"skills": [s.to_dict() for s in self.skills.values()]}
             with open(registry_file, "w", encoding="utf-8") as f:
                 json.dump(data, f, indent=2, ensure_ascii=False)
         except (OSError, TypeError, ValueError) as e:

{devsquad-4.5.17 → devsquad-4.5.18}/scripts/history_manager.py

@@ -79,7 +79,16 @@ class HistoryManager:
         """Get database connection with row factory."""
         conn = sqlite3.connect(self.db_path, check_same_thread=False)
         conn.row_factory = sqlite3.Row
-        conn.execute("PRAGMA journal_mode=WAL")  # Better concurrency
+        # V4.5.18: mirror the V4.5.17 code_graph_storage PRAGMA profile.
+        # Default SQLite uses journal=DELETE + full fsync which is ~10x
+        # slower per commit and starves the perf gate on cold CI runners
+        # when history_manager is forced to record many snapshots in a
+        # single test. WAL keeps journal append-only, NORMAL defers fsync
+        # to checkpoint boundary, busy_timeout makes concurrent writers
+        # wait instead of failing-fast under the 60s pytest-timeout.
+        conn.execute("PRAGMA journal_mode=WAL")
+        conn.execute("PRAGMA synchronous=NORMAL")
+        conn.execute("PRAGMA busy_timeout=5000")
         conn.execute("PRAGMA foreign_keys=ON")
         return conn
 

{devsquad-4.5.17 → devsquad-4.5.18}/skills/__init__.py

@@ -1,5 +1,5 @@
 """
-DevSquad V4.5.17 — Layered Sub-Skill Architecture
+DevSquad V4.5.18 — Layered Sub-Skill Architecture
 
 This package provides fine-grained, independently usable skills that wrap
 DevSquad's core modules. Each sub-skill is a thin handler (~50 lines) that

devsquad-4.5.17/scripts/collaboration/_version.py

@@ -1,2 +0,0 @@
-__version__ = "4.5.17"
-__version_info__ = (4, 6, 1)