Card-Forge / Card-Forge/forge

Spell Refactoring Additional Cost

Open
#2,472 10 comments 0 reactions 2 assignees Claimed by @MostCromulent View on GitHub
AI Game Mechanics keep
Dominant language
Java
Stars
2.7k
Forks
1.1k
Avg merge
1d 1h
Merged PRs (30d)
158

Description

Currently:
Spells that has "As an additional cost to cast this spell" like Discard a Card like `Tormenting Voice` use this as part of their cost for the spell

```
Name:Tormenting Voice
ManaCost:1 R
Types:Sorcery
A:SP$ Draw | Cost$ 1 R Discard<1/Card> | CostDesc$ As an additional cost to cast this spell, discard a card. | NumCards$ 2 | Defined$ You | SpellDescription$ Draw two cards.
```

Maybe better?
```
Name:Tormenting Voice
ManaCost:1 R
Types:Sorcery
S:Mode$ RaiseCost | ValidCard$ Card.Self | Activator$ You | Type$ Spell | Cost$ Discard<1/Card> | EffectZone$ All | Description$ As an additional cost to cast this spell, discard a card.
A:SP$ Draw | NumCards$ 2 | Defined$ You | SpellDescription$ Draw two cards.
```

it might help us for coping cards if the "intrinsic" cost of the spell is only its mana cost?

then we might better differ from which static ability the additional cost is coming from in case of "linked abilities"?

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.