rom24-quickmud-python 2.9.19__tar.gz → 2.9.20__tar.gz

{rom24_quickmud_python-2.9.19 → rom24_quickmud_python-2.9.20}/CHANGELOG.md

@@ -7,6 +7,11 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [2.9.20]
+
+### Fixed
+- **`do_group` missing TO_VICT / TO_NOTVICT broadcasts on add/remove** (`mud/commands/group_commands.py:do_group`): ROM `src/act_obj.c... src/act_comm.c:1850-1854` (add) emits three messages — `$N joins $n's group.` (TO_NOTVICT, onlookers), `You join $n's group.` (TO_VICT, the joined victim), `$N joins your group.` (TO_CHAR, the leader). The remove path at `:1841-1846` is symmetric: `$n removes $N from $s group.` / `$n removes you from $s group.` / `You remove $N from your group.`. Python returned only the TO_CHAR string, so victims never learned they had been added to (or removed from) a group, and onlookers never saw membership changes. Fix: emit ROM-shaped messages on both branches via `messages.append(...)`, iterating room occupants for TO_NOTVICT. One enforcement test in `tests/integration/test_do_group_notification.py` covering the add path (victim TO_VICT + onlooker TO_NOTVICT + leader/follower state).
+
 ## [2.9.19]
 
 ### Fixed

{rom24_quickmud_python-2.9.19/rom24_quickmud_python.egg-info → rom24_quickmud_python-2.9.20}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: rom24-quickmud-python
-Version: 2.9.19
+Version: 2.9.20
 Summary: A modern Python port of the ROM 2.4b6 MUD engine with full telnet server and JSON world loading
 Author-email: Mark Jedrzejczyk <mark.jedrzejczyk@gmail.com>
 Maintainer-email: Mark Jedrzejczyk <mark.jedrzejczyk@gmail.com>

{rom24_quickmud_python-2.9.19 → rom24_quickmud_python-2.9.20}/mud/commands/group_commands.py

@@ -259,16 +259,45 @@ def do_group(char: Character, args: str) -> str:
     if char_affected & AffectFlag.CHARM:
         return "You like your master too much to leave!"
 
-    # Already in group - remove
+    leader_name = _display_name(char)
+    victim_name_str = _display_name(victim)
+
+    # Already in group - remove (ROM src/act_comm.c:1838-1847)
     if is_same_group(victim, char) and char is not victim:
         victim.leader = None
-        victim_name = getattr(victim, "short_descr", None) or getattr(victim, "name", "them")
-        return f"You remove {victim_name} from your group."
-
-    # Add to group
+        # TO_VICT: "$n removes you from $s group."
+        victim_messages = getattr(victim, "messages", None)
+        if isinstance(victim_messages, list):
+            victim_messages.append(f"{leader_name} removes you from {leader_name}'s group.")
+        # TO_NOTVICT: "$n removes $N from $s group."
+        room = getattr(char, "room", None)
+        if room is not None:
+            notvict_msg = f"{leader_name} removes {victim_name_str} from {leader_name}'s group."
+            for occupant in list(getattr(room, "people", []) or []):
+                if occupant is char or occupant is victim:
+                    continue
+                occ_messages = getattr(occupant, "messages", None)
+                if isinstance(occ_messages, list):
+                    occ_messages.append(notvict_msg)
+        return f"You remove {victim_name_str} from your group."
+
+    # Add to group (ROM src/act_comm.c:1850-1854)
     victim.leader = char
-    victim_name = getattr(victim, "short_descr", None) or getattr(victim, "name", "someone")
-    return f"{victim_name} joins your group."
+    # TO_VICT: "You join $n's group."
+    victim_messages = getattr(victim, "messages", None)
+    if isinstance(victim_messages, list):
+        victim_messages.append(f"You join {leader_name}'s group.")
+    # TO_NOTVICT: "$N joins $n's group."
+    room = getattr(char, "room", None)
+    if room is not None:
+        notvict_msg = f"{victim_name_str} joins {leader_name}'s group."
+        for occupant in list(getattr(room, "people", []) or []):
+            if occupant is char or occupant is victim:
+                continue
+            occ_messages = getattr(occupant, "messages", None)
+            if isinstance(occ_messages, list):
+                occ_messages.append(notvict_msg)
+    return f"{victim_name_str} joins your group."
 
 
 def do_gtell(char: Character, args: str) -> str:

{rom24_quickmud_python-2.9.19 → rom24_quickmud_python-2.9.20}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "rom24-quickmud-python"
-version = "2.9.19"
+version = "2.9.20"
 description = "A modern Python port of the ROM 2.4b6 MUD engine with full telnet server and JSON world loading"
 readme = "README.md"
 license = "MIT"

{rom24_quickmud_python-2.9.19 → rom24_quickmud_python-2.9.20/rom24_quickmud_python.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: rom24-quickmud-python
-Version: 2.9.19
+Version: 2.9.20
 Summary: A modern Python port of the ROM 2.4b6 MUD engine with full telnet server and JSON world loading
 Author-email: Mark Jedrzejczyk <mark.jedrzejczyk@gmail.com>
 Maintainer-email: Mark Jedrzejczyk <mark.jedrzejczyk@gmail.com>

{rom24_quickmud_python-2.9.19 → rom24_quickmud_python-2.9.20}/rom24_quickmud_python.egg-info/SOURCES.txt

@@ -670,6 +670,7 @@ tests/integration/test_do_equipment.py
 tests/integration/test_do_examine_command.py
 tests/integration/test_do_exits_command.py
 tests/integration/test_do_follow_master_notification.py
+tests/integration/test_do_group_notification.py
 tests/integration/test_do_help_command.py
 tests/integration/test_do_inventory.py
 tests/integration/test_do_look_command.py

rom24_quickmud_python-2.9.20/tests/integration/test_do_group_notification.py

@@ -0,0 +1,101 @@
+"""Probe: do_group must notify victim and room per ROM.
+
+ROM ``src/act_comm.c:1850-1854`` (add path):
+
+    victim->leader = ch;
+    act_new ("$N joins $n's group.", ch, NULL, victim, TO_NOTVICT, POS_RESTING);
+    act_new ("You join $n's group.",  ch, NULL, victim, TO_VICT,   POS_SLEEPING);
+    act_new ("$N joins your group.",  ch, NULL, victim, TO_CHAR,   POS_SLEEPING);
+
+ROM ``src/act_comm.c:1840-1847`` (remove path):
+
+    victim->leader = NULL;
+    act_new ("$n removes $N from $s group.", ch, NULL, victim, TO_NOTVICT, ...);
+    act_new ("$n removes you from $s group.", ch, NULL, victim, TO_VICT,   ...);
+    act_new ("You remove $N from your group.", ch, NULL, victim, TO_CHAR,  ...);
+
+Python ``mud/commands/group_commands.py:do_group`` returns only the
+TO_CHAR string; the victim and onlookers receive nothing. Parallel
+to the do_follow notify gap closed in 2.9.19.
+"""
+
+from __future__ import annotations
+
+from mud.commands.dispatcher import process_command
+from mud.models.character import Character, character_registry
+from mud.models.constants import Position
+from mud.models.room import Room
+from mud.registry import room_registry
+
+
+def _make_room(vnum: int = 9411) -> Room:
+    room = Room(vnum=vnum, name="Group Room", description="", room_flags=0, sector_type=0)
+    room.people = []
+    room.contents = []
+    room_registry[vnum] = room
+    return room
+
+
+def test_do_group_add_notifies_victim_and_room():
+    """ROM src/act_comm.c:1850-1854 — victim and onlooker receive broadcasts on add."""
+    snapshot = list(character_registry)
+    character_registry.clear()
+    try:
+        room = _make_room()
+
+        leader = Character(
+            name="leader",
+            is_npc=False,
+            level=10,
+            room=room,
+            position=int(Position.STANDING),
+        )
+        leader.messages = []
+        room.people.append(leader)
+        character_registry.append(leader)
+
+        victim = Character(
+            name="victim",
+            is_npc=False,
+            level=10,
+            room=room,
+            position=int(Position.STANDING),
+        )
+        victim.messages = []
+        victim.master = leader
+        room.people.append(victim)
+        character_registry.append(victim)
+
+        onlooker = Character(
+            name="onlooker",
+            is_npc=False,
+            level=10,
+            room=room,
+            position=int(Position.STANDING),
+        )
+        onlooker.messages = []
+        room.people.append(onlooker)
+        character_registry.append(onlooker)
+
+        process_command(leader, "group victim")
+
+        # ROM: victim receives "You join $n's group." (TO_VICT)
+        joined_victim = "\n".join(victim.messages).lower()
+        assert "join" in joined_victim and "group" in joined_victim, (
+            f"victim must receive `You join $n's group.` "
+            f"(ROM src/act_comm.c:1853); victim.messages = {victim.messages!r}"
+        )
+
+        # ROM: onlooker receives "$N joins $n's group." (TO_NOTVICT)
+        joined_onlooker = "\n".join(onlooker.messages).lower()
+        assert "joins" in joined_onlooker and "group" in joined_onlooker, (
+            f"onlooker must receive `$N joins $n's group.` "
+            f"(ROM src/act_comm.c:1851); onlooker.messages = {onlooker.messages!r}"
+        )
+
+        # State per ROM line 1850.
+        assert victim.leader is leader
+    finally:
+        character_registry.clear()
+        character_registry.extend(snapshot)
+        room_registry.pop(9411, None)