agy-mcp-bridge 0.1.30 → 0.1.32

package/dist/src/agents/codex.js

@@ -1,7 +1,7 @@
 import { readFile } from 'node:fs/promises';
 import path from 'node:path';
 import { isQuotaError, isStrongQuotaError } from './quota.js';
-import { startProcess, terminateOwned } from '../antigravity/process.js';
+import { captureOutput } from './probe.js';
 const CAPABILITIES = ['code', 'review', 'test', 'research', 'long-run'];
 const REQUIRED_FLAGS = ['--json', '--sandbox', '--cd', '--output-last-message', '--ephemeral', '--skip-git-repo-check'];
 const MAX_RESULT_BYTES = 1024 * 1024;
@@ -86,20 +86,6 @@ export function codexAdapter(command, options = {}) {
 export function codexAgent(command, status, options = {}) {
     return { adapter: codexAdapter(command, options), status, enabled: true, prefixArgs: command.prefixArgs };
 }
-async function capture(command, args, cwd, extraEnvPassthrough) {
-    return new Promise((resolve, reject) => {
-        const child = startProcess(command.executable, [...command.prefixArgs, ...args], cwd, { extraEnvPassthrough });
-        let text = '';
-        let exceeded = false;
-        const timer = setTimeout(() => { exceeded = true; void terminateOwned(child).catch(reject); }, 10000);
-        const append = (chunk) => { if (text.length < 32768)
-            text += chunk.toString().slice(0, 32768 - text.length); };
-        child.stdout.on('data', append);
-        child.stderr.on('data', append);
-        child.once('error', error => { clearTimeout(timer); reject(error); });
-        child.once('close', code => { clearTimeout(timer); exceeded ? reject(new Error('Codex inspection timed out')) : resolve({ code, text }); });
-    });
-}
 /** Non-interactive `codex exec` support is the only contract this adapter relies on. */
 export async function detectCodex(command, options = {}) {
     const cwd = options.cwd ?? process.cwd();
@@ -108,8 +94,8 @@ export async function detectCodex(command, options = {}) {
         id: 'codex', label: 'Codex', available, version, executable: command.source, capabilities: CAPABILITIES, warnings
     });
     try {
-        const version = await capture(command, ['--version'], cwd, extra);
-        const help = await capture(command, ['exec', '--help'], cwd, extra);
+        const version = await captureOutput('Codex', command, ['--version'], cwd, extra);
+        const help = await captureOutput('Codex', command, ['exec', '--help'], cwd, extra);
         const versionText = version.text.trim().split(/\r?\n/)[0] ?? '';
         const parsed = /codex-cli\s+([^\s]+)/i.exec(versionText)?.[1] ?? (versionText || null);
         const missing = REQUIRED_FLAGS.filter(flag => !new RegExp(`(?:^|\\s)${flag.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')}(?=\\s|,|$)`).test(help.text));

package/dist/src/agents/opencode.js

@@ -1,4 +1,4 @@
-import { startProcess, terminateOwned } from '../antigravity/process.js';
+import { captureOutput } from './probe.js';
 import { isQuotaError } from './quota.js';
 export const CAPABILITIES = ['code', 'review', 'test', 'research', 'long-run'];
 // Verified against opencode v2.2.5: `opencode run` reads the message from stdin and, with `--format json`, writes one JSON event per line.
@@ -120,20 +120,6 @@ export function opencodeAdapter(command, options = {}) {
 export function opencodeAgent(command, status, options = {}) {
     return { adapter: opencodeAdapter(command, options), status, enabled: true, prefixArgs: command.prefixArgs };
 }
-function capture(command, args, cwd, extraEnvPassthrough) {
-    return new Promise((resolve, reject) => {
-        const child = startProcess(command.executable, [...command.prefixArgs, ...args], cwd, { extraEnvPassthrough });
-        let text = '';
-        let exceeded = false;
-        const timer = setTimeout(() => { exceeded = true; void terminateOwned(child).catch(reject); }, 10000);
-        const append = (chunk) => { if (text.length < 32768)
-            text += chunk.toString().slice(0, 32768 - text.length); };
-        child.stdout.on('data', append);
-        child.stderr.on('data', append);
-        child.once('error', error => { clearTimeout(timer); reject(error); });
-        child.once('close', code => { clearTimeout(timer); exceeded ? reject(new Error('OpenCode inspection timed out')) : resolve({ code, text }); });
-    });
-}
 /** Non-interactive `opencode run --format json` is the only contract this adapter relies on. */
 export async function detectOpenCode(command, opts = {}) {
     const cwd = opts.cwd ?? process.cwd();
@@ -142,8 +128,8 @@ export async function detectOpenCode(command, opts = {}) {
         id: 'opencode', label: 'OpenCode', available, version, executable: command.executable, capabilities: CAPABILITIES, warnings
     });
     try {
-        const version = await capture(command, ['--version'], cwd, extra);
-        const help = await capture(command, ['run', '--help'], cwd, extra);
+        const version = await captureOutput('OpenCode', command, ['--version'], cwd, extra);
+        const help = await captureOutput('OpenCode', command, ['run', '--help'], cwd, extra);
         const parsed = version.text.trim().split(/\r?\n/)[0]?.replace(/^opencode\s+/i, '').replace(/^v/i, '') || null;
         if (version.code !== 0)
             return status(false, null, ['opencode --version failed']);

package/dist/src/agents/probe.js

@@ -0,0 +1,20 @@
+import { startProcess, terminateOwned } from '../antigravity/process.js';
+/**
+ * Runs an agent CLI once to read its output (`--version`, `<subcommand> --help`) while it is being inspected. Bounded in time (10 s) and in
+ * size (32 KiB of combined stdout and stderr), and the process is killed with its tree when the time runs out. `label` names the CLI in the error.
+ */
+export function captureOutput(label, command, args, cwd, extraEnvPassthrough) {
+    return new Promise((resolve, reject) => {
+        const child = startProcess(command.executable, [...command.prefixArgs, ...args], cwd, { extraEnvPassthrough });
+        let text = '';
+        let exceeded = false;
+        const timer = setTimeout(() => { exceeded = true; void terminateOwned(child).catch(reject); }, 10000);
+        const append = (chunk) => { if (text.length < 32768)
+            text += chunk.toString().slice(0, 32768 - text.length); };
+        child.stdout.on('data', append);
+        child.stderr.on('data', append);
+        child.once('error', error => { clearTimeout(timer); reject(error); });
+        child.once('close', code => { clearTimeout(timer); exceeded ? reject(new Error(`${label} inspection timed out`)) : resolve({ code, text }); });
+    });
+}
+//# sourceMappingURL=probe.js.map

package/dist/src/agents/registry.js

@@ -54,23 +54,30 @@ export async function createAgentRegistry(config, antigravity) {
         askForApproval: settings.codex_ask_for_approval
     }));
     const opencodeEnabled = enabled.includes('opencode');
-    let opencodeStatus = {
+    const autoApproveWarning = (id) => `Agent "${id}" is configured to auto-approve permissions (opencode_auto_approve): tasks run with opencode --auto`;
+    // The OpenCode CLI is probed when the `opencode` agent is enabled, and also when an enabled opencode profile relies on it:
+    // OpenCode is opt-in, so a profile declared without `enabled` must not end up unavailable because the base agent is off.
+    const profileUsesOpencode = Object.entries(settings.profiles ?? {}).some(([id, profile]) => profile.adapter === 'opencode' && !profile.executable && (enabled.includes(id) || !settings.enabled));
+    let opencodeProbe = {
         id: 'opencode', label: 'OpenCode', available: false, version: null, executable: null,
         capabilities: CAPABILITIES, warnings: ['Disabled in the bridge config']
     };
     let opencodeCommand = { executable: 'opencode', prefixArgs: [], source: 'opencode' };
-    if (opencodeEnabled) {
+    if (opencodeEnabled || profileUsesOpencode) {
         try {
             opencodeCommand = await resolveCommand(settings.opencode_executable ?? 'opencode');
-            opencodeStatus = await detectOpenCode(opencodeCommand, { cwd: process.cwd(), extraEnvPassthrough: config.extra_env_passthrough });
+            opencodeProbe = await detectOpenCode(opencodeCommand, { cwd: process.cwd(), extraEnvPassthrough: config.extra_env_passthrough });
         }
         catch (error) {
-            opencodeStatus = { ...opencodeStatus, warnings: [`OpenCode CLI not found: ${error instanceof Error ? error.message : 'unknown error'}`] };
+            opencodeProbe = { ...opencodeProbe, warnings: [`OpenCode CLI not found: ${error instanceof Error ? error.message : 'unknown error'}`] };
         }
     }
-    const opencodeAutoWarning = 'Agent "opencode" is configured to auto-approve permissions (opencode_auto_approve): tasks run with opencode --auto';
+    // The base agent is only available when it was asked for; the probe above may exist just for a profile.
+    let opencodeStatus = opencodeEnabled
+        ? opencodeProbe
+        : { ...opencodeProbe, available: false, warnings: ['Disabled in the bridge config'] };
     if (opencodeEnabled && settings.opencode_auto_approve)
-        opencodeStatus = { ...opencodeStatus, warnings: [...opencodeStatus.warnings, opencodeAutoWarning] };
+        opencodeStatus = { ...opencodeStatus, warnings: [...opencodeStatus.warnings, autoApproveWarning('opencode')] };
     agents.push(opencodeAgent(opencodeCommand, opencodeStatus, {
         model: settings.opencode_model,
         autoApprove: settings.opencode_auto_approve
@@ -79,7 +86,7 @@ export async function createAgentRegistry(config, antigravity) {
         for (const [profileId, profile] of Object.entries(settings.profiles)) {
             const isProfileEnabled = enabled.includes(profileId) || (!settings.enabled && true);
             const adapterType = profile.adapter;
-            const label = profile.label ?? (adapterType === 'codex' ? `Codex (${profileId})` : `Antigravity (${profileId})`);
+            const label = profile.label ?? `${{ codex: 'Codex', antigravity: 'Antigravity', opencode: 'OpenCode' }[adapterType]} (${profileId})`;
             const profileWarnings = [];
             if (!isProfileEnabled)
                 profileWarnings.push('Disabled in the bridge config');
@@ -148,8 +155,8 @@ export async function createAgentRegistry(config, antigravity) {
             else if (adapterType === 'opencode') {
                 let cmd = opencodeCommand.executable;
                 let prefixArgs = opencodeCommand.prefixArgs;
-                let available = opencodeStatus.available;
-                let version = opencodeStatus.version;
+                let available = opencodeProbe.available;
+                let version = opencodeProbe.version;
                 if (profile.executable) {
                     try {
                         const resolved = await resolveCommand(profile.executable);
@@ -170,7 +177,7 @@ export async function createAgentRegistry(config, antigravity) {
                     version: version || null, executable: cmd || null,
                     capabilities: CAPABILITIES,
                     warnings: isProfileEnabled
-                        ? [...opencodeStatus.warnings.filter(w => !w.includes('Disabled') && w !== opencodeAutoWarning), ...profileWarnings, ...(settings.opencode_auto_approve ? [opencodeAutoWarning.replace('"opencode"', `"${profileId}"`)] : [])]
+                        ? [...(profile.executable ? [] : opencodeProbe.warnings.filter(w => !w.includes('Disabled'))), ...profileWarnings, ...(settings.opencode_auto_approve ? [autoApproveWarning(profileId)] : [])]
                         : profileWarnings
                 };
                 agents.push({

package/dist/src/cli.js

@@ -10,7 +10,7 @@ import { addSessionWorkspaces } from './config.js';
 import { recoverExecutionLock } from './tasks/execution-lock.js';
 import { renderSetup } from './setup.js';
 import { DEFAULT_TOOL_TIMEOUT_SECONDS, codexConfigPath, planCodexConfig, readCodexConfig, writeCodexConfig } from './codex-config.js';
-import { freebuffConfigPath, planFreebuffConfig, readFreebuffConfig, writeFreebuffConfig } from './freebuff-config.js';
+import { freebuffConfigPath, isTransientInstall, planFreebuffConfig, readFreebuffConfig, writeFreebuffConfig } from './freebuff-config.js';
 import { readWait } from './wait-setting.js';
 import { configuredPort } from './http.js';
 import { offerSkillInstall, offerSkillUpdate } from './onboarding.js';
@@ -217,6 +217,8 @@ async function setup(args) {
         return;
     }
     const clients = target === 'both' ? ['codex', 'claude'] : [target];
+    if (target === 'freebuff' && isTransientInstall(paths.packageRoot))
+        console.error(`Note: this copy of ${BIN} runs from a temporary npx cache, so the stdio command printed below stops working once npm clears it. Install it globally (npm install -g ${pkg.name}) and print it again before using it.`);
     console.log(renderSetup({ clients, mcpUrl, shells: shell, bin: BIN, waitSeconds: readWait(paths.stateDirectory) ?? undefined, stdio: stdioCommand() }));
 }
 /** The command a stdio client must run for THIS installation: the interpreter and the packaged stdio entry, both absolute. */
@@ -225,6 +227,11 @@ export function stdioCommand() {
 }
 /** Shows exactly what would change in Freebuff's .agents/mcp.json and writes it only after a yes (or --yes). */
 async function applyFreebuffSetup(project, yes) {
+    if (isTransientInstall(paths.packageRoot)) {
+        console.error(`Not changed: this copy of ${BIN} runs from a temporary npx cache (${paths.packageRoot}), which npm can delete, so Freebuff's config would point at a path that stops working. Install it first: npm install -g ${pkg.name}, then run ${BIN} setup freebuff --apply again.`);
+        process.exitCode = 1;
+        return;
+    }
     if (project !== undefined) {
         const info = await stat(project).catch(() => null);
         if (!info?.isDirectory())

package/dist/src/freebuff-config.js

@@ -10,6 +10,11 @@ import path from 'node:path';
  * diff stays readable. The caller shows the plan and asks before anything is written, and a backup is made first.
  */
 export const SERVER_KEY = 'antigravity';
+/**
+ * `npx` unpacks a package into a cache directory (`.../_npx/<hash>/node_modules/...`) that npm may delete at any time. An
+ * absolute path into it must not be written into a client's persistent config: it works today and breaks silently later.
+ */
+export const isTransientInstall = (packageRoot) => /[\\/]_npx[\\/]/.test(packageRoot);
 /** `~/.agents/mcp.json`, or `<project>/.agents/mcp.json`: Freebuff reads the file nearest to where it runs. */
 export const freebuffConfigPath = (options = {}) => path.join(options.project ?? options.home ?? os.homedir(), '.agents', 'mcp.json');
 /** The indent the file already uses, so re-serializing it does not reformat the servers someone else wrote. */

package/dist/src/isolation/apply.js

@@ -12,7 +12,7 @@ import { isOwned, normalizeOwn } from './worktree.js';
  */
 const MAX_PATCH_BYTES = 8 * 1024 * 1024;
 const MAX_FILES = 500;
-const gitEnvironment = () => ({
+export const gitEnvironment = () => ({
     PATH: process.env.PATH, PATHEXT: process.env.PATHEXT, SystemRoot: process.env.SystemRoot, COMSPEC: process.env.COMSPEC,
     TEMP: process.env.TEMP, TMP: process.env.TMP, TMPDIR: process.env.TMPDIR,
     GIT_CONFIG_NOSYSTEM: '1', GIT_CONFIG_GLOBAL: process.platform === 'win32' ? 'NUL' : '/dev/null',

package/dist/src/isolation/diff.js

@@ -0,0 +1,44 @@
+import { execFile } from 'node:child_process';
+import path from 'node:path';
+import { redact } from '../security/redact.js';
+import { gitEnvironment } from './apply.js';
+/**
+ * The content diff of one file in a task's worktree, for the owner to read before applying it. Read only: nothing is staged or written
+ * (`git add` would race with an agent that is still running), and the worktree is bridge-owned, so its files are not the owner's.
+ * Secrets are filtered the way task logs are.
+ */
+const MAX_DIFF_CHARS = 200_000;
+const NULL_DEVICE = process.platform === 'win32' ? 'NUL' : '/dev/null';
+/** A path inside the worktree, as the changes list reports it: relative, no drive, no `..`, no empty segment. */
+export function isRelativeFilePath(file) {
+    if (typeof file !== 'string' || !file || file.length > 512 || file.includes('\0'))
+        return false;
+    if (path.isAbsolute(file) || /^[A-Za-z]:/.test(file) || file.startsWith('/') || file.startsWith('\\'))
+        return false;
+    return !file.replace(/\\/g, '/').split('/').some(part => part === '' || part === '..');
+}
+function capture(cwd, args) {
+    return new Promise(resolve => {
+        execFile('git', ['-c', 'core.fsmonitor=false', ...args], { cwd, env: gitEnvironment(), timeout: 20000, maxBuffer: 1024 * 1024, encoding: 'utf8', windowsHide: true }, (error, stdout) => {
+            const code = error ? (typeof error.code === 'number' ? error.code : null) : 0;
+            resolve({ stdout: stdout ?? '', code, overflow: Boolean(error) && /maxBuffer/i.test(error.message) });
+        });
+    });
+}
+export async function worktreeFileDiff(worktree, file) {
+    if (!isRelativeFilePath(file))
+        throw new Error('path must be a relative path inside the worktree');
+    const literal = `:(literal)${file}`;
+    // Tracked files diff against the base commit; a file the agent created is not in the index, so it is shown as all added lines.
+    const tracked = (await capture(worktree, ['ls-files', '--error-unmatch', '-z', '--', literal])).code === 0;
+    const result = tracked
+        ? await capture(worktree, ['diff', 'HEAD', '--no-ext-diff', '--no-textconv', '--no-color', '--', literal])
+        : await capture(worktree, ['diff', '--no-index', '--no-ext-diff', '--no-textconv', '--no-color', '--', NULL_DEVICE, file]);
+    // `git diff --no-index` exits 1 when the files differ; 128 means the file is not there at all.
+    if (!result.stdout && !result.overflow)
+        return { path: file, diff: '', truncated: false, binary: false, changed: false };
+    const truncated = result.overflow || result.stdout.length > MAX_DIFF_CHARS;
+    const text = redact(result.stdout.slice(0, MAX_DIFF_CHARS));
+    return { path: file, diff: text, truncated, binary: /^Binary files .* differ$/m.test(text), changed: true };
+}
+//# sourceMappingURL=diff.js.map

package/dist/src/tasks/manager.js

@@ -7,6 +7,7 @@ import { DEFAULT_MAX_WAIT_SECONDS, delegateSchema, promptFor } from '../security
 import { inside, validateWorkspace } from '../security/workspace.js';
 import { appendDecision } from '../decisions.js';
 import { applyWorktreeChanges, repositoryRoot } from '../isolation/apply.js';
+import { worktreeFileDiff } from '../isolation/diff.js';
 import { redact } from '../security/redact.js';
 import { delegationState } from '../delegation.js';
 import { startProcess, terminateOwned } from '../antigravity/process.js';
@@ -744,14 +745,14 @@ export class TaskManager {
         this.decisionQueue = run.catch(() => undefined);
         return run;
     }
-    async reviewableTask(id) {
+    async reviewableTask(id, allowRunning = false) {
         const entry = this.entries.get(id);
         if (!entry)
             throw new Error('Unknown or expired task_id');
         const task = entry.task;
         if (task.isolation !== 'worktree' || !task.worktree)
             throw new Error('This task did not run in a worktree');
-        if (task.status === 'queued' || task.status === 'running')
+        if (!allowRunning && (task.status === 'queued' || task.status === 'running'))
             throw new Error('The task is still running');
         if (task.discarded_at)
             throw new Error('This task worktree was already discarded');
@@ -768,6 +769,11 @@ export class TaskManager {
     recordDecision(id, action, ok, detail) {
         return appendDecision(this.runtime.stateDirectory, { at: new Date().toISOString(), task_id: id, action, ok, detail });
     }
+    /** One file's content diff from a worktree task. Read only, so it is allowed while the task still runs. */
+    async fileDiff(id, file) {
+        const task = await this.reviewableTask(id, true);
+        return worktreeFileDiff(task.worktree, file);
+    }
     applyTask(id) {
         return this.enqueueDecision(async () => {
             const task = await this.reviewableTask(id);

package/dist/src/ui/routes.js

@@ -6,6 +6,7 @@ import { paths } from '../paths.js';
 import { knownUpdate, readPackageInfo } from '../update.js';
 import { workspaceChanges } from '../changes.js';
 import { readDecisions } from '../decisions.js';
+import { isRelativeFilePath } from '../isolation/diff.js';
 import { validModelId, writeModel } from '../models.js';
 import { readProgress, writeProgress } from '../progress.js';
 import { readUsage, summarizeUsage } from '../usage.js';
@@ -193,6 +194,16 @@ export async function handleUi(req, res, ctx) {
             json(res, 200, { task: await ctx.manager.discardTask(discard[1]) });
             return true;
         }
+        const fileDiff = pathname.match(new RegExp(`^/ui/api/tasks/(${UUID})/diff$`));
+        if (fileDiff && method === 'GET') {
+            const file = new URL(req.url ?? '', 'http://localhost').searchParams.get('path');
+            if (!isRelativeFilePath(file)) {
+                json(res, 400, { error: 'path must be a relative path inside the worktree' });
+                return true;
+            }
+            json(res, 200, await ctx.manager.fileDiff(fileDiff[1], file));
+            return true;
+        }
         if (pathname === '/ui/api/decisions' && method === 'GET') {
             json(res, 200, { decisions: await readDecisions(ctx.manager.stateDirectory) });
             return true;

package/docs/MULTI-AGENT-PLAN.md

@@ -245,12 +245,15 @@ ngân sách token theo mission — usage vẫn chỉ được ghi và tổng h
   không đổi nhánh). Bị từ chối, kèm lý do, khi: task chưa `completed`, có file ngoài `owns`, workspace không còn trong allowlist,
   workspace đang có sửa đổi chưa commit trên cùng file, patch không còn khớp (workspace đã đi tiếp), hoặc thay đổi có symlink/submodule.
 - `Bỏ worktree` xóa thư mục và nhánh `bridge/<id>` của task (giữ nguyên workspace).
+- Diff nội dung từng file (lát cắt 2): bấm một file ở "Thay đổi trong workspace" của task worktree để đọc diff so với commit gốc
+  (`GET /ui/api/tasks/<id>/diff?path=<file>`). Chỉ đọc, không stage gì (an toàn khi task còn chạy), secret được lọc như log, chỉ nhận đường dẫn
+  tương đối trong worktree. Task không dùng worktree chưa có diff nội dung.
 - Nhật ký quyết định ở `<state>/decisions.jsonl` (mỗi dòng: thời điểm, task, apply/discard, ok, chi tiết), hiện trong panel và ở `GET /ui/api/decisions`.
 - Chỉ chủ bridge làm được việc này qua web UI (`POST /ui/api/tasks/<id>/apply|discard` với `confirm: true`). Không có tool MCP tương ứng, nên một
   agent hay client không tự áp dụng kết quả của chính nó.
 
 Chưa làm của Phase 4: trang Settings cho agent ưu tiên / số task song song / autonomy mặc định (hiện là file cấu hình), hàng đợi duyệt hành động
-ngoài chính sách, nút hợp nhất theo nhánh (commit) và diff nội dung từng file.
+ngoài chính sách và nút hợp nhất theo nhánh (commit).
 
 ### Phase 5 — Tích hợp và hardening
 

package/docs/OPENCODE-INTEGRATION-PLAN.md

@@ -5,6 +5,24 @@
 
 Không thay đổi hành vi hiện tại của antigravity và codex. OpenCode nằm trong **Phase 3** theo `docs/MULTI-AGENT-PLAN.md`.
 
+## Đã làm khác thiết kế ban đầu (đọc mục này trước các mục 1–3)
+
+Các mục 1–3 bên dưới là bản thiết kế viết trước khi có code, giữ lại để biết lý do. Nơi nào mâu thuẫn với bảng này thì **bảng này đúng** (đã đối chiếu với `opencode` v2.2.5 và với code):
+
+| Thiết kế ban đầu | Thực tế |
+|---|---|
+| `opencode` vào `DEFAULT_ENABLED` | **Mặc định tắt**; chỉ chạy khi `agents.enabled` liệt kê. Profile OpenCode khai báo mà bỏ `agents.enabled` vẫn được kiểm tra CLI và báo lý do nếu không dùng được |
+| `opencode run <prompt>` qua argv | `opencode run --standalone --format json`, prompt qua **stdin** (argv bị giới hạn độ dài, và prompt bắt đầu bằng `-` bị đọc thành cờ) |
+| không nhắc service nền | **`--standalone` bắt buộc**: mặc định `opencode run` giao việc cho service nền dùng chung của OpenCode Desktop, hủy task không dừng agent và shell của agent nằm ngoài cây process mà bridge dọn (đo ở `docs/HANDOFF.md` mục 7) |
+| `OPENCODE_HOME` tách tài khoản | Biến này **không có tác dụng**; dùng `XDG_DATA_HOME` / `XDG_CONFIG_HOME` trong `profile.env` |
+| model `gpt-5` | Dạng `provider/model` (xem `opencode models`), ví dụ `opencode/big-pickle` |
+| `parseResult` đọc JSON đơn hoặc text thường | Đọc luồng JSONL: sự kiện `text` cuối là câu trả lời, `step_finish` cho lượt và token, `error` là thất bại |
+| `autonomy: "full"` có thể thêm `--auto` | **Không bao giờ**: `autonomy` do client chọn và chỉ là chỉ dẫn. `--auto` chỉ bật bằng `agents.opencode_auto_approve` trong config của chủ máy, và hiện trong cảnh báo của `agents_list` |
+| không có giới hạn log | `compactLine` bỏ `step_start` và rút `tool_use` về tên tool và trạng thái, để luồng JSON không vượt giới hạn output |
+| quyền mặc định không bàn tới | OpenCode **chạy lệnh shell và sửa file mà không hỏi** kể cả không có `--auto` (đo trên v2.2.5): mô hình quyền của nó lỏng hơn mặc định của `agy` và Codex `workspace-write`. Bật nó trong `agents.enabled` là chấp nhận điều đó |
+
+Mục 3 ("Câu hỏi cần trả lời") đã được trả lời bằng bảng trên; không còn câu hỏi mở.
+
 ---
 
 ## 1. Khai báo kỹ thuật

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agy-mcp-bridge",
-  "version": "0.1.30",
+  "version": "0.1.32",
   "description": "Local loopback HTTP MCP bridge that delegates bounded coding tasks to multiple agent CLIs (Antigravity, Codex) with optional parallel git worktrees",
   "license": "MIT",
   "keywords": [

package/ui/app.css

@@ -150,6 +150,15 @@ details.more .body { padding: 10px 12px 12px; }
 .change-list { margin: 0; padding: 0; list-style: none; display: grid; max-height: 360px; overflow: auto; border: 1px solid var(--border); border-radius: 10px; }
 .change-row { display: grid; grid-template-columns: 64px minmax(0, 1fr) auto; gap: 10px; align-items: center; padding: 7px 12px; border-top: 1px solid var(--border); font-size: 13px; }
 .change-row:first-child { border-top: 0; }
+.change-row.clickable { cursor: pointer; }
+.change-row.clickable:hover, .change-row.clickable:focus-visible { background: var(--surface-2); outline: none; }
+.diff-box { margin-top: 12px; display: grid; gap: 8px; }
+.diff { white-space: pre; overflow: auto; max-height: 460px; }
+.diff span { display: block; min-height: 1.3em; }
+.diff .add { color: var(--ok); background: var(--ok-soft); }
+.diff .del { color: var(--bad); background: var(--bad-soft); }
+.diff .hunk { color: var(--warn); }
+.diff .meta { color: var(--muted); }
 .change-path { overflow-wrap: anywhere; }
 .change-count { color: var(--muted); white-space: nowrap; font-size: 12.5px; }
 .change-kind { text-align: center; padding: 1px 6px; border-radius: 6px; font-size: 11.5px; font-weight: 650; background: var(--idle-soft); color: var(--idle); }

package/ui/app.js

@@ -375,6 +375,36 @@ function collapsible(key, title, ...body) {
 function more(key, ...body) { return collapsible(key, 'Xem thêm', ...body); }
 
 const CHANGE_LABEL = {modified: 'Sửa', deleted: 'Xóa', added: 'Thêm', 'type-changed': 'Đổi kiểu'};
+/* ---------- Content diff of one changed file (worktree tasks only; read only, secrets filtered on the server) ---------- */
+async function loadDiff(task, file) {
+  state.diff = {task_id: task.task_id, path: file, loading: true}; patch();
+  try { state.diff = {task_id: task.task_id, path: file, data: await api(`tasks/${task.task_id}/diff?path=${encodeURIComponent(file)}`)}; }
+  catch (error) { state.diff = {task_id: task.task_id, path: file, error: error.message}; }
+  patch();
+}
+/** A row is a button only where a diff can be read: a task that ran in a worktree, which the bridge owns. */
+function rowProps(task, file) {
+  if (task.isolation !== 'worktree' || task.discarded_at) return {class: 'change-row'};
+  const open = () => loadDiff(task, file);
+  return {class: 'change-row clickable', tabindex: '0', role: 'button', 'aria-label': `Xem diff của ${file}`, onclick: open,
+    onkeydown: event => { if (event.key === 'Enter' || event.key === ' ') { event.preventDefault(); open(); } }};
+}
+const diffLineClass = line => line.startsWith('+++') || line.startsWith('---') || line.startsWith('diff ') || line.startsWith('index ') ? 'meta'
+  : line.startsWith('@@') ? 'hunk' : line.startsWith('+') ? 'add' : line.startsWith('-') ? 'del' : '';
+function diffViewer(task) {
+  const current = state.diff && state.diff.task_id === task.task_id ? state.diff : null;
+  if (!current) return null;
+  const close = h('button', {class: 'btn small', onclick: () => { state.diff = null; patch(); }, text: 'Đóng'});
+  const head = h('div', {class: 'change-head'}, h('strong', {class: 'mono', text: current.path}), close);
+  if (current.loading) return h('div', {class: 'diff-box'}, head, h('div', {class: 'typing'}, h('span', {class: 'spinner'}), 'Đang đọc diff…'));
+  if (current.error) return h('div', {class: 'diff-box'}, head, h('div', {class: 'callout bad', text: current.error}));
+  const data = current.data;
+  if (!data.changed) return h('div', {class: 'diff-box'}, head, h('p', {class: 'muted', text: 'File này không có thay đổi so với commit gốc của worktree.'}));
+  if (data.binary) return h('div', {class: 'diff-box'}, head, h('p', {class: 'muted', text: 'File nhị phân: chỉ biết là có thay đổi, không hiện được nội dung.'}));
+  return h('div', {class: 'diff-box'}, head,
+    h('pre', {class: 'log diff'}, data.diff.split('\n').map(line => h('span', {class: diffLineClass(line), text: line}))),
+    data.truncated ? h('div', {class: 'callout warn', text: 'Diff dài nên chỉ hiện phần đầu.'}) : null);
+}
 function changesBody(task) {
   const c = state.changes && state.changes.task_id === task.task_id ? state.changes : null;
   const refresh = h('button', {class: 'btn small', onclick: () => loadChanges(task.task_id), disabled: Boolean(c && c.loading), text: c && c.data ? 'Làm mới' : 'Xem thay đổi'});
@@ -384,16 +414,18 @@ function changesBody(task) {
   const d = c.data;
   if (!d.available) return [h('div', {class: 'callout warn', text: d.reason || 'Không đọc được thay đổi.'}), refresh];
   const rows = [
-    ...d.changes.map(item => h('li', {class: 'change-row'}, h('span', {class: `change-kind ${item.status}`, text: CHANGE_LABEL[item.status] || item.status}),
+    ...d.changes.map(item => h('li', rowProps(task, item.path), h('span', {class: `change-kind ${item.status}`, text: CHANGE_LABEL[item.status] || item.status}),
       h('span', {class: 'change-path mono', text: item.path}),
       h('span', {class: 'change-count mono', text: item.added === null ? 'nhị phân' : `+${item.added} −${item.deleted}`}))),
-    ...d.untracked.map(item => h('li', {class: 'change-row'}, h('span', {class: 'change-kind untracked', text: 'Mới'}),
+    ...d.untracked.map(item => h('li', rowProps(task, item.path), h('span', {class: 'change-kind untracked', text: 'Mới'}),
       h('span', {class: 'change-path mono', text: item.path}), h('span', {class: 'change-count mono', text: item.bytes === null ? '' : bytes(item.bytes)})))
   ];
   return [
     h('div', {class: 'change-head'}, h('strong', {text: rows.length ? `${d.changes.length} file đã sửa hoặc xóa · ${d.untracked.length} file mới chưa theo dõi` : 'Không có thay đổi nào so với HEAD.'}), refresh),
-    h('p', {class: 'muted', text: `So với commit ${d.head ? d.head.slice(0, 8) : '(repo chưa có commit)'}; gồm cả thay đổi đã có từ trước khi task chạy. Chỉ hiện tên file và số dòng, không có nội dung.`}),
+    h('p', {class: 'muted', text: `So với commit ${d.head ? d.head.slice(0, 8) : '(repo chưa có commit)'}; gồm cả thay đổi đã có từ trước khi task chạy. ${task.isolation === 'worktree' && !task.discarded_at ? 'Danh sách chỉ có tên file và số dòng; nội dung đọc ở diff từng file.' : 'Chỉ hiện tên file và số dòng, không có nội dung.'}`}),
+    rows.length && task.isolation === 'worktree' && !task.discarded_at ? h('p', {class: 'muted', text: 'Bấm một file để đọc nội dung thay đổi (đã lọc secret).'}) : null,
     rows.length ? h('ul', {class: 'change-list'}, rows) : null,
+    diffViewer(task),
     d.truncated ? h('div', {class: 'callout warn', text: 'Chỉ hiện 500 mục đầu tiên của mỗi nhóm.'}) : null
   ];
 }
@@ -437,6 +469,7 @@ function reviewBody(task) {
       h('button', {class: 'btn primary small', disabled: Boolean(block), title: block || 'Ghi thay đổi vào workspace', onclick: () => applyTask(task), text: 'Áp dụng vào workspace'}),
       h('button', {class: 'btn danger small', disabled: Boolean(task.discarded_at), onclick: () => discardTask(task), text: 'Bỏ worktree'})),
     block && !task.applied_at && !task.discarded_at ? h('p', {class: 'muted', text: block}) : null,
+    !task.applied_at && !task.discarded_at ? h('p', {class: 'muted', text: 'Đọc diff từng file ở mục "Thay đổi trong workspace" trước khi áp dụng.'}) : null,
     mine.length ? h('div', {}, h('div', {class: 'msg-head', text: 'Nhật ký quyết định'}),
       h('ul', {class: 'change-list'}, mine.map(item => h('li', {class: 'change-row'},
         h('span', {class: `change-kind ${item.ok ? 'added' : 'deleted'}`, text: item.action === 'apply' ? 'Áp dụng' : 'Bỏ'}),
@@ -874,7 +907,7 @@ function patch() {
     setRegion('list', JSON.stringify([state.q, state.filter, state.selected, o.budget?.task_tokens, o.tasks.map(t => stable(t))]), () => taskItems(o));
     regions.count.textContent = taskCount(o);
     const d = state.detail;
-    const detailKey = JSON.stringify([state.selected, state.logTab, state.changes, state.decisions, d && {...d, activity: null, task: {...stable(d.task), stdout: '', stderr: '', stdout_bytes: 0, stderr_bytes: 0, stdout_truncated: false, stderr_truncated: false}}]);
+    const detailKey = JSON.stringify([state.selected, state.logTab, state.changes, state.decisions, state.diff, d && {...d, activity: null, task: {...stable(d.task), stdout: '', stderr: '', stdout_bytes: 0, stderr_bytes: 0, stdout_truncated: false, stderr_truncated: false}}]);
     const rebuilt = signatures.detail !== detailKey;
     setRegion('detail', detailKey, () => detailView());
     if (!rebuilt) patchLogs();