@jc_stack/ez-agents 0.1.0-beta.50 → 0.1.0-beta.52

package/test/intake-relay.test.ts

@@ -295,7 +295,7 @@ test('conversation menu is owner-only and removes the retired settings control',
     await f.relay.bot.handleUpdate(message(8, '/status'))
     assert.ok(!f.keyboards.at(-1)!.flat().some((button) => button.text === 'Scheduled tasks'))
     await f.relay.bot.handleUpdate(message(9, '/tasks'))
-    assert.match(f.replies.at(-1)!, /📅 Scheduled tasks[\s\S]*No active scheduled tasks/)
+    assert.match(f.replies.at(-1)!, /📅 Scheduled tasks[\s\S]*No scheduled tasks/)
     await f.relay.bot.handleUpdate(message(10, '/tasks 2'))
     assert.match(f.replies.at(-1)!, /Scheduled task 2 was not found/)
     assert.equal(f.launched.length, 0)
@@ -810,6 +810,13 @@ test('tasks list and detail show saved frequency and ordered setup path without
     assert.ok(detail.includes('Schedule\nEvery 30 minutes · America/New_York'))
     assert.ok(detail.includes(`Execution path\n${path}`))
     assert.deepEqual(await scheduler.get(saved.id),saved)
+    const paused=await scheduler.enable(saved.id,false)
+    await f.relay.bot.handleUpdate(message(3,'/tasks'))
+    assert.match(f.replies.at(-1)!,/0 active · 1 paused[\s\S]*⏸ Paused[\s\S]*Next · —/)
+    await f.relay.bot.handleUpdate(message(4,'/tasks 1'))
+    assert.match(f.replies.at(-1)!,/⏸ Paused[\s\S]*Next run\n—/)
+    assert.ok(f.replies.at(-1)!.includes(`Execution path\n${path}`))
+    assert.deepEqual(await scheduler.get(saved.id),paused)
     assert.equal(f.launched.length,0)
   }finally{await f.close()}
 })

package/test/plugin-manager.test.mjs

@@ -283,7 +283,7 @@ test('a lease from another PID namespace is live while its call container runs,
  // Isolated broker lease: its pid means nothing in this namespace (EPERM/ESRCH or an unrelated process).
  await fs.writeFile(lease,JSON.stringify({pid:1,container:'broker-call',pidNamespace:'pid:[4026531999]'}),{mode:0o600});
  const path0=process.env.PATH;process.env.PATH=f.fake+path.delimiter+path0;t.after(()=>{process.env.PATH=path0;});
- const docker=state=>fs.writeFile(path.join(f.fake,'docker'),`#!${process.execPath}\nconst a=process.argv.slice(2);if(a[0]==='container'&&a[1]==='inspect'){${state}}\n`,{mode:0o700});
+ const docker=async state=>{const file=path.join(f.fake,'docker');await fs.writeFile(file+'.next',`#!${process.execPath}\nconst a=process.argv.slice(2);if(a[0]==='container'&&a[1]==='inspect'){${state}}\n`,{mode:0o700});await fs.rename(file+'.next',file);};
  await docker(`console.log('true')`);
  await assert.rejects(locked(f.home,async()=>{}),/commands are active/); // never misreported as stale
  let activated=false;const drain=locked(f.home,async()=>{activated=true;},{drainInvocations:true});

package/test/run-handoff.test.ts

@@ -56,6 +56,9 @@ test('a graceful relay stop hands running host work to the next relay, which rec
     await rename(join(host, id + '.request.json'), join(host, id + '.running.json'))
     await appendFile(join(host, id + '.events'), JSON.stringify({ stream: 'stdout', text: 'working\n' }) + '\n')
   }
+  const delivered=await first.enqueueMessage(scheduled.id,'Delivered before replacement')
+  await first.claimOutbox(delivered.id)
+  await first.markOutboxSent(delivered.id,[17])
   await before.stop()
   assert.equal((await stat(join(dir, 'run-handoff.json'))).mode & 0o777, 0o600)
   const handoff = JSON.parse(await readFile(join(dir, 'run-handoff.json'), 'utf8'))
@@ -72,6 +75,7 @@ test('a graceful relay stop hands running host work to the next relay, which rec
     await after.adoptHandoff()
     await assert.rejects(readFile(join(dir, 'run-handoff.json')), { code: 'ENOENT' })
     assert.deepEqual((await runs.list()).map(run => run.status), ['running', 'running'])
+    assert.deepEqual(await runs.waitForDelivery(delivered.id,100),await first.waitForDelivery(delivered.id,100))
     // The new ledger accepts the run's result delivery and holds its recurring schedule.
     assert.ok(await runs.enqueueMessage(scheduled.id, 'Result after replacement'))
     await scheduler.tick(owner, runs, due + 60_000); await after.drainSources()

package/test/schedule-cli.test.ts

@@ -2,7 +2,7 @@ import test from 'node:test'
 import assert from 'node:assert/strict'
 import { execFile } from 'node:child_process'
 import { promisify } from 'node:util'
-import { mkdtemp, rm, writeFile } from 'node:fs/promises'
+import { mkdir, mkdtemp, rm, writeFile } from 'node:fs/promises'
 import { join } from 'node:path'
 import { tmpdir } from 'node:os'
 import { fileURLToPath } from 'node:url'
@@ -11,6 +11,7 @@ import { RunStore } from '../src/runs.js'
 import { initialPreset } from '../src/ai.js'
 import { serveTestLedger } from './helpers/ledger.js'
 import { Scheduler } from '../src/scheduler.js'
+import { Scripts } from '../src/scripts.js'
 import { callDeliverySocket, socketPathFor } from '../src/delivery-socket.js'
 const exec=promisify(execFile),bin=fileURLToPath(new URL('../bin/ezenciel-agents-schedule.mjs',import.meta.url))
 test('public scheduler CLI saves literal text, reads back, edits, pauses, and rejects external or finished callers',async t=>{
@@ -66,6 +67,28 @@ test('public scheduler CLI saves literal text, reads back, edits, pauses, and re
  await assert.rejects(exec(process.execPath,[bin,'list'],{env}),/owner-authorized/)
 })
 
+test('edit keeps a paused agent or script schedule paused and an enabled one enabled',async t=>{
+ const dir=await mkdtemp(join(tmpdir(),'ez-schedule-edit-'));t.after(()=>rm(dir,{recursive:true,force:true}))
+ const ledger=await serveTestLedger(dir);t.after(()=>ledger.stop())
+ const control=new ControlStore(dir,1000),workspace=join(dir,'workspace')
+ await control.requestPairing(101,101);await control.approveOwner(101)
+ await mkdir(workspace);await writeFile(join(workspace,'guard.mjs'),'')
+ await new Scripts(dir).save({id:'guard',owner:(await control.status()).owner!,workspace,entry:'guard.mjs',interpreter:'node',args:[],timeoutSeconds:60},true)
+ const env={...process.env,EZ_CONTROL_DIR:dir,EZ_RUN_ID:''}
+ const cli=async(...args:string[])=>JSON.parse((await exec(process.execPath,[bin,...args],{env})).stdout)
+ const tasks={agent:(text:string)=>['--model','gpt-6-astra','--text',text],script:(name:string)=>['--script','guard','--name',name]}
+ for(const [id,task] of Object.entries(tasks)){
+  await cli('create',id,...task('Original'),'--every-seconds','3600')
+  await cli('pause',id)
+  // Changing only the instructions, even with --now, must not resume or dispatch it.
+  const paused=await cli('edit',id,...task('Changed'),'--now')
+  assert.equal(paused.enabled,false);assert.equal(paused.nextEligibleAt,null)
+  assert.equal((await cli('show',id)).enabled,false)
+  await cli('resume',id)
+  assert.equal((await cli('edit',id,...task('Again'),'--every-seconds','3600')).enabled,true)
+ }
+})
+
 test('executor PATH exposes the extensionless scheduler command',async()=>{
  const command=fileURLToPath(new URL('../bin/ezenciel-agents-schedule',import.meta.url))
   assert.match((await exec(command,['--help'])).stdout,/Creates a scheduled task/)

package/test/scheduled-tasks.test.ts

@@ -38,10 +38,10 @@ test('scheduled task view is read-only, owner-bound, and shows active task promp
     scheduled: { id: saved.id, revision: saved.revision, dueAt: '2026-01-01T05:00:00.000Z', pairedAt: owner.pairedAt },
   })
   await runs.patch(run.id, { status: 'completed', endedAt: '2026-01-01T05:04:00.000Z' })
-  const active = await scheduler.listActiveReadOnly(await runs.list())
+  const active = await scheduler.listTasksReadOnly(await runs.list())
   const text = scheduledTasksText(active, owner)
 
-  assert.equal(text, '📅 Scheduled tasks · 1 active\n\nNext/last times UTC. Use /tasks 1 for task details.\n\n1 · Daily report\n  ● Active\n  Schedule · Weekdays at 09:00 · Asia/Dubai\n  Next · Thu, Jan 1 · 05:00 UTC\n  Last · ✓ Completed · Thu, Jan 1 · 05:04 UTC\n  Execution · codex (default login) · fixture-model · default effort\n  Read the ledger and send the owner a concise report.')
+  assert.equal(text, '📅 Scheduled tasks · 1 active · 0 paused\n\nNext/last times UTC. Use /tasks 1 for task details.\n\n1 · Daily report\n  ● Active\n  Schedule · Weekdays at 09:00 · Asia/Dubai\n  Next · Thu, Jan 1 · 05:00 UTC\n  Last · ✓ Completed · Thu, Jan 1 · 05:04 UTC\n  Execution · codex (default login) · fixture-model · default effort\n  Read the ledger and send the owner a concise report.')
   assert.match(scheduledTaskDetailText(active.find(schedule => schedule.id === saved.id)!, 1), /Last run\n✓ Completed · Thu, Jan 1 · 05:04 UTC/)
   assert.doesNotMatch(text, /Other owner task|This must never be visible/)
   assert.equal(await readFile(join(scheduleDir, 'owner-task.json'), 'utf8'), before)
@@ -62,29 +62,37 @@ test('active view follows existing cursor and run state without changing schedul
         id==='interrupted'?{everySeconds:60,start:new Date(due).toISOString()}:{at:new Date(due).toISOString()}})
   }
   await scheduler.save({...common,id:'recurring',name:'recurring',trigger:{everySeconds:60,start:new Date(due).toISOString()}})
-  const names = async () => (await scheduler.listActiveReadOnly(await runs.list())).map(s=>s.id).sort()
-  assert.deepEqual(await names(),['expired','interrupted','once','recurring','review'])
+  const names = async () => (await scheduler.listTasksReadOnly(await runs.list())).map(s=>s.id).sort()
+  assert.deepEqual(await names(),['expired','interrupted','once','paused','recurring','review'])
+  const paused = (await scheduler.listTasksReadOnly(await runs.list())).find(s=>s.id==='paused')!
+  assert.equal(paused.enabled,false)
+  assert.equal(paused.nextAt,null)
+  assert.match(scheduledTasksText([paused],owner),/0 active · 1 paused[\s\S]*⏸ Paused/)
+  assert.doesNotMatch(scheduledTasksText([paused],{...owner,pairedAt:'different'}),/⏸ Paused/)
+  assert.match(scheduledTaskDetailText({...paused,runState:'running'},1),/▶ Running · ⏸ Paused/)
+  assert.match(scheduledTaskDetailText({...paused,runState:'queued'},1),/◌ Queued · ⏸ Paused/)
   await scheduler.tick(owner,runs,due)
+  assert.ok(!(await runs.list()).some(r=>r.scheduled?.id==='paused'))
   // The empty conditional review consumes its occurrence without creating work.
-  assert.deepEqual(await names(),['expired','interrupted','once','recurring'])
+  assert.deepEqual(await names(),['expired','interrupted','once','paused','recurring'])
   const queued = (await runs.list()).find(r=>r.scheduled?.id==='once')!
-  const view = (await scheduler.listActiveReadOnly(await runs.list())).find(s=>s.id==='once')!
+  const view = (await scheduler.listTasksReadOnly(await runs.list())).find(s=>s.id==='once')!
   assert.equal(view.nextAt,null)
   assert.equal(view.runState,'queued')
   assert.match(scheduledTasksText([view],owner),/Queued/)
   await runs.patch(queued.id,{status:'running'})
-  assert.equal((await scheduler.listActiveReadOnly(await runs.list())).find(s=>s.id==='once')!.runState,'running')
+  assert.equal((await scheduler.listTasksReadOnly(await runs.list())).find(s=>s.id==='once')!.runState,'running')
   for (const run of await runs.list()) await runs.patch(run.id,run.scheduled?.id==='interrupted'
     ? {status:'failed',interrupted:true} : {status:'completed'})
   const before = await readdir(join(dir,'schedules'))
-  assert.deepEqual(await names(),['recurring'])
+  assert.deepEqual(await names(),['paused','recurring'])
   assert.deepEqual(await readdir(join(dir,'schedules')),before)
   // A held recurring revision remains hidden even when its cursor has a later occurrence.
   const interrupted = await scheduler.get('interrupted')
   assert.equal('everySeconds' in interrupted.trigger && interrupted.trigger.everySeconds,60)
   // A new schedule revision has its own occurrence; old terminal runs cannot hide it.
   await scheduler.save({...common,id:'once',name:'once',trigger:{at:new Date(due+60_000).toISOString()}})
-  assert.deepEqual(await names(),['once','recurring'])
+  assert.deepEqual(await names(),['once','paused','recurring'])
 })
 
 test('prompt preview is one bounded Unicode sentence and preserves stored input', async t => {
@@ -95,15 +103,15 @@ test('prompt preview is one bounded Unicode sentence and preserves stored input'
   const saved=await scheduler.save({id:'preview',name:'Preview',text,owner,
     execution:{...execution,preset:{...execution.preset,model:'gpt-5.6-terra',effort:'high'}},
     enabled:true,trigger:{at:'2027-01-01T00:00:00Z'}})
-  const output=scheduledTasksText(await scheduler.listActiveReadOnly([]),owner)
+  const output=scheduledTasksText(await scheduler.listTasksReadOnly([]),owner)
   assert.match(output,/gpt-5.6-terra · high/)
   assert.match(output,/Next · Fri, Jan 1 · 00:00 UTC/)
   assert.equal(Array.from(output.split('\n').at(-1)!.trim()).length,140)
   assert.ok(output.endsWith('…'))
   assert.doesNotMatch(output,/Do not display/)
   assert.equal((await scheduler.get(saved.id)).text,text)
-  const other=scheduledTasksText(await scheduler.listActiveReadOnly([]),{...owner,pairedAt:'other-pairing'})
-  assert.match(other,/No active scheduled tasks/)
+  const other=scheduledTasksText(await scheduler.listTasksReadOnly([]),{...owner,pairedAt:'other-pairing'})
+  assert.match(other,/No scheduled tasks/)
   assert.doesNotMatch(other,/Preview|🧪/)
 })
 
@@ -118,7 +126,7 @@ test('a legacy invalid selection does not prevent the active menu from rendering
   saved.execution.preset = {...saved.execution.preset,model:'gpt-5.6-terra',effort:'max'}
   await writeFile(file,JSON.stringify(saved))
 
-  const active = await scheduler.listActiveReadOnly([])
+  const active = await scheduler.listTasksReadOnly([])
   assert.match(scheduledTasksText(active,owner),/codex \(default login\) · gpt-5.6-terra · max/)
 })
 
@@ -128,7 +136,7 @@ test('detail view truncates long instructions and tolerates invalid timestamps',
   const scheduler = new Scheduler(dir)
   const saved = await scheduler.save({id:'long',name:'Long task',text:'/goal '+'x'.repeat(5000),owner,
     execution,enabled:true,trigger:{at:'2027-01-01T00:00:00Z'}})
-  const active = await scheduler.listActiveReadOnly([])
+  const active = await scheduler.listTasksReadOnly([])
   const detail = scheduledTaskDetailText(active.find(schedule => schedule.id === saved.id)!,1,1)
   assert.match(detail,/📅 1 of 1 · Long task/)
   assert.match(detail,/truncated; full text lives in the workspace/)
@@ -152,7 +160,7 @@ test('frequency describes saved rules without inventing a uniform interval for a
   ]
   for(const [i,[trigger,label]] of cases.entries()){
     const saved=await scheduler.save({id:`t-${i}`,name:`Task ${i}`,text:'Inspect saved work.',owner,execution,enabled:true,trigger})
-    const active=(await scheduler.listActiveReadOnly([])).find(s=>s.id===saved.id)!
+    const active=(await scheduler.listTasksReadOnly([])).find(s=>s.id===saved.id)!
     assert.ok(scheduledTasksText([active],owner).includes(`Schedule · ${label}\n`))
     assert.ok(scheduledTaskDetailText(active,1).includes(`Schedule\n${label}\n`))
   }

package/test/updates.test.mjs

@@ -14,7 +14,7 @@ import { atomic, snapshot, compose, prepareCommand } from '../src/plugins/manage
 import { bindUpdates } from '../src/updates/binding.mjs';
 import { TOOLS_LINE } from '../src/tools-line.mjs';
 import { status as runtimeStatus } from '../src/updates/status.mjs';
-import { providerEnvironment, queueAutomatic, waitForHostHeartbeat, withIdleUpgrade, withUpgrade } from '../src/updates/supervisor.mjs';
+import { idle, executorBindingArchive, executorVolumes, providerEnvironment, queueAutomatic, waitForHostHeartbeat, withIdleUpgrade, withUpgrade } from '../src/updates/supervisor.mjs';
 const exec=promisify(execFile);
 
 test('structured runtime output excludes warnings but failures retain diagnostics',async()=>{
@@ -573,10 +573,12 @@ for(const provider of ['pnpm','corepack']) test(`supervisor with only ${provider
  const interrupted=await read(path.join(jobPath(f.home,job.id),'job.json'));interrupted.status='applying';await atomic(path.join(jobPath(f.home,job.id),'job.json'),interrupted);
  await atomic(path.join(f.home,'registry.lock'),{pid:second.p.pid});
  const closed2=new Promise(r=>second.p.once('close',r));second.p.kill('SIGKILL');await closed2;
- const third=start();t.after(()=>third.p.kill('SIGTERM'));
+ const third=start(),closed3=new Promise(r=>third.p.once('close',r));t.after(()=>third.p.kill('SIGTERM'));
  await wait(async()=>{const j=await read(path.join(jobPath(f.home,job.id),'job.json'));return j.status==='rolled-back';});
  assert.equal((await read(path.join(f.home,'config.json'))).packageRoot,f.old);
- const closed3=new Promise(r=>third.p.once('close',r));third.p.kill('SIGTERM');assert.equal(await closed3,0,third.output());
+ // A rolled-back main update reloads automatically once native work is idle.
+ await wait(()=>third.p.exitCode!==null||third.p.signalCode!==null);
+ assert.equal(await closed3,0,third.output());
 });
 
 test('queued upgrades keep admission open while busy and release a raced or aborted pause',async t=>{
@@ -760,3 +762,62 @@ test('changed single-file broker binding retains full native drain and broker re
  assert.equal((await perform(f.home,job,{...r,refreshPluginBroker:true})).status,'completed');
  assert.equal(r.calls.filter(call=>call.includes('--force-recreate')).length,1);
 });
+
+test('handoff-compatible core upgrade builds with native work active and retires only at activation',async t=>{
+ const f=await fixture(t),job=await queued(f),r=runtime(f)
+ await fs.mkdir(path.join(f.agent.controlDir,'host-executor'),{recursive:true})
+ const claim=path.join(f.agent.controlDir,'host-executor','r_native.running.json')
+ await fs.writeFile(claim,'active native occurrence')
+ r.preserveRuns=true;r.retireHost=async()=>{assert.equal((await read(path.join(f.agent.controlDir,'upgrade-pause.json'))).id,job.id);r.calls.push(['retireHost'])}
+ const execute=r.execute
+ r.execute=async(command,args,options)=>{
+  if(args.includes('build')){await fs.access(claim);await assert.rejects(fs.access(path.join(f.agent.controlDir,'upgrade-pause.json')),{code:'ENOENT'});await assert.rejects(fs.access(path.join(f.home,'registry.lock')),{code:'ENOENT'})}
+  if(args.includes('stop')){
+   const backup=path.join(jobPath(f.home,job.id),'backup');
+   assert.equal(await fs.readFile(path.join(backup,'workspace','TOOLS.md'),'utf8'),'My notes\n','state backup finishes before relay shutdown');
+   await fs.access(path.join(backup,'control'));
+   await assert.rejects(fs.access(path.join(backup,'control','host-executor')),{code:'ENOENT'});
+  }
+  return execute(command,args,options)
+ }
+ assert.equal((await withUpgrade(f.agent.controlDir,job,new AbortController().signal,()=>perform(f.home,job,r),{preserveRuns:true,isIdle:async()=>false})).status,'completed')
+ assert.equal(r.calls.filter(c=>c[0]==='retireHost').length,1)
+ assert.ok(!r.calls.some(c=>c[0]==='stopHost'))
+ assert.equal(await fs.readFile(claim,'utf8'),'active native occurrence')
+ await assert.rejects(fs.access(path.join(f.agent.controlDir,'upgrade-pause.json')),{code:'ENOENT'})
+})
+
+test('failed handoff-compatible activation retires the candidate and rolls back without stopping native runs',async t=>{
+ const f=await fixture(t),job=await queued(f),r=runtime(f,{fail:(_command,args)=>args.includes('up')})
+ r.preserveRuns=true;r.retireHost=async()=>r.calls.push(['retireHost'])
+ assert.equal((await perform(f.home,job,r)).status,'rolled-back')
+ assert.equal(r.calls.filter(c=>c[0]==='retireHost').length,2)
+ assert.ok(!r.calls.some(c=>c[0]==='stopHost'))
+})
+
+
+test('missing executor startup ignores queued requests but never live native work',async t=>{
+ const root=await fs.mkdtemp(path.join(tmpdir(),'ez-executor-admission-'));t.after(()=>fs.rm(root,{recursive:true,force:true}));
+ const dir=path.join(root,'host-executor');await fs.mkdir(dir);
+ await fs.writeFile(path.join(dir,'queued.request.json'),'{}');
+ assert.equal(await idle(root),false);assert.equal(await idle(root,{pendingRequests:false}),true);
+ await fs.writeFile(path.join(dir,'live.running.json'),'{}');assert.equal(await idle(root,{pendingRequests:false}),false);
+ assert.equal(await fs.readFile(path.join(dir,'queued.request.json'),'utf8'),'{}');
+ await fs.rm(path.join(dir,'live.running.json'));
+ const server=createServer(socket=>socket.on('data',chunk=>{const req=JSON.parse(chunk.toString());socket.end(JSON.stringify({id:req.id,ok:true,result:{running:1}})+'\n');}));
+ await new Promise(resolve=>server.listen(path.join(root,'delivery.sock'),resolve));t.after(()=>new Promise(resolve=>server.close(resolve)));
+ assert.equal(await idle(root,{pendingRequests:false}),false);
+});
+test('executor preserves exact named volume identities and rejects missing declarations',()=>{
+ const worker={volumes:[{type:'volume',source:'registered-whatsapp-ipc'},{type:'bind',source:'/private/workspace'}]};
+ assert.deepEqual(executorVolumes({volumes:{'registered-whatsapp-ipc':{name:'existing-private-ipc'}}},worker),{'registered-whatsapp-ipc':{name:'existing-private-ipc',external:true}});
+ assert.throws(()=>executorVolumes({},worker),/Missing executor volume/);
+});
+test('executor archive extracts its exact private binding without host ownership changes',async t=>{
+ const root=await fs.mkdtemp(path.join(tmpdir(),'ez-executor-binding-'));t.after(()=>fs.rm(root,{recursive:true,force:true}));
+ const binding={agents:[{workspace:'/private/work',binDir:'/app/bin'}]},file=path.join(root,'binding.tar');
+ await fs.writeFile(file,executorBindingArchive(binding,1000),{mode:0o600});
+ await exec('tar',['-xf',file,'-C',root]);
+ const extracted=path.join(root,'executor.json');assert.deepEqual(await read(extracted),binding);assert.equal((await fs.stat(extracted)).mode&0o777,0o440);
+ for(const invalid of [-1,NaN,1.5,'1000',0o10000000])assert.throws(()=>executorBindingArchive(binding,invalid),/Invalid executor group/);
+});