Skip to content

feat: implement create_room in admin API#2965

Draft
kvirund wants to merge 5 commits intomasterfrom
feat/admin-api-create-room
Draft

feat: implement create_room in admin API#2965
kvirund wants to merge 5 commits intomasterfrom
feat/admin-api-create-room

Conversation

@kvirund
Copy link
Collaborator

@kvirund kvirund commented Mar 14, 2026

Summary

  • Implements HandleCreateRoom replacing the stub with a working implementation
  • Uses existing OLC save path (redit_save_internally) for room creation
  • Supports explicit vnum or auto-assignment of first available vnum in zone range

Closes #2964

Test plan

  • Create room with explicit vnum — returns success, room accessible via get_room
  • Create room without vnum — auto-assigns first available vnum in zone
  • Duplicate vnum — returns error "Vnum already in use"
  • Invalid zone — returns error "Zone not found"
  • Vnum outside zone range — returns error "Vnum is outside zone range"
  • Unit tests pass (336/336)

🤖 Generated with Claude Code

kvirund and others added 4 commits March 14, 2026 04:25
Replaces the stub with a working implementation that creates rooms
via the existing OLC save path (redit_save_internally). Supports
explicit vnum or auto-assignment of the first available vnum in zone.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Replaces stubs with working implementations that create objects and
mobs via existing OLC save paths (oedit_save_internally and
medit_save_internally). Both support explicit vnum or auto-assignment.

Also adds zone_vnum parameter to both create commands for consistency
with HandleCreateRoom and HandleCreateTrigger.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Implements list_zone_commands, add_zone_command, and delete_zone_command
endpoints. These expose zedit's zone reset command functionality
(M/O/G/E/P/D/R/T/V/F/Q commands) through the admin API, allowing
management of mob/object loading rules without in-game OLC.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Allows resetting a zone via API, triggering mob/object reload
using the existing ResetZone/DecayObjectsOnRepop functions.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@kvirund kvirund marked this pull request as draft March 14, 2026 05:48
…parser

- list_mobs and list_objects now include a 'name' field for web admin
- ParseMobUpdate now sets both SetCharAliases and set_npc_name so mobs
  can be targeted by players in-game

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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.

feat: implement create_room in admin API

1 participant