feat: initial proxybench evidence-first benchmark utility - #1
Merged
Merged
Conversation
pnlabs-dev
force-pushed
the
feat/initial-proxybench
branch
from
August 20, 2026 17:46
b75aee3 to
e992310
Compare
pnlabs-dev
commented
Aug 20, 2026
pnlabs-dev
left a comment
Owner
Author
There was a problem hiding this comment.
PN Labs final review — merge-ready
Second-pass review is complete on the clean one-commit branch. No remaining source-level blocker found.
Verified boundaries:
- measurement only; no network/proxy orchestration;
- unknown input fields rejected;
- no raw-line/path echo in handled validation errors;
- cost/rotation ratios fail closed under partial coverage;
- descriptive intervals and gates do not claim causality;
- proprietary routing/scoring/provider logic remains excluded;
- public hygiene gate covers credentials, credential-bearing URLs, IP literals, internal hostnames, sensitive filenames, and emails without echoing matches;
- package install, 20 tests, and installed CLI smoke checks pass across Python 3.10/3.11/3.12.
Active PR history is one commit directly on top of main. Recommendation: publish via squash/merge once ready.
pnlabs-dev
marked this pull request as ready for review
August 20, 2026 17:47
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
Introduces the first public PN Labs implementation of
proxybench: a zero-runtime-dependency, local-only utility for comparing sanitized proxy/web-retrieval outcomes by usable-result efficiency rather than proxy count.Core metrics
Public data boundary
The event schema accepts only
usable,latency_ms,cost_units,rotated, andoutcome. Unknown fields are rejected. URLs, IPv4/IPv6 addresses, provider identifiers, credentials, cookies, headers, payloads, customer identifiers, and production infrastructure context are not required and are outside the public schema.The CLI avoids intentionally echoing raw JSON lines, unsupported values, or local input paths in validation errors.
Product boundary
This repository is the transparent measurement baseline. Private routing/scoring implementation, provider selection, target × egress learning, promotion/demotion intelligence, private benchmark data, and production infrastructure remain outside OSS.
Security hardening
contents: read;Clean history
The branch was rebuilt from the reviewed final tree as one commit directly on top of
main.80dbf8a97a0a6ad4360a1b0f7acba40f59fd14f3e992310afb67cb07c87d60a8e2d8a9f68d468281Final validation
Exact clean-history head
e992310afb67cb07c87d60a8e2d8a9f68d468281:CI run:
32399449277.Review posture
Metric semantics, missing-data behavior, privacy/data minimization, IP/secret hygiene, CI packaging, commercial boundary, and active branch history have been reviewed. Descriptive intervals and operational gates are explicitly not presented as causal proof.