@givolt/i3-composer-agent 2.3.3 → 2.3.4

package/skills/i3-ui-prototype/scripts/prototype-hooks.cjs

@@ -4,7 +4,7 @@
 const fs=require('node:fs'),path=require('node:path'),os=require('node:os'),assert=require('node:assert/strict');
 const R=require('./codebuddy-registration.cjs'),C=require('./codebuddy-context.cjs'),E=require('./execution-evidence.cjs');
 function context(options) {
- assert.equal(options.host||'codebuddy','codebuddy','A+B supports CodeBuddy only');
+ const host=options.host||'codebuddy';assert.ok(['codebuddy','workbuddy'].includes(host),'A+B supports CodeBuddy and WorkBuddy');
  const project=fs.realpathSync(path.resolve(options.project||'.')),scope=options.scope;
  assert.ok(['project','global'].includes(scope),'Specify project or global installation scope');
  const base=scope==='project'?path.join(project,'.codebuddy'):C.globalBase(options);
@@ -12,7 +12,7 @@ function context(options) {
  const record=recordText?JSON.parse(recordText):null;
  const unified=path.join(base,'i3-composer-agent/hooks/i3-prototype.cjs');
  const scriptPath=options.entryPath||(fs.existsSync(unified)?unified:path.join(base,'skills/i3-ui-prototype/scripts/hosts/codebuddy.cjs'));
- return {project,scope,base,record,scriptPath};
+ return {project,scope,base,record,scriptPath,host};
 }
 function prepare(options,runtime={}) {
  const current=runtime.current||context(options),readText=runtime.readText||R.fileIO.read,action=options.action||'keep';
@@ -21,12 +21,8 @@ function prepare(options,runtime={}) {
   if(action==='disable'){const settings=R.config(readText(path.join(current.base,'settings.json')));assert.ok(!Object.values(settings.hooks||{}).flat().some(g=>g.hooks.some(h=>h.command?.includes(R.OWNER))),'Unknown owned-looking Hook has no ownership record; preserve it for review');}
   return {summary:{status:'not-enabled',scope:current.scope,changed:false},unchanged:true};
  }
- const product=options.product||current.record?.product;assert.ok(['ide','cli'].includes(product),'Use --codebuddy-product ide or cli when enabling Hooks');
- if(action!=='disable') {
-  const manifest=readText(path.join(current.base,'.i3-composer-agent.json'));
-  if(manifest)assert.ok(!JSON.parse(manifest).agents?.includes('workbuddy'),'Shared CodeBuddy/WorkBuddy registration cannot safely identify the product; preserve settings and use an exclusive CodeBuddy scope');
-  assert.ok(!options.agents?.includes('workbuddy')||current.scope!=='project','Do not enable CodeBuddy audit in a shared WorkBuddy project target');
- }
+ assert.ok(options.product===undefined||['auto','ide','cli'].includes(options.product),'Invalid Buddy product profile');
+ const product='auto'; // Legacy product arguments are hints, never a second new registration.
  const explicit=options.transcriptRoot,roots=explicit==='auto'?[]:explicit?[path.resolve(explicit)]:current.record?.transcriptRoots||[];
  const mode=explicit?(explicit==='auto'?'auto':'manual'):current.record?.transcriptMode||(roots.length?'manual':'auto');
  if(action!=='disable') {
@@ -34,7 +30,7 @@ function prepare(options,runtime={}) {
  }
  const settingsText=readText(path.join(current.base,'settings.json')),settings=R.config(settingsText);
  if(action!=='disable')assert.ok(!settings.disableAllHooks&&!settings.allowManagedHooksOnly,'Host policy prevents automatic auditing; preserve the existing policy');
- return R.plan({action,...current,product,nodePath:process.execPath,scriptDigest:runtime.scriptDigest||C.bundle(options.entrySource||(path.basename(current.scriptPath)==='i3-prototype.cjs'?current.scriptPath:path.join(__dirname,'hosts/codebuddy.cjs'))),currentText:settingsText,
+ return R.plan({action,...current,product,hosts:options.hosts,agents:options.agents,nodePath:process.execPath,scriptDigest:runtime.scriptDigest||C.bundle(options.entrySource||(path.basename(current.scriptPath)==='i3-prototype.cjs'?current.scriptPath:path.join(__dirname,'hosts/codebuddy.cjs'))),currentText:settingsText,
   transcriptMode:mode,transcriptRoots:roots,productVersion:options.productVersion||current.record?.productVersion||'unreported'});
 }
 function configure(options) {
@@ -51,8 +47,9 @@ function check(options,runtime={}) {
   return {...result,status:result.filesPresent?'files-present':'not-enabled'};
  }
  try {
-  const record=(runtime.registration||C.registration)(current.base),effective=(runtime.select||C.select)({project:current.project,product:record.product,home:options.home,env:options.env});
-  Object.assign(result,{product:record.product,productVersion:record.productVersion,registrationDigest:record.registrationDigest,effective:effective?.base===current.base,transcriptRoots:record.transcriptRoots,
+  const record=(runtime.registration||C.registration)(current.base),host=options.host||'codebuddy',products=record.product==='auto'?(host==='workbuddy'?['ide']:['ide','cli']):[record.product];
+  const effectiveByProduct=Object.fromEntries(products.map(product=>[product,(runtime.select||C.select)({project:current.project,product,host,home:options.home,env:options.env})?.base===current.base]));
+  Object.assign(result,{host,product:record.product,productVersion:record.productVersion,registrationDigest:record.registrationDigest,effective:Object.values(effectiveByProduct).every(Boolean),effectiveByProduct,transcriptRoots:record.transcriptRoots,
    transcript:{mode:record.transcriptMode||(record.transcriptRoots?.length?'manual':'auto'),status:'awaiting-native-event',invocationVerified:false}});
   let observation=null;
   const parent=path.join(current.base,'state/i3-prototype',C.projectKey(current.project));
@@ -62,18 +59,21 @@ function check(options,runtime={}) {
    const raw=R.fileIO.read(path.join(parent,name,'session.json'));if(!raw)continue;
    entries.push(...Object.values(JSON.parse(raw).bindings||{}));
   }
-  for(const entry of entries)if(entry.project===current.project&&entry.registrationDigest===record.registrationDigest&&entry.settingsDigest===record.settingsDigest&&(!observation||entry.lastHook.at>observation.lastHook.at))observation=entry;
+  for(const entry of entries)if((entry.host||'codebuddy')===host&&entry.project===current.project&&entry.registrationDigest===record.registrationDigest&&entry.settingsDigest===record.settingsDigest&&(!observation||entry.lastHook.at>observation.lastHook.at))observation=entry;
   if(observation) {
    result.transcript.path=observation.source.path;
    C.prefix(observation.source.path,observation.source.bytes,observation.source.digest);
    C.source({session_id:observation.sessionId,transcript_path:observation.source.path},record,current.project);
    Object.assign(result.transcript,{status:'verified',invocationVerified:true});
    result.observation={sessionId:observation.sessionId,projectKey:observation.projectKey,runId:observation.runId,event:observation.lastHook.event,at:observation.lastHook.at};
+   const observedProduct=observation.product||(record.product!=='auto'?record.product:null);
+   assert.ok(['ide','cli'].includes(observedProduct),'Observed native product identity missing');
+   Object.assign(result,{product:observedProduct,productVersion:observation.productVersion||'unreported',effective:effectiveByProduct[observedProduct]===true,identitySource:observation.identitySource||'legacy-profile',recovery:observation.recovery||null});
   }
   const recent=observation&&Date.now()-Date.parse(observation.lastHook.at)>=0&&Date.now()-Date.parse(observation.lastHook.at)<30*60*1000;
   return {...result,status:result.effective&&recent?'active':record.product==='cli'?'pending-host-review':'registered',
    activationScope:'Only the displayed project/session/configuration and adapter bytes; not all projects or business acceptance',
-   next:record.product==='cli'?'Review /hooks in CodeBuddy Code, then use a new session.':'Reload CodeBuddy and verify a real prototype write task.'};
+   next:'CLI: review /hooks; desktop: reload. Verify an actual Skill invocation and missing-Skill recovery in the chosen host.'};
  }catch(error){if(result.transcript)Object.assign(result.transcript,{status:'unverified',invocationVerified:false});return {...result,status:'failed',errors:[error.message]};}
 }
 function main(argv=process.argv.slice(2)) {

package/skills/i3-ui-prototype/scripts/prototype-locations.cjs

@@ -12,7 +12,7 @@ function globalSkillsDir(agent, home, env) {
   let base = home;
   if (config.base === 'config') base = env.XDG_CONFIG_HOME && path.isAbsolute(env.XDG_CONFIG_HOME) ? env.XDG_CONFIG_HOME : path.join(home, '.config');
   if (config.base === 'env') base = env[config.env]?.trim() || path.join(home, config.fallback);
-  if (config.env === 'CODEBUDDY_CONFIG_DIR' && (!path.isAbsolute(base) || /[\x00\r\n]/.test(base))) throw new Error('CODEBUDDY_CONFIG_DIR must be an absolute safe directory');
+  if (['CODEBUDDY_CONFIG_DIR','WORKBUDDY_CONFIG_DIR'].includes(config.env) && (!path.isAbsolute(base) || /[\x00\r\n]/.test(base))) throw new Error(`${config.env} must be an absolute safe directory`);
   if (config.alternatives && !fs.existsSync(path.join(home, path.dirname(config.path)))) {
     const alternative = config.alternatives.find(relative => fs.existsSync(path.join(home, path.dirname(relative))));
     if (alternative) return path.join(home, alternative);