Skip to content

Commit 2f5bae0

Browse files
[UPD] Copilot instructions
1 parent 341a5c5 commit 2f5bae0

File tree

1 file changed

+6
-19
lines changed

1 file changed

+6
-19
lines changed

.github/copilot-instructions.md

Lines changed: 6 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -38,8 +38,6 @@
3838

3939
* Confirmar que todos los archivos usados (vistas, seguridad, datos, reportes, wizards) estén referenciados en el manifest.
4040
* Verificar dependencias declaradas: que no falten módulos requeridos ni se declaren innecesarios.
41-
* **Regla de versión (obligatoria):**
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).
4341
* Solo hacerlo una vez por revisión, aunque haya múltiples archivos afectados.
4442

4543
---
@@ -61,7 +59,6 @@
6159

6260
* Verificar los archivos `ir.model.access.csv` para nuevos modelos: deben tener permisos mínimos necesarios.
6361
* No proponer abrir acceso global sin justificación.
64-
* Si se agregan nuevos modelos o campos de control de acceso, **recordar el bump de versión** (ver sección de manifest).
6562
* Si se cambian `record rules`, revisar especialmente combinaciones multi-compañía y multi-website.
6663

6764
### Seguridad y rendimiento del ORM
@@ -86,7 +83,7 @@
8683

8784
## Cambios estructurales y scripts de migración – **cuestiones generales**
8885

89-
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**.
86+
Cuando el diff sugiera **cambios de estructura de datos**, **siempre evaluar** si corresponde proponer un **script de migración** en `migrations/` (pre/post/end).
9087

9188
### Reglas generales de estructura de `migrations/`
9289

@@ -283,31 +280,21 @@ def migrate(cr, registry):
283280
| ------------------ | -------------------------------------------------------------------------------------------------------- |
284281
| Modelos | Relaciones válidas; constraints; uso adecuado de `@api.depends`; `super()` correcto |
285282
| Vistas XML | Herencias correctas; campos válidos; adaptación a cambios de versión (p.ej. `<list>` vs `<tree>`) |
286-
| Manifest | **Bump de versión obligatorio** si hay cambios estructurales en modelos/vistas/records .xml; archivos referenciados |
287283
| Seguridad | Accesos mínimos necesarios; reglas revisadas |
288284
| Migraciones | **Si hay cambios estructurales, sugerir script en `migrations/` (pre/post/end)** y describir qué hace |
289285
| Rendimiento / ORM | Evitar loops costosos; no SQL innecesario; aprovechar las optimizaciones del ORM de la versión |
290286
| Ortografía & typos | Errores evidentes corregibles sin modificar idioma ni estilo |
291287

292288
---
293289

294-
## Heurística práctica para el bump de versión (general)
295-
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)
297-
**Y** `__manifest__.py` no cambia `version`**Sugerir bump**.
298-
* **SI** hay scripts `migrations/pre_*.py` o `migrations/post_*.py` nuevos → **Sugerir al menos minor bump**.
299-
* **SI** hay cambios que rompen compatibilidad (renombres, cambios de tipo con impacto, limpieza masiva de datos) → **Sugerir minor/major** según impacto.
300-
301-
---
302-
303290
## Estilo del feedback (general)
304291

305292
* Ser breve, claro y útil. Ejemplos:
306293

307294
* “El campo `partner_id` no se encuentra referenciado en la vista.”
308295
* “Este método redefine `write()` sin usar `super()`.”
309296
* “Tip: hay un error ortográfico en el nombre del parámetro.”
310-
* **Bump + migración:** “Se renombra `old_ref``new_ref`: falta **bump de versión** y **pre-script** en `migrations/` para copiar valores antes del upgrade; añadir **post-script** para recompute del stored.”
297+
* **Migración:** “Se renombra `old_ref``new_ref`: falta **pre-script** en `migrations/` para copiar valores antes del upgrade; añadir **post-script** para recompute del stored.”
311298

312299
* Evitar explicaciones largas o reescrituras completas salvo que el cambio sea claro y necesario.
313300
* Priorizar comentarios en forma de **lista corta de puntos** (3–7 ítems) y frases breves en lugar de bloques de texto extensos.
@@ -316,10 +303,10 @@ def migrate(cr, registry):
316303

317304
## Resumen operativo para Copilot
318305

319-
1. **Detecta cambios estructurales en modelos, vistas o records .xml → exige bump de `version` en `__manifest__.py` si no está incrementada.**
320-
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.
321-
3. Distingue entre:
306+
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.
307+
2. Distingue entre:
322308

323309
* **cuestiones generales** (válidas para cualquier versión),
324310
* y **matices específicos de Odoo 18** (por ejemplo, uso de `<list>`, passkeys, tours y comportamiento del framework).
325-
4. Mantén el feedback **concreto, breve y accionable**.
311+
312+
3. Mantén el feedback **concreto, breve y accionable**.

0 commit comments

Comments
 (0)