neph1/LlamaTale
1
0
Fork 0
Code Issues 4 Pull requests Projects Releases 63 Packages Wiki Activity Actions

Remove EventSource-based WSGI API from IF mode #131

Merged
Copilot merged 4 commits from copilot/remove-old-eventsource-api into update-v0.42.0 2025-11-27 14:45:23 -05:00
Conversation 3 Commits 4 Files changed 11 +366 -1581
Copilot commented 2025-11-26 15:37:14 -05:00 (Migrated from github.com)
Copy link
  • Remove EventSource-based WSGI API from IF mode backend
  • Remove EventSource-based WSGI API from MUD mode backend
  • Convert MUD mode to use FastAPI/WebSocket (TaleMudFastAPIApp)
  • Remove EventSource polyfill files (eventsource.js, eventsource.min.js)
  • Update script.js to remove EventSource fallback code
  • Update story.html to remove eventsource.min.js reference
  • Remove TaleWsgiAppBase class entirely from if_browser_io.py
  • Update driver_mud.py to use TaleMudFastAPIApp
  • Update tests to remove WSGI-related test cases
  • Update WEBSOCKET.md documentation
  • Fix code review issues (PEP 8 spacing, magic numbers)
  • All tests pass
  • Code review passed
  • Security scan passed (0 alerts)
Original prompt

This section details on the original issue you should resolve

<issue_title>remove old eventsource based api</issue_title>
<issue_description>with websockets and fastapi implemented, i want to remove the old event source based wsgi app from if_browser and related files.
make sure existing functionality is not broken. only the older legacy api should be removed, from both backend and front end.</issue_description>

Comments on the Issue (you are @copilot in this section)

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

- [x] Remove EventSource-based WSGI API from IF mode backend - [x] Remove EventSource-based WSGI API from MUD mode backend - [x] Convert MUD mode to use FastAPI/WebSocket (TaleMudFastAPIApp) - [x] Remove EventSource polyfill files (eventsource.js, eventsource.min.js) - [x] Update script.js to remove EventSource fallback code - [x] Update story.html to remove eventsource.min.js reference - [x] Remove TaleWsgiAppBase class entirely from if_browser_io.py - [x] Update driver_mud.py to use TaleMudFastAPIApp - [x] Update tests to remove WSGI-related test cases - [x] Update WEBSOCKET.md documentation - [x] Fix code review issues (PEP 8 spacing, magic numbers) - [x] All tests pass - [x] Code review passed - [x] Security scan passed (0 alerts) <!-- START COPILOT CODING AGENT SUFFIX --> <details> <summary>Original prompt</summary> > > ---- > > *This section details on the original issue you should resolve* > > <issue_title>remove old eventsource based api</issue_title> > <issue_description>with websockets and fastapi implemented, i want to remove the old event source based wsgi app from if_browser and related files. > make sure existing functionality is not broken. only the older legacy api should be removed, from both backend and front end.</issue_description> > > ## Comments on the Issue (you are @copilot in this section) > > <comments> > </comments> > </details> - Fixes neph1/LlamaTale#130 <!-- START COPILOT CODING AGENT TIPS --> --- ✨ Let Copilot coding agent [set things up for you](https://github.com/neph1/LlamaTale/issues/new?title=✨+Set+up+Copilot+instructions&body=Configure%20instructions%20for%20this%20repository%20as%20documented%20in%20%5BBest%20practices%20for%20Copilot%20coding%20agent%20in%20your%20repository%5D%28https://gh.io/copilot-coding-agent-tips%29%2E%0A%0A%3COnboard%20this%20repo%3E&assignees=copilot) — coding agent works faster and does higher quality work when set up for your repo.
neph1 (Migrated from github.com) requested changes 2025-11-26 15:58:13 -05:00
neph1 (Migrated from github.com) left a comment
Copy link

@copilot great! let's remove it from mud as well, and anything in tale/web that still uses event source.

@copilot great! let's remove it from mud as well, and anything in tale/web that still uses event source.
Copilot commented 2025-11-26 16:14:25 -05:00 (Migrated from github.com)
Copy link

@copilot great! let's remove it from mud as well, and anything in tale/web that still uses event source.

Done in 998ad09. Removed EventSource from MUD mode as well:

  • Replaced TaleMudWsgiApp with TaleMudFastAPIApp using WebSocket
  • Removed eventsource.js and eventsource.min.js polyfill files
  • Removed EventSource fallback code from script.js
  • Removed eventsource.min.js reference from story.html
  • Removed TaleWsgiAppBase entirely from if_browser_io.py

Both IF and MUD modes now use WebSocket/FastAPI.

> @copilot great! let's remove it from mud as well, and anything in tale/web that still uses event source. Done in 998ad09. Removed EventSource from MUD mode as well: - Replaced `TaleMudWsgiApp` with `TaleMudFastAPIApp` using WebSocket - Removed `eventsource.js` and `eventsource.min.js` polyfill files - Removed EventSource fallback code from `script.js` - Removed eventsource.min.js reference from `story.html` - Removed `TaleWsgiAppBase` entirely from `if_browser_io.py` Both IF and MUD modes now use WebSocket/FastAPI.
neph1 (Migrated from github.com) approved these changes 2025-11-27 14:44:53 -05:00
Sign in to join this conversation.
Reviewers
No reviewers
neph1
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
neph1/LlamaTale!131
No description provided.