Conversation
📝 WalkthroughSummary by CodeRabbit
WalkthroughBy the Omnissiah’s decree, this update initiates the inclusion of two sacred data artefacts into the ChapterMaster project’s configuration. The first is a pict-capture file, “flesh_tearers.png”, sanctified for the chapter iconography. The second is a comprehensive JSON data scroll, “34.json”, encoding the full litany of the Flesh Tearers chapter: their gene-seed, livery, personnel, and chapter lore. No alterations to logic or exported machine-spirits are detected; the modifications are strictly in the domain of data and resource inclusion. Changes
Possibly related PRs
Suggested labels
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Actionable comments posted: 8
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
Plan: Pro
⛔ Files ignored due to path filters (1)
datafiles/images/creation/chapters/icons/flesh_tearers.pngis excluded by!**/*.png
📒 Files selected for processing (2)
ChapterMaster.yyp(2 hunks)datafiles/main/chapters/34.json(1 hunks)
🧰 Additional context used
📓 Path-based instructions (2)
`**/*.json`: - JSON files with comments are supported by the parser used by this team. Don't warn or request changes to remove them.
**/*.json: - JSON files with comments are supported by the parser used by this team. Don't warn or request changes to remove them.
datafiles/main/chapters/34.json
`**/*.*`: - Having humanly understandable and maintainable code is always the top most priority. - DRY (Don't repeat yourself) principle is also very important. - Ensure that the c...
**/*.*: - Having humanly understandable and maintainable code is always the top most priority.
- DRY (Don't repeat yourself) principle is also very important.
- Ensure that the code is compliant with the CODE_STYLE.MD: https://github.com/Adeptus-Dominus/ChapterMaster/blob/main/docs/CODE_STYLE.md
datafiles/main/chapters/34.jsonChapterMaster.yyp
🔇 Additional comments (4)
ChapterMaster.yyp (2)
139-139: Icon addition approved
The glyphflesh_tearers.pngis correctly inserted in alphabetical order underdatafiles/images/creation/chapters/icons.
Please verify that the file exists in the repository at the specified path and is referenced properly in the GMProject.
518-518: Chapter definition manifest approved
The sacred scroll34.jsonis placed correctly following33.jsonindatafiles/main/chapters.
Ensure the JSON conforms to the chapter schema and that the file is included in this PR.datafiles/main/chapters/34.json (2)
3-17: Sanctify Mutation Rates Configuration
By the Omnissiah, the mutation block venerates Omophagea at full potency (1.0) and purges all others to 0.0, perfectly aligning with the Flesh Tearers’ dread gene-seed.
68-78:Details
❌ Incorrect review comment
Guard Negative Marine Counts
Negative values forfifthandtenthcompanies (−100.0) may yield underflow or negative troop counts if not clamped. Ensure the engine handles these correctly.Run this script to flag negative entries:
🏁 Script executed:
#!/bin/bash jq '.chapter.extra_marines | to_entries[] | select(.value < 0)' datafiles/main/chapters/34.jsonLength of output: 173
By the Omnissiah, I shall scour the code for all worshippers of
extra_marinesto see if the Machine-Spirit already clamps these negative offerings:
🏁 Script executed:
#!/bin/bash rg -n "extra_marines" -C3Length of output: 11593
Dismiss Negative Marine Clamping Concern
By the Omnissiah, the negative values inextra_marinesare intentional subtractions, as documented in the chapter template (/* use negative numbers to subtract */). No underflow guard is required—these values correctly reduce company strengths.Likely an incorrect or invalid review comment.
Purpose
Describe your changes/additions
What can/needs to be improved/changed
Related things and/or additional context