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

package/test/ai.test.ts

@@ -106,13 +106,17 @@ test('OpenCode keeps same-provider models together and separates providers encod
     const other = { ...first, id: 'other', name: 'Other', model: 'openrouter/other' }
     const choice = await store.captureChoice(first)
     await store.saveNativeSession(choice.sessionId, 'ses_original')
-    const menu = createAiMenu(store, 'opencode', async () => [first, second, other].map(p => ({ ...p, efforts: [] })), dir, undefined, async () => true)
+    const closed: string[] = []
+    const menu = createAiMenu(store, 'opencode', async () => [first, second, other].map(p => ({ ...p, efforts: [] })), dir, undefined, async () => true,
+      undefined, async (session) => { if (session) closed.push(session.sessionId) })
     await store.savePreset(second)
     assert.equal((await menu.select(second, choice.sessionId)).fresh, false)
+    assert.deepEqual(closed, [])
     assert.equal((await store.captureChoice(first)).sessionId, choice.sessionId)
     await store.savePreset(other)
     assert.equal(await store.selectPreset(other.id, choice.sessionId), false)
     assert.equal((await menu.select(other, choice.sessionId)).fresh, true)
+    assert.deepEqual(closed, [choice.sessionId]) // switching provider closes the old conversation once
     const next = await store.captureChoice(first)
     assert.notEqual(next.sessionId, choice.sessionId)
     assert.equal((await store.executionSession(next)).nativeSessionId, undefined)
@@ -519,7 +523,7 @@ test('opencode model and variant selections validate against the installed catal
   await validateSelection({ id: 'default', name: 'Default', cli: 'opencode', model: 'opencode/big-pickle' }, catalog, available)
   await assert.rejects(
     validateSelection({ id: 'bad', name: 'Bad', cli: 'opencode', model: 'opencode/ling-free', effort: 'max' }, catalog, available),
-    /installed client catalog/)
+    /Effort max is not supported.*low, medium/)
   await assert.rejects(
     validateSelection({ id: 'missing', name: 'Missing', cli: 'opencode', model: 'opencode/unknown' }, catalog, available),
     /installed client catalog/)
@@ -672,3 +676,45 @@ test('Claude catalog projects only aliases and efforts from the installed native
   await validateSelection({ id: 'c', name: 'c', cli: 'claude', model: 'opus', effort: 'xhigh' }, models, async () => true)
   await assert.rejects(validateSelection({ id: 'c', name: 'c', cli: 'claude', model: 'opus-guess' }, models, async () => true), /not in the installed client catalog/)
 })
+
+test('Telegram Sonnet choice survives restart and reaches the native launch without Opus substitution', async () => {
+  const dir = await mkdtemp(join(tmpdir(), 'ez-sonnet-selection-'))
+  const priorPath = process.env.PATH
+  try {
+    const store = new ControlStore(dir, 1000)
+    await store.captureChoice({id:'opus',name:'Opus',cli:'claude',model:'opus',effort:'xhigh'})
+    const models = [{cli:'claude',model:'opus',name:'Opus',efforts:['xhigh']},
+      {cli:'claude',model:'sonnet',name:'Sonnet',efforts:['low','medium','high']}]
+    const menu = createAiMenu(store, 'claude', async()=>models, dir, undefined, async()=>true)
+    const replies: Array<{text:string; buttons:Array<{text:string;callback_data:string}>}> = []
+    const context = (data?:string) => ({callbackQuery:data ? {data} : undefined,answerCallbackQuery:async()=>({}),
+      reply:async(text:string, options?:{reply_markup?:{inline_keyboard?:Array<Array<{text:string;callback_data:string}>>}})=>{
+        replies.push({text,buttons:options?.reply_markup?.inline_keyboard?.flat() ?? []});return {} as never
+      }})
+    const click = async(text:string) => menu.handle(context(replies.at(-1)!.buttons.find(b=>b.text===text)!.callback_data) as never)
+    await menu.list(context() as never)
+    await click('claude');await click('Sonnet');await click('high')
+    const restarted = new ControlStore(dir,1000)
+    const captured = await restarted.captureChoice(menu.initial)
+    assert.equal(captured.preset.model,'sonnet');assert.equal(captured.preset.effort,'high')
+    const before = await restarted.status()
+    await assert.rejects(menu.saveSelection(models[1], 'xhigh'), /Effort xhigh is not supported.*low, medium, high/)
+    await assert.rejects(validateSelection({id:'s',name:'Sonnet',cli:'claude',model:'sonnet',effort:'high'}, [], async()=>true), /No models advertised for claude.*current selection unchanged/)
+    assert.deepEqual(await restarted.status(), before)
+    const {ownerRun} = await import('./helpers/owner-run.js')
+    const {startExecutorJob} = await import('../src/executor.js')
+    await mkdir(join(dir,'bin'))
+    await writeFile(join(dir,'bin','claude'), `#!${process.execPath}\nrequire('fs').writeFileSync('launched.json',JSON.stringify(process.argv.slice(2)));require('fs').readFileSync(0,'utf8')`, {mode:0o700})
+    process.env.PATH = join(dir,'bin') + ':' + priorPath
+    await ownerRun(dir,'r_sonnet_selected')
+    const job = await startExecutorJob(['synthetic request'], {workspace:dir,controlDir:dir,binDir:join(dir,'bin'),runId:'r_sonnet_selected',timeoutMs:5000,repairEnabled:false,
+      cli:captured.preset.cli,model:captured.preset.model,effort:captured.preset.effort,sessionId:captured.sessionId})
+    const closed = new Promise<number|null>(resolve=>job.child.once('close',resolve))
+    assert.equal(await closed,0)
+    assert.deepEqual(JSON.parse(await readFile(join(dir,'launched.json'),'utf8')).slice(-4), ['--model','sonnet','--effort','high'])
+    await job.cleanup()
+  } finally {
+    if (priorPath === undefined) delete process.env.PATH; else process.env.PATH = priorPath
+    await rm(dir,{recursive:true,force:true})
+  }
+})

package/test/application-channel.test.ts

@@ -450,3 +450,27 @@ test('administrator task grant replacement preserves the runtime file owner', {s
     const disabled=await stat(file);assert.equal(disabled.uid,20000);assert.equal(disabled.gid,20000)
   }finally{await rm(dir,{recursive:true,force:true})}
 })
+
+test('empty server Claude catalog rejects Sonnet chat selection without changing Opus xhigh', async t => {
+  const root = await mkdtemp(join(tmpdir(),'ez-app-missing-claude-'))
+  t.after(()=>rm(root,{recursive:true,force:true}))
+  const owned = await owner(root), control = new ControlStore(root,1000)
+  const initial = {id:'opus',name:'Opus',cli:'claude',model:'opus',effort:'xhigh'}
+  await control.captureChoice(initial)
+  const {createAiMenu} = await import('../src/menu.js')
+  let models: import('../src/ai.js').ModelChoice[] = []
+  const menu = createAiMenu(control,'claude',async()=>models,root,undefined,async()=>true)
+  const channel = new ApplicationChannel({controlDir:root,initial,wake:()=>{},cancel:async()=>{},aiControls:menu})
+  const binding = (await channel.bindings.register('app',token(),owned,true))!
+  const before = await control.status()
+  const input = {action:'model',expectedSession:before.activeSession!.sessionId,cli:'claude',model:'sonnet',effort:'high'}
+  await assert.rejects(channel.changeControls(binding.bindingId,input), /No models advertised for claude.*current selection unchanged/)
+  assert.deepEqual(await control.status(),before)
+  models = [{cli:'claude',model:'sonnet',name:'Sonnet',efforts:['low','medium','high']}]
+  await assert.rejects(channel.changeControls(binding.bindingId,{...input,effort:'xhigh'}), /Effort xhigh is not supported.*low, medium, high/)
+  assert.deepEqual(await control.status(),before)
+  await channel.changeControls(binding.bindingId,input)
+  const admitted = await channel.submit(binding.bindingId,{requestId:'sonnet',scope:'chat',text:'Synthetic request',followTelegram:true})
+  assert.equal(admitted.execution!.preset.model,'sonnet')
+  assert.equal(admitted.execution!.preset.effort,'high')
+})

package/test/application-controls.test.ts

@@ -53,7 +53,8 @@ test('private scope reset and model controls preserve running sessions and share
     {cli:'codex',model:'fixture-model',name:'Fixture',efforts:['low','high']},
     {cli:'opencode',model:'fixture-other',name:'Other',efforts:[]},
   ],root,join(root,'native-home'),async()=>true)
-  const channel = new ApplicationChannel({controlDir:root,initial:menu.initial,aiControls:menu,wake:()=>{},cancel:async()=>{}})
+  const closed:string[] = []
+  const channel = new ApplicationChannel({controlDir:root,initial:menu.initial,aiControls:menu,wake:()=>{},cancel:async()=>{},closeSession:async session=>{if(session)closed.push(session.sessionId)}})
   t.after(async()=>{await channel.stop();await rm(root,{recursive:true,force:true})})
   const token = randomBytes(32).toString('base64url')
   const binding = (await channel.bindings.register('private',token,owner))!
@@ -77,6 +78,7 @@ test('private scope reset and model controls preserve running sessions and share
   assert.equal((await call({action:'new',expectedSession:before.activeSessionId})).status,200)
   const next = await channel.scopeControls(binding.bindingId,'exercise')
   assert.notEqual(next.activeSessionId,before.activeSessionId)
+  assert.deepEqual(closed,[]) // a private scope runs under its binding, so no owner-authority close turn
   assert.equal(next.ai.selectedId,menu.initial.id)
   assert.equal((await call({action:'new',expectedSession:before.activeSessionId})).status,400)
   assert.equal((await control.executionSession(run.execution!)).nativeSessionId,'native-private')
@@ -89,6 +91,7 @@ test('private scope reset and model controls preserve running sessions and share
   assert.deepEqual((await control.status()).ai!.presets,presets)
   assert.equal((await call({action:'model',expectedSession:next.activeSessionId,cli:'codex',model:'fixture-model',effort:'high'})).status,200)
   const updated = await channel.submit(binding.bindingId,{requestId:'updated-model',scope:'exercise',text:'same engine'})
+  assert.deepEqual(closed,[])
   assert.equal(updated.execution!.sessionId,next.activeSessionId)
   assert.equal(updated.execution!.preset.effort,'high')
   assert.deepEqual((await new RunStore(root).get(after.id))!.execution,after.execution)

package/test/auth-profiles.test.ts

@@ -0,0 +1,187 @@
+import { ownerRun } from './helpers/owner-run.js'
+import assert from 'node:assert/strict'
+import test from 'node:test'
+import { lstat, mkdir, mkdtemp, readFile, rm, stat, writeFile } from 'node:fs/promises'
+import { spawn, spawnSync } from 'node:child_process'
+import { tmpdir } from 'node:os'
+import path from 'node:path'
+import { fileURLToPath } from 'node:url'
+import { startExecutorJob } from '../src/executor.js'
+import { ControlStore } from '../src/control-state.js'
+import { RunStore } from '../src/runs.js'
+import { serveHostExecutor } from '../src/host-executor.js'
+import { authProfileStatus, initialPreset, readModels, validateSelection, type AiPreset } from '../src/ai.js'
+
+const restoreEnv = (keys: string[]) => {
+  const prior = Object.fromEntries(keys.map(key => [key, process.env[key]]))
+  return () => { for (const [key, value] of Object.entries(prior)) value === undefined ? delete process.env[key] : process.env[key] = value }
+}
+
+test('each Claude and Codex profile runs only with its own home and credentials', {skip:process.platform==='win32'}, async t => {
+  const root = await mkdtemp(path.join(tmpdir(), 'ez-profiles-'))
+  t.after(() => rm(root, { recursive: true, force: true }))
+  t.after(restoreEnv(['PATH', 'HOME', 'EZ_ISOLATION', 'EZ_EXECUTOR_TRANSPORT']))
+  const bin = path.join(root, 'bin'), workspace = path.join(root, 'mind'), controlDir = path.join(root, 'control')
+  await mkdir(bin); await mkdir(workspace)
+  const observed = path.join(root, 'observed.json')
+  for (const cli of ['claude', 'codex']) await writeFile(path.join(bin, cli), `#!${process.execPath}
+const fs=require('fs'),p=require('path');let auth=null
+try{auth=fs.lstatSync(p.join(process.env.CODEX_HOME,'auth.json')).isSymbolicLink()?'link:'+fs.readlinkSync(p.join(process.env.CODEX_HOME,'auth.json')):'file'}catch{}
+fs.writeFileSync(${JSON.stringify(observed)},JSON.stringify({config:process.env.CLAUDE_CONFIG_DIR,storage:process.env.CLAUDE_SECURESTORAGE_CONFIG_DIR,token:process.env.CLAUDE_CODE_OAUTH_TOKEN,codexHome:process.env.CODEX_HOME,auth}))`, { mode: 0o700 })
+  process.env.PATH = bin + path.delimiter + process.env.PATH
+  process.env.HOME = root
+  delete process.env.EZ_ISOLATION; delete process.env.EZ_EXECUTOR_TRANSPORT
+  const tokens = { work: 'sk-ant-oat01-work_profile_token', personal: 'sk-ant-oat01-personal_profile_token' }
+  for (const [name, token] of Object.entries(tokens)) {
+    await mkdir(path.join(controlDir, 'cli', 'profiles', name, 'claude'), { recursive: true })
+    await writeFile(path.join(controlDir, 'cli', 'profiles', name, 'claude', 'oauth-token'), token, { mode: 0o600 })
+    await mkdir(path.join(controlDir, 'cli', 'profiles', name, 'codex'), { recursive: true })
+    await writeFile(path.join(controlDir, 'cli', 'profiles', name, 'codex', 'auth.json'), '{}', { mode: 0o600 })
+  }
+  let n = 0
+  const run = async (cli: string, authProfile?: string) => {
+    const runId = `r_profile_${n++}`
+    await ownerRun(controlDir, runId)
+    await rm(observed, { force: true })
+    const job = await startExecutorJob(['hello'], { workspace, controlDir, binDir: bin, cli, authProfile, runId, timeoutMs: 5000, repairEnabled: false })
+    assert.equal(await new Promise(resolve => job.child.once('close', resolve)), 0); await job.cleanup()
+    return JSON.parse(await readFile(observed, 'utf8'))
+  }
+  // Default bindings are unchanged: shared host login.
+  const claudeDefault = await run('claude')
+  assert.equal(claudeDefault.config, path.join(controlDir, 'cli', 'claude')); assert.equal(claudeDefault.storage, ''); assert.equal(claudeDefault.token, undefined)
+  const codexDefault = await run('codex')
+  assert.equal(codexDefault.codexHome, path.join(controlDir, 'cli', 'codex')); assert.equal(codexDefault.auth, `link:${path.join(root, '.codex', 'auth.json')}`)
+  // Named profiles use only their own store: no shared secure storage, no host link.
+  for (const [name, token] of Object.entries(tokens)) {
+    const claude = await run('claude', name)
+    assert.deepEqual(claude, { config: path.join(controlDir, 'cli', 'profiles', name, 'claude'), token, auth: null })
+    const codex = await run('codex', name)
+    assert.deepEqual(codex, { codexHome: path.join(controlDir, 'cli', 'profiles', name, 'codex'), auth: 'file' })
+  }
+  // Unprovisioned or unsupported profiles fail closed before any process starts.
+  for (const [cli, authProfile, error] of [['claude', 'missing', /not provisioned/], ['codex', 'missing', /not provisioned/], ['opencode', 'work', /Invalid auth profile/], ['claude', '../work', /Invalid auth profile/]] as const) {
+    await ownerRun(controlDir, `r_reject_${n}`); await rm(observed, { force: true })
+    await assert.rejects(startExecutorJob(['hello'], { workspace, controlDir, binDir: bin, cli, authProfile, runId: `r_reject_${n++}`, timeoutMs: 5000, repairEnabled: false }), error)
+    await assert.rejects(stat(observed))
+  }
+  assert.equal((await lstat(path.join(controlDir, 'cli', 'profiles'))).isDirectory(), true)
+  await assert.rejects(stat(path.join(controlDir, 'cli', 'profiles', 'missing')))
+})
+
+test('a conversation stays bound to its auth profile and never resumes under another', async () => {
+  const root = await mkdtemp(path.join(tmpdir(), 'ez-profile-sessions-'))
+  try {
+    const store = new ControlStore(root, 900000)
+    await store.aiState(initialPreset('claude'))
+    const work: AiPreset = { id: 'work', name: 'Claude work', cli: 'claude', authProfile: 'work', model: 'opus' }
+    const personal: AiPreset = { id: 'personal', name: 'Claude personal', cli: 'claude', authProfile: 'personal', model: 'opus' }
+    for (const preset of [work, personal]) await store.savePreset(preset)
+    const first = await store.captureChoice(initialPreset('claude'))
+    await store.markSessionStarted(first.sessionId)
+    // A profile change is a new conversation, never an in-place settings change.
+    assert.equal(await store.selectPreset('work', first.sessionId), false)
+    assert.equal(await store.selectPreset('work', first.sessionId, true), true)
+    const working = await store.captureChoice(initialPreset('claude'))
+    assert.notEqual(working.sessionId, first.sessionId)
+    assert.equal((await store.executionSession(working)).authProfile, 'work')
+    await store.markSessionStarted(working.sessionId)
+    // A choice that names another account for this session is rejected at launch.
+    await assert.rejects(store.executionSession({ sessionId: working.sessionId, preset: personal }), /matching CLI binding/)
+    await assert.rejects(store.executionSession({ sessionId: first.sessionId, preset: work }), /matching CLI binding/)
+    // Switching back restores that conversation's own account.
+    assert.equal(await store.selectPreset('personal', working.sessionId, true), true)
+    const restored = await store.switchSession(working.sessionId)
+    assert.equal(restored.authProfile, 'work')
+    assert.equal((await store.captureChoice(initialPreset('claude'))).preset.authProfile, 'work')
+  } finally { await rm(root, { recursive: true, force: true }) }
+})
+
+test('catalog offers provisioned profiles only; host refuses a request that changes the captured profile', {skip:process.platform==='win32'}, async t => {
+  const root = await mkdtemp(path.join(tmpdir(), 'ez-profile-host-'))
+  t.after(() => rm(root, { recursive: true, force: true }))
+  t.after(restoreEnv(['PATH']))
+  const workspace = path.join(root, 'mind'), controlDir = path.join(root, 'control'), bin = path.join(root, 'bin'), directory = path.join(controlDir, 'host-executor')
+  await mkdir(workspace, { recursive: true }); await mkdir(bin); await mkdir(directory, { recursive: true })
+  for (const cli of ['codex', 'claude']) await writeFile(path.join(bin, cli), '#!/bin/sh\nexit 0\n', { mode: 0o700 })
+  await mkdir(path.join(controlDir, 'cli', 'profiles', 'work', 'claude'), { recursive: true })
+  process.env.PATH = bin + path.delimiter + process.env.PATH
+  const catalog = await readModels(root, undefined, path.join(controlDir, 'cli', 'codex'), undefined, undefined, undefined, controlDir)
+  assert.deepEqual(catalog.filter(model => model.authProfile), [{ cli: 'claude', name: 'work · claude · client default', efforts: [], authProfile: 'work' }])
+  await validateSelection({ id: 'w', name: 'w', cli: 'claude', authProfile: 'work' }, catalog)
+  await assert.rejects(validateSelection({ id: 'p', name: 'p', cli: 'claude', authProfile: 'personal' }, catalog), /not provisioned/)
+
+  const runs = new RunStore(controlDir)
+  await ownerRun(controlDir, 'r_seed')
+  const execution = { sessionId: '00000000-0000-4000-8000-000000000001', preset: { id: 'work', name: 'work', cli: 'claude', authProfile: 'work' } }
+  for (const id of ['r_wrong', 'r_right', 'r_dropped']) {
+    await runs.create({ id, chatId: 101, telegramUserId: 101, texts: ['hello'], execution })
+    await runs.patch(id, { status: 'running' })
+  }
+  const abort = new AbortController(), launched: string[] = []
+  const server = serveHostExecutor({ cli: 'codex', agents: [{ name: 'test', workspace, controlDir, binDir: bin }] }, abort.signal, async (_texts, options) => {
+    launched.push(`${options.runId}:${options.authProfile}`)
+    return { child: spawn(process.execPath, ['-e', 'process.exit(0)']), cleanup: async () => {}, stdout: '' }
+  }, async () => catalog)
+  try {
+    const exit = async (id: string, options: Record<string, unknown>) => {
+      await writeFile(path.join(directory, `${id}.request.json`), JSON.stringify({ texts: ['hello'], options }))
+      for (let n = 0; n < 200; n++) {
+        const events = await readFile(path.join(directory, `${id}.events`), 'utf8').catch(() => '')
+        const code = events.match(/"stream":"exit","code":(\d+)/)?.[1]
+        if (code !== undefined) return { code: Number(code), events }
+        await new Promise(resolve => setTimeout(resolve, 20))
+      }
+      throw new Error('host request did not finish')
+    }
+    for (const [id, options] of [['r_wrong', { authProfile: 'personal' }], ['r_dropped', {}]] as const) {
+      const result = await exit(id, { cli: 'claude', sessionId: execution.sessionId, ...options })
+      assert.equal(result.code, 1); assert.match(result.events, /does not match its captured AI selection/)
+    }
+    assert.equal((await exit('r_right', { cli: 'claude', authProfile: 'work', sessionId: execution.sessionId })).code, 0)
+    assert.deepEqual(launched, ['r_right:work'])
+  } finally { abort.abort(); await server }
+})
+
+test('profile setup and diagnostics never print credentials or account identity', {skip:process.platform==='win32'}, async () => {
+  const root = await mkdtemp(path.join(tmpdir(), 'ez-profile-cli-'))
+  try {
+    const controlDir = path.join(root, 'control'), bin = path.join(root, 'bin')
+    await mkdir(bin)
+    await writeFile(path.join(bin, 'claude'), `#!/bin/sh
+if [ "$1" = auth ]; then
+  if [ -n "$CLAUDE_CODE_OAUTH_TOKEN" ]; then echo '{"loggedIn":true,"authMethod":"oauth_token","email":"owner@example.com","orgId":"org-secret"}'; exit 0; fi
+  echo '{"loggedIn":false,"authMethod":"none"}'; exit 1
+fi
+exit 0
+`, { mode: 0o700 })
+    await writeFile(path.join(bin, 'codex'), '#!/bin/sh\n[ -f "$CODEX_HOME/auth.json" ] && { echo "Logged in using ChatGPT (owner@example.com)" >&2; exit 0; }\necho "Not logged in"; exit 1\n', { mode: 0o700 })
+    const store = new ControlStore(controlDir, 900000); await store.aiState(initialPreset('claude'))
+    const env = { ...process.env, HOME: root, PATH: bin + path.delimiter + process.env.PATH, EZ_CONTROL_DIR: controlDir }
+    const ai = (...args: string[]) => spawnSync(process.execPath, [fileURLToPath(new URL('../bin/ezenciel-agents-ai.mjs', import.meta.url)), ...args], { env, encoding: 'utf8' })
+    for (const [cli, name] of [['claude', 'work'], ['claude', 'personal'], ['codex', 'work']]) {
+      const added = ai('profile', 'add', '--cli', cli, '--auth-profile', name)
+      assert.equal(added.status, 0, added.stderr)
+      const home = path.join(controlDir, 'cli', 'profiles', name, cli)
+      assert.equal(JSON.parse(added.stdout).home, home)
+      assert.equal((await stat(home)).mode & 0o777, 0o700)
+    }
+    assert.notEqual(ai('profile', 'add', '--cli', 'opencode', '--auth-profile', 'work').status, 0)
+    await writeFile(path.join(controlDir, 'cli', 'profiles', 'work', 'claude', 'oauth-token'), 'sk-ant-oat01-never_printed_token', { mode: 0o600 })
+    await writeFile(path.join(controlDir, 'cli', 'profiles', 'work', 'codex', 'auth.json'), '{"tokens":"never-printed"}', { mode: 0o600 })
+    const profiles = ai('profiles')
+    assert.equal(profiles.status, 0, profiles.stderr)
+    for (const secret of ['never_printed', 'never-printed', 'owner@example.com', 'org-secret']) assert.equal(profiles.stdout.includes(secret), false)
+    const byKey = Object.fromEntries((JSON.parse(profiles.stdout) as Awaited<ReturnType<typeof authProfileStatus>>).map(item => [`${item.cli}@${item.authProfile}`, item]))
+    assert.deepEqual(byKey['claude@work'], { cli: 'claude', authProfile: 'work', provisioned: true, sharesHostLogin: false, credentialFiles: ['oauth-token'], loggedIn: true, method: 'oauth_token' })
+    assert.equal(byKey['claude@personal'].loggedIn, false)
+    assert.deepEqual(byKey['codex@work'], { cli: 'codex', authProfile: 'work', provisioned: true, sharesHostLogin: false, credentialFiles: ['auth.json'], loggedIn: true, method: 'chatgpt' })
+    assert.equal(byKey['claude@null'].sharesHostLogin, true)
+    // Selecting a provisioned profile binds a fresh conversation to it.
+    const selected = ai('select', '--cli', 'claude', '--auth-profile', 'work')
+    assert.equal(selected.status, 0, selected.stderr)
+    assert.equal((await store.status()).activeSession?.authProfile, 'work')
+    assert.notEqual(ai('select', '--cli', 'claude', '--auth-profile', 'unknown').status, 0)
+    assert.equal((await store.status()).activeSession?.authProfile, 'work')
+  } finally { await rm(root, { recursive: true, force: true }) }
+})

package/test/intake-relay.test.ts

@@ -1,6 +1,6 @@
 import test from 'node:test'
 import assert from 'node:assert/strict'
-import { randomBytes } from 'node:crypto'
+import { randomBytes, randomUUID } from 'node:crypto'
 import { mkdtemp, readdir, rm } from 'node:fs/promises'
 import { tmpdir } from 'node:os'
 import { join } from 'node:path'
@@ -17,6 +17,8 @@ import { atomicTaskFile, Tasks } from '../src/tasks.js'
 import { EventSources } from '../src/event-sources.js'
 import { ownerRun } from './helpers/owner-run.js'
 import { packageVersion } from '../src/version.js'
+import { SESSION_CLOSE_PROMPT } from '../src/session-close.js'
+import type { ExecutorOptions } from '../src/executor.js'
 import type { Config } from '../src/config.js'
 
 const message = (id: number, text = 'hello'): Update => ({
@@ -32,6 +34,7 @@ const message = (id: number, text = 'hello'): Update => ({
 const fixture = async (overrides: Partial<Config> = {}) => {
   const dir = await mkdtemp(join(tmpdir(), 'ez-intake-relay-'))
   const launched: string[][] = []
+  const launchOptions: ExecutorOptions[] = []
   const replies: string[] = []
   const members = new Map<number, string>()
   const keyboards: { text: string; callback_data: string }[][][] = []
@@ -47,8 +50,9 @@ const fixture = async (overrides: Partial<Config> = {}) => {
     ...overrides,
   }
   const make = () => {
-    const relay = createRelay(config, async (texts) => {
+    const relay = createRelay(config, async (texts, options) => {
       launched.push(texts)
+      launchOptions.push(options)
       const child = spawn(process.execPath, ['-e', 'setTimeout(() => {}, 50)'])
       children.push(child)
       await once(child, 'spawn')
@@ -79,6 +83,7 @@ const fixture = async (overrides: Partial<Config> = {}) => {
   return {
     dir,
     launched,
+    launchOptions,
     replies,
     members,
     keyboards,
@@ -336,6 +341,35 @@ test('new conversation leaves already accepted messages pinned to the old conver
   } finally { await f.close() }
 })
 
+test('new conversation gives the started one a single silent close turn in its own native session', async () => {
+  const f = await fixture()
+  try {
+    const runs = new RunStore(f.dir), store = new ControlStore(f.dir, 1000)
+    const settled = async () => {
+      for (let i = 0; i < 200 && (await runs.list()).some((r) => ['queued', 'running'].includes(r.status)); i++)
+        await new Promise((resolve) => setTimeout(resolve, 10))
+    }
+    await f.relay.bot.handleUpdate(message(1, '/new'))
+    assert.equal(f.launched.length, 0) // an unstarted conversation has nothing to keep
+    await f.relay.bot.handleUpdate(message(2, 'My sister is called Ana'))
+    await f.relay.drainInbox(true)
+    await settled()
+    const first = (await store.getActiveSession())!
+    assert.equal(first.hasStarted, true)
+    await f.relay.bot.handleUpdate(message(3, '/new'))
+    await settled()
+    assert.deepEqual(f.launched.at(-1), [SESSION_CLOSE_PROMPT])
+    const close = (await runs.list()).find((r) => r.sessionClose)!
+    assert.equal(close.execution!.sessionId, first.sessionId)
+    assert.equal(close.chatId, undefined); assert.equal(close.messageId, undefined)
+    const options = f.launchOptions.at(-1)!
+    assert.equal(options.isResume, true)
+    assert.equal(options.promptSuffix, '') // no reply mechanism: nothing is delivered
+    assert.notEqual((await store.getActiveSession())!.sessionId, first.sessionId)
+    assert.equal((await runs.list()).filter((r) => r.sessionClose).length, 1)
+  } finally { await f.close() }
+})
+
 test('accepted DM survives relay restart and duplicate delivery creates exactly one run', async () => {
   const f = await fixture()
   try {
@@ -752,3 +786,30 @@ test('tasks command never reaches the executor on invalid input or foreign sende
     assert.equal(f.launched.length, 0)
   } finally { await f.close() }
 })
+
+test('tasks list and detail show saved frequency and ordered setup path without changing or launching work',async()=>{
+  const f=await fixture()
+  try{
+    const {Scheduler}=await import('../src/scheduler.js')
+    const control=new ControlStore(f.dir,1000),scheduler=new Scheduler(f.dir)
+    const preset={id:'primary',name:'Primary',cli:'claude',model:'opus',effort:'high'}
+    const fallbacks=[
+      {...preset,id:'second',authProfile:'claude2'},
+      {id:'third',name:'Third',cli:'codex',authProfile:'work',model:'gpt-6-sol',effort:'xhigh'},
+      {id:'fourth',name:'Fourth',cli:'pi',model:'anthropic/claude-sonnet',effort:'medium'}]
+    const saved=await scheduler.save({id:'analyst',name:'Analyst',text:'Read the saved role.',owner:(await control.status()).owner!,
+      execution:{sessionId:randomUUID(),preset,fallbacks},
+      enabled:true,trigger:{cron:'*/30 * * * *',timezone:'America/New_York',start:'2027-01-01T00:00:00Z'}})
+    const path='claude (default login) · opus · high → claude@claude2 · opus · high → codex@work · gpt-6-sol · xhigh → pi · anthropic/claude-sonnet · medium'
+    await f.relay.bot.handleUpdate(message(1,'/tasks'))
+    const list=f.replies.at(-1)!
+    assert.ok(list.includes('Schedule · Every 30 minutes · America/New_York'))
+    assert.ok(list.includes(`Execution · ${path}`))
+    await f.relay.bot.handleUpdate(message(2,'/tasks 1'))
+    const detail=f.replies.at(-1)!
+    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)
+    assert.equal(f.launched.length,0)
+  }finally{await f.close()}
+})

package/test/model-policy.test.ts

@@ -17,7 +17,7 @@ test('engine defaults stay omitted and explicit native settings survive every ad
   const preset={id:'chosen',name:'Chosen',cli,model:'native-model',effort:'ultra'}
   assert.deepEqual(executionDefaults(cli,preset),preset)
   await validateSelection(preset,[{cli,model:'native-model',name:'Native',efforts:['ultra']}],async()=>true)
-  await assert.rejects(validateSelection({...preset,effort:'unsupported'},[{cli,model:'native-model',name:'Native',efforts:['ultra']}],async()=>true),/installed client catalog/)
+  await assert.rejects(validateSelection({...preset,effort:'unsupported'},[{cli,model:'native-model',name:'Native',efforts:['ultra']}],async()=>true),/Effort unsupported is not supported.*Supported efforts: ultra/)
   assert.throws(()=>executionDefaults(cli,{effort:'bad option'}),/Invalid reasoning effort/)
  }
  assert.deepEqual(executionOverrides('codex',{model:'old',effort:'max'},'new'),{model:'new',effort:undefined})

package/test/plugin-broker.test.mjs

@@ -103,7 +103,8 @@ test('isolated broker authenticates an owned run, pins plugin revision, and writ
     assert.deepEqual(JSON.parse(listed.stdout), { sample: 'sample' });
     const detailed = await request(f.socket, { version: 1, id: randomUUID(), operation: 'manager', runId: f.runId, args: ['tools', 'list', '--details'] });
     assert.equal(detailed.ok, true);
-    assert.deepEqual(JSON.parse(detailed.stdout).sample.commands, ['ez sample --help']);
+    assert.deepEqual(JSON.parse(detailed.stdout).sample.aliases, ['sample']);
+    assert.equal(JSON.parse(detailed.stdout).sample.example, 'ez sample --help');
     assert.deepEqual(JSON.parse(detailed.stdout).sample.skillReads, [['ez', 'tools', 'skill', 'sample', 'SKILL.md']]);
     const skill = await request(f.socket, { version: 1, id: randomUUID(), operation: 'manager', runId: f.runId, args: ['tools', 'skill', 'sample', 'SKILL.md'] });
     assert.equal(skill.code, 0);
@@ -209,6 +210,9 @@ test('scheduled preflight uses authoritative configuration and rejects writes or
     registry.plugins.sample.manifest.commands.sample.exposure={receivesExternalContent:true,changesRecords:false,sendsExternally:false,requiresReview:false};
     await fs.writeFile(file,JSON.stringify(registry));
     const accepted=await request(f.socket,payload);assert.equal(accepted.ok,true);assert.equal(accepted.result.eligible,true);
+    await fs.writeFile(path.join(f.fake,'docker'), `#!${process.execPath}\nprocess.stdout.write('SECRET-token-xyz');\n`, {mode:0o700});
+    const invalid=await request(f.socket,payload);assert.equal(invalid.ok,false);
+    assert.equal(invalid.error,'Scheduled plugin preflight returned invalid output');assert(!JSON.stringify(invalid).includes('SECRET-token-xyz'));
     await scheduler.save({...schedule,enabled:false});
     assert.equal((await request(f.socket,payload)).ok,false);
   } finally {abort.abort();await serving;process.env.PATH=previousPath;}
@@ -246,3 +250,49 @@ test('task launch broker admits only its fixed application command and writes no
     const revoked=await request(f.socket,{version:1,id:randomUUID(),operation:'task-launch',runId:taskRun,stdin:JSON.stringify(input)});assert.equal(revoked.ok,false);
   }finally{abort.abort();await serving;process.env.PATH=oldPath;await ledger.stop();provider.closeAllConnections();await new Promise(r=>provider.close(r));}
 });
+
+// Native engines and Node scripts wire child stdin as a socket they may never
+// close. The isolated client must reach the broker instead of waiting for EOF,
+// while pipes and files still deliver their complete input.
+test('isolated client does not wait forever on an inherited open stdin socket', async t => {
+  const { createServer: createSocketServer } = await import('node:net');
+  const { spawn } = await import('node:child_process');
+  const { once } = await import('node:events');
+  const root = await fs.realpath(await fs.mkdtemp(path.join(tmpdir(), 'e-')));
+  t.after(() => fs.rm(root, { recursive: true, force: true }));
+  const socket = path.join(root, 'b.sock'), invokes = [];
+  const broker = createSocketServer(connection => {
+    let buffer = '';
+    connection.on('data', chunk => {
+      buffer += chunk;
+      if (!buffer.includes('\n')) return;
+      const frame = JSON.parse(buffer);
+      if (frame.operation === 'invoke') invokes.push(frame);
+      connection.end(JSON.stringify(frame.operation === 'resolve' ? { ok: true, revision, capability: 'c'.repeat(64) } : { ok: true, code: 0, stdout: 'ok' }) + '\n');
+    });
+  });
+  await new Promise(resolve => broker.listen(socket, resolve));
+  t.after(() => broker.close());
+  const client = new URL('../bin/ez', import.meta.url).pathname;
+  const env = { PATH: process.env.PATH, EZ_PLUGIN_BROKER_SOCKET: socket, EZ_RUN_ID: 'r_client' };
+  const finish = async child => {
+    let stdout = '';
+    child.stdout.on('data', chunk => stdout += chunk);
+    const timer = setTimeout(() => child.kill('SIGKILL'), 10_000);
+    const [code, signal] = await once(child, 'close');
+    clearTimeout(timer);
+    return { code, signal, stdout };
+  };
+  const open = spawn(process.execPath, [client, 'sample', 'read'], { env });
+  t.after(() => open.kill('SIGKILL'));
+  assert.deepEqual(await finish(open), { code: 0, signal: null, stdout: 'ok' });
+  assert.deepEqual(invokes.at(-1).args, ['read']);
+  assert.equal(invokes.at(-1).stdin, undefined);
+  const supplied = spawn(process.execPath, [client, 'sample', 'write'], { env });
+  supplied.stdin.end('payload');
+  assert.equal((await finish(supplied)).code, 0);
+  assert.equal(invokes.at(-1).stdin, 'payload');
+  const piped = spawn('/bin/sh', ['-c', '(sleep 1.5; printf late) | "$0" "$1" sample pipe', process.execPath, client], { env, stdio: ['ignore', 'pipe', 'inherit'] });
+  assert.equal((await finish(piped)).code, 0);
+  assert.equal(invokes.at(-1).stdin, 'late');
+});