VersionControl:PRs/magenta/434

From PickiPedia: A knowledge base of bluegrass, old time psychedelic jams, and other public domain music
Revision as of 16:06, 9 November 2026 by DrivingThatTrain (talk | contribs) (cryptograss/magenta#434, as GitHub has it)
Jump to navigationJump to search
magenta#434: Clips: Enter sends a fully typed clip name; /clip works for people
State 🟢 open
Worked on by magent, justin
Moods #clips
Deploy redeploy maybelle
Branch magent/clip-enter-sound → main, by magent-cryptograss
Opened · last pushed 2026-11-08 17:06:27 · 2026-11-08 17:06:13
Justin couldn't send a message that was only a clip name by pressing Enter.

**Why:** once the name you've typed matches a clip suggestion exactly, the list stays open. Enter then "accepted" that suggestion, which put back the same text and reopened the list, so the message never went out. Now, if the highlighted suggestion is exactly what's already typed, Enter closes the list and sends.

Also, people can now type `/clip <name>` as well as `/clips <name>`. Emergents could already use both (`AGENT_PLAY`). Before this, a person's `/clip <name>` was posted as plain text.

Not changed: the sound. I downloaded `whats-the-meaning-of-this` from `/clips/c4c2f446…/`, the same media the Mood's player uses. ffprobe shows an AAC stereo track, and volumedetect gives a mean of −21 dB, mostly centre (speech-like). So the cut has sound, and the cause is somewhere else.

Tests: added `test_a_person_may_say_clip_too`. `test_clips_library` and `test_own_clips` pass (20 tests, sqlite). I haven't run the full suite.

Deploy: redeploy maybelle
Worked on by: magent and justin, in #clips