ESP-029 — DDL TRUNCATE TABLE (FB / PG / MySQL)

[isaquepinheiro]

ESP-029 — DDL fluente: TRUNCATE TABLE

Handoff do /architect: esp.md, adr.md (ADR-029), plan.md, task-input.md.

Descrição

Introduzir CreateFluentDDLTruncateTable e interfaces de builder para gerar TRUNCATE TABLE por dialecto. PostgreSQL: opcionais RestartIdentity e Cascade. Firebird e MySQL: forma base; modificadores PG exclusivos → ENotSupportedException (ADR-029). Sem execução no motor.

ESP-028 (#43) pode seguir /develop / /release em paralelo — não misturar critérios.

Acceptance criteria

• Comportamento conforme ADR-029 e esp.md (API, serialização por dialecto, matriz de excepções).
• Testes DUnitX cobrindo PostgreSQL, Firebird e MySQL para o núcleo TRUNCATE TABLE e, em PG, os ramos opcionais definidos na ADR.
• Guia Docusaurus + npm run build em docs-src quando ficheiros .md forem alterados.
• Relatórios da esteira coerentes com esta tarefa; ROADMAP.md com ESP-029 entregue apenas após /release.

Fora de âmbito

• CHANGELOG/versão/tags fora de /release; TRUNCATE multi-tabela; partições MySQL sem ESP dedicada; execução no SGBD.

[isaquepinheiro]

Code review — REJECTED

Verdict: REJECTED (issue #44 / ESP-029)

Critical finding: Naming convention — \Source/Core/FluentSQL.DDL.Serialize.pas, \DDLCreateTableSQL\ (lines 100–110): local loop variable \I\ must use the \L\ prefix (e.g. \LI) to match \LParts\ / \LCol\ in the same block.

Next: /implement\ as rejection fix — re-read .claude/pipeline/review-report.md; minimal rename + loop update only.

(Project 16: no card found for #44 — status field not updated via API.)

[isaquepinheiro]

Code review — #44

Verdict: APPROVED WITH CAVEATS

Summary: ESP-029 (TRUNCATE TABLE) matches ADR-029 (interfaces, CreateFluentDDLTruncateTable, DDLTruncateTableSQL, PG/FB/MySQL matrix, tests in test.ddl.pas). Prior finding (I → LI in DDLCreateTableSQL) is fixed. npm run build (docs-src) passed in the review environment.

Caveat: .claude/pipeline/implement-report.md still describes only the naming correction — reconcile the modified-files list with the actual branch before /test.

Kanban: Issue added to Project 16; Status In review.

Next step: /test

[isaquepinheiro]

QA /test — APPROVED WITH CAVEATS (issue #44)

Verdict: APPROVED WITH CAVEATS — product behaviour for ESP-029 / ADR-029 is consistent with acceptance criteria; automated evidence recorded in .claude/pipeline/test-report.md.

Evidence (this session):

• 
  pm run build\ in \docs-src: PASSED
• 
  svars.bat\ (Studio 37.0) + \MSBuild\ \Test Delphi/Firebird_tests/PTestFluentSQLFirebird.dproj\ (Debug/Win32): PASSED
• \PTestFluentSQLFirebird.exe\ -b\ -exit:Continue\ -cm:Quiet: 113 tests passed, 0 failed

Caveats (non-blocking):

• .claude/pipeline/implement-report.md\ still reflects only the post-review naming correction, not the full ESP-029 delivery — reconcile for traceability if needed before /develop.
• First MSBuild attempt hit F2039 (output file); resolved by ensuring the output directory existed; final build produced \PTestFluentSQLFirebird.exe\ under \Test Delphi/Firebird_tests.

Kanban: Card moved to QA Testing on Project 16.

Next step: /develop\ (per pipeline).

[isaquepinheiro]

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