@givolt/i3-composer-agent 3.2.1 → 4.0.0

package/hooks/manage.cjs

@@ -1,169 +0,0 @@
-#!/usr/bin/env node
-'use strict';
-// CodeBuddy settings protocol: https://www.codebuddy.cn/docs/cli/hooks
-// IDE override semantics: https://www.codebuddy.cn/docs/ide/Features/Hooks
-const fs=require('node:fs'),path=require('node:path'),os=require('node:os'),crypto=require('node:crypto'),assert=require('node:assert/strict');
-const OWNER='i3-composer-agent-codebuddy-v1';
-const catalog=[
- {id:'session-start',name:'Skill 发现与路由',origin:'upstream',files:['session-start.sh'],events:['SessionStart'],dependencies:['bash','jq'],condition:'明确 skill-routing=hook；原生路由使用 native，避免重复注入'},
- {id:'sdd-cache',name:'官方文档请求缓存',origin:'upstream',files:['sdd-cache-pre.sh','sdd-cache-post.sh'],events:['PreToolUse','PostToolUse'],dependencies:['bash','jq','curl','sha256'],condition:'实际 WebFetch；仅源站 304 复用'},
- {id:'simplify-ignore',name:'代码块简化保护',origin:'upstream',files:['simplify-ignore.sh'],events:['PreToolUse','PostToolUse','Stop'],dependencies:['bash','jq','sha1'],condition:'使用 simplify-ignore 标记；会在运行期间替换标记块并在 Stop 恢复'},
-];
-const digest=value=>crypto.createHash('sha256').update(JSON.stringify(value)).digest('hex');
-const sha=bytes=>crypto.createHash('sha256').update(bytes).digest('hex');
-const json=value=>JSON.stringify(value,null,2)+'\n';
-function selection(value,action='check') {
- if(action==='enable')assert.ok(value!==undefined,'Choose explicit Hook groups before enabling; no groups are enabled by default');
- const ids=value===undefined?(['check','list'].includes(action)?catalog.map(x=>x.id):action==='disable'?['prototype']:[]):value==='all'?catalog.map(x=>x.id):value.split(',');
- assert.ok(ids.every(id=>id==='prototype'||catalog.some(item=>item.id===id)),'Unknown Hook group; choose session-start,sdd-cache,simplify-ignore or all');
- assert.ok(action!=='enable'||!ids.includes('prototype'),'Prototype audit Hooks are retired and cannot be enabled');
- return [...new Set(ids)];
-}
-function quote(value){assert.ok(typeof value==='string'&&!/[\x00\r\n]/.test(value),'Unsafe Hook argument');return "'"+value.replaceAll("'","'\\''")+"'";}
-function config(raw){const value=raw===null?{}:JSON.parse(raw);assert.ok(value&&typeof value==='object'&&!Array.isArray(value),'Settings must be an object');assert.ok(value.hooks===undefined||value.hooks&&typeof value.hooks==='object'&&!Array.isArray(value.hooks),'Invalid hooks configuration');for(const groups of Object.values(value.hooks||{})){assert.ok(Array.isArray(groups),'Invalid Hook groups');for(const group of groups)assert.ok(group&&Array.isArray(group.hooks),'Invalid Hook group');}return value;}
-function modules() {
- const roots=[path.resolve(__dirname,'..'),path.resolve(__dirname,'../..')];
- const candidates=roots.flatMap(root=>['i3-ui-prototype',['ui','prototype'].join('-')].map(id=>path.join(root,'skills',id,'scripts')));
- const scripts=candidates.find(dir=>fs.existsSync(path.join(dir,'prototype-hooks.cjs')));assert.ok(scripts,'Prototype audit implementation is missing; install the complete package');
- return {scripts,R:require(path.join(scripts,'codebuddy-registration.cjs')),P:require(path.join(scripts,'prototype-hooks.cjs'))};
-}
-function owned(group){return group.hooks.some(h=>typeof h.command==='string'&&h.command.includes(OWNER));}
-function validateOwned(settings,record,base,scope) {
- if(record){assert.equal(record.owner,OWNER,'Unknown Hook ownership');assert.equal(record.schemaVersion,1,'Unsupported Hook ownership');assert.equal(record.base,base,'Hook base differs');assert.equal(record.scope,scope,'Hook scope differs');assert.ok(record.groups&&typeof record.groups==='object'&&!Array.isArray(record.groups),'Invalid owned Hook groups');
-  for(const [id,item] of Object.entries(record.groups)){const known=catalog.find(x=>x.id===id&&x.origin==='upstream');assert.ok(known&&item&&item.groups,'Unknown owned Hook group');assert.deepEqual(Object.keys(item.groups).sort(),[...known.events].sort(),'Owned Hook events changed');}
- }
- const found=new Set();
- for(const [event,groups] of Object.entries(settings.hooks||{}))for(const group of groups.filter(owned)) {
-  const ids=Object.keys(record?.groups||{}).filter(id=>record.groups[id].groups[event]&&group.hooks.some(h=>h.command?.includes(`--group' '${id}'`)));
-  assert.equal(ids.length,1,'Hook ownership is missing; preserve the unknown command');const key=ids[0]+':'+event;
-  assert.ok(!found.has(key),'Duplicate owned Hook');assert.deepEqual(group,record.groups[ids[0]].groups[event],'Owned Hook was modified; preserve user changes');found.add(key);
- }
- for(const [id,item] of Object.entries(record?.groups||{}))for(const event of Object.keys(item.groups))assert.ok(found.has(id+':'+event),'Owned Hook registration is missing');
-}
-function requirements(id,deps) {
- for(const name of catalog.find(x=>x.id===id).dependencies){if(name==='sha256')assert.ok(deps.shasum||deps.sha256sum,'Missing shasum or sha256sum');else if(name==='sha1')assert.ok(deps.shasum||deps.sha1sum,'Missing shasum or sha1sum');else assert.ok(deps[name],`Missing ${name}; install it separately before enabling ${id}`);}
-}
-function planUpstream(options) {
- const {base,scope,product,action,record=null,settingsText=null,nodePath,hookDirectory,scriptDigest,dependencies={},skillRouting}=options;
- assert.ok(['keep','enable','disable'].includes(action),'Invalid Hook action');assert.ok(['project','global'].includes(scope),'Specify project or global scope');
- const settings=config(settingsText);validateOwned(settings,record,base,scope);
- const chosen=selection(options.groups,action).filter(id=>id!=='prototype'),enabled=new Set(Object.keys(record?.groups||{}));
- if(action==='enable')for(const id of chosen){if(id==='session-start')assert.equal(skillRouting,'hook','session-start requires explicit skill-routing=hook; native routing must not be duplicated');requirements(id,dependencies);enabled.add(id);}
- if(action==='disable')for(const id of chosen)enabled.delete(id);
- if(enabled.size){assert.ok(['ide','cli'].includes(product),'Specify CodeBuddy product ide or cli');if(record)assert.equal(product,record.product,'Disable the previous product profile before changing it');assert.match(scriptDigest,/^[a-f0-9]{64}$/);for(const p of [base,nodePath,hookDirectory])assert.ok(path.isAbsolute(p),'Absolute registration paths required');}
- // Remove only exact owned groups after validation. All user Hook groups survive.
- for(const event of Object.keys(settings.hooks||{}))settings.hooks[event]=settings.hooks[event].filter(g=>!owned(g));
- const groups={};
- for(const item of catalog.filter(x=>enabled.has(x.id))) {
-  const entry={product,scriptDigest,nodePath,bashPath:dependencies.bash||record?.groups[item.id]?.bashPath,skillRouting:item.id==='session-start'?'hook':null,groups:{}};
-  for(const event of item.events) {
-   const matcher=item.id==='sdd-cache'?'^WebFetch$':event==='PreToolUse'?'^Read$':'^(Edit|Write)$';
-   const argv=[nodePath,path.join(hookDirectory,'codebuddy-upstream.cjs'),'--registration',OWNER,'--group',item.id,'--base',base,'--scope',scope,'--product',product,'--event',event];
-   const group={...(event.endsWith('ToolUse')?{matcher}:{}),hooks:[{type:'command',command:argv.map(quote).join(' '),timeout:item.id==='sdd-cache'?15:20}]};
-   entry.groups[event]=group;settings.hooks||={};(settings.hooks[event]||=[]).push(group);
-  }
-  groups[item.id]=entry;
- }
- const recordAfter=enabled.size?{schemaVersion:1,owner:OWNER,base,scope,product,productVersion:options.productVersion||record?.productVersion||'unreported',groups}:null;
- const afterText=!record&&!chosen.length||!record&&action!=='enable'?settingsText:json(settings);
- return {afterText,recordAfter};
-}
-function apply(change,io=modules().R.fileIO) {
- const unlock=io.lock?io.lock(change.changes[0].file):()=>{},written=[];
- try {
-  for(const entry of change.changes)assert.equal(digest(io.read(entry.file)),digest(entry.before),'Hook configuration changed after planning');
-  try {for(const entry of change.changes)if(entry.before!==entry.after){io.write(entry.file,entry.after,digest(entry.before));written.push(entry);}}
-  catch(error){for(const entry of written.reverse())if(io.read(entry.file)===entry.after)io.write(entry.file,entry.before,digest(entry.after));throw error;}
-  return change.summary;
- }finally{unlock();}
-}
-function executable(name,env=process.env) {
- for(const folder of (env.PATH||'').split(path.delimiter)){const file=path.resolve(folder,name);try{if(fs.statSync(file).isFile()){fs.accessSync(file,fs.constants.X_OK);return file;}}catch{}}
- return null;
-}
-function dependencies(){return Object.fromEntries(['bash','jq','curl','shasum','sha256sum','sha1sum'].map(name=>[name,executable(name)]));}
-function location(options) {
- assert.ok(['codebuddy','workbuddy'].includes(options.host||'codebuddy'),'Hook management supports CodeBuddy and WorkBuddy');assert.ok(['project','global'].includes(options.scope),'Specify project or global scope');
- const project=fs.realpathSync(path.resolve(options.project||'.'));assert.ok(fs.statSync(project).isDirectory(),'Project must be a directory');
- return {project,base:options.scope==='project'?path.join(project,'.codebuddy'):modules().P.globalBase(options)};
-}
-function unmanaged(settings) {
- const files={ 'session-start':['session-start.sh'],'sdd-cache':['sdd-cache-pre.sh','sdd-cache-post.sh'],'simplify-ignore':['simplify-ignore.sh'] },result=[];
- for(const [event,groups] of Object.entries(settings.hooks||{}))for(const group of groups)for(const hook of group.hooks) {
-  if(typeof hook.command!=='string'||hook.command.includes(OWNER))continue;
-  for(const [id,names] of Object.entries(files))if(names.some(name=>hook.command.includes(name)))result.push({id,event,command:hook.command});
- }
- return result;
-}
-function pendingSimplify(base) {
- const parent=path.join(base,'cache/i3-composer-agent/hooks');if(!fs.existsSync(parent))return [];
- return fs.readdirSync(parent,{withFileTypes:true}).filter(entry=>entry.isDirectory()&&!entry.isSymbolicLink()).flatMap(entry=>{
-  const dir=path.join(parent,entry.name,'simplify-ignore');if(!fs.existsSync(dir))return [];
-  assert.ok(!fs.lstatSync(dir).isSymbolicLink(),'Simplify cache symlink requires review');return fs.readdirSync(dir).filter(file=>file.endsWith('.bak')).map(file=>path.join(dir,file));
- });
-}
-function prepare(options,runtime={}) {
- const {P,R}=runtime.modules||modules(),{project,base}=runtime.location||location(options),io=R.fileIO,requested=options.action||'keep',ids=selection(options.groups,requested),host=options.host||'codebuddy';
- const action=requested==='default'?'keep':requested;
- const settingsPath=path.join(base,'settings.json'),recordPath=path.join(base,'state/i3-composer-agent/hooks.json'),prototypePath=path.join(base,'state/i3-prototype/installation.json');
- const before=io.read(settingsPath),recordText=io.read(recordPath),record=recordText?JSON.parse(recordText):null,prototypeText=io.read(prototypePath);
- const settings=config(before);
- if(action==='enable') {
-  assert.ok(process.platform!=='win32','Buddy Hook quoting on Windows has not been validated');
-  assert.ok(!settings.disableAllHooks&&!settings.allowManagedHooksOnly,'Host policy prevents automatic Hooks; preserve the policy');
-  assert.equal(host,'codebuddy','WorkBuddy upstream Hook adapters are not supported');
-  for(const id of ids)assert.ok(!unmanaged(settings).some(item=>item.id===id),`Unmanaged ${id} Hook already exists; preserve it and resolve the duplicate before enabling`);
- }
- if(ids.includes('simplify-ignore')&&['enable','disable'].includes(action)) {
-  for(const candidate of [...new Set([base,path.join(project,'.codebuddy'),P.globalBase?P.globalBase(options):path.join(options.home||os.homedir(),'.codebuddy')])])assert.equal((runtime.pendingSimplify||pendingSimplify)(candidate).length,0,'Simplify protection is still active; finish the session and restore blocks before changing Hooks');
- }
- // Retire the entire prototype group, including shared Buddy registrations.
- // The legacy planner validates exact ownership; unknown/user-modified entries stop here.
- const prototypeChange=P.prepare({...options,project,action:'disable',hosts:[],entryPath:path.join(base,'i3-composer-agent/hooks/i3-prototype.cjs')});
- const upstream=planUpstream({...options,base,product:record?.product||options.product,action,record,settingsText:prototypeChange.unchanged?before:prototypeChange.afterText,nodePath:process.execPath,
-  hookDirectory:path.join(base,'i3-composer-agent/hooks'),scriptDigest:runtime.scriptDigest||require('./codebuddy-upstream.cjs').bundle(),dependencies:runtime.dependencies||dependencies()});
- const changes=[{file:settingsPath,before,after:upstream.afterText},{file:recordPath,before:recordText,after:upstream.recordAfter?json(upstream.recordAfter):null}];
- if(prototypeText&&!prototypeChange.unchanged) {
-  const archive=path.join(base,'state/i3-prototype/history','retired-'+sha(prototypeText)+'.json'),previous=io.read(archive);
-  assert.ok(previous===null||previous===prototypeText,'Retired registration history differs; preserve it for review');
-  changes.push({file:archive,before:previous,after:prototypeText});
- }
- changes.push({file:prototypePath,before:prototypeText,after:prototypeChange.unchanged?prototypeText:null});
- return {changes,summary:{host,scope:options.scope,base,product:record?.product||options.product||'auto',action,requestedAction:requested,groups:ids.filter(id=>id!=='prototype'),enabledHosts:[],
-  changed:changes.some(item=>item.before!==item.after),enabledGroups:Object.keys(upstream.recordAfter?.groups||{}),retiredGroups:['prototype'],prototypeRegistrationRemoved:!!prototypeText&&!prototypeChange.unchanged,
-  next:'Prototype audit Hooks are retired. Existing owned callbacks are removed; other Hook groups and user settings are preserved. Reload an affected host after cleanup.'}};
-}
-function check(options) {
- const {P,R}=modules(),{project,base}=location(options),io=R.fileIO,ids=selection(options.groups,options.action||'check'),settingsText=io.read(path.join(base,'settings.json')),settings=config(settingsText),deps=dependencies(),items=[];
- let record=null,error=null;try{record=require('./codebuddy-upstream.cjs').registration(base);}catch(e){error=e.message;}
- for(const id of ids) {
-  if(id==='prototype')continue;
-  const item=catalog.find(x=>x.id===id),entry=path.join(base,'i3-composer-agent/hooks','codebuddy-upstream.cjs');
-  const missingFiles=[entry,...item.files.map(file=>path.join(base,'i3-composer-agent/hooks',file))].filter(file=>!fs.existsSync(file));
-  const row={...item,entryPath:entry,filesPresent:missingFiles.length===0,missingFiles,status:missingFiles.length?'not-installed':'files-present',invocationVerified:false,nativeValidation:'pending-user'};
-  try{requirements(id,deps);}catch(e){row.dependencyIssue=e.message;}
-  if(error){row.status='failed';row.reason=error;}
-  else if(record?.groups[id])try {
-   const U=require('./codebuddy-upstream.cjs'),selected=U.effective({project,product:record.product,group:id,home:options.home,env:options.env});row.effective=selected?.base===base;
-   row.status=!row.effective?'shadowed':record.product==='cli'?'pending-host-review':'registered';
-   const raw=io.read(path.join(base,'state/i3-composer-agent/hook-observations',U.projectKey(project),id+'.json'));
-   if(raw){const observation=JSON.parse(raw);if(observation.settingsDigest===record.settingsDigest&&observation.scriptDigest===record.groups[id].scriptDigest){row.observation=observation;row.status=row.effective?'triggered':row.status;}}
-   row.product=record.product;row.productVersion=record.productVersion;
-  }catch(e){row.status='failed';row.reason=e.message;}
-  items.push(row);
- }
- const retired=P.check({...options,project});
- return {host:options.host||'codebuddy',scope:options.scope,base,readOnly:true,changed:false,status:items.some(item=>item.status==='failed')||retired.status==='failed'?'failed':'checked',hooks:items,retiredHooks:[{id:'prototype',...retired}],unmanagedHooks:unmanaged(settings),
-  next:'files-present is not enabled; registered is not triggered; triggered is not business acceptance. CLI: review /hooks; IDE: reload.'};
-}
-function configure(options){const change=prepare(options);return options.preview?change.summary:apply(change);}
-function main(argv=process.argv.slice(2)) {
- if(argv.includes('--help'))return {usage:'node manage.cjs list|check|enable|disable --host codebuddy|workbuddy --scope project|global --project <directory> [--groups <ids>|all] [--product auto|ide|cli] [--skill-routing hook] [--transcript-root <native-directory>|auto]',hooks:catalog,default:'No automatic Hook registration. Enable requires explicit upstream groups; prototype audit is retired and its old registrations are cleaned during upgrades',nativeValidation:'pending-user'};
- const options={action:argv.shift()||'list',host:'codebuddy',project:process.cwd()};
- assert.ok(['list','check','enable','disable','keep'].includes(options.action),'Choose list, check, enable or disable');
- while(argv.length){const arg=argv.shift();assert.ok(arg.startsWith('--')&&argv[0]&&!argv[0].startsWith('--'),'Hook option needs a value');const key=arg.slice(2).replace(/-([a-z])/g,(_,c)=>c.toUpperCase());assert.ok(['host','scope','project','product','productVersion','transcriptRoot','groups','skillRouting'].includes(key),'Unknown Hook option');options[key]=argv.shift();}
- if(options.skillRouting!==undefined)assert.ok(['native','hook'].includes(options.skillRouting),'skill-routing must be native or hook');
- return ['list','check'].includes(options.action)?check(options):configure(options);
-}
-module.exports={OWNER,catalog,digest,sha,json,quote,config,selection,modules,validateOwned,requirements,planUpstream,apply,dependencies,location,unmanaged,pendingSimplify,prepare,check,configure,main};
-if(require.main===module)try{const result=main();process.stdout.write(json(result));if(result.status==='failed')process.exitCode=1;}catch(error){process.stderr.write(error.message+'\n');process.exitCode=1;}

package/scripts/design-provider-discovery-test.js

@@ -1,190 +0,0 @@
-'use strict';
-const test = require('node:test');
-const assert = require('node:assert/strict');
-const fs = require('node:fs');
-const os = require('node:os');
-const path = require('node:path');
-const childProcess = require('node:child_process');
-const installationFile = path.join(__dirname, 'lib/installation.js');
-const skillsDirectory = fs.existsSync(installationFile) ? require(installationFile).loadInstallation().skillsDirectory : path.resolve(__dirname, '../skills');
-const moduleFile = path.join(skillsDirectory, 'i3-ui-prototype/scripts/design-provider-discovery.cjs');
-
-function fixture(t) {
-  const root = fs.mkdtempSync(path.join(os.tmpdir(), 'i3-provider-discovery-'));
-  t.after(() => fs.rmSync(root, { recursive: true, force: true }));
-  const home = path.join(root, 'home'), project = path.join(root, 'project');
-  fs.mkdirSync(home); fs.mkdirSync(project);
-  return { root, home, project, env: {} };
-}
-function write(file, value) { fs.mkdirSync(path.dirname(file), { recursive: true }); fs.writeFileSync(file, typeof value === 'string' ? value : JSON.stringify(value)); }
-function skill(root) { write(path.join(root, 'SKILL.md'), '# Fixture UI/UX'); return root; }
-function discover(options, run = () => { throw new Error('Unexpected host CLI call'); }) {
-  const original = childProcess.spawnSync;
-  childProcess.spawnSync = run;
-  delete require.cache[require.resolve(moduleFile)];
-  let api;
-  try { api = require(moduleFile); } finally { childProcess.spawnSync = original; }
-  return api.discoverProviderCandidates(options);
-}
-function codexPlugin(f, overrides = {}) {
-  const record = { pluginId: 'ui-ux-pro-max@ui-ux-pro-max-skill', name: 'ui-ux-pro-max', marketplaceName: 'ui-ux-pro-max-skill', version: '2.15.0', installed: true, enabled: true, ...overrides };
-  const cache = path.join(f.env.CODEX_HOME || path.join(f.home, '.codex'), 'plugins/cache/ui-ux-pro-max-skill/ui-ux-pro-max', record.version);
-  if (!record.version.includes('..')) {
-    write(path.join(cache, '.claude-plugin/plugin.json'), { name: 'ui-ux-pro-max', version: record.version, skills: './.claude/skills/' });
-    skill(path.join(cache, '.claude/skills/ui-ux-pro-max'));
-  }
-  return { record, cache, run: () => ({ status: 0, stdout: JSON.stringify({ installed: [record], available: [] }) }) };
-}
-function registryPlugin(f, host, records, enabled = true) {
-  const config = path.join(f.home, `.${host}`), id = 'ui-ux-pro-max@fixture-market';
-  write(path.join(config, 'settings.json'), { enabledPlugins: { [id]: enabled } });
-  write(path.join(config, 'plugins/installed_plugins.json'), { version: 2, plugins: { [id]: records } });
-  for (const record of records) {
-    write(path.join(record.installPath, host === 'claude' ? '.claude-plugin/plugin.json' : '.codebuddy-plugin/plugin.json'), { name: 'ui-ux-pro-max', version: record.version, ...(host === 'claude' ? { skills: './.claude/skills/' } : {}) });
-    skill(path.join(record.installPath, host === 'claude' ? '.claude/skills/ui-ux-pro-max' : 'skills/ui-ux-pro-max'));
-  }
-  return config;
-}
-
-test('explicit provider has precedence over host roots and does not invoke discovery', t => {
-  const f = fixture(t), root = path.join(f.root, 'explicit');
-  assert.deepEqual(discover({ ...f, provider: root, providerRoots: ['/ignored'], host: 'codex' }), { candidates: [{ root, source: 'explicit' }], diagnostics: [] });
-});
-test('host-exposed provider roots work without host context and preserve order', t => {
-  const f = fixture(t), roots = [path.join(f.root, 'first'), path.join(f.root, 'second')];
-  assert.deepEqual(discover({ ...f, providerRoots: roots }).candidates.map(c => c.root), roots);
-  assert.equal(discover({ ...f, providerRoots: [] }).contextRequired, undefined);
-});
-test('unknown host requires context and cannot read another host cache', t => {
-  const f = fixture(t); skill(path.join(f.home, '.claude/skills/ui-ux-pro-max'));
-  assert.equal(discover(f).contextRequired, true);
-  assert.equal(discover({ ...f, host: 'unknown' }).contextRequired, true);
-  assert.deepEqual(discover({ ...f, host: 'unknown' }).candidates, []);
-});
-test('discover false prevents automatic host access', t => {
-  const f = fixture(t); skill(path.join(f.home, '.agents/skills/ui-ux-pro-max'));
-  assert.deepEqual(discover({ ...f, host: 'codex', discover: false }), { candidates: [], diagnostics: [], contextRequired: true });
-});
-test('Codex returns project then global skill locations and the exact enabled plugin version', t => {
-  const f = fixture(t), plugin = codexPlugin(f);
-  const project = skill(path.join(f.project, '.agents/skills/ui-ux-pro-max'));
-  const global = skill(path.join(f.home, '.codex/skills/ui-ux-pro-max'));
-  const stale = path.join(f.home, '.codex/plugins/cache/ui-ux-pro-max-skill/ui-ux-pro-max/99.0.0/.claude/skills/ui-ux-pro-max'); skill(stale);
-  let command;
-  const result = discover({ ...f, host: 'codex' }, (...args) => { command = args; return plugin.run(); });
-  assert.deepEqual(result.candidates.map(c => c.root), [project, global, path.join(plugin.cache, '.claude/skills/ui-ux-pro-max')]);
-  assert.equal(command[0], 'codex');
-  assert.deepEqual(command[1], ['plugin', 'list', '--marketplace', 'ui-ux-pro-max-skill', '--json']);
-  assert.ok(command[2].timeout > 0 && command[2].timeout <= 15000);
-  assert.equal(command[2].shell, false);
-});
-test('Codex honors isolated CODEX_HOME and never scans the default cache', t => {
-  const f = fixture(t); f.env.CODEX_HOME = path.join(f.root, 'custom-codex');
-  const plugin = codexPlugin(f);
-  const result = discover({ ...f, host: 'codex' }, plugin.run);
-  assert.equal(result.candidates.at(-1).root, path.join(plugin.cache, '.claude/skills/ui-ux-pro-max'));
-});
-test('Codex disabled cached plugin is not a candidate', t => {
-  const f = fixture(t), plugin = codexPlugin(f, { enabled: false });
-  const result = discover({ ...f, host: 'codex' }, plugin.run);
-  assert.deepEqual(result.candidates, []);
-  assert.equal(result.diagnostics[0].availability, 'disabled');
-});
-test('Codex missing installation does not inspect cached versions', t => {
-  const f = fixture(t); codexPlugin(f);
-  const result = discover({ ...f, host: 'codex' }, () => ({ status: 0, stdout: '{"installed":[],"available":[]}' }));
-  assert.deepEqual(result.candidates, []);
-  assert.ok(result.diagnostics.every(d => d.availability === 'missing'));
-});
-for (const [name, run] of [
-  ['missing CLI', () => ({ status: null, error: { code: 'ENOENT' }, stdout: '' })],
-  ['failed CLI', () => ({ status: 1, stderr: 'failure', stdout: '' })],
-  ['invalid JSON', () => ({ status: 0, stdout: '{bad' })],
-  ['invalid schema', () => ({ status: 0, stdout: '{}' })],
-]) test(`Codex ${name} is a discovery error, not a missing installation`, t => {
-  const f = fixture(t), result = discover({ ...f, host: 'codex' }, run);
-  assert.equal(result.diagnostics[0].availability, 'discovery-error');
-});
-test('Codex rejects multiple active installed records instead of selecting a version', t => {
-  const f = fixture(t), { record } = codexPlugin(f);
-  const result = discover({ ...f, host: 'codex' }, () => ({ status: 0, stdout: JSON.stringify({ installed: [record, { ...record, version: '2.14.0' }] }) }));
-  assert.deepEqual(result.candidates, []);
-  assert.equal(result.diagnostics[0].availability, 'ambiguous');
-});
-for (const [name, change] of [
-  ['wrong manifest version', m => { m.version = '2.14.0'; }],
-  ['wrong manifest name', m => { m.name = 'different'; }],
-  ['escaping skills declaration', m => { m.skills = '../../../outside'; }],
-  ['missing skills declaration', m => { delete m.skills; }],
-]) test(`Codex rejects ${name}`, t => {
-  const f = fixture(t), plugin = codexPlugin(f), file = path.join(plugin.cache, '.claude-plugin/plugin.json');
-  const manifest = JSON.parse(fs.readFileSync(file)); change(manifest); write(file, manifest);
-  const result = discover({ ...f, host: 'codex' }, plugin.run);
-  assert.deepEqual(result.candidates, []);
-  assert.equal(result.diagnostics[0].availability, 'discovery-error');
-});
-test('Codex validates the active version as a single safe path segment', t => {
-  const f = fixture(t), plugin = codexPlugin(f, { version: '../../escape' });
-  assert.equal(discover({ ...f, host: 'codex' }, plugin.run).diagnostics[0].availability, 'discovery-error');
-});
-test('Codex rejects a child skill symlink escaping the registered plugin root', t => {
-  const f = fixture(t), plugin = codexPlugin(f), outside = skill(path.join(f.root, 'outside'));
-  const child = path.join(plugin.cache, '.claude/skills/ui-ux-pro-max');
-  fs.rmSync(child, { recursive: true }); fs.symlinkSync(outside, child, 'dir');
-  const result = discover({ ...f, host: 'codex' }, plugin.run);
-  assert.deepEqual(result.candidates, []);
-  assert.equal(result.diagnostics[0].availability, 'discovery-error');
-});
-for (const host of ['claude', 'workbuddy', 'codebuddy']) test(`${host} selects only enabled registered plugins`, t => {
-  const f = fixture(t), installPath = path.join(f.root, host, 'plugin');
-  registryPlugin(f, host, [{ scope: 'user', installPath, version: '2.0.0' }]);
-  const result = discover({ ...f, host });
-  assert.deepEqual(result.candidates.map(c => c.root), [path.join(installPath, host === 'claude' ? '.claude/skills/ui-ux-pro-max' : 'skills/ui-ux-pro-max')]);
-  registryPlugin(f, host, [{ scope: 'user', installPath, version: '2.0.0' }], false);
-  const disabled = discover({ ...f, host });
-  assert.deepEqual(disabled.candidates, []); assert.equal(disabled.diagnostics[0].availability, 'disabled');
-});
-test('missing optional registry is normal, while corrupt metadata is an error', t => {
-  const f = fixture(t);
-  assert.ok(discover({ ...f, host: 'workbuddy' }).diagnostics.every(d => d.availability === 'missing'));
-  write(path.join(f.home, '.workbuddy/plugins/installed_plugins.json'), '{invalid');
-  assert.equal(discover({ ...f, host: 'workbuddy' }).diagnostics[0].availability, 'discovery-error');
-});
-test('a registered plugin with missing enablement metadata needs diagnosis instead of installation', t => {
-  const f = fixture(t);
-  const config = registryPlugin(f, 'workbuddy', [{ scope: 'user', installPath: path.join(f.root, 'plugin'), version: '2.0.0' }]);
-  fs.unlinkSync(path.join(config, 'settings.json'));
-  const result = discover({ ...f, host: 'workbuddy' });
-  assert.deepEqual(result.candidates, []);
-  assert.equal(result.diagnostics[0].availability, 'discovery-error');
-});
-test('project plugin scope must match the current project path', t => {
-  const f = fixture(t), installPath = path.join(f.root, 'plugin');
-  registryPlugin(f, 'claude', [{ scope: 'project', projectPath: path.join(f.root, 'other-project'), installPath, version: '2.0.0' }]);
-  assert.deepEqual(discover({ ...f, host: 'claude-code' }).candidates, []);
-  registryPlugin(f, 'claude', [{ scope: 'project', projectPath: f.project, installPath, version: '2.0.0' }]);
-  assert.equal(discover({ ...f, host: 'claude-code' }).candidates.length, 1);
-  registryPlugin(f, 'claude', [{ scope: 'project', installPath, version: '2.0.0' }]);
-  assert.deepEqual(discover({ ...f, host: 'claude-code' }).candidates, []);
-});
-test('ambiguous matching registry entries do not pick one arbitrarily', t => {
-  const f = fixture(t);
-  registryPlugin(f, 'workbuddy', [
-    { scope: 'user', installPath: path.join(f.root, 'first'), version: '2.0.0' },
-    { scope: 'user', installPath: path.join(f.root, 'second'), version: '2.1.0' },
-  ]);
-  const result = discover({ ...f, host: 'workbuddy' });
-  assert.deepEqual(result.candidates, []); assert.equal(result.diagnostics[0].availability, 'ambiguous');
-});
-test('host selection never borrows another host global plugin', t => {
-  const f = fixture(t);
-  registryPlugin(f, 'claude', [{ scope: 'user', installPath: path.join(f.root, 'claude'), version: '2.0.0' }]);
-  registryPlugin(f, 'workbuddy', [{ scope: 'user', installPath: path.join(f.root, 'workbuddy'), version: '2.0.0' }]);
-  assert.deepEqual(discover({ ...f, host: 'codebuddy' }).candidates, []);
-});
-test('WorkBuddy uses its global config override and the project CodeBuddy skills directory', t => {
-  const f = fixture(t); f.env.WORKBUDDY_CONFIG_DIR = path.join(f.root, 'custom-workbuddy');
-  const projectRoot = skill(path.join(f.project, '.codebuddy/skills/ui-ux-pro-max'));
-  const globalRoot = skill(path.join(f.env.WORKBUDDY_CONFIG_DIR, 'skills/ui-ux-pro-max'));
-  assert.deepEqual(discover({ ...f, host: 'workbuddy' }).candidates.map(c => c.root), [projectRoot, globalRoot]);
-});

package/scripts/design-provider-test.js

@@ -1,75 +0,0 @@
-'use strict';
-const { test, mock } = require('node:test');
-const assert = require('node:assert/strict');
-const fs = require('node:fs');
-const os = require('node:os');
-const path = require('node:path');
-const cp = require('node:child_process');
-mock.method(cp, 'spawnSync', (command, args) => {
-  if (!args.includes('--json') || args[0] !== '-B') return { status: 1, stdout: '', stderr: 'No plugin CLI in isolated test' };
-  if (args[1].includes('query-fails')) return { status: 1, stdout: '', stderr: 'Python module unavailable' };
-  return { status: 0, stdout: JSON.stringify({ domain: 'ux', results: [{ name: 'Error summary' }] }), stderr: '' };
-});
-const installationFile = path.join(__dirname, 'lib/installation.js');
-const skillsDirectory = fs.existsSync(installationFile) ? require(installationFile).loadInstallation().skillsDirectory : path.resolve(__dirname, '../skills');
-const lib = require(path.join(skillsDirectory, 'i3-ui-prototype/scripts/prototype-lib.cjs'));
-function fixture(t) {
-  const home = fs.mkdtempSync(path.join(os.tmpdir(), 'i3-provider-'));
-  t.after(() => fs.rmSync(home, { recursive: true, force: true }));
-  return home;
-}
-function provider(home, name) {
-  const root = path.join(home, name);
-  for (const file of ['SKILL.md', 'scripts/search.py', 'scripts/core.py', 'scripts/design_system.py', ...['ux-guidelines', 'styles', 'colors', 'typography', 'products', 'landing', 'ui-reasoning'].map(name => `data/${name}.csv`)]) {
-    fs.mkdirSync(path.dirname(path.join(root, file)), { recursive: true });
-    fs.writeFileSync(path.join(root, file), 'fixture');
-  }
-  return root;
-}
-test('an incomplete project provider does not shadow a usable global provider', t => {
-  const home = fixture(t), local = path.join(home, 'project-provider'), global = provider(home, 'global-provider');
-  fs.mkdirSync(local);
-  const result = lib.checkDesignProvider({ providerRoots: [local, global] });
-  assert.equal(result.availability, 'available');
-  assert.equal(result.root, global);
-  assert.equal(result.attempts[0].availability, 'unusable');
-  assert.ok(result.attempts[0].missing.includes('scripts/search.py'));
-});
-test('a failed project query does not shadow a usable global provider', t => {
-  const home = fixture(t), local = provider(home, 'query-fails'), global = provider(home, 'global-provider');
-  const result = lib.checkDesignProvider({ providerRoots: [local, global] });
-  assert.equal(result.availability, 'available');
-  assert.equal(result.root, global);
-  assert.match(result.attempts[0].reason, /Python module unavailable/);
-});
-test('an explicit provider failure never silently selects a different provider', t => {
-  const home = fixture(t), selected = path.join(home, 'missing'), other = provider(home, 'other');
-  const result = lib.checkDesignProvider({ provider: selected, providerRoots: [other] });
-  assert.equal(result.availability, 'missing');
-  assert.deepEqual(result.searched, [selected]);
-});
-test('without host or provider the result requests context, not installation', t => {
-  const home = fixture(t);
-  const result = lib.initPrototype({ project: home, id: 'demo', home, env: {} });
-  assert.equal(result.provider.availability, 'needs-context');
-  assert.equal(result.choices[0].id, 'locate');
-  assert.ok(!result.choices.some(choice => choice.id === 'install'));
-  assert.equal(fs.existsSync(path.join(home, 'prototypes')), false);
-});
-test('all unusable providers preserve their individual failure reasons without asking to reinstall', t => {
-  const home = fixture(t), incomplete = path.join(home, 'incomplete'), failing = provider(home, 'query-fails');
-  fs.mkdirSync(incomplete);
-  const result = lib.initPrototype({ project: home, id: 'demo', providerRoots: [incomplete, failing] });
-  assert.equal(result.provider.availability, 'unusable');
-  assert.equal(result.provider.attempts.length, 2);
-  assert.equal(result.choices[0].id, 'repair');
-  assert.ok(!result.choices.some(choice => choice.id === 'install'));
-  assert.equal(fs.existsSync(path.join(home, 'prototypes')), false);
-});
-test('a business package version cannot masquerade as the later installed UIUX version', t => {
-  const home = fixture(t), root = provider(home, 'manual-provider');
-  fs.writeFileSync(path.join(home, 'package.json'), JSON.stringify({ name: 'human-business-project', version: '99.88.77' }));
-  assert.equal(lib.checkDesignProvider({ provider: root }).version, 'unversioned');
-  fs.writeFileSync(path.join(root, 'UPSTREAM-ORIGIN.json'), JSON.stringify({ packageName: 'ui-ux-pro-max-cli', version: '2.15.0' }));
-  assert.equal(lib.checkDesignProvider({ provider: root }).version, '2.15.0');
-});