diff --git a/CHANGELOG.md b/CHANGELOG.md index aa4f552..e839066 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -6,6 +6,9 @@ Changes prior to the next official version change will appear here. * bugfix in find_symbol tool (a bug fixed in LS) * merged the two overview tools (for dir and file) int a single one * one-click setup for Cline enabled + * Fix `CheckOnboardingPerformedTool`: + * Tool description was incompatible with project change + * Returned result was not as useful as it could be (now added list of memories) * Language Servers: * Add further file extensions considered by the language servers for Python (.pyi), JavaScript (.jsx) and TypeScript (.tsx, .jsx) diff --git a/src/serena/agent.py b/src/serena/agent.py index e13b1af..8082f6a 100644 --- a/src/serena/agent.py +++ b/src/serena/agent.py @@ -958,15 +958,13 @@ class InsertAtLineTool(Tool): class CheckOnboardingPerformedTool(Tool): """ - Checks whether the onboarding was already performed. + Checks whether project onboarding was already performed. """ def apply(self) -> str: """ - Check if onboarding was performed yet. - You should always call this tool in the beginning of the conversation, - before any question about code or the project is asked. - You will call this tool only once per conversation. + Checks whether project onboarding was already performed. + You should always call this tool before beginning to actually work on a project/after activating a project. """ list_memories_tool = self.agent.get_tool(ListMemoriesTool) memories = json.loads(list_memories_tool.apply()) @@ -976,7 +974,7 @@ class CheckOnboardingPerformedTool(Tool): + "You should perform onboarding by calling the `onboarding` tool before proceeding with the task." ) else: - return "Onboarding already performed, no need to perform it again." + return json.dumps({"result": "Onboarding already performed.", "available_memories": memories}) class OnboardingTool(Tool):