Skip to content

fix(QA-018): prohibir múltiples wires al mismo puerto de entrada#58

Closed
anlaco wants to merge 1 commit into
mainfrom
fix/qa-018-024-029
Closed

fix(QA-018): prohibir múltiples wires al mismo puerto de entrada#58
anlaco wants to merge 1 commit into
mainfrom
fix/qa-018-024-029

Conversation

@anlaco
Copy link
Copy Markdown
Owner

@anlaco anlaco commented Apr 7, 2026

Resumen

  • wire-exists-to-port? en model.red: comprueba si ya existe un wire al mismo nodo+puerto destino
  • add-wire en model.red: wrapper que aplica la guarda antes de hacer append (reemplaza append wires make-wire en todo canvas.red)
  • canvas.red: 5 puntos de creación de wire migrados a add-wire
  • 5 tests de regresión QA-018

QA-024 y QA-029

Verificadas: ya estaban corregidas en el código (no requieren cambios):

  • fp-default-label tiene fallback true → "Numeric" (línea 82)
  • save-panel-to-diagram usa item/value (línea 1049)

Test plan

  • 455/455 tests pasan
  • Intentar conectar 2 wires al mismo puerto de entrada → el segundo se rechaza silenciosamente

Closes QA-018

🤖 Generated with Claude Code

- model.red: wire-exists-to-port? verifica si ya existe wire al puerto destino
- model.red: add-wire wraps append+guard — reemplaza append directo en canvas
- canvas.red: 5 puntos de add-wire usan add-wire en lugar de append directo
- tests/test-model.red: 5 tests de regresión QA-018

QA-024 y QA-029 ya estaban corregidos en el código (verificado).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@anlaco
Copy link
Copy Markdown
Owner Author

anlaco commented Apr 8, 2026

Supersedido por PR #60 (refactor/fase4-estructural) que incluye este trabajo y más.

@anlaco anlaco closed this Apr 8, 2026
@anlaco anlaco deleted the fix/qa-018-024-029 branch April 9, 2026 21:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant