aethyr/Aethyr
2
0
Fork 0
Code Issues 281 Pull requests Projects Releases Packages Wiki Activity Actions

Epic: Sunset Legacy player.output() Direct Call Pattern #184

New issue
Open
opened 2026-03-15 23:08:50 +00:00 by freemo · 0 comments
freemo commented 2026-03-15 23:08:50 +00:00
Owner
Copy link

Background and Context

There are 111+ direct @player.output() calls across all command action files. This is the largest legacy pattern to migrate. A gradual approach uses a :legacy_output bridge event that maintains backward compatibility while call sites are converted incrementally.

Demonstrable Outcome

Zero player.output() calls in action command files. Output flows through Wisper events.

Acceptance Criteria

  • Bridge event created and functioning
  • All command categories migrated
  • Bridge removed after all callers migrated
## Background and Context There are 111+ direct `@player.output()` calls across all command action files. This is the largest legacy pattern to migrate. A gradual approach uses a `:legacy_output` bridge event that maintains backward compatibility while call sites are converted incrementally. ## Demonstrable Outcome Zero `player.output()` calls in action command files. Output flows through Wisper events. ## Acceptance Criteria - Bridge event created and functioning - All command categories migrated - Bridge removed after all callers migrated
freemo added this to the v1.2.0 milestone 2026-03-15 23:19:50 +00:00
freemo modified the milestone from v1.2.0 to v1.0.0 2026-03-16 00:27:53 +00:00
freemo self-assigned this 2026-03-16 01:26:58 +00:00
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
master
feature/m2-headless-renderer
dependabot/bundler/rdoc-6.3.2
dependabot/bundler/rexml-3.2.5
dependabot/bundler/json-tw-2.3
inprogress
dependabot/bundler/rake-tw-13.0
No results found.
Labels
Clear labels   
Blocked

Cannot proceed, depends on unresolved issue

  
Duplicate

Issue is a duplicate of an existing one

  
MoSCoW/Could Have

Desirable but not necessary, included if time permits

  
MoSCoW/Must Have

Essential, project cannot ship without it

  
MoSCoW/Should Have

Important but not strictly essential

  
Points/1

Estimated effort: 1 story points

  
Points/13

Estimated effort: 13 story points

  
Points/2

Estimated effort: 2 story points

  
Points/21

Estimated effort: 21 story points

  
Points/3

Estimated effort: 3 story points

  
Points/5

Estimated effort: 5 story points

  
Points/8

Estimated effort: 8 story points

  
Priority/Backlog

Not yet prioritized

  
Priority/Critical

Must be addressed immediately

  
Priority/High

Important, should be completed soon

  
Priority/Low

Nice to have, not time-sensitive

  
Priority/Medium

Normal priority, valid and should be addressed

  
State/Completed

Issue resolved and all PRs merged

  
State/In progress

Someone is actively working on this issue

  
State/In review

Implementation complete, PR submitted for review

  
State/Paused

Work temporarily suspended, typically blocked

  
State/Unverified

Issue created but not yet reviewed by a maintainer

  
State/Verified

Maintainer confirmed valid and actionable

  
State/Wont Do

Deliberate decision not to address this issue

  
Type/Bug

A defect in existing functionality

  
Type/Epic

Large body of work broken into multiple issues

  
Type/Feature

A new capability or user story

  
Type/Legendary

Major strategic pillar composed of multiple Epics

  
Type/Task

Technical or administrative work (not bug or feature)

  
Type/Testing

Test-only work item (TDD, test infrastructure, coverage)

No labels
Blocked
Duplicate
MoSCoW/Could Have
MoSCoW/Must Have
MoSCoW/Should Have
Points/1
Points/13
Points/2
Points/21
Points/3
Points/5
Points/8
Priority/Backlog
Priority/Critical
Priority/High
Priority/Low
Priority/Medium
State/Completed
State/In progress
State/In review
State/Paused
State/Unverified
State/Verified
State/Wont Do
Type/Bug
Type/Epic
Type/Feature
Type/Legendary
Type/Task
Type/Testing
Milestone
Clear milestone
No items
No milestone
v1.0.0
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
freemo
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.

Blocks
#171 Event-Driven Architecture: Wisper Expansion & Legacy Sunset
aethyr/Aethyr
Depends on
#185 Create :legacy_output bridge event for gradual player.output() migration
aethyr/Aethyr
#186 Replace player.output() in movement and perception commands
aethyr/Aethyr
#187 Replace player.output() in inventory, equipment, and combat commands
aethyr/Aethyr
#188 Replace player.output() in communication, info, settings, and meta commands
aethyr/Aethyr
#189 Replace player.output() in admin commands and remove :legacy_output bridge
aethyr/Aethyr
Reference: aethyr/Aethyr#184
No description provided.
Delete branch "%!s()"

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?