test-impact: the hook's body-edit command and the per-package TS/JS command share one signature

command_for takes the graph (for Java/C# concrete test classes) and the repository (for each TS/JS
file's package and runner): command_for(lang, files, classes, db=None, repo='.'). The edit hooks pass
the repository they located, so a TypeScript body edit gets the command its package runs, and a
multi-package plan stays on the hook's one line.

Co-authored-by: axiomcode-bot[bot] <334110751+axiomcode-bot[bot]@users.noreply.github.com>

Author: swapnilpaliwal-sd

plugins/axiomcode/hooks/_graphline.py

@@ -102,10 +102,12 @@ def _ti():
     return _TI or None
 
 
-def _command_for(lang, files, classes, db=None):
-    """the runnable command `axiomcode test-impact` prints, from the same function"""
-    try: return _ti().command_for(lang, files, classes, db)
+def _command_for(lang, files, classes, db=None, repo='.'):
+    """the runnable command `axiomcode test-impact` prints, from the same function; a TypeScript/JavaScript selection
+    can need one command per package and runner, joined with '; ' here because the hook's answer is one line"""
+    try: cmd = _ti().command_for(lang, files, classes, db, repo)
     except Exception: return None
+    return cmd.replace("\n", "; ") if cmd else cmd
 
 
 def _concrete(db, classes):
@@ -119,7 +121,7 @@ def _concrete(db, classes):
 SHOWN = 6
 
 
-def body_line(db, results):
+def body_line(db, results, repo='.'):
     """ONE line for an edit that changed only bodies: which declarations, how many tests reach them, how to run those.
     `results` is [(changed-declaration, impact-json)], the same pairs the blast-radius block is built from."""
     names = [d['symbol'] for d, _ in results]
@@ -149,7 +151,7 @@ def body_line(db, results):
     fl, cl = sorted(files), sorted(owners)
     if lang in ('java', 'csharp') and cl:
         cl = sorted(_concrete(db, cl))      # before the cut, so the command and the "+N more" count the same classes
-    cmd = _command_for(lang, fl[:SHOWN], cl[:SHOWN])
+    cmd = _command_for(lang, fl[:SHOWN], cl[:SHOWN], None, repo)
     more = (len({c.split('.')[-1] for c in cl}) if lang in ('java', 'csharp') and cl else len(fl)) - SHOWN
     tail = (f"; run: {cmd}" + (f" (+{more} more: axiomcode test-impact)" if more > 0 else '')) if cmd else "; axiomcode test-impact gives the command"
     return f"graph: body edit of {what}: {n} test(s) reach it{tail}"

plugins/axiomcode/hooks/changes.py

@@ -118,7 +118,7 @@ def impact(d):
     if len(decls) > 3: lines.append(f"  … +{len(decls) - 3} more: axiomcode changed --impact")
     if bodies:
         lines.append(_graphline.body_line(os.path.join(os.environ.get('AXIOMCODE_GRAPH') or os.path.join(cwd, '.axiomcode'), 'out', 'graph.sqlite'),
-                                          bodies + [(d, {}) for d in body[3:]]))
+                                          bodies + [(d, {}) for d in body[3:]], cwd))
     return lines
 
 def key(d): return f"{d['file']}:{d['symbol']}:{d['kind']}:{d.get('detail', '')}"

plugins/axiomcode/hooks/enrich.py

@@ -231,7 +231,7 @@ def names(xs, k=4): return ', '.join(f"{x['display']} {x['at'].split('/')[-1]}"
         for n in ch.get('notes', [])[:2]: lines.append(f"  added: {n}")
     if bodies:
         lines.append(_graphline.body_line(os.path.join(os.environ.get('AXIOMCODE_GRAPH') or os.path.join(cwd, '.axiomcode'), 'out', 'graph.sqlite'),
-                                          bodies + [(d, {}) for d in body[3:]]))
+                                          bodies + [(d, {}) for d in body[3:]], cwd))
 elif tool == 'Read':
     fp = _where._abs(inp.get('file_path', ''), scwd); rel = rel_of(fp)
     a = int(inp.get('offset') or 1); b = a + int(inp.get('limit') or 100000)

tests/test_command.py

@@ -103,7 +103,7 @@ def tree(files):
 # control: nothing configured anywhere, a *.test.ts file still gets the command it always got
 d = tree({'src/a.test.ts': ''})
 check("nothing configured: a *.test.ts file keeps `npx vitest run` (control)",
-      ti.command_for('typescript', ['src/a.test.ts'], [], d), 'npx vitest run src/a.test.ts')
+      ti.command_for('typescript', ['src/a.test.ts'], [], repo=d), 'npx vitest run src/a.test.ts')
 shutil.rmtree(d)
 
 # a runner's collection is read the way the runner reads it. Each shape: (files, the file, expected plan)