@consilioweb/payload-support 5.0.0 → 6.0.1

package/scripts/uninstall.mjs

@@ -0,0 +1,184 @@
+#!/usr/bin/env node
+
+/**
+ * `support-uninstall` — removes this plugin from a host application.
+ *
+ * Contract, deliberately identical to the other ConsilioWEB plugins:
+ *   - DRY RUN by default. Nothing is written without `--confirm`.
+ *   - a detection gate: the destructive step refuses to run unless this project
+ *     really uses the plugin (a source file references it, or it is a declared
+ *     dependency). `--force-db` overrides it.
+ *   - `--keep-data` cleans the code and the preferences but keeps every row.
+ *
+ * Usage:
+ *   npx support-uninstall                       list what would be removed
+ *   npx support-uninstall --confirm             do it
+ *   npx support-uninstall --confirm --keep-data code only, rows kept
+ *   npx support-uninstall --confirm --force-db  delete rows even if undetected
+ *   npx support-uninstall --extra-slugs=a,b     add renamed collections
+ */
+
+import fs from 'node:fs'
+import path from 'node:path'
+import { execSync, spawnSync } from 'node:child_process'
+import { fileURLToPath } from 'node:url'
+
+const HERE = path.dirname(fileURLToPath(import.meta.url))
+
+function readOwnPackageName() {
+  try {
+    const pkg = JSON.parse(fs.readFileSync(path.join(HERE, '..', 'package.json'), 'utf-8'))
+    if (typeof pkg.name === 'string' && pkg.name) return pkg.name
+  } catch {
+    // fall through
+  }
+  return '@consilioweb/payload-support'
+}
+
+const PACKAGE_NAME = readOwnPackageName()
+
+function detectPackageManager(dir) {
+  if (fs.existsSync(path.join(dir, 'pnpm-lock.yaml'))) return 'pnpm'
+  if (fs.existsSync(path.join(dir, 'yarn.lock'))) return 'yarn'
+  if (fs.existsSync(path.join(dir, 'bun.lockb')) || fs.existsSync(path.join(dir, 'bun.lock'))) return 'bun'
+  return 'npm'
+}
+
+function findSourceFiles(dir, acc = []) {
+  if (!fs.existsSync(dir)) return acc
+  for (const entry of fs.readdirSync(dir, { withFileTypes: true })) {
+    const full = path.join(dir, entry.name)
+    if (entry.isDirectory()) {
+      if (entry.name === 'node_modules' || entry.name === '.next' || entry.name.startsWith('.')) continue
+      findSourceFiles(full, acc)
+    } else if (/\.(?:ts|tsx|js|jsx|mjs|cjs)$/.test(entry.name)) {
+      acc.push(full)
+    }
+  }
+  return acc
+}
+
+function isDeclaredDependency(projectDir) {
+  try {
+    const pkg = JSON.parse(fs.readFileSync(path.join(projectDir, 'package.json'), 'utf-8'))
+    return Boolean(
+      pkg.dependencies?.[PACKAGE_NAME]
+      || pkg.devDependencies?.[PACKAGE_NAME]
+      || pkg.peerDependencies?.[PACKAGE_NAME]
+      || pkg.optionalDependencies?.[PACKAGE_NAME],
+    )
+  } catch {
+    return false
+  }
+}
+
+function run(cmd, cwd) {
+  console.log(`  $ ${cmd}`)
+  try {
+    execSync(cmd, { cwd, stdio: 'inherit' })
+    return true
+  } catch {
+    return false
+  }
+}
+
+function main() {
+  const projectDir = process.env.INIT_CWD || process.cwd()
+  const argv = process.argv.slice(2)
+  const confirm = argv.includes('--confirm')
+  const keepData = argv.includes('--keep-data')
+  const forceDb = argv.includes('--force-db')
+  const extraSlugs = argv.find((a) => a.startsWith('--extra-slugs='))
+  const pm = detectPackageManager(projectDir)
+
+  console.log('')
+  console.log(`  ${PACKAGE_NAME} — uninstall`)
+  console.log('  ─────────────────────────────────────────────')
+  console.log(`  Project:         ${projectDir}`)
+  console.log(`  Package manager: ${pm}`)
+  console.log(`  Mode:            ${confirm ? 'APPLY' : 'DRY RUN — nothing will be written'}`)
+  if (keepData) console.log('  --keep-data:     rows will be kept')
+  console.log('')
+
+  // ── 1. Where is the plugin referenced? ───────────────────
+  const sourceFiles = findSourceFiles(path.join(projectDir, 'src'))
+  const referencing = sourceFiles.filter((file) => {
+    try {
+      return fs.readFileSync(file, 'utf-8').includes(PACKAGE_NAME)
+    } catch {
+      return false
+    }
+  })
+
+  console.log('  [1/3] Source references')
+  if (referencing.length === 0) {
+    console.log('    none found under src/')
+  } else {
+    for (const file of referencing) console.log(`    ${path.relative(projectDir, file)}`)
+    console.log('')
+    console.log('    Remove the `supportPlugin(...)` call and its import yourself.')
+    console.log('    This script never rewrites your source: the plugin call carries')
+    console.log('    your configuration, and a regex that guesses where it ends is how')
+    console.log('    a config file gets silently corrupted.')
+  }
+  console.log('')
+
+  const declared = isDeclaredDependency(projectDir)
+  const detected = referencing.length > 0 || declared || forceDb
+
+  // ── 2. Data ──────────────────────────────────────────────
+  console.log('  [2/3] Data')
+  if (!detected) {
+    console.log(`    ${PACKAGE_NAME} was not found in this project`)
+    console.log('    (no source reference, not in package.json dependencies).')
+    console.log('    Skipping the data step. Force it with --force-db.')
+    console.log('')
+  } else {
+    // The Payload local API needs the host's TypeScript config loaded, which is
+    // exactly what `payload run` sets up. Spawning it beats reimplementing a
+    // TypeScript loader here.
+    const dataScript = path.join(HERE, 'uninstall-data.mjs')
+    const args = ['payload', 'run', dataScript]
+    if (confirm) args.push('--confirm')
+    if (keepData) args.push('--keep-data')
+    if (extraSlugs) args.push(extraSlugs)
+
+    const result = spawnSync('npx', args, { cwd: projectDir, stdio: 'inherit' })
+    if (result.status !== 0) {
+      console.log('')
+      console.log('    Could not run the data step through `npx payload run`.')
+      console.log('    Run it by hand from your project root:')
+      console.log(`      npx payload run ${dataScript} ${confirm ? '--confirm' : ''}`)
+      console.log('')
+    }
+  }
+
+  // ── 3. The package ───────────────────────────────────────
+  console.log('  [3/3] Package')
+  if (!confirm) {
+    const removeCmd = pm === 'npm' ? 'npm uninstall' : `${pm} remove`
+    console.log(`    would run: ${removeCmd} ${PACKAGE_NAME}`)
+    console.log(`    would run: ${pm === 'npm' ? 'npx' : pm} payload generate:importmap`)
+    console.log('')
+    console.log('  Dry run finished. Nothing was changed. Re-run with --confirm.')
+    console.log('')
+    return
+  }
+
+  if (referencing.length > 0) {
+    console.log('    Skipped: the plugin is still imported by the files listed above.')
+    console.log('    Remove those references first, then re-run.')
+    console.log('')
+    return
+  }
+
+  const removeCmd = pm === 'npm' ? 'npm uninstall' : `${pm} remove`
+  run(`${removeCmd} ${PACKAGE_NAME}`, projectDir)
+  run(`${pm === 'npm' ? 'npx' : pm} payload generate:importmap`, projectDir)
+
+  console.log('')
+  console.log('  Done.')
+  console.log('')
+}
+
+main()

package/scripts/verify-dist-imports.mjs

@@ -0,0 +1,84 @@
+#!/usr/bin/env node
+/**
+ * Every relative import in `dist/` must resolve to a file that is actually there.
+ *
+ * This catches one specific, silent failure mode of a multi-pass tsup build.
+ * The `bundle: false` pass preserves the source tree and rewrites nothing, so a
+ * relative import survives into the emitted file. If the target module is not
+ * itself in that pass's `entry` list, it is never emitted as a standalone file —
+ * it only exists inlined inside the bundled `dist/index.js`. The import then
+ * points at nothing.
+ *
+ * Nothing in the normal toolchain sees it. `tsc` type-checks the SOURCE tree,
+ * where the module exists. Vitest imports from `src/`, same thing. The build
+ * itself succeeds — tsup has no reason to object. The failure surfaces only when
+ * the HOST application's bundler resolves the published package, which is to say
+ * after publication, in someone else's build.
+ *
+ * `import type` is invisible here on purpose: TypeScript erases it, so it leaves
+ * no trace in the emitted file and cannot break anything. Only value imports do.
+ * That is why the defect hid for two releases in this package — every other
+ * importer of the same module used `import type`.
+ */
+import { existsSync, readdirSync, readFileSync, statSync } from 'node:fs'
+import { dirname, join, resolve } from 'node:path'
+
+const DIST = resolve(process.cwd(), 'dist')
+
+if (!existsSync(DIST)) {
+  console.error('verify-dist-imports: dist/ is missing — run the build first.')
+  process.exit(1)
+}
+
+/** Every emitted JavaScript file, at any depth. */
+function walk(dir) {
+  return readdirSync(dir, { withFileTypes: true }).flatMap((entry) => {
+    const path = join(dir, entry.name)
+    if (entry.isDirectory()) return walk(path)
+    return /\.(js|cjs|mjs)$/.test(entry.name) ? [path] : []
+  })
+}
+
+/**
+ * Relative specifiers only. Bare specifiers are the consumer's dependency
+ * problem, not ours, and `node:` builtins always resolve.
+ */
+const SPECIFIER =
+  /(?:^|[\s;{(])(?:import|export)\s+(?:[^'"]*?\sfrom\s+)?['"](\.[^'"]+)['"]|require\(\s*['"](\.[^'"]+)['"]\s*\)|import\(\s*['"](\.[^'"]+)['"]\s*\)/g
+
+/** Resolve the way Node and a bundler would: as written, then the usual suffixes. */
+function resolves(fromFile, specifier) {
+  const base = resolve(dirname(fromFile), specifier)
+  const candidates = [
+    base,
+    `${base}.js`, `${base}.cjs`, `${base}.mjs`,
+    join(base, 'index.js'), join(base, 'index.cjs'), join(base, 'index.mjs'),
+  ]
+  return candidates.some((c) => existsSync(c) && statSync(c).isFile())
+}
+
+const broken = []
+for (const file of walk(DIST)) {
+  const source = readFileSync(file, 'utf8')
+  for (const match of source.matchAll(SPECIFIER)) {
+    const specifier = match[1] || match[2] || match[3]
+    if (!specifier || resolves(file, specifier)) continue
+    broken.push({ file: file.replace(`${process.cwd()}/`, ''), specifier })
+  }
+}
+
+if (broken.length === 0) {
+  console.log('verify-dist-imports: every relative import in dist/ resolves.')
+  process.exit(0)
+}
+
+console.error(`verify-dist-imports: ${broken.length} unresolved relative import(s) in dist/.\n`)
+for (const { file, specifier } of broken) {
+  console.error(`  ${file}\n    imports '${specifier}' — not emitted`)
+}
+console.error(
+  '\nThe target is almost certainly missing from the `entry` list of the\n' +
+    '`bundle: false` pass in tsup.config.ts. Add it there, or move the value\n' +
+    'being imported into a module that pass already emits.',
+)
+process.exit(1)

package/src/collections/Tickets.ts

@@ -525,9 +525,16 @@ function createCascadeDelete(slugs: CollectionSlugs): CollectionBeforeDeleteHook
   return async ({ id, req }) => {
     const collections = [slugs.ticketMessages, slugs.ticketActivityLog, slugs.timeEntries, slugs.satisfactionSurveys]
     for (const slug of collections) {
+      // `req` must be forwarded: it carries the ambient transaction. Without
+      // it these cascades open their own connection and, on SQLite, deadlock
+      // against the transaction that is deleting the ticket — the delete then
+      // fails silently (Payload collects per-document errors instead of
+      // throwing) and the children survive the parent.
+      if (!req.payload.collections?.[slug as never]) continue
       await dbDelete(req.payload, slug, {
         where: { ticket: { equals: id } },
         overrideAccess: true,
+        req,
       })
     }
   }

package/src/collections/TimeEntries.ts

@@ -1,13 +1,19 @@
-import type { CollectionConfig, CollectionAfterChangeHook, CollectionAfterDeleteHook } from 'payload'
+import type { CollectionConfig, CollectionAfterChangeHook, CollectionAfterDeleteHook, PayloadRequest } from 'payload'
 import type { Payload } from 'payload'
 import type { CollectionSlugs } from '../utils/slugs'
 
 // ─── Hooks ───────────────────────────────────────────────
 
+/**
+ * `req` is forwarded to every operation so the rollup joins the caller's
+ * transaction. Reading or writing outside it would see a stale total and, on
+ * SQLite, contend with the very transaction that triggered the hook.
+ */
 async function recalculateTicketTime(
   payload: Payload,
   slugs: CollectionSlugs,
   ticketId: number | string,
+  req?: PayloadRequest,
 ): Promise<void> {
   // Sum all time entries for this ticket, paginating to avoid loading all at once
   let totalMinutes = 0
@@ -23,6 +29,7 @@ async function recalculateTicketTime(
       depth: 0,
       overrideAccess: true,
       select: { duration: true },
+      req,
     })
 
     for (const entry of entries.docs) {
@@ -38,6 +45,7 @@ async function recalculateTicketTime(
     id: ticketId,
     data: { totalTimeMinutes: totalMinutes },
     overrideAccess: true,
+    req,
   })
 }
 
@@ -45,7 +53,7 @@ function createRecalculateTicketTime(slugs: CollectionSlugs): CollectionAfterCha
   return async ({ doc, req }) => {
     if (!doc.ticket) return
     const ticketId = typeof doc.ticket === 'object' ? doc.ticket.id : doc.ticket
-    await recalculateTicketTime(req.payload, slugs, ticketId)
+    await recalculateTicketTime(req.payload, slugs, ticketId, req)
   }
 }
 
@@ -59,7 +67,7 @@ function createRecalculateTicketTimeOnDelete(slugs: CollectionSlugs): Collection
     if (!doc?.ticket) return
     const ticketId = typeof doc.ticket === 'object' ? doc.ticket.id : doc.ticket
     try {
-      await recalculateTicketTime(req.payload, slugs, ticketId)
+      await recalculateTicketTime(req.payload, slugs, ticketId, req)
     } catch (err) {
       // The entry is already gone; never fail the delete on the rollup refresh.
       console.error('[support] Failed to recalculate ticket time after delete:', err)

package/src/components/RichTextEditor/index.tsx

@@ -287,7 +287,6 @@ export const RichTextEditor = forwardRef<RichTextEditorHandle, Props>(function R
             fontSize: '14px',
             lineHeight: 1.6,
             color: '#1f2937',
-            outline: 'none',
             overflowY: 'auto',
           }}
         />
@@ -300,6 +299,7 @@ export const RichTextEditor = forwardRef<RichTextEditorHandle, Props>(function R
           type="file"
           accept="image/*"
           multiple
+          aria-label="Inserer une image"
           onChange={handleFileChange}
           style={{ display: 'none' }}
         />

package/src/components/TicketConversation/RewriteDropdown.tsx

@@ -76,7 +76,7 @@ export const RewriteDropdown: React.FC<{
               onMouseLeave={(e) => { (e.target as HTMLElement).style.background = 'transparent' }}
             >
               <span style={{ fontSize: 12, fontWeight: 600, color: 'var(--theme-text, #111)' }}>{style.label}</span>
-              <span style={{ fontSize: 10, color: 'var(--theme-elevation-500, #6b7280)' }}>{style.desc}</span>
+              <span style={{ fontSize: 10, color: 'var(--theme-elevation-650, #6b7280)' }}>{style.desc}</span>
             </button>
           ))}
         </div>

package/src/components/TicketConversation/components/AISummaryPanel.tsx

@@ -39,6 +39,7 @@ export function AISummaryPanel({
   return (
     <div style={{ marginBottom: '14px' }}>
       <button
+        type="button"
         onClick={() => { setShowAiSummary(!showAiSummary); if (!showAiSummary && !aiSummary) handleAiGenerate() }}
         style={{
           ...s.ghostBtn('#17807c', false),
@@ -61,6 +62,7 @@ export function AISummaryPanel({
             </h4>
             <div style={{ display: 'flex', gap: '6px' }}>
               <button
+                type="button"
                 onClick={handleAiGenerate}
                 disabled={aiGenerating}
                 style={{ ...s.outlineBtn('#17807c', aiGenerating), fontSize: '11px', padding: '4px 10px' }}
@@ -69,6 +71,7 @@ export function AISummaryPanel({
               </button>
               {aiSummary && !aiGenerating && (
                 <button
+                  type="button"
                   onClick={handleAiSave}
                   disabled={aiSaving || aiSaved}
                   style={{ ...s.btn(aiSaved ? '#16a34a' : '#2563eb', aiSaving), fontSize: '11px', padding: '4px 10px' }}

package/src/components/TicketConversation/components/ActionPanels.tsx

@@ -26,12 +26,13 @@ export function MergePanel({
       <div style={{ display: 'flex', gap: '8px', alignItems: 'center', marginBottom: '8px' }}>
         <input
           type="text"
+          aria-label="Numero du ticket cible"
           value={mergeTarget}
           onChange={(e) => { setMergeTarget(e.target.value); setMergeTargetInfo(null); setMergeError('') }}
           placeholder="TK-0001"
           style={{ ...s.input, width: '130px' }}
         />
-        <button onClick={handleMergeLookup} style={{ ...s.outlineBtn('#ec4899'), fontSize: '12px', padding: '6px 14px' }}>
+        <button type="button" onClick={handleMergeLookup} style={{ ...s.outlineBtn('#ec4899'), fontSize: '12px', padding: '6px 14px' }}>
           Rechercher
         </button>
         {mergeError && <span style={{ fontSize: '12px', color: '#be185d', fontWeight: 600 }}>{mergeError}</span>}
@@ -41,7 +42,7 @@ export function MergePanel({
           <span style={{ fontSize: '13px', fontWeight: 600 }}>
             {mergeTargetInfo.ticketNumber} — {mergeTargetInfo.subject}
           </span>
-          <button onClick={handleMerge} disabled={merging} style={{ ...s.btn('#ec4899', merging), color: '#fff', fontSize: '12px', padding: '6px 14px' }}>
+          <button type="button" onClick={handleMerge} disabled={merging} style={{ ...s.btn('#ec4899', merging), color: '#fff', fontSize: '12px', padding: '6px 14px' }}>
             {merging ? 'Fusion...' : 'Confirmer la fusion'}
           </button>
         </div>
@@ -76,6 +77,7 @@ export function ExtMessagePanel({
     <div style={{ padding: '14px 18px', borderRadius: '8px', backgroundColor: '#eef2ff', border: '1px solid #c7d2fe', marginBottom: '14px' }}>
       <h4 style={{ fontSize: '13px', fontWeight: 600, marginBottom: '10px', color: '#312e81' }}>Ajouter un message reçu (email, SMS, WhatsApp...)</h4>
       <textarea
+        aria-label="Contenu du message recu"
         value={extMsgBody}
         onChange={(e) => setExtMsgBody(e.target.value)}
         placeholder="Coller le contenu du message reçu..."
@@ -83,17 +85,18 @@ export function ExtMessagePanel({
         style={{ ...s.input, width: '100%', resize: 'vertical', marginBottom: '10px' }}
       />
       <div style={{ display: 'flex', gap: '8px', alignItems: 'center', flexWrap: 'wrap', marginBottom: '10px' }}>
-        <select value={extMsgAuthor} onChange={(e) => setExtMsgAuthor(e.target.value as 'client' | 'admin')} style={{ ...s.input, fontSize: '12px' }}>
+        <select aria-label="Auteur du message recu" value={extMsgAuthor} onChange={(e) => setExtMsgAuthor(e.target.value as 'client' | 'admin')} style={{ ...s.input, fontSize: '12px' }}>
           <option value="client">Envoyé par le client</option>
           <option value="admin">Envoyé par le support</option>
         </select>
         <input
           type="datetime-local"
+          aria-label="Date du message recu"
           value={extMsgDate}
           onChange={(e) => setExtMsgDate(e.target.value)}
           style={{ ...s.input, fontSize: '12px' }}
         />
-        <input ref={extFileInputRef} type="file" multiple onChange={handleExtFileChange} style={{ display: 'none' }} accept="image/*,.pdf,.doc,.docx,.txt,.zip" />
+        <input ref={extFileInputRef} type="file" multiple aria-label="Joindre des fichiers au message recu" onChange={handleExtFileChange} style={{ display: 'none' }} accept="image/*,.pdf,.doc,.docx,.txt,.zip" />
         <button type="button" onClick={() => extFileInputRef.current?.click()} style={{ ...s.ghostBtn('#6b7280'), fontSize: '12px', padding: '6px 12px' }}>
           + PJ
         </button>
@@ -108,7 +111,7 @@ export function ExtMessagePanel({
           ))}
         </div>
       )}
-      <button onClick={handleSendExtMsg} disabled={sendingExtMsg || !extMsgBody.trim()} style={s.btn('#1d2b4d', sendingExtMsg || !extMsgBody.trim())}>
+      <button type="button" onClick={handleSendExtMsg} disabled={sendingExtMsg || !extMsgBody.trim()} style={s.btn('#1d2b4d', sendingExtMsg || !extMsgBody.trim())}>
         {sendingExtMsg ? 'Ajout...' : 'Ajouter (sans notification)'}
       </button>
     </div>
@@ -186,6 +189,7 @@ export function ReminderPanel({
       )}
 
       <button
+        type="button"
         onClick={handleSendReminder}
         disabled={reminderSending || !clientEmail}
         style={{ ...s.btn(C.orange, reminderSending || !clientEmail), color: '#fff', fontSize: '13px', padding: '8px 18px' }}
@@ -207,11 +211,12 @@ export function SnoozePanel({ snoozeSaving, handleSnooze }: SnoozePanelProps) {
     <div style={{ padding: '14px 18px', borderRadius: '8px', backgroundColor: '#eef8f7', border: '1px solid #b8dcda', marginBottom: '14px' }}>
       <h4 style={{ fontSize: '13px', fontWeight: 600, marginBottom: '10px', color: '#5b21b6' }}>Snooze — masquer temporairement</h4>
       <div style={{ display: 'flex', gap: '8px', flexWrap: 'wrap', alignItems: 'center' }}>
-        <button onClick={() => handleSnooze(1)} disabled={snoozeSaving} style={{ ...s.outlineBtn('#17807c', snoozeSaving), fontSize: '12px' }}>1 jour</button>
-        <button onClick={() => handleSnooze(3)} disabled={snoozeSaving} style={{ ...s.outlineBtn('#17807c', snoozeSaving), fontSize: '12px' }}>3 jours</button>
-        <button onClick={() => handleSnooze(7)} disabled={snoozeSaving} style={{ ...s.outlineBtn('#17807c', snoozeSaving), fontSize: '12px' }}>1 semaine</button>
+        <button type="button" onClick={() => handleSnooze(1)} disabled={snoozeSaving} style={{ ...s.outlineBtn('#17807c', snoozeSaving), fontSize: '12px' }}>1 jour</button>
+        <button type="button" onClick={() => handleSnooze(3)} disabled={snoozeSaving} style={{ ...s.outlineBtn('#17807c', snoozeSaving), fontSize: '12px' }}>3 jours</button>
+        <button type="button" onClick={() => handleSnooze(7)} disabled={snoozeSaving} style={{ ...s.outlineBtn('#17807c', snoozeSaving), fontSize: '12px' }}>1 semaine</button>
         <input
           type="datetime-local"
+          aria-label="Snoozer jusqu'a une date precise"
           onChange={(e) => { if (e.target.value) handleSnooze(null, e.target.value) }}
           style={{ ...s.input, fontSize: '12px' }}
         />

package/src/components/TicketConversation/components/ClientHistory.tsx

@@ -96,6 +96,7 @@ export function ClientHistory({
           {/* Notes */}
           <h5 style={{ fontSize: '12px', fontWeight: 600, color: C.textSecondary, marginBottom: '8px' }}>Notes internes</h5>
           <textarea
+            aria-label="Notes internes sur le client"
             value={clientNotes}
             onChange={(e) => onNotesChange(e.target.value)}
             rows={3}

package/src/components/TicketConversation/components/CodeBlock.tsx

@@ -90,6 +90,7 @@ function SingleCodeBlock({ lang, code }: { lang: string; code: string }) {
           </span>
         </div>
         <button
+          type="button"
           onClick={handleCopy}
           style={{
             background: 'none',

package/src/components/TicketConversation/components/CodeBlockInserter.tsx

@@ -109,6 +109,7 @@ export function CodeBlockInserter({ onInsert, className, style }: CodeBlockInser
             <input
               ref={inputRef}
               type="text"
+              aria-label="Rechercher un langage"
               value={filter}
               onChange={(e) => setFilter(e.target.value)}
               onKeyDown={(e) => {
@@ -124,7 +125,6 @@ export function CodeBlockInserter({ onInsert, className, style }: CodeBlockInser
                 border: '1px solid #334155',
                 borderRadius: 6,
                 color: '#e2e8f0',
-                outline: 'none',
                 boxSizing: 'border-box',
               }}
             />

package/src/components/TicketConversation/components/QuickActions.tsx

@@ -48,6 +48,7 @@ export function QuickActions({
       <div style={{ display: 'flex', gap: '8px', flexWrap: 'wrap', marginBottom: '16px', alignItems: 'center' }}>
         {statusTransitions.map((a) => (
           <button
+            type="button"
             key={a.status}
             onClick={() => onStatusChange(a.status)}
             disabled={statusUpdating}
@@ -57,22 +58,22 @@ export function QuickActions({
           </button>
         ))}
         {snoozeUntil && new Date(snoozeUntil) > new Date() && (
-          <button onClick={onCancelSnooze} disabled={snoozeSaving} style={{ ...s.ghostBtn('#17807c', snoozeSaving), fontSize: '12px', padding: '5px 10px' }}>
+          <button type="button" onClick={onCancelSnooze} disabled={snoozeSaving} style={{ ...s.ghostBtn('#17807c', snoozeSaving), fontSize: '12px', padding: '5px 10px' }}>
             Annuler snooze
           </button>
         )}
         <span style={{ borderLeft: `1px solid ${C.border}`, height: '20px', margin: '0 4px' }} />
-        <button onClick={onToggleMerge} style={s.ghostBtn('#be185d')}>Fusionner</button>
-        <button onClick={onToggleExtMsg} style={s.ghostBtn('#4f46e5')}>+ Message reçu</button>
-        <button onClick={onToggleSnooze} style={s.ghostBtn('#17807c')}>Snooze</button>
+        <button type="button" onClick={onToggleMerge} style={s.ghostBtn('#be185d')}>Fusionner</button>
+        <button type="button" onClick={onToggleExtMsg} style={s.ghostBtn('#4f46e5')}>+ Message reçu</button>
+        <button type="button" onClick={onToggleSnooze} style={s.ghostBtn('#17807c')}>Snooze</button>
         {showReminderButton && (
-          <button onClick={onToggleReminder} style={s.ghostBtn('#ea580c')}>{'⏰'} Relancer</button>
+          <button type="button" onClick={onToggleReminder} style={s.ghostBtn('#ea580c')}>{'⏰'} Relancer</button>
         )}
-        <button onClick={onNextTicket} style={s.ghostBtn('#16a34a')}>Ticket suivant</button>
+        <button type="button" onClick={onNextTicket} style={s.ghostBtn('#16a34a')}>Ticket suivant</button>
         {scheduledClose && (
           <span style={{ display: 'inline-flex', alignItems: 'center', gap: '8px', padding: '5px 10px', borderRadius: '6px', backgroundColor: '#fff7ed', border: '1px solid #fed7aa', fontSize: '12px', color: '#9a3412', fontWeight: 600 }}>
             {'⏰'} Fermeture auto le {scheduledClose}
-            <button onClick={onCancelScheduledClose} disabled={cancelingClose} style={{ border: 'none', background: 'none', color: '#b91c1c', cursor: cancelingClose ? 'not-allowed' : 'pointer', fontWeight: 700, fontSize: '12px', textDecoration: 'underline', padding: 0, opacity: cancelingClose ? 0.5 : 1 }}>
+            <button type="button" onClick={onCancelScheduledClose} disabled={cancelingClose} style={{ border: 'none', background: 'none', color: '#b91c1c', cursor: cancelingClose ? 'not-allowed' : 'pointer', fontWeight: 700, fontSize: '12px', textDecoration: 'underline', padding: 0, opacity: cancelingClose ? 0.5 : 1 }}>
               {cancelingClose ? '...' : 'Annuler'}
             </button>
           </span>
@@ -90,6 +91,7 @@ export function QuickActions({
                 Ticket suivant : <strong>{nextTicketInfo}</strong>
               </span>
               <button
+                type="button"
                 onClick={() => { window.location.href = `/admin/support/ticket?id=${nextTicketId}` }}
                 style={{ ...s.btn(C.statusResolved), color: C.white, fontSize: '12px', padding: '5px 14px' }}
               >
@@ -99,7 +101,7 @@ export function QuickActions({
           ) : (
             <span style={{ fontSize: '13px', color: '#166534', fontWeight: 700 }}>{nextTicketInfo}</span>
           )}
-          <button onClick={onCloseNextTicket} style={{ border: 'none', background: 'none', color: C.textMuted, cursor: 'pointer', fontSize: '16px', fontWeight: 700, marginLeft: '8px' }}>&times;</button>
+          <button type="button" onClick={onCloseNextTicket} style={{ border: 'none', background: 'none', color: C.textMuted, cursor: 'pointer', fontSize: '16px', fontWeight: 700, marginLeft: '8px' }}>&times;</button>
         </div>
       )}
     </>

package/src/components/TicketConversation/components/TicketHeader.tsx

@@ -71,6 +71,7 @@ export function TicketHeader({
       )}
       <span style={{ marginLeft: 'auto', display: 'inline-flex', alignItems: 'center', gap: '4px' }}>
         <button
+          type="button"
           onClick={() => onCopyLink('admin')}
           title="Copier le lien admin"
           style={{ background: 'none', border: 'none', cursor: 'pointer', fontSize: '13px', opacity: copiedLink === 'admin' ? 1 : 0.5, padding: '2px 4px' }}
@@ -78,6 +79,7 @@ export function TicketHeader({
           {copiedLink === 'admin' ? '\u2705' : '\uD83D\uDCCB'}
         </button>
         <button
+          type="button"
           onClick={() => onCopyLink('client')}
           title="Copier le lien client"
           style={{ background: 'none', border: 'none', cursor: 'pointer', fontSize: '13px', opacity: copiedLink === 'client' ? 1 : 0.5, padding: '2px 4px' }}