Skip to content

fix(telemetry,workers): canonical netscript.* correlation/outcome floor on Flow-B product spans#626

Merged
rickylabs merged 1 commit into
mainfrom
fix/599-flowb-attribute-floor
Jul 11, 2026
Merged

fix(telemetry,workers): canonical netscript.* correlation/outcome floor on Flow-B product spans#626
rickylabs merged 1 commit into
mainfrom
fix/599-flowb-attribute-floor

Conversation

@rickylabs

Copy link
Copy Markdown
Owner

Closes #599

Summary

Implements the four #599 acceptance boxes (product gaps found by the T8 Flow-B merge gate, PR #598):

  1. traceJobExecution() now emits canonical netscript.outcome at running/completed/failed transitions (legacy job.status retained).
  2. Trigger ingress/detect/process spans emit netscript.correlation.id (event id) + netscript.outcome; queue enqueue/dequeue spans emit the pair from the message envelope's correlationId (TC-6/TC-7 floor).
  3. processWorkerJob()executeWorkerJob()executeDenoJob() thread correlationId into the rebuilt in-process JobMessage, so the callback handler inherits the product span's correlation value.
  4. T8 gate tightened, not forked: validate-flow-b-traces.ts correlatedSpans now covers ingress/process/enqueue/dequeue/execute/callback-boundary/fan-in, with the canonical outcome asserted on the full set and single-correlation-id equality across all seven.

No new public API; values added to existing attribute maps only. No deno.lock churn.

Lane + evidence

WSL Codex slice (thread 019f5117-bcd2…, worktree ns-wt-599) under the beta-7 orchestrator (df71d36c); worklog committed at .llm/runs/fix-599-flowb-attribute-floor--codex/worklog.md. Scoped check/lint 335 files 0 findings; telemetry 50/0, workers 24/0 tests. Standalone behavior.otel.traces cannot run without a live prepared runtime (worklog drift D2) — this PR's CI scaffold-runtime job is the live T8 verdict.

@rickylabs rickylabs added this to the 0.0.1-beta.7 milestone Jul 11, 2026
@rickylabs rickylabs added type:fix status:impl-eval area:plugins plugins/* and plugin-core packages wave:v1 gate:e2e area:telemetry priority:p2 Medium epic:telemetry-revamp Telemetry revamp epic (Spine-1 enabler) labels Jul 11, 2026
@rickylabs
rickylabs marked this pull request as ready for review July 11, 2026 12:38
@rickylabs
rickylabs merged commit f580d9f into main Jul 11, 2026
17 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

telemetry: canonical netscript.* correlation/outcome floor gaps on Flow-B product spans (found by T8 gate)

1 participant