Fix trail finding stale-any API filter · Entire

Fix trail finding stale-any API filter

bb784c6→main·

dipree·4w ago·2 files·+4 added/-2 removed

Sessions

7a52fa0734fcView transcript

[?
Validate Trail Finding API and CLIPi·GPT-5.5·3 steps](/content/gh/entireio/cli/session/019edc00-dd32-7126-ba30-bb8fc9d0bc0b#timeline-7a52fa0734fc/index.html)

Changes

2

612 unmodified lines

613
614
615
616
616
617
618
619
620
621

612 unmodified lines

if opts.Severity != "" {
        q.Set("severity", opts.Severity)
    }
    if opts.Stale != "" && opts.Stale != trailReviewStaleAny {
        // Unlike status=any (which the API treats as no status filter), stale=any is
        // semantically significant: omitting stale defaults to current-only server-side.
        if opts.Stale != "" {
            q.Set("stale", opts.Stale)
        }
    if opts.IncludeDismissed {

Mcmd/entire/cli/trail_review_cmd.go+3/-1

96 unmodified lines

97
98
99
100
100
101
102
103

96 unmodified lines

Limit:            25,
        Offset:           50,
    })
    want := "/api/v1/trails/trail%20id%2Fwith%20slash/reviews/comments?include_dismissed=true&limit=25&offset=50&severity=high%2Cmedium&status=open%2Cresolved"
    want := "/api/v1/trails/trail%20id%2Fwith%20slash/reviews/comments?include_dismissed=true&limit=25&offset=50&severity=high%2Cmedium&stale=any&status=open%2Cresolved"
    if got != want {
        t.Fatalf("trailReviewCommentsPath = %q, want %q", got, want)
    }

Mcmd/entire/cli/trail_review_cmd_test.go+1/-1