Split 1 of 7: canonical timeline foundations for the data platform - #1792
Draft
martien-wdy wants to merge 1 commit into
Draft
Split 1 of 7: canonical timeline foundations for the data platform#1792martien-wdy wants to merge 1 commit into
martien-wdy wants to merge 1 commit into
Conversation
Adds Roughtime consensus and CLOCK_BOOTTIME sampling so later chunks can stamp captured data on a single canonical timeline with an explicit uncertainty bound rather than on the wall clock. Part 1 of 7 in the split of the Wendy Data Platform change.
Contributor
AI Security ReviewNote Automated security review from Claude. Apply, adapt, silence with Input coverage: 7/7 changed files; 10,652/10,652 bytes reviewed; diff SHA-256 No security findings. |
Joannis
approved these changes
Aug 25, 2026
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
2 participants
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.
Problem
Pull request #1751 (the Wendy Data Platform) is +24032/-47 across 91 files. It is too large to review, and the repository's Claude Security Review job cannot run on it at all:
ghrefuses any diff over 20000 lines with HTTP 406. Merging it as one piece would put a new entitlement and a new socket on main with no automated security review.Cause
The change was developed as a single branch covering the whole feature: canonical time, an episode data model, protocol definitions, an entitlement, capture, upload and tooling. Nothing forced it apart as it grew.
Solution
Split it into seven stacked pull requests, each independently buildable and each small enough to be security reviewed. This is chunk 1 of 7. It is history surgery only. No behaviour was changed while splitting.
This chunk adds the canonical timeline foundations everything else stamps data against:
CLOCK_BOOTTIMEsampling, with a portable fallback for non Linux development builds.Deliberately left to later chunks
The episode data model that consumes this timeline (chunk 2), and every consumer of it. Nothing imports the new consensus entry point yet, which is why this chunk is small.
Depends on
Nothing. This chunk is based on
main.Verification
CC=/usr/bin/clang go build ./...succeeds.CC=/usr/bin/clang go test ./go/internal/agent/timesync/... ./go/internal/shared/roughtime/...passes.gofmt -l go/is empty andgo vetis clean for both packages.The full stack, in merge order: 1-timeline, 2-episode-model, 3-protos, 4-sensors-entitlement, 5-capture, 6-upload, 7-tools-and-example. The union of all seven is byte for byte equal to
feature/wendy-data-platform.