Agent trace export - #198
Open
davidabram wants to merge 5 commits into
Open
Conversation
Establish the read/export module boundary with batch-size, cursor, limit, and JavaScript-safe-integer validation before reader methods are added. Register the additive service module and cover the validation boundaries with unit tests.\n\nPlan: agent-trace-export-readers (T01) Co-authored-by: SCE <sce@crocoder.dev>
Define owned camelCase export rows for messages, parts, diff traces, and agent traces, including lowercase message roles and nullable metadata fields. Add serialization contract tests so the reader output shapes remain compatible with their wire format.\n\nPlan: agent-trace-export-readers, T02 Co-authored-by: SCE <sce@crocoder.dev>
The export service lacked read-only, cursor-based access to persisted messages and parts, preventing bounded incremental delivery while preserving database ownership and JavaScript-safe identifiers.\n\nAdd repository-scoped readers that validate cursors and limits, query rows in ascending ID order, decode message roles, enforce safe-integer bounds, and materialize owned batches. Add coverage for gaps, pagination, empty tails, invalid inputs, unsafe timestamps, and no mutation.\n\nPlan: agent-trace-export-readers, T03 Co-authored-by: SCE <sce@crocoder.dev>
The export reader was missing access to the diff_traces and agent_traces capture streams needed for complete incremental export. Add bounded cursor queries, typed row materialization, safe-integer validation, and raw payload passthrough for both streams, with coverage for pagination, nullability, validation, and read-only behavior.\n\nPlan: agent-trace-export-readers, task T04 Co-authored-by: SCE <sce@crocoder.dev>
Prove export readers compose with repository-scoped storage while retaining repository and source-instance metadata. Document the read-only incremental boundary, cursor semantics, stream queries, and excluded sync concerns for future control-plane integration.\n\nPlan: agent-trace-export-readers (T05) Co-authored-by: SCE <sce@crocoder.dev>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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.
No description provided.