@fontdata/ai-harness 2.4.0 → 2.5.1

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "fontdata",
-  "version": "2.4.0",
+  "version": "2.5.1",
   "description": "Harness de engenharia com IA da Fontdata para Claude Code",
   "author": { "name": "Fontdata" },
   "commands": ["./templates/commands"],

package/README.md

@@ -103,6 +103,21 @@ npx @fontdata/ai-harness init --comandos resolver-bug,nova-tarefa --sem-harness
 Com o plugin `fontdata@ai-harness` habilitado, o `init` não instala os hooks no projeto, para eles não rodarem
 duas vezes.
 
+### Permissões para o modo auto
+
+O `init` e o `update` gravam em `.claude/settings.json` as permissões que os fluxos usam, para rodarem sem pedir
+aprovação, inclusive no modo auto do Claude Code:
+
+- `permissions.allow`: o rastro, a checagem de conflitos, o `doctor` e a `memoria` do harness; `git add`,
+  `git commit` e `git push` (o `/git-commit-push`); a abertura de sub-agents (`Agent`); e consultas só de leitura
+  (`git status`, `diff`, `log`, `show`, `blame`, `gh pr view`, `gh pr diff`, `npm outdated`...);
+- `permissions.deny`: o push forçado (`--force`, `-f`, `--force-with-lease` e refspec com `+`), que vence o `allow`.
+
+Ficam de fora os comandos de teste e build, que variam por projeto, e os que regravam a instalação (`init`,
+`update`, `migrar`). O projeto acrescenta os seus no próprio `settings.json`; o harness nunca remove regras.
+Nos modos 1 e 2 nada é gravado no projeto: rode `npx @fontdata/ai-harness update --yes` num projeto já instalado,
+ou `init`, para receber as regras.
+
 ## Subcomandos do CLI
 
 | Subcomando | O que faz |

package/dist/install/plan.js

@@ -7,7 +7,7 @@ import { buildLockItem, LOCK_VERSION, readLock, writeLock } from './lock.js';
 import { HOOKS_ITEM_ID, PROJECT_HOOKS_DIR, projectHooksConfig } from './hooks.js';
 import { neededSkillsFor, resolveSelection } from './selection.js';
 import { COMMENTS_RULE_PATH, commentsHooksConfig, ensureCommentsBaseline } from '../comentarios/instalar.js';
-import { combineHooks, HARNESS_PERMISSIONS, mergeSettings } from './settings.js';
+import { combineHooks, HARNESS_DENY, HARNESS_PERMISSIONS, mergeSettings } from './settings.js';
 /** Rule de convenção com o que é específico do projeto (D-38): é do projeto, então fica fora do lock. */
 export const PROJECT_RULE_PATH = '.claude/rules/harness-projeto.md';
 /** Esqueleto da rule de projeto; os comandos `-legado` renomeados pelo update entram na última seção. */
@@ -161,8 +161,9 @@ export function executePlan(root, plan, pkg, now = new Date()) {
     applyAtomically(root, plan.files);
     const permission = needsEventPermission(plan);
     const hooks = combineHooks(plan.registerHooks ? projectHooksConfig() : {}, plan.registerComments ? commentsHooksConfig() : {});
-    if (Object.keys(hooks).length || permission)
-        mergeSettings(root, { hooks, allow: permission ? HARNESS_PERMISSIONS : [] }, now);
+    if (Object.keys(hooks).length || permission) {
+        mergeSettings(root, { hooks, allow: permission ? HARNESS_PERMISSIONS : [], deny: permission ? HARNESS_DENY : [] }, now);
+    }
     if (plan.registerComments)
         ensureCommentsBaseline(root);
     const items = new Map();
@@ -200,8 +201,11 @@ export function describePlan(plan) {
         lines.push('O CLAUDE.md atual será copiado para CLAUDE.md.bak.');
     if (plan.registerHooks)
         lines.push('Hooks do harness registrados em .claude/settings.json (sem duplicar os existentes).');
-    if (needsEventPermission(plan))
-        lines.push(`Permissões ${HARNESS_PERMISSIONS.join(' e ')} em .claude/settings.json, para o rastro e a checagem de conflitos não pedirem aprovação.`);
+    if (needsEventPermission(plan)) {
+        lines.push(`Permissões ${HARNESS_PERMISSIONS.join(', ')} em .claude/settings.json, para o rastro, a checagem de conflitos, ` +
+            'o /git-commit-push, os sub-agents e as consultas de leitura não pedirem aprovação (inclusive no modo auto); ' +
+            'o push forçado fica bloqueado em permissions.deny.');
+    }
     if (plan.registerComments) {
         lines.push('Padrão de comentários: rule carregada em toda sessão, hook que bloqueia a violação e catraca do legado (gerada só na primeira instalação).');
     }

package/dist/install/settings.js

@@ -5,8 +5,43 @@ export const SETTINGS_PATH = join('.claude', 'settings.json');
 export const EVENT_PERMISSION = 'Bash(npx @fontdata/ai-harness evento:*)';
 /** O /executar-plan checa (e aguarda) conflito de arquivos com outros trabalhos por este comando. */
 export const CONFLICTS_PERMISSION = 'Bash(npx @fontdata/ai-harness conflitos:*)';
+/** O /git-commit-push roda sem confirmação, o que inclui o modo auto do Claude Code. */
+export const GIT_PERMISSIONS = ['Bash(git add:*)', 'Bash(git commit:*)', 'Bash(git push:*)'];
+/** O /executar-plan e as skills de fluxo delegam a sub-agents a cada task e gate. */
+export const AGENT_PERMISSION = 'Agent';
+/**
+ * Comandos só de leitura que as skills, os comandos e os agents rodam. Ficam de fora os que regravam a instalação
+ * (`init`, `update`, `migrar`), os de teste e build, que variam por projeto, e o `npm audit`, cujo prefixo
+ * liberaria também o `npm audit fix`.
+ */
+export const READ_PERMISSIONS = [
+    'Bash(npx @fontdata/ai-harness doctor:*)',
+    'Bash(npx @fontdata/ai-harness memoria:*)',
+    'Bash(git status:*)',
+    'Bash(git diff:*)',
+    'Bash(git log:*)',
+    'Bash(git show:*)',
+    'Bash(git blame:*)',
+    'Bash(git describe:*)',
+    'Bash(git rev-list:*)',
+    'Bash(git rev-parse:*)',
+    'Bash(git remote -v)',
+    'Bash(git symbolic-ref --short refs/remotes/origin/HEAD)',
+    'Bash(gh pr view:*)',
+    'Bash(gh pr diff:*)',
+    'Bash(npm outdated:*)',
+];
 /** Regras de `permissions.allow` que o init e o update gravam para as skills e os agents do harness. */
-export const HARNESS_PERMISSIONS = [EVENT_PERMISSION, CONFLICTS_PERMISSION];
+export const HARNESS_PERMISSIONS = [EVENT_PERMISSION, CONFLICTS_PERMISSION, ...GIT_PERMISSIONS, AGENT_PERMISSION, ...READ_PERMISSIONS];
+/** O push liberado não vale para reescrever o histórico remoto: `deny` vence o `allow`. */
+export const HARNESS_DENY = [
+    'Bash(git push --force:*)',
+    'Bash(git push *--force*)',
+    'Bash(git push -f:*)',
+    'Bash(git push * -f)',
+    'Bash(git push * -f *)',
+    'Bash(git push * +*)',
+];
 function isPlainObject(value) {
     return typeof value === 'object' && value !== null && !Array.isArray(value);
 }
@@ -24,12 +59,12 @@ export function combineHooks(...configs) {
     }
     return combined;
 }
-/** A regra já está em `permissions.allow` do `.claude/settings.json` do projeto? */
-export function hasAllowRule(root, rule) {
+/** A regra já está em `permissions.allow` (ou na lista pedida) do `.claude/settings.json` do projeto? */
+export function hasAllowRule(root, rule, key = 'allow') {
     try {
         const settings = JSON.parse(readFileSync(join(root, SETTINGS_PATH), 'utf8'));
-        const allow = settings.permissions?.allow;
-        return Array.isArray(allow) && allow.includes(rule);
+        const list = settings.permissions?.[key];
+        return Array.isArray(list) && list.includes(rule);
     }
     catch {
         return false;
@@ -60,7 +95,11 @@ export function mergeSettings(root, additions, now = new Date()) {
         const invalid = !isPlainObject(parsed) ||
             (parsed.hooks !== undefined && !isPlainObject(parsed.hooks)) ||
             (parsed.permissions !== undefined && !isPlainObject(parsed.permissions)) ||
-            (isPlainObject(parsed.permissions) && parsed.permissions.allow !== undefined && !Array.isArray(parsed.permissions.allow));
+            (isPlainObject(parsed.permissions) &&
+                ['allow', 'deny'].some((key) => {
+                    const list = parsed.permissions[key];
+                    return list !== undefined && !Array.isArray(list);
+                }));
         if (invalid) {
             throw new Error(`O ${SETTINGS_PATH} está com JSON inválido; corrija-o antes de instalar os hooks.`);
         }
@@ -87,17 +126,19 @@ export function mergeSettings(root, additions, now = new Date()) {
         }
         next.hooks = merged;
     }
-    const rules = additions.allow ?? [];
-    if (rules.length) {
-        const permissions = { ...(settings.permissions ?? {}) };
-        const allow = [...(permissions.allow ?? [])];
+    for (const key of ['allow', 'deny']) {
+        const rules = additions[key] ?? [];
+        if (!rules.length)
+            continue;
+        const permissions = { ...(next.permissions ?? {}) };
+        const list = [...(permissions[key] ?? [])];
         for (const rule of rules) {
-            if (allow.includes(rule))
+            if (list.includes(rule))
                 continue;
-            allow.push(rule);
+            list.push(rule);
             changed = true;
         }
-        next.permissions = { ...permissions, allow };
+        next.permissions = { ...permissions, [key]: list };
     }
     if (!changed)
         return { changed: false, backup: null };

package/dist/update/update.js

@@ -9,7 +9,7 @@ import { HOOKS_ITEM_ID, isHarnessPluginEnabled, projectHooksConfig } from '../in
 import { hashContent, LOCK_PATH, LOCK_VERSION, readLock, writeLock } from '../install/lock.js';
 import { itemSources, packageInfo, PROJECT_RULE_PATH, projectRuleContent } from '../install/plan.js';
 import { neededSkillsFor } from '../install/selection.js';
-import { combineHooks, HARNESS_PERMISSIONS, hasAllowRule, mergeSettings } from '../install/settings.js';
+import { combineHooks, HARNESS_DENY, HARNESS_PERMISSIONS, hasAllowRule, mergeSettings } from '../install/settings.js';
 import { adoptLegacyInstall } from './legado.js';
 /** `<id>-legado.md`, com sufixo numérico quando o nome já está em uso. */
 function legacyPath(root, path) {
@@ -108,6 +108,10 @@ export function checkUpdate(root, catalog = loadCatalog(), legacy, home) {
         addItem('rule', 'comentarios', true);
         addItem('hook', 'comentarios', true);
     }
+    const permissoesFaltando = [
+        ...HARNESS_PERMISSIONS.filter((rule) => !hasAllowRule(root, rule)),
+        ...HARNESS_DENY.filter((rule) => !hasAllowRule(root, rule, 'deny')).map((rule) => `deny ${rule}`),
+    ];
     return {
         changes,
         lock,
@@ -115,7 +119,8 @@ export function checkUpdate(root, catalog = loadCatalog(), legacy, home) {
         adotada,
         registrarHooks: adotada && !pluginHooks,
         registrarComentarios: adotada,
-        permissaoAusente: !HARNESS_PERMISSIONS.every((rule) => hasAllowRule(root, rule)),
+        permissaoAusente: permissoesFaltando.length > 0,
+        permissoesFaltando,
     };
 }
 /**
@@ -154,7 +159,7 @@ export function applyUpdate(root, report, options, pkg, now = new Date()) {
         return { de: path, para };
     });
     const hooks = combineHooks(report.registrarHooks ? projectHooksConfig() : {}, report.registrarComentarios ? commentsHooksConfig() : {});
-    mergeSettings(root, { hooks, allow: HARNESS_PERMISSIONS }, now);
+    mergeSettings(root, { hooks, allow: HARNESS_PERMISSIONS, deny: HARNESS_DENY }, now);
     if (report.registrarComentarios)
         ensureCommentsBaseline(root);
     if (report.adotada) {
@@ -220,7 +225,11 @@ export function describeUpdate(report) {
                 'a rule de projeto e a permissão do `ai-harness evento`.',
         ]
         : [];
-    const permission = report.permissaoAusente && !report.adotada ? [`Faltam permissões do harness (${HARNESS_PERMISSIONS.join(', ')}) em .claude/settings.json; o --yes as acrescenta.`] : [];
+    const permission = report.permissaoAusente && !report.adotada
+        ? [
+            `Faltam permissões do harness em .claude/settings.json (${report.permissoesFaltando.join(', ')}); o --yes as acrescenta.`,
+        ]
+        : [];
     if (!report.changes.length) {
         const status = adopted.length || permission.length ? 'Nenhum arquivo a atualizar.' : 'A instalação está em dia: nenhum arquivo a atualizar.';
         return [...adopted, ...permission, status].join('\n');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fontdata/ai-harness",
-  "version": "2.4.0",
+  "version": "2.5.1",
   "description": "Harness de engenharia com IA da Fontdata para Claude Code: roda os slash commands direto na pasta do projeto via npx, sem instalar nada — ou instala-os em .claude/commands.",
   "type": "module",
   "bin": {

package/templates/commands/git-commit-push.md

@@ -25,14 +25,33 @@ No repositório atual, execute commit e push conforme o uso abaixo.
      - tipos aceitos: `feat`, `fix`, `refactor`, `docs`, `chore`, `style`, `test`
      - corpo detalhado com descrição completa do que foi feito no commit (itens objetivos)
    - Se o usuário **passou** texto após o comando: use esse texto como mensagem do commit (multilinha permitida).
-3. Execute **imediatamente** (sem etapa de confirmação):
-   - `git add -A`
+3. Execute **imediatamente** (sem etapa de confirmação), deixando de fora a configuração do Claude Code:
+   - `git add -A -- . ':(exclude,glob).claude/settings*.json' ':(exclude,glob).claude/hooks/**'` — o padrão glob
+     cobre `.claude/settings.json` e `.claude/settings.local.json`; com o nome exato, um arquivo que está no
+     `.gitignore` faz o `git add` falhar inteiro
+   - se algum desses caminhos já estava no stage (`git diff --cached --name-only`), tire-o com
+     `git restore --staged -- <caminho>`
    - `git commit` com a mensagem definida no passo 2 (título + corpo quando aplicável)
    - `git push`
+
+   Se as únicas mudanças forem nesses caminhos, não faça commit nem push: vá direto ao passo 4.
 4. Retorne um resumo final com branch, hash e status do push.
+5. **Configuração do Claude Code alterada.** Se `git status` mostrou mudança em `.claude/settings.json` ou em
+   `.claude/hooks/`, avise que ela ficou fora do commit: no modo auto, o agente versionar as próprias permissões e
+   hooks é bloqueado como `[Self-Modification]`, e quem aprova essa mudança é o usuário. Entregue o comando pronto
+   para ele rodar com `!` no prompt, que executa como o próprio usuário:
+
+   ```
+   ! git add .claude/settings.json .claude/hooks/ && git commit -m "chore: Atualiza a configuração do Claude Code" && git push
+   ```
+
+   Liste só os caminhos que mudaram. `.claude/settings.local.json` é pessoal e nunca vai para o repositório: se ele
+   aparecer no `git status`, sugira incluí-lo no `.gitignore`.
 
 ## Guardrails
 
 - Se não houver alterações, interrompa com aviso.
 - Não execute push forçado.
+- Não tente versionar a configuração do Claude Code por outro caminho quando o modo auto bloquear: o passo 5 é
+  do usuário.
 - Em caso de erro, pare e explique o próximo passo recomendado.