specfuse-loop 0.31.0__tar.gz → 0.31.1__tar.gz

{specfuse_loop-0.31.0/specfuse_loop.egg-info → specfuse_loop-0.31.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: specfuse-loop
-Version: 0.31.0
+Version: 0.31.1
 Summary: Local-first executor for the Specfuse Plan + Work Unit gate-cycle methodology.
 Author: Specfuse contributors
 License: Apache-2.0

{specfuse_loop-0.31.0 → specfuse_loop-0.31.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "specfuse-loop"
-version = "0.31.0"
+version = "0.31.1"
 description = "Local-first executor for the Specfuse Plan + Work Unit gate-cycle methodology."
 readme = "README.md"
 requires-python = ">=3.10"

specfuse_loop-0.31.1/specfuse/loop/data/VERSION

@@ -0,0 +1 @@
+0.31.1

{specfuse_loop-0.31.0 → specfuse_loop-0.31.1}/specfuse/loop/ids.py

@@ -405,6 +405,8 @@ def _rewrite_file(rel, text, renames, include_prose, plan, folders=()):
     if is_changelog and span is None:
         span = (len(lines), len(lines))
     pairs = [(o, n) for o, n in renames]
+    # Built once per file, not per ID per line (#3559).
+    owned = {o: [f for f in folders if f.startswith(o + "-")] for o, _ in pairs}
     out, fenced = [], False
     for no, line in enumerate(lines):
         if is_changelog and not (span[0] <= no < span[1]):
@@ -417,7 +419,15 @@ def _rewrite_file(rel, text, renames, include_prose, plan, folders=()):
         row = bool(is_roadmap and _ROW_ID_RE.match(line))
         heading = bool(is_roadmap and re.match(r"^#{1,6}\s", line))
         frontmatter = no <= fm_end or no in graph
+        low = line.lower()
+        # Every old ID is `FEAT-…` and its anchor `feat-…`: a line naming neither
+        # cannot match, so the per-ID loop is skipped (#3559).
+        if "feat-" not in low:
+            out.append(line)
+            continue
         for old, new in pairs:
+            if anchor(old) not in low:
+                continue
             line = _rewrite_line(
                 line, old, new,
                 structural=row or heading or frontmatter,
@@ -425,8 +435,9 @@ def _rewrite_file(rel, text, renames, include_prose, plan, folders=()):
                 events=is_events,
                 prose=include_prose and not fenced,
                 where=(rel, no + 1, plan),
-                folders=[f for f in folders if f.startswith(old + "-")],
+                folders=owned[old],
             )
+            low = line.lower()
         out.append(line)
     return "".join(out)
 

{specfuse_loop-0.31.0 → specfuse_loop-0.31.1}/specfuse/loop/loop.py

@@ -134,7 +134,7 @@ SPECFUSE_DIR = Path(".specfuse")
 REPO_ROOT = SPECFUSE_DIR.parent
 FEATURES_DIR = SPECFUSE_DIR / "features"
 VERIFICATION_PATH = SPECFUSE_DIR / "verification.yml"
-DRIVER_VERSION = "0.31.0"
+DRIVER_VERSION = "0.31.1"
 # Oldest scaffold layout this driver can drive. init.sh stamps the scaffold's own
 # version into `.specfuse/VERSION`; check_scaffold_version() fails loud at startup if
 # the consumer's scaffold is older than this, pointing at `specfuse upgrade`. Bump

{specfuse_loop-0.31.0 → specfuse_loop-0.31.1/specfuse_loop.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: specfuse-loop
-Version: 0.31.0
+Version: 0.31.1
 Summary: Local-first executor for the Specfuse Plan + Work Unit gate-cycle methodology.
 Author: Specfuse contributors
 License: Apache-2.0

{specfuse_loop-0.31.0 → specfuse_loop-0.31.1}/specfuse_loop.egg-info/SOURCES.txt

@@ -514,6 +514,7 @@ tests/test_maven_failure_excerpt.py
 tests/test_max_attempts_configurable.py
 tests/test_max_open_prs.py
 tests/test_migrate_all_repo_stays_valid.py
+tests/test_migrate_all_scales.py
 tests/test_migrate_legacy.py
 tests/test_miniyaml_equivalence.py
 tests/test_miniyaml_negative.py

specfuse_loop-0.31.1/tests/test_migrate_all_scales.py

@@ -0,0 +1,56 @@
+#
+# Copyright 2026 Specfuse contributors
+# Licensed under the Apache License, Version 2.0. See LICENSE.
+#
+"""`ids migrate --all` cost tracks the lines that name an ID, not lines x IDs (#3559)."""
+
+import time
+import unittest
+from unittest import mock
+
+from specfuse.loop import ids
+
+N_IDS = 200
+RENAMES = [(f"FEAT-2026-{n:04d}", f"FEAT-ACME-2026-{n:04d}") for n in range(1, N_IDS + 1)]
+FOLDERS = [f"FEAT-2026-{n:04d}-slug-{n}" for n in range(1, 101)]
+
+
+def _plan():
+    return ids.AllPlan(RENAMES, [], [], {}, {}, [], [])
+
+
+def _source(n_lines):
+    body = [f"value_{i} = compute({i})  # no feature mentioned here\n" for i in range(n_lines)]
+    body[10] = "see FEAT-2026-0007 and features/FEAT-2026-0042-slug-42/PLAN.md\n"
+    body[20] = "anchor [x](#feat-2026-0150)\n"
+    return "".join(body)
+
+
+class RewriteSkipsLinesWithoutAnId(unittest.TestCase):
+    def test_per_id_work_runs_only_on_lines_naming_that_id(self):
+        calls = []
+        real = ids._rewrite_line
+
+        def counting(line, old, *a, **kw):
+            calls.append(old)
+            return real(line, old, *a, **kw)
+
+        with mock.patch.object(ids, "_rewrite_line", counting):
+            out = ids._rewrite_file("src/mod.py", _source(1000), RENAMES, True, _plan(), FOLDERS)
+
+        self.assertEqual(sorted(calls), ["FEAT-2026-0007", "FEAT-2026-0042", "FEAT-2026-0150"])
+        self.assertIn("see FEAT-ACME-2026-0007 and features/FEAT-ACME-2026-0042-slug-42/", out)
+        self.assertIn("[x](#feat-acme-2026-0150)", out)
+
+    def test_large_file_with_many_ids_rewrites_quickly(self):
+        text = _source(20_000)
+        start = time.perf_counter()
+        out = ids._rewrite_file("src/mod.py", text, RENAMES, True, _plan(), FOLDERS)
+        elapsed = time.perf_counter() - start
+        self.assertIn("FEAT-ACME-2026-0007", out)
+        # Before #3559: ~20k lines x 200 IDs x (100 folder checks + 2 regexes), minutes.
+        self.assertLess(elapsed, 5.0, f"rewrite took {elapsed:.1f}s")
+
+
+if __name__ == "__main__":
+    unittest.main()

specfuse_loop-0.31.0/specfuse/loop/data/VERSION

@@ -1 +0,0 @@
-0.31.0