ESP-030 — DDL ALTER TABLE … RENAME COLUMN (FB / PG / MySQL)

[isaquepinheiro]

ESP-030 — DDL fluente ALTER TABLE … RENAME COLUMN

Descrição

Introduzir fábrica, interfaces e serialização para gerar texto SQL de renomeação de uma coluna em PostgreSQL, Firebird e MySQL, conforme matriz ADR-030. Contrato: só string + POO; sem execução no SGBD nem catálogo.

Contexto

Complementa as verticais ADD/DROP column (ESP-019/ESP-020). ESP-029 / #44 pode seguir /develop / /release em paralelo — não misturar critérios nesta issue.

Âmbito

• Interfaces IFluentDDLAlterTableRenameColumnDef / IFluentDDLAlterTableRenameColumnBuilder e builder em FluentSQL.DDL.pas
• Fábrica pública em FluentSQL.pas (nome e assinatura conforme ADR-030)
• DDLAlterTableRenameColumnSQL em FluentSQL.DDL.Serialize.pas (matriz PG / FB / MySQL)
• Testes DUnitX em Test Delphi/test.ddl.pas
• Guia em docs-src (ex. ddl-alter-table-rename-column.md) e api.md quando aplicável

Fora de âmbito

• Rename de tabela; múltiplas operações num único AsString; CHANGELOG / versão / tags (exclusivo /release); ramos só de um motor fora da matriz comum (ESP-016 ou SQL na aplicação).

Critérios de aceitação

• Comportamento conforme ADR-030 e esp.md (API, serialização por dialecto, exceções EArgumentException / ENotSupportedException com referência ESP-030 onde aplicável).
• Testes DUnitX cobrindo PostgreSQL, Firebird e MySQL (casos felizes e ramos de exceção definidos na ADR).
• Documentação Docusaurus com matriz dialecto × sintaxe × versão mínima; npm run build em docs-src quando ficheiros .md forem alterados.
• Relatórios da esteira sem contradição com a ESP; ROADMAP.md com ESP-030 marcado como entregue apenas após /release.

Referências de pipeline

• .claude/pipeline/esp.md (ESP-030), .claude/pipeline/adr.md (ADR-030), .claude/pipeline/plan.md, .claude/pipeline/task-input.md

[isaquepinheiro]

Code review — /review (ESP-030 / ADR-030)

Verdict: APPROVED WITH CAVEATS

Summary: Implementation matches ADR-030: PostgreSQL and MySQL emit \ALTER TABLE … RENAME COLUMN … TO …; Firebird emits \ALTER TABLE … ALTER … TO …. Interfaces, builder, \DDLAlterTableRenameColumnSQL, and \CreateFluentDDLAlterTableRenameColumn\ follow existing DDL patterns. \ est.ddl.pas\ (\TTestDDLAlterTableRenameColumn) covers the three dialects plus validation and unsupported-dialect branches. Docusaurus guide includes the dialect × syntax × minimum-version matrix; \�pi.md\ / sidebar updated per scope.

Caveat: DUnitX was not executed in the implement environment (MSB6003 / long command line per \implement-report.md). */test* should run \TTestDDLAlterTableRenameColumn\ (or the full DDL suite) where the Delphi test project builds and record evidence.

Kanban: Card added to Project 16 and set to In review for this review pass.

Next step: */test* — move Project 16 card to QA Testing when QA starts.

[isaquepinheiro]

QA /test — APPROVED WITH CAVEATS

Verdict: APPROVED WITH CAVEATS (issue #45, ESP-030 / ADR-030).

Automated checks

• 
  pm run build\ in \docs-src: PASSED
• MSBuild Rebuild \Test Delphi/Firebird_tests/PTestFluentSQLFirebird.dproj\ (Embarcadero rsvars + Delphi toolchain): FAILED — MSB6003 / DCC command line length (known limitation; same class as \implement-report).

Summary

• Serialization \DDLAlterTableRenameColumnSQL, API/factory, and \TTestDDLAlterTableRenameColumn\ in \ est.ddl.pas\ were verified by static review against ADR-030 (PostgreSQL/MySQL \RENAME COLUMN … TO …, Firebird \ALTER … TO …, validation and ESP-030 in exception paths).
• Runtime DUnitX for this suite was not obtained in this environment due to MSB6003; recommend CI or a shorter build path for green execution evidence.
• ROADMAP.md correctly leaves ESP-030 unchecked pending **/release**.

Next

• */develop* when ready (card on Project 16 set to QA Testing).

[isaquepinheiro]

Delivered - closing as part of Kanban hygiene round (ESP-060).