PR #62057: fix(discord): avoid silent native slash success fallback

• Repository: openclaw/openclaw
• Author: jb510
• State: open | merged: False
• Link: https://github.com/openclaw/openclaw/pull/62057

Description (problem / solution / changelog)

Summary

• unify native Discord builtin slash auth with shared command auth before dispatch
• return an explicit unauthorized response for builtin native slash commands when shared auth denies
• replace the generic native zero-output fallback with an explicit warning instead of ✅ Done.
• add a regression test covering zero-visible-output native slash dispatch

Fixes #58986.

Why

Discord native slash commands could appear to succeed with ✅ Done. even when no visible reply was produced. One real path was auth drift: native preflight allowed the command, deeper shared auth denied it, builtin handlers returned no payload, dispatch returned zero outputs, and Discord emitted a misleading success ack.

This patch makes builtin native slash commands fail explicitly when shared auth denies, and makes the remaining zero-output fallback explicit instead of looking like success.

Testing

Passed

• pnpm vitest run extensions/discord/src/monitor/native-command.plugin-dispatch.test.ts

Attempted but blocked by pre-existing checkout issues

• pnpm test:extension discord
  • failed in unrelated existing Discord tests in this checkout, including exec-approvals.test.ts, monitor.test.ts, and thread-bindings.lifecycle.test.ts
• pnpm test:contracts
  • not reached because pnpm test:extension discord was already red in this tree
• pnpm build && pnpm check
  • build/check path hung in this checkout and did not complete cleanly

AI assistance

This PR was prepared with AI assistance, with changes reviewed and validation results reported transparently.

Changed files

• extensions/discord/src/monitor/native-command.plugin-dispatch.test.ts (modified, +56/-0)
• extensions/discord/src/monitor/native-command.ts (modified, +2/-2)