Repository navigation
Add operation logging and release Runtime 1.4.0 - #23
Merged
Merged
Conversation
There was a problem hiding this comment.
Autter PR review for #23 terminated unexpectedly (runner stopped before review completed (SIGTERM)). Push a new commit to re-run the review.
Autter review failedAutter PR review for #23 terminated unexpectedly (runner stopped before review completed (SIGTERM)). Push a new commit to re-run the review. The merge gate is blocked until this review finishes successfully. Retry
Autter will post a new comment when the retry starts. |
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
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.
Customer operations can finish without throwing even when the intended result failed. Runtime 1.4.0 adds structured diagnostic logs and explicit operation outcomes, so a checkout can report reserved inventory, an unconfirmed payment, and skipped order creation with linked trace context.
The Node and Next.js server APIs record nested context, measured steps and one completed operation summary. The ingester accepts authenticated OTLP JSON/protobuf at
/v1/logsand creates the separateruntime_logstable through migration0011-runtime-logs. Ordinary logs remain diagnostic evidence; exceptions and failed outcomes use the existing trace issue pipeline. Buffering, retries, privacy scrubbing and flush failures are bounded and reported.Validation: Runtime builds passed; 40 Node tests, 8 browser tests and 67 ingester tests passed; browser size remains within its 5 kB budget. Isolated ClickHouse 24.8 validation covered schema migration, SDK ingestion and scoped linked evidence.
Rollout: deploy the ingester before publishing the 1.4.0 SDKs, then deploy platform evidence consumers. The browser package remains 1.3.3; Next.js requires the new Node package.
Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.