Skip to content

Plan: dialog helpers extraction mapping#1535

Merged
SorraTheOrc merged 2 commits into
mainfrom
feature/WL-0MO5O06K10079BS7-plan-dialog-helpers
Apr 19, 2026
Merged

Plan: dialog helpers extraction mapping#1535
SorraTheOrc merged 2 commits into
mainfrom
feature/WL-0MO5O06K10079BS7-plan-dialog-helpers

Conversation

@SorraTheOrc
Copy link
Copy Markdown
Member

Summary

Planning work item WL-0MO5O06K10079BS7 - extract dialog helpers to shared module.

Work Done

Created mapping of private helpers in DialogsComponent to proposed public API signatures:

  1. createList() -> createBlessedList(options) - Creates configured Blessed List
  2. createTextarea() -> createBlessedTextarea(options) - Creates configured Blessed Textarea
  3. createLabel() -> createDialogLabel(options) - Creates dialog section headers

Documented migration steps and compatibility notes for child work items.

Acceptance Criteria Met

  • New module planned with public signatures documented and migration steps outlined
  • A mapping is produced showing which helpers in DialogsComponent map to exported functions/classes
  • A dependency link to WL-0MNU782BD004HO2W is included in the description

Ready For

  • Review of the proposed API signatures
  • Decision on API shape (options object vs typed params) - open question

Child Work Items

This plan feeds into 6 child feature work items:

  • WL-0MO5SBA2C0011DXJ: Create shared dialog-helpers module
  • WL-0MO5SBPQW006ZZ3Q: Migrate DialogsComponent to use helpers
  • WL-0MO5SBU2L002DLY4: Textarea and cursor integration parity
  • WL-0MO5SBWCA0027QKV: Modal base and modals interop
  • WL-0MO5SBZ3U0007M40: Integration parity and visual smoke tests
  • WL-0MO5SC3MG002M6JA: Cleanup, docs and migration mapping

@SorraTheOrc SorraTheOrc merged commit 9cf7d18 into main Apr 19, 2026
4 checks passed
@SorraTheOrc SorraTheOrc deleted the feature/WL-0MO5O06K10079BS7-plan-dialog-helpers branch April 28, 2026 18:10
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.

1 participant