[PR #249] [MERGED] kb(dark-souls-remastered): resolve #147's open bullet questions, add probe and touch-trigger results #251

New issue

Closed

opened 2026-10-15 20:54:48 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-15 20:54:48 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/rehan-remade/universal-modder/pull/249
Author: @Ep1cNova
Created: 10/12/2026
Status: ✅ Merged
Merged: 10/12/2026
Merged by: @rehan-remade

Base: main ← Head: kb/dsr-portal-gun-resolve-flags

📝 Commits (5)

f160d99 kb(dark-souls-remastered): resolve the bullet-section flags, record warp-test base and App ver
8e2ae74 kb(dark-souls-remastered): wall touch trigger confirmed, fly-state probe never hits the player
e9bbead kb: DSR portal gun: review fixes
9770ceb Merge remote-tracking branch 'origin/main' into land/pr-249
2ceb4a6 kb: regenerate the index

📊 Changes

3 files changed (+82 additions, -62 deletions)

View changed files

📝 knowledge/INDEX.md (+1 -1)
📝 knowledge/games/dark-souls-remastered/portal-gun-prototype-via-dinput8-proxy-dll-player-camera-and.md (+79 -59)
📝 knowledge/index.json (+2 -2)

📄 Description

Follow-up to #147, answering the open points from the merge comment.

Bullet section: the three "unresolved" flags are resolved and removed

List / stride / size: the working code polls [BulletMan + 0x08], next at +0x348, 0x360-byte objects. Those are the real BulletIns. The 0x320-byte objects on [BulletMan + 0x28] / +0x308 are launchers: Spawn returns a launcher's handle, which holds state -1 and a NaN position. The old note had the two the wrong way round. Per-poll logs of both lists during the same throws settled it.
Impact state: 4. Every logged throw and shot went 2 → 4 on a world hit. State 3 (explosion) was never seen with the Prism Stone rows; whether exploding bullets pass through 3 is kept as an open question.
Ownerless bullets: the "unknown position" lines were out of date. An ownerless bullet starts at the requested transform, flies and collides normally; only its visual is missing.
Gotchas 7, 9, 10 and 14 are rewritten to match.

Warp test (Gotcha 3): the crashing write went to ChrCtrl [PlayerIns + 0x18] + 0x108..0x124, not [PlayerIns + 0x68]. A retry with +0x68 has not been done, so the wrong base is likely but still not confirmed as the cause.

App ver: 1.03.1 (title screen), added next to the Steam build id.

New results from today's run

The touch trigger for shot-placed wall portals fires in game.
New Gotcha 15: a blind destination behind a wall was rejected twice, then accepted, and the player fell into the void.
Fly-state, zero-speed probe bullets stay fixed but never hit the player (ownerless with NPC atk 3146, ownerless with PC atk 222, player-owned with 3146; the player stood inside each one).

um kb check passes; the index is regenerated.

🤖 Generated with Claude Code

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/rehan-remade/universal-modder/pull/249 **Author:** [@Ep1cNova](https://github.com/Ep1cNova) **Created:** 10/12/2026 **Status:** ✅ Merged **Merged:** 10/12/2026 **Merged by:** [@rehan-remade](https://github.com/rehan-remade) **Base:** `main` ← **Head:** `kb/dsr-portal-gun-resolve-flags` --- ### 📝 Commits (5) - [`f160d99`](https://github.com/rehan-remade/universal-modder/commit/f160d99c037d752d08d10fd99866e262abb34685) kb(dark-souls-remastered): resolve the bullet-section flags, record warp-test base and App ver - [`8e2ae74`](https://github.com/rehan-remade/universal-modder/commit/8e2ae74a615d3e906c9bde99175d0f41064cef6f) kb(dark-souls-remastered): wall touch trigger confirmed, fly-state probe never hits the player - [`e9bbead`](https://github.com/rehan-remade/universal-modder/commit/e9bbeadd18aae3c10824ddc9453f9ebaed5425e8) kb: DSR portal gun: review fixes - [`9770ceb`](https://github.com/rehan-remade/universal-modder/commit/9770ceb47fe74453de616618de95e06153a6cba0) Merge remote-tracking branch 'origin/main' into land/pr-249 - [`2ceb4a6`](https://github.com/rehan-remade/universal-modder/commit/2ceb4a60336321c969ed6580a0bf27ed1b776fa0) kb: regenerate the index ### 📊 Changes **3 files changed** (+82 additions, -62 deletions) <details> <summary>View changed files</summary> 📝 `knowledge/INDEX.md` (+1 -1) 📝 `knowledge/games/dark-souls-remastered/portal-gun-prototype-via-dinput8-proxy-dll-player-camera-and.md` (+79 -59) 📝 `knowledge/index.json` (+2 -2) </details> ### 📄 Description Follow-up to #147, answering the open points from the merge comment. **Bullet section: the three "unresolved" flags are resolved and removed** - **List / stride / size:** the working code polls `[BulletMan + 0x08]`, next at `+0x348`, 0x360-byte objects. Those are the real `BulletIns`. The 0x320-byte objects on `[BulletMan + 0x28]` / `+0x308` are *launchers*: `Spawn` returns a launcher's handle, which holds state -1 and a NaN position. The old note had the two the wrong way round. Per-poll logs of both lists during the same throws settled it. - **Impact state:** 4. Every logged throw and shot went 2 → 4 on a world hit. State 3 (explosion) was never seen with the Prism Stone rows; whether exploding bullets pass through 3 is kept as an open question. - **Ownerless bullets:** the "unknown position" lines were out of date. An ownerless bullet starts at the requested transform, flies and collides normally; only its visual is missing. - Gotchas 7, 9, 10 and 14 are rewritten to match. **Warp test (Gotcha 3):** the crashing write went to ChrCtrl `[PlayerIns + 0x18]` + 0x108..0x124, not `[PlayerIns + 0x68]`. A retry with `+0x68` has not been done, so the wrong base is likely but still not confirmed as the cause. **App ver:** 1.03.1 (title screen), added next to the Steam build id. **New results from today's run** - The touch trigger for shot-placed wall portals fires in game. - New Gotcha 15: a blind destination behind a wall was rejected twice, then accepted, and the player fell into the void. - Fly-state, zero-speed probe bullets stay fixed but never hit the player (ownerless with NPC atk 3146, ownerless with PC atk 222, player-owned with 3146; the player stood inside each one). `um kb check` passes; the index is regenerated. 🤖 Generated with [Claude Code](https://claude.com/claude-code) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi

2026-10-15 20:54:48 +00:00