land: configure-walk-event-test

Merged
AF
1h ago
+59-42

Accepted agent delivery; branch-keyed search descriptor included.


Summary by cubic

Replaces the sleep-based timing in the configure-before-walk test with an event handshake: the test hook now waits for a release file (capped by a delay) instead of sleeping unconditionally, and the test releases the walk only after checking the configure reply. Adds a harness benchmark slice descriptor for the event-driven walk test.

RetriggerView in GreptileConfidence Score: 5/5

The PR appears safe to merge; no actionable issues were found.

Summary

Replaces the configure test’s short timing check with a file-based hold.

  • The configure test holds the walk until its reply arrives.

No actionable issues found. Review was based on code inspection; tests were not run.

Diagram

sequenceDiagram
    participant Test
    participant AFT
    participant Walk
    Test->>AFT: Send configure on a thread
    AFT->>Walk: Start background walk
    Walk-->>Test: Write start signal
    Note over Walk: Wait for release file or time cap
    AFT-->>Test: Configure reply
    Test->>Test: Check success
    Test->>Walk: Write release file
    Walk->>Walk: Continue

Reviews (2) · Last reviewed commit: "release notes and search-quality descrip..." · Reviewed by Greptile