cecli-dev 0.100.11rc3__tar.gz → 0.100.11rc4__tar.gz

{cecli_dev-0.100.11rc3 → cecli_dev-0.100.11rc4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cecli-dev
-Version: 0.100.11rc3
+Version: 0.100.11rc4
 Summary: we also can't pronounce cecli
 Project-URL: Homepage, https://github.com/dwash96/cecli
 Project-URL: Documentation, https://github.com/dwash96/cecli

{cecli_dev-0.100.11rc3 → cecli_dev-0.100.11rc4}/cecli/_version.py

@@ -18,7 +18,7 @@ version_tuple: tuple[int | str, ...]
 commit_id: str | None
 __commit_id__: str | None
 
-__version__ = version = '0.100.11rc3'
-__version_tuple__ = version_tuple = (0, 100, 11, 'rc3')
+__version__ = version = '0.100.11rc4'
+__version_tuple__ = version_tuple = (0, 100, 11, 'rc4')
 
-__commit_id__ = commit_id = 'g11ba4d315'
+__commit_id__ = commit_id = 'g7cf5b1173'

{cecli_dev-0.100.11rc3 → cecli_dev-0.100.11rc4}/cecli/args.py

@@ -43,15 +43,10 @@ def get_parser(default_config_files, git_root):
     # List of valid edit formats for argparse validation & shtab completion.
     # Dynamically gather them from the registered coder classes so the list
     # stays in sync if new formats are added.
-    from cecli import coders as _cecli_coders
+    from cecli.coders import EDIT_FORMAT_MAP
+
+    edit_format_choices = sorted(EDIT_FORMAT_MAP.keys())
 
-    edit_format_choices = sorted(
-        {
-            c.edit_format
-            for c in _cecli_coders.__all__
-            if hasattr(c, "edit_format") and c.edit_format is not None
-        }
-    )
     group = parser.add_argument_group("Main model")
     group.add_argument(
         "files", metavar="FILE", nargs="*", help="files to edit with an LLM (optional)"

{cecli_dev-0.100.11rc3 → cecli_dev-0.100.11rc4}/cecli/coders/__init__.py

@@ -44,6 +44,26 @@ _MODULE_MAP = {
 }
 
 
+EDIT_FORMAT_MAP = {
+    "help": "HelpCoder",
+    "ask": "AskCoder",
+    "diff": "EditBlockCoder",
+    "diff-fenced": "EditBlockFencedCoder",
+    "whole": "WholeFileCoder",
+    "patch": "PatchCoder",
+    "udiff": "UnifiedDiffCoder",
+    "udiff-simple": "UnifiedDiffSimpleCoder",
+    "architect": "ArchitectCoder",
+    "editor-diff": "EditorEditBlockCoder",
+    "editor-whole": "EditorWholeFileCoder",
+    "editor-diff-fenced": "EditorDiffFencedCoder",
+    "context": "ContextCoder",
+    "agent": "AgentCoder",
+    "hashline": "HashLineCoder",
+    "subagent": "SubAgentCoder",
+}
+
+
 def __getattr__(name):
     if name in _MODULE_MAP:
         import importlib

{cecli_dev-0.100.11rc3 → cecli_dev-0.100.11rc4}/cecli/coders/base_coder.py

@@ -105,28 +105,6 @@ all_fences = [
     wrap_fence("sourcecode"),
 ]
 
-# Map of edit_format values to coder class names.
-# Used by Coder.create() to find the right coder class by edit_format
-# without importing all coder modules.
-EDIT_FORMAT_MAP = {
-    "help": "HelpCoder",
-    "ask": "AskCoder",
-    "diff": "EditBlockCoder",
-    "diff-fenced": "EditBlockFencedCoder",
-    "whole": "WholeFileCoder",
-    "patch": "PatchCoder",
-    "udiff": "UnifiedDiffCoder",
-    "udiff-simple": "UnifiedDiffSimpleCoder",
-    "architect": "ArchitectCoder",
-    "editor-diff": "EditorEditBlockCoder",
-    "editor-whole": "EditorWholeFileCoder",
-    "editor-diff-fenced": "EditorDiffFencedCoder",
-    "context": "ContextCoder",
-    "agent": "AgentCoder",
-    "hashline": "HashLineCoder",
-    "subagent": "SubAgentCoder",
-}
-
 
 class UsageMeta(type):
     """Metaclass that provides shared accumulator properties across all Coder subclasses.
@@ -336,6 +314,7 @@ class Coder(metaclass=UsageMeta):
                 uuid=from_coder.uuid,
                 parent_uuid=from_coder.parent_uuid,
                 repo=from_coder.repo,
+                summarizer=from_coder.summarizer,
             )
             use_kwargs.update(update)  # override to complete the switch
             use_kwargs.update(kwargs)  # override passed kwargs
@@ -351,7 +330,7 @@ class Coder(metaclass=UsageMeta):
             res = coders.CopyPasteCoder(main_model, io, args=args, **kwargs)
 
         if not res:
-            coder_name = EDIT_FORMAT_MAP.get(edit_format)
+            coder_name = coders.EDIT_FORMAT_MAP.get(edit_format)
             if coder_name:
                 coder_cls = getattr(coders, coder_name)
                 res = coder_cls(main_model, io, args=args, **kwargs)
@@ -373,10 +352,14 @@ class Coder(metaclass=UsageMeta):
 
             await res.initialize_mcp_tools()
 
-            res.original_kwargs = dict(kwargs)
+            # Store only small/primitive kwargs to avoid retaining large object references.
+            # Large objects (repo, mcp_manager, commands, summarizer, file_watcher, etc.)
+            # are either overridden during clone() or accessible from instance attributes.
+            _LARGE_KWARGS = {"repo", "mcp_manager", "commands", "summarizer", "file_watcher"}
+            res.original_kwargs = {k: v for k, v in kwargs.items() if k not in _LARGE_KWARGS}
             return res
 
-        valid_formats = list(EDIT_FORMAT_MAP.keys())
+        valid_formats = list(coders.EDIT_FORMAT_MAP.keys())
         raise UnknownEditFormat(edit_format, valid_formats)
 
     async def clone(self, **kwargs):

{cecli_dev-0.100.11rc3 → cecli_dev-0.100.11rc4}/cecli/coders/copypaste_coder.py

@@ -80,7 +80,7 @@ class CopyPasteCoder(Coder):
         target_coder_class = None
         if coders is not None and selected_edit_format:
             # Use EDIT_FORMAT_MAP from base_coder to find the right coder class by edit_format
-            from cecli.coders.base_coder import EDIT_FORMAT_MAP
+            from cecli.coders import EDIT_FORMAT_MAP
 
             coder_name = EDIT_FORMAT_MAP.get(selected_edit_format)
             if coder_name:

{cecli_dev-0.100.11rc3 → cecli_dev-0.100.11rc4}/cecli/commands/core.py

@@ -1,6 +1,7 @@
 import json
 import re
 import sys
+import weakref
 from pathlib import Path
 
 from cecli.commands.utils.registry import CommandRegistry
@@ -55,6 +56,18 @@ class ReloadProgramSignal(BaseException):
 class Commands:
     scraper = None
 
+    def _get_coder(self):
+        """Return coder via weak reference, or None if collected."""
+        if self._coder_ref is not None:
+            return self._coder_ref()
+        return None
+
+    def _set_coder(self, value):
+        """Store coder as weakref to break circular reference chains."""
+        self._coder_ref = weakref.ref(value) if value is not None else None
+
+    coder = property(_get_coder, _set_coder)
+
     def clone(self):
         cloned = Commands(
             self.io,
@@ -87,7 +100,8 @@ class Commands:
         original_read_only_fnames=None,
     ):
         self.io = io
-        self.coder = coder
+        # Use weak ref to avoid circular reference chains
+        self._coder_ref = weakref.ref(coder) if coder else None
         self.parser = parser
         self.args = args
         self.verbose = verbose

{cecli_dev-0.100.11rc3 → cecli_dev-0.100.11rc4}/cecli/helpers/skills.py

@@ -6,6 +6,7 @@ according to the Skills specification.
 """
 
 import re
+import weakref
 from dataclasses import dataclass, field
 from pathlib import Path
 from typing import Any, Dict, List, Optional
@@ -69,7 +70,7 @@ class SkillsManager:
         self.include_list = set(include_list) if include_list else None
         self.exclude_list = set(exclude_list) if exclude_list else set()
         self.git_root = Path(git_root).expanduser().resolve() if git_root else None
-        self.coder = coder  # Weak reference to coder instance
+        self._coder_ref = weakref.ref(coder) if coder else None  # Weak reference to coder instance
 
         # Cache for loaded skills
         self._skills_cache: Dict[str, SkillContent] = {}
@@ -105,6 +106,18 @@ class SkillsManager:
 
             # Save initial state from config
 
+    def _get_coder(self):
+        """Return coder via weak reference, or None if collected."""
+        if self._coder_ref is not None:
+            return self._coder_ref()
+        return None
+
+    def _set_coder(self, value):
+        """Store coder as weakref to break circular reference chains."""
+        self._coder_ref = weakref.ref(value) if value is not None else None
+
+    coder = property(_get_coder, _set_coder)
+
     def _save_state(self):
         """Save current mutable state to the global skill state store.
 

{cecli_dev-0.100.11rc3 → cecli_dev-0.100.11rc4}/cecli/repo.py

@@ -1,5 +1,6 @@
 import contextlib
 import os
+import sys
 import time
 from pathlib import Path, PurePosixPath
 
@@ -84,7 +85,9 @@ class GitRepo:
         self.models = models
 
         self.normalized_path = {}
-        self.tree_files = {}
+        # Single-entry file cache: (commit_sha, interned_set_of_paths)
+        # Using a single entry (not per-commit dict) limits unbounded memory growth
+        self._tree_cache = None
 
         self.attribute_author = attribute_author
         self.attribute_committer = attribute_committer
@@ -602,8 +605,8 @@ class GitRepo:
 
         files = set()
         if commit:
-            if commit in self.tree_files:
-                files = self.tree_files[commit]
+            if self._tree_cache is not None and self._tree_cache[0] == commit.hexsha:
+                files = self._tree_cache[1]
             else:
                 try:
                     iterator = commit.tree.traverse()
@@ -612,7 +615,8 @@ class GitRepo:
                         try:
                             blob = next(iterator)
                             if blob.type == "blob":  # blob is a file
-                                files.add(blob.path)
+                                # Use sys.intern() to deduplicate path strings in memory
+                                files.add(sys.intern(blob.path))
                         except IndexError:
                             # Handle potential index error during tree traversal
                             # without relying on potentially unassigned 'blob'
@@ -629,7 +633,10 @@ class GitRepo:
                     self.io.tool_output("Is your git repo corrupted?")
                     return []
                 files = set(self.normalize_path(path) for path in files)
-                self.tree_files[commit] = set(files)
+                # Use single-entry cache (not per-commit dict) to limit memory growth
+                # Store only the SHA string (not the Commit object) to avoid retaining
+                # the entire git object graph (tree, blobs, parent commits, etc.)
+                self._tree_cache = (commit.hexsha, files)
 
         # Add staged files
         index = self.repo.index

{cecli_dev-0.100.11rc3 → cecli_dev-0.100.11rc4}/cecli_dev.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cecli-dev
-Version: 0.100.11rc3
+Version: 0.100.11rc4
 Summary: we also can't pronounce cecli
 Project-URL: Homepage, https://github.com/dwash96/cecli
 Project-URL: Documentation, https://github.com/dwash96/cecli

{cecli_dev-0.100.11rc3 → cecli_dev-0.100.11rc4}/cecli_dev.egg-info/scm_version.json

@@ -1,7 +1,7 @@
 {
-  "tag": "0.100.11rc3",
+  "tag": "0.100.11rc4",
   "distance": 0,
-  "node": "g11ba4d31552be3bd51e2dc4fa32c676f0e5c783d",
+  "node": "g7cf5b117399a2cdca750cf8c2719fd3962028413",
   "dirty": false,
   "branch": "HEAD",
   "node_date": "2026-06-30"

{cecli_dev-0.100.11rc3 → cecli_dev-0.100.11rc4}/tests/scrape/test_scrape.py

@@ -39,7 +39,10 @@ class TestScrape:
             def get_announcements(self):
                 return []
 
-        return Commands(io, DummyCoder())
+        coder = DummyCoder()
+        cmds = Commands(io, coder)
+        cmds._test_coder = coder  # keep strong reference to prevent GC of weakref
+        return cmds
 
     @patch("cecli.commands.web.install_playwright")
     @patch("cecli.commands.web.Scraper")