login: drop persistLogin's unused errW parameter

6c240a0→main·

toothbrush·1mo ago·1 file·+3 added/-3 removed

With the legacy dual-write's warn-and-continue gone, persistLogin never writes to stderr.

Co-Authored-By: Claude Fable 5 noreply@anthropic.com

Sessions

b7c0ca0b79e1View transcript

[?
Auth Refactor: Eliminate Static FallbacksClaude Code·Fable 5.1m·3 steps

Changes

1

• cmd/entire/cli

• Mlogin.go+3/-3

194 unmodified lines

195
196
197
198
198
199
200
201
110 unmodified lines

312
313
314
315
315
316
317
318
319
320
321
321
322
323
324

194 unmodified lines

return fmt.Errorf("complete login: %w", err)
    
    return persistLogin(outW, errW, client.BaseURL(), token, refreshToken)
    return persistLogin(outW, client.BaseURL(), token, refreshToken)
}

// loginFlowFacts carries the environment facts that pick between the
110 unmodified lines

return fmt.Errorf("complete login: %w", err)
    
    return persistLogin(outW, errW, baseURL, token, refreshToken)
    return persistLogin(outW, baseURL, token, refreshToken)
}

// persistLogin validates the freshly-issued access token and records it in
// the shared contexts.json credential model. Shared by the device-code and
// browser flows.
func persistLogin(outW, errW io.Writer, baseURL, token, refreshToken string) error {
func persistLogin(outW io.Writer, baseURL, token, refreshToken string) error {
    if err := validateReceivedToken(token, baseURL, time.Now()); err != nil {
        return fmt.Errorf("reject login token: %w", err)
    }