Skip to content

Discord: add exec approval cleanup option#13205

Merged
thewilloftheshadow merged 3 commits intomainfrom
shadow/issue-13085-discord-approval-cleanup
Feb 10, 2026
Merged

Discord: add exec approval cleanup option#13205
thewilloftheshadow merged 3 commits intomainfrom
shadow/issue-13085-discord-approval-cleanup

Conversation

@thewilloftheshadow
Copy link
Copy Markdown
Member

@thewilloftheshadow thewilloftheshadow commented Feb 10, 2026

Closes #13085

Greptile Overview

Greptile Summary

Adds a new Discord exec-approvals config option (cleanupAfterResolve) that, when enabled, deletes exec approval DM messages after they’re approved/denied or time out. The option is plumbed through the Discord config types, Zod schema validation, and the exec-approvals handler logic, with docs updated accordingly.

Confidence Score: 4/5

  • Safe to merge after fixing a minor release-notes issue
  • The code changes are small and the new config option is correctly wired through types, schema, docs, and runtime behavior; the only verified issue is an incorrect PR reference in CHANGELOG.md.
  • CHANGELOG.md

@openclaw-barnacle openclaw-barnacle bot added docs Improvements or additions to documentation channel: discord Channel integration: discord maintainer Maintainer-authored PR labels Feb 10, 2026
@thewilloftheshadow thewilloftheshadow self-assigned this Feb 10, 2026
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 file reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

CHANGELOG.md Outdated
Comment on lines +20 to +22
### Fixes

- Discord: add exec approval cleanup option to delete DMs after approval/denial/timeout. (#13085) Thanks @thewilloftheshadow.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Incorrect PR reference

CHANGELOG.md lists this change as (#13085), but this review is for PR #13205. Please update the changelog entry to reference the correct PR number so release notes link to the right change.

Prompt To Fix With AI
This is a comment left during a code review.
Path: CHANGELOG.md
Line: 20:22

Comment:
**Incorrect PR reference**

`CHANGELOG.md` lists this change as `(#13085)`, but this review is for PR #13205. Please update the changelog entry to reference the correct PR number so release notes link to the right change.

How can I resolve this? If you propose a fix, please make it concise.

@thewilloftheshadow thewilloftheshadow force-pushed the shadow/issue-13085-discord-approval-cleanup branch from 15e53dd to cd24c01 Compare February 10, 2026 06:33
@thewilloftheshadow thewilloftheshadow merged commit 8ff1618 into main Feb 10, 2026
21 of 24 checks passed
@thewilloftheshadow thewilloftheshadow deleted the shadow/issue-13085-discord-approval-cleanup branch February 10, 2026 06:39
YanHaidao added a commit to YanHaidao/clawdbot that referenced this pull request Feb 10, 2026
* 'main' of github.com:YanHaidao/clawdbot: (94 commits)
  fix(auto-reply): prevent sender spoofing in group prompts
  Discord: add exec approval cleanup option (openclaw#13205)
  CI: extend stale timelines to be contributor-friendly (openclaw#13209)
  fix: enforce Discord agent component DM auth (openclaw#11254) (thanks @thedudeabidesai)
  refactor(security,config): split oversized files (openclaw#13182)
  Commands: add commands.allowFrom config
  CI: configure stale automation
  fix(signal): enforce mention gating for group messages (openclaw#13124)
  fix(ui): prioritize displayName over label in webchat session picker (openclaw#13108)
  Chore: add testflight auto-response
  Docker: include A2UI sources for bundle (openclaw#13114)
  fix: unify session maintenance and cron run pruning (openclaw#13083)
  docs: expand vulnerability reporting guidelines in SECURITY.md
  docs: add vulnerability reporting guidelines to CONTRIBUTING.md
  refactor: consolidate fetchWithTimeout into shared utility
  fix(memory): default batch embeddings to off
  Improve code analyzer for independent packages, CI: only run release-check on push to main
  fix(tools): correct Grok response parsing for xAI Responses API (openclaw#13049)
  chore(deps): update dependencies, remove hono pinning
  Update contributing, deduplicate more functions
  ...
Hansen1018 pushed a commit to Hansen1018/openclaw that referenced this pull request Feb 10, 2026
michaelleone pushed a commit to michaelleone/openclaw that referenced this pull request Feb 11, 2026
skyhawk14 pushed a commit to skyhawk14/openclaw that referenced this pull request Feb 13, 2026
zooqueen pushed a commit to hanzoai/bot that referenced this pull request Mar 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

channel: discord Channel integration: discord docs Improvements or additions to documentation maintainer Maintainer-authored PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Auto-cleanup Discord approval messages after resolution

2 participants