Store upgrade version in context instead of displaying immediately

- AuthCommand returns SUCCESS instead of FAILURE when no authentication configured
- CheckForUpdates pipeline step now stores available upgrade version in SessionContext instead of displaying it via prompt
- Updated tests to verify context state instead of prompt output

Author: inxilpro

app/Commands/AuthCommand.php

@@ -38,9 +38,9 @@ protected function showStatus(AuthManager $auth): int
 			warning('No authentication configured.');
 			note('Run `clave auth` to set up a Claude Code token.');
 
-			return self::FAILURE;
+			return self::SUCCESS;
 		}
-		
+
 		$method_label = match ($info['method']) {
 			'api_key' => 'API Key',
 			'oauth' => 'OAuth Token',

tests/Feature/Pipelines/Steps/CheckForUpdatesTest.php

@@ -4,7 +4,6 @@
 use App\Pipelines\Steps\CheckForUpdates;
 use Illuminate\Http\Client\ConnectionException;
 use Illuminate\Support\Facades\Http;
-use Laravel\Prompts\Prompt;
 
 function makeUpdateContext(): SessionContext
 {
@@ -24,7 +23,6 @@ function makeUpdateContext(): SessionContext
 		]),
 	]);
 
-	Prompt::fake();
 
 	$step = app(CheckForUpdates::class);
 	$context = makeUpdateContext();
@@ -38,20 +36,18 @@ function makeUpdateContext(): SessionContext
 
 	expect($next_called)->toBeTrue();
 	expect($result)->toBe($context);
-
-	Prompt::assertStrippedOutputDoesntContain('new version of Clave is available');
+	expect($context->upgrade_version_available)->toBeNull();
 
 	Http::assertSentCount(1);
 });
 
-test('shows update notice when newer version is available', function() {
+test('stores upgrade version when newer version is available', function() {
 	Http::fake([
 		'api.github.com/repos/glhd/clave/releases/latest' => Http::response([
 			'tag_name' => 'v99.99.99',
 		]),
 	]);
 
-	Prompt::fake();
 
 	$step = app(CheckForUpdates::class);
 	$context = makeUpdateContext();
@@ -65,18 +61,14 @@ function makeUpdateContext(): SessionContext
 
 	expect($next_called)->toBeTrue();
 	expect($result)->toBe($context);
-
-	Prompt::assertStrippedOutputContains('new version of Clave is available');
-	Prompt::assertStrippedOutputContains('v99.99.99');
+	expect($context->upgrade_version_available)->toBe('99.99.99');
 
 	Http::assertSentCount(1);
 });
 
 test('passes through silently when the HTTP request fails', function() {
 	Http::fake(fn() => throw new ConnectionException('Connection timed out'));
 
-	Prompt::fake();
-
 	$step = app(CheckForUpdates::class);
 	$context = makeUpdateContext();
 
@@ -89,8 +81,7 @@ function makeUpdateContext(): SessionContext
 
 	expect($next_called)->toBeTrue();
 	expect($result)->toBe($context);
-
-	Prompt::assertStrippedOutputDoesntContain('new version of Clave is available');
+	expect($context->upgrade_version_available)->toBeNull();
 });
 
 test('passes through silently when response is missing tag_name', function() {
@@ -100,7 +91,6 @@ function makeUpdateContext(): SessionContext
 		]),
 	]);
 
-	Prompt::fake();
 
 	$step = app(CheckForUpdates::class);
 	$context = makeUpdateContext();
@@ -114,8 +104,7 @@ function makeUpdateContext(): SessionContext
 
 	expect($next_called)->toBeTrue();
 	expect($result)->toBe($context);
-
-	Prompt::assertStrippedOutputDoesntContain('new version of Clave is available');
+	expect($context->upgrade_version_available)->toBeNull();
 
 	Http::assertSentCount(1);
 });