@piagent/platform 1.46.2 → 1.46.4

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.46.2" title="v1.46.2 docs · VI"><span class="dot"></span>v1.46.2</a>
+      <a class="version-badge" href="/whats-new" data-docs-version="v1.46.4" title="v1.46.4 docs · VI"><span class="dot"></span>v1.46.4</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,31 @@
 <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-46-2" data-release>
+<article class="release latest" id="v1-46-4" data-release>
+  <div class="release-meta"><a class="release-version" href="#v1-46-4">v1.46.4</a><time class="release-date" datetime="2026-10-29">29/10/2026</time></div>
+  <div class="release-body">
+    <h3 data-toc-skip id="v1-46-4-title">Một subagent thay vì bị chặn; bản cập nhật đọc được models.json có comment</h3>
+    <p>Session công ty không còn lặp lại lệnh gọi subagent bị chặn, và bản cập nhật không còn dừng khi file danh sách model của Pi có comment.</p>
+    <ul class="release-items">
+      <li data-kind="improved"><span class="kind kind-improved">Cải thiện</span><span>Lỗi khi cấu hình danh sách model không còn làm dừng cả bản cài: bản cài cảnh báo, nêu tên file và lệnh để chạy lại.</span></li>
+      <li data-kind="fixed"><span class="kind kind-fixed">Sửa lỗi</span><span>Agent xin 2–3 subagent mà member không yêu cầu từng bị chặn, và một số model gọi lại liên tục đến khi member phải dừng. Giờ một subagent chạy, và agent được báo lý do.</span></li>
+      <li data-kind="fixed"><span class="kind kind-fixed">Sửa lỗi</span><span>piagent-update lỗi khi models.json có comment //, dù Pi cho phép. Giờ file được đọc như Pi đọc, và file có comment được giữ nguyên.</span></li>
+    </ul>
+  </div>
+</article>
+<article class="release" id="v1-46-3" data-release>
+  <div class="release-meta"><a class="release-version" href="#v1-46-3">v1.46.3</a><time class="release-date" datetime="2026-10-29">29/10/2026</time></div>
+  <div class="release-body">
+    <h3 data-toc-skip id="v1-46-3-title">Một subagent thay vì bị chặn; bản cập nhật đọc được models.json có comment <span class="tag">không phát hành</span></h3>
+    <p>Đã tag nhưng chưa publish: tiêu đề bản phát hành dài quá giới hạn của trang nên kiểm tra release fail. Thay bằng 1.46.4.</p>
+    <ul class="release-items">
+      <li data-kind="improved"><span class="kind kind-improved">Cải thiện</span><span>Lỗi khi cấu hình danh sách model không còn làm dừng cả bản cài: bản cài cảnh báo, nêu tên file và lệnh để chạy lại.</span></li>
+      <li data-kind="fixed"><span class="kind kind-fixed">Sửa lỗi</span><span>Agent xin 2–3 subagent mà member không yêu cầu từng bị chặn, và một số model gọi lại liên tục đến khi member phải dừng. Giờ một subagent chạy, và agent được báo lý do.</span></li>
+      <li data-kind="fixed"><span class="kind kind-fixed">Sửa lỗi</span><span>piagent-update lỗi khi models.json có comment //, dù Pi cho phép. Giờ file được đọc như Pi đọc, và file có comment được giữ nguyên.</span></li>
+    </ul>
+  </div>
+</article>
+<article class="release" id="v1-46-2" data-release>
   <div class="release-meta"><a class="release-version" href="#v1-46-2">v1.46.2</a><time class="release-date" datetime="2026-10-29">29/10/2026</time></div>
   <div class="release-body">
     <h3 data-toc-skip id="v1-46-2-title">Tin gửi sau câu trả lời hiện ngay; WebUI hiện chữ dần</h3>
@@ -395,42 +419,38 @@
     </ul>
   </div>
 </article>
-<article class="release" id="v1-40-0" data-release>
-  <div class="release-meta"><a class="release-version" href="#v1-40-0">v1.40.0</a><time class="release-date" datetime="2026-10-24">24/10/2026</time></div>
-  <div class="release-body">
-    <h3 data-toc-skip id="v1-40-0-title">Project local cũng được báo cho Studio</h3>
-    <p>Cuộc trò chuyện công ty trong thư mục không có remote báo project local cho Studio, mọi lượt chạy đều báo, và member thấy mọi loại cảnh báo project.</p>
-    <ul class="release-items">
-      <li data-kind="new"><span class="kind kind-new">Mới</span><span>Thư mục không có remote đọc được được báo như project local, theo commit gốc của Git hoặc theo mã băm; đường dẫn không bao giờ rời khỏi máy.</span></li>
-      <li data-kind="new"><span class="kind kind-new">Mới</span><span>Key local của project Git giống nhau trên mọi máy có cùng lịch sử, kể cả bản clone và checkout không có remote <code>origin</code>.</span></li>
-      <li data-kind="improved"><span class="kind kind-improved">Cải thiện</span><span>Mọi lượt chạy đều báo project, không chỉ lượt đầu của cuộc trò chuyện, nên Studio đếm được từng lần dùng và biết máy nào chưa bao giờ báo.</span></li>
-      <li data-kind="improved"><span class="kind kind-improved">Cải thiện</span><span>Member thấy một dòng cho mọi cảnh báo: project của team khác, ngoài danh sách của team, chưa đăng ký cho team, hoặc thư mục không đọc được.</span></li>
-      <li data-kind="improved"><span class="kind kind-improved">Cải thiện</span><span>Thư mục chứa nhiều checkout báo từng checkout, tối đa sáu, và project local được gọi theo tên thư mục.</span></li>
-      <li data-kind="security"><span class="kind kind-security">Bảo mật</span><span>Ba cảnh báo bảo mật của dependency trong Pi host được chấp nhận tới 2026-11-07: từ chối dịch vụ và lỗi Pi không dùng tới, không lộ dữ liệu.</span></li>
-    </ul>
-    <p class="release-requires"><strong>Cần:</strong> Agent Studio 0.40.0</p>
-  </div>
-</article>
-<article class="release" id="v1-39-0" data-release>
-  <div class="release-meta"><a class="release-version" href="#v1-39-0">v1.39.0</a><time class="release-date" datetime="2026-10-23">23/10/2026</time></div>
-  <div class="release-body">
-    <h3 data-toc-skip id="v1-39-0-title">Cảnh báo khi key của team được dùng ngoài team</h3>
-    <p>Studio biết cuộc trò chuyện công ty làm trong repository nào và cảnh báo khi key của team bị dùng ngoài team; dashboard giữ ngôn ngữ và theme màu.</p>
-    <ul class="release-items">
-      <li data-kind="new"><span class="kind kind-new">Mới</span><span>Cuộc trò chuyện công ty báo cho Studio các repository đang làm, dùng cùng key <code>host/path</code> mà team memory đã gửi.</span></li>
-      <li data-kind="new"><span class="kind kind-new">Mới</span><span>Admin thấy cảnh báo trong Logs → &quot;Sai project&quot;; member thấy một dòng nhắc phía trên ô soạn tin và trong <code>piagent studio</code>.</span></li>
-      <li data-kind="improved"><span class="kind kind-improved">Cải thiện</span><span>Không có gì bị chặn, và team memory chạy như cũ.</span></li>
-      <li data-kind="fixed"><span class="kind kind-fixed">Sửa lỗi</span><span>Dashboard trước đây quay về ngôn ngữ và theme màu mặc định sau mỗi lần khởi động lại; giờ nó giữ cả hai.</span></li>
-    </ul>
-    <p class="release-requires"><strong>Cần:</strong> Agent Studio 0.39.0</p>
-  </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">59</span></summary>
+    <summary><span>Tháng 10/2026</span><span class="archive-count">61</span></summary>
+    <div class="archive-row" id="v1-40-0">
+      <div class="archive-head"><a class="release-version" href="#v1-40-0">v1.40.0</a> <span class="release-date">24/10/2026</span></div>
+      <div class="archive-body">
+        <p><strong>Project local cũng được báo cho Studio</strong> — Cuộc trò chuyện công ty trong thư mục không có remote báo project local cho Studio, mọi lượt chạy đều báo, và member thấy mọi loại cảnh báo project.</p>
+        <ul class="release-items">
+        <li data-kind="new"><span class="kind kind-new">Mới</span><span>Thư mục không có remote đọc được được báo như project local, theo commit gốc của Git hoặc theo mã băm; đường dẫn không bao giờ rời khỏi máy.</span></li>
+        <li data-kind="new"><span class="kind kind-new">Mới</span><span>Key local của project Git giống nhau trên mọi máy có cùng lịch sử, kể cả bản clone và checkout không có remote <code>origin</code>.</span></li>
+        <li data-kind="improved"><span class="kind kind-improved">Cải thiện</span><span>Mọi lượt chạy đều báo project, không chỉ lượt đầu của cuộc trò chuyện, nên Studio đếm được từng lần dùng và biết máy nào chưa bao giờ báo.</span></li>
+        <li data-kind="improved"><span class="kind kind-improved">Cải thiện</span><span>Member thấy một dòng cho mọi cảnh báo: project của team khác, ngoài danh sách của team, chưa đăng ký cho team, hoặc thư mục không đọc được.</span></li>
+        <li data-kind="improved"><span class="kind kind-improved">Cải thiện</span><span>Thư mục chứa nhiều checkout báo từng checkout, tối đa sáu, và project local được gọi theo tên thư mục.</span></li>
+        <li data-kind="security"><span class="kind kind-security">Bảo mật</span><span>Ba cảnh báo bảo mật của dependency trong Pi host được chấp nhận tới 2026-11-07: từ chối dịch vụ và lỗi Pi không dùng tới, không lộ dữ liệu.</span></li>
+        </ul>
+      </div>
+    </div>
+    <div class="archive-row" id="v1-39-0">
+      <div class="archive-head"><a class="release-version" href="#v1-39-0">v1.39.0</a> <span class="release-date">23/10/2026</span></div>
+      <div class="archive-body">
+        <p><strong>Cảnh báo khi key của team được dùng ngoài team</strong> — Studio biết cuộc trò chuyện công ty làm trong repository nào và cảnh báo khi key của team bị dùng ngoài team; dashboard giữ ngôn ngữ và theme màu.</p>
+        <ul class="release-items">
+        <li data-kind="new"><span class="kind kind-new">Mới</span><span>Cuộc trò chuyện công ty báo cho Studio các repository đang làm, dùng cùng key <code>host/path</code> mà team memory đã gửi.</span></li>
+        <li data-kind="new"><span class="kind kind-new">Mới</span><span>Admin thấy cảnh báo trong Logs → &quot;Sai project&quot;; member thấy một dòng nhắc phía trên ô soạn tin và trong <code>piagent studio</code>.</span></li>
+        <li data-kind="improved"><span class="kind kind-improved">Cải thiện</span><span>Không có gì bị chặn, và team memory chạy như cũ.</span></li>
+        <li data-kind="fixed"><span class="kind kind-fixed">Sửa lỗi</span><span>Dashboard trước đây quay về ngôn ngữ và theme màu mặc định sau mỗi lần khởi động lại; giờ nó giữ cả hai.</span></li>
+        </ul>
+      </div>
+    </div>
     <div class="archive-row" id="v1-38-2">
       <div class="archive-head"><a class="release-version" href="#v1-38-2">v1.38.2</a> <span class="release-date">23/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.46.2" title="v1.46.2 docs · VI"><span class="dot"></span>v1.46.2</a>
+      <a class="version-badge" href="/whats-new" data-docs-version="v1.46.4" title="v1.46.4 docs · VI"><span class="dot"></span>v1.46.4</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.46.2" title="v1.46.2 docs · VI"><span class="dot"></span>v1.46.2</a>
+      <a class="version-badge" href="/whats-new" data-docs-version="v1.46.4" title="v1.46.4 docs · VI"><span class="dot"></span>v1.46.4</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

@@ -257,7 +257,7 @@
     {
       "path": "tests",
       "fileCount": 636,
-      "sha256": "2684ca3bdb704a5c2ea6764048dbed440ec0a8000fdbe7b1309cd42c099a8af6"
+      "sha256": "7d8a3af48dd9179f3bf3921cae65d3e6d18d17ff87d3454c29247f8cf1b34b63"
     }
   ],
   "custodianComparisonRules": {

package/package.json

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

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

@@ -20,17 +20,27 @@ export function namedCount(words) {
 }
 // The copies a delegate call asks for, checked against the member's words;
 // null for a plain call. `split`: one line per copy naming its share of the work.
+// Copies the member did not ask for come back as `{ dropped }`: one helper runs
+// and the main agent is told why. A refusal here sent agents round in retries,
+// each quoting other words, until the member stopped the turn.
 export function fanOut(manager, role, args, harness) {
   const split = Array.isArray(args?.split) ? args.split.map(s => typeof s === 'string' ? s.trim().slice(0, 500) : '').filter(Boolean) : [];
   const n = split.length || (Number.isInteger(args?.copies) ? args.copies : 1);
-  if (n <= 1 && !args?.member_words) return null;
+  if (n <= 1) return null;
   if (harness) throw Error('managed-copies-invalid: the harness runs one helper per call.');
-  if (n < 2 || n > MAX_COPIES) throw Error(`managed-copies-invalid: ask for 2 to ${MAX_COPIES} ${role} helpers (copies, or one split line per helper).`);
   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))
-    throw Error(`managed-copies-not-asked: more than one ${role} helper runs only when the member asks for it. Quote in member_words the member's own words that ask for ${n} (verbatim, with the number); otherwise use one helper.`);
+  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).`);
   return { n, split: split.length ? split : null };
 }
+// The helper's answer, with what the main agent reads above it when it asked
+// for helpers the member did not (`asked`: what fanOut returned).
+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`;
+  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.'}`;

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

@@ -24,7 +24,7 @@ import { readPatchDigest, readPatchSnapshot, readChangedFiles, readHead } from '
 import { memberTurn } from './continuation.mjs';
 import { workflowPolicy, repositoryChecks, sessionPlanTool, currentPlan, workflowPrompt, RunProcess, completionGate, pendingBaseline, pendingPaths, pendingHead, processEditTools } from './workflow.mjs';
 import { workflowContextTool, workflowContextExtension } from './workflow-context.mjs';
-import { beforeDelegate, resolveTool } from './objections.mjs'; import { fanOut } from './copies.mjs';
+import { beforeDelegate, resolveTool } from './objections.mjs'; import { fanOut, withDroppedNote } from './copies.mjs';
 import { HELPER_CALLS, HELPER_ROLES, helperRoles, helperPrompt, webPrompt, delegateDescription, mainWebLookups, limitedWebTools } from './helper-roles.mjs'; import { delegateParameters, delegateArgs, SPEC_PROMPT } from './helper-settlement.mjs';
 import { loadAgentResources } from './agent-skills.mjs';
 import { loadPiagentThemes } from '../runtime/terminal/terminal-style.mjs';
@@ -469,7 +469,7 @@ export class ManagedSession {
     if (!enabled.includes(role)) throw Error(`managed-helper-not-configured: the company Harness has no ${role} subagent${enabled.length ? `; use ${enabled.join(', ')}` : ''}.`);
     // Studio lets each helper role run HELPER_CALLS times per run (one user
     // message), each helper the member asked for counted (copies.mjs). Say so at once.
-    const fan = fanOut(this.session?.sessionManager, role, more, harness);
+    const asked = fanOut(this.session?.sessionManager, role, more, harness), fan = asked?.dropped ? null : asked;
     if ((this.helperCalls.get(role) ?? 0) + (fan?.n ?? 1) > HELPER_CALLS) throw Error(`managed-helper-limit: the ${role} subagent already ran ${HELPER_CALLS} times for this user message. Go on with what it returned, or use it again after the next user message.`);
     beforeDelegate(this.run, role, harness, { since, review: this.review, changed: (this.ownPaths?.size ?? 0) > 0, asked: !!fan });
     const job = this.runHelper(role, task, signal, harness, title, role === 'review' && typeof since === 'string' && since.trim() ? since.trim() : null, fan); this.helpers.set(role, job); this.publishHelpers();
@@ -477,7 +477,7 @@ export class ManagedSession {
     // quiet: a beat each minute tells the dashboard's watchdog, which stops a
     // turn after 15 minutes without an event, that the turn is working.
     const beat = setInterval(() => this.notify({ type: 'managed_helper_progress', role }), this.helperBeatMs ?? HELPER_BEAT_MS); beat.unref?.();
-    try { return await job; } finally { clearInterval(beat); this.helpers.delete(role); this.publishHelpers(); }
+    try { return withDroppedNote(await job, role, asked); } finally { clearInterval(beat); this.helpers.delete(role); this.publishHelpers(); }
   }
   publishHelpers() {
     // One of each enabled helper role can run at a time.

package/packages/piagent-core/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@piagent/core",
-  "version": "1.46.2",
+  "version": "1.46.4",
   "type": "module",
   "private": true,
   "license": "MIT",

package/packages/piagent-core/runtime/managed-failure.mjs

@@ -96,7 +96,6 @@ const CODES = {
   'managed-review-by-harness': ['tool', 'the harness reviews the final patch; the agent does not ask for one'],
   'managed-review-once': ['tool', 'this change was reviewed already; ask again only after fixing blocking findings'],
   'managed-copies-invalid': ['tool', 'several helpers of a role run 2 or 3 at a time, at the member\'s request'],
-  'managed-copies-not-asked': ['tool', 'more than one helper of a role runs only when the member asks for it'],
   helper_run_limit_reached: ['tool', 'this subagent already ran 8 times for this message'],
   'managed-helper-cancelled': ['tool', 'the subagent was stopped'],
   'managed-helper-failed': ['failed', 'the subagent ended without an answer'],

package/scripts/claude-model-additions.mjs

@@ -1,5 +1,6 @@
 import fs from "node:fs";
 import path from "node:path";
+import { parsePiModelsJson } from "./pi-models-json.mjs";
 
 const additions = JSON.parse(fs.readFileSync(new URL("../catalog/claude-model-additions.json", import.meta.url), "utf8"));
 const object = value => value !== null && typeof value === "object" && !Array.isArray(value);
@@ -9,7 +10,11 @@ const object = value => value !== null && typeof value === "object" && !Array.is
 export function prepareClaudeModelAdditions(file) {
   if (fs.existsSync(file) && (!fs.lstatSync(file).isFile() || fs.lstatSync(file).isSymbolicLink())) throw new Error("Unsafe model configuration path");
   const before = fs.existsSync(file) ? fs.readFileSync(file, "utf8") : null;
-  const data = before === null ? {} : JSON.parse(before);
+  let data = {}, strict = true;
+  if (before !== null) {
+    try { ({ data, strict } = parsePiModelsJson(before)); }
+    catch (error) { throw new Error(`Cannot read model configuration ${file}: ${error.message}`); }
+  }
   if (!object(data) || (data.providers !== undefined && !object(data.providers))) throw new Error("Unsupported model configuration");
   data.providers ??= {};
   const provider = data.providers.anthropic ?? {};
@@ -19,6 +24,8 @@ export function prepareClaudeModelAdditions(file) {
   const models = provider.models ?? [];
   const missing = additions.models.filter(model => !models.some(existing => existing.id === model.id));
   if (missing.length === 0) return null;
+  // Rewriting would drop the member's comments; leave the file to them.
+  if (!strict) return { skipped: `${file} has comments; not adding ${missing.map(model => model.id).join(", ")} (add them by hand if wanted)`, apply() {} };
   data.providers.anthropic = { ...provider, models: [...models, ...missing] };
   return { apply() {
     const current = fs.existsSync(file) ? fs.readFileSync(file, "utf8") : null;

package/scripts/configure-model-scope.sh

@@ -168,7 +168,11 @@ function parseDefaultModel(input) {
 
 const parsedDefault = parseDefaultModel(defaultModelInput);
 const settingsBefore = fs.existsSync(settingsPath) ? fs.readFileSync(settingsPath, "utf8") : null;
-const settings = settingsBefore === null ? {} : JSON.parse(settingsBefore);
+let settings = {};
+if (settingsBefore !== null) {
+  try { settings = JSON.parse(settingsBefore); }
+  catch (error) { console.error(`FAIL: cannot read Pi settings ${settingsPath}: ${error.message}`); process.exit(1); }
+}
 
 // What Piagent last offered, so an update adds only models new to this
 // release and never puts back one the member removed.
@@ -213,6 +217,7 @@ if (dryRun) {
   console.log(`  preset: ${preset}`);
   console.log(`  default: ${settings.defaultProvider}/${settings.defaultModel}:${settings.defaultThinkingLevel}`);
   console.log(`  enabledModels: ${settings.enabledModels.length}`);
+  if (catalogEdit?.skipped) console.log(`  note: ${catalogEdit.skipped}`);
   for (const note of notes) console.log(`  ${note}`);
 }
 NODE

package/scripts/install-global.sh

@@ -881,10 +881,16 @@ fi
 
 if [[ "$CONFIGURE_MODEL_SCOPE" == true ]]; then
   echo "Configuring Pi model selector scope:"
-  if [[ "$RESET_MODEL_SCOPE" == true ]]; then
-    run_cmd bash "$PLATFORM_ROOT/scripts/configure-model-scope.sh" --preset "$MODEL_SCOPE_PRESET" --default-model "$DEFAULT_MODEL"
-  else
-    run_cmd bash "$PLATFORM_ROOT/scripts/configure-model-scope.sh" --preset "$MODEL_SCOPE_PRESET" --default-model "$DEFAULT_MODEL" --keep-member-choices
+  MODEL_SCOPE_ARGS=(--preset "$MODEL_SCOPE_PRESET" --default-model "$DEFAULT_MODEL")
+  if [[ "$RESET_MODEL_SCOPE" != true ]]; then
+    MODEL_SCOPE_ARGS+=(--keep-member-choices)
+  fi
+  # The model list is a convenience: a member's settings or models.json that the
+  # script cannot read must not stop the install (Pi and the package are already
+  # in place). The script restores settings.json when it fails part way.
+  if ! run_cmd bash "$PLATFORM_ROOT/scripts/configure-model-scope.sh" "${MODEL_SCOPE_ARGS[@]}"; then
+    echo "WARN: the Pi model selector scope was not changed; the install continues." >&2
+    echo "      Fix the file named above, then run: piagent-model-scope ${MODEL_SCOPE_ARGS[*]}" >&2
   fi
 fi
 

package/scripts/personal-pi-config.mjs

@@ -15,11 +15,14 @@ import fs from "node:fs";
 import os from "node:os";
 import path from "node:path";
 import { fileURLToPath } from "node:url";
+import { parsePiModelsJson } from "./pi-models-json.mjs";
 
 const COMPANY_PROVIDER = /^agent_watch_/;
 const SESSION_HEADER_EXTENSION = "agentwatch-studio-session.mjs";
 
 const readJson = (file) => { try { return JSON.parse(fs.readFileSync(file, "utf8")); } catch { return null; } };
+// models.json may carry comments, as Pi allows; a strict read would miss company providers in it.
+const readModels = (file) => { try { return parsePiModelsJson(fs.readFileSync(file, "utf8")).data; } catch { return null; } };
 const real = (file) => { try { return fs.realpathSync(file); } catch { return null; } };
 
 // The Piagent package a path belongs to (its root), or null.
@@ -47,7 +50,7 @@ export function packageVersion(agentDir, source) {
 
 export function inspect(agentDir) {
   const settingsFile = path.join(agentDir, "settings.json"), modelsFile = path.join(agentDir, "models.json");
-  const settings = readJson(settingsFile) ?? {}, models = readJson(modelsFile) ?? {};
+  const settings = readJson(settingsFile) ?? {}, models = readModels(modelsFile) ?? {};
   const managed = fs.existsSync(path.join(agentDir, "agent-watch-managed.json"));
   const packages = Array.isArray(settings.packages) ? settings.packages : [];
   const piagentPackages = packages.filter(isPiagentSource);
@@ -109,7 +112,7 @@ export function repair(agentDir, now = new Date()) {
     backups.push(backup(settingsFile, stamp)); writeJson(settingsFile, settings); changed.push(settingsFile);
   }
   if (kinds.has("company-providers")) {
-    const models = readJson(modelsFile);
+    const models = readModels(modelsFile);
     for (const name of Object.keys(models.providers ?? {})) if (COMPANY_PROVIDER.test(name)) delete models.providers[name];
     backups.push(backup(modelsFile, stamp)); writeJson(modelsFile, models); changed.push(modelsFile);
   }

package/scripts/pi-models-json.mjs

@@ -0,0 +1,13 @@
+// Pi reads models.json leniently: a BOM, `//` line comments and trailing
+// commas are allowed (Pi's stripJsonComments). Piagent reads it the same way,
+// and says whether the text was plain JSON, so a writer knows it would drop
+// the member's comments.
+const stripComments = text => text
+  .replace(/"(?:\\.|[^"\\])*"|\/\/[^\n]*/g, m => (m[0] === '"' ? m : ""))
+  .replace(/"(?:\\.|[^"\\])*"|,(\s*[}\]])/g, (m, tail) => tail ?? (m[0] === '"' ? m : ""));
+
+export function parsePiModelsJson(text) {
+  const body = text.replace(/^\uFEFF/, "");
+  try { return { data: JSON.parse(body), strict: true }; }
+  catch { return { data: JSON.parse(stripComments(body)), strict: false }; }
+}