agents-city 0.5.11 → 0.5.12

package/.claude-plugin/marketplace.json

@@ -9,7 +9,7 @@
     {
       "name": "city",
       "source": "./plugin",
-      "version": "0.5.11",
+      "version": "0.5.12",
       "description": "Operate one autonomous city seat, its domain, role, repo support agents, goal, recognised skills and explicit roads to other cities."
     }
   ]

package/benchmarks/committee/run.py

@@ -34,12 +34,12 @@ def table(rows):
     widths = [max(len(label), *(len(str(row[key])) for row in rows))
               for label, key in columns]
     lines = [
-        '  '.join(label.ljust(width) for (label, _), width in zip(columns, widths)),
+        '  '.join(label.ljust(width) for (label, _), width in zip(columns, widths, strict=False)),
         '  '.join('-' * width for width in widths),
     ]
     for row in rows:
         lines.append('  '.join(str(row[key]).ljust(width)
-                               for (_, key), width in zip(columns, widths)))
+                               for (_, key), width in zip(columns, widths, strict=False)))
     return '\n'.join(lines)
 
 

package/benchmarks/latency/live.py

@@ -42,7 +42,8 @@ def parse_args():
     parser.add_argument('--timeout', type=int, default=180,
                         help='seconds to wait for model positions (default: 180)')
     parser.add_argument('--json', action='store_true', help='print only the JSON report')
-    parser.add_argument('--no-save', action='store_true', help='do not append the local baseline ledger')
+    parser.add_argument('--no-save', action='store_true',
+                        help='do not append the local baseline ledger')
     parser.add_argument('--keep', action='store_true', help='keep the temporary city and logs')
     return parser.parse_args()
 
@@ -81,7 +82,7 @@ def main():
     os.symlink(os.path.join(ROOT, 'bin', 'agents-city.js'), os.path.join(tools, 'agents-city'))
     marker = f'CITY_NATIVE_{int(time.time())}_{os.getpid()}'
     actors = {runtime: f'{runtime}-agent' for runtime in selected}
-    for runtime, actor in actors.items():
+    for _runtime, actor in actors.items():
         repo = os.path.join(repos, actor)
         os.makedirs(repo)
         with open(os.path.join(repo, 'README.md'), 'w', encoding='utf-8') as stream:
@@ -227,7 +228,9 @@ def wait_ready(runtimes, actors, processes, streams, app, timeout):
         for runtime in runtimes:
             process = processes[runtime]
             if process.poll() is not None:
-                raise RuntimeError(f'{runtime} exited during startup: {tail(read_stream(streams[runtime]), 12)}')
+                raise RuntimeError(
+                    f'{runtime} exited during startup: '
+                    f'{tail(read_stream(streams[runtime]), 12)}')
             ready = ready and os.path.exists(
                 os.path.join(status_dir, f'{actors[runtime]}.json'))
         if ready:
@@ -256,7 +259,8 @@ def report_for(results, unavailable, thread, duration):
     return {
         'schema': 'agents-city/native-e2e-latency@1',
         'runAt': datetime.datetime.now(datetime.timezone.utc).isoformat(),
-        'scope': 'live task completion through bus, native provider protocol and committee position',
+        'scope': ('live task completion through bus, native provider protocol '
+                  'and committee position'),
         'thread': thread,
         'durationMs': duration,
         'results': results + unavailable,

package/benchmarks/stress/run.py

@@ -140,7 +140,8 @@ def run(args):
             raise RuntimeError('a Codex gateway exited during startup')
         startup_hubs = hub_pids(city)
         if len(startup_hubs) != 1:
-            raise RuntimeError(f'concurrent startup created {len(startup_hubs)} hubs: {startup_hubs}')
+            raise RuntimeError(
+                f'concurrent startup created {len(startup_hubs)} hubs: {startup_hubs}')
         report['startupMs'] = round((time.monotonic() - started) * 1000)
         report['startupHubProcesses'] = len(startup_hubs)
 
@@ -156,8 +157,11 @@ def run(args):
                 # recovery, not a stale endpoint. What must disappear is the
                 # stopped PID's ownership; the endpoint may validly already name
                 # the one new hub.
+                # Bound as a default: the lambda is called in this same
+                # iteration, so late binding does not bite — but correct by
+                # accident is not correct.
                 if not wait_for(
-                    lambda: endpoint_released_or_replaced(endpoint_path, stopped_pid), 4
+                    lambda pid=stopped_pid: endpoint_released_or_replaced(endpoint_path, pid), 4
                 ):
                     raise RuntimeError('the stopped hub still owns the endpoint')
             round_started = time.monotonic()
@@ -178,8 +182,8 @@ def run(args):
                     f'round {round_number} spectator feed did not represent all '
                     f'{args.agents} selected actors exactly once')
             if not wait_for(
-                lambda: round_complete(thread, claude_capture, codex_capture, metrics_path,
-                                       args.agents // 2, args.agents),
+                lambda t=thread: round_complete(t, claude_capture, codex_capture,
+                                                metrics_path, args.agents // 2, args.agents),
                 args.timeout,
             ):
                 raise RuntimeError(
@@ -197,7 +201,8 @@ def run(args):
                     f'Claude={claude_count}, Codex={codex_count}')
             if len(metrics) != args.agents:
                 raise RuntimeError(
-                    f'round {round_number} recorded {len(metrics)} acceptances, expected {args.agents}')
+                    f'round {round_number} recorded {len(metrics)} acceptances, '
+                    f'expected {args.agents}')
             latencies = sorted(int(row.get('totalToNativeAcceptMs', 0)) for row in metrics)
             round_report = {
                 'round': round_number,

package/bin/hall

@@ -28,6 +28,7 @@ sys.path.insert(0, AQUI)
 sys.path.insert(0, os.path.join(os.path.dirname(AQUI), 'plugin', 'scripts'))
 import serve  # noqa: E402
 import cities  # noqa: E402
+import runtime_processes  # noqa: E402  how a child is cut loose from a terminal
 
 def asegura_build():
     """hall.js, built if missing or older than its source. Same npm the map already

package/bin/test-contracts.py

@@ -358,7 +358,40 @@ def la_puerta_de_windows():
     afirma('· and the town hall is not one of them',
            'hall' not in conShell, str(conShell))
 
-    # 3. Every file this product reads or writes is UTF-8, said out loud. A
+    # 3. Every Python file this repo ships is one the linter actually reads.
+    #
+    #    `bin/hall` is Python with no `.py` on it, and ruff selects by a glob on
+    #    the extension — so the town hall was never linted, and shipped a call to
+    #    a module it had forgotten to import. `F821 Undefined name` would have
+    #    caught it before a tag did.
+    #
+    #    Third time in this repo that a tool chose files by their NAME instead of
+    #    by what they are: shellcheck's glob broke when a folder emptied and had
+    #    been missing seven scripts; the front door called the town hall a shell
+    #    program because it is long; and this.
+    import fnmatch  # noqa: PLC0415
+    import tomllib  # noqa: PLC0415
+
+    incluidos = tomllib.load(open(os.path.join(RAIZ, 'pyproject.toml'), 'rb'))
+    patrones = incluidos['tool']['ruff']['include']
+    seguidos = subprocess.run(['git', 'ls-files'], cwd=RAIZ, capture_output=True,
+                              text=True).stdout.split()
+    fuera = []
+    for rel in seguidos:
+        ruta = os.path.join(RAIZ, rel)
+        if not os.path.isfile(ruta) or rel.startswith('bin/test'):
+            continue
+        if not rel.endswith('.py'):
+            with open(ruta, 'rb') as f:
+                cabeza = f.readline()
+            if not (cabeza.startswith(b'#!') and b'python' in cabeza.lower()):
+                continue
+        if not any(fnmatch.fnmatch(rel, p) for p in patrones):
+            fuera.append(rel)
+    afirma('· every Python file this repo ships is one the linter reads',
+           not fuera, 'not covered by ruff include: ' + ', '.join(fuera))
+
+    # 4. Every file this product reads or writes is UTF-8, said out loud. A
     #    bare `open()` takes the console's code page on Windows, which is how a
     #    card with an em dash was written in cp1252 and read back as UTF-8 by
     #    the next door along — the first thing a Windows runner found.
@@ -380,14 +413,14 @@ def la_puerta_de_windows():
            not sinDecir, 'a bare open() takes the console code page: '
                          + ', '.join(sinDecir[:8]))
 
-    # 4. The launcher every window runs. Written in the language of the machine
+    # 5. The launcher every window runs. Written in the language of the machine
     #    it will run on — and what it carries is a plan, not a shell sentence.
     lanzador = open(os.path.join(RAIZ, 'plugin', 'scripts', 'launch.py'),
                     encoding='utf-8').read()
     afirma('· the launcher is written for the machine it will run on',
            "'#!/usr/bin/env bash'" in lanzador and '@echo off' in lanzador, '')
 
-    # 5. And the promise on the tin matches all of that. Windows is claimed
+    # 6. And the promise on the tin matches all of that. Windows is claimed
     #    because a machine says so, not because the code looks portable: the
     #    runner opens a city there every time this is pushed.
     afirma('· package.json says where this runs, and a runner agrees',

package/bin/test-navegador.py

@@ -31,7 +31,7 @@ sys.path.insert(0, os.path.join(RAIZ, "plugin", "scripts"))
 sys.path.insert(0, AQUI)
 
 import serve  # noqa: E402
-from testlib import afirma, resumen  # noqa: E402
+from testlib import afirma, detiene_hubs_de_ciudad, resumen  # noqa: E402
 
 
 def disco_de_prueba():
@@ -145,6 +145,63 @@ def bundle_al_dia():
     return "rebuilt from src/hall.ts"
 
 
+def la_puerta_que_abre_el_ayuntamiento():
+    """`bin/hall`, run as a person runs it — detached, and answering.
+
+    This is the check that was missing. The hall shipped calling a module it had
+    forgotten to import, and nothing here noticed, because every suite started
+    `serve.py` directly and none of them ever came through the door. A
+    `NameError` in `despega` is only reachable by launching one.
+
+    The linter would have caught it too — and now does, because `bin/hall` is
+    Python with no `.py` on it and ruff selects by a glob on the extension. Two
+    different holes, one bug through both.
+    """
+    print("  the door that opens the town hall")
+    casa = tempfile.mkdtemp(prefix="agents-city-hall-")
+    datos = os.path.join(casa, "app", "navtest", "home")
+    os.makedirs(datos)
+    ciudad_de_prueba(datos)
+    entorno = dict(os.environ, HOME=casa, AGENTS_CITY_HOME=os.path.join(casa, "app"),
+                   AGENTS_CITY_DATA=datos, AGENTS_CITY_USER="navtest",
+                   CITY_DIR=os.path.join(casa, "canal"))
+    entorno.pop("CITY_BUS_URL", None)
+    salida = ""
+    try:
+        r = subprocess.run([sys.executable, os.path.join(RAIZ, "bin", "hall"),
+                            "--no-browser"], capture_output=True, text=True,
+                           timeout=90, env=entorno, cwd=casa)
+        salida = r.stdout + r.stderr
+        afirma("· happy: the door starts a hall and hands back an address",
+               "http://127.0.0.1:" in salida and "Traceback" not in salida,
+               salida[-500:])
+        direccion = ""
+        for trozo in salida.split():
+            if trozo.startswith("http://127.0.0.1:"):
+                direccion = trozo
+                break
+        contesta = False
+        if direccion:
+            import urllib.error  # noqa: PLC0415
+            import urllib.request  # noqa: PLC0415
+
+            try:
+                with urllib.request.urlopen(direccion, timeout=5) as resp:
+                    contesta = resp.status == 200
+            except (OSError, urllib.error.URLError):
+                contesta = False
+        afirma("· happy: and that address really serves the page",
+               contesta, direccion or salida[-300:])
+        afirma("· non-happy: nothing it prints is a stack trace",
+               "NameError" not in salida and "Traceback" not in salida,
+               salida[-500:])
+    finally:
+        subprocess.run([sys.executable, os.path.join(RAIZ, "plugin", "scripts", "apaga.py"),
+                        datos, "--yes"], capture_output=True, timeout=60, env=entorno)
+        detiene_hubs_de_ciudad(datos)
+        shutil.rmtree(casa, ignore_errors=True)
+
+
 def un_chrome_que_no_arranca():
     """What the driver says when the browser never answers.
 
@@ -181,6 +238,7 @@ def un_chrome_que_no_arranca():
 def main():
     print()
     print("  the hall, in a browser that clicks")
+    la_puerta_que_abre_el_ayuntamiento()
     un_chrome_que_no_arranca()
     print(f"  · {bundle_al_dia()}")
     datos = tempfile.mkdtemp()

package/city/oven/collect.py

@@ -5,7 +5,10 @@ It exists because the PNGs come out of the browser in base64, and there is no
 point routing them through anything in between: the browser posts them here and
 here they are saved.
 """
-import base64, json, os, re
+import base64
+import json
+import os
+import re
 from http.server import BaseHTTPRequestHandler, HTTPServer
 
 DESTINO = os.path.join(os.path.dirname(__file__), '..', 'web', 'assets', 'sprites')
@@ -13,7 +16,9 @@ os.makedirs(DESTINO, exist_ok=True)
 
 class Manos(BaseHTTPRequestHandler):
     def do_OPTIONS(self):
-        self.send_response(204); self.cors(); self.end_headers()
+        self.send_response(204)
+        self.cors()
+        self.end_headers()
     def do_POST(self):
         n = int(self.headers.get('content-length', 0))
         cuerpo = self.rfile.read(n)
@@ -33,8 +38,10 @@ class Manos(BaseHTTPRequestHandler):
             json.dump(todas, open(metas, 'w'), indent=1, sort_keys=True)
             self.send_response(200)
         except Exception as e:
-            print('fallo:', e); self.send_response(500)
-        self.cors(); self.end_headers()
+            print('fallo:', e)
+            self.send_response(500)
+        self.cors()
+        self.end_headers()
     def cors(self):
         self.send_header('access-control-allow-origin', '*')
         self.send_header('access-control-allow-headers', '*')

package/city/web/sella.py

@@ -8,7 +8,10 @@ dist/index.html to /city.js?v=<hash of the bundle>, so a changed bundle is a
 changed URL and the browser fetches it. An unchanged bundle keeps its hash and
 stays cached, which is the whole point of the cache.
 """
-import hashlib, pathlib, re, sys
+import hashlib
+import pathlib
+import re
+import sys
 
 dist = pathlib.Path(__file__).parent / 'dist'
 for pagina, guion in (('index.html', 'city.js'),):

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agents-city",
-  "version": "0.5.11",
+  "version": "0.5.12",
   "description": "Run autonomous agent cities: one chair seat, role-aware repo agents, repo-owned skills, and explicit roads.",
   "bin": {
     "agents-city": "bin/agents-city.js"

package/plugin/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "city",
   "displayName": "Agents City",
-  "version": "0.5.11",
+  "version": "0.5.12",
   "description": "One autonomous city seat with a work domain, role, goal, repo support agents, recognised skills and explicit roads to other cities.",
   "author": {
     "name": "jlcases",