From 7d351e3422ba1aaa6687466c00389a6ea2efc4f8 Mon Sep 17 00:00:00 2001 From: Jacob <240170694+wignerStan@users.noreply.github.com> Date: Fri, 19 Dec 2025 02:05:08 +0800 Subject: [PATCH] fix: Remove duplicate LINEAR_API_KEY check and consolidate imports Addresses CodeRabbit review comment: - Consolidated split imports from core.auth - Removed unreachable duplicate LINEAR_API_KEY validation --- auto-claude/integrations/linear/updater.py | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/auto-claude/integrations/linear/updater.py b/auto-claude/integrations/linear/updater.py index cca8b805..d102642f 100644 --- a/auto-claude/integrations/linear/updater.py +++ b/auto-claude/integrations/linear/updater.py @@ -113,13 +113,13 @@ def _create_linear_client() -> ClaudeSDKClient: Create a minimal Claude client with only Linear MCP tools. Used for focused mini-agent calls. """ - from core.auth import get_sdk_env_vars, require_auth_token + from core.auth import ( + ensure_claude_code_oauth_token, + get_sdk_env_vars, + require_auth_token, + ) require_auth_token() # Raises ValueError if no token found - - # Ensure SDK can find the token - from core.auth import ensure_claude_code_oauth_token - ensure_claude_code_oauth_token() linear_api_key = get_linear_api_key() @@ -127,8 +127,6 @@ def _create_linear_client() -> ClaudeSDKClient: raise ValueError("LINEAR_API_KEY not set") sdk_env = get_sdk_env_vars() - if not linear_api_key: - raise ValueError("LINEAR_API_KEY not set") return ClaudeSDKClient( options=ClaudeAgentOptions(