Skip to content

Commit 301f8c0

Browse files
[UPD] Copilot instructions
1 parent af77ef8 commit 301f8c0

File tree

1 file changed

+6
-13
lines changed

1 file changed

+6
-13
lines changed

.github/copilot-instructions.md

Lines changed: 6 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@
1717
* Si ya existe un docstring, puede sugerirse un estilo básico acorde a PEP8, pero **no será un error** si faltan `return`, tipos o parámetros documentados.
1818
5. No proponer cambios puramente estéticos (espacios, comillas simples vs dobles, orden de imports, etc.).
1919
6. Mantener el feedback **muy conciso** en los PRs: priorizar pocos puntos claros, evitar párrafos largos y no repetir el contexto que ya está explicado en la descripción del PR.
20+
7. Sobre traducciones: usar `_()` o `self.env._()` es indistinto; solo marcar si hay mensajes de error o textos no traducidos que deban serlo.
2021

2122
---
2223

@@ -38,13 +39,7 @@
3839
* Confirmar que todos los archivos usados (vistas, seguridad, datos, reportes, wizards) estén referenciados en el manifest.
3940
* Verificar dependencias declaradas: que no falten módulos requeridos ni se declaren innecesarios.
4041
* **Regla de versión (obligatoria):**
41-
Siempre que el diff incluya **modificaciones en**:
42-
43-
* definición de campos o modelos (`models/*.py`, `wizards/*.py`),
44-
* vistas o datos XML (`views/*.xml`, `data/*.xml`, `report/*.xml`, `wizards/*.xml`),
45-
* seguridad (`security/*.csv`, `security/*.xml`),
46-
47-
**y el `__manifest__.py` no incrementa `version`, sugerir el bump de versión** (por ejemplo, `1.0.0 → 1.0.1`).
42+
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).
4843
* Solo hacerlo una vez por revisión, aunque haya múltiples archivos afectados.
4944

5045
---
@@ -288,7 +283,7 @@ def migrate(cr, registry):
288283
| ------------------ | -------------------------------------------------------------------------------------------------------- |
289284
| Modelos | Relaciones válidas; constraints; uso adecuado de `@api.depends`; `super()` correcto |
290285
| Vistas XML | Herencias correctas; campos válidos; adaptación a cambios de versión (p.ej. `<list>` vs `<tree>`) |
291-
| Manifest | **Bump de versión obligatorio** si hay cambios en modelos/vistas/seguridad/datos; archivos referenciados |
286+
| Manifest | **Bump de versión obligatorio** si hay cambios estructurales en modelos/vistas/records .xml; archivos referenciados |
292287
| Seguridad | Accesos mínimos necesarios; reglas revisadas |
293288
| Migraciones | **Si hay cambios estructurales, sugerir script en `migrations/` (pre/post/end)** y describir qué hace |
294289
| Rendimiento / ORM | Evitar loops costosos; no SQL innecesario; aprovechar las optimizaciones del ORM de la versión |
@@ -298,7 +293,7 @@ def migrate(cr, registry):
298293

299294
## Heurística práctica para el bump de versión (general)
300295

301-
* **SI** el diff toca cualquiera de: `models/`, `views/`, `data/`, `report/`, `security/`, `wizards/`
296+
* **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)
302297
**Y** `__manifest__.py` no cambia `version`**Sugerir bump**.
303298
* **SI** hay scripts `migrations/pre_*.py` o `migrations/post_*.py` nuevos → **Sugerir al menos minor bump**.
304299
* **SI** hay cambios que rompen compatibilidad (renombres, cambios de tipo con impacto, limpieza masiva de datos) → **Sugerir minor/major** según impacto.
@@ -321,12 +316,10 @@ def migrate(cr, registry):
321316

322317
## Resumen operativo para Copilot
323318

324-
1. **Detecta cambios en modelos/vistas/seguridad/datos → exige bump de `version` en `__manifest__.py`.**
319+
1. **Detecta cambios estructurales en modelos, vistas o records .xml → exige bump de `version` en `__manifest__.py` si no está incrementada.**
325320
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.
326321
3. Distingue entre:
327322

328323
* **cuestiones generales** (válidas para cualquier versión),
329324
* y **matices específicos de Odoo 18** (por ejemplo, uso de `<list>`, passkeys, tours y comportamiento del framework).
330-
4. Mantén el feedback **concreto, breve y accionable**.
331-
332-
[^odoo18]: Resumen basado en la documentación oficial de Odoo 18 Release Notes y artículos técnicos que analizan sus mejoras de rendimiento y UX.
325+
4. Mantén el feedback **concreto, breve y accionable**.

0 commit comments

Comments
 (0)