Infrastructure/eqs-platform
4
0
Fork 0
Code Issues 4 Pull requests 1 Projects Releases Packages Wiki Activity Actions

next.validation échoue sur la route de conformité que le socle monte lui-même (conformite.valider n'est pas .strict()) #159

New issue
Closed
opened 2026-10-15 08:09:29 +00:00 by nicolas-fryder · 0 comments
nicolas-fryder commented 2026-10-15 08:09:29 +00:00
Owner
Copy link

Ce qui ne va pas

routesDeConformiteNext (packages/platform-next/src/testing/routes-conformite.ts, #149) déclare le corps de POST __eqs/conformite/valider en z.object({ nom: z.string(), quantite: z.number() }) sans .strict(). Le contrôle next.validation (packages/platform-next/src/conformite.ts, autour de la ligne 317) envoie ce corps avec champQuiNExistePas: 1 et exige 422 : la route du socle rend 200. La règle ne peut donc jamais passer dans un satellite next-fullstack qui s'appuie sur les routes de conformité du socle, c'est-à-dire sur ce que #149 promettait de rendre jouable.

Toujours présent sur main en 0.24.0 (même ligne 54).

Reproduire

Le gabarit eqs-satellite-template, variante next-fullstack, @eqs/* 0.23.0 (eqs --version → 0.23.0) :

  1. Poser la variante sur un clone (variantes/next-fullstack/README.md, « Le choix se fait à l'étape 2 »), puis pnpm install.
  2. Retirer la dispense next.validation d'eqs-baseline.json.
  3. pnpm --filter web test:integration
 FAIL  test/integration/conformite.test.ts > conformité @eqs (profil next-fullstack) > next
AssertionError:
  [bloquant] next.validation (POST /api/__eqs/conformite/valider)
    un champ inconnu rend 200, pas 422
    → body: z.object({ … }).strict() dans withApi

La suite du gabarit ne fournit pas de routeValidation à elle : c'est demarrerRoutes qui remplit routeValidation, corpsValide et champNombre à partir de a.conformite (testing/index.ts, vers la ligne 144).

Pourquoi eqs-demo ne le voit pas : sa suite passe sa propre route (routeValidation: '/api/demo/valider'), qui est stricte. Le chemin « routes de conformité du socle seules » n'est joué par personne.

Le message est d'ailleurs trompeur : il dit au satellite d'écrire .strict() dans son withApi, alors que la route fautive est celle du socle, que le satellite ne peut pas modifier.

Ce que ça coûte

Contournable, pas bloquant : la variante du gabarit garde la dispense datée next.validation (jusqu'au 2026-12-31), dont la raison pointe désormais vers ce ticket. Mais ce que la dispense couvre n'est plus « le profil n'a pas la route » : c'est « la route du socle est fausse ». Tout satellite next-fullstack neuf hérite de la dispense, ou d'un rouge s'il la retire en croyant #149 livré. Environ une demi-heure pour remonter du rouge au fichier.

Ce que je proposerais

  1. .strict() sur le corps de conformite.valider : 
    body: z.object({ nom: z.string(), quantite: z.number() }).strict(),
  2. Un test du socle qui joue next.validation (et errors.validation) par demarrerRoutes sans routeValidation fournie par l'application, c'est-à-dire contre les seules routes de conformité. C'est le trou qui a laissé passer le défaut.
  3. Accessoirement : quand la route jugée est __eqs/conformite/*, que le faire du message dise que le défaut est dans le socle et pas chez le satellite.

Contournement, à retirer après correction

eqs-satellite-template, variantes/next-fullstack/eqs-baseline.json : dispense next.validation, raison renvoyant à ce ticket. À retirer à la montée qui corrige.

## Ce qui ne va pas `routesDeConformiteNext` (`packages/platform-next/src/testing/routes-conformite.ts`, #149) déclare le corps de `POST __eqs/conformite/valider` en `z.object({ nom: z.string(), quantite: z.number() })` **sans `.strict()`**. Le contrôle `next.validation` (`packages/platform-next/src/conformite.ts`, autour de la ligne 317) envoie ce corps avec `champQuiNExistePas: 1` et exige 422 : la route du socle rend 200. La règle **ne peut donc jamais passer** dans un satellite next-fullstack qui s'appuie sur les routes de conformité du socle, c'est-à-dire sur ce que #149 promettait de rendre jouable. Toujours présent sur `main` en 0.24.0 (même ligne 54). ## Reproduire Le gabarit `eqs-satellite-template`, variante next-fullstack, `@eqs/*` 0.23.0 (`eqs --version` → `0.23.0`) : 1. Poser la variante sur un clone (`variantes/next-fullstack/README.md`, « Le choix se fait à l'étape 2 »), puis `pnpm install`. 2. Retirer la dispense `next.validation` d'`eqs-baseline.json`. 3. `pnpm --filter web test:integration` ``` FAIL test/integration/conformite.test.ts > conformité @eqs (profil next-fullstack) > next AssertionError: [bloquant] next.validation (POST /api/__eqs/conformite/valider) un champ inconnu rend 200, pas 422 → body: z.object({ … }).strict() dans withApi ``` La suite du gabarit ne fournit pas de `routeValidation` à elle : c'est `demarrerRoutes` qui remplit `routeValidation`, `corpsValide` et `champNombre` à partir de `a.conformite` (`testing/index.ts`, vers la ligne 144). **Pourquoi eqs-demo ne le voit pas** : sa suite passe sa propre route (`routeValidation: '/api/demo/valider'`), qui est stricte. Le chemin « routes de conformité du socle seules » n'est joué par personne. Le message est d'ailleurs trompeur : il dit au satellite d'écrire `.strict()` dans son `withApi`, alors que la route fautive est celle du socle, que le satellite ne peut pas modifier. ## Ce que ça coûte Contournable, pas bloquant : la variante du gabarit garde la dispense datée `next.validation` (jusqu'au 2026-12-31), dont la raison pointe désormais vers ce ticket. Mais ce que la dispense couvre n'est plus « le profil n'a pas la route » : c'est « la route du socle est fausse ». Tout satellite next-fullstack neuf hérite de la dispense, ou d'un rouge s'il la retire en croyant #149 livré. Environ une demi-heure pour remonter du rouge au fichier. ## Ce que je proposerais 1. `.strict()` sur le corps de `conformite.valider` : ```ts body: z.object({ nom: z.string(), quantite: z.number() }).strict(), ``` 2. Un test du socle qui joue `next.validation` (et `errors.validation`) par `demarrerRoutes` **sans** `routeValidation` fournie par l'application, c'est-à-dire contre les seules routes de conformité. C'est le trou qui a laissé passer le défaut. 3. Accessoirement : quand la route jugée est `__eqs/conformite/*`, que le `faire` du message dise que le défaut est dans le socle et pas chez le satellite. ## Contournement, à retirer après correction `eqs-satellite-template`, `variantes/next-fullstack/eqs-baseline.json` : dispense `next.validation`, raison renvoyant à ce ticket. À retirer à la montée qui corrige.
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
dev
@eqs/tooling@0.24.3
@eqs/testing@0.24.3
@eqs/telemetry@0.24.3
@eqs/platform-next@0.24.3
@eqs/platform-nest@0.24.3
@eqs/http@0.24.3
@eqs/errors@0.24.3
@eqs/config@0.24.3
@eqs/cli@0.24.3
@eqs/auth@0.24.3
@eqs/tooling@0.24.2
@eqs/testing@0.24.2
@eqs/telemetry@0.24.2
@eqs/platform-next@0.24.2
@eqs/platform-nest@0.24.2
@eqs/http@0.24.2
@eqs/errors@0.24.2
@eqs/config@0.24.2
@eqs/cli@0.24.2
@eqs/auth@0.24.2
@eqs/tooling@0.24.1
@eqs/testing@0.24.1
@eqs/telemetry@0.24.1
@eqs/platform-next@0.24.1
@eqs/platform-nest@0.24.1
@eqs/http@0.24.1
@eqs/errors@0.24.1
@eqs/config@0.24.1
@eqs/cli@0.24.1
@eqs/auth@0.24.1
@eqs/tooling@0.24.0
@eqs/testing@0.24.0
@eqs/telemetry@0.24.0
@eqs/platform-next@0.24.0
@eqs/platform-nest@0.24.0
@eqs/http@0.24.0
@eqs/errors@0.24.0
@eqs/config@0.24.0
@eqs/cli@0.24.0
@eqs/auth@0.24.0
@eqs/tooling@0.23.0
@eqs/testing@0.23.0
@eqs/telemetry@0.23.0
@eqs/platform-next@0.23.0
@eqs/platform-nest@0.23.0
@eqs/http@0.23.0
@eqs/errors@0.23.0
@eqs/config@0.23.0
@eqs/cli@0.23.0
@eqs/auth@0.23.0
@eqs/tooling@0.22.0
@eqs/testing@0.22.0
@eqs/telemetry@0.22.0
@eqs/platform-next@0.22.0
@eqs/platform-nest@0.22.0
@eqs/http@0.22.0
@eqs/errors@0.22.0
@eqs/config@0.22.0
@eqs/cli@0.22.0
@eqs/auth@0.22.0
@eqs/tooling@0.21.0
@eqs/testing@0.21.0
@eqs/telemetry@0.21.0
@eqs/platform-next@0.21.0
@eqs/platform-nest@0.21.0
@eqs/http@0.21.0
@eqs/errors@0.21.0
@eqs/config@0.21.0
@eqs/cli@0.21.0
@eqs/auth@0.21.0
@eqs/tooling@0.20.0
@eqs/testing@0.20.0
@eqs/telemetry@0.20.0
@eqs/platform-next@0.20.0
@eqs/platform-nest@0.20.0
@eqs/http@0.20.0
@eqs/errors@0.20.0
@eqs/config@0.20.0
@eqs/cli@0.20.0
@eqs/auth@0.20.0
@eqs/tooling@0.19.0
@eqs/testing@0.19.0
@eqs/telemetry@0.19.0
@eqs/platform-next@0.19.0
@eqs/platform-nest@0.19.0
@eqs/http@0.19.0
@eqs/errors@0.19.0
@eqs/config@0.19.0
@eqs/cli@0.19.0
@eqs/auth@0.19.0
@eqs/tooling@0.17.1
@eqs/testing@0.17.1
@eqs/telemetry@0.17.1
@eqs/platform-next@0.17.1
@eqs/platform-nest@0.17.1
@eqs/http@0.17.1
@eqs/errors@0.17.1
@eqs/config@0.17.1
@eqs/cli@0.17.1
@eqs/auth@0.17.1
@eqs/tooling@0.17.0
@eqs/testing@0.17.0
@eqs/telemetry@0.17.0
@eqs/platform-next@0.17.0
@eqs/platform-nest@0.17.0
@eqs/http@0.17.0
@eqs/errors@0.17.0
@eqs/config@0.17.0
@eqs/cli@0.17.0
@eqs/auth@0.17.0
@eqs/tooling@0.16.0
@eqs/testing@0.16.0
@eqs/telemetry@0.16.0
@eqs/platform-next@0.16.0
@eqs/platform-nest@0.16.0
@eqs/http@0.16.0
@eqs/errors@0.16.0
@eqs/config@0.16.0
@eqs/cli@0.16.0
@eqs/auth@0.16.0
@eqs/tooling@0.15.0
@eqs/testing@0.15.0
@eqs/telemetry@0.15.0
@eqs/platform-next@0.15.0
@eqs/platform-nest@0.15.0
@eqs/http@0.15.0
@eqs/errors@0.15.0
@eqs/config@0.15.0
@eqs/cli@0.15.0
@eqs/auth@0.15.0
@eqs/tooling@0.14.0
@eqs/testing@0.14.0
@eqs/telemetry@0.14.0
@eqs/platform-next@0.14.0
@eqs/platform-nest@0.14.0
@eqs/http@0.14.0
@eqs/errors@0.14.0
@eqs/config@0.14.0
@eqs/cli@0.14.0
@eqs/auth@0.14.0
@eqs/tooling@0.13.0
@eqs/testing@0.13.0
@eqs/telemetry@0.13.0
@eqs/platform-next@0.13.0
@eqs/platform-nest@0.13.0
@eqs/http@0.13.0
@eqs/errors@0.13.0
@eqs/config@0.13.0
@eqs/cli@0.13.0
@eqs/auth@0.13.0
@eqs/tooling@0.12.2
@eqs/testing@0.12.2
@eqs/telemetry@0.12.2
@eqs/platform-next@0.12.2
@eqs/platform-nest@0.12.2
@eqs/http@0.12.2
@eqs/errors@0.12.2
@eqs/config@0.12.2
@eqs/cli@0.12.2
@eqs/auth@0.12.2
@eqs/tooling@0.12.1
@eqs/testing@0.12.1
@eqs/telemetry@0.12.1
@eqs/platform-next@0.12.1
@eqs/platform-nest@0.12.1
@eqs/http@0.12.1
@eqs/errors@0.12.1
@eqs/config@0.12.1
@eqs/cli@0.12.1
@eqs/auth@0.12.1
@eqs/tooling@0.12.0
@eqs/testing@0.12.0
@eqs/telemetry@0.12.0
@eqs/platform-next@0.12.0
@eqs/platform-nest@0.12.0
@eqs/http@0.12.0
@eqs/errors@0.12.0
@eqs/config@0.12.0
@eqs/cli@0.12.0
@eqs/auth@0.12.0
@eqs/tooling@0.11.2
@eqs/testing@0.11.2
@eqs/telemetry@0.11.2
@eqs/platform-next@0.11.2
@eqs/platform-nest@0.11.2
@eqs/http@0.11.2
@eqs/errors@0.11.2
@eqs/config@0.11.2
@eqs/cli@0.11.2
@eqs/auth@0.11.2
@eqs/tooling@0.11.1
@eqs/testing@0.11.1
@eqs/telemetry@0.11.1
@eqs/platform-next@0.11.1
@eqs/platform-nest@0.11.1
@eqs/http@0.11.1
@eqs/errors@0.11.1
@eqs/config@0.11.1
@eqs/cli@0.11.1
@eqs/auth@0.11.1
@eqs/tooling@0.11.0
@eqs/testing@0.11.0
@eqs/telemetry@0.11.0
@eqs/platform-next@0.11.0
@eqs/platform-nest@0.11.0
@eqs/http@0.11.0
@eqs/errors@0.11.0
@eqs/config@0.11.0
@eqs/cli@0.11.0
@eqs/auth@0.11.0
@eqs/tooling@0.10.1
@eqs/testing@0.10.1
@eqs/telemetry@0.10.1
@eqs/platform-next@0.10.1
@eqs/platform-nest@0.10.1
@eqs/http@0.10.1
@eqs/errors@0.10.1
@eqs/config@0.10.1
@eqs/cli@0.10.1
@eqs/auth@0.10.1
@eqs/tooling@0.10.0
@eqs/testing@0.10.0
@eqs/telemetry@0.10.0
@eqs/platform-next@0.10.0
@eqs/platform-nest@0.10.0
@eqs/http@0.10.0
@eqs/errors@0.10.0
@eqs/config@0.10.0
@eqs/cli@0.10.0
@eqs/auth@0.10.0
@eqs/tooling@0.9.0
@eqs/testing@0.9.0
@eqs/telemetry@0.9.0
@eqs/platform-next@0.9.0
@eqs/platform-nest@0.9.0
@eqs/http@0.9.0
@eqs/errors@0.9.0
@eqs/config@0.9.0
@eqs/cli@0.9.0
@eqs/auth@0.9.0
@eqs/tooling@0.8.1
@eqs/testing@0.8.1
@eqs/telemetry@0.8.1
@eqs/platform-next@0.8.1
@eqs/platform-nest@0.8.1
@eqs/http@0.8.1
@eqs/errors@0.8.1
@eqs/config@0.8.1
@eqs/cli@0.8.1
@eqs/auth@0.8.1
@eqs/tooling@0.8.0
@eqs/testing@0.8.0
@eqs/telemetry@0.8.0
@eqs/platform-next@0.8.0
@eqs/platform-nest@0.8.0
@eqs/http@0.8.0
@eqs/errors@0.8.0
@eqs/config@0.8.0
@eqs/cli@0.8.0
@eqs/auth@0.8.0
@eqs/tooling@0.7.0
@eqs/testing@0.7.0
@eqs/telemetry@0.7.0
@eqs/platform-next@0.7.0
@eqs/platform-nest@0.7.0
@eqs/http@0.7.0
@eqs/errors@0.7.0
@eqs/config@0.7.0
@eqs/cli@0.7.0
@eqs/auth@0.7.0
@eqs/tooling@0.6.0
@eqs/testing@0.6.0
@eqs/telemetry@0.6.0
@eqs/platform-next@0.6.0
@eqs/platform-nest@0.6.0
@eqs/http@0.6.0
@eqs/errors@0.6.0
@eqs/config@0.6.0
@eqs/cli@0.6.0
@eqs/auth@0.6.0
@eqs/tooling@0.5.1
@eqs/testing@0.5.1
@eqs/telemetry@0.5.1
@eqs/platform-next@0.5.1
@eqs/platform-nest@0.5.1
@eqs/http@0.5.1
@eqs/errors@0.5.1
@eqs/config@0.5.1
@eqs/cli@0.5.1
@eqs/auth@0.5.1
@eqs/tooling@0.5.0
@eqs/testing@0.5.0
@eqs/telemetry@0.5.0
@eqs/platform-next@0.5.0
@eqs/platform-nest@0.5.0
@eqs/http@0.5.0
@eqs/errors@0.5.0
@eqs/config@0.5.0
@eqs/cli@0.5.0
@eqs/auth@0.5.0
@eqs/tooling@0.4.0
@eqs/testing@0.4.0
@eqs/telemetry@0.4.0
@eqs/platform-next@0.4.0
@eqs/platform-nest@0.4.0
@eqs/http@0.4.0
@eqs/errors@0.4.0
@eqs/config@0.4.0
@eqs/cli@0.4.0
@eqs/auth@0.4.0
@eqs/tooling@0.2.0
@eqs/telemetry@0.2.0
@eqs/errors@0.2.0
@eqs/config@0.2.0
@eqs/tooling@0.1.0
Labels
Clear labels   
bug
Something is not working

  
duplicate
This issue or pull request already exists

  
enhancement
New feature

  
help wanted
Need some help

  
invalid
Something is wrong

  
question
More information is needed

  
wontfix
This won't be fixed

No labels
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Infrastructure/eqs-platform#159
No description provided.