troytt
/
diablo2-web
1
0
Fork 0
Code Issues 15 Pull Requests Packages Projects Releases Wiki Activity

[Skill #261] [ASS] Charged Bolt Sentry (1.13c Parity) #354

New Issue
Closed
opened 2026-09-22 02:40:27 +00:00 by troytt · 2 comments
troytt commented 2026-09-22 02:40:27 +00:00
Owner
Copy Link

Diablo II: Lord of Destruction v1.13c Skill Parity Specification

Field Value
Skill ID 261 (#261)
Skill Name Charged Bolt Sentry (skilldesc: charged bolt sentry)
Class ass (Assassin)
Required Level (reqlevel) 12
Prerequisites (reqskill1..3) Shock Field, -, -
Isolated Implementation Module src/game/skills/impl/ass/skill-261-charged-bolt-sentry.ts
Isolated Vitest Suite tests/skills/ass/skill-261-charged-bolt-sentry.test.ts

1. Authoritative 1.13c Skills.txt Function & Handler Dispatch

Function Slot 1.13c Value
srvstfunc (Server Start / Pre-Cast) -
srvdofunc (Server Action Frame 0x01/0x02) 45
srvprgfunc1..3 (Progressive Charge Release) - / - / -
cltstfunc / cltdofunc (Client Start / Action) - / -
cltprgfunc1..3 (Client Progressive Release) - / - / -
passive / checkfunc passive=- / checkfunc=-
auraevent1 / auraeventfunc1 - / -

2. Associated Missiles, States, Auras, Curses & Summons

Subsystem 1.13c Configuration
Server Missiles (srvmissile / a..c) - / a:-, b:-, c:-
Client Missiles (cltmissile / cltmissilea) - / -
Aura / Buff States (aurastate / auratgtstate) self:- / tgt:-
Aura / Buff Stats (aurastat1..3) -, -, -
Passive State & Stats (passivestate / passivestat1..3) state:- / -, -, -
Curse Priority (cursetype) -
Summon / Pet (summon / pettype / petmax) mon:chargeboltsentry / type:assassintrap / max:5

3. 1.13c Base Numerical & Synergy Formulas (Patch_D2.mpq)

  • Mana Cost: mana=13, lvlmana=0 (256-fixed-point << HitShift)
  • Physical / Elemental Damage: MinDam=-, MaxDam=-, EType=ltng, EMin=2, EMax=14, ELen=-
  • AST calc1..4: calc1=-, calc2=-, calc3=-, calc4=par1 + skill('Lightning Sentry'.blvl)/4
  • Synergy Calculations (blvl strictly base allocated STAT_SKILLPTS only):
    • EDmgSymPerCalc: (skill('Fire Trauma'.blvl) + skill('Lightning Sentry'.blvl) + skill('Death Sentry'.blvl)) * par8
    • DmgSymPerCalc: -
    • ELenSymPerCalc: -

4. R3 Verification & Merge Checklist

  • Isolated Git Worktree: Created via git worktree add ../wt-skill-261 -b feat/skill-261 from main.
  • 1.13c Implementation: src/game/skills/impl/ass/skill-261-charged-bolt-sentry.ts implements exact 1.13c srvstfunc/srvdofunc/srvprgfunc & numerical evaluation.
  • Automated Unit & Numerical Tests (slvl 1, 10, 20 + blvl Synergies): tests/skills/ass/skill-261-charged-bolt-sentry.test.ts passes 100% in Vitest verifying ManaCost, MinDmg, MaxDmg, ToHit, Duration, Radius, synergy scaling, and runtime combat/missile/state/summon effect.
  • Real-Browser Casting & Screenshot: Executed in headless Chromium (acts.html?arena=1&class=ass&skill=261&slvl=20&autocast=1) with zero console/page errors and PNG screenshot attached to this Issue.
  • Merge & Cleanup: Merged to main with fixes #<issue_number>, pushed to origin/main, verified Issue is closed, and removed worktree ../wt-skill-261 + branch feat/skill-261.
## Diablo II: Lord of Destruction v1.13c Skill Parity Specification | Field | Value | |---|---| | **Skill ID** | `261` (`#261`) | | **Skill Name** | **Charged Bolt Sentry** (`skilldesc: charged bolt sentry`) | | **Class** | `ass` (Assassin) | | **Required Level (`reqlevel`)** | `12` | | **Prerequisites (`reqskill1..3`)** | `Shock Field`, `-`, `-` | | **Isolated Implementation Module** | `src/game/skills/impl/ass/skill-261-charged-bolt-sentry.ts` | | **Isolated Vitest Suite** | `tests/skills/ass/skill-261-charged-bolt-sentry.test.ts` | --- ### 1. Authoritative 1.13c `Skills.txt` Function & Handler Dispatch | Function Slot | 1.13c Value | |---|---| | **`srvstfunc`** (Server Start / Pre-Cast) | `-` | | **`srvdofunc`** (Server Action Frame `0x01`/`0x02`) | `45` | | **`srvprgfunc1..3`** (Progressive Charge Release) | `-` / `-` / `-` | | **`cltstfunc` / `cltdofunc`** (Client Start / Action) | `-` / `-` | | **`cltprgfunc1..3`** (Client Progressive Release) | `-` / `-` / `-` | | **`passive` / `checkfunc`** | `passive=-` / `checkfunc=-` | | **`auraevent1` / `auraeventfunc1`** | `-` / `-` | ### 2. Associated Missiles, States, Auras, Curses & Summons | Subsystem | 1.13c Configuration | |---|---| | **Server Missiles (`srvmissile` / `a..c`)** | `-` / `a:-`, `b:-`, `c:-` | | **Client Missiles (`cltmissile` / `cltmissilea`)** | `-` / `-` | | **Aura / Buff States (`aurastate` / `auratgtstate`)** | `self:-` / `tgt:-` | | **Aura / Buff Stats (`aurastat1..3`)** | `-`, `-`, `-` | | **Passive State & Stats (`passivestate` / `passivestat1..3`)** | `state:-` / `-`, `-`, `-` | | **Curse Priority (`cursetype`)** | `-` | | **Summon / Pet (`summon` / `pettype` / `petmax`)** | `mon:chargeboltsentry` / `type:assassintrap` / `max:5` | ### 3. 1.13c Base Numerical & Synergy Formulas (`Patch_D2.mpq`) - **Mana Cost**: `mana=13`, `lvlmana=0` (256-fixed-point `<< HitShift`) - **Physical / Elemental Damage**: `MinDam=-`, `MaxDam=-`, `EType=ltng`, `EMin=2`, `EMax=14`, `ELen=-` - **AST `calc1..4`**: `calc1=-`, `calc2=-`, `calc3=-`, `calc4=par1 + skill('Lightning Sentry'.blvl)/4` - **Synergy Calculations (`blvl` strictly base allocated `STAT_SKILLPTS` only)**: - `EDmgSymPerCalc`: `(skill('Fire Trauma'.blvl) + skill('Lightning Sentry'.blvl) + skill('Death Sentry'.blvl)) * par8` - `DmgSymPerCalc`: `-` - `ELenSymPerCalc`: `-` --- ### 4. R3 Verification & Merge Checklist - [ ] **Isolated Git Worktree**: Created via `git worktree add ../wt-skill-261 -b feat/skill-261` from `main`. - [ ] **1.13c Implementation**: `src/game/skills/impl/ass/skill-261-charged-bolt-sentry.ts` implements exact 1.13c `srvstfunc`/`srvdofunc`/`srvprgfunc` & numerical evaluation. - [ ] **Automated Unit & Numerical Tests (`slvl` 1, 10, 20 + `blvl` Synergies)**: `tests/skills/ass/skill-261-charged-bolt-sentry.test.ts` passes 100% in Vitest verifying `ManaCost`, `MinDmg`, `MaxDmg`, `ToHit`, `Duration`, `Radius`, synergy scaling, and runtime combat/missile/state/summon effect. - [ ] **Real-Browser Casting & Screenshot**: Executed in headless Chromium (`acts.html?arena=1&class=ass&skill=261&slvl=20&autocast=1`) with zero console/page errors and PNG screenshot attached to this Issue. - [ ] **Merge & Cleanup**: Merged to `main` with `fixes #<issue_number>`, pushed to `origin/main`, verified Issue is `closed`, and removed worktree `../wt-skill-261` + branch `feat/skill-261`.
skill-261-charged-bolt-sentry.png
575 KiB
skill-261-charged-bolt-sentry.png
troytt added this to the All Player Skills (v1.13c Parity) milestone 2026-09-22 02:40:27 +00:00
troytt commented 2026-09-22 04:00:11 +00:00
Author
Owner
Copy Link

✅ Completed: [Skill #261] Charged Bolt Sentry (ASS) — 1.13c Parity Verified

  • Worktree & Branch: ../wt-skill-261 (feat/skill-261)
  • Skill Module: src/game/skills/impl/ass/skill-261-charged-bolt-sentry.ts
  • Vitest Suite: tests/skills/ass/skill-261-charged-bolt-sentry.test.ts
  • 1.13c Numbers (slvl 1 / 10 / 20): Mana 13 / 13 / 13, Phys 0-0 / 0-0 / 0-0, Elem 1-7 / 1-36 / 1-84 (ltng)
  • Chromium Arena Telemetry: ready=true, castCount=1, anim=S2:4, consoleErrors=0

Skill #261 Charged Bolt Sentry

### ✅ Completed: [Skill #261] Charged Bolt Sentry (`ASS`) — 1.13c Parity Verified - **Worktree & Branch**: `../wt-skill-261` (`feat/skill-261`) - **Skill Module**: `src/game/skills/impl/ass/skill-261-charged-bolt-sentry.ts` - **Vitest Suite**: `tests/skills/ass/skill-261-charged-bolt-sentry.test.ts` - **1.13c Numbers (slvl 1 / 10 / 20)**: Mana `13` / `13` / `13`, Phys `0-0` / `0-0` / `0-0`, Elem `1-7` / `1-36` / `1-84` (`ltng`) - **Chromium Arena Telemetry**: `ready=true, castCount=1, anim=S2:4, consoleErrors=0` ![Skill #261 Charged Bolt Sentry](https://git.projectdiablo2.cn/attachments/1d4175b1-6473-49cb-97c3-1d9866221e2f)
troytt commented 2026-09-25 17:30:08 +00:00
Author
Owner
Copy Link

1.13c Ground Truth Remediation Verified (Cohort 7)

  • Verified 1.13c assembly parity via D2MOO and MPQ data
  • Tests passing: tests/skills/ass/ (34 test files, 166 tests passing, 100% clean)
  • Gate tests passing: d2moo-skills-diff-gate.test.ts & global-master-parity-gate.test.ts
  • TypeScript typecheck: 0 errors
### 1.13c Ground Truth Remediation Verified (Cohort 7) - Verified 1.13c assembly parity via D2MOO and MPQ data - Tests passing: `tests/skills/ass/` (34 test files, 166 tests passing, 100% clean) - Gate tests passing: `d2moo-skills-diff-gate.test.ts` & `global-master-parity-gate.test.ts` - TypeScript typecheck: 0 errors
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
main
feat/automap-113c
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
All Player Skills (v1.13c Parity)
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: troytt/diablo2-web#354
No description provided.
Delete Branch "%!s(<nil>)"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?