Skip to content

Human Review Protocol R2 design: separate curation, transition feasibility, and execution #162

Description

@nulleimy

Status

DESIGN ONLY — no implementation authorization in this issue.

Source blocker: #159
Canonical checkpoint: feature/bundle-0-bootstrap @ f3292d6ac3db81e9d741eedfd2467ecc90c933c0

Goal

Replace the experimentally contaminated Human DJ Review R1 calibration path with a versioned R2 protocol that cleanly separates what APPLAYLIST chooses from how a human DJ happens to execute a mix.

Core invariant:

CURATION_REVIEW != TRANSITION_FEASIBILITY_REVIEW != HUMAN_EXECUTION_REVIEW

Historical Bundle 63 / R1 evidence remains immutable. R2 is a new protocol/schema, not an in-place reinterpretation of R1.

Proposed R2 evidence model

A. CurationReviewR2 — primary calibration truth for Competitive Curation

Evaluates the ordered set/segment only. No live transition execution is required.

Required dimensions, each 1..5:

  • energy_flow
  • dramaturgical_fit
  • set_coherence
  • alternative_usefulness

Required case-level fields:

  • curation_preference = plan_a | plan_b | tie | abstain
  • curation_confidence = 0..1
  • optional reason codes / note
  • observed timestamp
  • algorithm identity hidden

The reviewer instruction must explicitly say: judge track choice, ordering, progression and usefulness; do not use how well you personally executed a mix as the basis for curation preference.

Bundle 67 / Competitive Curation calibration may consume only curation_preference, never an overall mixed preference.

B. TransitionFeasibilityReviewR2 — separate APPLAYLIST transition-quality evidence

Evaluates a proposed transition plan, not arbitrary live DJ execution.

A transition is assessable only when the review artifact binds a deterministic TransitionReviewSpec, including at minimum:

  • outgoing track/segment identity
  • incoming track/segment identity
  • proposed outgoing mix window
  • proposed incoming mix window
  • transition duration or bars
  • tempo handling / target BPM when relevant
  • transition strategy
  • explicit evidence refs

Optional technique fields such as EQ/FX/stems/loop policy may be absent only if the dimension being rated does not depend on them.

Candidate dimensions:

  • phrase_window_quality
  • energy_handoff
  • spectral_compatibility
  • tempo_key_feasibility
  • transition_strategy_suitability
  • vocal_collision_risk only when explicit vocal evidence exists

Each dimension uses:

  • numeric rating when assessable, or
  • explicit not_assessable with reason code.

No forced score is allowed for missing vocal/phrase/spectral evidence.

Transition preference, if collected, must be separate:

  • transition_preference = plan_a | plan_b | tie | abstain | not_assessable

C. HumanExecutionReviewR2 — optional separate experiment

Measures the DJ's actual performance, not optimizer/curation truth.

Execution ratings are valid only if one of these is true:

  1. the exact execution recipe is standardized and bound to the review, or
  2. the research question explicitly studies free-form human execution.

Possible execution dimensions:

  • execution_smoothness
  • execution_phrase_accuracy
  • execution_energy_control

These values MUST NOT calibrate Bundle 67 Competitive Curation.

Preference isolation rules

  1. Remove the concept of a single mixed preference from R2.
  2. curation_preference is the only human target for Bundle 67 curation calibration.
  3. transition_preference is reserved for a future transition challenger/transition-planning model.
  4. execution outcome cannot alter curation calibration labels.
  5. ties and abstains remain first-class values.
  6. not_assessable remains distinct from tie/abstain.

R1 compatibility / migration

  • R1 rows and ledgers remain immutable historical evidence.
  • No R1 transition_smoothness or phrase_alignment score may be reclassified as R2 transition feasibility without a bound TransitionReviewSpec.
  • Existing qualitative track-selection notes may be retained as qualitative curation evidence only.
  • Existing unfilled 12-case CSV must not be auto-filled or coerced into R2.
  • New R2 reviewer artifacts receive new schema/protocol IDs and separate append-only ledgers/reports.

Calibration R3 design implications

Bundle 68 R2 calibration must be superseded for future real evidence by a curation-only calibration path that:

  • accepts complete CurationReviewR2 evidence;
  • does not require transition/execution dimensions;
  • binds blind assignment -> source plan/path exactly;
  • computes agreement, decisive agreement, tie behavior, abstain handling, confidence weighting, set-role coverage;
  • cannot read/use transition or execution preference fields even if present elsewhere;
  • cannot authorize optimizer activation.

Proposed tester acceptance properties BEFORE implementation

The Skill Tester must attempt to falsify this design against at least these attacks:

  1. Same track order, excellent DJ execution vs poor DJ execution -> curation label MUST remain identical.
  2. Missing transition spec -> transition feasibility MUST become not_assessable, never fabricated numeric evidence.
  3. Missing explicit vocal evidence -> vocal collision MUST be not_assessable.
  4. Reviewer prefers A only because they personally mixed A better -> that must not be representable as curation truth without an explicit curation judgment.
  5. Transition A better, curation B better -> protocol must preserve both without collapsing into one winner.
  6. All four curation dimensions complete but transition dimensions absent -> curation calibration must still be valid.
  7. Complete transition evidence but curation abstain -> no curation accuracy contribution.
  8. Legacy R1 submission -> cannot be silently accepted as R2.
  9. Duplicate/replayed review identity -> fail closed.
  10. Algorithm identity exposed -> blinded calibration invalid.
  11. A/B slot inversion -> deterministic source identity resolution must remain correct.
  12. Qualitative note only -> retained as note, never converted to score.
  13. HumanExecutionReview changes -> must not change curation report hash/verdict.
  14. Transition review changes -> must not change curation report hash/verdict.
  15. Curation review changes -> should change curation calibration deterministically.

Design gate

No implementation branch should be opened until a separate Skill Tester adversarial review of this proposal returns no unresolved HIGH/CRITICAL experimental-validity findings.

Authority

IMPLEMENTATION_AUTHORIZATION=NO
MERGE_AUTHORIZATION=NO
RELEASE_AUTHORIZATION=NO
DEPLOY_AUTHORIZATION=NO
PRODUCTION_ACTIVATION=NO
OPTIMIZER_RANKING_ACTIVATION=NO
PDM_TRAINING=NO

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions