Features to wire: Cleric domains + Druid circles audit follow-ups

[Geph]

Follow-ups from the Life/Light/Trickery/War Cleric domain wiring audit that need a design decision or new capability rather than a scoped bug fix. See the audit trace for how each finding was confirmed (real code, not just reasoning).

1. Design decision: wildcard presets always beat correct AI-mechanics data on fields the fingerprint doesn't cover

enrichFeatureLike (lib/import/enrich-import-modifiers.ts) applies SRD wildcard presets before merging AI-mechanics/phrase-detector detections. The cross-path dedup (isModifierRedundantAgainst / modifierInstanceFingerprint in lib/import/detect-feature-modifiers.ts) is fingerprint-based, and the uses fingerprint only includes type/fixedAmount/abilityModifier — not recharge. So if a preset's uses recharge is stale or wrong, a correct AI-mechanics usesRecharge hint for the same feature is silently discarded as a "duplicate," because the preset was added to linkedModifiers first.

Confirmed concretely via War Priest (now fixed as a one-off stale-preset correction — see PR/commit), but the underlying ordering + fingerprint blind spot is general: any preset with a uses pool is exposed to this whenever its recharge drifts from official text and a re-import supplies a correct AI-mechanics hint.

Options to evaluate:

• Make the uses fingerprint recharge-aware so a differing recharge is not treated as a duplicate (risk: could produce two coexisting uses pools where today's fingerprint intentionally treats near-duplicates as one).
• Change merge priority so AI/detector-sourced uses data can patch/override a preset's recharge instead of being dropped outright.
• Leave as-is and keep fixing individual stale presets as they're found during audits (status quo — cheap per-instance, but the class of bug will recur).

No code change proposed here — needs a decision on which direction before touching the shared dedup/merge logic, since it's load-bearing for every preset with a uses pool.

2. New capability: "bonus to a missed attack roll (self or ally, reaction)" has no wiring path

Guided Strike (War Domain, 2024 PHB — same pattern as the 2014 SRD version): "When you or a creature within 30 feet of you misses with an attack roll, you can expend a use of Channel Divinity and give that roll a +10 bonus... reaction if it's for another creature." Traced through the real pipeline — nothing wires (no mechanics[] kind, no phrase-detector rule, no preset).

The closest existing pattern is failedRollTriggerPreset (lib/compendium/enrich-srd-class-features.ts), used today only via hardcoded feature-name presets (Peerless Skill, Tactical Mind, Countercharm) — none targeting attack rolls or a reaction on someone else's roll.

Recommend: extend this into a proper mechanics[].kind (e.g. failed_roll_trigger) with rollKind including "attack", plus a targetScope: self | ally_in_range and useReaction flag, so this wires generically instead of needing a new hardcoded preset every time a class reuses the pattern (it will recur — Guided Strike is a classic).

3. To-do: narrative-only custom actions should still appear on the character sheet as actions

Per audit findings, these Channel-Divinity-cost (or similarly gated) features have real activation cost/timing but no matching characteristic/effect kind, so today they produce no sheet presence at all — not even a button — despite being real, frequently-used actions:

• Preserve Life (Life Domain) — Channel Divinity, splits 5×Cleric-level HP among Bloodied creatures in range, capped at half max HP each
• Radiance of the Dawn (Light Domain) — Channel Divinity, 30-ft emanation, Con save, 2d10+level radiant, half on success
• War God's Blessing (War Domain) — Channel Divinity, cast Shield of Faith/Spiritual Weapon without a slot, no concentration
• Invoke Duplicity (Trickery Domain) — Channel Divinity, illusory duplicate with cast/distract/move benefits
• Trickster's Transposition (Trickery Domain) — teleport-swap with the Invoke Duplicity illusion
• Improved Duplicity (Trickery Domain) — passive upgrade to Invoke Duplicity's illusion
• Guided Strike (War Domain) — see Add multiclassing and PDF imports to D&D 5.5e character builder #2 above; even before a real mechanical effect exists, it should still render as an activatable Channel Divinity action

Ask (from the audit follow-up): even where there's no modifier to compute, these should show up on the character sheet as actions with their narrative text attached (name, activation cost/resource spend, description) rather than being invisible. This likely means a lighter-weight "custom action, no mechanical effect" characteristic/registration path distinct from the fully-modeled effect kinds — worth scoping as its own change since it probably benefits every future "complex/narrative-only" feature, not just these four.

---

Audit source: Life/Light/Trickery/War Cleric domains, traced via enrichWildcardFeaturePresets / aiMechanicsToDetections / detectFeatureModifiers / mergeFeatureModifierDetections.

[Geph]

Follow-ups from the Druid circles audit (Land / Moon / Sea / Stars)

4. New capability: choice-gated, reselectable-per-long-rest spell lists

Circle of the Land Spells (Circle of the Land, level 3): "Whenever you finish a Long Rest, choose one type of land: arid, polar, temperate, or tropical... you have the spells listed for your Druid level and lower prepared." The source embeds four separate tables (Arid/Polar/Temperate/Tropical), one per land type, and the player's choice can change every long rest.

This doesn't fit the existing SpellsKnownEntry / alwaysPrepared + unlocksAtClassLevel model at all — that model assumes one fixed, non-reselectable list per feature. Auto-wiring today would have to arbitrarily pick one table's spells as permanently "always prepared" and silently drop the rest, which is wrong regardless of which one gets picked (confirmed via test — parseSubclassSpellTable's original single-<table>-match behavior did exactly this, defaulting to Arid). Fixed defensively for now: multi-<table> spell features are detected and left unwired, with a note in the import report (ImportReportSpellTable.ambiguousMultiTable) instead of a silent wrong grant.

The related Nature's Ward feature (level 10) grants "Resistance to a damage type associated with your current land choice" — same underlying gap, no mechanics possible until land-choice is modeled.

Proposed shape for real support: a "choice group" spell list — something like a spellChoiceGroup on SpellsKnownCharacteristic where the player picks one option from N named lists (each with its own per-level entries), re-pickable on a rest cadence (long_rest here, but the mechanism should be generic — e.g. Warlock's Book of Ancient Secrets swaps are a different cadence). This is a bigger, reusable primitive (domain/circle/oath "pick a subtype, get a matching spell list" is common across editions), not a one-off Circle of the Land special case — worth designing once rather than per-subclass.

5. New capability: no "in Wild Shape" sheet state

Druid's Wild Shape is a core, extremely common mechanic (not a one-off subclass feature) with no toggle at all in sheet-toggle-registry.ts (contrast with while_raging for Barbarian Rage, which several features already gate on). This breaks real Circle of the Moon features once wired:

• Circle Forms (level 3): "Until you leave the form, your AC equals 13 + WIS if higher" — wires today via ai.ac.formula with no gating, so it'd incorrectly apply outside Wild Shape too.
• Improved Circle Forms (level 6): "+WIS to Constitution saving throws" while in the form — no matching mechanics kind exists for "add a flat ability-mod bonus to one specific save," independent of the toggle problem.
• Lunar Form (level 14): "+2d10 Radiant, once/turn, on a Wild-Shape form's attack" — wires via on_hit_trigger with no gating, so it'd trigger on every attack, not just Wild-Shape ones.

Added a while_wild_shape builtin toggle this round (sheet-toggle-registry.ts) and confirmed ac/speed AI-mechanics builders now pass through requiresSheetToggle (they silently dropped it before). Still open: nothing on the sheet actually sets while_wild_shape automatically when the player activates Wild Shape — today it'd be a manual toggle the player flips themselves, same as while_raging is likely handled. Also open: no mechanics kind for "add ability modifier to a specific saving throw" (Improved Circle Forms) — closest existing pattern is saving_throw_alternate_ability (replaces the governing ability, not additive), so this needs either a new kind or a schema extension.

6. Confirmed non-functional (now fixed): new_toggles was a complete dead end

Traced end-to-end: new_toggles (declared on a class/subclass to introduce a new transformation state, e.g. the "Rage of the Gods" example used throughout the prompt guidance) was never read by any code path. getSheetToggleDefinition() only recognized the static builtin/optional lists plus magic_item:-prefixed ids; character-sheet-client.tsx's dynamic-toggle list was built from exactly two hardcoded sources (an "Elemental Mind" name-sniff and equipped magic items). No code anywhere read subclass.new_toggles/class.new_toggles, and the DB schema didn't even have a column for it. This affected every subclass that needed a brand-new transformation/conditional state, not just the Druid ones below — it just never surfaced because "Rage of the Gods" was always a hypothetical prompt example until this round.

Fixed this round: added new_toggles columns to classes/subclasses (schema + migration), threaded it through the import persist path (persist-import-content.ts, sanitize-import-content.ts — including hoisting new_toggles up from an individual feature if the LLM nests it there instead of on the subclass, which is what happened for Stormborn/Twinkling Constellations/Full of Stars below), and wired character-sheet-client.tsx to build real toggle definitions from a character's class/subclass new_toggles via a new sheetToggleDefinitionsFromNewToggles() helper. Confirms/unblocks:

• Stormborn (Circle of the Sea, level 10): Fly Speed + Cold/Lightning/Thunder resistance while Wrath of the Sea is active.
• Twinkling Constellations (Circle of the Stars, level 10) / Full of Stars (level 14): Fly Speed-while-Dragon-constellation and incorporeal resistance while in Starry Form.

Not done this round (flagging so it's not assumed complete): no dedicated editor UI for viewing/editing new_toggles on an existing compendium class/subclass — it round-trips transparently through the manual editors' save/load now, but there's no UI to see or hand-edit the list. Low priority unless someone needs to hand-author one outside the import flow.

---

Audit source: Circle of the Land / Moon / Sea / Stars Druid subclasses, traced via aiMechanicsToDetections / detectFeatureModifiers / parseSubclassSpellTable / enrichSubclassSpellTableFeatures / the sheet-toggle registry.

[Geph]

Follow-ups from the Monk warriors audit (Mercy / Shadow / Elements / Open Hand)

Most of this batch already had hand-authored SRD presets in place (Hand of Harm/Healing, Implements of Mercy, Physician's Touch, Shadow Arts, Shadow Step, Elemental Burst, Hand of Ultimate Mercy) or wired cleanly through existing/newly-generalized mechanics[] kinds (spells_known+spellcasting_ability, speed+requiresSheetToggle, and this round's new weapon_reach_modifier/movement_grant wiring). Two Warrior of Shadow features are genuinely unwired — no mechanics[] kind fits and no preset exists to fall back on (unlike e.g. Physician's Touch/Elemental Burst, which are text-only in the AI output but already covered by a preset):

7. New capability: conditionally waive another feature's activation precondition + append a bonus action-linked attack

Improved Shadow Step (Warrior of Shadow, level 11): "When you use your Shadow Step, you can expend 1 Focus Point to remove the requirement that you must start and end in Dim Light or Darkness for that use of the feature. As part of this Bonus Action, you can make an Unarmed Strike immediately after you teleport."

Two distinct gaps stacked on one feature:

• No mechanic models "spend a resource to waive an environmental precondition on a different, already-defined feature" (Shadow Step's dim-light/darkness requirement lives in its own preset text, not as a checkable limitation any other feature could reference/waive).
• No mechanic models "tack an extra attack onto an existing movement/bonus-action feature" — closest existing pattern is extra_attack (flat additional attack per turn), which doesn't carry the "only as part of this specific bonus action" conditioning.

8. New capability: self-granted temporary condition (Invisible) + partial-incorporeal movement, bundled with a cost waiver

Cloak of Shadows (Warrior of Shadow, level 17): grants the caster the Invisible condition and "partially incorporeal" movement (move through occupied spaces as Difficult Terrain; get shunted to the last unoccupied space if you end your turn in one) for up to 1 minute/until Incapacitated/until you end your turn in Bright Light, plus waives Flurry of Blows' Focus Point cost for the duration.

Confirmed no existing characteristic type models "grant the character a named condition (Invisible) for a duration" — closest is condition_immunity/damage_immunity, which suppress a condition rather than grant one. "Move through occupied spaces, get shunted out" also has no analog in movement_option's fields. This is a good candidate for a generic "self-buff: grant condition(s) + movement rule + waive resource cost on another feature, timed/toggle-gated" characteristic, since "become Invisible + weird movement rule, for a bit" is a recurring high-level-subclass-capstone shape (Assassinate-adjacent, Mistform-adjacent), not unique to this feature.

---

Audit source: Warrior of Mercy/Shadow/Elements/Open Hand (2024 PHB), traced via enrichWildcardFeaturePresets / aiMechanicsToDetections / resolvePresetKey / weapon-sheet-context.ts. Also fixed as scoped bugs this round (not tracked here since they were one-off corrections, not new-capability gaps): Quivering Palm's phantom "1/Long Rest" cap (2014-rules holdover not present in 2024 text), Wholeness of Body / Hand of Healing missing their "+ Wisdom modifier" heal bonus, and a Monk::Warrior of the Elements::Elemental Attunement name collision against an unrelated homebrew "Martial Form" wildcard preset.

[Geph]

Follow-ups from the Paladin oaths audit (Devotion / Glory / Ancients / Vengeance)

Skipped re-importing Oath of Devotion (already seeded + fully preset-wired). Fixed this round as general bugs (not tracked here — one-off corrections): Elder Champion's Regeneration was a silent no-op (turn_start_trigger's restoreResourceKey: "hit_points" was being treated as a nonexistent class-resource pool instead of a heal — now routes to the heal fields); requiresSheetToggle was being dropped for damage_resistance/condition_immunity/check_roll_modifier AI mechanics (now threaded through, check_roll_modifier via a limitations: [sheet_toggle] entry since it's an activation-effect, not a characteristic).

Two capstone/level-20 gaps left deliberately unwired this round (matches the existing hand-authored presets' current treatment — not a regression):

9. New capability: creature-type-scoped saving-throw advantage

Holy Nimbus (Oath of Devotion, level 20): "You have Advantage on any saving throw you are forced to make by a Fiend or an Undead." Attack/damage roll modifiers already support onlyVsCreatureTypes (lib/compendium/characteristic-modifiers.ts:529, evaluated via targetConditionMatches/creatureTypeMatches against a TargetConditionContext the player fills in when rolling), but check_roll_modifier FeatureEffects (saves/skills/abilities) have no equivalent — checkCategory: "save" matches all saves regardless of who's forcing them, and the AI mechanic's conditionNote field is explicitly freeform/unenforceable by design (lib/import/content-schema.ts:81). Wiring Holy Nimbus as-is would grant blanket advantage on every save, which is wrong. Recommend extending the onlyVsCreatureTypes/TargetConditionContext pattern to check_roll_modifier saves (a "who's forcing this save" prompt, mirroring the existing "who am I attacking" prompt) rather than a Holy-Nimbus-specific hack — likely to recur (Aura-of-Protection-adjacent and "ward against undead/fiends" is a common Paladin/Cleric trope).

10. New capability: "auto-hit on a miss" attack-roll override

Living Legend (Oath of Glory, level 20), Unerring Strike: "Once on each of your turns when you make an attack roll with a weapon and miss, you can cause that attack to hit instead." The existing preset (enrich-srd-class-features.ts:4225-4245) already wires Living Legend's other two benefits (Charismatic → checkAdvantage, Saving Throw Reroll → failedRollTriggerPreset) but has no equivalent for this one — there's no mechanic anywhere for "convert a missed attack roll into an automatic hit" (closest is checkRollMode: "replace_failure" on check_roll_modifier, which is scoped to ability/skill/save checks, not attack rolls, and rerolls rather than force-succeeds). Related to item #2 above (Guided Strike's "+10 to a missed roll") but more extreme — worth scoping together since both are "do something to a roll that already happened, once/turn" once someone needs either.

Also flagging, low priority: Aura of Alacrity's ally-facing half ("whenever an ally enters your Aura of Protection... the ally's Speed increases by 10 feet") has no mechanic in the source LLM output at all (self +10 ft. Speed is the only part wired) and the existing preset likewise only describes it in the aura's label (enrich-srd-class-features.ts:4197-4204) rather than modeling it — AuraCharacteristic supports save bonuses/condition immunities/half-cover for allies in the aura but not a speed grant. No action taken; noting so a future "aura grants allies X" feature request isn't a surprise.

---

Audit source: Oath of Devotion/Glory/Ancients/Vengeance (2024 PHB), traced via parse-ai-mechanics.ts's turn_start_trigger/check_roll_modifier/damage_resistance/condition_immunity cases, collect-feature-roll-modes.ts, and the existing // —— Paladin oaths (2024) —— preset block.