feat(installer): conditionally install hooks based on free/pro tier selection

[riaworks]

Problem

The installer copies all hook files to every project regardless of tier:

// ide-config-generator.js:678
const HOOKS_TO_COPY = [
  'synapse-engine.cjs',
  'code-intel-pretool.cjs',
  'precompact-session-digest.cjs',  // ← only useful with aios-pro
  'README.md',
];

precompact-session-digest.cjs is a PreCompact hook that:

1. Detects if aios-pro is installed
2. If yes → extracts a session digest before context compaction
3. If no → silent no-op (aios-pro not available, skipping)

For free/community users, this hook:

• Is always copied (unnecessary file)
• Is always registered in settings.local.json (unnecessary entry)
• Always results in a no-op (wasted process spawn on every compact)

Additionally, the hook is written as a module export (module.exports = async (context) => {...}) but registered as a command hook in settings.local.json. Command hooks must read JSON from stdin and write JSON to stdout — this hook does neither, causing errors when invoked as a command.

Proposed Solution

1. Tier-aware hook installation

During the wizard install step (where user selects free vs pro), conditionally select which hooks to copy:

const HOOKS_FREE = [
  'synapse-engine.cjs',
  'code-intel-pretool.cjs',
  'README.md',
];

const HOOKS_PRO = [
  ...HOOKS_FREE,
  'precompact-session-digest.cjs',
];

const HOOKS_TO_COPY = wizardState.tier === 'pro' ? HOOKS_PRO : HOOKS_FREE;

2. Rewrite precompact hook as command hook

precompact-session-digest.cjs currently exports a function (module pattern). For it to work as a Claude Code command hook registered in settings.local.json, it needs a stdin/stdout wrapper similar to synapse-engine.cjs:

// Read JSON from stdin
const input = await readStdin();
// Call the digest logic
await onPreCompact(input);
// Write result to stdout
process.stdout.write(JSON.stringify({ hookSpecificOutput: { ... } }));

Without this rewrite, even pro users will get errors when the hook is invoked as a command.

3. Dynamic registration in settings.local.json

createClaudeSettingsLocal() already reads from the hooks directory dynamically, so as long as step 1 doesn't copy the file for free users, it won't be registered. No change needed here.

Files to Modify

File	Change
packages/installer/src/wizard/ide-config-generator.js	Tier-aware HOOKS_TO_COPY
.claude/hooks/precompact-session-digest.cjs	Rewrite as command hook (stdin/stdout)
.aios-core/hooks/unified/runners/precompact-runner.js	May need adapter for command hook pattern

Context

• The HOOK_EVENT_MAP already correctly maps precompact-session-digest.cjs → PreCompact event (MIS-3.1)
• pro-detector.js already exists at bin/utils/pro-detector.js for tier detection at runtime
• The relative path fix for hook commands is available at fix(installer): use relative paths for hook commands in settings.local.json riaworks/aios-aiox-core#7

[nikolasdehor]

Excelente issue, @riaworks. Fiz uma análise detalhada do código envolvido e tenho algumas observações técnicas:

Análise do código atual

1. copyClaudeHooksFolder() não recebe wizardState

O ponto central do problema é que copyClaudeHooksFolder(projectRoot) recebe apenas o projectRoot — não tem acesso ao wizardState onde a seleção de tier do usuário está armazenada. A chamada fica em generateIDEConfigs() (linha ~541):

const hookFiles = await copyClaudeHooksFolder(projectRoot);

Enquanto generateIDEConfigs() já recebe wizardState como segundo argumento, esse contexto nunca é repassado para copyClaudeHooksFolder. A correção mínima seria mudar a assinatura:

async function copyClaudeHooksFolder(projectRoot, wizardState = {})

E propagar o parâmetro na chamada.

2. O hook já foi corrigido para stdin/stdout (MIS-3.1)

Li o precompact-session-digest.cjs atual e ele já implementa o padrão readStdin() + parse JSON, igual ao synapse-engine.cjs. A issue menciona que o hook "currently exports a function (module pattern)", mas o código atual já tem o wrapper correto com main() no entrypoint. Parece que a Story MIS-3.1 já resolveu isso — vale confirmar se a versão que está no repo é a atualizada.

3. precompact-runner.js já tem graceful degradation

O runner já faz detecção via isProAvailable() e retorna silenciosamente se pro não está disponível:

if (!proAvailable) {
  console.log('[PreCompact] aios-pro not available, skipping session digest');
  return;
}

Isso significa que o hook funciona como no-op para free users em runtime — não causa erros. Mas o ponto da issue continua válido: é um spawn desnecessário que poderia ser evitado na instalação.

4. createClaudeSettingsLocal() registra hooks dinamicamente

A função lê todos os .cjs do diretório .claude/hooks/ e registra no settings.local.json usando o HOOK_EVENT_MAP. Se o arquivo não for copiado (step 1 da proposta), ele automaticamente não será registrado. A proposta está correta — não precisa mudar createClaudeSettingsLocal().

Sugestão de implementação

A mudança é cirúrgica — só copyClaudeHooksFolder() precisa ser alterada:

async function copyClaudeHooksFolder(projectRoot, wizardState = {}) {
  // ... (existente)

  const HOOKS_FREE = [
    'synapse-engine.cjs',
    'code-intel-pretool.cjs',
    'README.md',
  ];

  const HOOKS_PRO = [
    ...HOOKS_FREE,
    'precompact-session-digest.cjs',
  ];

  // Tier detection: wizardState > runtime pro-detector > default free
  const isPro = wizardState.tier === 'pro' || 
    (wizardState.tier === undefined && isProAvailable());
  
  const HOOKS_TO_COPY = isPro ? HOOKS_PRO : HOOKS_FREE;
  // ... (resto igual)
}

O fallback para isProAvailable() quando wizardState.tier é undefined garante compatibilidade retroativa — se alguém chamar a função sem wizardState (ex: scripts de CI, testes), o comportamento atual é preservado.

Sobre testes

O packages/installer/tests/ já deve ter testes para copyClaudeHooksFolder. Seria bom adicionar:

• Teste com wizardState.tier = 'free' → não copia precompact-session-digest.cjs
• Teste com wizardState.tier = 'pro' → copia todos
• Teste sem wizardState (compatibilidade) → usa isProAvailable() como fallback

Posso abrir um PR com essa implementação se quiser. Boa issue!

[riaworks]

Olá nícolas eu estava com o PR pronto aqui, eu já tinha feito no meu fork e estava testando se não causava nada errado. Então aqui está o PR
#549

Obrigado pela disponibilidade

[nikolasdehor]

Faz muito sentido @riaworks. A instalação de hooks deveria ser condicional ao tier (free/pro). Atualmente o installer registra todos os hooks independente do tier, o que causa erros como o relatado na #528 (precompact-session-digest sem runner no tier free). Posso ajudar com a implementação — já tenho familiaridade com o installer e o hook system.