Skip to content

Commit 76086d8

Browse files
kylemclarenclaude
andcommitted
Workaround: Skip CLI tests due to sprite auth bug
The sprite CLI has a bug where `EnsureAuthenticated()` in auth_flow.go requires an active user (from `sprite login`) even when SPRITE_TOKEN env var is set. This causes `sprite create` to fail with "no authentication found" even after `sprite auth setup --token`. The bug is that `EnsureAuthenticated()` gates `GetOrgAndClient()` behind `activeUser != nil`, so the SPRITE_TOKEN check in `GetToken()` (added in PR #151) is never reached. This workaround sets SKIP_CLI_TESTS=true which skips test sprite creation but still generates docs from --help output. Related: - CLI auth fix (incomplete): superfly/sprite-env#151 - The fix to GetToken() works, but EnsureAuthenticated() needs updating Co-Authored-By: Claude Opus 4.5 <[email protected]>
1 parent 85d3ad8 commit 76086d8

File tree

1 file changed

+7
-0
lines changed

1 file changed

+7
-0
lines changed

.github/workflows/ci.yml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,14 @@ jobs:
5050
SPRITES_TEST_TOKEN: ${{ secrets.SPRITES_TEST_TOKEN }}
5151

5252
- name: Generate CLI Docs
53+
# WORKAROUND: Skip CLI tests due to auth bug in sprite CLI where
54+
# EnsureAuthenticated() requires an active user even when SPRITE_TOKEN
55+
# is set. See: https://github.com/superfly/sprite-env/pull/151
56+
# The --help output generation still works without auth.
57+
# TODO: Remove SKIP_CLI_TESTS once sprite CLI is fixed.
5358
run: pnpm generate:cli-docs
59+
env:
60+
SKIP_CLI_TESTS: 'true'
5461

5562
- name: Commit generated CLI docs
5663
run: |

0 commit comments

Comments
 (0)