CountersAddedFirstTime watcher implementation with counter type as an argument
- Dominant language
- Java
- Stars
- 2.4k
- Forks
- 940
- Avg merge
- 2d 12h
- Merged PRs (30d)
- 160
Description
Off the back of #15977
* We have `BoostCountersAddedFirstTimeWatcher` which matches on P1P1 counters being added for the first time each turn
* We have `CountersAddedFirstTimeWatcher` which matches on any counter being added for the first time each turn
* U.S.S. Enterprise-D, Galaxy-Class wants to match on CHARGE counters being added for the first time each turn
It would be nice to have a single class where the CounterType is either omitted to match on any counter type, or can be specified and then only trigger when one or more of that specific counter type is added. We can then clean up the custom class in the U.S.S. Enterprise-D, Galaxy-Class, and refactor the cards using `CountersAddedFirstTimeWatcher` or `BoostCountersAddedFirstTimeWatcher`
Contributor guide
No contributing guide indexed for this repository
Research direction
Start by reading BoostCountersAddedFirstTimeWatcher and CountersAddedFirstTimeWatcher, then inspect the U.S.S. Enterprise-D, Galaxy-Class implementation. The shared watcher should match any counter type when CounterType is omitted and only the specified type otherwise; refactor the affected cards and remove the custom class when the behavior is covered.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- game-dev
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100