Address comments

src/snowflake/cli/_plugins/connection/util.py

@@ -79,6 +79,8 @@ def get_ui_parameters(conn: SnowflakeConnection) -> Dict[UIParameter, Any]:
 
     parameters_to_fetch = [param.value for param in UIParameter]
 
+    # Parsing of the Json and the filtering is happening here in Snowflake CLI
+    # in order to avoid requiring a warehouse in Snowflake
     query = "call system$bootstrap_data_request('CLIENT_PARAMS_INFO')"
     *_, cursor = conn.execute_string(query)
 

src/snowflake/cli/_plugins/nativeapp/sf_sql_facade.py

@@ -18,6 +18,7 @@
 from textwrap import dedent
 from typing import Any, Dict, List
 
+from snowflake.cli._plugins.connection.util import UIParameter, get_ui_parameter
 from snowflake.cli._plugins.nativeapp.constants import (
     AUTHORIZE_TELEMETRY_COL,
     NAME_COL,
@@ -26,8 +27,6 @@
 from snowflake.cli._plugins.nativeapp.same_account_install_method import (
     SameAccountInstallMethod,
 )
-from snowflake.cli._plugins.connection.util import UIParameter, get_ui_parameter
-from snowflake.cli._plugins.nativeapp.constants import SPECIAL_COMMENT
 from snowflake.cli._plugins.nativeapp.sf_facade_constants import UseObjectType
 from snowflake.cli._plugins.nativeapp.sf_facade_exceptions import (
     CREATE_OR_UPGRADE_APPLICATION_EXPECTED_USER_ERROR_CODES,

src/snowflake/cli/api/config.py

@@ -286,10 +286,10 @@ def get_config_value(*path, key: str, default: Optional[Any] = Empty) -> Any:
         raise
 
 
-def get_config_bool_value(*path, key: str, default: Any = Empty) -> bool | None:
+def get_config_bool_value(*path, key: str, default: Optional[bool]) -> Optional[bool]:
     value = get_config_value(*path, key=key, default=None)
 
-    if value is None and default is not Empty:
+    if value is None:
         return default
 
     try: