@ouestfrance/sipa-bms-ui 8.44.0 → 8.45.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ouestfrance/sipa-bms-ui",
-  "version": "8.44.0",
+  "version": "8.45.0",
   "author": "Ouest-France BMS",
   "license": "ISC",
   "scripts": {

package/src/components/form/bms-input-code.linters.spec.ts

@@ -32,11 +32,22 @@ describe('htmlDiagnostics', () => {
     expect(diags[0].severity).toBe('error');
   });
 
-  it("positionne le diagnostic sur l'intégralité du contenu", () => {
+  it("positionne le diagnostic à proximité de l'erreur", () => {
+    // <div><p>Hello</div> — l'erreur est sur le </div> qui ferme mal le <p>
     const code = '<div><p>Hello</div>';
     const diags = htmlDiagnostics(code);
-    expect(diags[0].from).toBe(0);
-    expect(diags[0].to).toBe(code.length);
+    expect(diags[0].from).toBeGreaterThanOrEqual(0);
+    expect(diags[0].to).toBeGreaterThan(diags[0].from);
+    // le diagnostic doit pointer dans le contenu, pas sur l'intégralité
+    expect(diags[0].to).toBeLessThanOrEqual(code.length);
+  });
+
+  it('positionne le diagnostic sur la ligne correcte pour une erreur multi-lignes', () => {
+    const code = '<div>\n  <p>Hello\n</div>';
+    const diags = htmlDiagnostics(code);
+    expect(diags).toHaveLength(1);
+    // l'erreur est après le début du document
+    expect(diags[0].from).toBeGreaterThanOrEqual(0);
   });
 });
 

package/src/components/form/bms-input-code.linters.ts

@@ -16,16 +16,36 @@ import type { Extension } from '@codemirror/state';
  * Limitation : le parser XML est plus strict que le parser HTML du navigateur.
  * Les balises void HTML5 non fermées (`<br>`, `<img>`) sont considérées invalides.
  */
+const ROOT_TAG = '<root>';
+
+const htmlErrorOffset = (errorText: string, code: string): number => {
+  const match = errorText.match(/error on line (\d+) at column (\d+)/i);
+  if (!match) return 0;
+
+  const line = parseInt(match[1]);
+  // La colonne de la ligne 1 inclut le préfixe <root> — on le retire
+  const col = parseInt(match[2]) - (line === 1 ? ROOT_TAG.length : 0);
+
+  const lines = code.split('\n');
+  let offset = 0;
+  for (let i = 0; i < line - 1 && i < lines.length; i++) {
+    offset += lines[i].length + 1;
+  }
+  return Math.min(Math.max(0, offset + col - 1), code.length);
+};
+
 export const htmlDiagnostics = (code: string): Diagnostic[] => {
   if (!code.trim()) return [];
   const parser = new DOMParser();
-  const doc = parser.parseFromString(`<root>${code}</root>`, 'text/xml');
+  const doc = parser.parseFromString(`${ROOT_TAG}${code}</root>`, 'text/xml');
   const parserError = doc.querySelector('parsererror');
   if (!parserError) return [];
+
+  const from = htmlErrorOffset(parserError.textContent ?? '', code);
   return [
     {
-      from: 0,
-      to: code.length,
+      from,
+      to: Math.min(from + 1, code.length),
       severity: 'error',
       message: 'HTML invalide : balise mal formée ou non fermée',
     },