Skip to content

Commit 5a87371

Browse files
[UPD] Copilot instructions
1 parent cd2a37f commit 5a87371

1 file changed

Lines changed: 6 additions & 18 deletions

File tree

.github/copilot-instructions.md

Lines changed: 6 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,6 @@
6262
* Confirmar que todos los archivos usados (vistas, seguridad, datos, reportes, wizards) estén referenciados en el manifest.
6363
* Verificar dependencias declaradas: que no falten módulos requeridos ni se declaren innecesarios.
6464
* **Regla de versión (obligatoria):**
65-
Solo sugerir bump de versión si el `__manifest__.py` no incrementa `version` y se modificó la estructura de un modelo, una vista, o algún record .xml (ej. cambios en definición de campos, vistas XML, datos XML, seguridad).
6665
* Solo hacerlo una vez por revisión, aunque haya múltiples archivos afectados.
6766

6867
---
@@ -79,7 +78,6 @@
7978

8079
* Verificar los archivos `ir.model.access.csv` para nuevos modelos: deben tener permisos mínimos necesarios.
8180
* No proponer abrir acceso global sin justificación.
82-
* Si se agregan nuevos modelos o campos de control de acceso, **recordar el bump de versión** (ver sección de manifest).
8381
* En Odoo 19, poner atención especial a cambios de seguridad ligados a:
8482

8583
* integraciones de IA,
@@ -109,7 +107,7 @@
109107

110108
## Cambios estructurales y scripts de migración – **cuestiones generales**
111109

112-
Cuando el diff sugiera **cambios de estructura de datos**, **siempre evaluar** si corresponde proponer un **script de migración** en `migrations/` (pre/post/end) **y recordar el bump de versión**.
110+
Cuando el diff sugiera **cambios de estructura de datos**, **siempre evaluar** si corresponde proponer un **script de migración** en `migrations/` (pre/post/end).
113111

114112
### Reglas generales de estructura de `migrations/`
115113

@@ -276,23 +274,13 @@ En estos casos **normalmente corresponde** proponer migración (salvo notas en c
276274
| ------------------ | ------------------------------------------------------------------------------------------------------------ |
277275
| Modelos | Relaciones válidas; constraints; uso de `@api.depends`; `super()` correcto |
278276
| Vistas XML | Herencias correctas; campos válidos; adaptación a componentes modernos (IA, secciones, etc.) |
279-
| Manifest | **Bump de versión obligatorio** si hay cambios estructurales en modelos/vistas/records .xml; archivos referenciados |
280277
| Seguridad | Accesos mínimos necesarios; reglas revisadas, en especial para IA/VOIP/WhatsApp |
281278
| Migraciones | **Si hay cambios estructurales (lista actualizada), sugerir scripts en `migrations/` y describir qué hacen** |
282279
| Rendimiento / ORM | Evitar patrones anti-ORM; aprovechar las mejoras del ORM/registro en v19 |
283280
| Ortografía & typos | Errores evidentes corregibles sin cambiar estilo |
284281

285282
---
286283

287-
## Heurística práctica para el bump de versión
288-
289-
* **SI** el diff modifica la estructura de un modelo, una vista, o algún record .xml (ej. cambios en definición de campos, vistas XML, datos XML, seguridad)
290-
**Y** `__manifest__.py` no cambia `version`**Sugerir bump**.
291-
* **SI** hay scripts `migrations/pre_*.py` o `migrations/post_*.py` nuevos → **Sugerir al menos minor bump**.
292-
* **SI** hay cambios que rompen compatibilidad (renombres, tipos, limpieza agresiva de datos) → **Sugerir minor/major** según impacto.
293-
294-
---
295-
296284
## Estilo del feedback
297285

298286
* El feedback debe ser **breve, concreto y accionable**.
@@ -301,16 +289,16 @@ En estos casos **normalmente corresponde** proponer migración (salvo notas en c
301289

302290
* “La constraint nueva podría definirse como `models.Constraint` en lugar de `_sql_constraints`.”
303291
* “La vista de cotización no está aprovechando las secciones configurables introducidas en 19.”
304-
* “Se renombra el campo `state` almacenado; falta bump de versión y script de migración para copiar valores antes del upgrade.”
292+
* “Se renombra el campo `state` almacenado; falta script de migración para copiar valores antes del upgrade.”
305293

306294
---
307295

308296
## Resumen operativo para Copilot
309297

310-
1. **Detecta cambios estructurales en modelos, vistas o records .xml → exige bump de `version` en `__manifest__.py` si no está incrementada.**
311-
2. **Si hay cambio estructural (según la lista actualizada) → propone y describe script(s) de migración en `migrations/` (pre/post/end)**, con enfoque idempotente y en lotes.
312-
3. Distingue entre:
298+
1. **Si hay cambio estructural (según la lista actualizada) → propone y describe script(s) de migración en `migrations/` (pre/post/end)**, con enfoque idempotente y en lotes.
299+
2. Distingue entre:
313300

314301
* **cuestiones generales**,
315302
* y **matices específicos de Odoo 19**, por ej. preferir la nueva API de constraints/índices cuando corresponda.
316-
4. Mantén el feedback **concreto, breve y accionable**.
303+
304+
3. Mantén el feedback **concreto, breve y accionable**.

0 commit comments

Comments
 (0)