agy-mcp-bridge 0.1.30 → 0.1.31

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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agy-mcp-bridge",
-  "version": "0.1.30",
+  "version": "0.1.31",
   "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();