Skip to content

docs(quality): fix stale gate count + opt-in agent-lsp scaffold - #3902

Merged
diegosouzapw merged 1 commit into
release/v3.8.26from
docs/gate-count-and-agent-lsp
Jun 15, 2026
Merged

diegosouzapw merged 1 commit into
release/v3.8.26from
docs/gate-count-and-agent-lsp

Conversation

@diegosouzapw

Copy link
Copy Markdown
Collaborator

Fecha os 2 últimos gaps sem-secret da auditoria.

#9 — Contagem de gates desatualizada

CLAUDE.md e QUALITY_GATES.md diziam "35 CI quality gates across 6 jobs". O real:
~48 gate scripts (44 em scripts/check/ + 4 em scripts/quality/) wirados em 9 jobs que rodam gates no ci.yml (lint, quality-gate, quality-extended, docs-sync-strict, i18n-ui-coverage, i18n, pr-test-policy, test-vitest, sonarqube) + o job quality.yml (PR→release) + 3 nightlies. Também removi os sub-counts stale "lint (18 checks) + docs-sync-strict (12 checks)" (o lint já tem ~24 steps) pra não reintroduzir falsa precisão.

#3 — agent-lsp (Fase 7 Task 15)

Adicionado .mcp.json.example — scaffold opt-in de LSP-in-the-loop que expõe um TypeScript language server aos agentes (resolvem símbolos/diagnostics antes de editar; companheiro compile-before-claim do typecheck:core).

Por que .example e não .mcp.json: um .mcp.json auto-carregado afeta todas as sessões do repo; o bridge MCP↔LSP é escolha do operador e precisa ser verificado localmente. Como .example, nunca auto-carrega um servidor não-vetado — uma entrada quebrada só loga erro de conexão, não quebra sessões. Documentado em QUALITY_GATES.md.

Docs/scaffold apenas — sem código de produção, sem versão.

Closes the last two no-secret gaps from the gate-completeness audit:

#9 Stale doc count: CLAUDE.md and QUALITY_GATES.md claimed '35 CI quality gates
across 6 jobs' — the real tree has ~48 gate scripts (44 in scripts/check + 4 in
scripts/quality) wired across 9 gate-running ci.yml jobs (lint, quality-gate,
quality-extended, docs-sync-strict, i18n-ui-coverage, i18n, pr-test-policy,
test-vitest, sonarqube) plus the quality.yml fast-gates job and 3 nightly
workflows. Also dropped the stale 'lint (18 checks) + docs-sync-strict (12 checks)'
sub-counts (lint now has ~24 steps) to avoid re-introducing false precision.

#3 agent-lsp (Fase 7 Task 15): added .mcp.json.example — an OPT-IN LSP-in-the-loop
scaffold that exposes a TypeScript language server to coding agents (resolve
symbols/diagnostics before editing; compile-before-claim companion to typecheck:core).
Shipped as .example (not auto-loaded .mcp.json) because the MCP<->LSP bridge is the
operator's choice and must be verified locally; a broken entry only logs a connection
error and never breaks sessions. Documented in QUALITY_GATES.md.

Docs/scaffold only — no production code, no version bump.
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request introduces an opt-in agent-lsp configuration scaffold (.mcp.json.example) to expose a TypeScript language server to coding agents, and updates the documentation (CLAUDE.md and QUALITY_GATES.md) to reflect the increased number of quality-gate scripts (~48 scripts across 9 jobs). The review feedback correctly points out that adding .mcp.json.example to the project root violates the strict file placement rules in the repository style guide, and recommends updating the style guide to explicitly permit this configuration file.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread .mcp.json.example
@@ -0,0 +1,17 @@
{

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

O arquivo .mcp.json.example foi adicionado na raiz do projeto. No entanto, de acordo com as regras de File Placement & Organization (Seção 1) do Guia de Estilo do Repositório, a raiz do projeto (/) deve conter apenas um conjunto estrito de arquivos permitidos.\n\nComo .mcp.json.example (e o eventual .mcp.json) não estão listados explicitamente nas regras, isso constitui uma violação de estilo.\n\nRecomendação:\nAtualize o arquivo de regras correspondente (como AGENTS.md ou GEMINI.md onde essas regras estão definidas) para incluir explicitamente .mcp.json.example e .mcp.json na lista de arquivos permitidos na raiz do projeto, já que arquivos de configuração do MCP precisam residir na raiz para serem descobertos automaticamente pelos clientes MCP.

References
  1. A raiz do projeto deve conter apenas os arquivos explicitamente listados na Seção 1 do Guia de Estilo do Repositório. (link)

@diegosouzapw
diegosouzapw merged commit d0e4ce4 into release/v3.8.26 Jun 15, 2026
2 checks passed
@diegosouzapw
diegosouzapw deleted the docs/gate-count-and-agent-lsp branch June 17, 2026 00:54
tkgo11 pushed a commit to tkgo11/OmniRoute that referenced this pull request Sep 23, 2026
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.

1 participant