Skip to content

Enforce sight-required spell targeting - #117

Merged
dajiaohuang merged 2 commits into
mainfrom
fix/102-sight-required-spell-targeting
Aug 31, 2026
Merged

Enforce sight-required spell targeting#117
dajiaohuang merged 2 commits into
mainfrom
fix/102-sight-required-spell-targeting

Conversation

@dajiaohuang

Copy link
Copy Markdown
Contributor

Summary

  • share one authoritative visibility predicate between attacks and spells
  • reject sight-required spell targets when the caster is blinded or excluded by an explicit visibility ACL
  • preserve explicit visibility of hidden/invisible targets and non-sight spell behavior
  • prove rejected public casts consume no action, slot, randomness, concentration, state, or revision

Verification

  • full domain tests
  • full structured-spell MCP test file
  • attack-visibility MCP tests
  • independent read-only logic review: no findings
  • post-main-merge focused domain/MCP tests
  • Ruff and git diff --check

Closes #102

@dajiaohuang
dajiaohuang merged commit 680ff92 into main Aug 31, 2026
4 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.

Enforce caster visibility for sight-required spells

1 participant