hotfix: se mejoró UX modificando el tipo de cursor que se muestra al hacer hover sobre elementos interactuables y se restringió el input de horas estimadas a un rango de 0 a 200 pero permitiendo medias horas

Reviewed-on: #185

bun.lock

@@ -4,6 +4,7 @@
     "": {
       "name": "acad-ia-2",
       "dependencies": {
+        "@dnd-kit/react": "^0.3.2",
         "@radix-ui/react-alert-dialog": "^1.1.15",
         "@radix-ui/react-avatar": "^1.1.11",
         "@radix-ui/react-checkbox": "^1.3.3",
@@ -138,6 +139,18 @@
 
     "@csstools/css-tokenizer": ["@csstools/css-tokenizer@3.0.4", "", {}, "sha512-Vd/9EVDiu6PPJt9yAh6roZP6El1xHrdvIVGjyBsHR0RYwNHgL7FJPyIIW4fANJNG6FtyZfvlRPpFI4ZM/lubvw=="],
 
+    "@dnd-kit/abstract": ["@dnd-kit/abstract@0.3.2", "", { "dependencies": { "@dnd-kit/geometry": "^0.3.2", "@dnd-kit/state": "^0.3.2", "tslib": "^2.6.2" } }, "sha512-uvPVK+SZYD6Viddn9M0K0JQdXknuVSxA/EbMlFRanve3P/XTc18oLa5zGftKSGjfQGmuzkZ34E26DSbly1zi3Q=="],
+
+    "@dnd-kit/collision": ["@dnd-kit/collision@0.3.2", "", { "dependencies": { "@dnd-kit/abstract": "^0.3.2", "@dnd-kit/geometry": "^0.3.2", "tslib": "^2.6.2" } }, "sha512-pNmNSLCI8S9fNQ7QJ3fBCDjiT0sqBhUFcKgmyYaGvGCAU+kq0AP8OWlh0JSisc9k5mFyxmRpmFQcnJpILz/RPA=="],
+
+    "@dnd-kit/dom": ["@dnd-kit/dom@0.3.2", "", { "dependencies": { "@dnd-kit/abstract": "^0.3.2", "@dnd-kit/collision": "^0.3.2", "@dnd-kit/geometry": "^0.3.2", "@dnd-kit/state": "^0.3.2", "tslib": "^2.6.2" } }, "sha512-cIUAVgt2szQyz6JRy7I+0r+xeyOAGH21Y15hb5bIyHoDEaZBvIDH+OOlD9eoLjCbsxDLN9WloU2CBi3OE6LYDg=="],
+
+    "@dnd-kit/geometry": ["@dnd-kit/geometry@0.3.2", "", { "dependencies": { "@dnd-kit/state": "^0.3.2", "tslib": "^2.6.2" } }, "sha512-3UBPuIS7E3oGiHxOE8h810QA+0pnrnCtGxl4Os1z3yy5YkC/BEYGY+TxWPTQaY1/OMV7GCX7ZNMlama2QN3n3w=="],
+
+    "@dnd-kit/react": ["@dnd-kit/react@0.3.2", "", { "dependencies": { "@dnd-kit/abstract": "^0.3.2", "@dnd-kit/dom": "^0.3.2", "@dnd-kit/state": "^0.3.2", "tslib": "^2.6.2" }, "peerDependencies": { "react": "^18.0.0 || ^19.0.0", "react-dom": "^18.0.0 || ^19.0.0" } }, "sha512-1Opg1xw6I75Z95c+rF2NJa0pdGb8rLAENtuopKtJ1J0PudWlz+P6yL137xy/6DV43uaRmNGtsdbMbR0yRYJ72g=="],
+
+    "@dnd-kit/state": ["@dnd-kit/state@0.3.2", "", { "dependencies": { "@preact/signals-core": "^1.10.0", "tslib": "^2.6.2" } }, "sha512-dLUIkoYrIJhGXfF2wGLTfb46vUokEsO/OoE21TSfmahYrx7ysTmnwbePsznFaHlwgZhQEh6AlLvthLCeY21b1A=="],
+
     "@emnapi/core": ["@emnapi/core@1.8.1", "", { "dependencies": { "@emnapi/wasi-threads": "1.1.0", "tslib": "^2.4.0" } }, "sha512-AvT9QFpxK0Zd8J0jopedNm+w/2fIzvtPKPjqyw9jwvBaReTTqPBk9Hixaz7KbjimP+QNz605/XnjFcDAL2pqBg=="],
 
     "@emnapi/runtime": ["@emnapi/runtime@1.8.1", "", { "dependencies": { "tslib": "^2.4.0" } }, "sha512-mehfKSMWjjNol8659Z8KxEMrdSJDDot5SXMq00dM8BN4o+CLNXQ0xH2V7EchNHV4RmbZLmmPdEaXZc5H2FXmDg=="],
@@ -250,6 +263,8 @@
 
     "@napi-rs/wasm-runtime": ["@napi-rs/wasm-runtime@0.2.12", "", { "dependencies": { "@emnapi/core": "^1.4.3", "@emnapi/runtime": "^1.4.3", "@tybys/wasm-util": "^0.10.0" } }, "sha512-ZVWUcfwY4E/yPitQJl481FjFo3K22D6qF0DuFH6Y/nbnE11GY5uguDxZMGXPQ8WQ0128MXQD7TnfHyK4oWoIJQ=="],
 
+    "@preact/signals-core": ["@preact/signals-core@1.14.0", "", {}, "sha512-AowtCcCU/33lFlh1zRFf/u+12rfrhtNakj7UpaGEsmMwUKpKWMVvcktOGcwBBNiB4lWrZWc01LhiyyzVklJyaQ=="],
+
     "@radix-ui/number": ["@radix-ui/number@1.1.1", "", {}, "sha512-MkKCwxlXTgz6CFoJx3pCwn07GKp36+aZyu/u2Ln2VrA5DcdyCZkASEDBTd8x5whTQQL5CiYf4prXKLcgQdv29g=="],
 
     "@radix-ui/primitive": ["@radix-ui/primitive@1.1.3", "", {}, "sha512-JTF99U/6XIjCBo0wqkU5sK10glYe27MRRsfwoiq5zzOEZLHU3A3KCMa5X/azekYRCJ0HlwI0crAXS/5dEHTzDg=="],

package.json

@@ -17,6 +17,7 @@
     "ci:verify": "prettier --check . && eslint . && tsc --noEmit"
   },
   "dependencies": {
+    "@dnd-kit/react": "^0.3.2",
     "@radix-ui/react-alert-dialog": "^1.1.15",
     "@radix-ui/react-avatar": "^1.1.11",
     "@radix-ui/react-checkbox": "^1.3.3",

src/components/asignaturas/detalle/ContenidoTematico.tsx

@@ -1,3 +1,5 @@
+import { DragDropProvider } from '@dnd-kit/react'
+import { isSortable, useSortable } from '@dnd-kit/react/sortable'
 import { useParams } from '@tanstack/react-router'
 import {
   Plus,
@@ -11,7 +13,7 @@ import {
 import { useEffect, useRef, useState } from 'react'
 
 import type { ContenidoApi, ContenidoTemaApi } from '@/data/api/subjects.api'
-import type { FocusEvent, KeyboardEvent } from 'react'
+import type { FocusEvent, KeyboardEvent, ReactNode } from 'react'
 
 import {
   AlertDialog,
@@ -50,6 +52,95 @@ export interface UnidadTematica {
   temas: Array<Tema>
 }
 
+function createClientId(prefix: string) {
+  try {
+    const c = (globalThis as any).crypto
+    if (c && typeof c.randomUUID === 'function')
+      return `${prefix}-${c.randomUUID()}`
+  } catch {
+    // ignore
+  }
+  return `${prefix}-${Date.now()}-${Math.random().toString(16).slice(2)}`
+}
+
+function arrayMove<T>(array: Array<T>, fromIndex: number, toIndex: number) {
+  const next = array.slice()
+  const startIndex = fromIndex < 0 ? next.length + fromIndex : fromIndex
+  if (startIndex < 0 || startIndex >= next.length) return next
+  const endIndex = toIndex < 0 ? next.length + toIndex : toIndex
+  const [item] = next.splice(startIndex, 1)
+  next.splice(endIndex, 0, item)
+  return next
+}
+
+function renumberUnidades(unidades: Array<UnidadTematica>) {
+  return unidades.map((u, idx) => ({ ...u, numero: idx + 1 }))
+}
+
+function InsertUnidadOverlay({
+  onInsert,
+  position,
+}: {
+  onInsert: () => void
+  position: 'top' | 'bottom'
+}) {
+  return (
+    <div
+      className={cn(
+        'pointer-events-auto absolute right-0 left-0 z-30 flex justify-center',
+        // Match the `space-y-4` gap so the hover target is *between* units.
+        position === 'top' ? '-top-4 h-4' : '-bottom-4 h-4',
+      )}
+    >
+      <Button
+        type="button"
+        variant="outline"
+        size="sm"
+        className="bg-background/95 border-border/60 hover:bg-background cursor-pointer opacity-0 shadow-sm transition-opacity group-hover:opacity-100"
+        onClick={(e) => {
+          e.stopPropagation()
+          onInsert()
+        }}
+      >
+        <Plus className="mr-2 h-3 w-3" /> Nueva unidad
+      </Button>
+    </div>
+  )
+}
+
+function SortableUnidad({
+  id,
+  index,
+  registerContainer,
+  children,
+}: {
+  id: string
+  index: number
+  registerContainer: (el: HTMLDivElement | null) => void
+  children: (args: { handleRef: (el: HTMLElement | null) => void }) => ReactNode
+}) {
+  const { ref, handleRef, isDragSource, isDropTarget } = useSortable({
+    id,
+    index,
+  })
+
+  return (
+    <div
+      ref={(el) => {
+        ref(el)
+        registerContainer(el)
+      }}
+      className={cn(
+        'group relative',
+        isDragSource && 'opacity-80',
+        isDropTarget && 'ring-primary/20 ring-2',
+      )}
+    >
+      {children({ handleRef })}
+    </div>
+  )
+}
+
 function isRecord(value: unknown): value is Record<string, unknown> {
   return typeof value === 'object' && value !== null && !Array.isArray(value)
 }
@@ -100,20 +191,18 @@ function mapContenidoItem(value: unknown, index: number): ContenidoApi | null {
   if (Array.isArray(value.temas)) {
     temas = value.temas
       .map(mapTemaValue)
-      .filter((t): t is ContenidoTemaApi => t !== null)
+      .filter((x): x is ContenidoTemaApi => x !== null)
-  } else if (typeof value.temas === 'string' && value.temas.trim()) {
-    temas = value.temas
-      .split(/\r?\n|,/)
-      .map((t) => t.trim())
-      .filter(Boolean)
   }
 
-  return { unidad, titulo, temas }
+  return {
+    ...value,
+    unidad,
+    titulo,
+    temas,
+  }
 }
 
 function mapContenidoTematicoFromDb(value: unknown): Array<ContenidoApi> {
-  if (value == null) return []
-
   if (typeof value === 'string') {
     try {
       return mapContenidoTematicoFromDb(JSON.parse(value))
@@ -192,7 +281,16 @@ export function ContenidoTematico() {
   const [temaDraftHoras, setTemaDraftHoras] = useState('')
   const [temaOriginalHoras, setTemaOriginalHoras] = useState(0)
 
+  const didInitExpandedUnitsRef = useRef(false)
+
+  const unidadesRef = useRef<Array<UnidadTematica>>([])
+  useEffect(() => {
+    unidadesRef.current = unidades
+  }, [unidades])
+
   const persistUnidades = async (nextUnidades: Array<UnidadTematica>) => {
+    // A partir del primer guardado, ya respetamos lo que el usuario deje expandido.
+    didInitExpandedUnitsRef.current = true
     const payload = serializeUnidadesToApi(nextUnidades)
     await updateContenido.mutateAsync({
       subjectId: asignaturaId,
@@ -246,10 +344,17 @@ export function ContenidoTematico() {
     })
   }
 
+  const parseHorasEstimadas = (raw: string): number => {
+    const normalized = raw.trim().replace(',', '.')
+    const parsed = Number.parseFloat(normalized)
+    if (!Number.isFinite(parsed)) return 0
+
+    return parsed
+  }
+
   const commitEditTema = () => {
     if (!editingTema) return
-    const parsedHoras = Number.parseInt(temaDraftHoras, 10)
+    const horasEstimadas = parseHorasEstimadas(temaDraftHoras)
-    const horasEstimadas = Number.isFinite(parsedHoras) ? parsedHoras : 0
 
     const next = unidades.map((u) => {
       if (u.id !== editingTema.unitId) return u
@@ -303,28 +408,110 @@ export function ContenidoTematico() {
       data ? data.contenido_tematico : undefined,
     )
 
-    const transformed = contenido.map((u, idx) => ({
+    // 1. EL ESCUDO: Comparamos si nuestro estado local ya tiene esta info exacta
-      id: `u-${u.unidad || idx + 1}`,
+    // (Esto ocurre justo después de arrastrar, ya que actualizamos la UI antes que la BD)
-      numero: u.unidad || idx + 1,
+    const currentPayload = JSON.stringify(
-      nombre: u.titulo || 'Sin título',
+      serializeUnidadesToApi(unidadesRef.current),
-      temas: Array.isArray(u.temas)
+    )
-        ? u.temas.map((t: any, tidx: number) => ({
+
-            id: `t-${u.unidad || idx + 1}-${tidx + 1}`,
+    // Normalizamos la data de la BD para que tenga exactamente la misma forma que el payload
-            nombre: typeof t === 'string' ? t : t?.nombre || 'Tema',
+    const incomingPayload = JSON.stringify(
-            horasEstimadas: t?.horasEstimadas || 0,
+      contenido.map((u, idx) => ({
-          }))
+        unidad: u.unidad || idx + 1,
-        : [],
+        titulo: u.titulo || 'Sin título',
-    }))
+        temas: Array.isArray(u.temas)
+          ? u.temas.map((t) => {
+              if (typeof t === 'string') {
+                return {
+                  nombre: t,
+                  horasEstimadas: 0,
+                  descripcion: undefined,
+                }
+              }
+
+              return {
+                nombre: t.nombre || 'Tema',
+                horasEstimadas: t.horasEstimadas ?? 0,
+                descripcion: t.descripcion,
+              }
+            })
+          : [],
+      })),
+    )
+
+    // Si los datos son idénticos, abortamos el useEffect.
+    // ¡Nuestros IDs locales se salvan y no hay parpadeos!
+    if (currentPayload === incomingPayload && unidadesRef.current.length > 0) {
+      return
+    }
+
+    // 2. Si llegamos aquí, es la carga inicial o alguien más editó la BD desde otro lado.
+    // Reciclamos IDs buscando por CONTENIDO (nombre), NUNCA POR ÍNDICE.
+    const prevUnidades = [...unidadesRef.current]
+
+    const transformed = contenido.map((u, idx) => {
+      const dbTitulo = u.titulo || 'Sin título'
+
+      // Buscamos si ya existe una unidad con este mismo título
+      const existingUnitIndex = prevUnidades.findIndex(
+        (prev) => prev.nombre === dbTitulo,
+      )
+      let unidadId
+      let existingUnit = null
+
+      if (existingUnitIndex !== -1) {
+        existingUnit = prevUnidades[existingUnitIndex]
+        unidadId = existingUnit.id
+        prevUnidades.splice(existingUnitIndex, 1) // Lo sacamos de la lista para no repetirlo
+      } else {
+        unidadId = createClientId(`u-${u.unidad || idx + 1}`)
+      }
+
+      return {
+        id: unidadId,
+        numero: u.unidad || idx + 1,
+        nombre: dbTitulo,
+        temas: Array.isArray(u.temas)
+          ? u.temas.map((t: any, tidx: number) => {
+              const dbTemaNombre =
+                typeof t === 'string' ? t : t?.nombre || 'Tema'
+
+              // Reciclamos subtemas por nombre también
+              const existingTema = existingUnit?.temas.find(
+                (prevT) => prevT.nombre === dbTemaNombre,
+              )
+              const temaId = existingTema
+                ? existingTema.id
+                : createClientId(`t-${u.unidad || idx + 1}-${tidx + 1}`)
+
+              return {
+                id: temaId,
+                nombre: dbTemaNombre,
+                horasEstimadas:
+                  coerceNumber(
+                    typeof t === 'string' ? undefined : t?.horasEstimadas,
+                  ) ?? 0,
+              }
+            })
+          : [],
+      }
+    })
 
     setUnidades(transformed)
-    // Mantener las unidades ya expandidas si existen; si no, expandir la primera.
+
     setExpandedUnits((prev) => {
       const validIds = new Set(transformed.map((u) => u.id))
       const filtered = new Set(
         Array.from(prev).filter((id) => validIds.has(id)),
       )
-      if (filtered.size > 0) return filtered
+
-      return transformed.length > 0 ? new Set([transformed[0].id]) : new Set()
+      // Expandir la primera unidad solo una vez al llegar a la ruta.
+      // Luego, no auto-expandimos de nuevo (aunque `data` cambie).
+      if (!didInitExpandedUnitsRef.current && transformed.length > 0) {
+        return filtered.size > 0 ? filtered : new Set([transformed[0].id])
+      }
+
+      return filtered
     })
   }, [data])
 
@@ -353,7 +540,7 @@ export function ContenidoTematico() {
   // 3. Cálculo de horas (ahora dinámico basado en los nuevos datos)
   const totalHoras = unidades.reduce(
     (acc, u) =>
-      acc + u.temas.reduce((sum, t) => sum + (t.horasEstimadas || 0), 0),
+      acc + u.temas.reduce((sum, t) => sum + (t.horasEstimadas ?? 0), 0),
     0,
   )
 
@@ -364,16 +551,22 @@ export function ContenidoTematico() {
     setExpandedUnits(newExpanded)
   }
 
-  const addUnidad = () => {
+  const insertUnidadAt = (insertIndex: number) => {
-    const newNumero = unidades.length + 1
+    const newId = createClientId('u')
-    const newId = `u-${newNumero}`
     const newUnidad: UnidadTematica = {
       id: newId,
       nombre: 'Nueva Unidad',
-      numero: newNumero,
+      numero: 0,
       temas: [],
     }
-    const next = [...unidades, newUnidad]
+
+    const clampedIndex = Math.max(0, Math.min(insertIndex, unidades.length))
+    const next = renumberUnidades([
+      ...unidades.slice(0, clampedIndex),
+      newUnidad,
+      ...unidades.slice(clampedIndex),
+    ])
+
     setUnidades(next)
     setExpandedUnits((prev) => {
       const n = new Set(prev)
@@ -382,10 +575,40 @@ export function ContenidoTematico() {
     })
     setPendingScrollUnitId(newId)
 
-    // Abrir edición del título inmediatamente
     setEditingUnit(newId)
     setUnitDraftNombre(newUnidad.nombre)
     setUnitOriginalNombre(newUnidad.nombre)
+
+    void persistUnidades(next)
+  }
+
+  const handleReorderEnd = (event: any) => {
+    if (event?.canceled) return
+
+    const source = event?.operation?.source
+    if (!source) return
+
+    // Type-guard nativo de dnd-kit para asegurar que el elemento tiene metadata de orden
+    if (!isSortable(source)) return
+
+    // Extraemos las posiciones exactas calculadas por dnd-kit
+    const { initialIndex, index } = source.sortable
+
+    // Si lo soltó en la misma posición de la que salió, cancelamos
+    if (initialIndex === index) return
+
+    setUnidades((prev) => {
+      // Hacemos el movimiento usando los índices directos
+      const moved = arrayMove(prev, initialIndex, index)
+      const next = renumberUnidades(moved)
+
+      // Disparamos la persistencia hacia Supabase
+      void persistUnidades(next).catch((err) => {
+        console.error('No se pudo guardar el orden de unidades', err)
+      })
+
+      return next
+    })
   }
 
   // --- Lógica de Temas ---
@@ -451,158 +674,176 @@ export function ContenidoTematico() {
         </div>
       </div>
 
-      <div className="space-y-4">
+      <DragDropProvider onDragEnd={handleReorderEnd}>
-        {unidades.map((unidad) => (
+        <div className="space-y-4">
-          <div
+          {unidades.map((unidad, index) => (
-            key={unidad.id}
+            <SortableUnidad
-            ref={(el) => {
+              key={unidad.id}
-              if (el) unitContainerRefs.current.set(unidad.id, el)
+              id={unidad.id}
-              else unitContainerRefs.current.delete(unidad.id)
+              index={index}
-            }}
+              registerContainer={(el) => {
-          >
+                if (el) unitContainerRefs.current.set(unidad.id, el)
-            <Card className="overflow-hidden border-slate-200 shadow-sm">
+                else unitContainerRefs.current.delete(unidad.id)
-              <Collapsible
+              }}
-                open={expandedUnits.has(unidad.id)}
+            >
-                onOpenChange={() => toggleUnit(unidad.id)}
+              {({ handleRef }) => (
-              >
+                <>
-                <CardHeader className="border-b border-slate-100 bg-slate-50/50 py-3">
+                  <InsertUnidadOverlay
-                  <div className="flex items-center gap-3">
+                    position="bottom"
-                    <GripVertical className="h-4 w-4 cursor-grab text-slate-300" />
+                    onInsert={() => insertUnidadAt(index + 1)}
-                    <CollapsibleTrigger asChild>
+                  />
-                      <Button variant="ghost" size="sm" className="h-auto p-0">
-                        {expandedUnits.has(unidad.id) ? (
-                          <ChevronDown className="h-4 w-4" />
-                        ) : (
-                          <ChevronRight className="h-4 w-4" />
-                        )}
-                      </Button>
-                    </CollapsibleTrigger>
-                    <Badge className="bg-blue-600 font-mono">
-                      Unidad {unidad.numero}
-                    </Badge>
 
-                    {editingUnit === unidad.id ? (
+                  <Card className="overflow-hidden border-slate-200 shadow-sm">
-                      <Input
+                    <Collapsible
-                        ref={unitTitleInputRef}
+                      open={expandedUnits.has(unidad.id)}
-                        value={unitDraftNombre}
+                      onOpenChange={() => toggleUnit(unidad.id)}
-                        onChange={(e) => setUnitDraftNombre(e.target.value)}
+                    >
-                        onBlur={() => {
+                      <CardHeader className="border-b border-slate-100 bg-slate-50/50 py-3">
-                          if (cancelNextBlurRef.current) {
+                        <div className="flex items-center gap-3">
-                            cancelNextBlurRef.current = false
+                          <span
-                            return
+                            ref={handleRef as any}
-                          }
+                            className="inline-flex cursor-grab touch-none items-center text-slate-300"
-                          commitEditUnit()
+                            aria-label="Reordenar unidad"
-                        }}
+                          >
-                        onKeyDown={(e) => {
+                            <GripVertical className="h-4 w-4" />
-                          if (e.key === 'Enter') {
+                          </span>
-                            e.preventDefault()
+                          <CollapsibleTrigger asChild>
-                            e.currentTarget.blur()
+                            <Button
-                            return
+                              variant="ghost"
-                          }
+                              size="sm"
-                          if (e.key === 'Escape') {
+                              className="h-auto cursor-pointer p-0"
-                            e.preventDefault()
+                            >
-                            cancelNextBlurRef.current = true
+                              {expandedUnits.has(unidad.id) ? (
-                            cancelEditUnit()
+                                <ChevronDown className="h-4 w-4" />
-                            e.currentTarget.blur()
+                              ) : (
-                          }
+                                <ChevronRight className="h-4 w-4" />
-                        }}
+                              )}
-                        className="h-8 max-w-md bg-white"
+                            </Button>
-                      />
+                          </CollapsibleTrigger>
-                    ) : (
+                          <Badge className="bg-blue-600 font-mono">
-                      <CardTitle
+                            Unidad {unidad.numero}
-                        className="cursor-pointer text-base font-semibold transition-colors hover:text-blue-600"
+                          </Badge>
-                        onClick={() => beginEditUnit(unidad.id)}
-                      >
-                        {unidad.nombre}
-                      </CardTitle>
-                    )}
 
-                    <div className="ml-auto flex items-center gap-3">
+                          {editingUnit === unidad.id ? (
-                      <span className="flex items-center gap-1 text-xs font-medium text-slate-400">
+                            <Input
-                        <Clock className="h-3 w-3" />{' '}
+                              ref={unitTitleInputRef}
-                        {unidad.temas.reduce(
+                              value={unitDraftNombre}
-                          (sum, t) => sum + (t.horasEstimadas || 0),
+                              onChange={(e) =>
-                          0,
+                                setUnitDraftNombre(e.target.value)
-                        )}
+                              }
-                        h
+                              onBlur={() => {
-                      </span>
+                                if (cancelNextBlurRef.current) {
-                      <Button
+                                  cancelNextBlurRef.current = false
-                        variant="ghost"
+                                  return
-                        size="icon"
+                                }
-                        className="h-8 w-8 text-slate-400 hover:text-red-500"
+                                commitEditUnit()
-                        onClick={() =>
+                              }}
-                          setDeleteDialog({ type: 'unidad', id: unidad.id })
+                              onKeyDown={(e) => {
-                        }
+                                if (e.key === 'Enter') {
-                      >
+                                  e.preventDefault()
-                        <Trash2 className="h-4 w-4" />
+                                  e.currentTarget.blur()
-                      </Button>
+                                  return
-                    </div>
+                                }
-                  </div>
+                                if (e.key === 'Escape') {
-                </CardHeader>
+                                  e.preventDefault()
-                <CollapsibleContent>
+                                  cancelNextBlurRef.current = true
-                  <CardContent className="bg-white pt-4">
+                                  cancelEditUnit()
-                    <div className="ml-10 space-y-1 border-l-2 border-slate-50 pl-4">
+                                  e.currentTarget.blur()
-                      {unidad.temas.map((tema, idx) => (
+                                }
-                        <TemaRow
+                              }}
-                          key={tema.id}
+                              className="h-8 max-w-md bg-white"
-                          tema={tema}
+                            />
-                          index={idx + 1}
+                          ) : (
-                          isEditing={
+                            <CardTitle
-                            !!editingTema &&
+                              className="cursor-pointer text-base font-semibold transition-colors hover:text-blue-600"
-                            editingTema.unitId === unidad.id &&
+                              onClick={() => beginEditUnit(unidad.id)}
-                            editingTema.temaId === tema.id
+                            >
-                          }
+                              {unidad.nombre}
-                          draftNombre={temaDraftNombre}
+                            </CardTitle>
-                          draftHoras={temaDraftHoras}
+                          )}
-                          onBeginEdit={() => beginEditTema(unidad.id, tema.id)}
-                          onDraftNombreChange={setTemaDraftNombre}
-                          onDraftHorasChange={setTemaDraftHoras}
-                          onEditorBlurCapture={handleTemaEditorBlurCapture}
-                          onEditorKeyDownCapture={
-                            handleTemaEditorKeyDownCapture
-                          }
-                          onNombreInputRef={(el) => {
-                            temaNombreInputElRef.current = el
-                          }}
-                          onDelete={() =>
-                            setDeleteDialog({
-                              type: 'tema',
-                              id: tema.id,
-                              parentId: unidad.id,
-                            })
-                          }
-                        />
-                      ))}
-                      <Button
-                        variant="ghost"
-                        size="sm"
-                        className="mt-2 w-full justify-start text-blue-600 hover:bg-blue-50 hover:text-blue-700"
-                        onClick={() => addTema(unidad.id)}
-                      >
-                        <Plus className="mr-2 h-3 w-3" /> Añadir subtema
-                      </Button>
-                    </div>
-                  </CardContent>
-                </CollapsibleContent>
-              </Collapsible>
-            </Card>
-          </div>
-        ))}
-      </div>
 
-      <div className="flex justify-center pt-2">
+                          <div className="ml-auto flex items-center gap-3">
-        <Button
+                            <span className="flex cursor-default items-center gap-1 text-xs font-medium text-slate-400">
-          variant="outline"
+                              <Clock className="h-3 w-3" />{' '}
-          className="gap-2"
+                              {unidad.temas.reduce(
-          onClick={(e) => {
+                                (sum, t) => sum + (t.horasEstimadas || 0),
-            // Evita que Enter vuelva a disparar el click sobre el botón.
+                                0,
-            e.currentTarget.blur()
+                              )}
-            addUnidad()
+                              h
-          }}
+                            </span>
-        >
+                            <Button
-          <Plus className="h-4 w-4" /> Nueva unidad
+                              variant="ghost"
-        </Button>
+                              size="icon"
-      </div>
+                              className="h-8 w-8 cursor-pointer text-slate-400 hover:text-red-500"
+                              onClick={() =>
+                                setDeleteDialog({
+                                  type: 'unidad',
+                                  id: unidad.id,
+                                })
+                              }
+                            >
+                              <Trash2 className="h-4 w-4" />
+                            </Button>
+                          </div>
+                        </div>
+                      </CardHeader>
+                      <CollapsibleContent>
+                        <CardContent className="bg-white pt-4">
+                          <div className="ml-10 space-y-1 border-l-2 border-slate-50 pl-4">
+                            {unidad.temas.map((tema, idx) => (
+                              <TemaRow
+                                key={tema.id}
+                                tema={tema}
+                                index={idx + 1}
+                                isEditing={
+                                  !!editingTema &&
+                                  editingTema.unitId === unidad.id &&
+                                  editingTema.temaId === tema.id
+                                }
+                                draftNombre={temaDraftNombre}
+                                draftHoras={temaDraftHoras}
+                                onBeginEdit={() =>
+                                  beginEditTema(unidad.id, tema.id)
+                                }
+                                onDraftNombreChange={setTemaDraftNombre}
+                                onDraftHorasChange={setTemaDraftHoras}
+                                onEditorBlurCapture={
+                                  handleTemaEditorBlurCapture
+                                }
+                                onEditorKeyDownCapture={
+                                  handleTemaEditorKeyDownCapture
+                                }
+                                onNombreInputRef={(el) => {
+                                  temaNombreInputElRef.current = el
+                                }}
+                                onDelete={() =>
+                                  setDeleteDialog({
+                                    type: 'tema',
+                                    id: tema.id,
+                                    parentId: unidad.id,
+                                  })
+                                }
+                              />
+                            ))}
+                            <Button
+                              variant="ghost"
+                              size="sm"
+                              className="mt-2 w-full cursor-pointer justify-start text-blue-600 hover:bg-blue-50 hover:text-blue-700"
+                              onClick={() => addTema(unidad.id)}
+                            >
+                              <Plus className="mr-2 h-3 w-3" /> Añadir subtema
+                            </Button>
+                          </div>
+                        </CardContent>
+                      </CollapsibleContent>
+                    </Collapsible>
+                  </Card>
+                </>
+              )}
+            </SortableUnidad>
+          ))}
+        </div>
+      </DragDropProvider>
 
       <DeleteConfirmDialog
         dialog={deleteDialog}
@@ -667,6 +908,9 @@ function TemaRow({
           <Input
             type="number"
             value={draftHoras}
+            min={0}
+            max={200}
+            step={0.5}
             onChange={(e) => onDraftHorasChange(e.target.value)}
             className="h-8 w-16 bg-white"
           />
@@ -675,7 +919,7 @@ function TemaRow({
         <>
           <button
             type="button"
-            className="flex flex-1 items-center gap-3 text-left"
+            className="flex flex-1 cursor-pointer items-center gap-3 text-left"
             onClick={(e) => {
               e.stopPropagation()
               onBeginEdit()
@@ -690,7 +934,7 @@ function TemaRow({
             <Button
               variant="ghost"
               size="icon"
-              className="h-7 w-7 text-slate-400 hover:text-blue-600"
+              className="h-7 w-7 cursor-pointer text-slate-400 hover:text-blue-600"
               onClick={(e) => {
                 e.stopPropagation()
                 onBeginEdit()
@@ -701,7 +945,7 @@ function TemaRow({
             <Button
               variant="ghost"
               size="icon"
-              className="h-7 w-7 text-slate-400 hover:text-red-500"
+              className="h-7 w-7 cursor-pointer text-slate-400 hover:text-red-500"
               onClick={(e) => {
                 e.stopPropagation()
                 onDelete()

src/components/asignaturas/detalle/SaveAsignatura/ImprovementCardProps.tsx

@@ -0,0 +1,208 @@
+import { Check, Loader2, BookOpen, Clock, ListChecks } from 'lucide-react'
+import { useState } from 'react'
+
+import type { IASugerencia } from '@/types/asignatura'
+
+import { Button } from '@/components/ui/button'
+import {
+  useUpdateAsignatura,
+  useSubject,
+  useUpdateSubjectRecommendation,
+} from '@/data'
+import { cn } from '@/lib/utils'
+
+interface ImprovementCardProps {
+  sug: IASugerencia
+  asignaturaId: string
+  onApplied: (campoKey: string) => void
+}
+
+export function ImprovementCard({
+  sug,
+  asignaturaId,
+  onApplied,
+}: ImprovementCardProps) {
+  const { data: asignatura } = useSubject(asignaturaId)
+  const updateAsignatura = useUpdateAsignatura()
+  const updateRecommendation = useUpdateSubjectRecommendation()
+
+  const [isApplying, setIsApplying] = useState(false)
+
+  const handleApply = async () => {
+    if (!asignatura) return
+
+    setIsApplying(true)
+    try {
+      // 1. Identificar a qué columna debe ir el guardado
+      let patchData = {}
+
+      if (sug.campoKey === 'contenido_tematico') {
+        // Se guarda directamente en la columna contenido_tematico
+        patchData = { contenido_tematico: sug.valorSugerido }
+      } else if (sug.campoKey === 'criterios_de_evaluacion') {
+        // Se guarda directamente en la columna criterios_de_evaluacion
+        patchData = { criterios_de_evaluacion: sug.valorSugerido }
+      } else {
+        // Otros campos (ciclo, fines, etc.) se siguen guardando en el JSON de la columna 'datos'
+        patchData = {
+          datos: {
+            ...asignatura.datos,
+            [sug.campoKey]: sug.valorSugerido,
+          },
+        }
+      }
+
+      // 2. Ejecutar la actualización con la estructura correcta
+      await updateAsignatura.mutateAsync({
+        asignaturaId: asignaturaId as any,
+        patch: patchData as any,
+      })
+
+      // 3. Marcar la recomendación como aplicada
+      await updateRecommendation.mutateAsync({
+        mensajeId: sug.messageId,
+        campoAfectado: sug.campoKey,
+      })
+      console.log(sug.campoKey)
+
+      onApplied(sug.campoKey)
+    } catch (error) {
+      console.error('Error al aplicar mejora:', error)
+    } finally {
+      setIsApplying(false)
+    }
+  }
+
+  // --- FUNCIÓN PARA RENDERIZAR EL CONTENIDO DE FORMA SEGURA ---
+  const renderContenido = (valor: any) => {
+    // Si no es un array, es texto simple
+    if (!Array.isArray(valor)) {
+      return <p className="italic">"{String(valor)}"</p>
+    }
+
+    // --- CASO 1: CONTENIDO TEMÁTICO (Detectamos si el primer objeto tiene 'unidad') ---
+    if (valor[0]?.hasOwnProperty('unidad')) {
+      return (
+        <div className="space-y-3">
+          {valor.map((u: any, idx: number) => (
+            <div
+              key={idx}
+              className="rounded-md border border-teal-100 bg-white p-2 shadow-sm"
+            >
+              <div className="mb-1 flex items-center gap-2 border-b border-slate-50 pb-1 text-[11px] font-bold text-teal-800">
+                <BookOpen size={12} /> Unidad {u.unidad}: {u.titulo}
+              </div>
+              <ul className="space-y-1">
+                {u.temas?.map((t: any, tidx: number) => (
+                  <li
+                    key={tidx}
+                    className="flex items-start justify-between gap-2 text-[10px] text-slate-600"
+                  >
+                    <span className="leading-tight">• {t.nombre}</span>
+                    <span className="flex shrink-0 items-center gap-0.5 font-mono text-slate-400">
+                      <Clock size={10} /> {t.horasEstimadas}h
+                    </span>
+                  </li>
+                ))}
+              </ul>
+            </div>
+          ))}
+        </div>
+      )
+    }
+
+    // --- CASO 2: CRITERIOS DE EVALUACIÓN (Detectamos si tiene 'criterio') ---
+    if (valor[0]?.hasOwnProperty('criterio')) {
+      return (
+        <div className="space-y-2">
+          <div className="mb-1 flex items-center gap-2 text-[10px] font-bold text-slate-400 uppercase">
+            <ListChecks size={12} /> Desglose de evaluación
+          </div>
+          {valor.map((c: any, idx: number) => (
+            <div
+              key={idx}
+              className="flex items-center justify-between gap-3 rounded-md border border-slate-100 bg-white p-2 shadow-sm"
+            >
+              <span className="text-[11px] leading-tight text-slate-700">
+                {c.criterio}
+              </span>
+              <div className="flex shrink-0 items-center gap-1 rounded-full border border-orange-100 bg-orange-50 px-2 py-0.5 text-[10px] font-bold text-orange-600">
+                {c.porcentaje}%
+              </div>
+            </div>
+          ))}
+          {/* Opcional: Suma total para verificar que de 100% */}
+          <div className="pt-1 text-right text-[9px] font-medium text-slate-400">
+            Total:{' '}
+            {valor.reduce(
+              (acc: number, curr: any) => acc + (curr.porcentaje || 0),
+              0,
+            )}
+            %
+          </div>
+        </div>
+      )
+    }
+
+    // Caso por defecto (Array genérico)
+    return (
+      <pre className="text-[10px]">
+        {/* JSON.stringify(valor, null, 2)*/ 'hola'}
+      </pre>
+    )
+  }
+
+  // --- ESTADO APLICADO ---
+  if (sug.aceptada) {
+    return (
+      <div className="flex flex-col rounded-xl border border-slate-100 bg-white p-3 opacity-80 shadow-sm">
+        <div className="mb-3 flex items-center justify-between gap-4">
+          <span className="text-sm font-bold text-slate-800">
+            {sug.campoNombre}
+          </span>
+          <div className="flex items-center gap-1.5 rounded-full border border-slate-100 bg-slate-50 px-3 py-1 text-xs font-medium text-slate-400">
+            <Check size={14} />
+            Aplicado
+          </div>
+        </div>
+        <div className="rounded-lg border border-teal-100 bg-teal-50/30 p-3 text-xs leading-relaxed text-slate-500">
+          {renderContenido(sug.valorSugerido)}
+        </div>
+      </div>
+    )
+  }
+
+  // --- ESTADO PENDIENTE ---
+  return (
+    <div className="group flex flex-col rounded-xl border border-teal-100 bg-white p-3 shadow-sm transition-all hover:border-teal-200">
+      <div className="mb-3 flex items-center justify-between gap-4">
+        <span className="max-w-[150px] truncate rounded-lg border border-teal-100 bg-teal-50/50 px-2.5 py-1 text-[10px] font-bold tracking-wider text-teal-700 uppercase">
+          {sug.campoNombre}
+        </span>
+
+        <Button
+          size="sm"
+          disabled={isApplying || !asignatura}
+          className="h-8 w-auto bg-teal-600 px-4 text-xs font-semibold shadow-sm hover:bg-teal-700"
+          onClick={handleApply}
+        >
+          {isApplying ? (
+            <Loader2 size={14} className="mr-1.5 animate-spin" />
+          ) : (
+            <Check size={14} className="mr-1.5" />
+          )}
+          {isApplying ? 'Aplicando...' : 'Aplicar mejora'}
+        </Button>
+      </div>
+
+      <div
+        className={cn(
+          'rounded-lg border border-dashed border-slate-200 bg-slate-50/50 p-3 text-xs leading-relaxed text-slate-600',
+          !Array.isArray(sug.valorSugerido) && 'line-clamp-4 italic',
+        )}
+      >
+        {renderContenido(sug.valorSugerido)}
+      </div>
+    </div>
+  )
+}

src/components/ui/radio-group.tsx

@@ -0,0 +1,43 @@
+import * as React from "react"
+import { CircleIcon } from "lucide-react"
+import { RadioGroup as RadioGroupPrimitive } from "radix-ui"
+
+import { cn } from "@/lib/utils"
+
+function RadioGroup({
+  className,
+  ...props
+}: React.ComponentProps<typeof RadioGroupPrimitive.Root>) {
+  return (
+    <RadioGroupPrimitive.Root
+      data-slot="radio-group"
+      className={cn("grid gap-3", className)}
+      {...props}
+    />
+  )
+}
+
+function RadioGroupItem({
+  className,
+  ...props
+}: React.ComponentProps<typeof RadioGroupPrimitive.Item>) {
+  return (
+    <RadioGroupPrimitive.Item
+      data-slot="radio-group-item"
+      className={cn(
+        "aspect-square size-4 shrink-0 rounded-full border border-input text-primary shadow-xs transition-[color,box-shadow] outline-none focus-visible:border-ring focus-visible:ring-[3px] focus-visible:ring-ring/50 disabled:cursor-not-allowed disabled:opacity-50 aria-invalid:border-destructive aria-invalid:ring-destructive/20 dark:bg-input/30 dark:aria-invalid:ring-destructive/40",
+        className
+      )}
+      {...props}
+    >
+      <RadioGroupPrimitive.Indicator
+        data-slot="radio-group-indicator"
+        className="relative flex items-center justify-center"
+      >
+        <CircleIcon className="absolute top-1/2 left-1/2 size-2 -translate-x-1/2 -translate-y-1/2 fill-primary" />
+      </RadioGroupPrimitive.Indicator>
+    </RadioGroupPrimitive.Item>
+  )
+}
+
+export { RadioGroup, RadioGroupItem }

src/components/wizard/WizardResponsiveHeader.tsx

@@ -5,16 +5,24 @@ export function WizardResponsiveHeader({
   wizard,
   methods,
   titleOverrides,
+  hiddenStepIds,
 }: {
   wizard: any
   methods: any
   titleOverrides?: Record<string, string>
+  hiddenStepIds?: Array<string>
 }) {
-  const idx = wizard.utils.getIndex(methods.current.id)
+  const hidden = new Set(hiddenStepIds ?? [])
-  const totalSteps = wizard.steps.length
+  const visibleSteps = (wizard.steps as Array<any>).filter(
-  const currentIndex = idx + 1
+    (s) => s && !hidden.has(s.id),
-  const hasNextStep = idx < totalSteps - 1
+  )
-  const nextStep = wizard.steps[currentIndex]
+
+  const idx = visibleSteps.findIndex((s) => s.id === methods.current.id)
+  const safeIdx = idx >= 0 ? idx : 0
+  const totalSteps = visibleSteps.length
+  const currentIndex = Math.min(safeIdx + 1, totalSteps)
+  const hasNextStep = safeIdx < totalSteps - 1
+  const nextStep = visibleSteps[safeIdx + 1]
 
   const resolveTitle = (step: any) => titleOverrides?.[step?.id] ?? step?.title
 
@@ -45,10 +53,11 @@ export function WizardResponsiveHeader({
 
       <div className="hidden sm:block">
         <wizard.Stepper.Navigation className="border-border/60 rounded-xl border bg-slate-50 p-2">
-          {wizard.steps.map((step: any) => (
+          {visibleSteps.map((step: any, visibleIdx: number) => (
             <wizard.Stepper.Step
               key={step.id}
               of={step.id}
+              icon={visibleIdx + 1}
               className="whitespace-nowrap"
             >
               <wizard.Stepper.Title>

src/data/api/ai.api.ts

@@ -247,3 +247,131 @@ export async function update_recommendation_applied_status(
 
   return true
 }
+
+// --- FUNCIONES DE ASIGNATURA ---
+
+export async function create_subject_conversation(subjectId: string) {
+  const supabase = supabaseBrowser()
+  const { data, error } = await supabase.functions.invoke(
+    'create-chat-conversation/asignatura/conversations', // Ruta corregida
+    {
+      method: 'POST',
+      body: {
+        asignatura_id: subjectId,
+        instanciador: 'alex',
+      },
+    },
+  )
+  if (error) throw error
+  return data // Retorna { conversation_asignatura: { id, ... } }
+}
+
+export async function ai_subject_chat_v2(payload: {
+  conversacionId: string
+  content: string
+  campos?: Array<string>
+}) {
+  const supabase = supabaseBrowser()
+  const { data, error } = await supabase.functions.invoke(
+    `create-chat-conversation/conversations/asignatura/${payload.conversacionId}/messages`, // Ruta corregida
+    {
+      method: 'POST',
+      body: {
+        content: payload.content,
+        campos: payload.campos || [],
+      },
+    },
+  )
+  if (error) throw error
+  return data
+}
+
+export async function getConversationBySubject(subjectId: string) {
+  const supabase = supabaseBrowser()
+  const { data, error } = await supabase
+    .from('conversaciones_asignatura') // Tabla corregida
+    .select('*')
+    .eq('asignatura_id', subjectId)
+    .order('creado_en', { ascending: false })
+
+  if (error) throw error
+  return data ?? []
+}
+
+export async function getMessagesBySubjectConversation(conversationId: string) {
+  const supabase = supabaseBrowser()
+  const { data, error } = await supabase
+    .from('asignatura_mensajes_ia' as any)
+    .select('*')
+    .eq('conversacion_asignatura_id', conversationId)
+    .order('fecha_creacion', { ascending: true })
+
+  if (error) throw error
+  return data ?? []
+}
+
+export async function update_subject_recommendation_applied(
+  mensajeId: string,
+  campoAfectado: string,
+) {
+  const supabase = supabaseBrowser()
+
+  // 1. Obtener propuesta actual
+  const { data: msgData, error: fetchError } = await supabase
+    .from('asignatura_mensajes_ia')
+    .select('propuesta')
+    .eq('id', mensajeId)
+    .single()
+
+  if (fetchError) throw fetchError
+  const propuestaActual = msgData?.propuesta as any
+
+  // 2. Marcar como aplicada
+  const nuevaPropuesta = {
+    ...propuestaActual,
+    recommendations: (propuestaActual.recommendations || []).map((rec: any) =>
+      rec.campo_afectado === campoAfectado ? { ...rec, aplicada: true } : rec,
+    ),
+  }
+
+  // 3. Update
+  const { error: updateError } = await supabase
+    .from('asignatura_mensajes_ia')
+    .update({ propuesta: nuevaPropuesta })
+    .eq('id', mensajeId)
+
+  if (updateError) throw updateError
+  return true
+}
+
+export async function update_subject_conversation_status(
+  conversacionId: string,
+  nuevoEstado: 'ARCHIVADA' | 'ACTIVA',
+) {
+  const supabase = supabaseBrowser()
+  const { data, error } = await supabase
+    .from('conversaciones_asignatura')
+    .update({ estado: nuevoEstado })
+    .eq('id', conversacionId)
+    .select()
+    .single()
+
+  if (error) throw error
+  return data
+}
+
+export async function update_subject_conversation_name(
+  conversacionId: string,
+  nuevoNombre: string,
+) {
+  const supabase = supabaseBrowser()
+  const { data, error } = await supabase
+    .from('conversaciones_asignatura')
+    .update({ nombre: nuevoNombre }) // Asumiendo que la columna es 'titulo' según tu código previo, o cambia a 'nombre'
+    .eq('id', conversacionId)
+    .select()
+    .single()
+
+  if (error) throw error
+  return data
+}

src/data/api/subjects.api.ts

@@ -42,8 +42,19 @@ const EDGE = {
 export type BuscarBibliografiaRequest = {
   searchTerms: {
     q: string
-    maxResults: number
+  }
+
+  google: {
     orderBy?: 'newest' | 'relevance'
+    langRestrict?: string
+    startIndex?: number
+    [k: string]: unknown
+  }
+
+  openLibrary: {
+    language?: string
+    page?: number
+    sort?: string
     [k: string]: unknown
   }
 }
@@ -82,20 +93,22 @@ export type GoogleBooksVolume = {
   [k: string]: unknown
 }
 
+export type OpenLibraryDoc = Record<string, unknown>
+
+export type EndpointResult =
+  | { endpoint: 'google'; item: GoogleBooksVolume }
+  | { endpoint: 'open_library'; item: OpenLibraryDoc }
+
 export async function buscar_bibliografia(
   input: BuscarBibliografiaRequest,
-): Promise<Array<GoogleBooksVolume>> {
+): Promise<Array<EndpointResult>> {
   const q = input.searchTerms.q
-  const maxResults = input.searchTerms.maxResults
 
   if (typeof q !== 'string' || q.trim().length < 1) {
     throw new Error('q es requerido')
   }
-  if (!Number.isInteger(maxResults) || maxResults < 0 || maxResults > 40) {
-    throw new Error('maxResults debe ser entero entre 0 y 40')
-  }
 
-  return await invokeEdge<Array<GoogleBooksVolume>>(
+  return await invokeEdge<Array<EndpointResult>>(
     EDGE.buscar_bibliografia,
     input,
     { headers: { 'Content-Type': 'application/json' } },
@@ -219,7 +232,7 @@ export async function subjects_bibliografia_list(
   const { data, error } = await supabase
     .from('bibliografia_asignatura')
     .select(
-      'id,asignatura_id,tipo,cita,tipo_fuente,biblioteca_item_id,creado_por,creado_en,actualizado_en',
+      'id,asignatura_id,tipo,cita,referencia_biblioteca,referencia_en_linea,creado_por,creado_en,actualizado_en',
     )
     .eq('asignatura_id', subjectId)
     .order('tipo', { ascending: true })

src/data/hooks/useAI.ts

@@ -1,9 +1,9 @@
 import { useMutation, useQuery, useQueryClient } from '@tanstack/react-query'
+import { useEffect } from 'react'
 
 import {
   ai_plan_chat_v2,
   ai_plan_improve,
-  ai_subject_chat,
   ai_subject_improve,
   create_conversation,
   get_chat_history,
@@ -13,10 +13,17 @@ import {
   update_recommendation_applied_status,
   update_conversation_title,
   getMessagesByConversation,
+  update_subject_conversation_status,
+  update_subject_recommendation_applied,
+  getMessagesBySubjectConversation,
+  getConversationBySubject,
+  ai_subject_chat_v2,
+  create_subject_conversation,
+  update_subject_conversation_name,
 } from '../api/ai.api'
+import { supabaseBrowser } from '../supabase/client'
 
-// eslint-disable-next-line node/prefer-node-protocol
+import type { UUID } from 'node:crypto'
-import type { UUID } from 'crypto'
 
 export function useAIPlanImprove() {
   return useMutation({ mutationFn: ai_plan_improve })
@@ -90,22 +97,58 @@ export function useConversationByPlan(planId: string | null) {
 }
 
 export function useMessagesByChat(conversationId: string | null) {
-  return useQuery({
+  const queryClient = useQueryClient()
-    // La queryKey debe ser única; incluimos el ID para que se refresque al cambiar de chat
+  const supabase = supabaseBrowser()
-    queryKey: ['conversation-messages', conversationId],
 
-    // Solo ejecutamos la función si el ID no es null o undefined
+  const query = useQuery({
+    queryKey: ['conversation-messages', conversationId],
     queryFn: () => {
       if (!conversationId) throw new Error('Conversation ID is required')
       return getMessagesByConversation(conversationId)
     },
-
-    // Importante: 'enabled' controla que no se dispare la petición si no hay ID
     enabled: !!conversationId,
-
-    // Opcional: Mantener los datos previos mientras se carga la nueva conversación
     placeholderData: (previousData) => previousData,
   })
+
+  useEffect(() => {
+    if (!conversationId) return
+
+    // Suscribirse a cambios en los mensajes de ESTA conversación
+    const channel = supabase
+      .channel(`realtime-messages-${conversationId}`)
+      .on(
+        'postgres_changes',
+        {
+          event: '*', // Escuchamos INSERT y UPDATE
+          schema: 'public',
+          table: 'plan_mensajes_ia',
+          filter: `conversacion_plan_id=eq.${conversationId}`,
+        },
+        (payload) => {
+          // Opción A: Invalidar la query para que React Query haga refetch (más seguro)
+          queryClient.invalidateQueries({
+            queryKey: ['conversation-messages', conversationId],
+          })
+
+          /* Opción B: Actualización manual del caché (más rápido/fluido)
+             if (payload.eventType === 'INSERT') {
+               queryClient.setQueryData(['conversation-messages', conversationId], (old: any) => [...old, payload.new])
+             } else if (payload.eventType === 'UPDATE') {
+               queryClient.setQueryData(['conversation-messages', conversationId], (old: any) => 
+                 old.map((m: any) => m.id === payload.new.id ? payload.new : m)
+               )
+             }
+          */
+        },
+      )
+      .subscribe()
+
+    return () => {
+      supabase.removeChannel(channel)
+    }
+  }, [conversationId, queryClient, supabase])
+
+  return query
 }
 
 export function useUpdateRecommendationApplied() {
@@ -137,10 +180,6 @@ export function useAISubjectImprove() {
   return useMutation({ mutationFn: ai_subject_improve })
 }
 
-export function useAISubjectChat() {
-  return useMutation({ mutationFn: ai_subject_chat })
-}
-
 export function useLibrarySearch() {
   return useMutation({ mutationFn: library_search })
 }
@@ -157,3 +196,142 @@ export function useUpdateConversationTitle() {
     },
   })
 }
+
+// Asignaturas
+
+export function useAISubjectChat() {
+  const qc = useQueryClient()
+
+  return useMutation({
+    mutationFn: async (payload: {
+      subjectId: UUID
+      content: string
+      campos?: Array<string>
+      conversacionId?: string
+    }) => {
+      let currentId = payload.conversacionId
+
+      // 1. Si no hay ID, creamos la conversación de asignatura
+      if (!currentId) {
+        const response = await create_subject_conversation(payload.subjectId)
+        currentId = response.conversation_asignatura.id
+      }
+
+      // 2. Enviamos mensaje al endpoint de asignatura
+      const result = await ai_subject_chat_v2({
+        conversacionId: currentId!,
+        content: payload.content,
+        campos: payload.campos,
+      })
+
+      return { ...result, conversacionId: currentId }
+    },
+    onSuccess: (data) => {
+      // Invalidamos mensajes para que se refresque el chat
+      qc.invalidateQueries({
+        queryKey: ['subject-messages', data.conversacionId],
+      })
+    },
+  })
+}
+
+export function useConversationBySubject(subjectId: string | null) {
+  return useQuery({
+    queryKey: ['conversation-by-subject', subjectId],
+    queryFn: () => getConversationBySubject(subjectId!),
+    enabled: !!subjectId,
+  })
+}
+
+export function useMessagesBySubjectChat(conversationId: string | null) {
+  const queryClient = useQueryClient()
+
+  const query = useQuery({
+    queryKey: ['subject-messages', conversationId],
+    queryFn: async () => {
+      if (!conversationId) throw new Error('Conversation ID is required')
+      return getMessagesBySubjectConversation(conversationId)
+    },
+    enabled: !!conversationId,
+    placeholderData: (previousData) => previousData,
+  })
+
+  useEffect(() => {
+    if (!conversationId) return
+
+    const supabase = supabaseBrowser()
+
+    // Suscripción a cambios en la tabla específica para esta conversación
+    const channel = supabase
+      .channel(`subject_messages_${conversationId}`)
+      .on(
+        'postgres_changes',
+        {
+          event: 'UPDATE', // Solo nos interesan las actualizaciones (cuando pasa de PROCESANDO a COMPLETADO)
+          schema: 'public',
+          table: 'asignatura_mensajes_ia',
+          filter: `conversacion_asignatura_id=eq.${conversationId}`,
+        },
+        (payload) => {
+          // Si el mensaje se completó o dio error, invalidamos la caché para traer los datos nuevos
+          if (
+            payload.new.estado === 'COMPLETADO' ||
+            payload.new.estado === 'ERROR'
+          ) {
+            queryClient.invalidateQueries({
+              queryKey: ['subject-messages', conversationId],
+            })
+          }
+        },
+      )
+      .subscribe()
+
+    return () => {
+      supabase.removeChannel(channel)
+    }
+  }, [conversationId, queryClient])
+
+  return query
+}
+
+export function useUpdateSubjectRecommendation() {
+  const qc = useQueryClient()
+
+  return useMutation({
+    mutationFn: (payload: { mensajeId: string; campoAfectado: string }) =>
+      update_subject_recommendation_applied(
+        payload.mensajeId,
+        payload.campoAfectado,
+      ),
+    onSuccess: () => {
+      // Refrescamos los mensajes para ver el check de "aplicado"
+      qc.invalidateQueries({ queryKey: ['subject-messages'] })
+    },
+  })
+}
+
+export function useUpdateSubjectConversationStatus() {
+  const qc = useQueryClient()
+
+  return useMutation({
+    mutationFn: (payload: { id: string; estado: 'ARCHIVADA' | 'ACTIVA' }) =>
+      update_subject_conversation_status(payload.id, payload.estado),
+    onSuccess: () => {
+      qc.invalidateQueries({ queryKey: ['conversation-by-subject'] })
+    },
+  })
+}
+
+export function useUpdateSubjectConversationName() {
+  const qc = useQueryClient()
+
+  return useMutation({
+    mutationFn: (payload: { id: string; nombre: string }) =>
+      update_subject_conversation_name(payload.id, payload.nombre),
+    onSuccess: () => {
+      qc.invalidateQueries({ queryKey: ['conversation-by-subject'] })
+      // También invalidamos los mensajes si el título se muestra en la cabecera
+      qc.invalidateQueries({ queryKey: ['subject-messages'] })
+    },
+  })
+}

src/routeTree.gen.ts

@@ -31,6 +31,8 @@ import { Route as PlanesPlanIdAsignaturasAsignaturaIdDocumentoRouteImport } from
 import { Route as PlanesPlanIdAsignaturasAsignaturaIdContenidoRouteImport } from './routes/planes/$planId/asignaturas/$asignaturaId/contenido'
 import { Route as PlanesPlanIdAsignaturasAsignaturaIdBibliografiaRouteImport } from './routes/planes/$planId/asignaturas/$asignaturaId/bibliografia'
 import { Route as PlanesPlanIdDetalleAsignaturasNuevaRouteImport } from './routes/planes/$planId/_detalle/asignaturas/nueva'
+import { Route as PlanesPlanIdAsignaturasAsignaturaIdBibliografiaIndexRouteImport } from './routes/planes/$planId/asignaturas/$asignaturaId/bibliografia/index'
+import { Route as PlanesPlanIdAsignaturasAsignaturaIdBibliografiaNuevaRouteImport } from './routes/planes/$planId/asignaturas/$asignaturaId/bibliografia/nueva'
 
 const LoginRoute = LoginRouteImport.update({
   id: '/login',
@@ -156,6 +158,18 @@ const PlanesPlanIdDetalleAsignaturasNuevaRoute =
     path: '/nueva',
     getParentRoute: () => PlanesPlanIdDetalleAsignaturasRoute,
   } as any)
+const PlanesPlanIdAsignaturasAsignaturaIdBibliografiaIndexRoute =
+  PlanesPlanIdAsignaturasAsignaturaIdBibliografiaIndexRouteImport.update({
+    id: '/',
+    path: '/',
+    getParentRoute: () => PlanesPlanIdAsignaturasAsignaturaIdBibliografiaRoute,
+  } as any)
+const PlanesPlanIdAsignaturasAsignaturaIdBibliografiaNuevaRoute =
+  PlanesPlanIdAsignaturasAsignaturaIdBibliografiaNuevaRouteImport.update({
+    id: '/nueva',
+    path: '/nueva',
+    getParentRoute: () => PlanesPlanIdAsignaturasAsignaturaIdBibliografiaRoute,
+  } as any)
 
 export interface FileRoutesByFullPath {
   '/': typeof IndexRoute
@@ -174,12 +188,14 @@ export interface FileRoutesByFullPath {
   '/planes/$planId/mapa': typeof PlanesPlanIdDetalleMapaRoute
   '/planes/$planId/': typeof PlanesPlanIdDetalleIndexRoute
   '/planes/$planId/asignaturas/nueva': typeof PlanesPlanIdDetalleAsignaturasNuevaRoute
-  '/planes/$planId/asignaturas/$asignaturaId/bibliografia': typeof PlanesPlanIdAsignaturasAsignaturaIdBibliografiaRoute
+  '/planes/$planId/asignaturas/$asignaturaId/bibliografia': typeof PlanesPlanIdAsignaturasAsignaturaIdBibliografiaRouteWithChildren
   '/planes/$planId/asignaturas/$asignaturaId/contenido': typeof PlanesPlanIdAsignaturasAsignaturaIdContenidoRoute
   '/planes/$planId/asignaturas/$asignaturaId/documento': typeof PlanesPlanIdAsignaturasAsignaturaIdDocumentoRoute
   '/planes/$planId/asignaturas/$asignaturaId/historial': typeof PlanesPlanIdAsignaturasAsignaturaIdHistorialRoute
   '/planes/$planId/asignaturas/$asignaturaId/iaasignatura': typeof PlanesPlanIdAsignaturasAsignaturaIdIaasignaturaRoute
   '/planes/$planId/asignaturas/$asignaturaId/': typeof PlanesPlanIdAsignaturasAsignaturaIdIndexRoute
+  '/planes/$planId/asignaturas/$asignaturaId/bibliografia/nueva': typeof PlanesPlanIdAsignaturasAsignaturaIdBibliografiaNuevaRoute
+  '/planes/$planId/asignaturas/$asignaturaId/bibliografia/': typeof PlanesPlanIdAsignaturasAsignaturaIdBibliografiaIndexRoute
 }
 export interface FileRoutesByTo {
   '/': typeof IndexRoute
@@ -196,12 +212,13 @@ export interface FileRoutesByTo {
   '/planes/$planId/mapa': typeof PlanesPlanIdDetalleMapaRoute
   '/planes/$planId': typeof PlanesPlanIdDetalleIndexRoute
   '/planes/$planId/asignaturas/nueva': typeof PlanesPlanIdDetalleAsignaturasNuevaRoute
-  '/planes/$planId/asignaturas/$asignaturaId/bibliografia': typeof PlanesPlanIdAsignaturasAsignaturaIdBibliografiaRoute
   '/planes/$planId/asignaturas/$asignaturaId/contenido': typeof PlanesPlanIdAsignaturasAsignaturaIdContenidoRoute
   '/planes/$planId/asignaturas/$asignaturaId/documento': typeof PlanesPlanIdAsignaturasAsignaturaIdDocumentoRoute
   '/planes/$planId/asignaturas/$asignaturaId/historial': typeof PlanesPlanIdAsignaturasAsignaturaIdHistorialRoute
   '/planes/$planId/asignaturas/$asignaturaId/iaasignatura': typeof PlanesPlanIdAsignaturasAsignaturaIdIaasignaturaRoute
   '/planes/$planId/asignaturas/$asignaturaId': typeof PlanesPlanIdAsignaturasAsignaturaIdIndexRoute
+  '/planes/$planId/asignaturas/$asignaturaId/bibliografia/nueva': typeof PlanesPlanIdAsignaturasAsignaturaIdBibliografiaNuevaRoute
+  '/planes/$planId/asignaturas/$asignaturaId/bibliografia': typeof PlanesPlanIdAsignaturasAsignaturaIdBibliografiaIndexRoute
 }
 export interface FileRoutesById {
   __root__: typeof rootRouteImport
@@ -221,12 +238,14 @@ export interface FileRoutesById {
   '/planes/$planId/_detalle/mapa': typeof PlanesPlanIdDetalleMapaRoute
   '/planes/$planId/_detalle/': typeof PlanesPlanIdDetalleIndexRoute
   '/planes/$planId/_detalle/asignaturas/nueva': typeof PlanesPlanIdDetalleAsignaturasNuevaRoute
-  '/planes/$planId/asignaturas/$asignaturaId/bibliografia': typeof PlanesPlanIdAsignaturasAsignaturaIdBibliografiaRoute
+  '/planes/$planId/asignaturas/$asignaturaId/bibliografia': typeof PlanesPlanIdAsignaturasAsignaturaIdBibliografiaRouteWithChildren
   '/planes/$planId/asignaturas/$asignaturaId/contenido': typeof PlanesPlanIdAsignaturasAsignaturaIdContenidoRoute
   '/planes/$planId/asignaturas/$asignaturaId/documento': typeof PlanesPlanIdAsignaturasAsignaturaIdDocumentoRoute
   '/planes/$planId/asignaturas/$asignaturaId/historial': typeof PlanesPlanIdAsignaturasAsignaturaIdHistorialRoute
   '/planes/$planId/asignaturas/$asignaturaId/iaasignatura': typeof PlanesPlanIdAsignaturasAsignaturaIdIaasignaturaRoute
   '/planes/$planId/asignaturas/$asignaturaId/': typeof PlanesPlanIdAsignaturasAsignaturaIdIndexRoute
+  '/planes/$planId/asignaturas/$asignaturaId/bibliografia/nueva': typeof PlanesPlanIdAsignaturasAsignaturaIdBibliografiaNuevaRoute
+  '/planes/$planId/asignaturas/$asignaturaId/bibliografia/': typeof PlanesPlanIdAsignaturasAsignaturaIdBibliografiaIndexRoute
 }
 export interface FileRouteTypes {
   fileRoutesByFullPath: FileRoutesByFullPath
@@ -253,6 +272,8 @@ export interface FileRouteTypes {
     | '/planes/$planId/asignaturas/$asignaturaId/historial'
     | '/planes/$planId/asignaturas/$asignaturaId/iaasignatura'
     | '/planes/$planId/asignaturas/$asignaturaId/'
+    | '/planes/$planId/asignaturas/$asignaturaId/bibliografia/nueva'
+    | '/planes/$planId/asignaturas/$asignaturaId/bibliografia/'
   fileRoutesByTo: FileRoutesByTo
   to:
     | '/'
@@ -269,12 +290,13 @@ export interface FileRouteTypes {
     | '/planes/$planId/mapa'
     | '/planes/$planId'
     | '/planes/$planId/asignaturas/nueva'
-    | '/planes/$planId/asignaturas/$asignaturaId/bibliografia'
     | '/planes/$planId/asignaturas/$asignaturaId/contenido'
     | '/planes/$planId/asignaturas/$asignaturaId/documento'
     | '/planes/$planId/asignaturas/$asignaturaId/historial'
     | '/planes/$planId/asignaturas/$asignaturaId/iaasignatura'
     | '/planes/$planId/asignaturas/$asignaturaId'
+    | '/planes/$planId/asignaturas/$asignaturaId/bibliografia/nueva'
+    | '/planes/$planId/asignaturas/$asignaturaId/bibliografia'
   id:
     | '__root__'
     | '/'
@@ -299,6 +321,8 @@ export interface FileRouteTypes {
     | '/planes/$planId/asignaturas/$asignaturaId/historial'
     | '/planes/$planId/asignaturas/$asignaturaId/iaasignatura'
     | '/planes/$planId/asignaturas/$asignaturaId/'
+    | '/planes/$planId/asignaturas/$asignaturaId/bibliografia/nueva'
+    | '/planes/$planId/asignaturas/$asignaturaId/bibliografia/'
   fileRoutesById: FileRoutesById
 }
 export interface RootRouteChildren {
@@ -467,6 +491,20 @@ declare module '@tanstack/react-router' {
       preLoaderRoute: typeof PlanesPlanIdDetalleAsignaturasNuevaRouteImport
       parentRoute: typeof PlanesPlanIdDetalleAsignaturasRoute
     }
+    '/planes/$planId/asignaturas/$asignaturaId/bibliografia/': {
+      id: '/planes/$planId/asignaturas/$asignaturaId/bibliografia/'
+      path: '/'
+      fullPath: '/planes/$planId/asignaturas/$asignaturaId/bibliografia/'
+      preLoaderRoute: typeof PlanesPlanIdAsignaturasAsignaturaIdBibliografiaIndexRouteImport
+      parentRoute: typeof PlanesPlanIdAsignaturasAsignaturaIdBibliografiaRoute
+    }
+    '/planes/$planId/asignaturas/$asignaturaId/bibliografia/nueva': {
+      id: '/planes/$planId/asignaturas/$asignaturaId/bibliografia/nueva'
+      path: '/nueva'
+      fullPath: '/planes/$planId/asignaturas/$asignaturaId/bibliografia/nueva'
+      preLoaderRoute: typeof PlanesPlanIdAsignaturasAsignaturaIdBibliografiaNuevaRouteImport
+      parentRoute: typeof PlanesPlanIdAsignaturasAsignaturaIdBibliografiaRoute
+    }
   }
 }
 
@@ -521,8 +559,26 @@ const PlanesPlanIdDetalleRouteChildren: PlanesPlanIdDetalleRouteChildren = {
 const PlanesPlanIdDetalleRouteWithChildren =
   PlanesPlanIdDetalleRoute._addFileChildren(PlanesPlanIdDetalleRouteChildren)
 
+interface PlanesPlanIdAsignaturasAsignaturaIdBibliografiaRouteChildren {
+  PlanesPlanIdAsignaturasAsignaturaIdBibliografiaNuevaRoute: typeof PlanesPlanIdAsignaturasAsignaturaIdBibliografiaNuevaRoute
+  PlanesPlanIdAsignaturasAsignaturaIdBibliografiaIndexRoute: typeof PlanesPlanIdAsignaturasAsignaturaIdBibliografiaIndexRoute
+}
+
+const PlanesPlanIdAsignaturasAsignaturaIdBibliografiaRouteChildren: PlanesPlanIdAsignaturasAsignaturaIdBibliografiaRouteChildren =
+  {
+    PlanesPlanIdAsignaturasAsignaturaIdBibliografiaNuevaRoute:
+      PlanesPlanIdAsignaturasAsignaturaIdBibliografiaNuevaRoute,
+    PlanesPlanIdAsignaturasAsignaturaIdBibliografiaIndexRoute:
+      PlanesPlanIdAsignaturasAsignaturaIdBibliografiaIndexRoute,
+  }
+
+const PlanesPlanIdAsignaturasAsignaturaIdBibliografiaRouteWithChildren =
+  PlanesPlanIdAsignaturasAsignaturaIdBibliografiaRoute._addFileChildren(
+    PlanesPlanIdAsignaturasAsignaturaIdBibliografiaRouteChildren,
+  )
+
 interface PlanesPlanIdAsignaturasAsignaturaIdRouteRouteChildren {
-  PlanesPlanIdAsignaturasAsignaturaIdBibliografiaRoute: typeof PlanesPlanIdAsignaturasAsignaturaIdBibliografiaRoute
+  PlanesPlanIdAsignaturasAsignaturaIdBibliografiaRoute: typeof PlanesPlanIdAsignaturasAsignaturaIdBibliografiaRouteWithChildren
   PlanesPlanIdAsignaturasAsignaturaIdContenidoRoute: typeof PlanesPlanIdAsignaturasAsignaturaIdContenidoRoute
   PlanesPlanIdAsignaturasAsignaturaIdDocumentoRoute: typeof PlanesPlanIdAsignaturasAsignaturaIdDocumentoRoute
   PlanesPlanIdAsignaturasAsignaturaIdHistorialRoute: typeof PlanesPlanIdAsignaturasAsignaturaIdHistorialRoute
@@ -533,7 +589,7 @@ interface PlanesPlanIdAsignaturasAsignaturaIdRouteRouteChildren {
 const PlanesPlanIdAsignaturasAsignaturaIdRouteRouteChildren: PlanesPlanIdAsignaturasAsignaturaIdRouteRouteChildren =
   {
     PlanesPlanIdAsignaturasAsignaturaIdBibliografiaRoute:
-      PlanesPlanIdAsignaturasAsignaturaIdBibliografiaRoute,
+      PlanesPlanIdAsignaturasAsignaturaIdBibliografiaRouteWithChildren,
     PlanesPlanIdAsignaturasAsignaturaIdContenidoRoute:
       PlanesPlanIdAsignaturasAsignaturaIdContenidoRoute,
     PlanesPlanIdAsignaturasAsignaturaIdDocumentoRoute:

src/routes/planes/$planId/_detalle/iaplan.tsx

@@ -13,8 +13,8 @@ import {
   X,
   MessageSquarePlus,
   Archive,
-  RotateCcw,
   Loader2,
+  Sparkles,
 } from 'lucide-react'
 import { useState, useEffect, useRef, useMemo } from 'react'
 
@@ -22,10 +22,17 @@ import type { UploadedFile } from '@/components/planes/wizard/PasoDetallesPanel/
 
 import { ImprovementCard } from '@/components/planes/detalle/Ia/ImprovementCard'
 import ReferenciasParaIA from '@/components/planes/wizard/PasoDetallesPanel/ReferenciasParaIA'
+import { Avatar, AvatarFallback } from '@/components/ui/avatar'
 import { Button } from '@/components/ui/button'
 import { Drawer, DrawerContent } from '@/components/ui/drawer'
 import { ScrollArea } from '@/components/ui/scroll-area'
 import { Textarea } from '@/components/ui/textarea'
+import {
+  Tooltip,
+  TooltipContent,
+  TooltipProvider,
+  TooltipTrigger,
+} from '@/components/ui/tooltip'
 import {
   useAIPlanChat,
   useConversationByPlan,
@@ -98,14 +105,14 @@ function RouteComponent() {
   const [openIA, setOpenIA] = useState(false)
   const { mutateAsync: sendChat, isPending: isLoading } = useAIPlanChat()
   const { mutate: updateStatusMutation } = useUpdateConversationStatus()
-
+  const [isSyncing, setIsSyncing] = useState(false)
   const [activeChatId, setActiveChatId] = useState<string | undefined>(
     undefined,
   )
   const { data: lastConversation, isLoading: isLoadingConv } =
     useConversationByPlan(planId)
   const { data: mensajesDelChat, isLoading: isLoadingMessages } =
-    useMessagesByChat(activeChatId)
+    useMessagesByChat(activeChatId ?? null) // Si es undefined, pasa null
   const [selectedArchivoIds, setSelectedArchivoIds] = useState<Array<string>>(
     [],
   )
@@ -152,10 +159,6 @@ function RouteComponent() {
     )
   }, [availableFields, filterQuery, selectedFields])
 
-  const activeChatData = useMemo(() => {
-    return lastConversation?.find((chat: any) => chat.id === activeChatId)
-  }, [lastConversation, activeChatId])
-
   const chatMessages = useMemo(() => {
     if (!activeChatId || !mensajesDelChat) return []
 
@@ -231,7 +234,7 @@ function RouteComponent() {
     scrollToBottom()
   }, [chatMessages, isLoading])
 
-  useEffect(() => {
+  /* useEffect(() => {
     // Verificamos cuáles campos de la lista "selectedFields" ya no están presentes en el texto del input
     const camposActualizados = selectedFields.filter((field) =>
       input.includes(field.label),
@@ -241,7 +244,7 @@ function RouteComponent() {
     if (camposActualizados.length !== selectedFields.length) {
       setSelectedFields(camposActualizados)
     }
-  }, [input, selectedFields])
+  }, [input, selectedFields]) */
 
   useEffect(() => {
     if (isLoadingConv || isSending) return
@@ -274,6 +277,7 @@ function RouteComponent() {
     isSending,
     messages.length,
     chatMessages.length,
+    messages,
   ])
 
   useEffect(() => {
@@ -288,7 +292,7 @@ function RouteComponent() {
     setInput((prev) =>
       injectFieldsIntoInput(prev || 'Mejora este campo:', [field]),
     )
-  }, [availableFields])
+  }, [availableFields, routerState.location.state])
 
   const createNewChat = () => {
     setActiveChatId(undefined) // Al ser undefined, el próximo handleSend creará uno nuevo
@@ -300,7 +304,7 @@ function RouteComponent() {
       },
     ])
     setInput('')
-    setSelectedFields([])
+    // setSelectedFields([])
   }
 
   const archiveChat = (e: React.MouseEvent, id: string) => {
@@ -404,17 +408,16 @@ function RouteComponent() {
     if (isSending || (!rawText.trim() && selectedFields.length === 0)) return
 
     const currentFields = [...selectedFields]
+    const finalContent = buildPrompt(rawText, currentFields)
     setIsSending(true)
-    setOptimisticMessage(rawText)
+    setOptimisticMessage(finalContent)
-
-    // Limpiar input inmediatamente para feedback visual
     setInput('')
-    setSelectedFields([])
+    // setSelectedFields([])
 
     try {
       const payload = {
-        planId,
+        planId: planId as any,
-        content: buildPrompt(rawText, currentFields),
+        content: finalContent,
         conversacionId: activeChatId,
         campos:
           currentFields.length > 0
@@ -423,13 +426,12 @@ function RouteComponent() {
       }
 
       const response = await sendChat(payload)
-
+      setIsSyncing(true)
-      // IMPORTANTE: Si es un chat nuevo, actualizar el ID antes de invalidar
       if (response.conversacionId && response.conversacionId !== activeChatId) {
         setActiveChatId(response.conversacionId)
       }
 
-      // Invalidar ambas para asegurar que la lista de la izquierda y los mensajes se
+      // ESPERAMOS a que la caché se actualice antes de quitar el "isSending"
       await Promise.all([
         queryClient.invalidateQueries({
           queryKey: ['conversation-by-plan', planId],
@@ -440,12 +442,27 @@ function RouteComponent() {
       ])
     } catch (error) {
       console.error('Error:', error)
-    } finally {
-      setIsSending(false)
       setOptimisticMessage(null)
+    } finally {
+      // Solo ahora quitamos los indicadores de carga
+      setIsSending(false)
+      // setOptimisticMessage(null)
     }
   }
 
+  useEffect(() => {
+    if (!isSyncing || !mensajesDelChat || mensajesDelChat.length === 0) return
+
+    // Forzamos el tipo a 'any' o a tu interfaz de mensaje para saltarnos la unión de tipos compleja
+    const ultimoMensajeDB = mensajesDelChat[mensajesDelChat.length - 1] as any
+
+    // Ahora la validación es directa y no debería dar avisos de "unnecessary"
+    if (ultimoMensajeDB?.respuesta) {
+      setIsSyncing(false)
+      setOptimisticMessage(null)
+    }
+  }, [mensajesDelChat, isSyncing])
+
   const totalReferencias = useMemo(() => {
     return (
       selectedArchivoIds.length +
@@ -497,76 +514,99 @@ function RouteComponent() {
                 <div
                   key={chat.id}
                   onClick={() => setActiveChatId(chat.id)}
-                  className={`group relative flex w-full cursor-pointer items-center gap-3 rounded-lg px-3 py-3 text-sm transition-colors ${
+                  className={`group relative flex w-full items-center justify-between overflow-hidden rounded-lg px-3 py-3 text-sm transition-colors ${
                     activeChatId === chat.id
                       ? 'bg-slate-100 font-medium text-slate-900'
                       : 'text-slate-600 hover:bg-slate-50'
                   }`}
                 >
-                  <FileText size={16} className="shrink-0 opacity-40" />
+                  {/* LADO IZQUIERDO: Icono + Texto con Tooltip */}
+                  <div className="flex min-w-0 flex-1 items-center gap-3">
+                    <FileText size={16} className="shrink-0 opacity-40" />
 
-                  <span
+                    <TooltipProvider delayDuration={400}>
-                    ref={editingChatId === chat.id ? editableRef : null}
+                      <Tooltip>
-                    contentEditable={editingChatId === chat.id}
+                        <TooltipTrigger asChild>
-                    suppressContentEditableWarning={true}
+                          {/* Este contenedor es el que obliga al span a truncarse */}
-                    className={`truncate pr-14 transition-all outline-none ${
+                          <div className="max-w-[calc(100%-48px)] min-w-0 flex-1">
-                      editingChatId === chat.id
+                            <span
-                        ? 'min-w-[50px] cursor-text rounded bg-white px-1 ring-1 ring-teal-500'
+                              ref={
-                        : 'cursor-pointer'
+                                editingChatId === chat.id ? editableRef : null
+                              }
+                              contentEditable={editingChatId === chat.id}
+                              suppressContentEditableWarning={true}
+                              className={`block truncate outline-none ${
+                                editingChatId === chat.id
+                                  ? 'max-h-20 min-w-[100px] cursor-text overflow-y-auto rounded bg-white px-1 break-all shadow-sm ring-1 ring-teal-500'
+                                  : 'cursor-pointer'
+                              }`}
+                              onDoubleClick={(e) => {
+                                e.stopPropagation()
+                                setEditingChatId(chat.id)
+                              }}
+                              onKeyDown={(e) => {
+                                if (e.key === 'Enter') {
+                                  e.preventDefault()
+                                  e.currentTarget.blur()
+                                }
+                                if (e.key === 'Escape') {
+                                  setEditingChatId(null)
+                                  e.currentTarget.textContent =
+                                    chat.nombre || ''
+                                }
+                              }}
+                              onBlur={(e) => {
+                                if (editingChatId === chat.id) {
+                                  const newTitle =
+                                    e.currentTarget.textContent?.trim() || ''
+                                  if (newTitle && newTitle !== chat.nombre) {
+                                    updateTitleMutation({
+                                      id: chat.id,
+                                      nombre: newTitle,
+                                    })
+                                  }
+                                  setEditingChatId(null)
+                                }
+                              }}
+                            >
+                              {chat.nombre ||
+                                `Chat ${chat.creado_en.split('T')[0]}`}
+                            </span>
+                          </div>
+                        </TooltipTrigger>
+
+                        {/* Tooltip: Solo aparece si no estás editando y el texto es largo */}
+                        {editingChatId !== chat.id && (
+                          <TooltipContent
+                            side="right"
+                            className="max-w-[280px] break-all"
+                          >
+                            {chat.nombre || 'Conversación'}
+                          </TooltipContent>
+                        )}
+                      </Tooltip>
+                    </TooltipProvider>
+                  </div>
+
+                  {/* LADO DERECHO: Acciones con shrink-0 para que no se muevan */}
+                  <div
+                    className={`flex shrink-0 items-center gap-1 pl-2 opacity-0 transition-opacity group-hover:opacity-100 ${
+                      activeChatId === chat.id ? 'bg-slate-100' : 'bg-slate-50'
                     }`}
-                    onDoubleClick={(e) => {
-                      e.stopPropagation()
-                      setEditingChatId(chat.id)
-                    }}
-                    onKeyDown={(e) => {
-                      if (e.key === 'Enter') {
-                        e.preventDefault()
-                        const newTitle = e.currentTarget.textContent || ''
-                        updateTitleMutation(
-                          { id: chat.id, nombre: newTitle },
-                          {
-                            onSuccess: () => setEditingChatId(null),
-                          },
-                        )
-                      }
-                      if (e.key === 'Escape') {
-                        setEditingChatId(null)
-
-                        e.currentTarget.textContent = chat.nombre || ''
-                      }
-                    }}
-                    onBlur={(e) => {
-                      if (editingChatId === chat.id) {
-                        const newTitle = e.currentTarget.textContent || ''
-                        if (newTitle !== chat.nombre) {
-                          updateTitleMutation({ id: chat.id, nombre: newTitle })
-                        }
-                        setEditingChatId(null)
-                      }
-                    }}
-                    onClick={(e) => {
-                      if (editingChatId === chat.id) e.stopPropagation()
-                    }}
                   >
-                    {chat.nombre || `Chat ${chat.creado_en.split('T')[0]}`}
-                  </span>
-
-                  {/* ACCIONES */}
-                  <div className="absolute right-2 flex items-center gap-1 opacity-0 group-hover:opacity-100">
                     <button
                       onClick={(e) => {
                         e.stopPropagation()
                         setEditingChatId(chat.id)
-                        // Pequeño timeout para asegurar que el DOM se actualice antes de enfocar
                         setTimeout(() => editableRef.current?.focus(), 50)
                       }}
-                      className="p-1 text-slate-400 hover:text-teal-600"
+                      className="rounded-md p-1 text-slate-400 transition-colors hover:text-teal-600"
                     >
                       <Send size={12} className="rotate-45" />
                     </button>
                     <button
                       onClick={(e) => archiveChat(e, chat.id)}
-                      className="p-1 text-slate-400 hover:text-amber-600"
+                      className="rounded-md p-1 text-slate-400 transition-colors hover:text-amber-600"
                     >
                       <Archive size={14} />
                     </button>
@@ -574,24 +614,26 @@ function RouteComponent() {
                 </div>
               ))
             ) : (
-              /* ... Resto del código de archivados (sin cambios) ... */
+              /* Sección de archivados */
-              <div className="animate-in fade-in slide-in-from-left-2">
+              <div className="animate-in fade-in slide-in-from-left-2 px-1">
                 <p className="mb-2 px-2 text-[10px] font-bold text-slate-400 uppercase">
                   Archivados
                 </p>
                 {archivedChats.map((chat) => (
                   <div
                     key={chat.id}
-                    className="group relative mb-1 flex w-full items-center gap-3 rounded-lg bg-slate-50/50 px-3 py-2 text-sm text-slate-400"
+                    className="group relative mb-1 flex w-full items-center justify-between overflow-hidden rounded-lg bg-slate-50/50 px-3 py-2 text-sm text-slate-400"
                   >
-                    <Archive size={14} className="shrink-0 opacity-30" />
+                    <div className="flex min-w-0 flex-1 items-center gap-3">
-                    <span className="truncate pr-8">
+                      <Archive size={14} className="shrink-0 opacity-30" />
-                      {chat.nombre ||
+                      <span className="block min-w-0 flex-1 truncate">
-                        `Archivado ${chat.creado_en.split('T')[0]}`}
+                        {chat.nombre ||
-                    </span>
+                          `Archivado ${chat.creado_en.split('T')[0]}`}
+                      </span>
+                    </div>
                     <button
                       onClick={(e) => unarchiveChat(e, chat.id)}
-                      className="absolute right-2 p-1 opacity-0 group-hover:opacity-100 hover:text-teal-600"
+                      className="ml-2 shrink-0 rounded bg-slate-50/80 p-1 opacity-0 transition-opacity group-hover:opacity-100 hover:text-teal-600"
                     >
                       <RotateCcw size={14} />
                     </button>
@@ -647,42 +689,55 @@ function RouteComponent() {
                 </div>
               ) : (
                 <>
-                  {chatMessages.map((msg: any) => (
+                  {chatMessages.map((msg: any) => {
-                    <div
+                    const isAI = msg.role === 'assistant'
-                      key={msg.id}
+                    const isUser = msg.role === 'user'
-                      className={`flex max-w-[85%] flex-col ${
+                    // IMPORTANTE: Asegúrate de que msg.id contenga la info de procesamiento o pásala en el map
-                        msg.role === 'user'
+                    const isProcessing = msg.isProcessing
-                          ? 'ml-auto items-end'
+
-                          : 'items-start'
+                    return (
-                      }`}
-                    >
                       <div
-                        className={`relative rounded-2xl p-3 text-sm whitespace-pre-wrap shadow-sm transition-all duration-300 ${
+                        key={msg.id}
-                          msg.role === 'user'
+                        className={`flex max-w-[85%] flex-col ${
-                            ? 'rounded-tr-none bg-teal-600 text-white'
+                          isUser ? 'ml-auto items-end' : 'items-start'
-                            : `rounded-tl-none border bg-white text-slate-700 ${
-                                // --- LÓGICA DE REFUSAL ---
-                                msg.isRefusal
-                                  ? 'border-red-200 bg-red-50/50 ring-1 ring-red-100'
-                                  : 'border-slate-100'
-                              }`
                         }`}
                       >
-                        {/* Icono opcional de advertencia si es refusal */}
+                        <div
-                        {msg.isRefusal && (
+                          className={`relative rounded-2xl p-3 text-sm whitespace-pre-wrap shadow-sm transition-all duration-300 ${
-                          <div className="mb-1 flex items-center gap-1 text-[10px] font-bold text-red-500 uppercase">
+                            isUser
-                            <span>Aviso del Asistente</span>
+                              ? 'rounded-tr-none bg-teal-600 text-white'
-                          </div>
+                              : `rounded-tl-none border bg-white text-slate-700 ${
-                        )}
+                                  msg.isRefusal
+                                    ? 'border-red-200 bg-red-50/50 ring-1 ring-red-100'
+                                    : 'border-slate-100'
+                                }`
+                          }`}
+                        >
+                          {/* Aviso de Refusal */}
+                          {msg.isRefusal && (
+                            <div className="mb-1 flex items-center gap-1 text-[10px] font-bold text-red-500 uppercase">
+                              <span>Aviso del Asistente</span>
+                            </div>
+                          )}
 
-                        {msg.content}
+                          {/* CONTENIDO CORRECTO: Usamos msg.content */}
+                          {isAI && isProcessing ? (
+                            <div className="flex items-center gap-2 py-1">
+                              <div className="flex gap-1">
+                                <span className="h-1.5 w-1.5 animate-bounce rounded-full bg-teal-500" />
+                                <span className="h-1.5 w-1.5 animate-bounce rounded-full bg-teal-500 [animation-delay:-0.15s]" />
+                                <span className="h-1.5 w-1.5 animate-bounce rounded-full bg-teal-500 [animation-delay:-0.3s]" />
+                              </div>
+                            </div>
+                          ) : (
+                            msg.content // <--- CAMBIO CLAVE
+                          )}
 
-                        {!msg.isRefusal &&
+                          {/* Recomendaciones */}
-                          msg.suggestions &&
+                          {isAI && msg.suggestions?.length > 0 && (
-                          msg.suggestions.length > 0 && (
                             <div className="mt-4">
                               <ImprovementCard
-                                suggestions={msg.suggestions}
+                                suggestions={msg.suggestions} // Usamos el nombre normalizado en el flatMap
                                 dbMessageId={msg.dbMessageId}
                                 planId={planId}
                                 currentDatos={data?.datos}
@@ -693,32 +748,30 @@ function RouteComponent() {
                               />
                             </div>
                           )}
-                      </div>
-                    </div>
-                  ))}
-
-                  {optimisticMessage && (
-                    <div className="animate-in fade-in slide-in-from-right-2 ml-auto flex max-w-[85%] flex-col items-end">
-                      <div className="rounded-2xl rounded-tr-none bg-teal-600/70 p-3 text-sm whitespace-pre-wrap text-white shadow-sm">
-                        {optimisticMessage}
-                      </div>
-                    </div>
-                  )}
-
-                  {isSending && (
-                    <div className="animate-in fade-in slide-in-from-left-2 flex flex-col items-start duration-300">
-                      <div className="rounded-2xl rounded-tl-none border border-slate-200 bg-white p-4 shadow-sm">
-                        <div className="flex items-center gap-2">
-                          <div className="flex gap-1">
-                            <span className="h-1.5 w-1.5 animate-bounce rounded-full bg-teal-500 [animation-delay:-0.3s]" />
-                            <span className="h-1.5 w-1.5 animate-bounce rounded-full bg-teal-500 [animation-delay:-0.15s]" />
-                            <span className="h-1.5 w-1.5 animate-bounce rounded-full bg-teal-500" />
-                          </div>
-                          <span className="text-[10px] font-medium tracking-tight text-slate-400 uppercase">
-                            Esperando respuesta...
-                          </span>
                         </div>
                       </div>
+                    )
+                  })}
+
+                  {(isSending || isSyncing) && (
+                    <div className="animate-in fade-in slide-in-from-bottom-2 flex gap-4">
+                      <Avatar className="h-9 w-9 shrink-0 border bg-teal-600 text-white shadow-sm">
+                        <AvatarFallback>
+                          <Sparkles size={16} className="animate-pulse" />
+                        </AvatarFallback>
+                      </Avatar>
+                      <div className="flex flex-col items-start gap-2">
+                        <div className="rounded-2xl rounded-tl-none border border-slate-200 bg-white p-4 shadow-sm">
+                          <div className="flex gap-1">
+                            <span className="h-1.5 w-1.5 animate-bounce rounded-full bg-slate-400 [animation-delay:-0.3s]"></span>
+                            <span className="h-1.5 w-1.5 animate-bounce rounded-full bg-slate-400 [animation-delay:-0.15s]"></span>
+                            <span className="h-1.5 w-1.5 animate-bounce rounded-full bg-slate-400"></span>
+                          </div>
+                        </div>
+                        <span className="text-[10px] font-medium text-slate-400 italic">
+                          La IA está analizando tu solicitud...
+                        </span>
+                      </div>
                     </div>
                   )}
                 </>

src/types/supabase.ts

@@ -263,35 +263,35 @@ export type Database = {
         Row: {
           actualizado_en: string
           asignatura_id: string
-          biblioteca_item_id: string | null
           cita: string
           creado_en: string
           creado_por: string | null
           id: string
+          referencia_biblioteca: string | null
+          referencia_en_linea: string | null
           tipo: Database['public']['Enums']['tipo_bibliografia']
-          tipo_fuente: Database['public']['Enums']['tipo_fuente_bibliografia']
         }
         Insert: {
           actualizado_en?: string
           asignatura_id: string
-          biblioteca_item_id?: string | null
           cita: string
           creado_en?: string
           creado_por?: string | null
           id?: string
+          referencia_biblioteca?: string | null
+          referencia_en_linea?: string | null
           tipo: Database['public']['Enums']['tipo_bibliografia']
-          tipo_fuente?: Database['public']['Enums']['tipo_fuente_bibliografia']
         }
         Update: {
           actualizado_en?: string
           asignatura_id?: string
-          biblioteca_item_id?: string | null
           cita?: string
           creado_en?: string
           creado_por?: string | null
           id?: string
+          referencia_biblioteca?: string | null
+          referencia_en_linea?: string | null
           tipo?: Database['public']['Enums']['tipo_bibliografia']
-          tipo_fuente?: Database['public']['Enums']['tipo_fuente_bibliografia']
         }
         Relationships: [
           {
@@ -474,6 +474,7 @@ export type Database = {
           estado: Database['public']['Enums']['estado_conversacion']
           id: string
           intento_archivado: number
+          nombre: string | null
           openai_conversation_id: string
         }
         Insert: {
@@ -486,6 +487,7 @@ export type Database = {
           estado?: Database['public']['Enums']['estado_conversacion']
           id?: string
           intento_archivado?: number
+          nombre?: string | null
           openai_conversation_id: string
         }
         Update: {
@@ -498,6 +500,7 @@ export type Database = {
           estado?: Database['public']['Enums']['estado_conversacion']
           id?: string
           intento_archivado?: number
+          nombre?: string | null
           openai_conversation_id?: string
         }
         Relationships: [