Repository navigation
feat: explicit agent attribution and recorded spend changes - #85
Merged
Merged
Conversation
Compose bounded exact agent accounting with existing request accounting. Add shared opt-in request-tag transport, project-scoped Langfuse attribution, distinct-suffix coverage, sensitive-label acknowledgement, and frozen-source fixture provenance. Preserve default reports, projections, collector summaries and base accounting. Code authors: Vijay and Codex Signed-off-by: Vijay <evijay@gmail.com>
Use both disclosure flags in the routing smoke command and exercise agent ID, tag partitions and selected label disclosure with and without networking. Code authors: Vijay and Codex. Signed-off-by: Vijay <evijay@gmail.com>
Keep all assertions and cases. Allow fifteen minutes for the hosted SQL steps including pinned image pulls; retain existing job-wide limits. Clarify that the new tag transport and two-flag disclosure target the next minor, not the released v0.13.1 binary. No trigger or publication condition changes. Code authors: Vijay and Codex. Signed-off-by: Vijay <evijay@gmail.com>
Restore ci.yml exactly to the published manifest. Main CI already passed its existing SQL limit; only packaged verification retains the bounded fifteen-minute allowance. Leave all calibration evidence and assertions unchanged. Code authors: Vijay and Codex. Signed-off-by: Vijay <evijay@gmail.com>
Signed-off-by: Vijay <evijay@gmail.com>
Signed-off-by: Vijay <evijay@gmail.com>
Signed-off-by: Vijay <evijay@gmail.com>
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.
Summary
Verification
Local verification uses nice 10, GOMAXPROCS=1 and serial package builds/tests. No local Docker services or external model calls.
Review And Release
CI follow-ups update the restricted-network smoke invocation for the second
disclosure flag and add three agent cases. The packaged SQL verification step
has a bounded 15-minute allowance including pinned image pulls: the full suite
already completed with 67 passes and seven optional skips, but exceeded the old
ten-minute step budget. No cases or assertions were removed. The published
calibration-bound ci.yml remains byte-identical, and job-wide limits plus all
publication triggers and conditions are unchanged.
Draft for Claude review before merge. Merge only after that review and green final-head checks. No release, tag or installer-pin update in this PR. The new feature documentation is explicitly unreleased.