agent-office-studio 0.6.0 → 0.7.0

package/server/index.js

@@ -341,6 +341,7 @@ guests = new Guests(DATA, {
   links,
   office: {
     jobs: () => state.jobs,
+    repoPath: (name) => repoPath(name),
     createJob: (body, extra) => createJob(body, extra),
     replyJob: (job, body) => replyJob(job, body),
     stopJob: (job, why) => stopJob(job, why),
@@ -740,8 +741,9 @@ function start(job, agentId) {
     // hooks, or memory: at Full autonomy those would all be the guest's.
     args.push('--strict-mcp-config', '--setting-sources', 'project,local', '--settings', JSON.stringify({ autoMemoryEnabled: false }));
   }
-  const allowed = job.guest ? GUEST_TOOLS : job.away ? [...OFFICE_TOOLS, ...awayTools(job)] : OFFICE_TOOLS;
-  args.push('--mcp-config', mcpFile, '--allowedTools', allowed.join(','), '--append-system-prompt', job.guest ? guestPrompt(agentId, job) : officePrompt(agentId, job) + awayPrompt(job));
+  // A guest's job can be away too: one of our agents, lent to the guest, in the guest's own project.
+  const allowed = [...(job.guest ? GUEST_TOOLS : OFFICE_TOOLS), ...(job.away ? awayTools(job) : [])];
+  args.push('--mcp-config', mcpFile, '--allowedTools', allowed.join(','), '--append-system-prompt', (job.guest ? guestPrompt(agentId, job) : officePrompt(agentId, job)) + awayPrompt(job));
   // Being given work in a channel is how you end up in it.
   try { joinChannel(channelOf(job.channel), agentId); } catch {}
   const first = job.sessionId ? withClearHint(job, job.pending) : job.pending;
@@ -1024,12 +1026,15 @@ function finish(job, status, error) {
   const text = status === 'done' ? job.lastTurn?.text || 'Done.'
     : status === 'stopped' ? 'Stopped, as asked.'
     : `I hit a wall: ${error || 'something went wrong'}`;
-  post(job.agentId, text, { jobId: job.id, status, denials: job.denials });
+  // A borrowed agent's chat belongs to the borrower: the host's feed carries
+  // the finished job over the link, and remoteEnded posts it in their office.
+  // Nothing about a guest's job goes in our own chat.
+  if (!job.guest) post(job.agentId, text, { jobId: job.id, status, denials: job.denials });
   // The result lands in the job's thread, which the boss may never open, so
   // the chief says so in #general. Not for a retry (the work isn't over), a
   // stop (the boss did that), or a colleague's handed-off piece (their asker
   // gets it, and the asker's own finish is the one worth announcing).
-  if (!retry && status !== 'stopped' && !job.requestedBy) {
+  if (!retry && status !== 'stopped' && !job.requestedBy && !job.guest) {
     const line = awaiting ? `${nameOf(job.agentId)} is waiting on you for "${clip(job.title, 80)}": ${clip(job.asking.summary, 200)}`
       : status === 'done' ? `${nameOf(job.agentId)} finished "${clip(job.title, 80)}".${briefOf(job.lastTurn?.text) ? `\n\n${briefOf(job.lastTurn.text)}` : ''}` : `${nameOf(job.agentId)} hit a wall on "${clip(job.title, 80)}": ${clip(error || 'something went wrong', 200)}`;
     post('chief', `**${awaiting ? 'Waiting on you' : status === 'done' ? 'Done' : 'Failed'}** · ${line}`, { kind: 'finished', status, awaiting, jobId: job.id, thread: null, channel: GENERAL });
@@ -1404,7 +1409,7 @@ function remoteEnded(job) {
   const name = guests.remoteStatus().find((o) => o.link === job.remote.link)?.staff.find((s) => s.id === job.agentId)?.name || 'Their agent';
   const last = job.log.at(-1);
   const how = { done: 'finished', failed: 'hit a wall on', stopped: 'stopped' }[job.status] || 'stopped';
-  const said = last?.kind === 'result' && last.summary ? `\n\n${clip(last.summary, 600)}` : job.error ? ` ${job.error}` : '';
+  const said = last?.kind === 'result' && last.summary ? `\n\n${clip(last.summary, 4000)}` : job.error ? ` ${job.error}` : '';
   post('chief', `${name} at @${job.remote.host} ${how} “${job.title}”.${said}`, { jobId: job.id, channel: job.channel, thread: job.thread || null });
 }
 
@@ -1466,7 +1471,7 @@ function chat({ text, mode = 'edit', model = '', files, thread, channel }) {
   if (remote) {
     if (files.length) return void say(`Files can’t go to @${remote.host}’s office. Paste the text instead.`);
     guests.startRemote({ link: remote.link, staff: remote.staff, text: remote.clean, mode, model, thread: root?.id || msg.id, channel: where })
-      .then(({ job, repo, capped }) => say(`${remote.staff.name} at @${remote.host} is on it, in ${repo}.${capped ? ` @${remote.host} allows up to ${MODES[job.mode]?.label || job.mode}, so it runs at that.` : ''}`, { jobId: job.id, thread: root ? root.id : null }))
+      .then(({ job, repo, home, capped }) => say(`${remote.staff.name} at @${remote.host} is on it, in ${home ? `your ${repo}, on @${remote.host}’s account. Their changes go into a copy you review.` : `${repo}.`}${capped ? ` @${remote.host} allows up to ${MODES[job.mode]?.label || job.mode}, so it runs at that.` : ''}`, { jobId: job.id, thread: root ? root.id : null }))
       .catch((err) => say(err.message, { thread: root ? root.id : null }));
     return;
   }