exis_ray 0.9.0 → 0.10.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c87f049dd7216e8ac33a9844b77cd1c863f9248130b329994002aa654c0c47ca
-  data.tar.gz: 1752d432a8437178121b36a0b28293f52f96b99d13a5c40589f312fb8f9b6236
+  metadata.gz: 78d2b6cbc5e3da104fee7f3d2f481311acd2da7971d80295edf0bbad6eaab178
+  data.tar.gz: c54ebfc75e0c5abf640506b97b41b4b86bfe1f05a8d17a2d484941dae0fb21d4
 SHA512:
-  metadata.gz: 72f092befb65f8eb03fe851a302ec49ac47fb155a288ed1103923766f0322da6e5704e760d8c6fa7e4865d78666ddc29b7f7c5e486f22120de1c0ff4a072777e
-  data.tar.gz: 57e552a85bc42d9cb659d49a429c85131175350542856378d432063b9558f264a636c93d4a25ee57b51717718c5bcb03a2f86fbff3676ce85fedfde4562f5509
+  metadata.gz: e6642e744b2b9fc0d798e419664d76ebe2a0beba38722b1db09ac501d408938695990d4d752a0b0bf9cad78aa1a62a3939113e1444a623df890cabb8a4f0ad31
+  data.tar.gz: e96e7936add04a739ac1f325d1b768437c82381f5252420f2ab0eb72d6ad51e50bc272c7b136e4f1c0096d59f50b4b9fdc227ec05086e374e385ad543d5d47e8

data/CHANGELOG.md

@@ -1,3 +1,11 @@
+## [0.10.0] - 2026-05-25
+
+### Documentación
+- **`SKILL.md`/`README` describían la auto-inyección como incondicional** (#11): la prosa de "Flujo runtime" en `SKILL.md` y "Flujo de propagación" en `README.md` listaba los campos auto-inyectados (`root_id`, `trace_id`, `source`, `request_id`, ...) sin desglosar los guards. `JsonFormatter#inject_tracer_context` corta su bloque con `return unless Tracer.root_id`, y `request_id` se emite **fuera** de ese guard (distinto ciclo de vida — fix de #9). Sin esa precisión, un implementador en `box_radius_manager` concluyó (apoyándose en la doc) que "`root_id` ya cubre la trazabilidad, `request_id` es redundante" y removió un workaround necesario; en su entrypoint sin trace header `root_id` era nil → logs sin `request_id`/`source`/`root_id`. Agregada subsección "Condiciones de emisión por campo" en `SKILL.md` con tabla "campo → condición → entrypoint que la garantiza", callout en el README arriba de "Campos auto-inyectados", y enlace cruzado a `docs/behavior/behavior.md` + `docs/glossary/glossary.md` (que ya tenían el detalle).
+
+### Correcciones
+- **Clave `task` duplicada en JSON cuando `TaskMonitor.run` loguea lifecycle** (#12): `TaskMonitor` emitía `task=#{task_name}` en los KV strings de `task_started`/`task_finished` (3 sites: `task_monitor.rb:32,41,48`); `JsonFormatter#inject_tracer_context` ya inyecta la misma clave desde `Tracer.task` con Symbol — al colapsar Symbol/String en `JSON.generate`, `json` emitía `warning: detected duplicate key "task"` y la línea entera va a romper en `json` 3.0. **Fix B** (origen): los KV de lifecycle ya no emiten `task=...`; la clave viene exclusivamente del formatter vía `Tracer.task`. **Fix A** (defensa en profundidad): `JsonFormatter` normaliza todas las claves del payload a String antes de `JSON.generate`, deduplicando con precedencia "última inserción gana" (developer payload pisa contexto canónico inyectado por Tracer, igual que `log_fields`). Cubre futuros casos análogos Tracer ↔ developer KV sobre la misma clave.
+
 ## [0.9.0] - 2026-05-19
 
 ### Correcciones

data/README.md

@@ -52,7 +52,7 @@ ExisRay opera en tres capas que se combinan automáticamente:
 **Flujo de propagación:**
 
 1. Un request/job/mensaje llega al servicio. El middleware correspondiente hidrata el `Tracer` con el header entrante. **Todo entrypoint garantiza un `root_id`**: si no llega trace header (servicio que es punto de entrada, no eslabón intermedio), genera uno fresco. En HTTP además captura el `request_id` de Rails.
-2. `JsonFormatter` inyecta automáticamente `root_id`, `trace_id`, `source`, `request_id` y el contexto de negocio (`user_id`, `isp_id`, `correlation_id`) en cada línea de log. Como el entrypoint siempre asegura `root_id`, `source` (campo mandatorio) nunca falta.
+2. `JsonFormatter` inyecta `root_id`, `trace_id`, `source`, `request_id` y el contexto de negocio (`user_id`, `isp_id`, `correlation_id`) en cada línea de log. **Cada campo tiene un guard:** `root_id`/`trace_id`/`source`/`task`/`sidekiq_job` salen solo si `Tracer.root_id` está presente; `request_id` se emite fuera de ese guard (distinto ciclo de vida). Como todo entrypoint asegura `root_id` (genera uno fresco si no llega header), `source` (mandatorio) nunca falta. Detalle por campo: tabla [Campos auto-inyectados](#campos-auto-inyectados) más abajo.
 3. Cuando el servicio llama a otro servicio (HTTP, Sidekiq, RabbitMQ), el middleware de salida genera un nuevo header con `Tracer.generate_trace_header`, que incluye el `root_id` original, el `self_id` del servicio actual, el `CalledFrom` y el tiempo acumulado.
 4. El servicio destino repite desde el paso 1. El `root_id` se mantiene constante a lo largo de toda la cadena.
 
@@ -339,7 +339,9 @@ config.logger.formatter = ExisRay::JsonFormatter
 
 ### Campos auto-inyectados
 
-`JsonFormatter` inyecta estos campos automáticamente en cada línea. **Nunca** los incluyas manualmente en tus logs:
+`JsonFormatter` inyecta estos campos automáticamente en cada línea. **Nunca** los incluyas manualmente en tus logs.
+
+> **No es incondicional.** `root_id`/`trace_id`/`source`/`task`/`sidekiq_job` están gateados por `inject_tracer_context`'s `return unless Tracer.root_id`. Los 4 entrypoints (HTTP, Sidekiq server, BugBunny consumer, TaskMonitor) garantizan el `root_id` (fresco si no llega header), por eso `source` (mandatorio) nunca falta — pero si el código loguea fuera de un entrypoint (boot, código inicializador, hilos sueltos), esos campos pueden faltar. `request_id` se emite fuera de ese guard y tiene distinto ciclo de vida.
 
 | Campo | Condición |
 |:------|:----------|

data/lib/exis_ray/json_formatter.rb

@@ -61,13 +61,29 @@ module ExisRay
       inject_current_tags(payload)
       process_message(payload, msg)
 
-      "#{JSON.generate(payload.compact, { ascii_only: false })}\n"
+      "#{JSON.generate(normalize_keys(payload), { ascii_only: false })}\n"
     rescue StandardError
       fallback_message(severity, timestamp, msg)
     end
 
     private
 
+    # Normaliza todas las claves del payload a strings y deduplica con precedencia
+    # "última inserción gana" (developer payload pisa contexto canónico inyectado
+    # antes, igual que `log_fields`). Previene el warning `duplicate key` que `json`
+    # emite cuando coexisten `:task` (Symbol, vía Tracer) y `"task"` (String, vía
+    # KV string developer) — `json` 3.0 lo va a transformar en error.
+    #
+    # @param payload [Hash]
+    # @return [Hash{String => Object}]
+    def normalize_keys(payload)
+      payload.each_with_object({}) do |(key, value), result|
+        next if value.nil?
+
+        result[key.to_s] = value
+      end
+    end
+
     # @return [String, nil]
     def service_version
       ExisRay.configuration.service_version

data/lib/exis_ray/task_monitor.rb

@@ -29,7 +29,7 @@ module ExisRay
         curr.correlation_id = ExisRay::Tracer.correlation_id
       end
 
-      log_event(:info, "component=exis_ray event=task_started task=#{task_name} outcome=started")
+      log_event(:info, "component=exis_ray event=task_started outcome=started")
 
       # Bloque de ejecución con o sin tags dependiendo de la configuración
       execute_with_optional_tags(&block)
@@ -38,14 +38,14 @@ module ExisRay
       human_time = ExisRay::Tracer.format_duration(duration_s)
 
       log_event(:info,
-                "component=exis_ray event=task_finished task=#{task_name} " \
+                "component=exis_ray event=task_finished " \
                 "outcome=success duration_s=#{duration_s} duration_human=\"#{human_time}\"")
     rescue StandardError => e
       duration_s = ExisRay::Tracer.current_duration_s
       human_time = ExisRay::Tracer.format_duration(duration_s)
 
       log_event(:error,
-                "component=exis_ray event=task_finished task=#{task_name} " \
+                "component=exis_ray event=task_finished " \
                 "outcome=failed duration_s=#{duration_s} duration_human=\"#{human_time}\" " \
                 "error_class=#{e.class} error_message=#{e.message.inspect} " \
                 "exception.type=#{e.class} exception.message=#{e.message.inspect} " \

data/lib/exis_ray/version.rb

@@ -2,5 +2,5 @@
 
 module ExisRay
   # Versión actual de la gema.
-  VERSION = "0.9.0"
+  VERSION = "0.10.0"
 end

data/skill/SKILL.md

@@ -11,7 +11,7 @@ Para el complemento del estándar de logging Wispro (regla Data First, mapeo Ope
 
 ### Artefactos de detalle (RFC-008)
 
-Este SKILL.md **resume e indexa**; el contrato y el significado de detalle viven en `docs/<capa>/`. **Version-lock por construcción:** `gemspec.files` empaqueta `docs/**` en el mismo tag que este `SKILL.md`; los links son rutas relativas dentro del paquete del release (nunca rama/`HEAD`/URL flotante). Contrato resumido anclado a **v0.8.0 + fix issue #9** (commit `00cf803`):
+Este SKILL.md **resume e indexa**; el contrato y el significado de detalle viven en `docs/<capa>/`. **Version-lock por construcción:** `gemspec.files` empaqueta `docs/**` en el mismo tag que este `SKILL.md`; los links son rutas relativas dentro del paquete del release (nunca rama/`HEAD`/URL flotante). Contrato resumido anclado a **v0.10.0** (post fix issues #9, #11, #12):
 
 - [`docs/behavior/behavior.md`](../docs/behavior/behavior.md) — secuencias de hidratación de trace por entrypoint + emisión en logs (parcial, incremental).
 - [`docs/glossary/glossary.md`](../docs/glossary/glossary.md) — lenguaje ubicuo del bounded context (`root_id`, `trace_id`, `source`, `request_id`, `entrypoint`, ...).
@@ -79,7 +79,7 @@ ExisRay unifica trazabilidad distribuida, logging estructurado JSON, contexto de
 2. `Tracer.parse_trace_id` extrae `root_id`, `self_id`, `called_from`, `total_time_so_far`
 3. `ExisRay.sync_correlation_id` asigna `Tracer.correlation_id` a `Current.correlation_id`
 4. Controller ejecuta `before_action` para setear `Current.user_id`, `Current.isp_id`
-5. `JsonFormatter` intercepta cada `Rails.logger.*` e inyecta automaticamente: `time`, `level`, `severity_number`, `service`, `service_version`, `deployment_environment`, `request_id` (fuera del guard de `root_id` — distinto ciclo de vida), `root_id`, `trace_id`, `source`, `user_id`, `isp_id`, `correlation_id`. Como el entrypoint siempre garantiza `root_id`, `source` (mandatorio) nunca falta. El developer aporta `component` (modulo de negocio) y `event` (que paso); estos NO son auto-inyectados porque dependen del call site, no del contexto de ejecucion.
+5. `JsonFormatter` intercepta cada `Rails.logger.*` e inyecta el contexto de ejecucion en cada linea. **No es incondicional:** cada campo tiene un guard especifico (ver tabla "Condiciones de emision" mas abajo). En particular `inject_tracer_context` corta el bloque `root_id`/`trace_id`/`source`/`task`/`sidekiq_job` con `return unless Tracer.root_id` — la invariante "todo entrypoint garantiza `root_id`" es lo que hace que `source` (mandatorio) nunca falte. `request_id` se emite **fuera** de ese guard (distinto ciclo de vida que `root_id`). El developer aporta `component` (modulo de negocio) y `event` (que paso); estos NO son auto-inyectados porque dependen del call site, no del contexto de ejecucion.
 6. `LogSubscriber` emite un unico Hash al finalizar el request con campos default (`component`, `event`, `method`, `path`, `http_route`, `format`, `controller`, `action`, `http_status`, `duration_s`, `duration_human`, `view_runtime_s`, `db_runtime_s`, `user_agent_original`, `server_address`, y en error `error_class`/`error_message`/`exception.*`).
 7. En llamadas salientes, `FaradayMiddleware`/`ActiveResourceInstrumentation` inyectan `propagation_trace_header` con `Tracer.generate_trace_header`
 8. Al finalizar, `ActiveSupport::CurrentAttributes` hace reset automatico
@@ -295,6 +295,26 @@ Casteo automatico: integers, floats, objetos JSON (`{...}`, `[...]`). Filtra cla
 
 Por eso `component` y `event` jamas se auto-inyectan, aunque el estandar Wispro los exija.
 
+#### Condiciones de emision por campo
+
+La auto-inyeccion **no es incondicional**: cada campo tiene un guard. `inject_tracer_context` corta su bloque con `return unless Tracer.root_id`, asi que `root_id`/`trace_id`/`source`/`task`/`sidekiq_job` solo se emiten cuando hay trace context activo. Los 4 entrypoints (HTTP, Sidekiq server, BugBunny consumer, TaskMonitor) garantizan ese `root_id` (fresco si no llega header), por eso `source` (mandatorio) nunca falta en una linea originada por un entrypoint. `request_id` se emite **fuera** del guard de `root_id` — distinto ciclo de vida.
+
+| Campo | Condicion de emision | Entrypoint que la garantiza |
+|:------|:---------------------|:----------------------------|
+| `time`, `level`, `severity_number`, `service`, `service_version`, `deployment_environment` | Siempre (no depende de Tracer/Current) | — |
+| `request_id` | `Tracer.request_id` presente. **Fuera del guard de `root_id`** (issue #9 Gap C): distinto ciclo de vida (UUID v4 de Rails vs formato X-Ray). | HTTP (via `ActionDispatch::RequestId`). Otros entrypoints solo si la app lo setea explicitamente. |
+| `root_id` | `Tracer.root_id` presente. **Gatea todo el bloque de tracer context.** | Los 4 entrypoints garantizan `root_id` fresco si no llega trace header (issue #9 Gap A). |
+| `source` | `Tracer.source` presente **y** `root_id` presente (esta dentro del bloque gateado). | Idem `root_id`. Como `source` es mandatorio del estandar Wispro, la invariante "todo entrypoint garantiza `root_id`" es lo que evita que falte. |
+| `trace_id` | `Tracer.trace_id` presente **y** `root_id` presente. Solo cuando el servicio es **eslabon intermedio** (recibio trace header upstream). | — (entrypoint que no recibe header tiene `root_id` fresco pero `trace_id` nil). |
+| `sidekiq_job` | `Tracer.sidekiq_job` presente **y** `root_id` presente. | Sidekiq `ServerMiddleware`. |
+| `task` | `Tracer.task` presente **y** `root_id` presente. | `TaskMonitor.run`. |
+| `correlation_id` | `Current.correlation_id` presente. | `ExisRay.sync_correlation_id` (HTTP middleware lo llama; otros entrypoints solo si la app lo invoca). |
+| `user_id`, `isp_id` | `Current.<attr>` no nil. | Lo setea la app (login, before_actions, etc.). |
+| `Current.log_fields` (cualquier key) | La subclass de `Current` overrideo el hook y retorno un Hash no vacio. | — |
+| `tags` | Rails tagged logging activo. **Antipatron con JSON** (rompe el formato) — ver FAQ. | — |
+
+Para el detalle por entrypoint (que setea que y cuando), ver [`docs/behavior/behavior.md`](../docs/behavior/behavior.md). Para el significado de cada campo, [`docs/glossary/glossary.md`](../docs/glossary/glossary.md).
+
 #### Ejemplos: KV vs Hash producen output equivalente
 
 ```ruby

data/skills.yml

@@ -2,48 +2,34 @@ mcps:
   - github
   - clickup
 skills:
-  skill-manager:
-    repo: sequre/ai_knowledge
-    scope: local
   yard:
     repo: sequre/ai_knowledge
-    scope: local
   quality-code:
     repo: sequre/ai_knowledge
-    scope: local
   gem-release:
     repo: sequre/ai_knowledge
-    scope: local
-  skill-builder:
+  dev-structure:
     repo: sequre/ai_knowledge
-    scope: local
-  ai-reports:
+  dev-compose:
     repo: sequre/ai_knowledge
-    scope: local
-    environment:
-      space_id: "${AI_REPORTS_SPACE_ID}"
-      bug_reports_list_id: "${AI_REPORTS_BUG_REPORTS_LIST_ID}"
-      improvements_list_id: "${AI_REPORTS_IMPROVEMENTS_LIST_ID}"
-  agent-review:
+  dev-enrich:
     repo: sequre/ai_knowledge
-    scope: local
-    environment:
-      space_id: "${AGENT_REVIEW_SAPCE_ID}"
-      list_id: "${AGENT_LIST_ID}"
-  action-plan:
+  skill-feedback:
+    repo: sequre/ai_knowledge
+  agent-issue:
+    repo: sequre/ai_knowledge
+  dev-flow:
+    repo: sequre/ai_knowledge
+  matrix-element:
     repo: sequre/ai_knowledge
     environment:
-      space_id: "${AGENT_REVIEW_SAPCE_ID}"
-      list_id: "${AGENT_ACTION_PLAN_LIST_ID}"
+      homeserver: "https://matrix.cloud.wispro.co"
+      auth_token: "${MATRIX_AUTH_TOKEN}"
+      rooms:
+        agents: "!VCHwQXgmXdyhhhPhoz:matrix.cloud.wispro.co"
   documentation-writer:
     repo: github/awesome-copilot
     path: skills/documentation-writer
-    scope: local
-  find-skills:
-    repo: vercel-labs/skills
-    path: skills/find-skills
-    scope: local
   opentelemetry:
     repo: bobmatnyc/claude-mpm-skills
     path: universal/observability/opentelemetry
-    scope: local

data/spec/exis_ray/json_formatter_spec.rb

@@ -463,6 +463,72 @@ RSpec.describe ExisRay::JsonFormatter do
     end
   end
 
+  describe "dedup de claves Symbol/String (issue #12)" do
+    def stub_tracer_with_task(task_value)
+      stub_const("ExisRay::Tracer", Module.new do
+        define_singleton_method(:service_name) { "test-service" }
+        define_singleton_method(:root_id) { "1-abc-def" }
+        define_singleton_method(:trace_id) { nil }
+        define_singleton_method(:request_id) { nil }
+        define_singleton_method(:source) { "task" }
+        define_singleton_method(:sidekiq_job) { nil }
+        define_singleton_method(:task) { task_value }
+      end)
+    end
+
+    it "no emite warning de `duplicate key` cuando KV string y Tracer setean la misma key" do
+      stub_tracer_with_task("billing:invoices")
+
+      warning = nil
+      original_warn = Warning.method(:warn)
+      Warning.singleton_class.define_method(:warn) { |msg, **| warning = msg }
+
+      begin
+        call("event=task_started task=billing:invoices outcome=started")
+      ensure
+        Warning.singleton_class.define_method(:warn, original_warn)
+      end
+
+      expect(warning).to be_nil
+    end
+
+    it "developer payload (KV string) pisa al contexto canónico inyectado por Tracer" do
+      stub_tracer_with_task("billing:invoices")
+
+      result = call("event=foo task=override outcome=success")
+
+      expect(result["task"]).to eq("override")
+    end
+
+    it "emite la key como String aunque internamente fuese Symbol" do
+      stub_tracer_with_task("billing:invoices")
+
+      json_str = formatter.call(severity, timestamp, progname, "event=task_started outcome=started")
+
+      # Si quedaran ambas (`:task` y `"task"`) el JSON tendría dos veces la key.
+      expect(json_str.scan(/"task"\s*:/).size).to eq(1)
+      expect(JSON.parse(json_str)["task"]).to eq("billing:invoices")
+    end
+
+    it "no incluye keys con valor nil en el output" do
+      stub_const("ExisRay::Tracer", Module.new do
+        define_singleton_method(:service_name) { "test-service" }
+        define_singleton_method(:root_id) { nil }
+        define_singleton_method(:trace_id) { nil }
+        define_singleton_method(:request_id) { nil }
+        define_singleton_method(:source) { nil }
+        define_singleton_method(:sidekiq_job) { nil }
+        define_singleton_method(:task) { nil }
+      end)
+
+      result = call("event=foo")
+
+      expect(result).not_to have_key("root_id")
+      expect(result).not_to have_key("task")
+      expect(result).not_to have_key("source")
+    end
+  end
+
   describe "#filter_sensitive_hash (privado)" do
     it "filtra claves sensibles en el nivel raíz" do
       result = formatter.send(:filter_sensitive_hash, { user: "gabriel", password: "secret" })

data/spec/exis_ray/task_monitor_spec.rb

@@ -49,7 +49,22 @@ RSpec.describe ExisRay::TaskMonitor do
         expect(start_log[0]).to eq(:info)
         expect(start_log[1]).to include("outcome=started")
         expect(start_log[1]).to include("event=task_started")
-        expect(start_log[1]).to include("task=billing:invoices")
+      end
+
+      it "no emite `task=` en el KV (la clave viene del formatter vía Tracer.task)" do
+        captured_task = nil
+        described_class.run("billing:invoices") { captured_task = ExisRay::Tracer.task }
+
+        # task_name vive en el Tracer durante la ejecución del bloque
+        expect(captured_task).to eq("billing:invoices")
+
+        # Los KV de lifecycle NO incluyen `task=...` — evita la duplicación con
+        # `JsonFormatter#inject_tracer_context` (issue #12).
+        lifecycle = captured_logs.map { |_, msg| msg }.select { |m| m.include?("event=task_") }
+        expect(lifecycle).not_to be_empty
+        lifecycle.each do |msg|
+          expect(msg).not_to match(/\btask=/)
+        end
       end
 
       it "loguea task_finished con outcome=success (breaking rename v0.6.0)" do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: exis_ray
 version: !ruby/object:Gem::Version
-  version: 0.9.0
+  version: 0.10.0
 platform: ruby
 authors:
 - Gabriel Edera
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-05-19 00:00:00.000000000 Z
+date: 2026-05-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -90,8 +90,8 @@ licenses:
 metadata:
   homepage_uri: https://github.com/gedera/exis_ray
   source_code_uri: https://github.com/gedera/exis_ray
-  changelog_uri: https://github.com/gedera/exis_ray/blob/v0.9.0/CHANGELOG.md
-  documentation_uri: https://github.com/gedera/exis_ray/blob/v0.9.0/skill
+  changelog_uri: https://github.com/gedera/exis_ray/blob/v0.10.0/CHANGELOG.md
+  documentation_uri: https://github.com/gedera/exis_ray/blob/v0.10.0/skill
 post_install_message:
 rdoc_options: []
 require_paths: