convert-separate-verifier

name	convert-separate-verifier
description	Convert a Harbor benchmark task from Harbor's shared verifier mode (default) to separate verifier mode. Use when the user asks to "convert this task to separate verifier", "make the verifier run in its own container", or asks about Harbor's separate-verifier environment for a specific task.
allowed-tools	Read, Edit, Write, Bash, Glob, Grep

You convert a single Harbor benchmark task from shared verifier mode (verifier runs in the agent's container after the agent finishes) to separate verifier mode (verifier runs in its own fresh container, sees only declared artifacts + connections to surviving sidecars).

Reference docs: https://www.harborframework.com/docs/tasks#verifier-environment-shared-vs-separate

Before you start — always make a new worktree

Do the conversion in a fresh git worktree on a new branch (e.g., convert-<task-name>), not the user's main checkout. Do all edits from the worktree.

Mechanics you must know before editing

These are gotchas confirmed against Harbor 0.7.1 source by running an actual oracle through the converted hello-world. The docs are thin; do not invent assumptions beyond these.

• The TOML key is environment_mode = "separate" under [verifier].
• artifacts = [...] lives at the TOP LEVEL of task.toml, not under [verifier]. TOML scoping: anything written after a [section] header belongs to that section. Place artifacts above the first [section] (typically right after schema_version). Putting it under [verifier] silently drops it from the resolved task config and the verifier sees no artifacts.
• The verifier image's build context is tasks/<name>/tests/ — tests/Dockerfile builds with tests/ as the context. Hardcoded in Harbor.
• In separate mode, Harbor does NOT upload tests/ into the verifier container (skip_tests_upload=True). The verifier image must own /tests/* itself. So tests/Dockerfile must COPY . /tests/ (or otherwise place test.sh and any supporting files at /tests/). Forgetting this manifests as RewardFileNotFoundError because test.sh never runs.
• Artifacts declared in artifacts = ["/app/...", ...] land at the same absolute path in the verifier container. No translation: if you declared /app/hello.txt, read it at /app/hello.txt. BUT — the parent directory must exist in the verifier image, or the upload fails with Could not find the file /app in container. Add RUN mkdir -p /app (or the appropriate parent) to tests/Dockerfile.
• The agent container is stopped BEFORE the verifier container starts. Declared artifacts are downloaded to a host-side dir between phases, then re-uploaded into the fresh verifier container. Sidecars (declared in docker-compose.yaml) persist across the handoff. /logs/verifier/ is bind-mounted between host and the verifier container for its lifetime, which is how reward.txt reaches the host the same way it always did.
• If [verifier.environment] is omitted, Harbor uses a fresh copy of the top-level [environment] as the verifier env — but tests/Dockerfile is still the image source. Omit [verifier.environment] unless you genuinely need different resource limits or a different OS than the agent env.
• Bake external dependencies into the verifier image (uv, language toolchains, system libs, downloaded oracle models, etc.). Don't rely on test.sh to apt-get install, curl ... | sh, pip install, or otherwise reach out at verify time. Runtime installs are flaky (intermittent network, rate limits, upstream version churn) and slow down every trial; baking them into tests/Dockerfile is more reliable and faster.

Core rules — non-negotiable

1. Minimal changes. Make the smallest edit set that achieves separate mode. Do not refactor, rename, restructure, "clean up", add tests, or improve unrelated parts of the task.
2. Preserve task semantics. The agent's experience, instruction text, and grading criteria must remain equivalent. The agent should still solve the same problem; the only difference is where the verifier runs.
3. Concise instruction additions. If the conversion requires the agent to ship a manifest (e.g., requirements.txt), add one short sentence to instruction.md — not a paragraph. Example: Write your Python dependencies to /app/requirements.txt. Nothing more.
4. Verify before declaring done. After converting, re-read everything (instruction, task.toml, tests, Dockerfile, solution) and check:
   • Does the verifier still have access to everything it reads?
   • Are all artifacts the verifier needs declared in artifacts or already in the verifier image?
   • Will the solution script (solution/solve.sh) still produce a passing reward?
   • Did anything in instruction.md become inconsistent with the new setup?
5. Stop and ask if unsure. If you encounter ambiguity (e.g., it's unclear whether a path is an agent artifact or a verifier-image asset, or whether moving a service to a sidecar will change the test's nature), DO NOT guess. Post a comment on the PR using gh pr comment <PR> --body "<text>" describing the issue and 2–3 possible approaches, then STOP and wait for input. Do not push speculative changes.

When to skip conversion entirely

Do not attempt conversion when:

• The user has not specified which PR/task. Ask.
• The task uses unusual harness primitives (e.g., custom pre-verifier hooks, baked oracle blobs in the agent image) that need owner input on where they should live. Post a comment.

Conversion playbook

The conversion shape depends on which bucket the task falls in. To decide, read the whole task: task.toml, instruction.md, everything under tests/, everything under solution/, and everything under environment/.

Bucket 1 — FILES (verifier reads a fixed set of agent output files)

Symptom: the verifier opens specific paths under /app/output/, /app/, /results/, etc., and compares them to constants or /tests/-shipped ground truth. No subprocess.run/importlib/sys.path.insert for agent code.

Conversion:

1. In task.toml, under [verifier], set environment_mode = "separate".

2. Add an artifacts = ["/app/path/to/output.json", "/app/another/file.csv"] list with only the paths the verifier actually reads. Do not over-declare.

3. Add tests/Dockerfile that pre-installs runtime deps, copies the test scripts in, and pre-creates artifact parent directories. Minimal example:

   FROM python:3.13-slim-bookworm
   RUN apt-get update && apt-get install -y --no-install-recommends curl && rm -rf /var/lib/apt/lists/*
   RUN curl -LsSf https://astral.sh/uv/0.9.7/install.sh | sh
   ENV PATH="/root/.local/bin:${PATH}"
   COPY . /tests/
   RUN mkdir -p /app
   

   Adapt the FROM line and pre-installed tools to whatever the verifier needs (e.g., torch base, R, JDK). Adapt the mkdir -p to whatever parent directories your artifacts need. The COPY . /tests/ is mandatory.

4. If the existing test.sh does runtime apt-get install / curl ... | sh for tools that should be in the image, drop those lines and rely on the verifier image instead.

5. If the verifier compares against a file currently baked into the agent image (e.g., /root/data/ground_truth.json chmod 600), that file is gone in separate mode — the verifier container is fresh from tests/Dockerfile and has nothing from the agent image. Move the file out of environment/ into tasks/<name>/tests/ground_truth.json (a checked-in file in the repo). tests/Dockerfile's COPY . /tests/ will bring it in as /tests/ground_truth.json. Update the verifier code to read the new path. Bonus: the agent never sees this file at all now — strictly better anti-cheat than the old chmod-600 trick.

6. Do not change instruction.md (no agent-visible change).

7. Verify by re-reading tests/test_*.py and confirming every path it opens is either: (a) in the artifacts list (same absolute path), (b) in /tests/, or (c) installed by tests/Dockerfile.

Worked example: tasks/hello-world/ — a minimal FILES conversion that declares /app/hello.txt as an artifact, ships uv + the tests via tests/Dockerfile, and has a trivial test.sh.

Bucket 2 — CODE+PACKAGES (verifier executes agent code)

Symptom: the verifier does sys.path.insert(0, "/app"), importlib.import_module(...), subprocess.run([sys.executable, "/app/...py"], cwd="/app"), make / cargo build / lake build / bazel build etc. against /app, or runs an agent-built binary. Anything where verifier code paths execute agent code paths.

Once agent code runs in the verifier container, the verifier needs the agent's runtime — which means any packages the agent might have installed during their task. Harbor tasks are typically outcome-based: the agent has internet access and root in the agent container, and unless the instruction explicitly restricts the toolset (and the tests enforce that restriction), they can install whatever they want. So in separate mode the agent must pin those installs so the verifier image can reproduce them.

Conversion:

1. In task.toml, under [verifier], set environment_mode = "separate".
2. Add an artifacts list declaring the agent's source tree paths — only the ones the verifier needs (e.g., ["/app/src/", "/app/output/"], not ["/app/"]).
3. Add a manifest requirement to instruction.md, scoped to the task's actual stack. Identify what the agent might install (language ecosystem from the base image + system layer) and add one sentence per relevant manifest:
   • Python tasks: Pin any Python packages needed to run your code in /app/requirements.txt.
   • Node tasks: agent already needs package.json for any Node project, so the language manifest is implicit. No instruction change needed for the language layer.
   • Rust tasks: agent already needs Cargo.toml for any Rust project. Language manifest is implicit.
   • Go tasks: agent already needs go.mod. Implicit.
   • System (apt) layer — add this for any task where the agent could plausibly apt install system libs (most Linux-based tasks): Pin any apt packages needed to run your code in /app/apt-packages.txt, one per line.
   • Skip a layer only if the instruction already restricts the agent from using it AND the tests enforce that restriction.
   • Add every declared manifest path to artifacts.
4. NEVER allow arbitrary setup.sh or install.sh as the dep manifest — that's a reward-hack surface (the script would run with verifier privileges). Only allow declarative manifests pinned per ecosystem.
5. If the verifier already does heavy setup itself (already installs uv + pytest, downloads model checkpoints, etc.), leave that in place. The verifier image just additionally installs the agent's manifest.
6. Do not change anything that doesn't have to change.

Bucket 3 — LIVE_STATE (verifier inspects a running process or in-memory state)

First sub-classify by reading environment/docker-compose.yaml if present:

• SIDECAR: the service the verifier inspects is in docker-compose.yaml as a separate service (e.g., webapp:, db:, localstack:, loadgen:). It persists across handoff.
• AGENT_STATE: the service runs inside the agent container (started by Dockerfile ENTRYPOINT, entrypoint.sh, or test.sh).
• MIXED: both.

SIDECAR conversion: usually the easiest case.

1. [verifier] environment_mode = "separate".
2. Ensure the verifier container can reach the sidecar by service hostname (Harbor handles this if the sidecar is on the same docker-compose network — verify).
3. Declare any small agent output files the verifier also reads (e.g., a flag file).
4. No instruction changes.

AGENT_STATE conversion: this is where you stop and think. Two options:

1. Restructure to multi-container. Move the service from the agent's environment/Dockerfile ENTRYPOINT to a new sidecar in environment/docker-compose.yaml. The agent operates against it as a client. The configured state persists in a Docker volume across handoff. The verifier reconnects via service hostname.
   • This works when the agent's interaction model is already client-side (REST/CLI/protocol calls to the service).
   • This may NOT work cleanly when the agent's interaction model assumes shell access to the service container (e.g., editing /etc/foo.conf, running systemctl, attaching debuggers).
2. Re-launch in verifier from agent's config. If the agent's deliverable is config/code that produces the live system, declare those as artifacts; the verifier brings up the service from them in its own image.
   • This changes the test from "agent operates the system" to "agent ships config and the verifier reproduces" — which may be a meaningful semantic shift. Ask the user/PR author before committing to this.

If option 1 is viable and clean, do it. If unsure, post a PR comment with the two options and the trade-offs. Wait.

MIXED conversion: usually same as SIDECAR for the sidecar parts plus AGENT_STATE for the rest. Often the simplest fix is to ensure all measurement completes during the agent phase and the verifier just reads stored results from the sidecar.

Verification after conversion

After making the edits, re-read end-to-end and confirm:

1. task.toml is valid TOML; the [verifier] block contains environment_mode = "separate" and an appropriate artifacts list.
2. Every file path the verifier opens is reachable: in artifacts, in /tests/, in the verifier image (Dockerfile additions), or via a persistent sidecar connection.
3. The agent's instruction does not mention any path or behavior that no longer matches the new setup. The added manifest sentence(s) are concise and scoped to the task's actual stack.
4. solution/solve.sh still produces a passing trajectory: it writes the same artifacts the verifier expects AND (if you added a manifest requirement) writes the manifest file(s) itself, so the oracle keeps scoring 1.0.
5. The canary comment is present at the top of every modified text file.
6. Byte-diff any duplicated assets between agent and verifier. When the task ships shared scripts/data/fixtures in both environment/ and tests/, run diff -q environment/<path> tests/<path> (or diff -qr for directories) and confirm they byte-match — unless the divergence is intentional and documented in verification_explanation (e.g., the verifier intentionally lacks the ground-truth image the agent must reverse-engineer). Also confirm pinned external dependencies match across the two Dockerfiles (FONT_REV, model SHAs, base image, package versions). Silent drift here — one copy updated, the other forgotten — produces correctness bugs that no other check catches.

Then run the full local check pipeline per the downstream benchmark's CONTRIBUTING.md (Running checks locally section). All four are mandatory on every conversion, no exceptions. Do not skip them because the change "looks trivial" or "is just a one-line mode flip" — silent failures in this conversion (TOML scoping, missing artifact parent dirs, forgotten COPY . /tests/) are invisible to re-reading and only surface when the oracle actually runs. A "minimal edit" is exactly the case where you're most likely to skip and most likely to miss a bug.

# Static checks (run all of them on the task directory)
for check in ci_checks/check-*.sh; do bash "$check" tasks/<task-name>; done

# Rubric review (LLM-judge against rubrics/task-implementation.toml)
harbor check tasks/<task-name> -m anthropic/claude-opus-4-8 -r rubrics/task-implementation.toml -o /tmp/rubric.json

# Oracle agent (must score 1.0)
harbor run -p tasks/<task-name> -a oracle

# Nop agent (must score < 1.0 — confirms the verifier isn't a no-op)
harbor run -p tasks/<task-name> -a nop

Of these, the oracle run is the load-bearing one — it's what actually proves the agent→verifier handoff is wired correctly. If you only have time for one, run that one. But run all four; they're cheap relative to the cost of reporting a broken conversion as done.

If any check, rubric criterion, or the oracle fails, fix it before declaring done. If a failure looks like it's caused by the conversion itself rather than a pre-existing issue (e.g., the oracle now drops to 0.0 because an artifact path is wrong), do not paper over it — go back and fix the root cause.

Output

Report to the user:

• Which bucket the task fell in.
• The exact set of edits made (file paths + brief description).
• Anything you were uncertain about that you asked on the PR (link the comment).
• A short paragraph explaining what the agent must now produce that they didn't have to before (if anything). Keep it tight.

Do not commit, push, or open a PR unless the user explicitly asks. Stop after the changes are made and verified.

When the user does ask you to push, also post a comment on the PR via gh pr comment <PR> --body "<text>" with a concise but complete summary of the conversion:

• Which bucket and one-line rationale.
• Edits made, by file (e.g., task.toml: added [verifier] environment="separate" and artifacts = [...]; instruction.md: added one-sentence requirements.txt requirement; solution/solve.sh: now writes requirements.txt).
• Any major design decisions or workarounds — especially anything where you chose between multiple viable approaches (e.g., "moved the oracle blob from /root/data in the agent image to /tests/ rather than redesigning the anti-cheat") so the reviewer can sanity-check the choice.
• Local verification results (static checks, rubric review, oracle, nop) — just a one-liner per check.

Keep the comment scannable. Headings or bullets are fine; paragraphs are not.

The separate verifier label is applied automatically by the Auto-label separate verifier workflow (.github/workflows/auto-label-separate-verifier.yml) whenever a PR's task.toml sets environment_mode = "separate" under [verifier]. You do not need to add or remove the label manually — pushing the converted task.toml is sufficient. If the label doesn't appear within a minute of the push, check the workflow run for that PR (or confirm the downstream benchmark has both the workflow and the separate verifier label defined).

When the conversion taught you something new

If during this conversion you encountered a case the skill didn't cover — an out-of-bucket pattern, a workaround you had to invent, a gotcha that wasn't documented, a category of task that needs different handling — open a separate PR into harbor-framework/benchmark-template updating this skill file (downstream repos pull template changes via the template remote). Keep the update concise: add a sentence, a bullet, or at most a short subsection that captures what future agents need to know. Do not rewrite the skill. Branch name: skill/convert-separate-verifier-learning-<short-slug>. PR title: convert-separate-verifier: <one-line learning>. PR body: link to the original conversion PR + a few sentences on the new case and how the skill addition addresses it.

Do this only when the learning generalizes. A one-off quirk specific to a single task does not warrant a skill update — that goes in the original conversion PR's comment.