[gomminjae]

문제

TokenStore.saveAccessToken이 KeychainStorage.save의 반환값(성공 여부)을 버렸다. Keychain 저장이 실패하면:

• 인메모리엔 토큰이 남아 이번 세션은 로그인 상태처럼 보임
• 하지만 다음 실행 때 Keychain에 없어 조용히 로그아웃

fix/stability-hardening(#107)이 save를 -> Bool로 바꿔뒀지만, 콜러가 그 값을 안 썼다. 이 PR이 콜러까지 연결한다.

변경

• TokenStorageProtocol.saveAccessToken → @discardableResult -> Bool
• TokenStore: 저장 성공 시에만 인메모리 반영 → 세션 상태가 실제 저장과 어긋나지 않음. 로그아웃/삭제는 보안상 인메모리 항상 클리어
• LoginError.tokenPersistenceFailed 추가
• AuthRepositoryImpl.login/signup: 저장 실패 시 throw → UI가 로그인 실패로 처리·재시도 가능. (login은 저장 로직을 do/catch 밖으로 빼 에러가 networkError로 삼켜지지 않게)

검증

• tuist generate + xcodebuild build-for-testing 성공
• AuthTests 통과

🤖 Generated with Claude Code

[gemini-code-assist]

Code Review

This pull request updates the token storage mechanism to return a boolean indicating success or failure when saving or deleting an access token. It also updates the authentication repository to handle token persistence failures by throwing a new tokenPersistenceFailed error. The review feedback suggests refactoring the saveAccessToken method in TokenStore.swift to use a guard let statement instead of if let, which reduces nesting and improves code readability in accordance with Swift style guidelines.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

[gemini-code-assist]

if let token과 else 블록을 사용하는 대신, guard let token else 패턴을 사용하면 중첩(nesting)을 줄이고 "Happy Path"(토큰 저장 성공 흐름)를 함수의 최상위 레벨에 유지할 수 있어 Swift 스타일 가이드에 더 부합하며 가독성이 향상됩니다.

Suggested change

	if let token {
	KeychainStorage.save(token, key: Key.accessToken)
	let saved = KeychainStorage.save(token, key: Key.accessToken)
	if saved {
	lock.withLock { state in
	state.token = token
	state.loaded = true
	}
	}
	return saved
	} else {
	KeychainStorage.delete(key: Key.accessToken)
	let deleted = KeychainStorage.delete(key: Key.accessToken)
	// 로그아웃/삭제 시 보안상 인메모리는 항상 비운다 (Keychain 삭제 실패와 무관).
	lock.withLock { state in
	state.token = nil
	state.loaded = true
	}
	return deleted
	}
	guard let token else {
	let deleted = KeychainStorage.delete(key: Key.accessToken)
	// 로그아웃/삭제 시 보안상 인메모리는 항상 비운다 (Keychain 삭제 실패와 무관).
	lock.withLock { state in
	state.token = nil
	state.loaded = true
	}
	return deleted
	}
	let saved = KeychainStorage.save(token, key: Key.accessToken)
	if saved {
	lock.withLock { state in
	state.token = token
	state.loaded = true
	}
	}
	return saved

References

1. Swift API Design Guidelines 및 일반적인 Swift 스타일 가이드에서는 조기 탈출(early exit)을 위해 guard 문을 사용하고, 함수의 주요 흐름(happy path)을 최소한의 들여쓰기로 유지하는 것을 권장합니다. ^(link)