Skip to content

tests/run.sh: a long run root truncates the row stamp so test_lane_helpers.sh's clause (c) TAIL case fails locally while CI's plain pytest passes #180

Closed
#215
Closed
tests/run.sh: a long run root truncates the row stamp so test_lane_helpers.sh's clause (c) TAIL case fails locally while CI's plain pytest passes#180
#215

Description

@brettheap
brettheap
opened on Oct 6, 2026

Under tests/run.sh, tests/test_lane_helpers.sh's case "the row's stamp states clause (c)'s two facts ... TAIL" FAILS, while the same suite run by plain python3 -m pytest (which CI uses) passes. Found 2026-10-06 by lane openRepoTools-1's #170 writer at the base of PR #174 (feat/lane-worktrees-report), reproduced locally on Eagle.

Cause as observed: tests/run.sh places the run under a LONG run root (its per-run temp root beneath the checkout-adjacent paths #169 moved it to), and the row stamp written by the lane tooling is CUT before its dir …; window … tail — the stamp is bounded (the state cell's 240-character cap, or the stamp's own width) and the long path consumes the budget, so the clause (c) facts the case asserts are not there to read. Under plain pytest the shorter default temp root leaves room and the tail survives.

Why it matters: tests/run.sh is THE documented way to run the suite locally (AGENTS.md), so a person who follows the document sees a red case that CI never shows, and a case that depends on the length of the temp root is a case whose answer depends on where it is run. Related: #169 (run.sh's temp roots and process group), #77 (the suite's duration; this is the same run root), #137 (run.sh scoping).

Wants: either the suite's estate for that case uses a short, fixed sandbox path independent of the run root, or the row stamp's bounding is made to keep the clause (c) tail ahead of the directory text (truncate the dir, never the facts) — and a note in tests/run.sh or AGENTS.md until then. Unowned; filed so it is not lost across the account swaps of 2026-10-06. Filed by lane openRepoTools-1 at lane openRepoTools-3's request.

Activity

  1. brettheap commented on Oct 6, 2026

    @brettheap
    brettheap
    on Oct 6, 2026
    ContributorAuthor

    Deferred under the close-out direction of 2026-10-06. Diagnosis from the stopped writer: this is a path-length limit, not a run.sh bug per se — lane-start's cut_to_line cuts the row stamp at 230 characters; in the clause (c) case that leaves about 49 characters for TMPDIR. Under tests/run.sh TMPDIR is about 71 characters (~/.local/state/openRepoTools/tmp/<UTC>-<pid>/tmp), and on the macOS CI runner the default TMPDIR is about 49, so the case is already at the edge there. Moving the run root to a short path also requires updating lane-worktrees' sandbox_items, which finds killed runs' roots only under the state directory.

    Lane: openRepoTools-3

  2. brettheap
    added a commit that references this issue on Oct 11, 2026

    Date the T010 record against #192's and #190's landings, correct its …

    22bdb2e

  3. brettheap commented on Oct 11, 2026

    @brettheap
    brettheap
    on Oct 11, 2026
    ContributorAuthor

    CLAIMED — lane openRepoTools-4, session e545e636-93fd-4b80-81b1-48177346b887@Eagle, 2026-10-11T16:46:57Z, for #180

    Logged in lanes/log/openRepoTools-4.md at ae729f9375b133ad04951a4dd6ae864edde0f106.

    The three reads (lane-collision-protocol Rule 1):

    1. existing `CLAIMED —` comments on opensoft/openRepoTools#180:
   none
2. `gh pr list --repo opensoft/openRepoTools --state all --search "#180"`, kept where the row or its body names `#180`:
   none
3. `git ls-remote --heads git@github.com:opensoft/openRepoTools` branches naming `180` as a whole token:
   none
  4. brettheap
    added 5 commits that reference this issue on Oct 11, 2026

    Point the three "no review verdict posted" rows of the T011 record at…

    17dd54f

    Close W22's findings 2-4 on 83bcec7 in the T011 record: date the sect…

    118a2e2

    Seed the row-stamp test's workspace from a data table, and name its c…

    929c31e

    Triad migration evidence of 2026-10-08 and 2026-10-09: test roots and…

    983d8c7

    Read the clause (c) TAIL case on a checkout at a short fixed path, an…

    61a7378
  5. brettheap
    closed this as completedin #215on Oct 18, 2026
  6. brettheap
    added 2 commits that reference this issue on Oct 18, 2026

    T018: verify 88ee9d9: 224 passed, 1 failed (test_the_lane_helper_suit…

    3d0642c

    Merge main (f5c3bcd) into fix/rare-state-instructions-73, so CI tests…

    828d2cf

  7. brettheap commented on Oct 19, 2026

    @brettheap
    brettheap
    on Oct 19, 2026
    ContributorAuthor

    RELEASED — lane openRepoTools-4, session e545e636-93fd-4b80-81b1-48177346b887@Eagle, 2026-10-19T11:58:47Z, for #180 — #180 is closed by PR #215 (61a7378); lane 4's claim ends

  8. brettheap
    added 2 commits that reference this issue on Oct 19, 2026

    Make the lane tools look inside a triad's legs and paired feature tre…

    55aae16

    D's record, fourth touch: lane 3's 231 (the start turn followed a sta…

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions