Skip to content

Remove stale src/harness/ stub agent-runner (superseded by modular refactor + upcoming runner) #11

Description

@cmungall

Proposal

Remove the stale src/harness/ stub agent-runner. It was the original sketch of
the skill-invocation harness but was never implemented, and it has been
superseded by the modular src/folds/ work + the agent runner we're about to
build (Step 3).

Why it's dead weight

Remove

src/harness/__init__.py
src/harness/run_skill1.py
src/harness/run_skill2.py
src/harness/run_pipeline.py
src/harness/oracle.py
src/harness/exemplar_selection.py   # (review: exemplar logic may be worth salvaging)

Dependents to handle in the same change

These import src.harness and are themselves NotImplementedError stubs of the
old design — remove them too, or rewrite against the new runner once it exists:

  • experiments/phase_a_crossval.py (from src.harness.run_pipeline/run_skill1/run_skill2/oracle import …)
  • experiments/phase_b_prospective.py (from src.harness.run_pipeline import …)

Explicitly NOT in scope (keep)

  • src/schemas/not orphaned; src/metrics/skill1_metrics.py (CuratorBundle)
    and src/metrics/skill2_structural.py (TargetSchema) use it.
  • src/metrics/ — the scoring side (Step 4). May need updating for the modular
    world later, but that's a separate review, not part of this removal.

Acceptance criteria

  • src/harness/ removed.
  • No remaining imports of src.harness anywhere (experiments removed or rewired).
  • src/folds/ and src/metrics/ unaffected; full test suite still green.
  • If exemplar_selection logic is worth keeping, relocate it (e.g. into the
    new runner) rather than dropping it silently.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Fields

    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions