openclaw - 💡(How to fix) Fix feat(exec): add optional description field for intent-based hook审批 [1 participants]

TL;DR

Add an optional description field to the exec tool schema to enable intent-based approval flows in hooks.

Guidance

• Update the exec tool schema to include the optional description field as proposed.
• Modify hook scripts to utilize the new description field for smarter approval decisions, such as the provided isDestructive function example.
• Test the updated hook scripts with various description values to ensure correct approval flow behavior.
• Consider adding documentation for the new description field to inform users of its purpose and usage.

Example

{
  "command": "rm -rf ./node_modules",
  "description": "Clean install dependencies"
}

And the corresponding hook approval logic:

const isDestructive = desc => [delete, remove, drop, clean].some(k => desc.includes(k));
if (isDestructive(description)) {
  // prompt user for confirmation
}

Notes

The proposed solution is fully backward-compatible, as the description field is optional. However, the effectiveness of the intent-based approval flows relies on users providing accurate and descriptive values for the description field.

Recommendation

Apply the proposed workaround by adding the description field to the exec tool schema, as it enables smarter hooks and safer defaults without introducing significant changes or dependencies.