Skip to content

Help embed shows one random marital-wisdom quip instead of the full list - #8

Merged
CommunityPoke merged 1 commit into
mainfrom
devin/help-random-wisdom
Oct 18, 2026
Merged

CommunityPoke merged 1 commit into
mainfrom
devin/help-random-wisdom

Conversation

@devin-ai-integration

Copy link
Copy Markdown
Contributor

Summary

!matr help (and bare !matr) previously dumped all 30 _HELP_QUIPS into the embed description under a "Marital wisdom" section. Each help invocation now picks exactly one quip at random and appends it as a single Discord subtext line (-# <quip>), keeping the embed short while the pool stays in the code.

  • _help_embed(prefix, cmd, rng=None) picks (rng or random).choice(_HELP_QUIPS) and renders **Marital wisdom**\n-# {quip} at the bottom of the description; the command listing is unchanged. The optional rng makes tests deterministic.
  • All three call sites (matr group callback, matr help, on_command_error's unknown-action path) share the behaviour.
  • Tests updated: exactly one -# line per invocation from the pool, seeded RNG picks deterministically, different seeds can yield different quips, embed stays under 4096 chars, !matr and !matr help behave identically.
  • README's "Marital wisdom" bullet updated to describe the single random quip.

Link to Devin session: https://app.devin.ai/sessions/2a733256c8ef43ee8e45d295a9db3c9e
Open in Devin Desktop: https://app.devin.ai/desktop/session/2a733256c8ef43ee8e45d295a9db3c9e?variant=devin
Requested by: @CommunityPoke

…l quip dump

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

  • Disable automatic comment, CI, and merge conflict monitoring

@CommunityPoke
CommunityPoke merged commit 985f50e into main Oct 18, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant