bug: only require CI=true and AccessToken when making API requests by burmudar · Pull Request #1278 · sourcegraph/src-cli

added 5 commits

March 25, 2026 09:58

keegancsmith

@burmudar

- rename `RequireAccessToken` to `checkIfCIAccessTokenRequired`
- provide more context in error on why it happened

@burmudar

@burmudar

@burmudar burmudar deleted the wb/bug-src-token-and-ci branch

March 26, 2026 09:49

burmudar added a commit that referenced this pull request

Mar 26, 2026
…1278)

* check when inCI and whether we required Access Token

* fast fail in APIClient when CI=true and no Access Token

* use apiClient constructor in search_jobs

* fail fast in login / auth token when CI=true and AccessToken not set

* add comment

* review feedback

- rename `RequireAccessToken` to `checkIfCIAccessTokenRequired`
- provide more context in error on why it happened

* use sentinel error

* use lib errors

(cherry picked from commit 0e5e006)

burmudar added a commit that referenced this pull request

Mar 26, 2026
…1278)

* check when inCI and whether we required Access Token

* fast fail in APIClient when CI=true and no Access Token

* use apiClient constructor in search_jobs

* fail fast in login / auth token when CI=true and AccessToken not set

* add comment

* review feedback

- rename `RequireAccessToken` to `checkIfCIAccessTokenRequired`
- provide more context in error on why it happened

* use sentinel error

* use lib errors

(cherry picked from commit 0e5e006)