Skip to content

Use current MUD metadata consistently across wiki publication and details - #159

Merged
moshehbenavraham merged 1 commit into
masterfrom
fix/wiki-reference-metadata
Nov 5, 2026
Merged

moshehbenavraham merged 1 commit into
masterfrom
fix/wiki-reference-metadata

Conversation

@moshehbenavraham

Copy link
Copy Markdown
Member

Live publication checks found that item lists/filters and detail pages decoded different spell effects and wear slots. Share one effect decoder and slot mapping, reconcile all four affect banks with current MUD definitions, and preserve masks above 32 bits. Bound race restrictions to the MUD's 32-bit field so races above 32 cannot inherit lower race bits; include Dragoon and correct outdated race names.

Remove duplicate decoding tables from the detail service and cover the Misty Vale shirt's effects, belt attachment, higher-bit affects, race-mask wrapping, and current class/race identities. Backend configuration, formatting, lint, types, MUD-write verification, 26 focused tests and build pass. All 132 current affect-mask positions match the selected MUD source.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 5, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-05T21:34:24.663936Z 7c0daa0 PR opened
🔒 Security Review ✅ Completed 2026-11-05T21:36:43.943176Z 7c0daa0 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@moshehbenavraham
moshehbenavraham merged commit c8ca83f into master Nov 5, 2026
6 checks passed
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