egutierrez
a3f75d61ec
Reorganizacion de dev/issues en subcarpetas (completed/, cpp/, gamedev/, kanban/, trading/, imagegen/, matrix/) y cambios acumulados en cmd/fn/pyrunner, .claude/commands y settings. Trabajo de otro LLM/sesion, commiteado a peticion del usuario para desbloquear el working tree. Excluido logs/ardour_mcp_server.log (ruido).
49 lines
1.5 KiB
Markdown
49 lines
1.5 KiB
Markdown
---
|
|
id: "0125"
|
|
title: "deploy_server: anadir flag --db a cmdServe"
|
|
status: pendiente
|
|
type: bugfix
|
|
domain:
|
|
- deploy
|
|
scope: app
|
|
priority: media
|
|
depends: []
|
|
blocks: []
|
|
related:
|
|
- "0121a"
|
|
created: 2026-05-19
|
|
updated: 2026-05-19
|
|
tags: [deploy_server, cli, idempotency, ausente-ready]
|
|
---
|
|
|
|
# 0125 — deploy_server `--db` flag
|
|
|
|
Origen: detectado lateral por `fn-recopilador design-e2e apps/deploy_server` en 0121a.
|
|
|
|
## Problema
|
|
|
|
`cmdServe` en `apps/deploy_server/server.go` solo expone `--port`. No hay forma de pasar BD efimera para tests/e2e — el server siempre abre `operations.db` del cwd. Esto rompe idempotencia de `smoke` checks (mezcla datos test con prod).
|
|
|
|
## Decision
|
|
|
|
Anadir flag `--db PATH` a `cmdServe`. Default = `operations.db` del cwd (compatibilidad). Cuando se pasa, el server abre la BD apuntada.
|
|
|
|
## Tareas
|
|
|
|
1. Editar `apps/deploy_server/server.go` — parse `--db` en `cmdServe`.
|
|
2. Pasar el path a la apertura de SQLite.
|
|
3. Verificar que migraciones aplican igual sobre BD efimera.
|
|
4. Actualizar propuesta 0121a `deploy_server.yaml` removiendo gotcha del check `smoke`.
|
|
|
|
## Acceptance
|
|
|
|
- [ ] `./deploy_server serve --port 9190 --db /tmp/x.db` abre BD en `/tmp/x.db` y aplica migraciones idempotente.
|
|
- [ ] Sin `--db`, comportamiento actual preservado.
|
|
- [ ] `smoke` check del 0121a pasa con `/tmp/deploy_server_e2e.db`.
|
|
|
|
## DoD
|
|
|
|
- **Donde**: terminal (`./deploy_server serve --help`).
|
|
- **Latencia**: cambio no afecta latencia runtime.
|
|
- **Onboarding**: "Para e2e/tests de deploy_server pasa `--db /tmp/...`."
|