indent 0.1.13__tar.gz → 0.1.15__tar.gz

{indent-0.1.13 → indent-0.1.15}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: indent
-Version: 0.1.13
+Version: 0.1.15
 Summary: Indent is an AI Pair Programmer
 Author-email: Sashank Thupukari <sashank@exponent.run>
 Requires-Python: <3.13,>=3.10

{indent-0.1.13 → indent-0.1.15}/exponent/__init__.py

@@ -28,7 +28,7 @@ version_tuple: VERSION_TUPLE
 commit_id: COMMIT_ID
 __commit_id__: COMMIT_ID
 
-__version__ = version = '0.1.13'
-__version_tuple__ = version_tuple = (0, 1, 13)
+__version__ = version = '0.1.15'
+__version_tuple__ = version_tuple = (0, 1, 15)
 
 __commit_id__ = commit_id = None

{indent-0.1.13 → indent-0.1.15}/exponent/commands/common.py

@@ -132,13 +132,13 @@ async def check_inside_git_repo(settings: Settings) -> None:
             )
         )
         click.echo(
-            "This is a check to make sure you are running Exponent from the root of your project."
+            "This is a check to make sure you are running Indent from the root of your project."
         )
 
         click.echo(f"\nCurrent directory: {click.style(os.getcwd(), fg='cyan')}")
 
         click.echo("\nRecommendation:")
-        click.echo("  Run Exponent from the root directory of your codebase.")
+        click.echo("  Run Indent from the root directory of your codebase.")
         click.echo("\nExample:")
         click.echo(
             f"  If your project is in {click.style('~/my-project', fg='cyan')}, run:"
@@ -147,7 +147,7 @@ async def check_inside_git_repo(settings: Settings) -> None:
 
         # Tell the user they can run exponent config --no-git-warning to disable this check
         click.echo(
-            f"\nYou can run {click.style('exponent config --set-git-warning-disabled', fg='green')} to disable this check."
+            f"\nYou can run {click.style('indent config --set-git-warning-disabled', fg='green')} to disable this check."
         )
 
         if not click.confirm(
@@ -165,26 +165,26 @@ def check_running_from_home_directory(require_confirmation: bool = True) -> bool
     if os.path.expanduser("~") == os.getcwd():
         click.echo(
             click.style(
-                "\nWarning: Running Exponent from Home Directory",
+                "\nWarning: Running Indent from Home Directory",
                 fg="yellow",
                 bold=True,
             )
         )
         click.echo(
-            "Running Exponent from your home directory can cause unexpected issues."
+            "Running Indent from your home directory can cause unexpected issues."
         )
         click.echo("\nRecommendation:")
-        click.echo("  Run Exponent from the root directory of your codebase.")
+        click.echo("  Run Indent from the root directory of your codebase.")
         click.echo("\nExample:")
         click.echo(
             f"  If your project is in {click.style('~/my-project', fg='cyan')}, run:"
         )
-        click.echo(f"  {click.style('cd ~/my-project && exponent run', fg='green')}")
+        click.echo(f"  {click.style('cd ~/my-project && indent run', fg='green')}")
 
         if require_confirmation:
             if not click.confirm(
                 click.style(
-                    f"\nDo you want to continue running Exponent from {os.getcwd()}?",
+                    f"\nDo you want to continue running indent from {os.getcwd()}?",
                     fg="yellow",
                 ),
                 default=True,
@@ -214,7 +214,7 @@ def run_until_complete(coro: Coroutine[Any, Any, Any]) -> Any:
     except ExponentError as e:
         click.secho(f"Encountered error: {e}", fg="red")
         click.secho(
-            "The Exponent team has been notified, "
+            "The Indent team has been notified, "
             "please try again and reach out if the problem persists.",
             fg="yellow",
         )
@@ -392,7 +392,7 @@ async def set_login_complete(api_key: str, base_api_url: str, base_ws_url: str)
         # than the one used in the user's request...
         # This should never happen
         raise HandledExponentError(
-            "Invalid API key, login to https://exponent.run to find your API key."
+            "Invalid API key, login to https://indent.com to find your API key."
         )
 
 

{indent-0.1.13 → indent-0.1.15}/exponent/commands/run_commands.py

@@ -97,6 +97,10 @@ def run(
         create_chat(api_key, base_api_url, base_ws_url, ChatSource.CLI_RUN)
     )
 
+    if timeout_seconds is not None and timeout_seconds <= 0:
+        click.secho("Error: --timeout-seconds must be a positive integer", fg="red")
+        sys.exit(1)
+
     if chat_uuid is None:
         sys.exit(1)
 

{indent-0.1.13 → indent-0.1.15}/exponent/core/remote_execution/cli_rpc_types.py

@@ -2,6 +2,7 @@ from typing import TYPE_CHECKING, Any
 
 import msgspec
 import yaml
+from pydantic_ai.format_as_xml import format_as_xml
 
 if TYPE_CHECKING:
     from exponent_server.core.tools.edit_tool import (
@@ -28,6 +29,7 @@ class ToolInput(msgspec.Struct, tag_field="tool_name", omit_defaults=True):
         return msgspec.to_builtins(self)  # type: ignore[no-any-return]
 
     def to_text(self) -> str:
+        """This text is purely used for human debugging, it's not fed to the llm"""
         d = msgspec.to_builtins(self)
         del d["tool_name"]
         return yaml.dump(d)
@@ -41,7 +43,7 @@ class ToolResult(msgspec.Struct, tag_field="tool_name", omit_defaults=True):
         This provides a default textual representation of the tool result. Override it as needed for your tool."""
         d = msgspec.to_builtins(self)
         del d["tool_name"]
-        return yaml.dump(d)
+        return format_as_xml(d, include_root_tag=False, item_tag="item")
 
 
 class ErrorToolResult(ToolResult, tag="error"):

{indent-0.1.13 → indent-0.1.15}/exponent/core/remote_execution/client.py

@@ -5,7 +5,7 @@ import json
 import logging
 import time
 import uuid
-from collections.abc import AsyncGenerator, Callable, Generator
+from collections.abc import AsyncGenerator, Callable, Coroutine, Generator
 from contextlib import asynccontextmanager
 from dataclasses import dataclass
 from typing import Any, TypeVar, cast
@@ -30,6 +30,7 @@ from exponent.core.remote_execution.cli_rpc_types import (
     CliRpcRequest,
     CliRpcResponse,
     ErrorResponse,
+    ErrorToolResult,
     GetAllFilesRequest,
     GetAllFilesResponse,
     HttpRequest,
@@ -63,10 +64,10 @@ from exponent.core.remote_execution.types import (
     CreateChatResponse,
     GitInfo,
     HeartbeatInfo,
-    PrReviewWorkflowInput,
     RemoteExecutionResponseType,
     RunWorkflowRequest,
     StreamingCodeExecutionRequest,
+    WorkflowInput,
     WorkflowTriggerRequest,
     WorkflowTriggerResponse,
 )
@@ -182,6 +183,8 @@ class RemoteExecutionClient:
         """Handle an incoming websocket message.
         Returns None to continue processing, or a REMOTE_EXECUTION_CLIENT_EXIT_INFO to exit."""
 
+        self._last_request_time = time.time()
+
         msg_data = json.loads(msg)
         if msg_data["type"] != "request":
             return None
@@ -544,7 +547,7 @@ class RemoteExecutionClient:
         return cast(dict[str, Any], response.json())
 
     async def trigger_workflow(
-        self, workflow_name: str, workflow_input: PrReviewWorkflowInput
+        self, workflow_name: str, workflow_input: WorkflowInput
     ) -> WorkflowTriggerResponse:
         response = await self.api_client.post(
             "/api/remote_execution/trigger_workflow",
@@ -609,33 +612,36 @@ class RemoteExecutionClient:
                     response=GetAllFilesResponse(files=files),
                 )
             elif isinstance(request.request, BatchToolExecutionRequest):
-                results: list[ToolResultType] = []
+                coros: list[Coroutine[Any, Any, ToolResultType]] = []
                 for tool_input in request.request.tool_inputs:
-                    try:
-                        if isinstance(tool_input, BashToolInput):
-                            raw_result = await execute_bash_tool(
+                    if isinstance(tool_input, BashToolInput):
+                        coros.append(
+                            execute_bash_tool(
                                 tool_input,
                                 self.working_directory,
                                 should_halt=self.get_halt_check(request.request_id),
                             )
-                        else:
-                            raw_result = await execute_tool(  # type: ignore[assignment]
-                                tool_input, self.working_directory
-                            )
-                        tool_result = truncate_result(raw_result)
-                        results.append(tool_result)
-                    except Exception as e:
-                        logger.error(f"Error executing tool {tool_input}: {e}")
-                        from exponent.core.remote_execution.cli_rpc_types import (
-                            ErrorToolResult,
                         )
+                    else:
+                        coros.append(execute_tool(tool_input, self.working_directory))
 
-                        results.append(ErrorToolResult(error_message=str(e)))
+                results: list[ToolResultType | BaseException] = await asyncio.gather(
+                    *coros, return_exceptions=True
+                )
+
+                processed_results: list[ToolResultType] = []
+                for result in results:
+                    if not isinstance(result, BaseException):
+                        processed_results.append(truncate_result(result))
+                    else:
+                        processed_results.append(
+                            ErrorToolResult(error_message=str(result))
+                        )
 
                 return CliRpcResponse(
                     request_id=request.request_id,
                     response=BatchToolExecutionResponse(
-                        tool_results=results,
+                        tool_results=processed_results,
                     ),
                 )
             elif isinstance(request.request, HttpRequest):

indent-0.1.15/exponent/core/remote_execution/tool_type_utils.py

@@ -0,0 +1,39 @@
+from typing import Any
+
+import msgspec
+
+from exponent.core.remote_execution.cli_rpc_types import ToolResult
+
+
+def to_mostly_xml(tool_result: ToolResult) -> str:
+    """
+    This provides a default textual representation of the tool result. Override it as needed for your tool."""
+    d = msgspec.to_builtins(tool_result)
+    del d["tool_name"]
+    return to_mostly_xml_helper(d)
+
+
+def to_mostly_xml_helper(
+    d: Any,
+) -> str:
+    if isinstance(d, dict):
+        # No outer wrapper at top level, each field gets XML tags
+        parts = []
+        for key, value in d.items():
+            if isinstance(value, list):
+                # Handle lists with item tags
+                list_items = "\n".join(
+                    f"<item>\n{to_mostly_xml_helper(item)}\n</item>" for item in value
+                )
+                parts.append(f"<{key}>\n{list_items}\n</{key}>")
+            elif isinstance(value, dict):
+                # Nested dict
+                parts.append(f"<{key}>\n{to_mostly_xml_helper(value)}\n</{key}>")
+            else:
+                # Scalar value
+                parts.append(f"<{key}>\n{value!s}\n</{key}>")
+        return "\n".join(parts)
+    elif isinstance(d, list):
+        raise ValueError("Lists are not allowed at the top level")
+    else:
+        return str(d)

{indent-0.1.13 → indent-0.1.15}/exponent/core/remote_execution/types.py

@@ -49,9 +49,17 @@ class PrReviewWorkflowInput(BaseModel):
     pr_number: int
 
 
+class SlackWorkflowInput(BaseModel):
+    channel_id: str
+    thread_ts: str
+
+
+WorkflowInput = PrReviewWorkflowInput | SlackWorkflowInput
+
+
 class WorkflowTriggerRequest(BaseModel):
     workflow_name: str
-    workflow_input: PrReviewWorkflowInput
+    workflow_input: WorkflowInput
 
 
 class WorkflowTriggerResponse(BaseModel):
@@ -509,6 +517,10 @@ class ChatMode(str, Enum):
     DATABASE = "DATABASE"  # chat with database connection
     WORKFLOW = "WORKFLOW"
 
+    @classmethod
+    def requires_cli(cls, mode: "ChatMode") -> bool:
+        return mode not in [cls.DATABASE]
+
 
 class ChatSource(str, Enum):
     CLI_SHELL = "CLI_SHELL"
@@ -516,6 +528,7 @@ class ChatSource(str, Enum):
     WEB = "WEB"
     DESKTOP_APP = "DESKTOP_APP"
     VSCODE_EXTENSION = "VSCODE_EXTENSION"
+    SLACK_APP = "SLACK_APP"
 
 
 class CLIConnectedState(BaseModel):

{indent-0.1.13 → indent-0.1.15}/exponent/utils/version.py

@@ -135,8 +135,8 @@ def _get_upgrade_command_str(version: str) -> str:
 
 def _new_version_str(current_version: str, new_version: str) -> str:
     return (
-        f"\n{click.style('A new Exponent version is available:', fg='cyan')} {new_version} (current: {current_version})\n"
-        f"See {click.style('https://docs.exponent.run/installation', underline=True)} for details.\n"
+        f"\n{click.style('A new Indent version is available:', fg='cyan')} {new_version} (current: {current_version})\n"
+        f"See {click.style('https://docs.indent.com/help_and_resources/troubleshooting#installation-methods', underline=True)} for details.\n"
     )
 
 
@@ -196,15 +196,15 @@ def upgrade_exponent(
 
     if result.returncode != 0:
         click.secho(
-            "\nFailed to upgrade Exponent. See https://docs.exponent.run/installation for help, or reach out to team@exponent.run.",
+            "\nFailed to upgrade Indent. See https://docs.indent.com/help_and_resources/troubleshooting#installation-methods for help, or reach out to team@indent.com",
             fg="red",
         )
         sys.exit(2)
 
-    click.secho(f"Successfully upgraded Exponent to version {new_version}!", fg="green")
+    click.secho(f"Successfully upgraded Indent to version {new_version}!", fg="green")
 
-    click.echo("Re-run exponent to use the latest version.")
-    sys.exit(1)
+    click.echo("Re-run indent to use the latest version.")
+    sys.exit(0)
 
 
 def _upgrade_thread_worker(