docs(positioning): observability story#649
Merged
Merged
Conversation
…ery read side (#445) Reframe the observability pillar around the build-efficiency law: the verification turn ("did the change work across every process?") is the expensive turn, and NetScript closes it inside the framework. telemetry.md: - elevator-pitch intro (verification-turn / agent self-verify framing) - "one grouped trace" story spine (Flow-B: trigger → scheduler → queue → job.execute → SDK rpc.client callback; createFanInLinks span links; gate-enforced; gen_ai.* for AI turns) - telemetry ports/adapters + #402 attribute convention mechanism paragraph - service/RPC/HTTP tracing: registerORPCInstrumentation + first-party createHonoTracingMiddleware - new "Close the loop" section: typed TelemetryQueryPort read side + the single factual Encore comparison (MCP trace access vs in-language port) index.md: pitch-led intro, "read a trace back in code" next-step, convention link. All present-tense API claims trace to deno doc (query.ts/orpc.ts) + canonical README (PR #610); published beta.7 surface only. deno task verify green (build + 23019 links resolve + 27 caveats resolve). Refs #445 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01DiTbVL2p3zLYc9AxxpK8iG
5 tasks
rickylabs
marked this pull request as ready for review
July 11, 2026 13:36
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #445
Workstream D6 of epic #401: pitch + "one grouped trace" Flow-B story spine + ports/adapters and #402 convention mechanism on
observability/telemetry.md, plus a new "Close the loop: read the trace back in code" section (typedTelemetryQueryPortread side:createAspireTelemetryQuery/createTelemetryQuery, query surface, read models, Standard Schema validators) with the single factual comparison (Encore MCP-server trace access vs in-language port);observability/index.md— verification-turn pitch + convention link. Claims re-verified viadeno doc+ the canonical telemetry README (PR #610); the correlation-id mention stays within published-beta.7 naming-convention language (the #626 floor is unreleased).Validation:
deno task verifygreen (23,019/23,019 links; 27/27 caveats); positioning-law grep clean (Jaeger/Grafana/Honeycomb are neutral OTLP-backend references). Worklog on-branch at.llm/runs/docs-445-positioning-observability--docs/worklog.md.Lane note: started on Fable, spend-limit terminated mid-edit; completed by an Opus 4.8 resume agent that reviewed-not-trusted the partial edits (run eval D6). Validation verdict remains #450. Docs-only diff →
ci:skip-e2e+ci:skip-scaffold.