@piagent/platform 1.51.1 → 1.52.0

package/docs-site/releases.json

@@ -1,4 +1,69 @@
 [
+  {
+    "version": "1.52.0",
+    "date": "2026-11-08",
+    "status": "stable",
+    "title": {
+      "en": "Company helpers get an advisor and a general helper that works in its own clone",
+      "vi": "Helper công ty có thêm advisor và general làm việc trong bản clone riêng"
+    },
+    "summary": {
+      "en": "The company Harness declares every helper, and the main agent now calls them by working names: explore, scout, advisor, general and review. General changes code only in its own clone and hands back a git ref to merge.",
+      "vi": "Harness công ty khai báo mọi helper, và agent chính gọi chúng bằng tên công việc: explore, scout, advisor, general và review. General chỉ sửa code trong bản clone riêng và trả về một git ref để merge."
+    },
+    "requires": {
+      "en": "Agent Studio 0.51.0 and Agent Watch 0.15.20 (Windows 0.1.19) for the advisor and general helpers",
+      "vi": "Agent Studio 0.51.0 và Agent Watch 0.15.20 (Windows 0.1.19) để dùng helper advisor và general"
+    },
+    "items": [
+      {
+        "kind": "new",
+        "en": "advisor gives a second opinion before a hard decision: a recommendation, the options it rejected and the risks.",
+        "vi": "advisor cho ý kiến thứ hai trước quyết định khó: khuyến nghị, các phương án đã loại và rủi ro."
+      },
+      {
+        "kind": "new",
+        "en": "general takes large work on macOS: it edits, runs checks and commits in its own clone, never your files, and returns refs/general/<id>.",
+        "vi": "general nhận việc lớn trên macOS: sửa, chạy check và commit trong clone riêng, không đụng file của bạn, rồi trả về refs/general/<id>."
+      },
+      {
+        "kind": "breaking",
+        "en": "Helpers are called explore, scout, advisor, general and review; the old names research and verify are gone, and the verify helper is removed.",
+        "vi": "Helper được gọi là explore, scout, advisor, general và review; tên cũ research và verify không còn, helper verify bị bỏ."
+      },
+      {
+        "kind": "fixed",
+        "en": "The WebUI names an advisor or general failure correctly instead of calling it a review failure.",
+        "vi": "WebUI ghi đúng lỗi của advisor hay general thay vì gọi là lỗi của review."
+      }
+    ]
+  },
+  {
+    "version": "1.51.2",
+    "date": "2026-11-03",
+    "status": "stable",
+    "title": {
+      "en": "Running only the tests that cover a change works in every test tool",
+      "vi": "Chạy riêng phần test liên quan dùng được với mọi công cụ test"
+    },
+    "summary": {
+      "en": "When you chose to run only the related tests, the harness took many single-test commands for the whole suite and refused them. It now reads one file, filter or module the same way in each tool.",
+      "vi": "Khi bạn chọn chỉ chạy phần test liên quan, harness hiểu nhầm nhiều lệnh chạy một test là chạy toàn bộ rồi chặn lại. Giờ lệnh chạy một file, một bộ lọc hay một module được đọc đúng ở từng công cụ."
+    },
+    "requires": null,
+    "items": [
+      {
+        "kind": "fixed",
+        "en": "Maven -Dtest and -Dit.test before the goal, Gradle --tests and :module:test, node --import tsx --test, flutter, cargo test NAME and others now count as part of the suite.",
+        "vi": "Maven -Dtest và -Dit.test đặt trước goal, Gradle --tests và :module:test, node --import tsx --test, flutter, cargo test TÊN và các lệnh khác giờ được tính là một phần."
+      },
+      {
+        "kind": "fixed",
+        "en": "Commands that only compile or list tests (test-compile, -DskipTests, --listTests) no longer count as a test run, and a line that also runs the whole suite still asks.",
+        "vi": "Lệnh chỉ biên dịch hay liệt kê test (test-compile, -DskipTests, --listTests) không còn bị tính là chạy test; dòng lệnh có kèm chạy toàn bộ vẫn được hỏi."
+      }
+    ]
+  },
   {
     "version": "1.51.1",
     "date": "2026-11-03",

package/docs-site/runtime.html

@@ -37,7 +37,7 @@
         <span class="brand-mark"><img src="/assets/piagent-logo.svg" alt="" width="28" height="28" /></span>
         <span class="brand-name"><b>Piagent</b> <span>Platform</span></span>
       </a>
-      <a class="version-badge" href="/whats-new" data-docs-version="v1.51.1" title="v1.51.1 docs · VI"><span class="dot"></span>v1.51.1</a>
+      <a class="version-badge" href="/whats-new" data-docs-version="v1.52.0" title="v1.52.0 docs · VI"><span class="dot"></span>v1.52.0</a>
       <nav class="top-nav" aria-label="Tài liệu">
         <a href="/quickstart">Tài liệu</a>
         <a href="/whats-new">Changelog</a>

package/docs-site/scope.html

@@ -37,7 +37,7 @@
         <span class="brand-mark"><img src="/assets/piagent-logo.svg" alt="" width="28" height="28" /></span>
         <span class="brand-name"><b>Piagent</b> <span>Platform</span></span>
       </a>
-      <a class="version-badge" href="/whats-new" data-docs-version="v1.51.1" title="v1.51.1 docs · VI"><span class="dot"></span>v1.51.1</a>
+      <a class="version-badge" href="/whats-new" data-docs-version="v1.52.0" title="v1.52.0 docs · VI"><span class="dot"></span>v1.52.0</a>
       <nav class="top-nav" aria-label="Tài liệu">
         <a href="/quickstart">Tài liệu</a>
         <a href="/whats-new">Changelog</a>

package/docs-site/security.html

@@ -37,7 +37,7 @@
         <span class="brand-mark"><img src="/assets/piagent-logo.svg" alt="" width="28" height="28" /></span>
         <span class="brand-name"><b>Piagent</b> <span>Platform</span></span>
       </a>
-      <a class="version-badge" href="/whats-new" data-docs-version="v1.51.1" title="v1.51.1 docs · VI"><span class="dot"></span>v1.51.1</a>
+      <a class="version-badge" href="/whats-new" data-docs-version="v1.52.0" title="v1.52.0 docs · VI"><span class="dot"></span>v1.52.0</a>
       <nav class="top-nav" aria-label="Tài liệu">
         <a href="/quickstart">Tài liệu</a>
         <a href="/whats-new">Changelog</a>

package/docs-site/team.html

@@ -37,7 +37,7 @@
         <span class="brand-mark"><img src="/assets/piagent-logo.svg" alt="" width="28" height="28" /></span>
         <span class="brand-name"><b>Piagent</b> <span>Platform</span></span>
       </a>
-      <a class="version-badge" href="/whats-new" data-docs-version="v1.51.1" title="v1.51.1 docs · VI"><span class="dot"></span>v1.51.1</a>
+      <a class="version-badge" href="/whats-new" data-docs-version="v1.52.0" title="v1.52.0 docs · VI"><span class="dot"></span>v1.52.0</a>
       <nav class="top-nav" aria-label="Tài liệu">
         <a href="/quickstart">Tài liệu</a>
         <a href="/whats-new">Changelog</a>

package/docs-site/update.html

@@ -37,7 +37,7 @@
         <span class="brand-mark"><img src="/assets/piagent-logo.svg" alt="" width="28" height="28" /></span>
         <span class="brand-name"><b>Piagent</b> <span>Platform</span></span>
       </a>
-      <a class="version-badge" href="/whats-new" data-docs-version="v1.51.1" title="v1.51.1 docs · VI"><span class="dot"></span>v1.51.1</a>
+      <a class="version-badge" href="/whats-new" data-docs-version="v1.52.0" title="v1.52.0 docs · VI"><span class="dot"></span>v1.52.0</a>
       <nav class="top-nav" aria-label="Tài liệu">
         <a href="/quickstart">Tài liệu</a>
         <a href="/whats-new">Changelog</a>
@@ -153,9 +153,9 @@
   <h2 id="pin-cho-team">Pin một bản cho cả team</h2>
   <p>Khi rollout cho team, cài đúng bản Pi host và đúng bản platform, rồi chạy installer ở chế độ stable. Bước <code>--dry-run</code> in ra những gì sẽ đổi.</p>
   <div class="command-card">
-    <header><span>Cài pin theo bản</span><button class="clip-btn" data-clip="npm install -g --ignore-scripts @earendil-works/pi-coding-agent@1.3.0&#10;npm install -g --ignore-scripts @piagent/platform@1.51.1&#10;piagent-install --stable --dry-run&#10;piagent-install --stable">Lấy lệnh</button></header>
+    <header><span>Cài pin theo bản</span><button class="clip-btn" data-clip="npm install -g --ignore-scripts @earendil-works/pi-coding-agent@1.3.0&#10;npm install -g --ignore-scripts @piagent/platform@1.52.0&#10;piagent-install --stable --dry-run&#10;piagent-install --stable">Lấy lệnh</button></header>
     <pre><code>npm install -g --ignore-scripts @earendil-works/pi-coding-agent@1.3.0
-npm install -g --ignore-scripts @piagent/platform@1.51.1
+npm install -g --ignore-scripts @piagent/platform@1.52.0
 piagent-install --stable --dry-run
 piagent-install --stable</code></pre>
   </div>

package/docs-site/whats-new.html

@@ -37,7 +37,7 @@
         <span class="brand-mark"><img src="/assets/piagent-logo.svg" alt="" width="28" height="28" /></span>
         <span class="brand-name"><b>Piagent</b> <span>Platform</span></span>
       </a>
-      <a class="version-badge" href="/whats-new" data-docs-version="v1.51.1" title="v1.51.1 docs · VI"><span class="dot"></span>v1.51.1</a>
+      <a class="version-badge" href="/whats-new" data-docs-version="v1.52.0" title="v1.52.0 docs · VI"><span class="dot"></span>v1.52.0</a>
       <nav class="top-nav" aria-label="Tài liệu">
         <a href="/quickstart">Tài liệu</a>
         <a href="/whats-new" aria-current="page">Changelog</a>
@@ -108,7 +108,32 @@
 <div class="release-filter" role="toolbar" aria-label="Lọc theo loại thay đổi"><button type="button" data-filter-kind="all" aria-pressed="true">Tất cả</button><button type="button" data-filter-kind="new" aria-pressed="false">Mới</button><button type="button" data-filter-kind="improved" aria-pressed="false">Cải thiện</button><button type="button" data-filter-kind="fixed" aria-pressed="false">Sửa lỗi</button><button type="button" data-filter-kind="security" aria-pressed="false">Bảo mật</button></div>
 <section class="section releases" data-releases>
 <h2 id="latest" class="sr-only">Mới nhất</h2>
-<article class="release latest" id="v1-51-1" data-release>
+<article class="release latest" id="v1-52-0" data-release>
+  <div class="release-meta"><a class="release-version" href="#v1-52-0">v1.52.0</a><time class="release-date" datetime="2026-11-08">08/11/2026</time></div>
+  <div class="release-body">
+    <h3 data-toc-skip id="v1-52-0-title">Helper công ty có thêm advisor và general làm việc trong bản clone riêng</h3>
+    <p>Harness công ty khai báo mọi helper, và agent chính gọi chúng bằng tên công việc: explore, scout, advisor, general và review. General chỉ sửa code trong bản clone riêng và trả về một git ref để merge.</p>
+    <ul class="release-items">
+      <li data-kind="breaking"><span class="kind kind-breaking">Thay đổi lớn</span><span>Helper được gọi là explore, scout, advisor, general và review; tên cũ research và verify không còn, helper verify bị bỏ.</span></li>
+      <li data-kind="new"><span class="kind kind-new">Mới</span><span>advisor cho ý kiến thứ hai trước quyết định khó: khuyến nghị, các phương án đã loại và rủi ro.</span></li>
+      <li data-kind="new"><span class="kind kind-new">Mới</span><span>general nhận việc lớn trên macOS: sửa, chạy check và commit trong clone riêng, không đụng file của bạn, rồi trả về refs/general/&lt;id&gt;.</span></li>
+      <li data-kind="fixed"><span class="kind kind-fixed">Sửa lỗi</span><span>WebUI ghi đúng lỗi của advisor hay general thay vì gọi là lỗi của review.</span></li>
+    </ul>
+    <p class="release-requires"><strong>Cần:</strong> Agent Studio 0.51.0 và Agent Watch 0.15.20 (Windows 0.1.19) để dùng helper advisor và general</p>
+  </div>
+</article>
+<article class="release" id="v1-51-2" data-release>
+  <div class="release-meta"><a class="release-version" href="#v1-51-2">v1.51.2</a><time class="release-date" datetime="2026-11-03">03/11/2026</time></div>
+  <div class="release-body">
+    <h3 data-toc-skip id="v1-51-2-title">Chạy riêng phần test liên quan dùng được với mọi công cụ test</h3>
+    <p>Khi bạn chọn chỉ chạy phần test liên quan, harness hiểu nhầm nhiều lệnh chạy một test là chạy toàn bộ rồi chặn lại. Giờ lệnh chạy một file, một bộ lọc hay một module được đọc đúng ở từng công cụ.</p>
+    <ul class="release-items">
+      <li data-kind="fixed"><span class="kind kind-fixed">Sửa lỗi</span><span>Maven -Dtest và -Dit.test đặt trước goal, Gradle --tests và :module:test, node --import tsx --test, flutter, cargo test TÊN và các lệnh khác giờ được tính là một phần.</span></li>
+      <li data-kind="fixed"><span class="kind kind-fixed">Sửa lỗi</span><span>Lệnh chỉ biên dịch hay liệt kê test (test-compile, -DskipTests, --listTests) không còn bị tính là chạy test; dòng lệnh có kèm chạy toàn bộ vẫn được hỏi.</span></li>
+    </ul>
+  </div>
+</article>
+<article class="release" id="v1-51-1" data-release>
   <div class="release-meta"><a class="release-version" href="#v1-51-1">v1.51.1</a><time class="release-date" datetime="2026-11-03">03/11/2026</time></div>
   <div class="release-body">
     <h3 data-toc-skip id="v1-51-1-title">Task để chạy qua đêm không còn đứng chờ câu trả lời tới sáng</h3>
@@ -397,40 +422,38 @@
     </ul>
   </div>
 </article>
-<article class="release" id="v1-44-0" data-release>
-  <div class="release-meta"><a class="release-version" href="#v1-44-0">v1.44.0</a><time class="release-date" datetime="2026-10-28">28/10/2026</time></div>
-  <div class="release-body">
-    <h3 data-toc-skip id="v1-44-0-title">Giữ Pi của bạn, chọn giao diện Terminal, thêm repo skill <span class="tag">không phát hành</span></h3>
-    <p>Đã tag nhưng chưa publish: advisory undici mới trong Pi host làm kiểm tra runtime fail. Thay bằng 1.44.1.</p>
-    <ul class="release-items">
-      <li data-kind="new"><span class="kind kind-new">Mới</span><span>/style chọn giao diện Terminal: system, dark, light của Pi hoặc ocean, paper, contrast, focus của Piagent. Terminal công ty nhớ lựa chọn.</span></li>
-      <li data-kind="new"><span class="kind kind-new">Mới</span><span>piagent skills add mattpocock/skills thêm repo skill GitHub ghim một commit; skill dùng được dạng /skill:tên trong Pi, Terminal công ty và dashboard.</span></li>
-      <li data-kind="fixed"><span class="kind kind-fixed">Sửa lỗi</span><span>Setup, install, update không còn hạ pi mới hơn của dev (vd 1.4.0) về bản pin 1.3.0: Piagent giữ bản riêng trong ~/.piagent/pi-host cho phiên công ty, dashboard và lệnh piagent pi.</span></li>
-      <li data-kind="fixed"><span class="kind kind-fixed">Sửa lỗi</span><span>piagent-uninstall --with-host gỡ bản Pi riêng của Piagent và giữ nguyên pi mới hơn của dev.</span></li>
-    </ul>
-  </div>
-</article>
-<article class="release" id="v1-43-9" data-release>
-  <div class="release-meta"><a class="release-version" href="#v1-43-9">v1.43.9</a><time class="release-date" datetime="2026-10-27">27/10/2026</time></div>
-  <div class="release-body">
-    <h3 data-toc-skip id="v1-43-9-title">Đủ skill đã cài, và mở file agent gửi lại</h3>
-    <p>Skill trong plugin Claude và Codex, như superpowers, đến được agent; file agent gửi lại mở ngay trong web; hộp chọn thư mục chạy trên Windows.</p>
-    <ul class="release-items">
-      <li data-kind="new"><span class="kind kind-new">Mới</span><span>File agent gắn link trong câu trả lời mở ngay trong explorer: code đúng dòng, ảnh hiện hình, PDF và file office hiện nội dung chữ.</span></li>
-      <li data-kind="improved"><span class="kind kind-improved">Cải thiện</span><span>Skill và lệnh trong plugin Claude và Codex của dev được liệt kê và đọc được, lấy bản mới nhất; gõ /plugin:skill cũng được.</span></li>
-      <li data-kind="fixed"><span class="kind kind-fixed">Sửa lỗi</span><span>Chọn thư mục project mở lại được hộp chọn thư mục của Windows, kể cả khi dashboard chạy nền.</span></li>
-      <li data-kind="fixed"><span class="kind kind-fixed">Sửa lỗi</span><span>Lượt chỉ sửa tài liệu hay file Spec Kit không bị harness bắt chạy check của repo, và approval cho check đã bị từ chối thì không bị hỏi lại.</span></li>
-      <li data-kind="fixed"><span class="kind kind-fixed">Sửa lỗi</span><span>Câu trả lời agent lặp lại sau mỗi lần harness trả về chỉ hiện một lần, không lặp theo từng vòng.</span></li>
-      <li data-kind="fixed"><span class="kind kind-fixed">Sửa lỗi</span><span>Trên Linux và Windows (WSL2), check tìm được JDK ngoài PATH: SDKMAN, /usr/lib/jvm hoặc ~/.jdks.</span></li>
-    </ul>
-  </div>
-</article>
 </section>
 <section class="section archive">
   <h2 id="archive">Các bản trước</h2>
   <p class="section-lead">Bản cũ hơn, gom theo tháng. Mở một tháng để xem từng thay đổi.</p>
   <details class="release-archive">
-    <summary><span>Tháng 10/2026</span><span class="archive-count">73</span></summary>
+    <summary><span>Tháng 10/2026</span><span class="archive-count">75</span></summary>
+    <div class="archive-row" id="v1-44-0">
+      <div class="archive-head"><a class="release-version" href="#v1-44-0">v1.44.0</a> <span class="release-date">28/10/2026</span></div>
+      <div class="archive-body">
+        <p><strong>Giữ Pi của bạn, chọn giao diện Terminal, thêm repo skill</strong> <span class="tag">không phát hành</span> — Đã tag nhưng chưa publish: advisory undici mới trong Pi host làm kiểm tra runtime fail. Thay bằng 1.44.1.</p>
+        <ul class="release-items">
+        <li data-kind="new"><span class="kind kind-new">Mới</span><span>/style chọn giao diện Terminal: system, dark, light của Pi hoặc ocean, paper, contrast, focus của Piagent. Terminal công ty nhớ lựa chọn.</span></li>
+        <li data-kind="new"><span class="kind kind-new">Mới</span><span>piagent skills add mattpocock/skills thêm repo skill GitHub ghim một commit; skill dùng được dạng /skill:tên trong Pi, Terminal công ty và dashboard.</span></li>
+        <li data-kind="fixed"><span class="kind kind-fixed">Sửa lỗi</span><span>Setup, install, update không còn hạ pi mới hơn của dev (vd 1.4.0) về bản pin 1.3.0: Piagent giữ bản riêng trong ~/.piagent/pi-host cho phiên công ty, dashboard và lệnh piagent pi.</span></li>
+        <li data-kind="fixed"><span class="kind kind-fixed">Sửa lỗi</span><span>piagent-uninstall --with-host gỡ bản Pi riêng của Piagent và giữ nguyên pi mới hơn của dev.</span></li>
+        </ul>
+      </div>
+    </div>
+    <div class="archive-row" id="v1-43-9">
+      <div class="archive-head"><a class="release-version" href="#v1-43-9">v1.43.9</a> <span class="release-date">27/10/2026</span></div>
+      <div class="archive-body">
+        <p><strong>Đủ skill đã cài, và mở file agent gửi lại</strong> — Skill trong plugin Claude và Codex, như superpowers, đến được agent; file agent gửi lại mở ngay trong web; hộp chọn thư mục chạy trên Windows.</p>
+        <ul class="release-items">
+        <li data-kind="new"><span class="kind kind-new">Mới</span><span>File agent gắn link trong câu trả lời mở ngay trong explorer: code đúng dòng, ảnh hiện hình, PDF và file office hiện nội dung chữ.</span></li>
+        <li data-kind="improved"><span class="kind kind-improved">Cải thiện</span><span>Skill và lệnh trong plugin Claude và Codex của dev được liệt kê và đọc được, lấy bản mới nhất; gõ /plugin:skill cũng được.</span></li>
+        <li data-kind="fixed"><span class="kind kind-fixed">Sửa lỗi</span><span>Chọn thư mục project mở lại được hộp chọn thư mục của Windows, kể cả khi dashboard chạy nền.</span></li>
+        <li data-kind="fixed"><span class="kind kind-fixed">Sửa lỗi</span><span>Lượt chỉ sửa tài liệu hay file Spec Kit không bị harness bắt chạy check của repo, và approval cho check đã bị từ chối thì không bị hỏi lại.</span></li>
+        <li data-kind="fixed"><span class="kind kind-fixed">Sửa lỗi</span><span>Câu trả lời agent lặp lại sau mỗi lần harness trả về chỉ hiện một lần, không lặp theo từng vòng.</span></li>
+        <li data-kind="fixed"><span class="kind kind-fixed">Sửa lỗi</span><span>Trên Linux và Windows (WSL2), check tìm được JDK ngoài PATH: SDKMAN, /usr/lib/jvm hoặc ~/.jdks.</span></li>
+        </ul>
+      </div>
+    </div>
     <div class="archive-row" id="v1-43-8">
       <div class="archive-head"><a class="release-version" href="#v1-43-8">v1.43.8</a> <span class="release-date">27/10/2026</span></div>
       <div class="archive-body">

package/docs-site/windows.html

@@ -37,7 +37,7 @@
         <span class="brand-mark"><img src="/assets/piagent-logo.svg" alt="" width="28" height="28" /></span>
         <span class="brand-name"><b>Piagent</b> <span>Platform</span></span>
       </a>
-      <a class="version-badge" href="/whats-new" data-docs-version="v1.51.1" title="v1.51.1 docs · VI"><span class="dot"></span>v1.51.1</a>
+      <a class="version-badge" href="/whats-new" data-docs-version="v1.52.0" title="v1.52.0 docs · VI"><span class="dot"></span>v1.52.0</a>
       <nav class="top-nav" aria-label="Tài liệu">
         <a href="/quickstart">Tài liệu</a>
         <a href="/whats-new">Changelog</a>

package/docs-site/workflows.html

@@ -37,7 +37,7 @@
         <span class="brand-mark"><img src="/assets/piagent-logo.svg" alt="" width="28" height="28" /></span>
         <span class="brand-name"><b>Piagent</b> <span>Platform</span></span>
       </a>
-      <a class="version-badge" href="/whats-new" data-docs-version="v1.51.1" title="v1.51.1 docs · VI"><span class="dot"></span>v1.51.1</a>
+      <a class="version-badge" href="/whats-new" data-docs-version="v1.52.0" title="v1.52.0 docs · VI"><span class="dot"></span>v1.52.0</a>
       <nav class="top-nav" aria-label="Tài liệu">
         <a href="/quickstart">Tài liệu</a>
         <a href="/whats-new">Changelog</a>

package/evals/private-holdout-v1/public-exposure.v2.json

@@ -256,8 +256,8 @@
     },
     {
       "path": "tests",
-      "fileCount": 651,
-      "sha256": "3e29d3643596a6ed8d960833b13df079ee337aaa431e059e81af1b662311bcb8"
+      "fileCount": 653,
+      "sha256": "577758e269e6cb8d79d32cec4c354be0ea343404c897c24280dc50f86002ea21"
     }
   ],
   "custodianComparisonRules": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@piagent/platform",
-  "version": "1.51.1",
+  "version": "1.52.0",
   "license": "MIT",
   "publishConfig": {
     "access": "public",

package/packages/piagent-core/managed/check-questions.mjs

@@ -1,6 +1,7 @@
 import { randomUUID } from 'node:crypto';
 import { memberQuestions } from './member-questions.mjs';
 import { alwaysBypass, effectivePermission } from './permission.mjs';
+import { checkScope, suitePart, unquoted } from './test-scope.mjs';
 import { digestsOnce, knownFast, markCheckStart, missingSuites, pushQuestion, pushRepositories, reviewBeforePush } from './check-evidence.mjs';
 
 // How much to test is the member's decision: a one-line fix should not cost a
@@ -12,53 +13,8 @@ import { digestsOnce, knownFast, markCheckStart, missingSuites, pushQuestion, pu
 // overnight) is taken back and the cheap answer goes on: only the tests that
 // cover the change, and no push. Before this a turn waited until morning.
 export const UNANSWERED_MS = 10 * 60_000;
+export { checkScope, suitePart };
 
-// How much of the suite a test command covers: 'partial' (a filter, one file, one
-// package), 'suite' (the whole test command) or null when it runs no tests.
-const VERSION_ONLY = /(?:^|\s)(?:--version|-V|--help)(?:\s|$)/;
-const FILTERED = /(?:^|\s)(?:-k|-t|--grep|--testNamePattern|--test-name-pattern|--filter|--only|--testPathPattern)(?:[\s=]|$)|\bgo\s+test\b.*\s-run[\s=]|\bpytest\b.*\s::|\.only\b/;
-const TEST_RUNNER = /\b(npm|pnpm|yarn|bun)\s+(run\s+)?test(:\S+)?(?=\s|$)|\bnode\s+--test\b|\b(vitest|jest|mocha|pytest|rspec|phpunit)\b|\bplaywright\s+test\b|\bpython3?\s+-m\s+(pytest|unittest)\b|\b(go|cargo|swift|dotnet|deno)\s+test\b|\bmake\s+test\b|\bxcodebuild\b.*\btest\b|\b(gradle|gradlew|mvn|mvnw)\b.*\b(test|verify)\b/;
-// Test commands start a command: `npm install -D vitest` or `rm jest.config.ts` only name one.
-const TEST_START = new RegExp(`^(?:${TEST_RUNNER.source})`);
-// A function body, a group or a loop body opens a part as well.
-const LAUNCHER = /^(?:[\w-]+\s*\(\)\s*\{?\s*|\{\s+|\(\s*|(?:do|then|else)\s+|\w+=\S*\s+|time\s+|nohup\s+|timeout\s+\S+\s+|npx\s+(?:-\S+\s+)*|bunx\s+|pnpm\s+(?:exec|dlx)\s+|uv\s+run\s+|poetry\s+run\s+|yarn\s+(?=jest|vitest|mocha|playwright)|node_modules\/\.bin\/|\.\/)+/;
-const partsOf = bare => bare.split(/\s*(?:\|\|?|&&|;)\s*/).map(part => part.replace(/\s\d*>.*$/, '').replace(/[\s)}]+$/, '').trim());
-const NOT_TESTS = /\b(lint|typecheck|type-check|tsc|eslint|biome|ruff|mypy|pyright|vet|clippy|format|build)\b/;
-const TARGET_FLAG = /^(-p|--package|-w|--workspace|--tests)$|^(-Dtest=|-only-testing:|--workspace=|--package=)/;
-// `go test -p 4` sets how many packages run at once; for cargo -p names a package.
-const GO_TEST = /^go\s+test\b/;
-const VALUE_FLAG = /^(-c|--config|--reporter|--project|--workers|--timeout|--rootDir|--root|-f|--file|-o|--output)$/;
-// A file or folder named on the command line; a flag's own value names none.
-const targetsPart = (args, go = false) => args.some((a, i) => {
-  // A variable: a function's arguments, a loop's file, $(go list …). The caller chose them, so this is not the whole command.
-  if (a.startsWith('$')) return true;
-  if (go && a === '-p') return false;
-  if (TARGET_FLAG.test(a)) return true;
-  if (go && args[i - 1] === '-p') return false;
-  if (a.startsWith('-') || VALUE_FLAG.test(args[i - 1] ?? '') || a.includes('*') || a.endsWith('...')) return false;
-  return a.includes('/') || /\.(test|spec)\.\w+$|_test\.\w+$|^test_\w+\.\w+$/.test(a);
-});
-// What a command runs, without the text it merely quotes (a commit message, an echo).
-const unquoted = text => text.replace(/"[^"]*"|'[^']*'/g, quoted => quoted.startsWith('"$') ? '$' : '""');
-export function checkScope(command, declared = []) {
-  const text = String(command ?? '').replace(/\s+/g, ' ').trim();
-  if (!text || /^(echo|printf|grep|rg|cat|sed|awk|head|tail|less|git|ls|find)\b/.test(text) || VERSION_ONLY.test(text) || /--collect-only/.test(text)) return null;
-  const bare = unquoted(text);
-  for (const part of partsOf(bare)) {
-    const own = part.replace(LAUNCHER, ''), runner = own.match(TEST_START);
-    if (!runner) continue;
-    if (FILTERED.test(own)) return 'partial';
-    return !declared.includes(text) && targetsPart(own.slice(runner[0].length).split(' ').filter(Boolean), GO_TEST.test(own)) ? 'partial' : 'suite';
-  }
-  // A script the repository declares: it cannot be narrowed from here.
-  const named = declared.find(d => partsOf(bare).some(part => part.startsWith(unquoted(d))));
-  return named && !NOT_TESTS.test(named) ? 'suite' : null;
-}
-// The test step of a command, without what surrounds it: the same step costs the same.
-export function suitePart(command) {
-  for (const part of partsOf(unquoted(String(command ?? '').replace(/\s+/g, ' ').trim()))) { const own = part.replace(LAUNCHER, ''); if (TEST_START.test(own)) return own; }
-  return String(command ?? '').trim();
-}
 // The member's own words asking for the whole suite: nothing to ask them then.
 const RUN_ALL = String.raw`(?:full|all(?:\s+the)?|whole(?:\s+the)?|entire(?:\s+the)?|the\s+(?:full|whole|entire))`;
 const FULL_WORDS = new RegExp([

package/packages/piagent-core/managed/copies.mjs

@@ -1,3 +1,4 @@
+import { callName } from './role-names.mjs';
 import { memberMessages, memberRequest } from './objections.mjs';
 
 // Several helpers of one role, at the member's request only ("use 3
@@ -30,7 +31,7 @@ export function fanOut(manager, role, args, harness) {
   if (harness) throw Error('managed-copies-invalid: the harness runs one helper per call.');
   const words = normal(args?.member_words);
   if (words.length < 3 || !memberRequest(memberMessages(manager)).some(text => normal(text).includes(words)) || namedCount(words) < n || !HELPER_WORDS.test(words)) return { dropped: n };
-  if (n > MAX_COPIES) throw Error(`managed-copies-invalid: ask for 2 to ${MAX_COPIES} ${role} helpers (copies, or one split line per helper).`);
+  if (n > MAX_COPIES) throw Error(`managed-copies-invalid: ask for 2 to ${MAX_COPIES} ${callName(role)} helpers (copies, or one split line per helper).`);
   return { n, split: split.length ? split : null };
 }
 // The helper's answer, with what the main agent reads above it when it asked
@@ -38,12 +39,12 @@ export function fanOut(manager, role, args, harness) {
 export function withDroppedNote(result, role, asked) {
   const n = asked?.dropped, first = result?.content?.[0];
   if (!n || first?.type !== 'text') return result;
-  const note = `One ${role} helper ran, not ${n}: the member's own words did not ask for ${n} ${role} helpers. Go on with this answer; run several only when the member asks for that many.\n\n`;
+  const note = `One ${callName(role)} helper ran, not ${n}: the member's own words did not ask for ${n} ${callName(role)} helpers. Go on with this answer; run several only when the member asks for that many.\n\n`;
   return { ...result, content: [{ ...first, text: note + first.text }, ...result.content.slice(1)] };
 }
 // What one copy reads under its brief.
 export function copyNote(role, i, fan) {
-  return `\n\nThe member asked for ${fan.n} ${role} helpers; you are ${role} helper ${i + 1} of ${fan.n}. ${fan.split ? `Your share of the work: ${fan.split[i]}. The others cover the rest.` : 'The others have the same brief: answer on your own judgement.'}`;
+  return `\n\nThe member asked for ${fan.n} ${callName(role)} helpers; you are ${callName(role)} helper ${i + 1} of ${fan.n}. ${fan.split ? `Your share of the work: ${fan.split[i]}. The others cover the rest.` : 'The others have the same brief: answer on your own judgement.'}`;
 }
 
 const jsonOf = reply => {
@@ -53,18 +54,17 @@ const jsonOf = reply => {
 const unique = (list, key) => { const seen = new Set(); return list.filter(x => { const k = key(x); if (seen.has(k)) return false; seen.add(k); return true; }); };
 const SEVERITY = { blocking: 0, major: 1, minor: 2 };
 // One answer from the copies: each helper's prose (its JSON left out), then
-// one JSON block joining their findings, verdicts and objections, so the
-// review, verify and objection readers see every copy. `parsed`: every copy
-// gave its JSON. A finding or a failed claim any copy reports stands.
-export function mergeCopies(role, label, answers, { findingsOf, issuesOf, verdictsOf }) {
+// one JSON block joining their findings and objections, so the
+// review and objection readers see every copy. `parsed`: every copy
+// gave its JSON. A finding any copy reports stands.
+export function mergeCopies(role, label, answers, { findingsOf, issuesOf }) {
   const n = answers.length, room = Math.floor(12000 / n);
   const findings = role === 'review' ? unique(answers.flatMap(a => findingsOf(a.reply) ?? []), f => `${f.severity}|${f.file}|${f.line}|${f.issue}`).sort((a, b) => SEVERITY[a.severity] - SEVERITY[b.severity]) : null;
-  const verdicts = role === 'verify' ? unique(answers.flatMap(a => verdictsOf(a.reply) ?? []), v => `${v.status}|${v.claim}`) : null;
   const issues = unique(answers.flatMap(a => issuesOf(a.reply) ?? []), i => `${i.kind}|${i.detail}`).slice(0, 5);
   const parsed = answers.every(a => jsonOf(a.reply));
   const summaries = answers.map(a => jsonOf(a.reply)?.summary).filter(s => typeof s === 'string').map(s => s.slice(0, 200));
-  const json = { ...(findings ? { findings: findings.slice(0, 40) } : {}), ...(verdicts ? { verdicts: verdicts.slice(0, 40) } : {}), brief_issues: issues,
-    summary: `${n} ${role} helpers${summaries.length ? ': ' + summaries.join(' | ') : ''}`.slice(0, 900) };
+  const json = { ...(findings ? { findings: findings.slice(0, 40) } : {}), brief_issues: issues,
+    summary: `${n} ${callName(role)} helpers${summaries.length ? ': ' + summaries.join(' | ') : ''}`.slice(0, 900) };
   const body = answers.map((a, i) => `### ${label} helper ${i + 1} of ${n}\n${String(a.reply).replace(/```json\s*[\s\S]*?```/g, '').trim().slice(0, room)}`).join('\n\n');
   return { reply: `${body}\n\n\`\`\`json\n${JSON.stringify(json)}\n\`\`\``, usage: answers.reduce((total, a) => total + a.usage, 0),
     ...(findings ? { findings, parsed, blocking: findings.filter(f => f.severity === 'blocking').length } : {}) };

package/packages/piagent-core/managed/general-clone.mjs

@@ -0,0 +1,111 @@
+import fs from 'node:fs';
+import os from 'node:os';
+import path from 'node:path';
+import { randomUUID } from 'node:crypto';
+import { execFile } from 'node:child_process';
+import { promisify } from 'node:util';
+import { cacheDirectory } from './language-environment.mjs';
+
+// The general helper changes files, so it never works in the member's
+// project. It works in its own clone: a separate repository with its own
+// .git (refs, config, hooks), no remote, and no writable path to the project
+// or its refs. A linked worktree would share the project's .git, and the
+// sandbox opens a shared git directory for writing. The clone is `--shared`: it
+// borrows the project's object files (read-only inside the sandbox, by the
+// boundary's readRoots) instead of copying them, so it costs about what a
+// worktree costs. The only crossing is takeBack below, run by this runtime,
+// not by the helper: it copies the clone's commits into the project as one
+// private ref, refs/general/<id>, and touches nothing else. The main agent
+// merges that ref itself.
+export const GENERAL_ENTRY = 'agent-watch-general';
+const OWNER = '.piagent-owner';
+const run = promisify(execFile);
+// The clone's own configuration is the helper's to write. The runtime
+// therefore reads it only through git commands that cannot start a program
+// the clone names: these override what a clone's config could set.
+const SAFE = ['-c', 'core.fsmonitor=false', '-c', 'core.hooksPath=/dev/null', '-c', 'core.alternateRefsCommand=', '-c', 'protocol.file.allow=always'];
+const gitEnv = () => ({ PATH: '/usr/bin:/bin', HOME: os.tmpdir(), GIT_CONFIG_GLOBAL: '/dev/null', GIT_CONFIG_NOSYSTEM: '1', GIT_TERMINAL_PROMPT: '0', LANG: 'en_US.UTF-8' });
+const git = (exe, cwd, args) => run(exe, [...SAFE, ...args], { cwd, env: gitEnv(), maxBuffer: 4 << 20, timeout: 120_000 }).then(r => r.stdout.trim());
+
+// A clone of the project's HEAD. Work not yet committed in the project is not
+// in it: `dirty` lists those paths so the main agent can be told.
+export async function cloneProject(managed) {
+  // The clone's isolation is proven under the macOS sandbox only; the Linux (bubblewrap) path is not, so it stays closed.
+  if (process.platform !== 'darwin') throw Error('managed-general-unavailable: the general helper is not verified on this platform yet; do the work yourself.');
+  const exe = managed.boundary.git, project = managed.cwd;
+  let top, base, dirty, objects;
+  try {
+    top = fs.realpathSync(await git(exe, project, ['rev-parse', '--show-toplevel']));
+    base = await git(exe, project, ['rev-parse', '--verify', 'HEAD']);
+    objects = fs.realpathSync(path.resolve(project, await git(exe, project, ['rev-parse', '--git-path', 'objects'])));
+    dirty = (await git(exe, project, ['status', '--porcelain', '-uall'])).split('\n').filter(Boolean).map(l => l.replace(/^\S{1,2}\s+/, '')).slice(0, 50);
+  } catch { throw Error('managed-general-needs-git: the general helper needs the project to be a git repository with a commit; commit first or do the work yourself.'); }
+  if (top !== project) throw Error('managed-general-needs-git: the general helper needs the project folder to be the repository root.');
+  const id = randomUUID().slice(0, 8), dir = fs.mkdtempSync(path.join(fs.realpathSync(os.tmpdir()), 'piagent-general-'));
+  try {
+    await git(exe, dir, ['clone', '-q', '--shared', '--no-tags', '--template=', '--', project, path.join(dir, 'work')]);
+    await git(exe, path.join(dir, 'work'), ['remote', 'remove', 'origin']);
+  } catch (error) { fs.rmSync(dir, { recursive: true, force: true }); throw Error(`managed-general-clone-failed: ${String(error?.message).slice(0, 200)}`); }
+  const token = randomUUID(); // the proof, in the folder and in the session's record, that this runtime made it
+  fs.writeFileSync(path.join(dir, OWNER), token, { mode: 0o600 });
+  const real = fs.realpathSync(path.join(dir, 'work'));
+  return { id, base, dirty, token, objects, cache: cacheDirectory(managed.boundary.userHome, real), root: dir, dir: real };
+}
+
+// The clone's folder, and the package cache the sandbox keyed to it (no later run shares it).
+export function discardClone(clone) { fs.rmSync(clone.root, { recursive: true, force: true }); fs.rmSync(clone.cache, { recursive: true, force: true }); }
+
+// Bring the clone's commits into the project as refs/general/<id> and
+// discard the clone. Returns what the main agent needs to merge them.
+export async function takeBack(managed, clone) {
+  const exe = managed.boundary.git, project = managed.cwd;
+  try {
+    // The helper writes the clone's .git: a gitfile or alternates would point
+    // the fetch at a path outside it.
+    const dotgit = path.join(clone.dir, '.git');
+    const alternates = path.join(dotgit, 'objects/info/alternates');
+    if (!fs.lstatSync(dotgit).isDirectory() || fs.readFileSync(alternates, 'utf8').trim() !== clone.objects) throw Error('the clone was changed to point outside itself');
+    const ref = `refs/general/${clone.id}`;
+    await git(exe, project, ['fetch', '-q', '--no-tags', '--no-recurse-submodules', '--no-write-fetch-head', clone.dir, `+HEAD:${ref}`]);
+    const head = await git(exe, project, ['rev-parse', '--verify', ref]);
+    if (head === clone.base) { await git(exe, project, ['update-ref', '-d', ref]); return { ref: null, head, base: clone.base, files: [], note: 'The general helper made no commit, so nothing came back. Only committed work is returned.' }; }
+    if (!(await git(exe, project, ['merge-base', '--is-ancestor', clone.base, head]).then(() => true, () => false))) { await git(exe, project, ['update-ref', '-d', ref]); throw Error('the clone no longer descends from the commit it started from'); }
+    const files = (await git(exe, project, ['diff', '--numstat', `${clone.base}..${head}`])).split('\n').filter(Boolean).map(l => { const [added, deleted, ...name] = l.split('\t'); return `+${added} -${deleted} ${name.join('\t')}`; }).slice(0, 80);
+    return { ref, head, base: clone.base, files, note: null };
+  } finally { discardClone(clone); }
+}
+
+// What the main agent reads with the helper's answer.
+export function takeBackText(clone, back) {
+  const dirty = clone.dirty.length ? ` The helper did not see ${clone.dirty.length} uncommitted path(s) in the project (${clone.dirty.slice(0, 8).join(', ')}${clone.dirty.length > 8 ? ', …' : ''}).` : '';
+  if (!back.ref) return `\n\n${back.note}${dirty}`;
+  return `\n\nThe general helper's commits are in ${back.ref} (${back.base.slice(0, 9)}..${back.head.slice(0, 9)}); your files are unchanged. Merge or cherry-pick that ref yourself, run the checks on the result, then ask for review; delete the ref afterwards (git update-ref -d ${back.ref}). Changed:\n${back.files.join('\n')}${dirty}`;
+}
+
+// One entry per general run, so a later turn knows what ran and what came back;
+// "running" with the folder and its token is also the record of what to recover.
+export function recordGeneral(managed, clone, status, back = null) {
+  managed.session.sessionManager.appendCustomEntry(GENERAL_ENTRY, { id: clone.id, base: clone.base, status, root: clone.root, token: clone.token, ...(back?.ref ? { ref: back.ref, head: back.head } : {}), at: new Date().toISOString() });
+}
+// A crash or a kill skips the cleanup above. When the conversation next runs a
+// general helper, a run its record still calls "running" (and this process
+// is not running) has its clone removed only if the folder is provably the one
+// this runtime made: a real folder, ours, directly in the temporary folder,
+// holding the token the record names. Anything else is only reported.
+export function sweepGeneral(managed) {
+  const last = new Map(), tmp = fs.realpathSync(os.tmpdir()), removed = [], unproven = [];
+  for (const e of managed.session.sessionManager.getEntries?.() ?? []) if (e.type === 'custom' && e.customType === GENERAL_ENTRY) last.set(e.data.id, e.data);
+  for (const d of last.values()) {
+    if (d.status !== 'running' || typeof d.root !== 'string' || managed.liveClones?.has(d.root)) continue;
+    let st; try { st = fs.lstatSync(d.root); } catch { continue; } // already gone
+    let owned = false;
+    try {
+      owned = st.isDirectory() && !st.isSymbolicLink() && st.uid === process.getuid() && path.dirname(d.root) === tmp && path.basename(d.root).startsWith('piagent-general-')
+        && fs.readFileSync(path.join(d.root, OWNER), 'utf8') === d.token;
+    } catch { /* no marker: not proven */ }
+    if (owned) { fs.rmSync(d.root, { recursive: true, force: true }); removed.push(d.id); } else unproven.push(d.root);
+  }
+  return { removed, unproven };
+}
+export const sweepText = ({ removed, unproven }) => (removed.length ? `\n\nAn earlier general run (${removed.join(', ')}) was interrupted; its clone folder was removed and any work in it was not returned.` : '')
+  + (unproven.length ? `\n\nA folder an earlier general run recorded could not be proven to be the one it made, so it was left in place: ${unproven.join(', ')}.` : '');