indent 0.1.7__tar.gz → 0.1.8__tar.gz

{indent-0.1.7 → indent-0.1.8}/.gitignore

@@ -9,10 +9,11 @@ __pycache__/
 .env.production
 .env.*_db
 .gcp-sa-credentials.json
-.indent-gh-app.pem
+*.pem
 workflow_input.json
 .anthropic.log.json
 CLAUDE.local.md
+indent.local.md
 
 # C extensions
 *.so

{indent-0.1.7 → indent-0.1.8}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: indent
-Version: 0.1.7
+Version: 0.1.8
 Summary: Indent is an AI Pair Programmer
 Author-email: Sashank Thupukari <sashank@exponent.run>
 Requires-Python: <3.13,>=3.10
@@ -15,7 +15,7 @@ Requires-Dist: eval-type-backport<0.3,>=0.2.0
 Requires-Dist: git-python>=1.0.3
 Requires-Dist: gitignore-parser<0.2,>=0.1.11
 Requires-Dist: gql[httpx,websockets]<4,>=3.5.0
-Requires-Dist: httpx<0.28,>=0.27.0
+Requires-Dist: httpx>=0.28.1
 Requires-Dist: ipykernel<7,>=6.29.4
 Requires-Dist: jupyter-client<9,>=8.6.1
 Requires-Dist: msgspec>=0.19.0

indent-0.1.8/exponent/__init__.py

@@ -0,0 +1 @@
+__version__ = "0.1.8"  # Keep in sync with pyproject.toml

{indent-0.1.7 → indent-0.1.8}/exponent/core/graphql/queries.py

@@ -15,6 +15,19 @@ EVENTS_FOR_CHAT_QUERY: str = """query EventsForChat($chatUuid: UUID!) {
             ... on TextMessage {
               text
             }
+            ... on ToolCallMessage {
+              messageId
+              toolUseId
+              toolName
+              toolInput {
+                ... on BashToolInput {
+                  command
+                }
+                ... on ReadToolInput {
+                  filePath
+                }
+              }
+            }
             ... on ToolResultMessage {
               messageId
               toolUseId

{indent-0.1.7 → indent-0.1.8}/exponent/core/graphql/subscriptions.py

@@ -76,6 +76,19 @@ INDENT_EVENTS_SUBSCRIPTION = """
                 content
               }
             }
+          }
+           ... on ToolCallMessage {
+            messageId
+            toolUseId
+            toolName
+            toolInput {
+              ... on BashToolInput {
+                command
+              }
+              ... on ReadToolInput {
+                filePath
+              }
+            }
           }
           ... on PartialToolResultMessage {
             messageId

{indent-0.1.7 → indent-0.1.8}/exponent/core/remote_execution/cli_rpc_types.py

@@ -1,5 +1,6 @@
 import msgspec
 import yaml
+from typing import Any
 
 
 class PartialToolResult(msgspec.Struct, tag_field="tool_name", omit_defaults=True):
@@ -9,6 +10,17 @@ class PartialToolResult(msgspec.Struct, tag_field="tool_name", omit_defaults=Tru
 class ToolInput(msgspec.Struct, tag_field="tool_name", omit_defaults=True):
     """Concrete subclasses describe the full input schema for a tool."""
 
+    def to_llm(self) -> dict[str, Any]:
+        """Convert ToolInput to LLM-friendly typed dict format.
+
+        Returns a dictionary with the tool parameters, excluding the tool_name
+        which is handled separately by the LLM integration layer.
+
+        Returns:
+            self by default, which msgspec will serialize appropriately.
+        """
+        return msgspec.to_builtins(self)  # type: ignore[no-any-return]
+
 
 class ToolResult(msgspec.Struct, tag_field="tool_name", omit_defaults=True):
     """Concrete subclasses return data from a tool execution."""
@@ -107,6 +119,7 @@ class GrepToolResult(ToolResult, tag=GREP_TOOL_NAME):
 EDIT_TOOL_NAME = "edit"
 
 
+# This is only used in the CLI. The server side type is edit_tool.py
 class EditToolInput(ToolInput, tag=EDIT_TOOL_NAME):
     file_path: str
     old_string: str
@@ -139,13 +152,13 @@ class BashToolResult(ToolResult, tag=BASH_TOOL_NAME):
     stopped_by_user: bool
 
 
-
 class HttpRequest(msgspec.Struct, tag="http_fetch_cli"):
     url: str
     method: str = "GET"
     headers: dict[str, str] | None = None
     timeout: int | None = None
 
+
 class HttpResponse(msgspec.Struct, tag="http_fetch_cli"):
     status_code: int | None = None
     content: str | None = None
@@ -153,6 +166,7 @@ class HttpResponse(msgspec.Struct, tag="http_fetch_cli"):
     duration_ms: int | None = None
     headers: dict[str, str] | None = None
 
+
 ToolInputType = (
     ReadToolInput
     | WriteToolInput
@@ -188,6 +202,10 @@ class TerminateRequest(msgspec.Struct, tag="terminate"):
     pass
 
 
+class SwitchCLIChatRequest(msgspec.Struct, tag="switch_cli_chat"):
+    new_chat_uuid: str
+
+
 class BatchToolExecutionRequest(msgspec.Struct, tag="batch_tool_execution"):
     tool_inputs: list[ToolInputType]
 
@@ -204,6 +222,10 @@ class BatchToolExecutionResponse(msgspec.Struct, tag="batch_tool_execution"):
     tool_results: list[ToolResultType]
 
 
+class SwitchCLIChatResponse(msgspec.Struct, tag="switch_cli_chat"):
+    pass
+
+
 class CliRpcRequest(msgspec.Struct):
     request_id: str
     request: (
@@ -212,6 +234,7 @@ class CliRpcRequest(msgspec.Struct):
         | TerminateRequest
         | HttpRequest
         | BatchToolExecutionRequest
+        | SwitchCLIChatRequest
     )
 
 
@@ -232,4 +255,5 @@ class CliRpcResponse(msgspec.Struct):
         | TerminateResponse
         | BatchToolExecutionResponse
         | HttpResponse
+        | SwitchCLIChatResponse
     )

{indent-0.1.7 → indent-0.1.8}/exponent/core/remote_execution/client.py

@@ -36,6 +36,8 @@ from exponent.core.remote_execution.cli_rpc_types import (
     ToolExecutionRequest,
     ToolExecutionResponse,
     ToolResultType,
+    SwitchCLIChatRequest,
+    SwitchCLIChatResponse,
 )
 from exponent.core.remote_execution.code_execution import (
     execute_code_streaming,
@@ -168,6 +170,21 @@ class RemoteExecutionClient:
                 )
             )
             return None
+        elif isinstance(request.request, SwitchCLIChatRequest):
+            await websocket.send(
+                json.dumps(
+                    {
+                        "type": "result",
+                        "data": msgspec.to_builtins(
+                            CliRpcResponse(
+                                request_id=request.request_id,
+                                response=SwitchCLIChatResponse(),
+                            )
+                        ),
+                    }
+                )
+            )
+            return SwitchCLIChat(new_chat_uuid=request.request.new_chat_uuid)
         else:
             if isinstance(request.request, ToolExecutionRequest) and isinstance(
                 request.request.tool_input, BashToolInput
@@ -502,6 +519,11 @@ class RemoteExecutionClient:
                     "TerminateRequest should not be handled by handle_request"
                 )
 
+            elif isinstance(request.request, SwitchCLIChatRequest):
+                raise ValueError(
+                    "SwitchCLIChatRequest should not be handled by handle_request"
+                )
+
             raise ValueError(f"Unhandled request type: {type(request)}")
 
         except Exception as e:

{indent-0.1.7 → indent-0.1.8}/exponent/core/remote_execution/files.py

@@ -10,8 +10,6 @@ from exponent.core.remote_execution.cli_rpc_types import ErrorToolResult, GrepTo
 from exponent.core.remote_execution.types import (
     FileAttachment,
     FilePath,
-    GetAllTrackedFilesRequest,
-    GetAllTrackedFilesResponse,
     GetFileAttachmentRequest,
     GetFileAttachmentResponse,
     GetFileAttachmentsRequest,
@@ -209,16 +207,6 @@ async def get_matching_files(
     )
 
 
-async def get_all_tracked_files(
-    request: GetAllTrackedFilesRequest,
-    working_directory: str,
-) -> GetAllTrackedFilesResponse:
-    return GetAllTrackedFilesResponse(
-        correlation_id=request.correlation_id,
-        files=await get_all_non_ignored_files(working_directory),
-    )
-
-
 async def search_files(
     path_str: str,
     file_pattern: str | None,

{indent-0.1.7 → indent-0.1.8}/exponent/core/remote_execution/http_fetch.py

@@ -19,24 +19,28 @@ DEFAULT_USER_AGENT = "Indent-HTTP-Client/1.0"
 async def fetch_http_content(http_request: HttpRequest) -> HttpResponse:
     """
     Fetch content from an HTTP URL and return the response.
-    
+
     Args:
         http_request: HttpRequest containing URL, method, headers, and timeout
-        
+
     Returns:
         HttpResponse with status code, content, and error message if any
     """
     logger.info(f"Fetching {http_request.method} {http_request.url}")
-    
+
     try:
         # Set up timeout
-        timeout = http_request.timeout if http_request.timeout is not None else DEFAULT_TIMEOUT
-        
+        timeout = (
+            http_request.timeout
+            if http_request.timeout is not None
+            else DEFAULT_TIMEOUT
+        )
+
         # Set up headers with default User-Agent
         headers = http_request.headers or {}
         if "User-Agent" not in headers:
             headers["User-Agent"] = DEFAULT_USER_AGENT
-            
+
         # Create HTTP client with timeout
         async with httpx.AsyncClient(timeout=timeout) as client:
             # Make the HTTP request
@@ -45,23 +49,27 @@ async def fetch_http_content(http_request: HttpRequest) -> HttpResponse:
                 url=http_request.url,
                 headers=headers,
             )
-            
+
             # Get response content as text
             try:
                 content = response.text
             except UnicodeDecodeError:
                 # If content can't be decoded as text, provide a fallback
                 content = f"Binary content ({len(response.content)} bytes)"
-                logger.warning(f"Could not decode response content as text for {http_request.url}")
-            
-            logger.info(f"HTTP {http_request.method} {http_request.url} -> {response.status_code}")
-            
+                logger.warning(
+                    f"Could not decode response content as text for {http_request.url}"
+                )
+
+            logger.info(
+                f"HTTP {http_request.method} {http_request.url} -> {response.status_code}"
+            )
+
             return HttpResponse(
                 status_code=response.status_code,
                 content=content,
                 error_message=None,
             )
-            
+
     except httpx.TimeoutException:
         error_msg = f"Request to {http_request.url} timed out after {timeout} seconds"
         return HttpResponse(
@@ -69,7 +77,7 @@ async def fetch_http_content(http_request: HttpRequest) -> HttpResponse:
             content="",
             error_message=error_msg,
         )
-        
+
     except httpx.RequestError as e:
         error_msg = f"Request error for {http_request.url}: {str(e)}"
         return HttpResponse(
@@ -77,11 +85,11 @@ async def fetch_http_content(http_request: HttpRequest) -> HttpResponse:
             content="",
             error_message=error_msg,
         )
-        
+
     except Exception as e:
         error_msg = f"Unexpected error fetching {http_request.url}: {str(e)}"
         return HttpResponse(
             status_code=None,
             content="",
             error_message=error_msg,
-        )
+        )

indent-0.1.8/exponent/core/remote_execution/system_context.py

@@ -0,0 +1,27 @@
+import getpass
+import os
+import platform
+
+from anyio import Path as AsyncPath
+
+from exponent.core.remote_execution.git import get_git_info
+from exponent.core.remote_execution.languages import python_execution
+from exponent.core.remote_execution.types import (
+    SystemInfo,
+)
+from exponent.core.remote_execution.utils import safe_read_file
+
+
+async def get_system_info(working_directory: str) -> SystemInfo:
+    return SystemInfo(
+        name=getpass.getuser(),
+        cwd=working_directory,
+        os=platform.system(),
+        shell=_get_user_shell(),
+        git=await get_git_info(working_directory),
+        python_env=python_execution.get_python_env_info(),
+    )
+
+
+def _get_user_shell() -> str:
+    return os.environ.get("SHELL", "bash")

{indent-0.1.7 → indent-0.1.8}/exponent/core/remote_execution/types.py

@@ -229,10 +229,6 @@ Namespace = Literal[
     "get_file_attachment",
     "get_file_attachments",
     "get_matching_files",
-    "system_context",
-    "get_all_tracked_files",
-    "halt",
-    "switch_cli_chat",
     "error",
     "create_checkpoint",
     "rollback_to_checkpoint",
@@ -355,27 +351,6 @@ class GetMatchingFilesResponse(RemoteExecutionResponse):
     files: list[RemoteFile]
 
 
-class GetAllTrackedFilesResponse(RemoteExecutionResponse):
-    namespace: ClassVar[Namespace] = "get_all_tracked_files"
-
-    files: list[RemoteFile]
-
-
-class SystemContextResponse(RemoteExecutionResponse):
-    namespace: ClassVar[Namespace] = "system_context"
-
-    exponent_txt: str | None
-    system_info: SystemInfo | None
-
-
-class HaltResponse(RemoteExecutionResponse):
-    namespace: ClassVar[Namespace] = "halt"
-
-
-class SwitchCLIChatResponse(RemoteExecutionResponse):
-    namespace: ClassVar[Namespace] = "switch_cli_chat"
-
-
 class ErrorResponse(RemoteExecutionResponse):
     namespace: ClassVar[Namespace] = "error"
     # The namespace of the request that caused the error.
@@ -460,15 +435,6 @@ class StreamingCodeExecutionRequest(
     timeout: int
 
 
-class HaltRequest(RemoteExecutionRequest[HaltResponse]):
-    namespace: ClassVar[Namespace] = "halt"
-
-
-class SwitchCLIChatRequest(RemoteExecutionRequest[SwitchCLIChatResponse]):
-    namespace: ClassVar[Namespace] = "switch_cli_chat"
-    new_chat_uuid: str
-
-
 class FileWriteRequest(RemoteExecutionRequest[FileWriteResponse]):
     namespace: ClassVar[Namespace] = "file_write"
 
@@ -504,14 +470,6 @@ class GetMatchingFilesRequest(RemoteExecutionRequest[GetMatchingFilesResponse]):
     search_term: str
 
 
-class GetAllTrackedFilesRequest(RemoteExecutionRequest[GetAllTrackedFilesResponse]):
-    namespace: ClassVar[Namespace] = "get_all_tracked_files"
-
-
-class SystemContextRequest(RemoteExecutionRequest[SystemContextResponse]):
-    namespace: ClassVar[Namespace] = "system_context"
-
-
 class CreateCheckpointRequest(RemoteExecutionRequest[CreateCheckpointResponse]):
     namespace: ClassVar[Namespace] = "create_checkpoint"
 
@@ -557,12 +515,8 @@ RemoteExecutionRequestType = Union[
     GetFileAttachmentRequest,
     GetFileAttachmentsRequest,
     GetMatchingFilesRequest,
-    SystemContextRequest,
-    GetAllTrackedFilesRequest,
     CommandRequest,
-    HaltRequest,
     StreamingCodeExecutionRequest,
-    SwitchCLIChatRequest,
     CreateCheckpointRequest,
     RollbackToCheckpointRequest,
 ]
@@ -576,11 +530,7 @@ RemoteExecutionResponseType = Union[
     GetFileAttachmentResponse,
     GetFileAttachmentsResponse,
     GetMatchingFilesResponse,
-    GetAllTrackedFilesResponse,
-    SystemContextResponse,
     CommandResponse,
-    HaltResponse,
-    SwitchCLIChatResponse,
     ErrorResponse,
     CreateCheckpointResponse,
     RollbackToCheckpointResponse,

{indent-0.1.7 → indent-0.1.8}/exponent/core/remote_execution/utils.py

@@ -35,16 +35,12 @@ from exponent.core.remote_execution.types import (
     FilePath,
     FileWriteRequest,
     FileWriteResponse,
-    GetAllTrackedFilesRequest,
-    GetAllTrackedFilesResponse,
     GetFileAttachmentRequest,
     GetFileAttachmentResponse,
     GetFileAttachmentsRequest,
     GetFileAttachmentsResponse,
     GetMatchingFilesRequest,
     GetMatchingFilesResponse,
-    HaltRequest,
-    HaltResponse,
     ListFilesRequest,
     ListFilesResponse,
     RemoteExecutionMessage,
@@ -61,10 +57,6 @@ from exponent.core.remote_execution.types import (
     StreamingCodeExecutionResponse,
     StreamingCodeExecutionResponseChunk,
     SupportedLanguage,
-    SwitchCLIChatRequest,
-    SwitchCLIChatResponse,
-    SystemContextRequest,
-    SystemContextResponse,
 )
 from exponent.core.types.command_data import NaturalEditContent
 from exponent.core.types.event_types import (
@@ -78,82 +70,6 @@ from exponent.utils.version import get_installed_version
 ### Serde
 
 
-def deserialize_message_data(
-    message_data: RemoteExecutionMessageData | str,
-) -> RemoteExecutionMessage:
-    if isinstance(message_data, str):
-        message_data = RemoteExecutionMessageData.model_validate_json(message_data)
-    if message_data.direction == "request":
-        return deserialize_request_data(cast(RemoteExecutionRequestData, message_data))
-    elif message_data.direction == "response":
-        return deserialize_response_data(
-            cast(RemoteExecutionResponseData, message_data)
-        )
-    else:
-        # type checking trick, if you miss a namespace then
-        # this won't typecheck due to the input parameter
-        # having a potential type other than no-return
-        assert_unreachable(message_data.direction)
-
-
-def deserialize_request_data(
-    request_data: RemoteExecutionRequestData | str,
-) -> RemoteExecutionRequestType:
-    request: RemoteExecutionRequestType
-    if isinstance(request_data, str):
-        request_data = RemoteExecutionRequestData.model_validate_json(request_data)
-    if request_data.direction != "request":
-        raise ValueError(f"Expected request, but got {request_data.direction}")
-    if request_data.namespace == "code_execution":
-        request = CodeExecutionRequest.model_validate_json(request_data.message_data)
-    elif request_data.namespace == "file_write":
-        request = FileWriteRequest.model_validate_json(request_data.message_data)
-    elif request_data.namespace == "list_files":
-        request = ListFilesRequest.model_validate_json(request_data.message_data)
-    elif request_data.namespace == "get_file_attachment":
-        request = GetFileAttachmentRequest.model_validate_json(
-            request_data.message_data
-        )
-    elif request_data.namespace == "get_file_attachments":
-        request = GetFileAttachmentsRequest.model_validate_json(
-            request_data.message_data
-        )
-    elif request_data.namespace == "get_matching_files":
-        request = GetMatchingFilesRequest.model_validate_json(request_data.message_data)
-    elif request_data.namespace == "system_context":
-        request = SystemContextRequest.model_validate_json(request_data.message_data)
-    elif request_data.namespace == "get_all_tracked_files":
-        request = GetAllTrackedFilesRequest.model_validate_json(
-            request_data.message_data
-        )
-    elif request_data.namespace == "command":
-        request = CommandRequest.model_validate_json(request_data.message_data)
-    elif request_data.namespace == "halt":
-        request = HaltRequest.model_validate_json(request_data.message_data)
-    elif request_data.namespace == "streaming_code_execution":
-        request = StreamingCodeExecutionRequest.model_validate_json(
-            request_data.message_data
-        )
-    elif request_data.namespace == "switch_cli_chat":
-        request = SwitchCLIChatRequest.model_validate_json(request_data.message_data)
-    elif request_data.namespace == "streaming_code_execution_chunk":
-        assert False, "Streaming code execution chunk is a response, not a request"
-    elif request_data.namespace == "error":
-        assert False, "Error is a response, not a request"
-    elif request_data.namespace == "create_checkpoint":
-        request = CreateCheckpointRequest.model_validate_json(request_data.message_data)
-    elif request_data.namespace == "rollback_to_checkpoint":
-        request = RollbackToCheckpointRequest.model_validate_json(
-            request_data.message_data
-        )
-    else:
-        # type checking trick, if you miss a namespace then
-        # this won't typecheck due to the input parameter
-        # having a potential type other than no-return
-        request = assert_unreachable(request_data.namespace)
-    return truncate_message(request)
-
-
 def deserialize_response_data(
     response_data: RemoteExecutionResponseData | str,
 ) -> RemoteExecutionResponseType:
@@ -188,18 +104,9 @@ def deserialize_response_data(
         response = GetFileAttachmentsResponse.model_validate_json(
             response_data.message_data
         )
-    elif response_data.namespace == "system_context":
-        response = SystemContextResponse.model_validate_json(response_data.message_data)
-    elif response_data.namespace == "get_all_tracked_files":
-        response = GetAllTrackedFilesResponse.model_validate_json(
-            response_data.message_data
-        )
+
     elif response_data.namespace == "command":
         response = CommandResponse.model_validate_json(response_data.message_data)
-    elif response_data.namespace == "halt":
-        response = HaltResponse.model_validate_json(response_data.message_data)
-    elif response_data.namespace == "switch_cli_chat":
-        response = SwitchCLIChatResponse.model_validate_json(response_data.message_data)
     elif response_data.namespace == "error":
         response = ErrorResponse.model_validate_json(response_data.message_data)
     elif response_data.namespace == "create_checkpoint":
@@ -424,10 +331,8 @@ def truncate_message(response: GetMatchingFilesRequest) -> GetMatchingFilesReque
 def truncate_message(
     response: GetMatchingFilesResponse,
 ) -> GetMatchingFilesResponse: ...
-@overload
-def truncate_message(response: SystemContextRequest) -> SystemContextRequest: ...
-@overload
-def truncate_message(response: SystemContextResponse) -> SystemContextResponse: ...
+
+
 @overload
 def truncate_message(
     response: RemoteExecutionRequestType,

{indent-0.1.7 → indent-0.1.8}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "indent"
-version = "0.1.7"
+version = "0.1.8"
 description = "Indent is an AI Pair Programmer"
 authors = [{ name = "Sashank Thupukari", email = "sashank@exponent.run" }]
 requires-python = ">=3.10,<3.13"
@@ -14,7 +14,7 @@ dependencies = [
     "diff-match-patch>=20230430,<20230431",
     "gitignore-parser>=0.1.11,<0.2",
     "gql[httpx, websockets]>=3.5.0,<4",
-    "httpx>=0.27.0,<0.28",
+    "httpx>=0.28.1",
     "ipykernel>=6.29.4,<7",
     "jupyter-client>=8.6.1,<9",
     "packaging~=24.1",
@@ -50,7 +50,7 @@ dev = [
     "pytest>=8.3.3,<9",
     "pytest-asyncio>=0.24.0,<0.25",
     "pytest-cov>=5.0.0,<6",
-    "pytest-httpx>=0.30.0,<0.31",
+    "pytest-httpx>=0.31.0",
     "pytest-xdist>=3.6.1,<4",
     "types-passlib>=1.7.7.20240819,<2",
     "requests>=2.0.0,<3",

indent-0.1.7/exponent/__init__.py

@@ -1 +0,0 @@
-__version__ = "0.1.7"  # Keep in sync with pyproject.toml

indent-0.1.7/exponent/core/remote_execution/system_context.py

@@ -1,54 +0,0 @@
-import getpass
-import os
-import platform
-
-from anyio import Path as AsyncPath
-
-from exponent.core.remote_execution.git import get_git_info
-from exponent.core.remote_execution.languages import python_execution
-from exponent.core.remote_execution.types import (
-    SystemContextRequest,
-    SystemContextResponse,
-    SystemInfo,
-)
-from exponent.core.remote_execution.utils import safe_read_file
-
-EXPONENT_TXT_FILENAMES = [
-    "exponent.txt",
-]
-
-
-async def get_system_context(
-    request: SystemContextRequest, working_directory: str
-) -> SystemContextResponse:
-    return SystemContextResponse(
-        correlation_id=request.correlation_id,
-        exponent_txt=await _read_exponent_txt(working_directory),
-        system_info=await get_system_info(working_directory),
-    )
-
-
-async def get_system_info(working_directory: str) -> SystemInfo:
-    return SystemInfo(
-        name=getpass.getuser(),
-        cwd=working_directory,
-        os=platform.system(),
-        shell=_get_user_shell(),
-        git=await get_git_info(working_directory),
-        python_env=python_execution.get_python_env_info(),
-    )
-
-
-async def _read_exponent_txt(working_directory: str) -> str | None:
-    for filename in EXPONENT_TXT_FILENAMES:
-        file_path = AsyncPath(os.path.join(working_directory, filename.lower()))
-        exists = await file_path.exists()
-
-        if exists:
-            return await safe_read_file(file_path)
-
-    return None
-
-
-def _get_user_shell() -> str:
-    return os.environ.get("SHELL", "bash")