Que se guarden las seriaciones

fix #175
fix #151
fix #180

bun.lock

@@ -4,7 +4,6 @@
     "": {
       "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",
@@ -139,18 +138,6 @@
 
     "@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=="],
@@ -263,8 +250,6 @@
 
     "@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,7 +17,6 @@
     "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,5 +1,3 @@
-import { DragDropProvider } from '@dnd-kit/react'
-import { isSortable, useSortable } from '@dnd-kit/react/sortable'
 import { useParams } from '@tanstack/react-router'
 import {
   Plus,
@@ -13,7 +11,7 @@ import {
 import { useEffect, useRef, useState } from 'react'
 
 import type { ContenidoApi, ContenidoTemaApi } from '@/data/api/subjects.api'
-import type { FocusEvent, KeyboardEvent, ReactNode } from 'react'
+import type { FocusEvent, KeyboardEvent } from 'react'
 
 import {
   AlertDialog,
@@ -52,95 +50,6 @@ 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)
 }
@@ -191,18 +100,20 @@ function mapContenidoItem(value: unknown, index: number): ContenidoApi | null {
   if (Array.isArray(value.temas)) {
     temas = value.temas
       .map(mapTemaValue)
-      .filter((x): x is ContenidoTemaApi => x !== null)
+      .filter((t): t is ContenidoTemaApi => t !== null)
+  } else if (typeof value.temas === 'string' && value.temas.trim()) {
+    temas = value.temas
+      .split(/\r?\n|,/)
+      .map((t) => t.trim())
+      .filter(Boolean)
   }
 
-  return {
+  return { unidad, titulo, temas }
-    ...value,
-    unidad,
-    titulo,
-    temas,
-  }
 }
 
 function mapContenidoTematicoFromDb(value: unknown): Array<ContenidoApi> {
+  if (value == null) return []
+
   if (typeof value === 'string') {
     try {
       return mapContenidoTematicoFromDb(JSON.parse(value))
@@ -281,16 +192,7 @@ 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,
@@ -344,17 +246,10 @@ 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 horasEstimadas = parseHorasEstimadas(temaDraftHoras)
+    const parsedHoras = Number.parseInt(temaDraftHoras, 10)
+    const horasEstimadas = Number.isFinite(parsedHoras) ? parsedHoras : 0
 
     const next = unidades.map((u) => {
       if (u.id !== editingTema.unitId) return u
@@ -408,110 +303,28 @@ export function ContenidoTematico() {
       data ? data.contenido_tematico : undefined,
     )
 
-    // 1. EL ESCUDO: Comparamos si nuestro estado local ya tiene esta info exacta
+    const transformed = contenido.map((u, idx) => ({
-    // (Esto ocurre justo después de arrastrar, ya que actualizamos la UI antes que la BD)
+      id: `u-${u.unidad || idx + 1}`,
-    const currentPayload = JSON.stringify(
-      serializeUnidadesToApi(unidadesRef.current),
-    )
-
-    // Normalizamos la data de la BD para que tenga exactamente la misma forma que el payload
-    const incomingPayload = JSON.stringify(
-      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,
+      nombre: u.titulo || 'Sin título',
       temas: Array.isArray(u.temas)
-          ? u.temas.map((t: any, tidx: number) => {
+        ? u.temas.map((t: any, tidx: number) => ({
-              const dbTemaNombre =
+            id: `t-${u.unidad || idx + 1}-${tidx + 1}`,
-                typeof t === 'string' ? t : t?.nombre || 'Tema'
+            nombre: typeof t === 'string' ? t : t?.nombre || 'Tema',
-
+            horasEstimadas: t?.horasEstimadas || 0,
-              // 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
-      // Expandir la primera unidad solo una vez al llegar a la ruta.
+      return transformed.length > 0 ? new Set([transformed[0].id]) : new Set()
-      // 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])
 
@@ -540,7 +353,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,
   )
 
@@ -551,22 +364,16 @@ export function ContenidoTematico() {
     setExpandedUnits(newExpanded)
   }
 
-  const insertUnidadAt = (insertIndex: number) => {
+  const addUnidad = () => {
-    const newId = createClientId('u')
+    const newNumero = unidades.length + 1
+    const newId = `u-${newNumero}`
     const newUnidad: UnidadTematica = {
       id: newId,
       nombre: 'Nueva Unidad',
-      numero: 0,
+      numero: newNumero,
       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)
@@ -575,40 +382,10 @@ 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 ---
@@ -674,25 +451,15 @@ export function ContenidoTematico() {
         </div>
       </div>
 
-      <DragDropProvider onDragEnd={handleReorderEnd}>
       <div className="space-y-4">
-          {unidades.map((unidad, index) => (
+        {unidades.map((unidad) => (
-            <SortableUnidad
+          <div
             key={unidad.id}
-              id={unidad.id}
+            ref={(el) => {
-              index={index}
-              registerContainer={(el) => {
               if (el) unitContainerRefs.current.set(unidad.id, el)
               else unitContainerRefs.current.delete(unidad.id)
             }}
           >
-              {({ handleRef }) => (
-                <>
-                  <InsertUnidadOverlay
-                    position="bottom"
-                    onInsert={() => insertUnidadAt(index + 1)}
-                  />
-
             <Card className="overflow-hidden border-slate-200 shadow-sm">
               <Collapsible
                 open={expandedUnits.has(unidad.id)}
@@ -700,19 +467,9 @@ export function ContenidoTematico() {
               >
                 <CardHeader className="border-b border-slate-100 bg-slate-50/50 py-3">
                   <div className="flex items-center gap-3">
-                          <span
+                    <GripVertical className="h-4 w-4 cursor-grab text-slate-300" />
-                            ref={handleRef as any}
-                            className="inline-flex cursor-grab touch-none items-center text-slate-300"
-                            aria-label="Reordenar unidad"
-                          >
-                            <GripVertical className="h-4 w-4" />
-                          </span>
                     <CollapsibleTrigger asChild>
-                            <Button
+                      <Button variant="ghost" size="sm" className="h-auto p-0">
-                              variant="ghost"
-                              size="sm"
-                              className="h-auto cursor-pointer p-0"
-                            >
                         {expandedUnits.has(unidad.id) ? (
                           <ChevronDown className="h-4 w-4" />
                         ) : (
@@ -728,9 +485,7 @@ export function ContenidoTematico() {
                       <Input
                         ref={unitTitleInputRef}
                         value={unitDraftNombre}
-                              onChange={(e) =>
+                        onChange={(e) => setUnitDraftNombre(e.target.value)}
-                                setUnitDraftNombre(e.target.value)
-                              }
                         onBlur={() => {
                           if (cancelNextBlurRef.current) {
                             cancelNextBlurRef.current = false
@@ -763,7 +518,7 @@ export function ContenidoTematico() {
                     )}
 
                     <div className="ml-auto flex items-center gap-3">
-                            <span className="flex cursor-default items-center gap-1 text-xs font-medium text-slate-400">
+                      <span className="flex items-center gap-1 text-xs font-medium text-slate-400">
                         <Clock className="h-3 w-3" />{' '}
                         {unidad.temas.reduce(
                           (sum, t) => sum + (t.horasEstimadas || 0),
@@ -774,12 +529,9 @@ export function ContenidoTematico() {
                       <Button
                         variant="ghost"
                         size="icon"
-                              className="h-8 w-8 cursor-pointer text-slate-400 hover:text-red-500"
+                        className="h-8 w-8 text-slate-400 hover:text-red-500"
                         onClick={() =>
-                                setDeleteDialog({
+                          setDeleteDialog({ type: 'unidad', id: unidad.id })
-                                  type: 'unidad',
-                                  id: unidad.id,
-                                })
                         }
                       >
                         <Trash2 className="h-4 w-4" />
@@ -802,14 +554,10 @@ export function ContenidoTematico() {
                           }
                           draftNombre={temaDraftNombre}
                           draftHoras={temaDraftHoras}
-                                onBeginEdit={() =>
+                          onBeginEdit={() => beginEditTema(unidad.id, tema.id)}
-                                  beginEditTema(unidad.id, tema.id)
-                                }
                           onDraftNombreChange={setTemaDraftNombre}
                           onDraftHorasChange={setTemaDraftHoras}
-                                onEditorBlurCapture={
+                          onEditorBlurCapture={handleTemaEditorBlurCapture}
-                                  handleTemaEditorBlurCapture
-                                }
                           onEditorKeyDownCapture={
                             handleTemaEditorKeyDownCapture
                           }
@@ -828,7 +576,7 @@ export function ContenidoTematico() {
                       <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"
+                        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
@@ -838,12 +586,23 @@ export function ContenidoTematico() {
                 </CollapsibleContent>
               </Collapsible>
             </Card>
-                </>
+          </div>
-              )}
-            </SortableUnidad>
         ))}
       </div>
-      </DragDropProvider>
+
+      <div className="flex justify-center pt-2">
+        <Button
+          variant="outline"
+          className="gap-2"
+          onClick={(e) => {
+            // Evita que Enter vuelva a disparar el click sobre el botón.
+            e.currentTarget.blur()
+            addUnidad()
+          }}
+        >
+          <Plus className="h-4 w-4" /> Nueva unidad
+        </Button>
+      </div>
 
       <DeleteConfirmDialog
         dialog={deleteDialog}
@@ -908,9 +667,6 @@ 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"
           />
@@ -919,7 +675,7 @@ function TemaRow({
         <>
           <button
             type="button"
-            className="flex flex-1 cursor-pointer items-center gap-3 text-left"
+            className="flex flex-1 items-center gap-3 text-left"
             onClick={(e) => {
               e.stopPropagation()
               onBeginEdit()
@@ -934,7 +690,7 @@ function TemaRow({
             <Button
               variant="ghost"
               size="icon"
-              className="h-7 w-7 cursor-pointer text-slate-400 hover:text-blue-600"
+              className="h-7 w-7 text-slate-400 hover:text-blue-600"
               onClick={(e) => {
                 e.stopPropagation()
                 onBeginEdit()
@@ -945,7 +701,7 @@ function TemaRow({
             <Button
               variant="ghost"
               size="icon"
-              className="h-7 w-7 cursor-pointer text-slate-400 hover:text-red-500"
+              className="h-7 w-7 text-slate-400 hover:text-red-500"
               onClick={(e) => {
                 e.stopPropagation()
                 onDelete()

src/data/api/plans.api.ts

@@ -207,7 +207,7 @@ export async function plan_asignaturas_list(
   const { data, error } = await supabase
     .from('asignaturas')
     .select(
-      'id,plan_estudio_id,horas_academicas,horas_independientes,estructura_id,codigo,nombre,tipo,creditos,numero_ciclo,linea_plan_id,orden_celda,estado,datos,contenido_tematico,asignatura_hash,tipo_origen,meta_origen,creado_por,actualizado_por,creado_en,actualizado_en',
+      'id,plan_estudio_id,horas_academicas,horas_independientes,estructura_id,codigo,nombre,tipo,creditos,numero_ciclo,linea_plan_id,orden_celda,estado,datos,contenido_tematico,asignatura_hash,tipo_origen,meta_origen,creado_por,actualizado_por,creado_en,actualizado_en,prerrequisito_asignatura_id',
     )
     .eq('plan_estudio_id', planId)
     .order('numero_ciclo', { ascending: true, nullsFirst: false })

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

@@ -76,7 +76,7 @@ const mapAsignaturasToAsignaturas = (
       // Mapeo directo de los nuevos campos de la API
       hd: asig.horas_academicas ?? 0,
       hi: asig.horas_independientes ?? 0,
-      prerrequisitos: [],
+      prerrequisito_asignatura_id: asig.prerrequisito_asignatura_id ?? null,
     }
   })
 }
@@ -336,6 +336,7 @@ function MapaCurricularPage() {
       horas_independientes?: TablesUpdate<'asignaturas'>['horas_independientes']
       numero_ciclo?: TablesUpdate<'asignaturas'>['numero_ciclo']
       linea_plan_id?: TablesUpdate<'asignaturas'>['linea_plan_id']
+      prerrequisito_asignatura_id?: string | null
     }
     const patch: Partial<AsignaturaPatch> = {
       nombre: editingData.nombre,
@@ -345,6 +346,7 @@ function MapaCurricularPage() {
       horas_independientes: editingData.hi,
       numero_ciclo: editingData.ciclo,
       linea_plan_id: editingData.lineaCurricularId,
+      prerrequisito_asignatura_id: editingData.prerrequisito_asignatura_id,
       tipo: editingData.tipo.toUpperCase() as TipoAsignatura, // Asegurar que coincida con el ENUM (OBLIGATORIA/OPTATIVA)
     }
 
@@ -490,7 +492,7 @@ function MapaCurricularPage() {
     e: React.FocusEvent<HTMLSpanElement>,
     id: string,
   ) => {
-    const nuevoNombre = e.currentTarget.textContent?.trim() || ''
+    const nuevoNombre = e.currentTarget.textContent.trim() || ''
 
     // Buscamos la línea original para comparar
     const lineaOriginal = lineas.find((l) => l.id === id)
@@ -935,65 +937,55 @@ function MapaCurricularPage() {
               {/* Fila 4: Seriación (Prerrequisitos) */}
               <div className="space-y-2">
                 <label className="text-xs font-bold text-slate-500 uppercase">
-                  Seriación (Prerrequisitos)
+                  Seriación (Prerrequisito)
                 </label>
                 <Select
-                  value={seriacionValue}
+                  // Cambiamos a manejo de valor único basado en el ID de la columna
+                  value={editingData.prerrequisito_asignatura_id || undefined}
                   onValueChange={(val) => {
-                    if (val === 'none') {
+                    console.log(editingData)
-                      setSeriacionValue('')
+
-                      return
-                    }
-                    if (!editingData.prerrequisitos.includes(val)) {
                     setEditingData({
                       ...editingData,
-                        prerrequisitos: [...editingData.prerrequisitos, val],
+                      prerrequisito_asignatura_id: val === 'none' ? null : val,
                     })
-                    }
-                    setSeriacionValue('')
                   }}
                 >
-                  <SelectTrigger>
+                  <SelectTrigger className="w-full bg-white">
                     <SelectValue placeholder="Seleccionar asignatura..." />
                   </SelectTrigger>
                   <SelectContent>
                     <SelectItem value="none">-- Sin Seriación --</SelectItem>
 
                     {asignaturas
-                      .filter((m) => m.id !== editingData.id)
+                      .filter((asig) => {
-                      .map((m) => (
+                        // 1. No es la misma materia
-                        <SelectItem key={m.id} value={m.id}>
+                        const noEsMisma = asig.id !== editingData.id
-                          {m.nombre} ({m.clave})
+                        // 2. El ciclo debe ser estrictamente MENOR
+                        const esCicloMenor =
+                          asig.ciclo !== null &&
+                          editingData.ciclo !== null &&
+                          asig.ciclo < editingData.ciclo
+
+                        return noEsMisma && esCicloMenor
+                      })
+                      .sort(
+                        (a, b) =>
+                          (a.ciclo || 0) - (b.ciclo || 0) ||
+                          a.nombre.localeCompare(b.nombre),
+                      )
+                      .map((asig) => (
+                        <SelectItem key={asig.id} value={asig.id}>
+                          <span className="font-bold text-teal-600">
+                            [C{asig.ciclo}]
+                          </span>{' '}
+                          {asig.nombre}
                         </SelectItem>
                       ))}
                   </SelectContent>
                 </Select>
 
-                {/* Visualización de los prerrequisitos seleccionados */}
+                {/* Visualización del Prerrequisito con el Nombre */}
-                <div className="mt-2 flex flex-wrap gap-2">
-                  {editingData.prerrequisitos.map((pre) => (
-                    <Badge
-                      key={pre}
-                      variant="secondary"
-                      className="bg-slate-100 text-slate-600"
-                    >
-                      {pre}
-                      <button
-                        className="ml-1 hover:text-red-500"
-                        onClick={() => {
-                          setEditingData({
-                            ...editingData,
-                            prerrequisitos: editingData.prerrequisitos.filter(
-                              (p) => p !== pre,
-                            ),
-                          })
-                        }}
-                      >
-                        ×
-                      </button>
-                    </Badge>
-                  ))}
-                </div>
               </div>
 
               {/* Fila 5: Tipo */}

src/types/plan.ts

@@ -50,7 +50,7 @@ export interface Asignatura {
   orden?: number
   hd: number // <--- Añadir
   hi: number // <--- Añadir
-  prerrequisitos: Array<string>
+  prerrequisito_asignatura_id: string | null
 }
 
 export interface Plan {

src/types/supabase.ts

@@ -156,6 +156,7 @@ export type Database = {
           plan_estudio_id: string
           tipo: Database['public']['Enums']['tipo_asignatura']
           tipo_origen: Database['public']['Enums']['tipo_origen'] | null
+          prerrequisito_asignatura_id?: string
         }
         Insert: {
           actualizado_en?: string