Skip to content

v0.6.2 - #29

Merged
pnxenopoulos merged 2 commits into
mainfrom
feature/v0.6.2
Jul 27, 2026
Merged

v0.6.2#29
pnxenopoulos merged 2 commits into
mainfrom
feature/v0.6.2

Conversation

@pnxenopoulos

Copy link
Copy Markdown
Owner

Two fixes and a workspace version bump 0.6.1 → 0.6.2.

active_modifiers reported stale stack counts

First reported on Discord. The active_modifiers change-detector read a modifier's stack_count once, the first tick it saw the modifier, and never again. Every later in-place update hit the already-tracked branch and was dropped, so:

  • a stacking modifier that climbed (e.g. a Spellslinger Headshots debuff going 2 → 3 → 4) kept reporting its first-seen value
  • the removed row echoed that stale value, not the final total (every apply/remove pair had identical stacks — the tell-tale sign).

In-place stack changes are now captured: a new event value "changed" is emitted on the tick a live modifier's stacks moves, and removed reports the final count. applied / removed semantics are otherwise unchanged. The boon-dev active-modifiers command gains the same changed events.

Opening a demo no longer requires a match ID

Demo(path) used to resolve the match ID (and game mode) from CCitadelGameRulesProxy on the first tick and raise
DemoMessageError: could not resolve match ID … if it couldn't — rejecting a demo that was otherwise fully parseable (partial captures, sandbox / custom content).

  • Demo.match_id is now int | None — recorded when present, None otherwise.
  • Demo.game_mode (previously resolved in the same step, so it failed together with the match ID) now falls back to 0 when the game-rules entity is absent.

Such demos now open normally, instead of failing during construction.

@pnxenopoulos
pnxenopoulos merged commit 27b88b2 into main Jul 27, 2026
15 checks passed
@pnxenopoulos
pnxenopoulos deleted the feature/v0.6.2 branch July 27, 2026 23:30
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.

1 participant