ai-agentmanager 1.7.0 → 1.9.0

package/src/manager.js

@@ -29,6 +29,12 @@ import {
   parseJsonReply,
   loadPlanFile,
   planApprovedPrompt,
+  intentMarkdown,
+  intentPath,
+  readSpecDoc,
+  specDocsInfo,
+  specPath,
+  specPrompt,
   planPath,
   planReviewPrompt,
 } from './brief.js';
@@ -93,6 +99,7 @@ async function repoLayout(dir) {
 import { spawn } from 'node:child_process';
 import * as git from './git.js';
 import { parseDiff } from '../public/review.js';
+import { issueSummary, leadingIssue, wordsAfterLink } from './jira.js';
 import { conflictMessage, findConflicts, related, repoKey, summarize as summarizeConflicts } from './conflicts.js';
 import { answerMessage, loadQuestions, questionsPath, questionsPrompt } from './inbox.js';
 import { STUCK_DEFAULTS, detectStuck } from './stuck.js';
@@ -335,6 +342,8 @@ export class AgentManager extends EventEmitter {
     const prs = Object.values(agent.prs ?? {});
     return {
       ...agent,
+      // Intent, spec and plan files that exist, with when each changed (task agents).
+      ...(NO_REPO.has(agent.kind) ? {} : { specDocs: specDocsInfo(agent.id) }),
       results: this.results(agent, rt, prs),
       priority: agent.priority ?? 2,
       activeAt: this.activeAt(agent, rt),
@@ -667,9 +676,15 @@ export class AgentManager extends EventEmitter {
     };
     for (const pr of refs.prs) agent.prs[pr.url] = { ...pr, source: 'intent' };
     for (const bug of refs.bugs) agent.bugs[bug.key] = { ...bug, source: 'intent' };
+    // A request that starts with an issue link: describe it by the issue's summary (fetched below),
+    // and until then by the words after the link.
+    const issue = leadingIssue(intent);
+    if (issue) agent.synopsis = { text: wordsAfterLink(intent), source: 'request', key: issue.key };
     this.store.put(agent);
     this.registerProject(agent);
     this.changed(agent);
+    this.writeIntent(agent.id);
+    if (issue) this.fetchSynopsis(agent.id).catch(() => {});
 
     this.plugins.emit('agent.created', { agent: this.auditView(agent) });
     this.prepare(agent, { start, workspace }).catch(() => {});
@@ -682,6 +697,50 @@ export class AgentManager extends EventEmitter {
     return this.view(agent);
   }
 
+  /** Fetch the summary of the issue an agent's request starts with; it becomes its description. */
+  async fetchSynopsis(ref) {
+    const agent = this.store.get(this.store.resolve(ref)?.id ?? ref);
+    const issue = agent && leadingIssue(agent.intent);
+    if (!issue) return null;
+    const summary = await issueSummary(issue, this.jiraOptions ?? {});
+    const cur = this.store.get(agent.id);
+    if (!summary || !cur) return null;
+    const bug = cur.bugs?.[issue.key];
+    this.patch(cur.id, {
+      synopsis: { text: summary, source: 'jira', key: issue.key },
+      ...(bug && !bug.title ? { bugs: { ...cur.bugs, [issue.key]: { ...bug, title: summary } } } : {}),
+    });
+    this.writeIntent(cur.id);
+    return summary;
+  }
+
+  /** INTENT.md for a task agent: the request as given and where it came from (see brief.js). */
+  writeIntent(id) {
+    const agent = this.store.get(id);
+    if (!agent || NO_REPO.has(agent.kind)) return;
+    try {
+      fs.mkdirSync(briefDir(id), { recursive: true });
+      fs.writeFileSync(intentPath(id), intentMarkdown(agent));
+    } catch {}
+  }
+
+  /** One of an agent's intent, spec and plan files. */
+  specDoc(ref, name) {
+    const agent = this.mustGet(ref);
+    if (!['intent', 'spec', 'plan'].includes(name)) throw Object.assign(new Error('Unknown file (intent, spec or plan)'), { status: 400 });
+    return readSpecDoc(agent.id, name) ?? { name, markdown: null };
+  }
+
+  /** Fill in summaries still missing (after Jira settings change); returns how many it found. */
+  async refreshSynopses() {
+    let found = 0;
+    for (const a of this.store.list()) {
+      if (a.synopsis?.source === 'jira' || !leadingIssue(a.intent)) continue;
+      if (await this.fetchSynopsis(a.id).catch(() => null)) found++;
+    }
+    return found;
+  }
+
   prepare(agent, { start, workspace }) {
     const controller = new AbortController();
     const { signal } = controller;
@@ -951,6 +1010,13 @@ export class AgentManager extends EventEmitter {
       prompts.push(briefPrompt(path.join(dir, path.basename(briefPath(agent.id)))));
       if (this.config.inbox !== false) prompts.push(questionsPrompt(path.join(dir, path.basename(questionsPath(agent.id)))));
     }
+    // SPEC.md before changing code, and PLAN.md kept as it goes (unless the plan review asks for the plan).
+    if (task && this.config.specs !== false) {
+      fs.mkdirSync(briefDir(agent.id), { recursive: true });
+      const dir = fs.realpathSync(briefDir(agent.id));
+      writableDir ??= dir;
+      prompts.push(specPrompt(path.join(dir, path.basename(specPath(agent.id))), path.join(dir, path.basename(planPath(agent.id))), { withPlan: !agent.planReview }));
+    }
     if (agent.planReview && task) {
       fs.mkdirSync(briefDir(agent.id), { recursive: true });
       const file = path.join(fs.realpathSync(briefDir(agent.id)), path.basename(planPath(agent.id)));
@@ -1976,7 +2042,9 @@ export class AgentManager extends EventEmitter {
           .join('\n\n');
       }
     }
-    return { intent: agent.intent, status, digest, repo, briefMarkdown };
+    // The spec and plan go along with the brief: what done means, and how it gets there.
+    const extra = ['spec', 'plan'].map((n) => readSpecDoc(agent.id, n)?.markdown.trim()).filter(Boolean);
+    return { intent: agent.intent, status, digest, repo, briefMarkdown: [briefMarkdown, ...extra].filter(Boolean).join('\n\n') };
   }
 
   /** The engine (and model) that writes an agent's summaries: its own when it can make a one-shot call, else Claude Code. */

package/src/review.js

@@ -2,6 +2,7 @@
 // back to the agent as one message, and commit / push / open a pull request from the UI.
 import * as git from './git.js';
 import { hash } from './judge.js';
+import { prSections } from './brief.js';
 import { NO_REPO } from './manager.js';
 import { ghBin } from './tracker.js';
 
@@ -193,8 +194,15 @@ export class Review {
     }
     const args = ['pr', 'create', '--head', branch];
     if (target) args.push('--base', target);
-    if (String(title).trim()) args.push('--title', String(title).trim(), '--body', String(body ?? ''));
-    else args.push('--fill');
+    // The description: your text (else the commits), then the intent, spec and plan.
+    const sections = prSections(agent.id);
+    if (String(title).trim() || sections) {
+      const log = base ? await git.run('git', ['log', '--reverse', '--format=%s', `${base.sha}..HEAD`], opts).catch(() => '') : '';
+      const subjects = log.split('\n').filter(Boolean);
+      const t = String(title).trim() || subjects[0] || agent.name;
+      const lead = String(body ?? '').trim() || (subjects.length > 1 ? subjects.map((x) => `- ${x}`).join('\n') : '');
+      args.push('--title', t, '--body', [lead, sections].filter(Boolean).join('\n\n'));
+    } else args.push('--fill');
     if (draft) args.push('--draft');
     let out;
     try {

package/src/server.js

@@ -33,6 +33,7 @@ import { Previews, parseUrl } from './preview.js';
 import { LiveBrowser } from './livebrowser.js';
 import * as remote from './remote.js';
 import * as service from './service.js';
+import { jiraView, saveJira } from './jira.js';
 import { Store } from './store.js';
 import { about, latestRelease } from './about.js';
 
@@ -60,7 +61,9 @@ function isAllowedOrigin(origin, port) {
 }
 
 const PUBLIC_REMOTE_API = new Set(['POST /api/login', 'GET /api/session']);
-const LOCAL_ONLY_API = /^\/api\/(remote(\/|$)|notify(\/|$)|hub(\/|$)|triggers(\/|$)|update(\/|$)|service$|editors$|agents\/[\w-]+\/open-in$|shutdown$)/;
+// Updates are not here: a signed-in device (a phone) can install the newest release too. It only
+// ever installs an official release from where this copy came from (git tag or npm) and restarts.
+const LOCAL_ONLY_API = /^\/api\/(remote(\/|$)|notify(\/|$)|hub(\/|$)|triggers(\/|$)|service$|jira$|editors$|agents\/[\w-]+\/open-in$|shutdown$)/;
 const HOOK_PATH = '/api/hooks/agents';
 const NO_REPO_KINDS = new Set(['general', 'assistant']);
 
@@ -237,6 +240,7 @@ export async function startServer({ port = PORT, host = HOST } = {}) {
       return manager.document(req.params.id, req.params.kind);
     }),
   );
+  api.get('/agents/:id/specs/:name', wrap((req) => manager.specDoc(req.params.id, req.params.name)));
   api.post('/agents/:id/plan', wrap((req) => manager.startPlanning(req.params.id)));
   api.post('/agents/:id/plan/approve', wrap((req) => manager.approvePlan(req.params.id, req.body?.note)));
   api.post('/agents/:id/plan/changes', wrap((req) => manager.requestPlanChanges(req.params.id, req.body?.note)));
@@ -508,6 +512,17 @@ export async function startServer({ port = PORT, host = HOST } = {}) {
   api.get('/agents/:id/memory', wrap((req) => memory.read(req.params.id)));
   api.put('/agents/:id/memory', wrap((req) => memory.write(req.params.id, req.body?.text)));
   api.post('/agents/:id/learn', wrap((req) => memory.learn(req.params.id, req.body?.text, { by: req.headers['x-agentmgr-agent'] ? undefined : 'you' })));
+  // Jira: the site and API token used to show an issue's summary for agents created from its link.
+  api.get('/jira', wrap(() => jiraView()));
+  api.put(
+    '/jira',
+    wrap(async (req) => {
+      const view = saveJira(req.body ?? {});
+      // Agents already created from links get their summaries now (in the background).
+      manager.refreshSynopses().catch(() => {});
+      return view;
+    }),
+  );
   // Start at login: the login service (launchd or systemd) that brings the daemon back after a reboot.
   const serviceSupported = ['darwin', 'linux'].includes(process.platform);
   const serviceView = async () => ({ supported: serviceSupported, ...(serviceSupported ? await service.status().catch(() => ({ installed: false, loaded: false })) : { installed: false, loaded: false }) });