fix(clients): aligne adresse structurée sur convention Lucid snakeCase

La migration 1778800000100_enrich_clients_for_invoicing avait créé les
colonnes `address_line1` et `address_line2` (sans underscore avant le
chiffre), mais Lucid v22 auto-snake-case `addressLine1` → `address_line_1`
côté écriture. Résultat : tous les INSERT dans `clients` cassaient avec
`column "address_line_1" of relation "clients" does not exist`.

Bug latent — surfacé en lançant la suite functional complète après
`node ace migration:run`. Affectait 20 tests Clients/Dashboard/Invoices.

Fix : migration de rename qui aligne `address_line1` → `address_line_1`
et `address_line2` → `address_line_2`. Le `RENAME COLUMN` préserve les
données existantes en prod. Modèle Client simplifié (les déclarations
manuelles ne sont plus nécessaires depuis que `schema.ts` les a régen).

Bonus : fix du test `invoices.spec.ts` "liste paginée + meta total/page"
qui créait 3 factures, ce qui dépasse la nouvelle limite Free 2 (ADR-023).
Posé un `gracePeriodEndsAt` futur sur l'org du test pour bypass le quota.

État après ce commit : 127 tests verts (60 unit + 67 functional).

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

apps/api/app/models/client.ts

@@ -6,36 +6,14 @@ import Invoice from '#models/invoice'
 
 export default class Client extends ClientSchema {
   /**
-   * Champs ajoutés par la migration `1778800000100_enrich_clients_for_invoicing`
+   * Le champ `address` (existant, string libre) est conservé pour les
-   * (SIREN/TVA intra/adresse structurée). Déclarations manuelles en attendant
+   * clients importés avant l'éditeur natif. Les champs structurés
-   * que `schema.ts` soit régénéré par `node ace migration:run`.
+   * (`address_line_1`, `address_zip`, etc.) viennent de la migration
-   *
+   * `1778800000100_enrich_clients_for_invoicing` + alignement Lucid via
-   * Le champ `address` (existant, string libre) est conservé pour les clients
+   * `1779000000100_rename_client_address_line_columns`. Le nouveau code
-   * importés avant la feature ; le nouveau code lit en priorité ces champs
+   * lit en priorité les champs structurés et retombe sur `address` s'ils
-   * structurés et retombe sur `address` s'ils sont vides.
+   * sont vides.
    */
-  @column()
-  declare siren: string | null
-
-  @column()
-  declare tvaIntra: string | null
-
-  @column()
-  declare addressLine1: string | null
-
-  @column()
-  declare addressLine2: string | null
-
-  @column()
-  declare addressZip: string | null
-
-  @column()
-  declare addressCity: string | null
-
-  /** ISO 3166-1 alpha-2 (ex. "FR"). */
-  @column()
-  declare addressCountry: string | null
-
   @belongsTo(() => Organization)
   declare organization: BelongsTo<typeof Organization>
 

apps/api/database/migrations/1779000000100_rename_client_address_line_columns.ts

@@ -0,0 +1,32 @@
+import { BaseSchema } from '@adonisjs/lucid/schema'
+
+/**
+ * Renomme `address_line1` → `address_line_1` (et idem line2) pour aligner
+ * avec la convention Lucid snakeCase (Adonis v7 traite les digits comme
+ * des tokens séparés : `addressLine1` → `address_line_1`).
+ *
+ * La migration `1778800000100_enrich_clients_for_invoicing_table` avait
+ * créé les colonnes sans underscore avant le chiffre, ce qui causait des
+ * INSERT en erreur (`column "address_line_1" of relation "clients" does
+ * not exist`) puisque Lucid auto-snake-case sur write. Cette migration
+ * corrige rétroactivement.
+ *
+ * Aucune perte de donnée — `RENAME COLUMN` préserve le contenu.
+ */
+export default class extends BaseSchema {
+  protected tableName = 'clients'
+
+  async up() {
+    this.schema.alterTable(this.tableName, (table) => {
+      table.renameColumn('address_line1', 'address_line_1')
+      table.renameColumn('address_line2', 'address_line_2')
+    })
+  }
+
+  async down() {
+    this.schema.alterTable(this.tableName, (table) => {
+      table.renameColumn('address_line_1', 'address_line1')
+      table.renameColumn('address_line_2', 'address_line2')
+    })
+  }
+}

apps/api/tests/functional/invoices.spec.ts

@@ -281,7 +281,12 @@ test.group('Invoices — GET /invoices', (group) => {
   group.each.setup(() => testUtils.db().withGlobalTransaction())
 
   test('liste paginée + meta total/page', async ({ client, assert }) => {
-    const { bearer } = await createTestUser()
+    const { bearer, org } = await createTestUser()
+    // Grace period pour bypass le quota Free 2 factures (cf. ADR-023).
+    // Sans ça, la 3e facture serait rejetée en 402 et le test verrait
+    // meta.total=2 au lieu de 3.
+    org.gracePeriodEndsAt = DateTime.utc().plus({ months: 2 })
+    await org.save()
     const c = await createClient(client, bearer, 'Liste Spec')
 
     for (let i = 0; i < 3; i++) {