cobro_digital 1.8.0 → 1.9.0

data/lib/cobro_digital.rb

@@ -7,6 +7,10 @@ require "cobro_digital/transaccion"
 require "cobro_digital/micrositio"
 require "cobro_digital/meta"
 require "savon"
+require "net/http"
+require "uri"
+require "digest"
+require "json"
 
 module CobroDigital
 
@@ -19,6 +23,18 @@ module CobroDigital
 
   TIMEOUT = 300
 
+  # Nivel de log del cliente SOAP. Default `:error` para no filtrar el `sid`
+  # ni PII del pagador a los logs de la app. Subir a `:debug` solo para
+  # troubleshooting explícito (vía ENV['COBRODIGITAL_LOG_LEVEL']).
+  # Se sanea el string vacío (`COBRODIGITAL_LOG_LEVEL=`) que produciría `:""`.
+  _log_level = ENV['COBRODIGITAL_LOG_LEVEL'].to_s
+  LOG_LEVEL  = (_log_level.empty? ? 'error' : _log_level).to_sym
+  DEBUG_LOG  = (LOG_LEVEL == :debug)
+
+  # Nodo del mensaje SOAP que transporta credenciales (`sid`) y PII del pagador.
+  # Savon lo enmascara como `***FILTERED***` en el log, incluso en `:debug`.
+  LOG_FILTERS = [:parametros_de_entrada].freeze
+
   module Https
     POST = 'Post'
     GET  = 'Get'
@@ -26,30 +42,45 @@ module CobroDigital
 
   class Client
 
+    # NOTA: `sid` y `request_xml` contienen datos sensibles (credencial del
+    # comercio y XML con sid + PII del pagador). No loguear estos accessors.
     attr_accessor :id_comercio, :sid, :client_to_use, :http_method, :pagadores, :boletas, :transacciones, :micrositios, :requests, :request_xml
 
     def initialize(attrs={})
       @id_comercio    = attrs[:id_comercio]
       @sid            = attrs[:sid]
-      @client_to_use  = attrs[:con_client].present? ? attrs[:con_client] : CobroDigital::SOAP
+      @client_to_use  = attrs[:con_client].to_s.empty? ? CobroDigital::SOAP : attrs[:con_client]
       # @with_handshake = attrs[:handshake].present? ? attrs[:handshake] : true
       @pagadores      = []
       @boletas        = []
       @transacciones  = []
       @micrositios    = []
       @request_xml    = nil
+
+      unless CobroDigital::CLIENTS.include?(@client_to_use)
+        raise ArgumentError, "client_to_use inválido: #{@client_to_use.inspect} (esperado uno de #{CobroDigital::CLIENTS.inspect})"
+      end
     end
 
     def soap_client(params)
+      # `log: true` para que se sigan registrando errores SOAP/HTTP; la
+      # verbosidad la controla `log_level` (default :error → no loguea el body).
+      # `filters` enmascara el nodo con sid + PII como ***FILTERED*** si algo se
+      # llega a loguear. ADVERTENCIA: con COBRODIGITAL_LOG_LEVEL=debug el XML
+      # formateado incluye el sid en claro — no habilitar debug en producción.
       client = Savon.client(
         wsdl: CobroDigital::WSDL,
-        log_level: :debug,
-        pretty_print_xml: true,
+        log: true,
+        log_level: CobroDigital::LOG_LEVEL,
+        filters: CobroDigital::LOG_FILTERS,
+        pretty_print_xml: CobroDigital::DEBUG_LOG,
         open_timeout: CobroDigital::TIMEOUT,
         read_timeout: CobroDigital::TIMEOUT
       )
       operation = client.operation(:webservice_cobrodigital)
       request = operation.build(message: { 'parametros_de_entrada' => params.to_json })
+      # Contrato público: `@request_xml` siempre disponible tras el call. Retiene
+      # el XML con sid + PII en memoria — el consumidor no debe loguearlo.
       @request_xml = request.pretty
 
       client.call(:webservice_cobrodigital, message: { 'parametros_de_entrada' => params.to_json })
@@ -58,12 +89,12 @@ module CobroDigital
     def https_client(params)
       case http_method
       when CobroDigital::Https::POST
-        uri = URI(CobroDigital::URI)
-        req = "Net::HTTP::#{http_method}".constantize.new(uri)
+        uri = ::URI.parse(CobroDigital::URI)
+        req = Net::HTTP::Post.new(uri)
         req.set_form_data(params)
       when CobroDigital::Https::GET
-        uri = URI([CobroDigital::URI, URI.encode_www_form(data)].join('?'))
-        req = "Net::HTTP::#{http_method}".constantize.new(uri)
+        uri = ::URI.parse([CobroDigital::URI, ::URI.encode_www_form(params)].join('?'))
+        req = Net::HTTP::Get.new(uri)
       end
 
       Net::HTTP.start(

data/skill/SKILL.md

@@ -0,0 +1,87 @@
+---
+name: cobro-digital
+description: >-
+  Gema adaptadora del Web Service v3 de CobroDigital (pasarela de pago).
+  Modela las operaciones del WS — crear/editar/verificar pagadores, generar/
+  inhabilitar boletas, obtener código de barras, consultar transacciones y
+  actividad de micrositios, y agrupar operaciones en batch (meta) — y resuelve
+  el transporte SOAP (default, vía savon) o HTTPS hacia el endpoint del comercio.
+  Activala cuando trabajes con cobranzas/boletas/pagadores contra CobroDigital,
+  cuando integres o depures la comunicación con su webservice, o cuando un host
+  consuma la gema `cobro_digital`.
+triggers:
+  - "cobrodigital"
+  - "cobro digital"
+  - "generar boleta"
+  - "crear pagador"
+  - "consultar transacciones cobrodigital"
+  - "webservice de pago cobrodigital"
+  - "gema cobro_digital"
+---
+
+# cobro_digital
+
+## Qué es / cuándo usar
+
+Adaptador Ruby del WS v3 de CobroDigital (pasarela de pago argentina). Úsalo
+para invocar las operaciones del webservice desde un host Ruby/Rails. La gema
+no persiste estado ni define excepciones propias: arma el payload, lo envía y
+decodifica la respuesta.
+
+## Contrato resumido
+
+**Patrón de uso:** cada operación es una subclase de `CobroDigital::Operador`
+construida por un método de clase y ejecutada con `#call(id_comercio, sid)`.
+
+```ruby
+op = CobroDigital::Boleta.generar(id, buscar, vencimientos, importes, concepto, plantilla)
+op.call(comercio_id, comercio_sid) # ejecuta; guarda #response
+op.parse_response                  # => { resultado: Bool, log: [..], datos: [..] }
+```
+
+**Superficie pública** (detalle: [`docs/interface/interface.md`](../docs/interface/interface.md)):
+
+- `CobroDigital::Pagador` — `.crear`, `.editar`, `.verificar`, `.codigo_electronico`, `.estructura_de_datos`
+- `CobroDigital::Boleta` — `.generar`, `.inhabilitar`, `.obtener_codigo_de_barras`
+- `CobroDigital::Transaccion` — `.consultar(desde, hasta, filtros)` + constantes `FILTRO_*`
+- `CobroDigital::Micrositio` — `.consultar_actividad`
+- `CobroDigital::Meta` — `.meta(objs)` (batch de operaciones en una llamada)
+- `CobroDigital::Client` — transporte; `CobroDigital::VERSION`
+
+**Config:** `ENV['ENDPOINT_COBRODIGITAL']` (default `https://cobro.digital:14365`)
+y `ENV['COBRODIGITAL_LOG_LEVEL']` (default `error`). Credenciales `id_comercio`/`sid`
+se pasan por argumento en cada `#call`, no por env.
+
+**Gotchas:**
+- `parse_response` **no levanta excepción** ante `resultado: false` — el unhappy
+  path se inspecciona vía `resultado`/`log`. Fallos de transporte propagan
+  `Savon::*` / `Net::HTTP` / `JSON::ParserError` sin envolver.
+- El WS **no controla duplicados** (`crear_pagador` es no-idempotente): el
+  control de unicidad es del consumidor.
+- `client_to_use` inválido (≠ `'soap'`/`'https'`) → `ArgumentError` en `Client.new`.
+- **No usar `COBRODIGITAL_LOG_LEVEL=debug` en producción**: el XML formateado del
+  request expone el `sid` + PII del pagador.
+- Requiere Ruby `>= 2.7, < 3.0` (stack savon 2.12). Desde v1.9.0 **no** depende de
+  ActiveSupport (solo stdlib).
+
+## Índice de artefactos
+
+| capa | doc | estado |
+|---|---|---|
+| interfaz | [`docs/interface/interface.md`](../docs/interface/interface.md) | API Ruby pública |
+| consumidas | [`docs/consumed/cobrodigital.md`](../docs/consumed/cobrodigital.md) | WS de CobroDigital (§a/§b/§d; §c/§e pendientes de enrich) |
+| configuración | [`docs/config/configuracion.md`](../docs/config/configuracion.md) | inventario base |
+| topología | [`docs/topology/topology.md`](../docs/topology/topology.md) | deps + modos de transporte |
+| test | [`docs/test/testing.md`](../docs/test/testing.md) | RSpec (suite mínima) |
+| comportamiento | [`docs/behavior/behavior.md`](../docs/behavior/behavior.md) | operación simple · batch meta |
+| glosario | [`docs/glossary/glossary.md`](../docs/glossary/glossary.md) | términos del dominio |
+| datos · api · errores · eventos · seguridad · multi-tenancy · data-lifecycle | — | n/a (ver Mapa de conocimiento en `AGENTS.md`) |
+| release | — | pendiente (`/gem-release`) |
+
+## Uso correcto / gotchas
+
+- Para varias operaciones en una sola llamada al WS, construí cada operación y
+  pasalas a `CobroDigital::Meta.meta([...])`.
+- Las fechas de `Boleta.generar`, `Transaccion.consultar` y `Micrositio.consultar_actividad`
+  se formatean internamente a `%Y%m%d` — pasá objetos `Date`/`Time`, no strings.
+- El handshake (`MD5` de `Time.now`) se regenera por request automáticamente.

data/skills.yml

@@ -0,0 +1,17 @@
+mcps:
+  - github
+  - clickup
+skills:
+  multi-vendor-feedback:
+  yard:
+  quality-code:
+  gem-release:
+  arch-structure:
+  arch-compose:
+  arch-enrich:
+  skill-feedback:
+  agent-issue:
+  bug-report:
+  dev-flow:
+  documentation-writer:
+  matrix-element:

metadata

@@ -1,42 +1,43 @@
 --- !ruby/object:Gem::Specification
 name: cobro_digital
 version: !ruby/object:Gem::Version
-  version: 1.8.0
+  version: 1.9.0
 platform: ruby
 authors:
 - g.edera
+autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-03-30 00:00:00.000000000 Z
+date: 2026-06-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: bundler
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 2.6.6
+        version: 13.2.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 2.6.6
+        version: 13.2.1
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 13.2.1
+        version: '3.13'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 13.2.1
+        version: '3.13'
 - !ruby/object:Gem::Dependency
   name: savon
   requirement: !ruby/object:Gem::Requirement
@@ -58,9 +59,13 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/main.yml"
+- ".github/workflows/release.yml"
 - ".gitignore"
 - ".rspec"
-- ".travis.yml"
+- AGENTS.md
+- CHANGELOG.md
+- CLAUDE.md
 - CODE_OF_CONDUCT.md
 - Gemfile
 - LICENSE.txt
@@ -69,6 +74,13 @@ files:
 - bin/console
 - bin/setup
 - cobro_digital.gemspec
+- docs/behavior/behavior.md
+- docs/config/configuracion.md
+- docs/consumed/cobrodigital.md
+- docs/glossary/glossary.md
+- docs/interface/interface.md
+- docs/test/testing.md
+- docs/topology/topology.md
 - lib/cobro_digital.rb
 - lib/cobro_digital/boleta.rb
 - lib/cobro_digital/meta.rb
@@ -77,10 +89,14 @@ files:
 - lib/cobro_digital/pagador.rb
 - lib/cobro_digital/transaccion.rb
 - lib/cobro_digital/version.rb
+- skill/SKILL.md
+- skills.yml
 homepage: https://github.com/gedera/cobrodigital
 licenses:
 - MIT
-metadata: {}
+metadata:
+  documentation_uri: https://github.com/gedera/cobrodigital/blob/v1.9.0/skill
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -88,14 +104,18 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.7'
+  - - "<"
+    - !ruby/object:Gem::Version
+      version: '3.0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.6
+rubygems_version: 3.1.6
+signing_key:
 specification_version: 4
 summary: Adaptador CobroDigital
 test_files: []

data/.travis.yml

@@ -1,4 +0,0 @@
-language: ruby
-rvm:
-  - 1.8.7
-before_install: gem install bundler -v 1.10.6