adjusted some of test shell helperps and updated py version. removed shared context printout.

Author: cagostino

README.md

@@ -6,7 +6,7 @@
 # npcsh
 
 
-- `npcsh` is a python-based command-line tool designed to integrate Large Language Models (LLMs) and Agents into one's daily workflow by making them available and easily configurable through the command line shell.
+- `npcsh` is a python-based AI Agent framework designed to integrate Large Language Models (LLMs) and Agents into one's daily workflow by making them available and easily configurable through a command line shell as well as an extensible python library.
 
 - **Smart Interpreter**: `npcsh` leverages the power of LLMs to understand your natural language commands and questions, executing tasks, answering queries, and providing relevant information from local files and the web.
 
@@ -17,10 +17,9 @@
 
 * **Extensible with Python:**  `npcsh`'s python package provides useful functions for interacting with LLMs, including explicit coverage for popular providers like ollama, anthropic, openai, gemini, deepseek, and openai-like providers. Each macro has a corresponding function and these can be used in python scripts. `npcsh`'s functions are purpose-built to simplify NPC interactions but NPCs are not required for them to work if you don't see the need.
 
-* **Simple, Powerful CLI:**  Use the `npc` CLI commands to set up a flask server so you can expose your NPC team for use as a backend service. You can also use the `npc` CLI to run SQL models defined in your project, execute assembly lines, and verify the integrity of your NPC team's interrelations. `npcsh`'s NPCs take advantage of jinja templating to reference other NPCs and tools in their properties, and the `npc` CLI can be used to verify these references.
-
-* **Shell Strengths:** Execute bash commands directly. Use your favorite command-line tools like VIM, Emacs, ipython, sqlite3, git. Pipe the output of these commands to LLMs or pass LLM results to bash commands.
+* **Simple, Powerful CLI:**  Use the `npc` CLI commands to run `npcsh` macros or commands from one's regular shell. Set up a flask server so you can expose your NPC team for use as a backend service. You can also use the `npc` CLI to run SQL models defined in your project, execute assembly lines, and verify the integrity of your NPC team's interrelations. `npcsh`'s NPCs take advantage of jinja templating to reference other NPCs and tools in their properties, and the `npc` CLI can be used to verify these references.
 
+* **Powerful Tool integrations:** `npcsh` has built-in tools for users to have agents execute code, analyze data, generate images, search the web, and more. Tools can be defined in YAML files as part of project-specific `npc_team`s or in the global `~/.npcsh/npc_team` directory or simply in python scripts. Once compiled, the tools can be used as macros in the `npc` cli as well as `/{tool_name}` commands in the `npcsh` shell.
 
 
 Interested to stay in the loop and to hear the latest and greatest about `npcsh` ? Be sure to sign up for the [npcsh newsletter](https://forms.gle/n1NzQmwjsV4xv1B2A)!

npcsh/llm_funcs.py

@@ -885,7 +885,7 @@ def check_llm_command(
         Relevant shared context for the npc:
         {npc.shared_context}
         """
-        print("shared_context: " + str(npc.shared_context))
+        # print("shared_context: " + str(npc.shared_context))
     # print(prompt)
 
     prompt += f"""

setup.py

@@ -77,7 +77,6 @@ def get_setup_message():
         "librosa",
         "openai",
         "jinja2",
-        "pyautogui",
         "pandas",
         "matplotlib",
         "IPython",
@@ -87,6 +86,7 @@ def get_setup_message():
         "PyYAML",
         "langchain",
         "langchain_community",
+        "pyautogui",
         # "openai-whisper",
         # "pyaudio",
         # "gtts",

tests/test_shell_helpers.py

@@ -69,11 +69,6 @@ def test_db():
     os.unlink(db_path)
 
 
-@pytest.fixture
-def command_history():
-    return CommandHistory()
-
-
 @pytest.fixture
 def npc_compiler():
     # Get the absolute path to the npc_team directory
@@ -85,82 +80,73 @@ def npc_compiler():
     return NPCCompiler(npc_dir, ":memory:")
 
 
-def test_execute_slash_commands(command_history, npc_compiler, test_db):
+def test_execute_slash_commands(npc_compiler, test_db):
     """Test various slash commands"""
 
-    result = execute_command("/help", command_history, test_db, npc_compiler)
+    result = execute_command("/help", test_db, npc_compiler)
     assert "Available Commands" in result["output"]
 
 
-def test_execute_command_with_model_override(command_history, npc_compiler, test_db):
+def test_execute_command_with_model_override(npc_compiler, test_db):
     """Test command execution with model override"""
     result = execute_command(
         "@gpt-4o-mini What is 2+2?",
-        command_history,
         test_db,
         npc_compiler,
     )
     assert result["output"] is not None
 
 
-def test_execute_command_who_was_simon_bolivar(command_history, npc_compiler, test_db):
+def test_execute_command_who_was_simon_bolivar(npc_compiler, test_db):
     """Test the command for querying information about Simón Bolívar."""
     result = execute_command(
         "What country was Simon Bolivar born in?",
-        command_history,
         test_db,
         npc_compiler,
     )
     assert "venezuela" in str(result["output"]).lower()
 
 
-def test_execute_command_capital_of_france(command_history, npc_compiler, test_db):
+def test_execute_command_capital_of_france(npc_compiler, test_db):
     """Test the command for querying the capital of France."""
-    result = execute_command(
-        "What is the capital of France?", command_history, test_db, npc_compiler
-    )
+    result = execute_command("What is the capital of France?", test_db, npc_compiler)
     assert "paris" in str(result["output"]).lower()
 
 
-def test_execute_command_weather_info(command_history, npc_compiler, test_db):
+def test_execute_command_weather_info(npc_compiler, test_db):
     """Test the command for getting weather information."""
     result = execute_command(
-        "what is the weather in Tokyo?", command_history, test_db, npc_compiler
+        "search the web for the weather in Tokyo?", test_db, npc_compiler
     )
     print(result)  # Add print for debugging
     assert "tokyo" in str(result["output"]).lower()
 
 
-def test_execute_command_linked_list_implementation(
-    command_history, npc_compiler, test_db
-):
+def test_execute_command_linked_list_implementation(npc_compiler, test_db):
     """Test the command for querying linked list implementation in Python."""
     result = execute_command(
         " Tell me a way to implement a linked list in Python?",
-        command_history,
         test_db,
         npc_compiler,
     )
     assert "class Node:" in str(result["output"])
     assert "class LinkedList:" in str(result["output"])
 
 
-def test_execute_command_inquiry_with_npcs(command_history, npc_compiler, test_db):
+def test_execute_command_inquiry_with_npcs(npc_compiler, test_db):
     """Test inquiry using NPCs."""
     result = execute_command(
         "/search -p duckduckgo who is the current us president",
-        command_history,
         test_db,
         npc_compiler,
     )
     assert "President" in result["output"]  # Check for presence of expected output
 
 
-def test_execute_command_rag_search(command_history, npc_compiler, test_db):
+def test_execute_command_rag_search(npc_compiler, test_db):
     """Test the command for a RAG search."""
     result = execute_command(
         "/rag -f dummy_linked_list.py linked list",
-        command_history,
         test_db,
         npc_compiler,
     )