Remove round filter from wait-for-state entirely

Follow-up to 1.0.8's --min-round patch. Investigation with
jeehut@fline.dev (DojoKame author) concluded that the round filter
adds no safety at all — both status-field resets already prevent any
stale match:

- Generator signalling ready-for-eval writes evaluatorStatus=pending.
  So a later evaluatorStatus==done can only be THIS round's verdict.
- Evaluator's verdict write does not touch generatorStatus (it stays
  at ready-for-eval). When the Generator starts its next round it
  writes generatorStatus=working first, then ready-for-eval for the
  new round. So the next generatorStatus==ready-for-eval after an
  Evaluator verdict is always a NEW round's signal.

With the resets, field-value watchers are race-free without a round
gate. Keeping a round gate on top only surfaces exact-match or
stale-read bugs of the kind 1.0.8 was trying to work around.

Drops `--round` and `--min-round` flags. Using either now prints an
error pointing at the new semantics. Skill docs updated for both
Generator (post-round) and Evaluator (next-round + completion)
watchers — all three invocations simplify to the same pattern
`wait-for-state.sh MISSION FIELD VALUE`.

Bumps version to 1.0.9.

Author: Jeehut

.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "tandemkit",
-  "version": "1.0.8",
+  "version": "1.0.9",
   "description": "Describe your goal, approve the spec, then step away — Claude and Codex loop together until it's right.",
   "author": {
     "name": "Cihat Gündüz",

scripts/wait-for-state.sh

@@ -10,45 +10,43 @@ unset _TC _NV
 # TandemKit — wait-for-state
 # Blocks until a State.json field matches an expected value (or any change occurs).
 #
-# Usage: wait-for-state.sh <mission-folder> [<field> <value1> [value2 ...]] [--quiet] [--round N] [--min-round N]
+# Usage: wait-for-state.sh <mission-folder> [<field> <value1> [value2 ...]] [--quiet]
 #   mission-folder:  absolute path to the TandemKit/NNN-MissionName/ folder
 #   field:           JSON field to check (e.g., evaluatorStatus, generatorStatus)
 #   value1..N:       acceptable values — script exits when field matches any of them
 #   --quiet:         suppress all output except final READY line (for Codex)
-#   --round N:       also require State.json "round" field to equal N exactly
-#   --min-round N:   also require State.json "round" field to be >= N (catches up if the
-#                    other agent raced ahead past N between wake events). Use this from the
-#                    Evaluator side when the Generator runs autonomously (Feedback-06 mode).
-#                    Mutually exclusive with --round.
 #
 # If no field/values given: blocks until State.json content changes (any field).
 #
+# NOTE: There is no round filter. Both agents reset the other side's status field
+# at the start of each round (Generator writes evaluatorStatus=pending when
+# signalling ready-for-eval; Evaluator's verdict write keeps generatorStatus=
+# ready-for-eval but the next round-start flips generatorStatus=working first),
+# so a field-value match can never fire on a stale value from a previous round.
+# Dropping the round filter removes an exact-match race condition that used to
+# hang watchers when the counterparty raced ahead past the expected round number.
+#
 # Exit 0 = condition met. Output includes current state summary.
 # Exit 1 = error (missing files, bad args).
 
 MISSION_DIR="$1"
 shift
 
-# Check for --quiet, --round, --min-round flags (can appear anywhere in remaining args)
+# Check for --quiet flag (can appear anywhere in remaining args).
 QUIET=false
-REQUIRED_ROUND=""
-MIN_ROUND=""
 _ARGS=()
 while [[ $# -gt 0 ]]; do
    case "$1" in
       --quiet) QUIET=true; shift ;;
-      --round) REQUIRED_ROUND="$2"; shift 2 ;;
-      --min-round) MIN_ROUND="$2"; shift 2 ;;
+      --round|--min-round)
+         echo "ERROR: $1 has been removed in TandemKit 1.0.9 — watchers now rely on status-field resets, not round numbers. Drop this flag." >&2
+         exit 1
+         ;;
       *) _ARGS+=("$1"); shift ;;
    esac
 done
 set -- "${_ARGS[@]+"${_ARGS[@]}"}"
 
-if [[ -n "$REQUIRED_ROUND" && -n "$MIN_ROUND" ]]; then
-   echo "ERROR: --round and --min-round are mutually exclusive" >&2
-   exit 1
-fi
-
 STATE_FILE="$MISSION_DIR/State.json"
 
 if [[ ! -d "$MISSION_DIR" ]]; then
@@ -91,14 +89,6 @@ with open('$STATE_FILE') as f:
 " 2>/dev/null || echo "unknown"
 }
 
-read_round() {
-   python3 -c "
-import json
-with open('$STATE_FILE') as f:
-   print(json.load(f).get('round', 0))
-" 2>/dev/null || echo "0"
-}
-
 file_hash() {
    md5 -q "$STATE_FILE" 2>/dev/null || md5sum "$STATE_FILE" 2>/dev/null | cut -d' ' -f1 || echo "none"
 }
@@ -110,29 +100,6 @@ check_condition() {
       return 1
    fi
 
-   # Check round constraint first (if specified)
-   if [[ -n "$REQUIRED_ROUND" ]]; then
-      local actual_round
-      actual_round=$(read_round)
-      if [[ "$actual_round" != "$REQUIRED_ROUND" ]]; then
-         return 1
-      fi
-   fi
-
-   # Check min-round constraint (>=). Useful when the other agent may race past N
-   # between our wake events — fires as soon as round has reached N or later.
-   if [[ -n "$MIN_ROUND" ]]; then
-      local actual_round
-      actual_round=$(read_round)
-      # Numeric comparison; guard against non-integer output by defaulting to 0.
-      if ! [[ "$actual_round" =~ ^[0-9]+$ ]]; then
-         actual_round=0
-      fi
-      if (( actual_round < MIN_ROUND )); then
-         return 1
-      fi
-   fi
-
    local current
    current=$(read_field)
 
@@ -168,10 +135,7 @@ INITIAL_HASH=$(file_hash)
 # Not ready — enter watch loop
 if [[ "$QUIET" != true ]]; then
    if [[ -n "$FIELD" ]]; then
-      _round_note=""
-      [[ -n "$REQUIRED_ROUND" ]] && _round_note=" (round == $REQUIRED_ROUND)"
-      [[ -n "$MIN_ROUND" ]] && _round_note=" (round >= $MIN_ROUND)"
-      echo "WAITING: $FIELD not yet ${VALUES[*]:-non-null}$_round_note. Watching $STATE_FILE..."
+      echo "WAITING: $FIELD not yet ${VALUES[*]:-non-null}. Watching $STATE_FILE..."
    else
       echo "WAITING: Watching $STATE_FILE for any change..."
    fi

skills/evaluator/SKILL.md

@@ -253,12 +253,18 @@ The user invokes this skill with `/tandemkit:evaluator NNN-MissionName`. First r
 
 After writing your verdict, IMMEDIATELY start TWO background watchers:
 
-1. **Next round watcher (use `--min-round`, NOT `--round`):**
+1. **Next round watcher:**
 ```bash
-bash "$HOME/.claude/plugins/cache/FlineDev/tandemkit/latest/scripts/wait-for-state.sh" "$(pwd)/TandemKit/NNN-MissionName" generatorStatus ready-for-eval --min-round N+1
+bash "$HOME/.claude/plugins/cache/FlineDev/tandemkit/latest/scripts/wait-for-state.sh" "$(pwd)/TandemKit/NNN-MissionName" generatorStatus ready-for-eval
 ```
 
-Why `--min-round` and not `--round`: if the Generator is running autonomously (never-stop mode), it may advance past round N+1 while you are still writing your verdict. `--round N+1` is exact-match and would miss a Generator that already raced to round N+2 or N+3 — the condition window on round=N+1 may only have existed for milliseconds between the Generator's writes. `--min-round N+1` fires as soon as the Generator has reached at least N+1 AND is in `ready-for-eval`, which is the semantic you actually want: "wake me up when there is at least one new round to evaluate".
+No round filter is needed. Your verdict write leaves `generatorStatus` at its
+value from the Generator's ready-for-eval signal; the Generator's next round-
+start flips it to `"working"` first (or sometimes straight to `"ready-for-eval"`
+again for the new round). Either way, the next time `generatorStatus ==
+ready-for-eval` is the next round's signal, regardless of how many rounds the
+Generator has burned through since your last wake. Read `round` from
+`State.json` at wake time to know which `Round-NN.md` to evaluate.
 
 2. **Completion watcher:**
 ```bash

skills/generator/SKILL.md

@@ -81,9 +81,9 @@ The user invokes this skill with `/tandemkit:generator NNN-MissionName`. First r
 6. **Signal the Evaluator**: Update State.json — `generatorStatus: "ready-for-eval"`, `evaluatorStatus: "pending"`, `phase: "evaluation"`, `round: N`. Read-modify-write only your fields.
 7. **Wait for evaluation**: Use `wait-for-state.sh`:
    ```bash
-   bash "$HOME/.claude/plugins/cache/FlineDev/tandemkit/latest/scripts/wait-for-state.sh" "$(pwd)/TandemKit/NNN-MissionName" evaluatorStatus done --round N
+   bash "$HOME/.claude/plugins/cache/FlineDev/tandemkit/latest/scripts/wait-for-state.sh" "$(pwd)/TandemKit/NNN-MissionName" evaluatorStatus done
    ```
-   Run with `run_in_background: true`. When `evaluatorStatus` is `"done"` and round matches, read `Evaluator/Round-NN.md`.
+   Run with `run_in_background: true`. When `evaluatorStatus` flips to `"done"`, read `Evaluator/Round-NN.md` where `N` is whatever `round` is in State.json at that moment. No round filter is needed — step 6 reset `evaluatorStatus` to `"pending"` when signalling, so the next `"done"` is always your round's verdict.
 
 ════════════════════════════════════════
   → DONE — Waiting for Evaluator
@@ -213,10 +213,10 @@ If the user says "abort": confirm, set State.json `phase: "abandoned"`, Config.j
 Use `wait-for-state.sh` for ALL State.json watching. Do NOT use raw watchman-wait.
 
 ```bash
-bash "$HOME/.claude/plugins/cache/FlineDev/tandemkit/latest/scripts/wait-for-state.sh" "$(pwd)/TandemKit/NNN-MissionName" evaluatorStatus done --round N
+bash "$HOME/.claude/plugins/cache/FlineDev/tandemkit/latest/scripts/wait-for-state.sh" "$(pwd)/TandemKit/NNN-MissionName" evaluatorStatus done
 ```
 
-Run with `run_in_background: true`. The script checks immediately, then enters a watch loop. When it prints "READY", re-read State.json. The `--round N` parameter ensures you don't match stale values from a previous round.
+Run with `run_in_background: true`. The script checks immediately, then enters a watch loop. When it prints "READY", re-read State.json. No round filter — the status-field reset at step 6 guarantees the next `done` is your round's verdict.
 
 ## File Reading Limits