Skip to content

feat(skilleval): bounded cloud generation and isolated artifact replay - #267

Merged
Dong1017 merged 2 commits into
mainfrom
issue253-cloud-generation
Oct 30, 2026
Merged

Dong1017 merged 2 commits into
mainfrom
issue253-cloud-generation

Conversation

@Dong1017

Copy link
Copy Markdown
Collaborator

Governing Issue and delivery slice

Refs #253

  • Delivers an explicit generation-only path for the bounded public parser comparison without requiring a contributor-local Linux/WSL agent.
  • The provider returns source through the existing full-context submit-only client; a trusted controller writes only bounded parser.py bytes. This is not an iterative coding-agent experiment.
  • A separate historical accounting wrapper preserves the normal PR gate, binds compared revisions independently from the accounting PR, and requires prospective approval/protocol/plan hashes, fresh configuration, durable PR/day claims and a maximum of two requests. No default provider configuration, ledger or skill mapping is changed.
  • Manual main-only Linux replay accepts a fixed bounded data-only bundle, requalifies the host with known controls, and runs sources only through the existing bubblewrap verifier. No API credential is passed to Actions.
  • Original capture INCONCLUSIVE/errors remain visible; independent artifact verdicts and comparison outcomes are separate. No efficacy or Issue acceptance is claimed.

Changed artifacts and evidence

Requirement Artifact Evidence Remaining gap
No local-agent execution dependency Generation schema/prompt, trusted writer, retained source 22 offline generation tests, including malformed/oversized output, broker enforcement and remote-report integrity Actual paid capture not run
Account historical pair honestly tools/skill_eval_historical_accounting.py 13 new tests; 83 accounting/auto/budget tests passed; existing request guard tests passed Operational receipt/config/ledger admission still required
Independent isolated replay tools/skill_eval_cloud_replay.py, manual workflow Fixed schema/hashes, no arbitrary grader/path, qualification prerequisite; replay tests passed Actual host/run evidence separate
Data-only transfer tools/skill_eval_cloud_export.py Export-to-ingress interoperability, missing/changed approval and tampered source rejected Actual bundle not produced

Checks and dependencies

  • Integrated generation/accounting/replay/qualification selection: 60 tests PASS.
  • Broader generation/local/engineering/compare/provider selection: 236 tests PASS, excluding real-sandbox tests; these do not qualify this host.
  • Ruff on all changed Python files PASS; git diff --check PASS.
  • Base includes merged feat(skilleval): add manual strict verifier qualification #266 at 37166b6f7323c76a091e423a7163ac99c4ae01ac.
  • Tested local tree 4504542c565e66e1a1fde432e8dc36ae163f522d; published head has the identical tree.
  • No paid model call, real model efficacy experiment or external artifact replay has been run by this PR's unit tests.

Knowledge authorship declaration

Code/workflow/operational docs only; no protected skill or knowledge asset changes. Material implementation is agent-assisted; model identifiers are not asserted here.

Remaining work and Issue closure

Keep #253 open. After current-head review/CI: verify actual Linux qualification, freeze the precise prospective protocol and fresh cost/accounting inputs, capture the bounded pair, export its source bytes, independently replay, and publish the measured result with limits. A negative or inconclusive result must remain truthful. This PR is software delivery, not completion of the experiment.

Retain generation-only parser artifacts, bind historical experiment accounting separately from compared revisions, and replay fixed data-only bundles on qualified Linux hosts. Preserve original inconclusive captures and report independent artifact checks separately.

Signed-off-by: Codex <codex@openai.com>
Agent-assisted-by: Codex design_cloud_trial253 (implementation)
Agent-assisted-by: Codex verify_trial_cost_bounds (accounting implementation)
Agent-assisted-by: Codex find_skilleval_unblock (replay implementation)
@github-actions

github-actions Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

Skill evaluation

Automatic execution / cumulative PR budget: {"reason":"Trusted configuration or pricing missing, invalid or expired","remaining_usd":"unknown","reserved_or_unresolved_usd":"unknown","spent_or_legacy_usd":"unknown","state":"blocked"}

Advisory: obtain missing evidence

Retained evidence availability: planned / unexecuted. No approval or merge authorization is issued.

  • Retained evidence is planned / unexecuted.
  • No supported comparison cases were selected.
  • Uncovered/unsupported scope needs a case or explicit reviewer disposition.

PLANNED: execution is not authorized by knowing this hash.
Outcomes: {"fix":0,"inconclusive":0,"missing_pair":0,"regression":0,"unchanged_correct":0,"unchanged_incorrect":0}
Earlier revision results are stale for this plan.

Selected comparisons: 0 · Estimated task calls: 0.
Uncovered/unsupported changes: 13. Gaps never count as passes.

Applicable only while these revisions and inputs match the current PR.
Paid execution requires trusted label authorization and cumulative PR reservation. Unknown accounting blocks new paid calls; planning is not execution.
Pipeline evidence only; offline fixed outputs do not establish skill efficacy.
Evidence and logs · Retention: 30 days.
Outcomes are historical; verifiable only before Unix time 1795935885 while the linked artifact is available. Expired evidence is unverified.

Plan bindings, budget, frozen exam and advisory details

Unchanged-correct is valid neutral evidence, not incremental benefit.
Content/source review and authorship admission remain separate.
Trusted regrading checks retained answers, not model/hardware provenance or a hostile executor; this is not an efficacy certification.

Per-case outcomes (discovery excluded; regressions and gaps first):

[]

Showing at most 20 of 0 pairs; the evidence link contains all outcomes, including omitted regressions/gaps.

Plan SHA-256: 58abb114efaaf2cb9a7da3e949f29e8c7b7513d3f2f5fa2de7744e853fbe47ab
Base: 37166b6f7323c76a091e423a7163ac99c4ae01ac · Head: f77f73539d4a5dc781ab1a956d60c49565d676bc
Trusted implementation: 37166b6f7323c76a091e423a7163ac99c4ae01ac

Model/settings: {"effort":"not_applicable","name":"claude-opus-5-5"}
Budget: {"comparisons":0,"estimated_calls":0,"limits_per_comparison":{"campaign_seconds":600,"max_task_calls":2,"repeats":1,"timeout_seconds":180,"token_stop_new_trials":200000},"provider_config_sha256":"b28d80f8194fbe0f5a9f01515efb24b218979fd45f702776560da276f4b30d04","reservation":null}

Skill Comparison Cases Estimated calls

Provider/settings: {"client":{"api_version":"2023-06-01","endpoint":"https://yibuapi.com/v1/messages","kind":"anthropic-messages-full-context-v1","max_output_tokens":4096,"max_read_bytes":32768,"max_request_bytes":262144,"max_requests":1,"max_tool_calls":12,"provider":"yibuapi"},"limits":{"token_stop_new_trials":200000},"model":{"effort":"not_applicable","name":"claude-opus-5-5"},"pricing":{"currency":"USD","expires_at":1793271228,"group":"default","max_input_tokens":1000000,"model":"claude-opus-5-5","rate_evidence":"https://github.com/InferMatrixLab/InferRSI/issues/215#issuecomment-6467531341","rates":{"cache_read":"0.4","cache_write":"10","input":"8","output":"40"},"request_fee":"0","unit":"per_million_tokens"},"spend_limit":{"amount":"200","currency":"USD"}}

Uncovered / unsupported paths (13)
  • uncovered: ".github/workflows/skill-eval-cloud-replay.yml"
  • uncovered: "docs/operations.md"
  • uncovered: "docs/skill-eval-cloud-replay.md"
  • uncovered: "docs/skill-eval-generation-only.md"
  • uncovered: "src/inferrsi/harness/skill_eval_engineering.py"
  • uncovered: "src/inferrsi/harness/skill_evaluation.py"
  • uncovered: "tests/test_skill_eval_cloud_replay.py"
  • uncovered: "tests/test_skill_eval_generation.py"
  • uncovered: "tests/test_skill_eval_historical_accounting.py"
  • uncovered: "tools/skill_eval_cloud_export.py"
  • uncovered: "tools/skill_eval_cloud_replay.py"
  • uncovered: "tools/skill_eval_compare.py"
  • uncovered: "tools/skill_eval_historical_accounting.py"

@Dong1017
Dong1017 marked this pull request as ready for review October 30, 2026 06:59
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-30T07:07:18.009109Z f77f735 New commits
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

Keep both capture and capture-report phases inert on every host. Preserve explicit deferred verification and independently replay only in the verifier phase.

Signed-off-by: Codex <codex@openai.com>
Agent-assisted-by: Codex design_cloud_trial253 (implementation)

@Dong1017 Dong1017 left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Scoped software LGTM at f77f735 / tree c194a5219f24375f14609ca051cd9616c8fddcb9, base 37166b6. The PR body retains its initial tree/test snapshot; this review covers the final published tree, including the later no-execution-in-generation correction.

Independent review verified generation capture and its automatic report never invoke artifact verification even on a capable host; only the separate trusted replay path runs parsers. Replay preserves original deferred provenance, uses bounded strict JSON plus exact revision/hash bindings, and requalifies the same host with good/bad controls. Existing 19-check grading remains unchanged. Historical accounting separates charge PR from skill revisions, binds full approval digest and stable experiment identity, preserves previous reservations, and retains per-request dual-ledger/CAS guards with no retry/refund. Earlier integration findings (deferred replay eligibility, integrity error precedence, export binding, and ledger plan digest) are fixed with regressions.

Verification: independent 60 focused tests passed; final implementation regression run 192 tests and Ruff passed. Current-head Ready CI 41017811917 and policy checks passed. Native review 6516945855 completed on this head without findings; no unresolved review objections identified.

Advisory 6516942992 remains zero-comparison/unexecuted, not efficacy evidence. This is software infrastructure delivery only. Actual model execution still requires valid effective pricing, approved operational configuration and accounting, and authorized credential access. No new credential-consuming job is included here. Hosted qualification of the earlier verifier is not blanket approval of this changed tree; replay requalifies before evaluating artifacts. No real paired result or #253 acceptance is claimed. COMMENT only, not formal APPROVE.

@Dong1017
Dong1017 merged commit f93df05 into main Oct 30, 2026
10 checks passed
@Dong1017
Dong1017 deleted the issue253-cloud-generation branch October 30, 2026 08:24
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