fix: Set TerminalSkill.HandlesExecution to false to prevent direct LLM command execution.
This commit is contained in:
@@ -8,7 +8,7 @@ public class TerminalSkill : ISkill
|
|||||||
public string Description => "Translates an intent into a bash command and runs it in the background.";
|
public string Description => "Translates an intent into a bash command and runs it in the background.";
|
||||||
public string[] Hotwords => new[] { "System terminal", "System command" };
|
public string[] Hotwords => new[] { "System terminal", "System command" };
|
||||||
|
|
||||||
public bool HandlesExecution => true;
|
public bool HandlesExecution => false;
|
||||||
|
|
||||||
public string GetSystemPrompt(string rawTranscript)
|
public string GetSystemPrompt(string rawTranscript)
|
||||||
{
|
{
|
||||||
@@ -19,6 +19,8 @@ Output ONLY the raw bash command to achieve this task. Do not include markdown f
|
|||||||
|
|
||||||
public void Execute(string llmResult)
|
public void Execute(string llmResult)
|
||||||
{
|
{
|
||||||
|
// HandlesExecution is false because we are not retarded enough
|
||||||
|
// to let the LLM execute commands directly
|
||||||
try
|
try
|
||||||
{
|
{
|
||||||
Console.WriteLine($"[TerminalSkill] Executing: {llmResult}");
|
Console.WriteLine($"[TerminalSkill] Executing: {llmResult}");
|
||||||
|
|||||||
Reference in New Issue
Block a user