lily/mudserver
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix smoke tests and resolve CI timeouts #1

Merged
lily merged 2 commits from mcp-integration into main 2026-03-17 10:00:20 -06:00
Conversation 0 Commits 2 Files Changed 11 +394 -127
lily commented 2026-03-17 09:57:37 -06:00
Owner
Copy Link

This PR fixes the CI timeouts and broken smoke tests caused by the change in character spawn location.

Changes:

  • Updated Movement Paths: Adjusted Test 1 and Test 6 movement commands to correctly navigate from the new Central Bridge spawn point to the Rusty Tankard and Dark Alley.
  • Fixed Test 6 Hang: Implemented a subshell command pipe with a sleep 8 for the combat test. This keeps the SSH session open long enough for tick-based combat to process while allowing subsequent commands (stats, quit) to be sent.
  • Ensured Clean Disconnects: Added explicit quit commands to all test sessions to prevent sessions from hanging until server-side timeouts.

Verified locally and confirmed passing in Gitea Actions run #12.

This PR fixes the CI timeouts and broken smoke tests caused by the change in character spawn location. ### Changes: - **Updated Movement Paths:** Adjusted Test 1 and Test 6 movement commands to correctly navigate from the new `Central Bridge` spawn point to the `Rusty Tankard` and `Dark Alley`. - **Fixed Test 6 Hang:** Implemented a subshell command pipe with a `sleep 8` for the combat test. This keeps the SSH session open long enough for tick-based combat to process while allowing subsequent commands (`stats`, `quit`) to be sent. - **Ensured Clean Disconnects:** Added explicit `quit` commands to all test sessions to prevent sessions from hanging until server-side timeouts. Verified locally and confirmed passing in Gitea Actions run #12.
lily added 2 commits 2026-03-17 09:57:38 -06:00
Add JSON-RPC interface and refactor for MCP support
All checks were successful
Smoke tests / Build and smoke test (push) Successful in 1h0m43s
Details
dd517d8851
Fix smoke tests: update movement for new spawn point and resolve Test 6 timeout
All checks were successful
Smoke tests / Build and smoke test (push) Successful in 1m49s
Details
ebdfa16aa5
lily merged commit df757ba37d into main 2026-03-17 10:00:20 -06:00
lily deleted branch mcp-integration 2026-03-17 10:00:34 -06:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
ai-user lily
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: lily/mudserver#1
Delete Branch "mcp-integration"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?