fix(ignis): allow safe Dagger blinks in midair #1

Merged
dmg merged 1 commits from us-011-restore-dagger-blinking-including-midair into main 2026-10-12 19:22:26 +00:00
Owner

US-011: safe Dagger blinking, including midair

Canonical story: https://git.garvis.dev/dmg/somc-okf/src/branch/main/user-stories/purpur-ignis/us-011-restore-dagger-blinking-including-midair.md

  • Permit unsupported exits after the existing swept-body collision, hazard, chunk and boundary checks; retain the near-feet hazardous-surface rejection.
  • Keep the existing teleport, targeting, eligibility/receipt/reset, reusable item/Vault protection and shared two-second cooldown behavior. No hunger gate/debit, flight or fall immunity.
  • Add native paid recipe/result-slot crafting and actual PDC/durable receipt proof, registered interaction and incoming item-use handling, CraftPlayer/native PLUGIN teleport state and outgoing position packets; verify actual entrypoint listener registration.
  • Cover ground/air forward and behind-player movement, aim/look, zero hunger, target exclusions, safety/protection refusals, traded/reset copies, both hands, Vault denial, cooldown failure/cancellation/spam and no blocked-target fallback.

Verification

  • Untouched baseline: 158 tests (123 ordinary / 35 native).
  • Meaningful initial RED: grounded crafted-item control passes; unsupported airborne interaction produces no movement. Fixture/bootstrap failures are not behavioral RED.
  • Expanded byte-identical pre-fix safety replay: 41 cases, seven airborne forward/target assertion failures; grounded controls and preserved refusals pass.
  • Final Dagger matrix: 43 native cases pass.
  • Final ./gradlew clean check jar: 201 tests (123 ordinary / 78 native), zero failures/errors/skips.
  • Tested source commit: 7e60224096b12aac34083ef924ff8ddad3684139.
  • Tested tree: 9ede829880f508821d9e9b2fe061844be4dee907; base 3f721e928a17e3b8fc2cfafdcc980d8de27fc129.
  • Frozen 130 source files / 110 native runtime JARs unchanged; input-manifest SHA-256 07a316d7f2725f522f2d379b4969b3fcbc1e0690dbf88a2e512fc7cae7ae86e5.
  • Original PR CI 3266 rejected an overlong commit-body line before tests; push CI 3265 passed. A guarded rebase only reworded the message; fresh affected tests and the full 201-test suite pass, with the identical tree and all frozen inputs unchanged. The branch update used an explicit lease against the last authorized remote head, not an implicit lease.
  • Source diff/secret review and canonical wiki validation pass (one pre-existing unrelated frontmatter warning).

Evidence and approval boundaries

Server/session, scheduler/lookup, finite-world/ray and visibility ports are modeled; selected native methods and temporary-file persistence run unchanged. This is not live client/terrain/protection verification. The complete reported grounded failure is not reproduced or diagnosed; grounded controls pass before and after, while unsupported-air forward and target failures are reproduced and fixed.

Approved scope is US-011 only. This session is the approved source integration coordinator: satisfy current PR checks/reviews, merge through Gitea and verify equality with the tested tree before marking done. Final plugin security review follows the verified merge and is currently pending. Source publication may trigger a release but does not authorize deployment. No announcement, configuration, credentials, dependencies, CI/tooling, GitOps or live-server changes.

## US-011: safe Dagger blinking, including midair Canonical story: https://git.garvis.dev/dmg/somc-okf/src/branch/main/user-stories/purpur-ignis/us-011-restore-dagger-blinking-including-midair.md - Permit unsupported exits after the existing swept-body collision, hazard, chunk and boundary checks; retain the near-feet hazardous-surface rejection. - Keep the existing teleport, targeting, eligibility/receipt/reset, reusable item/Vault protection and shared two-second cooldown behavior. No hunger gate/debit, flight or fall immunity. - Add native paid recipe/result-slot crafting and actual PDC/durable receipt proof, registered interaction and incoming item-use handling, CraftPlayer/native PLUGIN teleport state and outgoing position packets; verify actual entrypoint listener registration. - Cover ground/air forward and behind-player movement, aim/look, zero hunger, target exclusions, safety/protection refusals, traded/reset copies, both hands, Vault denial, cooldown failure/cancellation/spam and no blocked-target fallback. ## Verification - Untouched baseline: 158 tests (123 ordinary / 35 native). - Meaningful initial RED: grounded crafted-item control passes; unsupported airborne interaction produces no movement. Fixture/bootstrap failures are not behavioral RED. - Expanded byte-identical pre-fix safety replay: 41 cases, seven airborne forward/target assertion failures; grounded controls and preserved refusals pass. - Final Dagger matrix: 43 native cases pass. - Final `./gradlew clean check jar`: **201 tests (123 ordinary / 78 native), zero failures/errors/skips**. - Tested source commit: `7e60224096b12aac34083ef924ff8ddad3684139`. - Tested tree: `9ede829880f508821d9e9b2fe061844be4dee907`; base `3f721e928a17e3b8fc2cfafdcc980d8de27fc129`. - Frozen 130 source files / 110 native runtime JARs unchanged; input-manifest SHA-256 `07a316d7f2725f522f2d379b4969b3fcbc1e0690dbf88a2e512fc7cae7ae86e5`. - Original PR CI 3266 rejected an overlong commit-body line before tests; push CI 3265 passed. A guarded rebase only reworded the message; fresh affected tests and the full 201-test suite pass, with the identical tree and all frozen inputs unchanged. The branch update used an explicit lease against the last authorized remote head, not an implicit lease. - Source diff/secret review and canonical wiki validation pass (one pre-existing unrelated frontmatter warning). ## Evidence and approval boundaries Server/session, scheduler/lookup, finite-world/ray and visibility ports are modeled; selected native methods and temporary-file persistence run unchanged. This is not live client/terrain/protection verification. The complete reported grounded failure is not reproduced or diagnosed; grounded controls pass before and after, while unsupported-air forward and target failures are reproduced and fixed. Approved scope is US-011 only. This session is the approved source integration coordinator: satisfy current PR checks/reviews, merge through Gitea and verify equality with the tested tree before marking done. Final plugin security review follows the verified merge and is currently pending. Source publication may trigger a release but **does not authorize deployment**. No announcement, configuration, credentials, dependencies, CI/tooling, GitOps or live-server changes.
dmg added 1 commit 2026-10-12 19:14:31 +00:00
fix(ignis): allow safe Dagger blinks in midair
CI / build (pull_request) Successful in 1m22s
CI / build (push) Successful in 1m15s
7e60224096
Retain swept collision, hazards, loaded chunks and protection checks
without requiring a supported exit.

Add native paid crafting, item-use packet, target movement and safeguard
regression coverage; 201 tests pass.
dmg force-pushed us-011-restore-dagger-blinking-including-midair from db28062e25 to 7e60224096 2026-10-12 19:14:31 +00:00 Compare
dmg merged commit 6a4d0c530b into main 2026-10-12 19:22:26 +00:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: dmg/purpur-ignis#1