parche-cli 0.6.9 → 0.7.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/CHANGELOG.md +16 -0
- package/README.md +7 -3
- package/data/known-sdks.json +1 -0
- package/package.json +1 -1
- package/packs/arca.wsfe.condicion-iva-receptor.json +9 -7
- package/src/ast.js +3 -2
- package/src/codemods/arca-rules.js +20 -1
- package/src/codemods/arca-wsfe-condicion-iva-php.js +27 -3
- package/src/codemods/arca-wsfe-condicion-iva.js +58 -8
- package/src/context.js +12 -2
- package/src/report.js +8 -2
package/CHANGELOG.md
CHANGED
|
@@ -2,6 +2,22 @@
|
|
|
2
2
|
|
|
3
3
|
Los cambios de cada versión de `parche-cli` y de la Action `Pepo08/parche@v0`. En npm están la 0.1.1, la 0.5.1 y de la 0.6.0 en adelante. La 0.1.0 y de la 0.2.0 a la 0.5.0 no se publicaron en npm; lo que traían está incluido en la 0.5.1. La fecha de cada versión es la del cambio; la de publicación figura en npm y en los [Releases](https://github.com/Pepo08/parche/releases).
|
|
4
4
|
|
|
5
|
+
## 0.7.0 — 2026-10-24
|
|
6
|
+
|
|
7
|
+
ARCA: Parche encuentra a los repos que creen estar migrados y no lo están, reconoce `@arcasdk/core` (la segunda librería de ARCA más descargada en npm) y avisa de la librería vieja en más casos. Salió de los pendientes del escaneo de 174 repos públicos (frente B, puntos 12 a 14).
|
|
8
|
+
|
|
9
|
+
### Nuevo
|
|
10
|
+
- **Clave casi bien escrita:** si el detalle del comprobante lleva `CondicionIVAReceptor` (sin `Id`), `CondicionIvaReceptorId` o `condicionIVAReceptorId`, Parche la renombra a `CondicionIVAReceptorId` sin tocar el valor, en JS y en PHP. Ese elemento no existe en el WSDL de WSFEv1: el dato nunca llegaba y desde el 1/12/2026 ARCA rechaza el CAE. Pasaba en un repo real con integración SOAP propia, en cuatro archivos. El PR sale borrador, para revisar que el valor sea un código válido. Solo se toca dentro de objetos o arrays con al menos dos claves de WSFEv1 (`DocTipo`, `CbteDesde`, `ImpTotal`…), nunca en tests ni en modelos propios con un nombre parecido. Si el comprobante tiene las dos claves, no edita y pide revisar.
|
|
11
|
+
- **`@arcasdk/core`:** `arca.electronicBillingService.createVoucher(...)` y `createNextVoucher(...)` reciben los mismos arreglos que `afip.js` (consumidor final, DNI con B o C, PR parcial). Su comprobante usa las mismas claves de WSFEv1 (tipos de la 2.0.2, leídos en el paquete de npm). Probado sobre cinco repos públicos que la usan.
|
|
12
|
+
- **Guía "Cómo agregar `CondicionIVAReceptorId` antes del 1/12/2026"** (`docs/guia-condicion-iva-receptor.md`): qué cambia, el campo en afip.js, afip.php, @arcasdk/core y SOAP propio, los códigos por clase de comprobante, los tres errores que aparecieron en repos reales y cómo probarlo en homologación. Para quien prefiere hacerlo a mano.
|
|
13
|
+
|
|
14
|
+
### Cambios
|
|
15
|
+
- **`ts-afip-ws`:** se avisa como librería vieja. Es un fork de afip.js 0.x y su última versión (1.3.2, de 2023) trae `wsfe.wsdl` y `wsfe-production.wsdl` sin el campo. El aviso dice qué hacer: pasar a `@afipsdk/afip.js` 1.x o actualizar el WSDL que trae la librería (lo mismo para `eesnaola/afip.php`, que no tiene una 1.x).
|
|
16
|
+
- **El aviso de librería vieja también sale cuando Parche no puede leer los datos del comprobante** (por ejemplo, si llegan de otra función). Antes salía solo en los arreglos automáticos y en lo que pedía un dato con el objeto a la vista.
|
|
17
|
+
- **Cuerpo del issue:** el pie dice "¿Te sirvió este aviso?" en vez de "¿Este PR te sirvió?".
|
|
18
|
+
- **Checklist:** "confirmar que cada cliente con CUIT tiene su condición" sale solo si queda algo pendiente. En un PR que arregla todo, como uno de puros consumidores finales, no aplicaba (nuevo campo de pack: `review_checklist_if_pending_es`).
|
|
19
|
+
- **Pack `arca.wsfe.condicion-iva-receptor` 2026.10.24:** fuentes releídas el 24/10/2026 (sin prórroga del 1/12); la explicación ya no cita una versión del manual de WSFEv1, que la página de ARCA rotula distinto; suma `facturas` (LaPyme) y las llamadas de `@arcasdk/core` a la detección.
|
|
20
|
+
|
|
5
21
|
## 0.6.9 — 2026-10-23
|
|
6
22
|
|
|
7
23
|
La Action muestra lo que no abre un PR, y la plantilla del workflow anda en repos PHP. Salió de probar cada comando sobre los ejemplos antes de mandar los primeros mensajes de venta.
|
package/README.md
CHANGED
|
@@ -130,7 +130,7 @@ Cómo contribuir, las reglas para el código y cómo se saca una versión: [`CON
|
|
|
130
130
|
|
|
131
131
|
| Pack | Proveedor | Tipo | Fecha límite | Arreglo |
|
|
132
132
|
|---|---|---|---|---|
|
|
133
|
-
| `arca.wsfe.condicion-iva-receptor` | ARCA | rompe | 2026-12-01 | sí, en JS (`afip.js`) y PHP (`afipsdk/afip.php` y forks): consumidor final y DNI B/C; con `DocTipo` en una variable, consumidor final al ejecutarse (PR parcial) |
|
|
133
|
+
| `arca.wsfe.condicion-iva-receptor` | ARCA | rompe | 2026-12-01 | sí, en JS (`afip.js` y `@arcasdk/core`) y PHP (`afipsdk/afip.php` y forks): consumidor final y DNI B/C; con `DocTipo` en una variable, consumidor final al ejecutarse (PR parcial) |
|
|
134
134
|
| `mercadopago.payments-to-orders` | Mercado Pago | migración | — | sí (creación de pagos en SDK Node v2) |
|
|
135
135
|
| `ai.model-retirements` | Anthropic, OpenAI | retiro | varias | solo detección (también modelos fine-tuned de OpenAI, `ft:…`). De OpenAI, los modelos que su página de deprecaciones apaga desde 2026: texto, código, imagen, video, voz, transcripción y realtime; y los productos que cierran: Assistants API, prompts reutilizables (`pmpt_…`, `v1/prompts`), la plataforma de Evals, ChatKit con un workflow de Agent Builder y la Videos API (cerrada el 24/9/2026). De Anthropic, los modelos retirados o con fecha, también por su alias (`claude-sonnet-4-5`, que apunta a `claude-sonnet-4-5-20250929` y se retira el 30/11/2026), `temperature`, `top_p` y `top_k` con un valor que no es el de por defecto en Claude 4.7 y posteriores, que dan error 400 (SDK de JS/TS y de Python, y HTTP directo), y los mismos tres parámetros con cualquier modelo en la API version `2026-10-21`, que usan el SDK de TypeScript 1.0.0 y el de Python 2.0.0 (del 21/10/2026): dice si el pedido ya falla, si falla en una instalación nueva o si va a fallar cuando suban el SDK |
|
|
136
136
|
|
|
@@ -159,6 +159,8 @@ Para reportar una vulnerabilidad, y el detalle de todo esto: [`SECURITY.md`](SEC
|
|
|
159
159
|
|
|
160
160
|
## ARCA en PHP
|
|
161
161
|
|
|
162
|
+
**¿Tenés que agregar el campo a mano?** La guía paso a paso, para afip.js, afip.php, @arcasdk/core y SOAP propio: [Cómo agregar `CondicionIVAReceptorId` antes del 1/12/2026](docs/guia-condicion-iva-receptor.md).
|
|
163
|
+
|
|
162
164
|
Para `afipsdk/afip.php` y sus forks (`fertandil87/afip.php`, `ivanalemunioz/afip.php`, `eesnaola/afip.php`), Parche busca `$afip->ElectronicBilling->CreateVoucher($data)` y `CreateNextVoucher($data)` (también en minúsculas: PHP no distingue) y agrega una línea por factura:
|
|
163
165
|
|
|
164
166
|
```php
|
|
@@ -171,7 +173,9 @@ Para `afipsdk/afip.php` y sus forks (`fertandil87/afip.php`, `ivanalemunioz/afip
|
|
|
171
173
|
- Copia el estilo del array: comillas de las claves, indentación (espacios o tabs), alineación de los `=>`, coma final y fin de línea (LF o CRLF).
|
|
172
174
|
- Saltea `vendor/`.
|
|
173
175
|
|
|
174
|
-
**La versión de la librería importa.** Con `afip.js`/`afip.php` 0.x
|
|
176
|
+
**La versión de la librería importa.** Con `afip.js`/`afip.php` 0.x, `eesnaola/afip.php` y `ts-afip-ws` (un fork de afip.js 0.x; su última versión, la 1.3.2, trae el WSDL sin el campo) el pedido se arma con un WSDL de 2021 que no tiene `CondicionIVAReceptorId`: aunque el campo esté en el código, no llega a ARCA. Parche lee la versión instalada (`composer.lock`, `package-lock.json`, `node_modules`) y, en esos casos, el arreglo sale como **"revisar"** y lo que pide un dato lleva el mismo aviso: qué actualizar para que el campo viaje. También avisa si el repo tiene un `.wsdl` de WSFEv1 sin el campo.
|
|
177
|
+
|
|
178
|
+
**Clave casi bien escrita.** Si el detalle del comprobante lleva `CondicionIVAReceptor` (sin `Id`), `CondicionIvaReceptorId` o `condicionIVAReceptorId`, ese elemento no existe en el WSDL: el dato nunca llega y el repo parece migrado. Parche renombra la clave a `CondicionIVAReceptorId` sin tocar el valor, en JS y en PHP, y el PR sale **borrador** para revisar que el valor sea un código válido. Solo lo hace dentro de objetos o arrays con al menos dos claves de WSFEv1 (`DocTipo`, `CbteDesde`, `ImpTotal`…), para no tocar modelos propios con un nombre parecido, y nunca en tests.
|
|
175
179
|
|
|
176
180
|
**Otros lenguajes:** en Python, C#, Java, Go y VB.NET, Parche detecta integraciones SOAP propias y clientes SOAP generados sin el campo (no los reescribe).
|
|
177
181
|
|
|
@@ -203,7 +207,7 @@ Cuando `DocTipo` no es un número fijo (`DocTipo: docTipo`), Parche no puede sab
|
|
|
203
207
|
## Límites del MVP
|
|
204
208
|
|
|
205
209
|
- JS/TS (ESM y CommonJS) y, para ARCA, PHP. Siguen Python y Java.
|
|
206
|
-
- ARCA: cubre `@afipsdk/afip.js` y `afipsdk/afip.php` (con sus forks). Las integraciones SOAP propias (en cualquier lenguaje) se detectan pero no se reescriben.
|
|
210
|
+
- ARCA: cubre `@afipsdk/afip.js`, `@arcasdk/core` (`arca.electronicBillingService.createVoucher` y `createNextVoucher`) y `afipsdk/afip.php` (con sus forks). Las integraciones SOAP propias (en cualquier lenguaje) se detectan pero no se reescriben, salvo el renombre de una clave casi bien escrita.
|
|
207
211
|
- IA, parámetros de Claude 4.7 y posteriores: avisa solo si el modelo del pedido se lee en el mismo archivo (un literal, una constante asignada una vez o el valor por defecto de una variable de entorno) y el parámetro es un número escrito en el código. Si el modelo llega de otro archivo o de la configuración, no avisa: no se puede saber si el pedido va a fallar.
|
|
208
212
|
- IA, API version `2026-10-21` de Anthropic: la versión del SDK sale de `package-lock.json`, `node_modules` o el rango de `package.json`, y en Python solo de `requirements.txt` (todavía no de `pyproject.toml`, `poetry.lock` ni `uv.lock`). Un rango sin número (`latest`, `*`) cuenta como versión desconocida. Por HTTP directo avisa solo si el archivo manda `anthropic-version: 2026-10-21`. Los SDKs de Go, Java, .NET, Ruby y PHP no se leen.
|
|
209
213
|
- IA, alias de Anthropic: avisa los de Claude Sonnet 4.5, Opus 4.5 y Haiku 4.5 (`claude-sonnet-4-5`, `claude-opus-4-5`, `claude-haiku-4-5`), escritos enteros entre comillas. Los alias de modelos ya retirados (`claude-3-5-sonnet-latest`, `claude-3-7-sonnet-latest` y otros) todavía no: la documentación actual de Anthropic ya no los nombra.
|
package/data/known-sdks.json
CHANGED
|
@@ -15,6 +15,7 @@
|
|
|
15
15
|
"afip.ts": { "provider": "ARCA (ex AFIP)", "category": "facturación" },
|
|
16
16
|
"ts-afip-ws": { "provider": "ARCA (ex AFIP)", "category": "facturación" },
|
|
17
17
|
"facturajs": { "provider": "ARCA (ex AFIP)", "category": "facturación" },
|
|
18
|
+
"facturas": { "provider": "ARCA (ex AFIP)", "category": "facturación" },
|
|
18
19
|
"openai": { "provider": "OpenAI", "category": "ia" },
|
|
19
20
|
"@anthropic-ai/sdk": { "provider": "Anthropic", "category": "ia" },
|
|
20
21
|
"@google/generative-ai": { "provider": "Google (Gemini)", "category": "ia" },
|
package/package.json
CHANGED
|
@@ -1,24 +1,24 @@
|
|
|
1
1
|
{
|
|
2
2
|
"id": "arca.wsfe.condicion-iva-receptor",
|
|
3
|
-
"version": "2026.10.
|
|
3
|
+
"version": "2026.10.24",
|
|
4
4
|
"provider": "ARCA (ex AFIP)",
|
|
5
5
|
"title": "Condición frente al IVA del receptor obligatoria en WSFEv1",
|
|
6
6
|
"kind": "breaking",
|
|
7
7
|
"severity": "critical",
|
|
8
8
|
"deadline": "2026-12-01",
|
|
9
|
-
"last_verified": "2026-10-
|
|
9
|
+
"last_verified": "2026-10-24",
|
|
10
10
|
"summary_es": "Desde el 1/12/2026 ARCA rechaza automáticamente las solicitudes de CAE que no informen CondicionIVAReceptorId.",
|
|
11
|
-
"why_es": "La RG 5616/2024 hizo obligatorio informar la condición frente al IVA del receptor en cada comprobante electrónico. El manual del desarrollador de WSFEv1
|
|
11
|
+
"why_es": "La RG 5616/2024 hizo obligatorio informar la condición frente al IVA del receptor en cada comprobante electrónico. El manual del desarrollador de WSFEv1 fija el 1 de diciembre de 2026 como fecha desde la cual el campo es obligatorio: si viaja vacío, nulo o no viaja, el pedido de CAE se rechaza y no se puede facturar.",
|
|
12
12
|
"sources": [
|
|
13
|
-
{ "title": "VATupdate: Argentina makes recipient VAT status mandatory
|
|
13
|
+
{ "title": "VATupdate: Argentina makes recipient VAT status mandatory in electronic invoices", "url": "https://www.vatupdate.com/2026/09/09/argentina-makes-recipient-vat-status-mandatory-in-electronic-invoices/" },
|
|
14
14
|
{ "title": "iProfesional: ARCA rechazará facturas sin la condición del cliente frente al IVA", "url": "https://www.iprofesional.com/impuestos/464104-nuevo-requisito-arca-puede-trabar-emision-facturas" },
|
|
15
15
|
{ "title": "Tabla de códigos de FEParamGetCondicionIvaReceptor", "url": "https://app.spomsolutions.com/blog/condicion-iva-receptor-rg-5616" },
|
|
16
16
|
{ "title": "Afip SDK: factura electrónica (Node y PHP, CreateVoucher / CreateNextVoucher)", "url": "https://docs.afipsdk.com/siguientes-pasos/web-services/factura-electronica" },
|
|
17
17
|
{ "title": "afipsdk/afip.php en Packagist", "url": "https://packagist.org/packages/afipsdk/afip.php" }
|
|
18
18
|
],
|
|
19
19
|
"detect": {
|
|
20
|
-
"packages": ["@afipsdk/afip.js", "@arcasdk/core", "afip.ts", "ts-afip-ws", "facturajs", "afipsdk/afip.php", "fertandil87/afip.php", "ivanalemunioz/afip.php", "eesnaola/afip.php"],
|
|
21
|
-
"calls": ["ElectronicBilling.createVoucher", "ElectronicBilling.createNextVoucher", "ElectronicBilling->CreateVoucher", "ElectronicBilling->CreateNextVoucher"],
|
|
20
|
+
"packages": ["@afipsdk/afip.js", "@arcasdk/core", "afip.ts", "ts-afip-ws", "facturajs", "facturas", "afipsdk/afip.php", "fertandil87/afip.php", "ivanalemunioz/afip.php", "eesnaola/afip.php"],
|
|
21
|
+
"calls": ["ElectronicBilling.createVoucher", "ElectronicBilling.createNextVoucher", "ElectronicBilling->CreateVoucher", "ElectronicBilling->CreateNextVoucher", "electronicBillingService.createVoucher", "electronicBillingService.createNextVoucher"],
|
|
22
22
|
"raw_markers": ["FECAESolicitar", "FECAEDetRequest"],
|
|
23
23
|
"field": "CondicionIVAReceptorId"
|
|
24
24
|
},
|
|
@@ -44,8 +44,10 @@
|
|
|
44
44
|
"codemod": "arca-wsfe-condicion-iva",
|
|
45
45
|
"review_checklist_es": [
|
|
46
46
|
"Probar la emisión en homologación (wsfehomo) con cada tipo de comprobante que emitís.",
|
|
47
|
-
"Confirmar que cada cliente con CUIT tiene guardada su condición frente al IVA y que se envía en cada CAE.",
|
|
48
47
|
"Pedir la condición frente al IVA en el alta de clientes nuevos.",
|
|
49
48
|
"Validar los códigos con FEParamGetCondicionIvaReceptor (la tabla puede cambiar)."
|
|
49
|
+
],
|
|
50
|
+
"review_checklist_if_pending_es": [
|
|
51
|
+
"Confirmar que cada cliente con CUIT tiene guardada su condición frente al IVA y que se envía en cada CAE."
|
|
50
52
|
]
|
|
51
53
|
}
|
package/src/ast.js
CHANGED
|
@@ -23,11 +23,12 @@ export function eolOf(source) {
|
|
|
23
23
|
return crlf > lf ? '\r\n' : '\n';
|
|
24
24
|
}
|
|
25
25
|
|
|
26
|
-
// Aplica
|
|
26
|
+
// Aplica ediciones de texto { offset, text, remove } de atrás para adelante, así no se corren los offsets.
|
|
27
|
+
// Sin `remove` es una inserción; con `remove`, `text` reemplaza esa cantidad de caracteres desde `offset`.
|
|
27
28
|
export function applyEdits(source, edits) {
|
|
28
29
|
let out = source;
|
|
29
30
|
for (const e of [...edits].sort((a, b) => b.offset - a.offset)) {
|
|
30
|
-
out = out.slice(0, e.offset) + e.text + out.slice(e.offset);
|
|
31
|
+
out = out.slice(0, e.offset) + e.text + out.slice(e.offset + (e.remove || 0));
|
|
31
32
|
}
|
|
32
33
|
return out;
|
|
33
34
|
}
|
|
@@ -79,6 +79,25 @@ export function partialLeftMessage(pending) {
|
|
|
79
79
|
return `Tiene el arreglo parcial de Parche (solo DocTipo 99 → 5). ${pending.message}`;
|
|
80
80
|
}
|
|
81
81
|
|
|
82
|
+
// Clave casi igual al campo (frente B, punto 13): `CondicionIVAReceptor` sin Id, `CondicionIvaReceptorId`,
|
|
83
|
+
// `condicionIVAReceptorId`. Ese elemento no existe en el WSDL de WSFEv1, así que el dato nunca viaja y el repo
|
|
84
|
+
// parece migrado (pablocla/erpia lo tenía en cuatro archivos). Solo se toca dentro del detalle de un comprobante
|
|
85
|
+
// (dos o más claves de WSFEv1), para no renombrar modelos o DTOs propios que se llamen parecido.
|
|
86
|
+
const FIELD = 'CondicionIVAReceptorId';
|
|
87
|
+
export const NEAR_HINT = /condicion_?iva_?receptor/i;
|
|
88
|
+
const DETAIL_KEYS = new Set(['Concepto', 'DocTipo', 'DocNro', 'CbteDesde', 'CbteHasta', 'CbteFch', 'ImpTotal', 'ImpNeto', 'ImpIVA', 'MonId', 'MonCotiz']);
|
|
89
|
+
export const isNearMissKey = (key) => typeof key === 'string' && key !== FIELD && /^condicion_?iva_?receptor(?:_?id)?$/i.test(key);
|
|
90
|
+
export const isVoucherDetail = (keys) => keys.filter((k) => DETAIL_KEYS.has(k)).length >= 2;
|
|
91
|
+
|
|
92
|
+
export function nearMissMessage(key, pack) {
|
|
93
|
+
return `\`${key}\` no es un campo de WSFEv1: ARCA espera \`${FIELD}\`, así que hoy el dato no llega y desde el ${dmy(pack.deadline)} ARCA rechaza el CAE. `
|
|
94
|
+
+ 'Lo renombré sin tocar el valor: revisá que sea un código de FEParamGetCondicionIvaReceptor.';
|
|
95
|
+
}
|
|
96
|
+
|
|
97
|
+
export function nearMissBothMessage(key) {
|
|
98
|
+
return `El comprobante tiene \`${key}\` y también \`${FIELD}\`. \`${key}\` no es un campo de WSFEv1: se puede borrar. Revisá que \`${FIELD}\` lleve el valor correcto.`;
|
|
99
|
+
}
|
|
100
|
+
|
|
82
101
|
// "El repo ya usa CondicionIVAReceptorId en otro lado": prefijo para bajar a "revisar" lo que no se pudo leer
|
|
83
102
|
// (en repos ya migrados, el objeto suele salir de un builder que sí pone el campo).
|
|
84
103
|
export function alreadyMigratedPrefix(context) {
|
|
@@ -92,7 +111,7 @@ export function libraryCaveat(context, ecosystem) {
|
|
|
92
111
|
const lib = (context?.arcaLibs || []).find((l) => l.ecosystem === ecosystem && l.travels !== 'si');
|
|
93
112
|
if (lib) {
|
|
94
113
|
return `Ojo: con ${lib.name} ${lib.version} el pedido se arma con un WSDL de 2021 que no tiene este campo`
|
|
95
|
-
+ `${lib.travels === 'no' ? '' : ' (en JS falta confirmarlo)'}: para que llegue a ARCA hay que actualizar la librería a 1.x.`;
|
|
114
|
+
+ `${lib.travels === 'no' ? '' : ' (en JS falta confirmarlo)'}: para que llegue a ARCA hay que ${lib.fix || 'actualizar la librería a 1.x'}.`;
|
|
96
115
|
}
|
|
97
116
|
const wsdl = context?.staleWsdl?.[0];
|
|
98
117
|
if (wsdl) return `Ojo: el repo tiene un WSDL de WSFEv1 sin este campo (\`${wsdl}\`): si el cliente SOAP lo usa, el campo no viaja hasta actualizarlo.`;
|
|
@@ -8,8 +8,10 @@ import { parsePhp, walkPhp, phpLiteral, phpKey } from '../php.js';
|
|
|
8
8
|
import {
|
|
9
9
|
PACK, decideCondition, validCodesText, missingMessage, appliedMessage, soapDetection, ARCA_HINT, libraryCaveat, alreadyMigratedPrefix,
|
|
10
10
|
canApplyAtRuntime, PARTIAL_COMMENT, partialAppliedMessage, partialPending, partialLeftMessage,
|
|
11
|
+
NEAR_HINT, isNearMissKey, isVoucherDetail, nearMissMessage, nearMissBothMessage,
|
|
11
12
|
} from './arca-rules.js';
|
|
12
13
|
import { emptyContext } from '../context.js';
|
|
14
|
+
import { isTestFile } from '../paths.js';
|
|
13
15
|
|
|
14
16
|
// En PHP los nombres de método no distinguen mayúsculas: createVoucher también es válido.
|
|
15
17
|
// El inventario usa el nombre oficial, así un mismo método no aparece dos veces.
|
|
@@ -112,7 +114,7 @@ function resolveData(arg, scope, call, scopes) {
|
|
|
112
114
|
export default function transform({ source, file, pack, context = emptyContext() }) {
|
|
113
115
|
const findings = [];
|
|
114
116
|
const usages = [];
|
|
115
|
-
if (!ARCA_HINT.test(source)) return { output: null, findings, usages }; // prefiltro (ver el codemod de JS)
|
|
117
|
+
if (!ARCA_HINT.test(source) && !NEAR_HINT.test(source)) return { output: null, findings, usages }; // prefiltro (ver el codemod de JS)
|
|
116
118
|
const caveat = libraryCaveat(context, 'composer');
|
|
117
119
|
const migrated = alreadyMigratedPrefix(context);
|
|
118
120
|
let ast;
|
|
@@ -129,6 +131,28 @@ export default function transform({ source, file, pack, context = emptyContext()
|
|
|
129
131
|
const { scopes, calls } = collect(ast);
|
|
130
132
|
const edits = [];
|
|
131
133
|
const edited = new Set(); // el mismo $data en dos llamadas: se inserta una sola vez (ver el codemod de JS)
|
|
134
|
+
|
|
135
|
+
// Punto 13 del frente B: 'CondicionIVAReceptor' => $x (sin Id) se renombra, como en JS. Solo cambia el texto
|
|
136
|
+
// entre las comillas de la clave; el array queda marcado para que la llamada no reciba un campo nuevo.
|
|
137
|
+
let renamed = false;
|
|
138
|
+
if (!isTestFile(file)) {
|
|
139
|
+
walkPhp(ast, (node) => {
|
|
140
|
+
if (node.kind !== 'array') return;
|
|
141
|
+
const entries = node.items.filter((it) => it && it.kind === 'entry');
|
|
142
|
+
const near = entries.find((e) => isNearMissKey(phpKey(e)));
|
|
143
|
+
if (!near || !isVoucherDetail(entries.map(phpKey))) return;
|
|
144
|
+
edited.add(node);
|
|
145
|
+
if (entries.some((e) => phpKey(e) === FIELD)) {
|
|
146
|
+
findings.push(finding(PACK, file, near, 'needs-review', nearMissBothMessage(phpKey(near))));
|
|
147
|
+
return;
|
|
148
|
+
}
|
|
149
|
+
const { start, end } = near.key.loc;
|
|
150
|
+
edits.push({ offset: start.offset + 1, remove: end.offset - start.offset - 2, text: FIELD });
|
|
151
|
+
renamed = true;
|
|
152
|
+
findings.push(finding(PACK, file, near, 'needs-review', nearMissMessage(phpKey(near), pack) + (caveat ? ` ${caveat}` : '')));
|
|
153
|
+
});
|
|
154
|
+
}
|
|
155
|
+
|
|
132
156
|
for (const { node: call, scope, method } of calls) {
|
|
133
157
|
usages.push({ provider: 'arca', op: `ElectronicBilling->${method}`, line: call.loc.start.line });
|
|
134
158
|
const data = resolveData(call.arguments[0], scope, call, scopes);
|
|
@@ -136,7 +160,7 @@ export default function transform({ source, file, pack, context = emptyContext()
|
|
|
136
160
|
const entries = data.arr ? data.arr.items.filter((it) => it && it.kind === 'entry') : null;
|
|
137
161
|
if (!entries || entries.some((e) => e.unpack || e.byRef)) {
|
|
138
162
|
const msg = data.hint || `No pude leer los datos del comprobante (${data.reason || 'el array usa ...$otro o referencias'}). Agregá ${FIELD} a mano, ${HINT_TAIL}`;
|
|
139
|
-
findings.push(finding(PACK, file, call, migrated ? 'needs-review' : 'needs-input', (migrated || '') + msg));
|
|
163
|
+
findings.push(finding(PACK, file, call, migrated ? 'needs-review' : 'needs-input', (migrated || '') + msg + (caveat ? ` ${caveat}` : '')));
|
|
140
164
|
continue;
|
|
141
165
|
}
|
|
142
166
|
const byKey = (k) => entries.find((e) => phpKey(e) === k);
|
|
@@ -175,7 +199,7 @@ export default function transform({ source, file, pack, context = emptyContext()
|
|
|
175
199
|
findings.push(finding(PACK, file, call, status, appliedMessage(d.status) + (caveat ? ` ${caveat}` : '')));
|
|
176
200
|
}
|
|
177
201
|
|
|
178
|
-
const soap = soapDetection(source, file, context);
|
|
202
|
+
const soap = renamed ? null : soapDetection(source, file, context); // ya manda el campo (ver el codemod de JS)
|
|
179
203
|
if (soap) { findings.push(soap.finding); usages.push(soap.usage); }
|
|
180
204
|
|
|
181
205
|
return { output: edits.length ? applyEdits(source, edits) : null, findings, usages };
|
|
@@ -2,13 +2,15 @@
|
|
|
2
2
|
// Agrega CondicionIVAReceptorId a cada pedido de CAE hecho con afip.js cuando se puede inferir
|
|
3
3
|
// sin datos del negocio, y marca el resto para que una persona complete el dato.
|
|
4
4
|
import {
|
|
5
|
-
jFor, detectQuote, eolOf, offsetAt, applyEdits, getProp, isQuotedKey, isPlainProp, literalValue, resolveBinding, finding,
|
|
5
|
+
jFor, detectQuote, eolOf, offsetAt, applyEdits, getProp, isQuotedKey, isPlainProp, keyName, literalValue, resolveBinding, finding,
|
|
6
6
|
} from '../ast.js';
|
|
7
7
|
import {
|
|
8
8
|
PACK, decideCondition, validCodesText, missingMessage, appliedMessage, soapDetection, ARCA_HINT, libraryCaveat, alreadyMigratedPrefix,
|
|
9
9
|
canApplyAtRuntime, PARTIAL_COMMENT, partialAppliedMessage, partialPending, partialLeftMessage,
|
|
10
|
+
NEAR_HINT, isNearMissKey, isVoucherDetail, nearMissMessage, nearMissBothMessage,
|
|
10
11
|
} from './arca-rules.js';
|
|
11
12
|
import { emptyContext } from '../context.js';
|
|
13
|
+
import { isTestFile } from '../paths.js';
|
|
12
14
|
|
|
13
15
|
const VOUCHER_METHODS = new Set(['createVoucher', 'createNextVoucher']);
|
|
14
16
|
const WRAPPERS = new Set(['TSAsExpression', 'TSNonNullExpression', 'TSTypeAssertion', 'TSSatisfiesExpression', 'ParenthesizedExpression']);
|
|
@@ -18,7 +20,10 @@ const unwrap = (n) => {
|
|
|
18
20
|
while (n && WRAPPERS.has(n.type)) n = n.expression;
|
|
19
21
|
return n;
|
|
20
22
|
};
|
|
21
|
-
|
|
23
|
+
// afip.ElectronicBilling (afip.js) o arca.electronicBillingService (@arcasdk/core): las dos reciben el mismo
|
|
24
|
+
// comprobante, con las claves de WSFEv1 (tipos de @arcasdk/core 2.0.2, leídos en su paquete de npm el 24/10/2026).
|
|
25
|
+
const BILLING_MEMBERS = new Set(['ElectronicBilling', 'electronicBillingService']);
|
|
26
|
+
const isEBMember = (n) => !!(n && n.type === 'MemberExpression' && !n.computed && n.property.type === 'Identifier' && BILLING_MEMBERS.has(n.property.name));
|
|
22
27
|
|
|
23
28
|
// ¿`node` es `<algo>.ElectronicBilling`? También con casts de TS y con un alias en el mismo scope
|
|
24
29
|
// (`const eb = afip.ElectronicBilling; eb.createVoucher(...)`): los dos aparecieron en repos reales.
|
|
@@ -32,6 +37,13 @@ function isElectronicBilling(node, p) {
|
|
|
32
37
|
return false;
|
|
33
38
|
}
|
|
34
39
|
|
|
40
|
+
// Para el inventario: `ElectronicBilling` o `electronicBillingService`, también si la llamada pasa por un alias.
|
|
41
|
+
function billingName(node, p) {
|
|
42
|
+
let n = unwrap(node);
|
|
43
|
+
if (n && n.type === 'Identifier') n = unwrap(resolveBinding(p, n.name)?.node.init);
|
|
44
|
+
return isEBMember(n) ? n.property.name : 'ElectronicBilling';
|
|
45
|
+
}
|
|
46
|
+
|
|
35
47
|
function voucherCallee(p) {
|
|
36
48
|
const callee = unwrap(p.node.callee);
|
|
37
49
|
if (!callee || callee.type !== 'MemberExpression' || callee.computed || callee.property.type !== 'Identifier') return null;
|
|
@@ -119,7 +131,8 @@ const suggestionFor = (d, pack) => `'CondicionIVAReceptorId': <condición del cl
|
|
|
119
131
|
export default function transform({ source, file, pack, context = emptyContext() }) {
|
|
120
132
|
const findings = [];
|
|
121
133
|
const usages = [];
|
|
122
|
-
|
|
134
|
+
const arcaFile = ARCA_HINT.test(source);
|
|
135
|
+
if (!arcaFile && !NEAR_HINT.test(source)) return { output: null, findings, usages };
|
|
123
136
|
const caveat = libraryCaveat(context, 'npm');
|
|
124
137
|
const migrated = alreadyMigratedPrefix(context);
|
|
125
138
|
const j = jFor(file);
|
|
@@ -128,8 +141,10 @@ export default function transform({ source, file, pack, context = emptyContext()
|
|
|
128
141
|
root = j(source);
|
|
129
142
|
} catch (err) {
|
|
130
143
|
// Antes se descartaba en silencio: un archivo de ARCA que no parsea es justo el que hay que mirar.
|
|
131
|
-
|
|
132
|
-
|
|
144
|
+
if (arcaFile) {
|
|
145
|
+
findings.push({ pack: PACK, file, line: err.loc?.line || null, status: 'needs-input',
|
|
146
|
+
message: `No pude analizar este archivo (${err.message.split('\n')[0]}). Revisá a mano las llamadas a ElectronicBilling.` });
|
|
147
|
+
}
|
|
133
148
|
return { output: null, findings, usages };
|
|
134
149
|
}
|
|
135
150
|
let changed = false;
|
|
@@ -138,9 +153,31 @@ export default function transform({ source, file, pack, context = emptyContext()
|
|
|
138
153
|
// texto y el árbol no se entera, sin esto se insertaba dos veces → clave duplicada y TypeScript no compila.
|
|
139
154
|
const edited = new Set();
|
|
140
155
|
|
|
156
|
+
// Punto 13 del frente B: la clave casi bien escrita se renombra; el nombre correcto lo fija el WSDL.
|
|
157
|
+
// Va antes que las llamadas, así el comprobante renombrado no recibe además un CondicionIVAReceptorId nuevo.
|
|
158
|
+
let renamed = false;
|
|
159
|
+
if (!isTestFile(file)) {
|
|
160
|
+
root.find(j.ObjectExpression).forEach((p) => {
|
|
161
|
+
const obj = p.node;
|
|
162
|
+
const props = obj.properties.filter(isPlainProp);
|
|
163
|
+
const near = props.find((pr) => isNearMissKey(keyName(pr)));
|
|
164
|
+
if (!near || !isVoucherDetail(props.map(keyName))) return;
|
|
165
|
+
edited.add(obj);
|
|
166
|
+
if (getProp(obj, 'CondicionIVAReceptorId')) {
|
|
167
|
+
findings.push(finding(PACK, file, near, 'needs-review', nearMissBothMessage(keyName(near))));
|
|
168
|
+
return;
|
|
169
|
+
}
|
|
170
|
+
edits.push(renameKey(source, near));
|
|
171
|
+
renamed = true;
|
|
172
|
+
changed = true;
|
|
173
|
+
findings.push(finding(PACK, file, near, 'needs-review', nearMissMessage(keyName(near), pack) + (caveat ? ` ${caveat}` : '')));
|
|
174
|
+
});
|
|
175
|
+
}
|
|
176
|
+
|
|
141
177
|
root.find(j.CallExpression).filter((p) => voucherCallee(p) !== null).forEach((p) => {
|
|
142
178
|
const call = p.node;
|
|
143
|
-
|
|
179
|
+
const callee = voucherCallee(p);
|
|
180
|
+
usages.push({ provider: 'arca', op: `${billingName(callee.object, p)}.${callee.property.name}`, line: call.loc?.start.line });
|
|
144
181
|
const arg = call.arguments[0];
|
|
145
182
|
let obj = null;
|
|
146
183
|
let binding = null;
|
|
@@ -151,7 +188,9 @@ export default function transform({ source, file, pack, context = emptyContext()
|
|
|
151
188
|
}
|
|
152
189
|
if (!obj || otherSpread(obj)) {
|
|
153
190
|
// Si el repo ya usa el campo en otro lado, lo más probable es que el builder ya lo ponga: "revisar".
|
|
154
|
-
|
|
191
|
+
// El aviso de librería vieja también va acá (frente B, punto 14: en los repos de jretondo los datos llegan de afuera).
|
|
192
|
+
findings.push(finding(PACK, file, call, migrated ? 'needs-review' : 'needs-input',
|
|
193
|
+
(migrated || '') + unreadableMessage(j, root, p, arg, obj) + (caveat ? ` ${caveat}` : '')));
|
|
155
194
|
return;
|
|
156
195
|
}
|
|
157
196
|
if (getProp(obj, 'CondicionIVAReceptorId') || edited.has(obj)) return; // ya está migrado (o ya lo editamos)
|
|
@@ -196,12 +235,23 @@ export default function transform({ source, file, pack, context = emptyContext()
|
|
|
196
235
|
findings.push(finding(PACK, file, call, status, appliedMessage(d.status) + (caveat ? ` ${caveat}` : '')));
|
|
197
236
|
});
|
|
198
237
|
|
|
199
|
-
|
|
238
|
+
// Con la clave renombrada el archivo ya manda el campo: el aviso de SOAP sin CondicionIVAReceptorId sobra.
|
|
239
|
+
const soap = renamed ? null : soapDetection(source, file, context);
|
|
200
240
|
if (soap) { findings.push(soap.finding); usages.push(soap.usage); }
|
|
201
241
|
|
|
202
242
|
return { output: changed ? applyEdits(source, edits) : null, findings, usages };
|
|
203
243
|
}
|
|
204
244
|
|
|
245
|
+
// `CondicionIVAReceptor: x` → `CondicionIVAReceptorId: x`, con las mismas comillas y sin tocar el valor.
|
|
246
|
+
// Si era abreviada (`{ CondicionIVAReceptor }`), queda `CondicionIVAReceptorId: CondicionIVAReceptor`.
|
|
247
|
+
function renameKey(source, prop) {
|
|
248
|
+
const start = offsetAt(source, prop.key.loc.start);
|
|
249
|
+
const end = offsetAt(source, prop.key.loc.end);
|
|
250
|
+
if (prop.shorthand) return { offset: start, remove: end - start, text: `CondicionIVAReceptorId: ${keyName(prop)}` };
|
|
251
|
+
const q = isQuotedKey(prop) ? source[start] : '';
|
|
252
|
+
return { offset: start, remove: end - start, text: `${q}CondicionIVAReceptorId${q}` };
|
|
253
|
+
}
|
|
254
|
+
|
|
205
255
|
// Cuando no se puede leer el comprobante, decir DÓNDE hay que agregar el dato: la función que lo arma
|
|
206
256
|
// (y de qué módulo viene) o que llega como parámetro. Lo encontrado en repos reales: payload = buildVoucherPayload(...).
|
|
207
257
|
function unreadableMessage(j, root, p, arg, obj) {
|
package/src/context.js
CHANGED
|
@@ -59,8 +59,13 @@ export const openRange = (range) => /^\s*(?:>=?|\*|x|latest)/i.test(range || '')
|
|
|
59
59
|
// afip.js / afip.php ≥ 1.0: mandan JSON a la API de Afip SDK → sí.
|
|
60
60
|
// 0.x: arman el SOAP con un WSDL de 2021 incluido en la librería, sin el campo → no (PHP) o sin confirmar (JS).
|
|
61
61
|
// fertandil87/afip.php: actualizó su WSDL (lo incluye) → sí. eesnaola/afip.php: WSDL de 2021 → igual que 0.x.
|
|
62
|
+
// ts-afip-ws: fork de afip.js 0.x (frente B, punto 14: los repos de jretondo). Su última versión, la 1.3.2 de 2023,
|
|
63
|
+
// trae wsfe.wsdl y wsfe-production.wsdl sin el campo (leído en el paquete de npm el 24/10/2026) → sin confirmar.
|
|
62
64
|
const ARCA_LIBS = {
|
|
63
|
-
npm: {
|
|
65
|
+
npm: {
|
|
66
|
+
'@afipsdk/afip.js': (v) => (major(v) >= 1 ? 'si' : 'sin-confirmar'),
|
|
67
|
+
'ts-afip-ws': () => 'sin-confirmar',
|
|
68
|
+
},
|
|
64
69
|
composer: {
|
|
65
70
|
'afipsdk/afip.php': (v) => (major(v) >= 1 ? 'si' : 'no'),
|
|
66
71
|
'ivanalemunioz/afip.php': (v) => (major(v) >= 1 ? 'si' : 'no'),
|
|
@@ -69,13 +74,18 @@ const ARCA_LIBS = {
|
|
|
69
74
|
},
|
|
70
75
|
};
|
|
71
76
|
const major = (v) => Number(String(v).split('.')[0]);
|
|
77
|
+
// Qué hacer para que el campo viaje, cuando no es "actualizar la librería a 1.x" (los forks no tienen 1.x).
|
|
78
|
+
const FIXES = {
|
|
79
|
+
'ts-afip-ws': 'pasar a @afipsdk/afip.js 1.x o actualizar el WSDL que trae la librería',
|
|
80
|
+
'eesnaola/afip.php': 'pasar a afipsdk/afip.php 1.x o actualizar el WSDL que trae la librería',
|
|
81
|
+
};
|
|
72
82
|
|
|
73
83
|
export function arcaLibraries(versions) {
|
|
74
84
|
const out = [];
|
|
75
85
|
for (const [eco, libs] of Object.entries(ARCA_LIBS)) {
|
|
76
86
|
for (const [name, rule] of Object.entries(libs)) {
|
|
77
87
|
const v = versions[eco]?.[name];
|
|
78
|
-
if (v) out.push({ ecosystem: eco, name, version: v.version, source: v.source, travels: rule(v.version) });
|
|
88
|
+
if (v) out.push({ ecosystem: eco, name, version: v.version, source: v.source, travels: rule(v.version), ...(FIXES[name] ? { fix: FIXES[name] } : {}) });
|
|
79
89
|
}
|
|
80
90
|
}
|
|
81
91
|
return out;
|
package/src/report.js
CHANGED
|
@@ -158,9 +158,15 @@ export function prBody({ pack, findings, changed, verification, ciApproval = fal
|
|
|
158
158
|
out.push('- CI de este PR: lo abrió GitHub Actions con el `GITHUB_TOKEN`, así que GitHub deja sus workflows esperando aprobación. Alguien con permiso de escritura tiene que tocar **Approve workflows to run**. Para que corran solos, pasale a la Action un token de una GitHub App o un token personal (input `token`).');
|
|
159
159
|
}
|
|
160
160
|
}
|
|
161
|
-
|
|
161
|
+
// Los ítems que dependen de un dato del negocio (por ejemplo, los clientes con CUIT) solo van si queda algo
|
|
162
|
+
// pendiente: en un PR que arregla todo, como uno de puros consumidores finales, no aplican (frente B, punto 12).
|
|
163
|
+
const checklist = [...pack.review_checklist_es, ...(pending.length ? pack.review_checklist_if_pending_es || [] : [])];
|
|
164
|
+
out.push('', asIssue ? '### Checklist' : '### Checklist antes de mergear', '', ...checklist.map((x) => `- [ ] ${x}`), '');
|
|
162
165
|
out.push('### Fuentes', '', ...pack.sources.map((s) => `- [${s.title}](${s.url})`), '');
|
|
163
|
-
|
|
166
|
+
const feedback = asIssue
|
|
167
|
+
? '¿Te sirvió este aviso? Cerralo cuando lo resuelvas, o contestá si algo no aplica: Parche aprende de cada resultado.'
|
|
168
|
+
: '¿Este PR te sirvió? Mergealo, editalo o cerralo: Parche aprende de cada resultado.';
|
|
169
|
+
out.push(`<sub>Pack \`${pack.id}\` v${pack.version}, verificado el ${pack.last_verified}. Resumen: ${Object.entries(c).map(([k, v]) => `${LABEL[k]} ${v}`).join(' · ')}. ${feedback} [¿Qué es Parche y por qué recibiste esto?](https://github.com/Pepo08/parche/blob/main/docs/por-que-recibi-este-pr.md)</sub>`, '');
|
|
164
170
|
return out.join('\n');
|
|
165
171
|
}
|
|
166
172
|
|