Update listener internal slots to map<name, structure>
Nobody has claimed this yet.
- Dominant language
- HTML
- Stars
- 45
- Forks
- 21
- PR merge metrics
- No merged PRs in 30d
Description
Currently internal slots for [[readHandlers]], [[writeHandlers]], etc. are a map from name to a handler function.
Besides that function, the steps require saving |options| and other information as well.
So the internal slots will need to contain an object like
dictionary HandlerData {
required function handler;
sequence<InteractionOptions> subscriberOptions;
sequence<any> subscriberData;
}
We could also add the |interaction| for convenience, but it can me retrieved using |name|.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the listener internal slot definitions and the steps that read or write [[readHandlers]], [[writeHandlers]], and related maps. Update the slot values to retain the handler, subscriberOptions, and subscriberData described in the issue, then verify that all affected listener steps use the new structure.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- api
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100