Skip to content

Commit ee2a702

Browse files
committed
docs(claude): record agent standing permissions and operational lessons
Add two sections to CLAUDE.md: - Permissões permanentes: the agent may, without per-action confirmation, read anything, inspect files / run any tests / analyze results, run and watch CI, read/reply to PR comments, create branches and write code, and commit/push/ open PRs (iterating until green). Merge, third-party branch deletion, force-push and destructive ops still need explicit confirmation. - Lições operacionais: the concrete mistakes made this session (commit only on a feature branch, never commit red, incremental build lies, re-read after a failed edit, one command at a time when debugging, CI status caching, out-of-order results, one PR at a time, CPM for new deps, local format false-positives).
1 parent e1ba69d commit ee2a702

1 file changed

Lines changed: 27 additions & 0 deletions

File tree

CLAUDE.md

Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -90,3 +90,30 @@ O handoff esperado do Claude Design são quatro entregáveis (`tokens.css`, `com
9090
- Siga `plan.md` em ordem; um PR por item, pequeno e independente.
9191
- Todo PR fecha um critério de aceite da spec e vem com testes.
9292
- Mudou uma decisão? Atualize `NeoReports-Decisoes.md` no mesmo PR.
93+
94+
## Permissões permanentes do agente (concedidas pelo mantenedor)
95+
96+
O agente tem permissão **permanente** para executar, sem pedir confirmação a cada vez, o ciclo completo de trabalho de um PR:
97+
98+
- **Leitura irrestrita.** Qualquer comando de leitura/inspeção em qualquer parte do sistema: ler/listar/buscar arquivos, `git` de leitura (`status`, `log`, `diff`, `show`, `branch`, `ls-files`, …), inspecionar build/testes, etc.
99+
- **Inspecionar arquivos, rodar quaisquer testes e analisar os resultados.**
100+
- **Executar e acompanhar os checks/CI.** Disparar/aguardar o CI, ler os resultados (`gh run`, `gh pr checks`) e diagnosticar falhas.
101+
- **Ler e responder comentários do PR no GitHub** (`gh pr view`/`comment`, review comments).
102+
- **Criar branch e escrever o código necessário.**
103+
- **Commit, push e abrir PR** (`git commit`, `git push`, `gh pr create`).
104+
- **Esperar o CI, corrigir o que for necessário, commitar e fazer push de novo** — iterar até o PR ficar verde.
105+
106+
Ainda exigem confirmação explícita: **merge de PR**, deletar branch remota de terceiros, force-push, e ações destrutivas/irreversíveis (`reset --hard` em remoto, mudar visibilidade do repo, dropar schema, etc.).
107+
108+
## Lições operacionais (erros já cometidos — não repetir)
109+
110+
1. **Confirme a branch ANTES de commitar.** Sempre `git branch --show-current` antes de `git add`/`commit`. Já commitei direto no `master` por engano (só não quebrou porque sou admin). Todo trabalho em branch própria.
111+
2. **NUNCA commite/pushe com build ou teste vermelho.** Antes de `git commit`, leia a linha real `Build succeeded`/`Passed!`/`Failed!` do escopo alterado. Se houver `Failed: N>0` ou `Build FAILED`, não commite. Já afirmei "verde" sem verificar e mergeei PR vermelho — inaceitável.
112+
3. **Build local incremental MENTE.** `obj/bin` em cache já mascarou erro real de compilação (CS0246) que só apareceu no CI. Na dúvida, `--no-incremental` ou apague `obj/bin` e rebuilde do zero antes de confiar no verde.
113+
4. **Edit que falha com "file modified since read" NÃO foi aplicado.** Re-leia e refaça; nunca assuma que entrou. Docs (ADR/plan) já não entraram em commits por isso.
114+
5. **Um comando por vez ao diagnosticar.** Lotes grandes em paralelo cancelam em cascata no primeiro erro e corrompem a visão de estado.
115+
6. **CI tem cache de status.** `gh pr checks` / `commits/<sha>/check-runs` pode mostrar run antigo. Confirme `headRefOid` do PR == seu último commit antes de confiar no resultado.
116+
7. **Resultados podem chegar fora de ordem.** Ao notar incoerência, PARE e rode uma checagem sequencial `git branch/status/log` antes de agir.
117+
8. **Um PR por vez.** Não abrir vários simultâneos sem o mantenedor pedir; fechar (mergear) o atual antes do próximo.
118+
9. **Novas dependências (inclusive de teste) vão no CPM** (`build/Directory.Packages.props`). `NU1010` = `PackageVersion` faltando.
119+
10. **`dotnet format --verify-no-changes` dá falso-positivo local** por autocrlf (CRLF no working tree vs LF no repo); o CI faz checkout LF e passa. Não persiga `ENDOFLINE` local; confie no step de format do CI.

0 commit comments

Comments
 (0)