Skip to content

fix(coderabbit): tone_instructions sob limite de 250 chars#100

Merged
adm01-debug merged 1 commit into
mainfrom
chore/fix-coderabbit-config
May 9, 2026
Merged

fix(coderabbit): tone_instructions sob limite de 250 chars#100
adm01-debug merged 1 commit into
mainfrom
chore/fix-coderabbit-config

Conversation

@adm01-debug
Copy link
Copy Markdown
Owner

@adm01-debug adm01-debug commented May 8, 2026

🐰 Fix CodeRabbit config

CodeRabbit estava reportando parsing error e usando default settings em todos os PRs (vide PR #99) porque tone_instructions excedeu o limite de 250 chars do schema.

Validation error: String must contain at most 250 character(s) at "tone_instructions"

Mudança

  • Antes: 538 chars (longo, com lista detalhada de prioridades)
  • Depois: 221 chars (resumido)

Instruções detalhadas continuam em path_instructions (que não tem limite de tamanho).

Validação

  • yaml-language-server schema OK
  • Char count: 221/250

Diff

1 arquivo, +1/-10 linhas.

Summary by CodeRabbit

  • Chores
    • Atualizada a diretriz de tom de mensagens para uma versão mais curta e direta, mantendo ênfase em bugs reais, segurança, performance e regressões, e priorização de issues críticas (secrets, SQL injection, Promises, tipos e operações destrutivas).

Review Change Stack

Copilot AI review requested due to automatic review settings May 8, 2026 16:52
@vercel
Copy link
Copy Markdown

vercel Bot commented May 8, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
promo-gifts Ready Ready Preview, Comment May 9, 2026 2:26pm

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 8, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: f46641e0-be5e-4812-b9cb-508b257f185e

📥 Commits

Reviewing files that changed from the base of the PR and between c5b2b55 and 865fdb8.

📒 Files selected for processing (1)
  • .coderabbit.yaml
✅ Files skipped from review due to trivial changes (1)
  • .coderabbit.yaml

Walkthrough

Atualização do bloco tone_instructions em .coderabbit.yaml (pt-BR): texto reescrito de forma mais curta e direta, mantendo foco em bugs reais, segurança, performance e prioridades como secrets hardcoded, SQL injection, Promises sem await/catch, any/unknown sem narrowing e operações destrutivas sem backup.

Changes

Atualização de Configuração

Layer / File(s) Summary
Instruções de Tom PT-BR
.coderabbit.yaml
tone_instructions simplificado em pt-BR; mantém critérios de prioridade alta (secrets hardcoded, SQL injection, Promises sem await/catch, any/unknown sem narrowing, operações destrutivas sem backup).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutos

Possibly related PRs

  • adm01-debug/zapp-web#86: Modifica o mesmo bloco tone_instructions em .coderabbit.yaml, com reescrita similar das instruções de tom em PT-BR.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed O título é direto e descreve com precisão o problema corrigido: reduzir tone_instructions para ficar dentro do limite de 250 caracteres.
Description check ✅ Passed A descrição cobre o problema (erro de parsing, config padrão usada), a solução (redução de 538 para 221 chars), validação e diff. Faltam apenas seções opcionais do template.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/fix-coderabbit-config

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Ajusta a configuração do CodeRabbit para voltar a ser validada pelo schema e evitar que o bot caia em default settings por erro de parsing (limite de 250 chars em tone_instructions).

Changes:

  • Reduz tone_instructions para caber no limite do schema (<= 250 chars).
  • Mantém instruções detalhadas em path_instructions para preservar a granularidade por caminho.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

CodeRabbit estava reportando parsing error e usando default settings em
todos os PRs por causa de tone_instructions excedendo o limite de 250
caracteres do schema.

Antes: 538 chars (longo, com lista de prioridades)
Depois: 221 chars (resumido — instruções detalhadas continuam em path_instructions)

Validação: yaml-language-server + cálculo manual de chars.

Detectado no review automático do PR #99.
@adm01-debug adm01-debug force-pushed the chore/fix-coderabbit-config branch from c5b2b55 to 865fdb8 Compare May 9, 2026 14:25
@adm01-debug adm01-debug merged commit d11c06b into main May 9, 2026
14 of 15 checks passed
@adm01-debug adm01-debug deleted the chore/fix-coderabbit-config branch May 9, 2026 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants