Repository navigation
Conversation
Signed-off-by: Srinivasa Dinakar Jainti <dinakarjs@gmail.com>
Contributor
Author
|
@imran-siddique Please independently review the two bounded delivery-test contributions, each on its own scope:
All returned technical workflows for both heads succeed. Existing formal review requests remain; this single comment avoids separate duplicate reminders. Please identify remaining changes or disposition each bounded PR. Prior test results are not a fresh local rerun, independent approval or production-delivery certification. Prepared with OpenAI Codex. |
This branch has not been deployed
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.
Ready for independent review — 31 October 2026
Readiness assessed at
57e75a9aeacbdb68328b4a9297afbb9141aaf0d0: current metadata is mergeable; all returned PR-triggered technical workflows succeeded, and the public discussion contains no explicit unresolved draft hold or code-change request. Changed-file scope and current CONTRIBUTING requirements were checked. This is a readiness assessment, not a new full source audit, fresh suite run, independent approval or complete branch-protection inventory.Synthetic delivery/lost-ack regression coverage only; no durable recovery, exactly-once delivery or Collector interoperability claim. The real Collector experiment remains separate in #100.
Independent maintainer review and normal merge gates remain required; no merge or release is authorized by this transition. Earlier test counts apply to their stated revisions; obsolete baseline CI failures below are superseded by the successful current-head runs. Prepared with OpenAI Codex.
Problem
Operational telemetry delivery and accepted evidence have different guarantees. Regression tests should preserve that boundary when projections disappear, duplicate, reorder, or fail, and when a persistence callback commits but loses its acknowledgement.
Changes
Validation
node --import tsx --test test/*.test.ts; npm pack --dry-run passed.Limits
Synthetic emitters and an in-memory store do not prove durable storage, crash recovery, OTLP wire behavior or collector interoperability. Python's real SDK exporter test is not a TypeScript exporter parity claim. The original local test environment did not supply Docker or a Collector; this PR's evidence remains synthetic. The documentation records the remaining version-pinned receiver/backend acceptance criteria.
Ready for independent maintainer review; this PR is not a draft.