@starterlib/eslint-config 0.10.13 → 0.11.0

package/README.md

@@ -24,6 +24,8 @@ export default configurationDuSocle({ prefixe: 'ct', racine: import.meta.dirname
 
 `role: 'hote'` pour la coquille : un remote n'a pas le droit de FOURNIR le noyau, le routeur ou le
 client HTTP (seul l'hote les fournit), l'hote si. `ignores` ajoute des chemins a ne pas lire.
+`variantesReservees` (P08-11) : les variantes que l'application reserve a des emplacements (regle
+`variantes-reservees` du greffon ; Aplomb reserve `stButton variante="danger"` a l'arret d'urgence).
 
 `.stylelintrc.json` :
 
@@ -48,14 +50,20 @@ Chaque regle repond a une ligne du plan SPA §4 ou de l'ADR 0029.
 - Donnees : aucun tri local (`sort`, `toSorted`) dans un fichier qui importe les tableaux du socle
   (`@starterlib/ui/listes`) : il ne trierait que la page chargee ; le tri passe a l'API (lot 224).
 - Gabarits (`@starterlib/eslint-plugin`) : aucun element interactif nu, aucun texte litteral, aucun
-  pipe de locale d'Angular ; code : un titre par route, aucun toast d'erreur, aucun texte libre dans
+  pipe de locale d'Angular, aucun style local (`no-style-literal`, P08-11), aucune variante reservee
+  hors de ses emplacements ; code : un titre par route, aucun toast d'erreur, aucun texte libre dans
   l'adresse.
-- Feuilles (stylelint) : aucune couleur ecrite (un derive de jeton est admis), aucun `px` non nul sur
-  un espacement, une police ou un rayon, aucun `z-index` en nombre, ni `order` ni sens inverse.
+- Feuilles (stylelint) : aucune couleur ecrite (un derive de jeton est admis ; `lab()`, `lch()`,
+  `oklab()`, `oklch()` et `color()` comptent), aucun `px` non nul sur un espacement, une police ou un
+  rayon, aucun `z-index` en nombre, ni `order` ni sens inverse ; P08-11 : un rayon, une ombre et une
+  police viennent d'un jeton `--st-*`, et aucune declaration `--st-*` hors des reglages de niveau 3.
+  Le depot du socle, qui FAIT le design, n'etend que les regles communes (`REGLES_COMMUNES`).
 
 Les essais (`*.spec.ts`) pilotent le DOM et montent des routes et des composants hotes : `window`,
 `document`, les routes sans titre et les gabarits en ligne y sont admis. L'hote de developpement d'un
 remote (`src/hote-dev`, jamais livre) ne parle qu'aux developpeurs : textes litteraux admis.
 
 Une exception s'ecrit sur place, avec sa raison : `// eslint-disable-next-line <regle> -- <raison>`,
-`// stylelint-disable-next-line <regle> -- <raison>`.
+`// stylelint-disable-next-line <regle> -- <raison>` (stylelint refuse une exception sans raison).
+Une exception aux regles de STYLE (P08-11) est aussi DATEE et se journalise dans le CHANGELOG de
+l'application : `-- AAAA-MM-JJ : <raison>` ; l'etape `lint` de `starter-web verify` le verifie.

package/index.mjs

@@ -158,7 +158,9 @@ const FOURNI_PAR_L_HOTE = [
  * `import.meta.dirname` de son `eslint.config.mjs` (le projet TypeScript) ; `role` : `remote` (par
  * defaut) ou `hote` ; `ignores` : des chemins de plus a ne pas lire ; `nomDesElements` : le nom
  * ENREGISTRE sous lequel la SPA definit ses custom elements (`<nom>-*`, lot 299, registre des custom
- * elements du socle frontal), absent tant qu'elle n'en definit aucun.
+ * elements du socle frontal), absent tant qu'elle n'en definit aucun ; `variantesReservees` : les
+ * variantes que l'application reserve a certains emplacements (P08-11, regle `variantes-reservees` ;
+ * Aplomb : `stButton variante="danger"` a l'arret d'urgence seul, DEC-105).
  */
 export function configurationDuSocle({
   prefixe,
@@ -166,6 +168,7 @@ export function configurationDuSocle({
   role = 'remote',
   ignores = [],
   nomDesElements,
+  variantesReservees = [],
 } = {}) {
   if (typeof prefixe !== 'string' || !PREFIXE.test(prefixe) || prefixe === 'st') {
     throw new Error(
@@ -292,6 +295,11 @@ export function configurationDuSocle({
         'socle/no-raw-interactive': 'error',
         'socle/no-literal-text': 'error',
         'socle/no-angular-locale-pipes': 'error',
+        // Lot 281 : l'affichage se decide par une permission (*stPermission), jamais par un role.
+        'socle/no-role-comparison': 'error',
+        // P08-11 : seul le socle fait le design ; les variantes reservees par l'application.
+        'socle/no-style-literal': 'error',
+        'socle/variantes-reservees': ['error', { reservees: variantesReservees }],
       },
     },
     {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@starterlib/eslint-config",
-  "version": "0.10.13",
+  "version": "0.11.0",
   "description": "Configuration ESLint d'une SPA du socle frontal starter, parametree par son prefixe, et configuration stylelint de ses feuilles. Remplace les eslint.config.mjs recopies.",
   "license": "UNLICENSED",
   "type": "module",
@@ -18,7 +18,7 @@
   },
   "peerDependencies": {
     "@eslint/js": "^10.0.1",
-    "@starterlib/eslint-plugin": "^0.10.0",
+    "@starterlib/eslint-plugin": "^0.11.0",
     "angular-eslint": "^22.5.0",
     "eslint": "^10.10.0",
     "typescript-eslint": "^8.70.0"

package/stylelint.mjs

@@ -8,9 +8,10 @@
  * { "extends": ["@starterlib/eslint-config/stylelint"] }
  * ```
  *
- * - aucune couleur ecrite (hexadecimal, nom, `rgb()`, `hsl()`...) : un jeton `--st-*`, qui suit le
- *   theme. Une couleur DERIVEE d'un jeton (`rgb(from var(--st-couleur-primaire) r g b / 10%)`) en est
- *   un usage, pas une invention : elle est admise ;
+ * - aucune couleur ecrite (hexadecimal, nom, `rgb()`, `hsl()`, `hwb()`, `lab()`, `lch()`,
+ *   `oklab()`, `oklch()`, `color()`) : un jeton `--st-*`, qui suit le theme. Une couleur DERIVEE d'un
+ *   jeton (`rgb(from var(--st-couleur-primaire) r g b / 10%)`) en est un usage, pas une invention :
+ *   elle est admise ;
  * - aucun `px` NON NUL sur un espacement, une police ou un rayon : les jetons de rythme, de
  *   typographie et de forme, qui suivent la densite et le zoom du texte. Un `0px` de repli
  *   (`var(--st-hauteur-entete, 0px)`, `env(safe-area-inset-bottom, 0px)` dans un `calc()`, ou un `0`
@@ -19,32 +20,104 @@
  * - ni `order` ni sens inverse (`row-reverse`, `column-reverse`) : l'ordre du DOM est l'ordre visuel,
  *   celui de la lecture et de la tabulation.
  *
- * Une exception s'ecrit sur place, avec sa raison :
- * `// stylelint-disable-next-line <regle> -- <raison>`.
+ * P08-11 (seul le socle fait le design, DEC-100 d'Aplomb), pour une SPA seulement (le depot du
+ * socle, qui fait le design, n'etend que `REGLES_COMMUNES`) : un rayon, une ombre (`box-shadow`,
+ * `text-shadow`, `drop-shadow()`) et une police (`font`, `font-family`) viennent d'un jeton
+ * `--st-*` (ou `0`, `none`, un mot-cle CSS) ; aucune declaration `--st-*` hors des reglages de
+ * niveau 3 (`REGLAGES_DE_NIVEAU_3`). Les gabarits sont gardes par la regle `no-style-literal` de
+ * `@starterlib/eslint-plugin`, avec les memes refus.
+ *
+ * Une exception s'ecrit sur place, nommee et DATEE, et se journalise dans le CHANGELOG de
+ * l'application : `// stylelint-disable-next-line <regle> -- AAAA-MM-JJ : <raison>`.
+ */
+
+/**
+ * Les reglages de niveau 3 qu'une SPA peut poser : la liste du README des jetons (« Niveau 3 ») et le
+ * reglage d'affichage `--st-prefixe-de-ton`. La MEME que `REGLAGES_DE_NIVEAU_3` de
+ * `@starterlib/eslint-plugin`, recopiee : cette configuration se lit aussi dans le depot du socle, ou
+ * le greffon ne se resout pas. Une garde du depot confronte les trois.
  */
+export const REGLAGES_DE_NIVEAU_3 = Object.freeze([
+  'anim-chargement',
+  'anim-respiration',
+  'calendrier-jour-taille',
+  'carte-rayon',
+  'champ-largeur-libelle',
+  'commande-hauteur',
+  'commande-rayon',
+  'coquille-largeur-navigation',
+  'liste-hauteur-max',
+  'prefixe-de-ton',
+  'separateur-marge',
+  'surface-rayon',
+  'tableau-cellule-bloc',
+  'tableau-cellule-ligne',
+  'tableau-largeur-case',
+]);
 
 /** Une fonction de couleur qui n'est pas derivee d'un jeton du socle. */
-const COULEUR_ECRITE = '/(^|[^-\\w])(rgba?|hsla?|hwb)\\((?!\\s*from\\s+var\\(--st-)/';
+const COULEUR_ECRITE =
+  '/(^|[^-\\w])(rgba?|hsla?|hwb|lab|lch|oklab|oklch|color)\\((?!\\s*from\\s+var\\(--st-)/';
 
 /** Une mesure en pixels non nulle, negative comprise. */
 const PIXELS_NON_NULS = '/(^|[^\\w.])-?(?!0px\\b)(\\d*\\.)?\\d+px\\b/';
 
-export default {
-  rules: {
-    'color-no-hex': true,
-    'color-named': 'never',
-    'declaration-property-value-disallowed-list': {
-      '/.*/': [COULEUR_ECRITE],
-      '/^(margin|padding|gap|row-gap|column-gap|inset)/': [PIXELS_NON_NULS],
-      '/^(font-size|line-height|letter-spacing|word-spacing)$/': [PIXELS_NON_NULS],
-      '/^border(-(start|end)-(start|end)|-(top|bottom)-(left|right))?-radius$/': [PIXELS_NON_NULS],
-      'flex-direction': ['/reverse/'],
-      'flex-flow': ['/reverse/'],
-    },
-    'declaration-property-value-allowed-list': {
-      'z-index': ['/^var\\(--st-/', '/^calc\\(var\\(--st-/', 'auto', '0'],
-    },
-    'property-disallowed-list': ['order'],
+/** Une ombre portee par un filtre, ecrite plutot que tiree d'un jeton. */
+const OMBRE_DE_FILTRE = '/drop-shadow\\((?!\\s*var\\(--st-)/';
+
+/** Une valeur tiree d'un jeton du socle, ou un mot-cle CSS. */
+const DE_JETON = [
+  '/^var\\(\\s*--st-/',
+  '/^calc\\(.*var\\(\\s*--st-/',
+  'inherit',
+  'initial',
+  'unset',
+  'revert',
+];
+
+/** Un rayon, quel que soit le coin. */
+const RAYON = '/^border(-(start|end)-(start|end)|-(top|bottom)-(left|right))?-radius$/';
+
+/** Une declaration `--st-*` qui n'est pas un reglage de niveau 3. */
+const JETON_REDEFINI = `/^--st-(?!(?:${REGLAGES_DE_NIVEAU_3.join('|')})$)/`;
+
+/** Les regles du socle ET des SPA : couleurs, pixels, plans, ordre du DOM. */
+export const REGLES_COMMUNES = Object.freeze({
+  'color-no-hex': true,
+  'color-named': 'never',
+  'declaration-property-value-disallowed-list': {
+    '/.*/': [COULEUR_ECRITE],
+    '/^(margin|padding|gap|row-gap|column-gap|inset)/': [PIXELS_NON_NULS],
+    '/^(font-size|line-height|letter-spacing|word-spacing)$/': [PIXELS_NON_NULS],
+    [RAYON]: [PIXELS_NON_NULS],
+    'flex-direction': ['/reverse/'],
+    'flex-flow': ['/reverse/'],
+  },
+  'declaration-property-value-allowed-list': {
+    'z-index': ['/^var\\(--st-/', '/^calc\\(var\\(--st-/', 'auto', '0'],
+  },
+  'property-disallowed-list': ['order'],
+});
+
+/** P08-11 : ce qu'une SPA n'ecrit jamais, la ou le socle fait le design. */
+const REGLES_DES_SPA = {
+  ...REGLES_COMMUNES,
+  'declaration-property-value-disallowed-list': {
+    ...REGLES_COMMUNES['declaration-property-value-disallowed-list'],
+    '/^(filter|backdrop-filter)$/': [OMBRE_DE_FILTRE],
   },
+  'declaration-property-value-allowed-list': {
+    ...REGLES_COMMUNES['declaration-property-value-allowed-list'],
+    [RAYON]: [...DE_JETON, '0'],
+    '/^(box-shadow|text-shadow)$/': [...DE_JETON, 'none'],
+    '/^(font|font-family)$/': DE_JETON,
+  },
+  'property-disallowed-list': [...REGLES_COMMUNES['property-disallowed-list'], JETON_REDEFINI],
+};
+
+export default {
+  rules: REGLES_DES_SPA,
+  // Une exception sans raison ne se lit pas a la revue.
+  reportDescriptionlessDisables: true,
   overrides: [{ files: ['**/*.scss'], customSyntax: 'postcss-scss' }],
 };