mirror of
https://github.com/abhigyanpatwari/GitNexus.git
synced 2026-10-01 02:01:24 +00:00
* feat(eval): a scriptable stand-in for Anthropic and OpenAI Every defect this harness shipped last round was invisible to its own tests for one reason: the tests exercised a layer BELOW where the code runs. The usage log was never written because the proxy is a subprocess with a constructed environment. The callback could not be imported because LiteLLM loads it by path, not as a package. Failures went unrecorded because only the async hook was overridden. CI or review caught all three; no unit test could, because each called the function directly instead of driving the path that calls it. This closes that gap without spending money. It speaks the two wire protocols the harness actually depends on - Anthropic Messages, streaming and not, and OpenAI Responses - so a run can go through the real sandbox, the real CLI, the real gateway and the real usage callback with only the model faked. The runner already supports pointing at it: --base-url is the same path the free-model proxy documentation uses. Scripted rather than simulated. A test decides what the model says, which tools it asks for, and exactly what usage it reports. That last part is what makes provider-native accounting testable at all: real cache hits are not reproducible on demand, but a declared cache_read of 44,000 is. One Reply served down both protocols is also the cleanest demonstration that the same billed work is stated as a sum on one side and as a whole on the other. Tool blocks are the mechanism for artifact-producing cells. The CLI runs what it is asked to run, so a scripted Write block makes it write that file inside the sandbox for real - no model deciding anything. The end-to-end test drives the real proxy against the mock and asserts the usage log records the provider's own arithmetic through the Anthropic-shaped translation. It SKIPS here, because litellm's console script is absent in this environment, so it is unverified until CI runs it - the same footing the bubblewrap canary started on, and that one found a real bug on its first CI run. Not yet built: driving a whole sweep against this. That needs a scripted reply sequence that carries a cell to a scored artifact, which is the next step and the point of the exercise. 668 eval tests pass, 17 skipped; the two test_model_gateway.py failures are the pre-existing environmental ones. * test(eval): run a real session against the scripted provider The mock only proves something once the harness runs against it. This adds the stand-in CLI and the first integration tests that use it, so a session goes through the real code with only the model faked. tests/fixtures/fake_claude.py does what the CLI does at the two boundaries the harness depends on: it calls ANTHROPIC_BASE_URL for a turn, EXECUTES the tool blocks that come back, and prints the stream-json sequence the parent parses. Everything between - the session runner, the event-stream parse, the usage extraction, the artifact capture, the scorer - stays real. Four tests, chosen for the layers that have actually broken here: the usage a provider reported survives to the row, a scripted Write produces an artifact parse_review_output accepts, the prompt the harness meant to send is what arrived, and an upstream 529 lands as a failed session rather than a usable measurement. Writing the stand-in found two things worth keeping. The prompt arrives on STDIN under "-p --input-format text"; scanning argv for a non-flag token picks up a flag's value instead, and the prompt-fidelity test is what caught it. And three of these tests had been holding a sandbox they never applied, since no command_prefix is passed - that implied coverage which was not there, so the sandbox is gone from them and stays only in the artifact test, which needs its review directory. What these do NOT cover, checked rather than assumed: making the stand-in write in place instead of atomically still passes. On the host-unsafe backend there is no read-only mount to refuse it, so the atomic-write requirement remains a bubblewrap mount property that only the real-sandbox canary can prove. Dropping cache_read from the recorded usage does fail, so that half is genuinely pinned. 672 eval tests pass, 17 skipped; the two test_model_gateway.py failures are the environmental ones. * fix(eval): the usage adapter read a shape the callback never receives Running the gateway against the scripted provider proved the accounting merged in #3220 does not work, and the same run showed why nothing had caught it. LiteLLM does not hand a logger the upstream body. It normalises usage into its own Chat-Completions-shaped object first, so an OpenAI Responses reply reaches the callback as prompt_tokens / prompt_tokens_details.cached_tokens - never the input_tokens / input_tokens_details the shipped adapter reads. Every field came back unknown. The observed call_type is "anthropic_messages" as well, because Claude Code calls the Anthropic-shaped endpoint, so canonical_provider returned None and normalize_usage would have refused outright. Both were assumptions about a boundary I had only read about. The unit tests agreed with them because their fixture was written in the same wrong shape, so producer and consumer were consistent and both wrong - the exact failure the producer/consumer round trip exists to catch, one layer further out. Adds a LITELLM_NORMALIZED adapter for the object that actually arrives. The arithmetic is still OpenAI's - prompt_tokens is the whole, the details are subsets - so ordinary input is recovered by subtraction. The Responses adapter stays for a raw upstream body, which the mock still serves and tests directly. An unrecognised provider is still refused rather than guessed. The fixtures now carry the measured shape, and the end-to-end test asserts it through a real proxy: 48k prompt tokens with 44k cached is read back as 3k ordinary rather than as silence. 676 eval tests pass, 16 skipped, none failing. * test(eval): run a whole sweep offline, with negative controls The layers between a model turn and a promotion decision had never been exercised together. Unit tests covered each alone, and the paid runs that would have covered the composition kept dying, so the contracts BETWEEN them went unverified - which is where this harness has repeatedly shipped bugs. Drives runner.main() the way the workflow does. Real task selection, hidden oracle capture, sandbox, CLI subprocess, artifact capture, scoring against the oracle, aggregation, health guard and promotion gate. Only the model is scripted. Getting to green meant satisfying nine real contracts nothing had exercised end to end, and each failure was the harness correctly refusing bad evidence: --unsafe-no-bwrap is restricted to the paired review arms; ce_* needs a plugin carrying ce-plan, ce-work and ce-code-review; candidate_* needs an overlay; the clone needs .gitnexus/meta.json with indexedAt and lastCommit; the evidence gate needs a Skill request with a non-error result; review findings need exactly ten fields with severity in critical/high/medium/low; and the hidden labels use a DIFFERENT schema from the review output - line_start/line_end, six fields. That last one only a real run surfaces. Three negative controls, because a scorer that cannot be wrong measures nothing. A finding in the wrong place is tp=0 fp=1 fn=1 and oracle-failed, while its evidence stays VALID - being wrong is a quality result, not a broken measurement. Approving defective code is a miss with no false positive, and precision is None rather than 0, because it is undefined with no predictions. One run cannot promote: the gate says it needs three valid paired runs. A fourth control exists because a mutation demanded it. Forcing skill_was_invoked_events to return True left every other test here passing, so nothing pinned the gate that separates measuring a SKILL from measuring a model. Writing it turned up behaviour worth recording rather than assuming: a skill-not-invoked row still carries its score AND still counts toward the arm median, because aggregate() drops EXCLUDED_ERROR_KINDS and evidence_valid=False and skill-not-invoked is neither. The health guard stops the sweep, so a single-run sweep cannot promote on it, but a mixed run's median would include a cell whose skill never ran. Pinned as-is so it cannot change silently in either direction; changing it is a promotion-semantics decision, not a test fix. Two provisioning steps are stubbed and neither is harness logic: the pinned runtime mounts (no node_modules in a worktree) and the sanitized graph build (needs the gitnexus CLI at a mounted path). Containment is host-unsafe here; bubblewrap stays with the real-sandbox canary. 681 eval tests pass, 16 skipped, none failing. Runs in ~18s. * fix(eval): an uninvoked skill must not move the arm's quality median Found by the offline sweep: a skill-not-invoked row still carried its score into the arm's quality median. aggregate()'s filter dropped EXCLUDED_ERROR_KINDS and evidence_valid=False, and skill-not-invoked is neither, so an arm could be credited for a review it never performed with the skill under test - which is the one thing an arm exists to measure. Excluded from the QUALITY metrics only. Cost and duration still count that row, because the session really ran and really was billed, and the promotion gate still sees it, because it has its own vocabulary for a candidate that never loaded its skill. Two wider fixes were tried and abandoned, both because the tests said so rather than because I reasoned it out first. Reusing the health guard's evidence_failed predicate also excluded transcript-missing rows, but test_aggregate_excludes_session_error_rows_from_medians pins those as counting: that session ran, only its transcript is unverifiable. Excluding the row from `valid` outright turned a candidate whose skill never loaded from keep_incumbent into insufficient_evidence - the safety property held either way, but the decision vocabulary is promotion semantics and not mine to change on a measurement fix. Mutation-checked: putting the rows back into the quality median fails the new test. Both directions asserted, since a filter that excludes everything would also pass - a wrong-but-valid review still moves quality, because being wrong is exactly what a quality median should reflect. 682 eval tests pass, 16 skipped. * test(eval): run the offline sweep unstubbed in the job that can, and probe CLI identity Items 5 and 6 turned out to be one change. The containment (ubuntu) job already installs bubblewrap, the pinned Claude CLI, node_modules and a built GitNexus - everything the sweep's two provisioning stubs stand in for. So the stubs are not a property of the test, only of a machine that lacks those things. GITNEXUS_REQUIRE_FULL_SWEEP=1 makes the sweep run with nothing stubbed: real containment instead of --unsafe-no-bwrap, the real runtime mounts, the real sanitized graph. Set in that job, following the GITNEXUS_REQUIRE_BWRAP_CANARY pattern already there. The gate FAILS on a missing piece rather than degrading to the stubbed path, which is the point - a green tick that silently tested less is what the bubblewrap canary was written to prevent. Verified both states here: default green, and gate-on fails on this machine rather than skipping, since it cannot create user namespaces. Item 7 is an experiment, not an answer. Per-cell attribution needs an identifier that travels WITH the request, because one proxy serves the whole sweep and anything read from its environment is identical for every call. What the real CLI sends is not documented anywhere I can check, and guessing a wire format is exactly how the last three accounting bugs happened. So the probe drives the REAL pinned CLI against the mock and records the identity-bearing headers and body keys that arrive. It asserts only that a request was made; the recorded evidence is the deliverable, and the job log preserves it. Skips without CLAUDE_CANARY_BIN. Two guards caught this rather than review: the repo pins the containment job's env and its exact test list, so both had to be updated deliberately - which is the guard working, not friction. 682 eval tests pass, 17 skipped. * test(eval): make the offline sweep cross-task, so a scheduler change is checkable The sweep fixture had one task, and a single task cannot show the thing a cross-task scheduler changes: waves are per-task, so ordering, packing and a breaker spanning a task boundary are all invisible with one. A second task with its defect in a DIFFERENT file, and its own hidden labels, makes per-task routing observable. The scripted reply is now task-aware, which matters for the same reason: replying with the first task's finding scores the second task wrong. The load-bearing assertion is that each task scored against ITS OWN oracle. That is the dangerous failure mode of interleaving cells from different tasks - a mis-routed context or artifact scores one task against another's labels, and every row still looks green. Mutation-checked: pointing every cell at the first task's oracle snapshot fails it. This is the safety net the packed-scheduler wiring needs. Measured earlier against the real sweep_packed_cells, that change is worth -27% on a cold sweep and -37% weekly, with breaker fidelity holding at three injected failure positions - but it restructures a 125-line loop across ~92 names that also holds graph prefetch, reuse selection, oracle staging and the canary drop. Landing that on top of a one-task fixture would have been unverifiable, which is why this comes first and separately. 682 eval tests pass, 17 skipped. * fix(eval): commit the stand-in CLI's executable bit The file was created and chmod +x'd locally, but committed 100644 - so the mode existed only in my working tree. Any fresh checkout, CI included, gets a non-executable file and every cell dies with "required executable is not an executable regular file". Found by accident: checking out origin/main and back to compare a flaky test restored the file from the index and stripped the bit, which turned 5 green tests into 9 failures. Without that detour this would have failed on the first CI run instead. Same shape as the bugs this branch exists to catch - something that works only because of local state, breaking where the code actually runs. * fix(eval): apply code review findings Seven local reviewers and an independent cross-model pass. The headline is that a fix I added in this branch was worse than the gap it closed. Reverted the aggregate() quality-median filter. Excluding skill-not-invoked rows from the quality metrics left valid_runs and excluded_runs still counting them, so the promotion gate saw N clean runs while the median came from fewer. The dropped rows are systematically an arm's worst, so it biased toward PROMOTING - reproduced: one real run at 0.9 plus two uninvoked rows at 0.0 gave the gate 3 valid runs, zero exclusions and a 0.9 median, flipping keep_incumbent to promote. Three verdict fields compounded it: they are all() reducers still reading the wider set, so one uninvoked cell flipped a whole arm. Five reviewers found the two halves independently. Closing it honestly needs a scored-run count plus a paired-equality check in the gate, which is promotion semantics rather than an aggregation fix. The gap is now pinned by a test that states why the half-fix was reverted. Stopped forging the absence of CI. The runner refuses --unsafe-no-bwrap when CI is set because that mode runs sessions with bypassPermissions behind a boundary its own docstring calls "not a security boundary"; the sweep test deleted CI to get past it, so eval / locked pytest ran an uncontained agent sweep on the runner holding the checkout and credentials. It skips under CI instead - the containment job still runs it for real with GITNEXUS_REQUIRE_FULL_SWEEP=1. The stand-in CLI was lying in three ways. It never set is_error, so a refused write read as a completed one. It had no Skill branch at all, so honoring is_error revealed the evidence gate had been satisfied by a tool the fixture never ran - the gate was measuring the fixture, not a skill. And a reply with no usage became four zero-valued fields plus a fabricated cost, which is exactly the unknown-is-not-zero confusion the accounting it feeds exists to prevent. A provider failure also crashed the subprocess with no terminal result event. The identity probe never ran anywhere. test_mock_provider.py was in no job's file list, and the only job setting CLAUDE_CANARY_BIN runs a fixed list. My commit message claimed the next containment run would produce the answer; it would not have. Now wired in, with the CI-shape test updated to pin it. Also: the regex-miss fallback wrote a predictable name in shared /tmp through a symlink-following stage, now scoped to the test's own directory; and the canonical_provider docstring plus the callback comment still asserted a call_type branch the code no longer has. Deferred as design decisions rather than review fixes: the containment sweep uses the stand-in CLI rather than the pinned real one, the full-sweep path bypasses the gateway so native usage accounting is unexercised there, _normalize_litellm duplicates the Responses algorithm, and OPENAI_RESPONSES is now unreachable from canonical_provider. 682 eval tests pass, 17 skipped, ruff clean. * fix(eval): carry scripted tools over the Responses protocol Review round on #3235. Three real items; five more were already fixed ina20f94e1cand are answered on their threads rather than re-fixed. `_openai_response` emitted only an `output_text` item and never read `reply.tools`, so a reply scripted with a Write or Skill crossed the gateway with the tool silently dropped. Responses is the protocol the gateway is configured for BECAUSE it carries tool use, so the mock was wrong about the wire on the one path that matters most. Function-call items now accompany the message. Mutation-checked: reverting the emit fails the new test on "the scripted tool must cross the Responses path". The artifact session now takes `command_prefix` and `require_pid_namespace` from the sandbox the way `run_arm` does instead of calling `run_claude` bare. On host-unsafe `command_prefix_for` returns `[]` by construction, so this pins the wiring, not the isolation - the comment says so rather than implying more. CodeQL's three unused-variable reports on one line were one finding: a call whose result is entirely discarded. Unpack nothing there. 683 passed, 17 skipped. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> * fix(eval): forward the usage the provider reported instead of zero-filling it Review round on #3235; all three findings valid. The stand-in CLI defaulted absent cache fields to 0. That fabricated a complete measurement out of an incomplete reply, and the second-order effect was worse than the first: `runner_sessions` requires all four USAGE_FIELDS before it calls a session measured, so a stand-in that always emitted four fields made that guard unfirable from any offline test. It was always satisfied. It now forwards exactly what arrived. `Reply`'s cache fields accept None to script absence, since a consumer that cannot tell "omitted" from "zero" is the bug this harness exists to catch. Mutation-checked: restoring the zero-fill fails the new test. Also corrected a comment claiming aggregate() excludes skill-not-invoked rows from the quality median. It does not - that was the filter reverted ina20f94e1cfor inverting a promotion, and the comment survived the revert describing the opposite of what the test pins. Dropped an unused monkeypatch fixture arg. 684 passed, 17 skipped. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> * fix(eval): keep an omitted cache field omitted on the Responses wire Review round on #3235. The main finding is a miss in my own previous commit: that one taught the Anthropic path to forward absence instead of zero-filling, but `_openai_response` still serialized both `input_tokens_details` keys unconditionally. Collapsing None to 0 is right for the arithmetic - an unreported field adds nothing to the total - and wrong on the wire, because `_int_or_none` reads an absent key as unknown and a present 0 as a measured zero. So a reply scripted with `cache_read_input_tokens=None` was indistinguishable from a provider-reported zero on exactly one of the two protocols. Half-applying the invariant was arguably worse than not applying it: the Anthropic test passing made the pair look covered. Mutation-checked: restoring the unconditional keys fails the new test. Also: the module docstring claimed the stand-in executes the tool blocks that come back, without noting Bash is stubbed; and dropped an unused tmp_path fixture arg. 685 passed, 17 skipped. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> * fix(eval): validate every usage field the stand-in forwards Review round on #3235. The guard checked input_tokens and output_tokens for type and sign, but the forwarding comprehension passed the cache fields through unchecked whenever present. The parent's well_formed test only asks whether the four keys are PRESENT, so a negative, boolean, or non-integer cache value rode into a `success` result and was recorded as a usable measurement. Same shape as the previous two rounds: the required half of a pair was handled and the optional half was not. A field good enough to report is good enough to check. Mutation-checked: dropping the added clause fails all three parametrized cases (negative, boolean, string). 688 passed, 17 skipped. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> * docs(eval): say which stand-in tools execute and which are modelled Review round on #3235. The previous commit's docstring fix said "Write and Skill really run" while correcting the Bash claim. Only Write really runs: Skill validates the request and returns a synthetic result. Fourth round of the same shape - the reported half of a pair gets fixed and the sibling keeps the overclaim. Both docstrings now name each of the three branches and what it actually does. 688 passed, 17 skipped. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Gergo Magyar <gergomagyar0@gmail.com> Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
327 lines
14 KiB
Python
327 lines
14 KiB
Python
"""What the proxy writes must outlive the translation that follows it.
|
|
|
|
Claude Code receives an Anthropic-shaped response, which has nowhere to put
|
|
OpenAI's cached_tokens, cache_write_tokens or reasoning_tokens. If those are not
|
|
captured before the translation, the only remaining record of them is a bill.
|
|
"""
|
|
|
|
from __future__ import annotations
|
|
|
|
import contextlib
|
|
import json
|
|
from pathlib import Path
|
|
from types import SimpleNamespace
|
|
|
|
import pytest
|
|
|
|
from workflow_bench import litellm_usage_callback, provider_usage
|
|
from workflow_bench.litellm_usage_callback import USAGE_LOG_ENV_VAR, ProviderUsageLogger
|
|
from workflow_bench.model_gateway import (
|
|
OpenAIGateway,
|
|
USAGE_CALLBACK_MODULE,
|
|
openai_litellm_config,
|
|
write_openai_litellm_config,
|
|
)
|
|
from workflow_bench.provider_usage import (
|
|
ANTHROPIC,
|
|
LITELLM_NORMALIZED,
|
|
USAGE_ENV_VARS,
|
|
normalize_usage,
|
|
)
|
|
|
|
|
|
class _Usage:
|
|
"""Stands in for the provider usage model LiteLLM hands the callback."""
|
|
|
|
def __init__(self, payload: dict) -> None:
|
|
self._payload = payload
|
|
|
|
def model_dump(self) -> dict:
|
|
return self._payload
|
|
|
|
|
|
def _openai_response(usage: dict) -> SimpleNamespace:
|
|
return SimpleNamespace(
|
|
id="resp_68f2c1",
|
|
# The model that actually answered, which is not the role the caller asked for.
|
|
model="gpt-5.6-sol-2026-08-01",
|
|
usage=_Usage(usage),
|
|
)
|
|
|
|
|
|
# The shape a callback actually receives: LiteLLM normalises usage into its own
|
|
# Chat-Completions-style object before any logger sees it, so an OpenAI reply
|
|
# arrives as prompt_tokens / prompt_tokens_details. Confirmed against a real
|
|
# proxy in tests/test_mock_provider.py; a fixture in the wire shape would test
|
|
# an object this code path never gets.
|
|
NATIVE = {
|
|
"prompt_tokens": 48_000,
|
|
"prompt_tokens_details": {"cached_tokens": 44_000, "cache_write_tokens": 1_000},
|
|
"completion_tokens": 900,
|
|
"completion_tokens_details": {"reasoning_tokens": 640},
|
|
}
|
|
|
|
|
|
@pytest.fixture
|
|
def logged(tmp_path: Path, monkeypatch: pytest.MonkeyPatch):
|
|
log = tmp_path / "provider_usage.jsonl"
|
|
monkeypatch.setenv(USAGE_LOG_ENV_VAR, str(log))
|
|
|
|
def emit(usage: dict) -> dict:
|
|
ProviderUsageLogger()._append(
|
|
"success",
|
|
{"model": "claude-sonnet-4-5", "custom_llm_provider": "openai", "call_type": "responses"},
|
|
_openai_response(usage),
|
|
0.0,
|
|
1.0,
|
|
)
|
|
return json.loads(log.read_text().splitlines()[-1])
|
|
|
|
return emit
|
|
|
|
|
|
def test_native_openai_usage_survives_the_anthropic_translation(logged) -> None:
|
|
event = logged(NATIVE)
|
|
native = event["native_usage"]
|
|
# Verbatim: the fields an Anthropic-shaped response cannot carry.
|
|
assert native["prompt_tokens_details"]["cached_tokens"] == 44_000
|
|
assert native["prompt_tokens_details"]["cache_write_tokens"] == 1_000
|
|
assert native["completion_tokens_details"]["reasoning_tokens"] == 640
|
|
assert event["response_id"] == "resp_68f2c1"
|
|
|
|
|
|
def test_the_actual_model_is_recorded_separately_from_the_requested_role(logged) -> None:
|
|
"""Pricing must follow what answered, not what the caller named."""
|
|
|
|
event = logged(NATIVE)
|
|
assert event["requested_model"] == "claude-sonnet-4-5"
|
|
assert event["actual_model"] == "gpt-5.6-sol-2026-08-01"
|
|
assert "cell_id" not in event, "a proxy-wide variable cannot identify a cell"
|
|
|
|
|
|
def test_the_captured_event_normalizes_with_openai_arithmetic(logged) -> None:
|
|
"""Capture and normalization must agree end to end, not just in isolation."""
|
|
|
|
event = logged(NATIVE)
|
|
# The provider the LOG recorded, not one the test supplies - passing
|
|
# OPENAI_RESPONSES by hand here is what hid the adapter-key mismatch.
|
|
# LITELLM_NORMALIZED, not OPENAI_RESPONSES: a proxy callback never sees the
|
|
# upstream body. Measured against a real gateway - the Responses adapter
|
|
# found none of its keys there and reported every field unknown.
|
|
assert event["provider"] == LITELLM_NORMALIZED
|
|
assert event["provider_label"] == "openai"
|
|
usage = normalize_usage(event["provider"], event["native_usage"])
|
|
assert usage.total_input_tokens == 48_000
|
|
assert usage.ordinary_input_tokens == 3_000
|
|
assert usage.cache_read_input_tokens == 44_000
|
|
assert usage.complete
|
|
|
|
|
|
def test_usage_without_details_normalizes_to_unknown_rather_than_zero(logged) -> None:
|
|
"""The mutation the accounting must not survive: dropped details, silent zeros."""
|
|
|
|
stripped = {k: v for k, v in NATIVE.items() if k != "prompt_tokens_details"}
|
|
event = logged(stripped)
|
|
usage = normalize_usage(event["provider"], event["native_usage"])
|
|
assert usage.cache_read_input_tokens is None
|
|
assert usage.ordinary_input_tokens is None
|
|
assert not usage.complete
|
|
|
|
|
|
def test_a_failed_request_is_still_accounted_for(logged, tmp_path: Path) -> None:
|
|
"""The money was spent whether or not the cell produced an artifact."""
|
|
|
|
import asyncio
|
|
|
|
logger = ProviderUsageLogger()
|
|
args = ({"model": "claude-sonnet-4-5"}, _openai_response(NATIVE), 0.0, 1.0)
|
|
# Every hook LiteLLM can call, not the private helper underneath them: the
|
|
# sync failure hook was missing entirely and _append could never show that.
|
|
logger.log_failure_event(*args)
|
|
asyncio.run(logger.async_log_failure_event(*args))
|
|
events = [json.loads(line) for line in (tmp_path / "provider_usage.jsonl").read_text().splitlines()]
|
|
assert len(events) == 2, "both failure hooks must record"
|
|
assert all(e["status"] == "failure" for e in events)
|
|
|
|
|
|
def test_every_public_outcome_hook_records(tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> None:
|
|
"""Overriding a subset silently drops whichever path LiteLLM actually uses."""
|
|
|
|
import asyncio
|
|
|
|
monkeypatch.setenv(USAGE_LOG_ENV_VAR, str(tmp_path / "usage.jsonl"))
|
|
logger = ProviderUsageLogger()
|
|
args = ({"model": "m"}, _openai_response(NATIVE), 0.0, 1.0)
|
|
logger.log_success_event(*args)
|
|
logger.log_failure_event(*args)
|
|
asyncio.run(logger.async_log_success_event(*args))
|
|
asyncio.run(logger.async_log_failure_event(*args))
|
|
|
|
events = [json.loads(line) for line in (tmp_path / "usage.jsonl").read_text().splitlines()]
|
|
assert [e["status"] for e in events] == ["success", "failure", "success", "failure"]
|
|
|
|
|
|
def test_the_logger_never_raises_into_the_proxy(tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> None:
|
|
"""Accounting is evidence, not control flow."""
|
|
|
|
monkeypatch.setenv(USAGE_LOG_ENV_VAR, str(tmp_path / "missing-dir" / "usage.jsonl"))
|
|
ProviderUsageLogger()._append("success", {}, object(), 0.0, 1.0)
|
|
|
|
|
|
def test_no_log_is_written_when_the_destination_is_unset(tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> None:
|
|
monkeypatch.delenv(USAGE_LOG_ENV_VAR, raising=False)
|
|
ProviderUsageLogger()._append("success", {}, _openai_response(NATIVE), 0.0, 1.0)
|
|
assert not list(tmp_path.iterdir())
|
|
|
|
|
|
def test_the_generated_config_loads_the_callback_from_beside_itself(tmp_path: Path) -> None:
|
|
"""LiteLLM resolves the dotted path relative to the config directory."""
|
|
|
|
config = write_openai_litellm_config(tmp_path / "litellm.yaml", ["gpt-5.6-sol"])
|
|
assert openai_litellm_config(["gpt-5.6-sol"])["litellm_settings"]["callbacks"] == [
|
|
f"{USAGE_CALLBACK_MODULE}.handler"
|
|
]
|
|
installed = config.parent / f"{USAGE_CALLBACK_MODULE}.py"
|
|
assert installed.is_file(), "the proxy cannot import a callback that was never placed"
|
|
# Importing it, not grepping it: a text search passes even when the module
|
|
# cannot load, which is exactly how a package-relative import survived
|
|
# review here. This is the deployment configuration, so load it the way the
|
|
# proxy does - by path, as a top-level module.
|
|
import importlib.util
|
|
|
|
spec = importlib.util.spec_from_file_location(USAGE_CALLBACK_MODULE, installed)
|
|
assert spec is not None and spec.loader is not None
|
|
module = importlib.util.module_from_spec(spec)
|
|
spec.loader.exec_module(module)
|
|
assert isinstance(module.handler, module.ProviderUsageLogger)
|
|
|
|
|
|
def test_the_gateway_forwards_the_usage_environment_into_the_proxy(
|
|
tmp_path: Path, monkeypatch: pytest.MonkeyPatch
|
|
) -> None:
|
|
"""The proxy is a separate process with a constructed environment.
|
|
|
|
Popen(env=...) replaces the parent environment rather than extending it, so
|
|
a variable the callback reads is simply absent unless the gateway forwards
|
|
it by name. Without this the accounting looks configured and silently
|
|
records nothing on every request - the in-process tests above cannot see
|
|
that, because they never cross the subprocess boundary.
|
|
"""
|
|
|
|
for name in USAGE_ENV_VARS:
|
|
monkeypatch.setenv(name, f"value-for-{name}")
|
|
captured: dict[str, dict[str, str]] = {}
|
|
|
|
class _Popen:
|
|
def __init__(self, *_a, **kwargs):
|
|
captured["env"] = kwargs["env"]
|
|
raise RuntimeError("stop before launching a real proxy")
|
|
|
|
# The console-script resolver runs before Popen and is absent in this
|
|
# environment (the same reason two gateway tests fail here); the argv it
|
|
# builds is not what this test is about.
|
|
monkeypatch.setattr(
|
|
"workflow_bench.model_gateway.litellm_proxy_argv",
|
|
lambda **_k: ["/bin/true"],
|
|
)
|
|
monkeypatch.setattr("workflow_bench.model_gateway.subprocess.Popen", _Popen)
|
|
gateway = OpenAIGateway(
|
|
openai_api_key="sk-test",
|
|
model_names=["gpt-5.6-sol"],
|
|
work_dir=tmp_path,
|
|
)
|
|
with contextlib.suppress(Exception):
|
|
gateway.__enter__()
|
|
|
|
env = captured.get("env")
|
|
assert env is not None, "the proxy was never constructed"
|
|
for name in USAGE_ENV_VARS:
|
|
assert env.get(name) == f"value-for-{name}", f"{name} never reached the proxy"
|
|
# The credential allowlist is still an allowlist, not the parent environment.
|
|
assert "PATH" in env and len(env) < 40
|
|
|
|
|
|
def test_an_unresolvable_provider_is_refused_rather_than_guessed() -> None:
|
|
"""LiteLLM says "openai" for Chat Completions too, and it counts differently."""
|
|
|
|
from workflow_bench.provider_usage import canonical_provider
|
|
|
|
# Every openai call reaching this callback has already been normalised by
|
|
# LiteLLM, whatever endpoint it used - the observed call_type for a Claude
|
|
# Code request through the gateway is "anthropic_messages". The adapter has
|
|
# to match the object in hand, not the protocol on the wire.
|
|
assert canonical_provider("openai", "responses") == LITELLM_NORMALIZED
|
|
assert canonical_provider("openai", "anthropic_messages") == LITELLM_NORMALIZED
|
|
assert canonical_provider("anthropic", "completion") == ANTHROPIC
|
|
# An unrecognised provider is still refused rather than guessed.
|
|
assert canonical_provider("some-new-provider", "responses") is None
|
|
assert canonical_provider(None, None) is None
|
|
|
|
|
|
def test_request_identity_cannot_come_from_the_proxy_environment() -> None:
|
|
"""One proxy serves the whole sweep, so its environment identifies the sweep.
|
|
|
|
attach_openai_gateway wraps all of _run_sweep, and cells run concurrently
|
|
under --workers, interleaving requests through that single process. Any
|
|
variable forwarded at launch is therefore constant for every event it ever
|
|
records. Pinned so a future change does not reintroduce a per-cell
|
|
environment variable that would silently stamp one value on every request.
|
|
"""
|
|
|
|
assert USAGE_ENV_VARS == (
|
|
"GITNEXUS_BENCH_PROVIDER_USAGE",
|
|
"GITNEXUS_BENCH_SWEEP_ID",
|
|
), "a per-cell variable here would be constant across concurrent cells"
|
|
|
|
|
|
def test_a_request_records_its_session_so_attribution_stays_possible(logged) -> None:
|
|
"""The per-request half of identity, recorded even when the provider omits it."""
|
|
|
|
event = logged(NATIVE)
|
|
assert "session_id" in event, "absent attribution is still a fact about the run"
|
|
|
|
|
|
def test_the_callback_imports_the_way_litellm_actually_loads_it(tmp_path: Path) -> None:
|
|
"""By path, as a top-level module, with no parent package and no sys.path entry.
|
|
|
|
LiteLLM resolves a dotted callback through spec_from_file_location against
|
|
the config directory, so the copied file is not part of workflow_bench when
|
|
it runs. A relative or sibling import therefore raises ImportError and the
|
|
proxy exits before becoming ready - which the in-package tests cannot see,
|
|
because they import it as workflow_bench.litellm_usage_callback.
|
|
"""
|
|
|
|
import importlib.util
|
|
import shutil
|
|
|
|
source = Path(litellm_usage_callback.__file__)
|
|
installed = tmp_path / f"{USAGE_CALLBACK_MODULE}.py"
|
|
shutil.copy(source, installed)
|
|
|
|
spec = importlib.util.spec_from_file_location(USAGE_CALLBACK_MODULE, installed)
|
|
assert spec is not None and spec.loader is not None
|
|
module = importlib.util.module_from_spec(spec)
|
|
spec.loader.exec_module(module) # ImportError here is the proxy refusing to start
|
|
assert hasattr(module, "handler")
|
|
|
|
|
|
def test_the_callbacks_copied_constants_match_the_canonical_ones() -> None:
|
|
"""The copies are deliberate; drifting apart silently is not.
|
|
|
|
The callback cannot import from the package (see the test above), so it
|
|
carries its own literals. These assertions are what keep the duplication
|
|
honest.
|
|
"""
|
|
|
|
assert litellm_usage_callback.USAGE_LOG_ENV_VAR == provider_usage.USAGE_LOG_ENV_VAR
|
|
assert litellm_usage_callback.SWEEP_ID_ENV_VAR == provider_usage.SWEEP_ID_ENV_VAR
|
|
for label, call_type in (
|
|
("openai", "responses"),
|
|
("openai", "completion"),
|
|
("openai", None),
|
|
("anthropic", "completion"),
|
|
("mystery", "responses"),
|
|
):
|
|
assert litellm_usage_callback.canonical_provider(label, call_type) == provider_usage.canonical_provider(
|
|
label, call_type
|
|
), f"resolver drifted for {label!r}/{call_type!r}"
|