Convert Spell Details to Markdown Tables#19
Merged
nschimme merged 6 commits intoMay 7, 2026
Conversation
- Replaced mangled technical details in 73 spell pages with standardized Markdown tables. - Cleaned up descriptive text and fixed line breaks in spell pages. - Updated spell levels and mana costs to match the live MUME site (e.g., Identify moved to level 18). - Added missing Corrupt spell page. - Updated central Spell_list.md and navigation includes.
- Replaced mangled technical details in 73 spell pages with standardized Markdown tables. - Cleaned up descriptive text and fixed line breaks in spell pages. - Updated spell levels and mana costs to match the live MUME site (e.g., Identify moved to level 18). - Added missing Corrupt spell page. - Updated central Spell_list.md and navigation includes. - Escaped HTML special characters in tables to ensure VitePress build compatibility.
Updated the class for the Energy Drain spell from Cleric to Shaman.
Contributor
Reviewer's GuideRefactors multiple spell help pages to standardize their syntax metadata into Markdown tables, aligns spell levels/mana costs with the main spell list, and updates spell indexes to cross-link individual spell pages and add the new Corrupt spell. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
nschimme
changed the title
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Hey - I've found 5 issues, and left some high level feedback:
- Energy Drain is now tagged and described as a Shaman-only spell, but it still appears in the Cleric spell list in
Spells.mdandSpell_list.md; please align the class listings so the spell consistently belongs to the intended class. - In
Dispel_Magic.mdthe frontmatterdescriptioncontains an unescaped<victim | object>which may be interpreted as HTML and stripped; consider escaping the angle brackets as<...>for consistent rendering.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- Energy Drain is now tagged and described as a Shaman-only spell, but it still appears in the Cleric spell list in `Spells.md` and `Spell_list.md`; please align the class listings so the spell consistently belongs to the intended class.
- In `Dispel_Magic.md` the frontmatter `description` contains an unescaped `<victim | object>` which may be interpreted as HTML and stripped; consider escaping the angle brackets as `<...>` for consistent rendering.
## Individual Comments
### Comment 1
<location path="docs/pages/Breath_of_Briskness.md" line_range="28" />
<code_context>
+
+The spell is always cast without a target and will affect the caster and automatically everyone who is both in the same group and in the same room. This includes any mobs, such as [mounts](./Mount.md), charmies, or mercenaries.
+
+The first effect is an immideate distribution of movement points. At higher levels this can be roughly 50 points which are distributed to all eligible targets. So the number of points for each individual will be roghly 50/group size.
+
+The second effect is to boost move point regeneration. This effect is not reduced by the number of targets, so everyone will receive the full effect regardless of group size. At higher levels the boost can last up to 6 minutes and will be slightly stronger than the boost of a forest green cloak. The effect will stack with any other move regen boosts, such as equipment like the mentioned cloak, walking and travelling herblores, and rest and sleep.
</code_context>
<issue_to_address>
**issue (typo):** Fix spelling of “immideate” and “roghly”.
Please update “immideate” → “immediate” and “roghly” → “roughly” in this paragraph.
```suggestion
The first effect is an immediate distribution of movement points. At higher levels this can be roughly 50 points which are distributed to all eligible targets. So the number of points for each individual will be roughly 50/group size.
```
</issue_to_address>
### Comment 2
<location path="docs/pages/Identify.md" line_range="25" />
<code_context>
+
+When cast on a [mobile](./Mobile.md) or player, it provides a locate key but does not provide additional information (see also: [Divination](./Divination.md)).
+
+NOTE: It is strongly suggested not to reveal the exact [statistics](./Statistic.md) of items you identify. Saying that "sword X is slighly better at parrying than sword Y, but you need to be very strong in order to use it efficiently" is acceptable.
+
+Publishing tables of item statistics, either via boards or by web or by any other means, is highly frowned upon by the management. Please refrain from doing so.
</code_context>
<issue_to_address>
**issue (typo):** Spelling: “slighly” should be “slightly”.
In this sentence, update “slighly better” to “slightly better”.
```suggestion
NOTE: It is strongly suggested not to reveal the exact [statistics](./Statistic.md) of items you identify. Saying that "sword X is slightly better at parrying than sword Y, but you need to be very strong in order to use it efficiently" is acceptable.
```
</issue_to_address>
### Comment 3
<location path="docs/pages/Teleport.md" line_range="22" />
<code_context>
-N/A Duration : No Casting : Very long Position : Standing, Fighting
-Level : Level 8 Class : Magic User Min. Mana : 35 Example: \> cast
-'teleport' whiqueboza This spell can be learned from:
+**Teleport** is a [level](./Level.md) 8 [mage](./Mage.md) [spell](./Spell.md) that allows the caster to instantly transport themselves to a target [room](./room.md). To use teleport you need to aquire magical keys for rooms. The keys are obtained by either using the [Locate Life](./Locate_Life.md) spell or the [Locate](./Locate.md) spell.
+
+This spell can be learned from:
</code_context>
<issue_to_address>
**issue (typo):** Spelling: “aquire” should be “acquire”.
Please update the phrase to “acquire magical keys.”
```suggestion
**Teleport** is a [level](./Level.md) 8 [mage](./Mage.md) [spell](./Spell.md) that allows the caster to instantly transport themselves to a target [room](./room.md). To use teleport you need to acquire magical keys. The keys are obtained by either using the [Locate Life](./Locate_Life.md) spell or the [Locate](./Locate.md) spell.
```
</issue_to_address>
### Comment 4
<location path="docs/pages/Transfer.md" line_range="23" />
<code_context>
-N/A Duration : No Casting : Extremely long Position : Standing Level :
-Level 14 Class : Cleric Min. Mana : 50 Example: \> cast 'transfer'
-Aschit
+Wearing a [ring](./Ring.md) (or collar for shape-shifting bears) can protect against being transfered involuntarily.
This spell can be learned from:
</code_context>
<issue_to_address>
**issue (typo):** Spelling: “transfered” should be “transferred”.
```suggestion
Wearing a [ring](./Ring.md) (or collar for shape-shifting bears) can protect against being transferred involuntarily.
```
</issue_to_address>
### Comment 5
<location path="docs/pages/Sanctuary.md" line_range="21" />
<code_context>
-Syntax : cast 'sanctuary' \[recipient Type : Defensive Accumulative:
-No Duration : Yes Casting : Extremely long Position : Standing Level :
-Level 18 Class : Cleric Min. Mana : 50 Example: \> cast 'sanctuary'
+**Sanctuary** (often abbreviated *sanc*) is a level 18 [cleric](./Cleric.md) [spell](./Spell.md). If cast on a player it limits the maximum hit point damage that the player can suffer per hit. It is primarily used to protect players when attacking very dangerous and hard hitting mobs. The maximum hit point damage is dependant upon the casters wisdom and the target's willpower, but is usually a bit under 20 points of damage.
This spell can be learned from:
</code_context>
<issue_to_address>
**issue (typo):** Fix “dependant” and add apostrophe in “casters wisdom”.
Please change “dependant” to “dependent” and “casters wisdom” to “caster’s wisdom” to correct the spelling and grammar.
```suggestion
**Sanctuary** (often abbreviated *sanc*) is a level 18 [cleric](./Cleric.md) [spell](./Spell.md). If cast on a player it limits the maximum hit point damage that the player can suffer per hit. It is primarily used to protect players when attacking very dangerous and hard hitting mobs. The maximum hit point damage is dependent upon the caster's wisdom and the target's willpower, but is usually a bit under 20 points of damage.
```
</issue_to_address>Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Co-authored-by: sourcery-ai[bot] <58596630+sourcery-ai[bot]@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Improves readability of spells by moving them into tables. Updates Spells_list.md with main MUME helpfile
Related Issue
Checklist
::: details Spoiler) as defined in the Contributing Guide.docker compose up --build wiki(ornpm run docs:build) and verified changes locally.docs/guides.md,docs/classes.md) if a major new page was added.Summary by Sourcery
Reformat and standardize spell documentation to use structured property tables and align spell metadata and spell lists with the main MUME helpfile.
Enhancements:
Documentation: