Skip to content

Standardize eval policy, budgets, session metadata, traces, and cost accounting - #2902

Open
miguelg719 wants to merge 126 commits into
evals/consolidation-04-cdp-diagnosticsfrom
evals/consolidation-12-shared-runner-contract
Open

miguelg719 wants to merge 126 commits into
evals/consolidation-04-cdp-diagnosticsfrom
evals/consolidation-12-shared-runner-contract

Conversation

@miguelg719

@miguelg719 miguelg719 commented Sep 8, 2026

Copy link
Copy Markdown
Collaborator

Defines one eval-owned autonomy policy and a shared result contract for external harnesses. Records policy channel/version, effective budget and unit, requested configuration, browser identity, execution reason, normalized usage and trace/timing metadata.

Unknown usage and subscription costs remain unavailable. Astra estimates also remain unavailable until per-request context-tier accounting exists. Other supported direct-provider estimates include the dated catalog source. Generic task JSON is preserved in results and verifier trajectories. Failed facade calls remain failures regardless of tool-output text; runner-owned telemetry determines run-level session loss. PR13 wires browser identity and budget helpers into existing adapters.

Validation: isolated shared-runner changes passed all 896 eval tests and eval typecheck. The complete repaired stack passed 30 build/typecheck tasks and 29 unit-test tasks, including 999 eval tests. Fresh CI passed on this PR’s current head.

Stack base: evals/consolidation-04-cdp-diagnostics. Review this PR relative to its immediate predecessor.

Reviewer entry points:

Readiness: CI passed on 61aa53cfdb. Human review status is unchanged.

Current open stack: #2893#2894#2902#2903#2889#2904#2905#2906#2907.

…nsolidation-03-capture-recovery

# Conflicts:
#	packages/integrations/core/src/facade/tools.ts
#	packages/integrations/core/tests/facade-tools.test.ts
@miguelg719
miguelg719 added this pull request to stack #2956 September 17, 2026 07:14
@miguelg719
miguelg719 removed this pull request from stack #2956 September 17, 2026 07:15
@miguelg719
miguelg719 changed the base branch from evals/consolidation-11-verifier-results to main September 17, 2026 07:23
@miguelg719
miguelg719 changed the base branch from main to evals/consolidation-11-verifier-results September 17, 2026 07:24
@miguelg719
miguelg719 force-pushed the evals/consolidation-11-verifier-results branch from e9cf972 to e0f2b1e Compare September 17, 2026 08:09
@miguelg719
miguelg719 changed the base branch from evals/consolidation-11-verifier-results to main September 21, 2026 21:02
@miguelg719
miguelg719 requested a review from a team as a code owner September 21, 2026 21:02
# Conflicts:
#	packages/integrations/core/tests/facade-tools.test.ts

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant