Skip to content

eamuse: include card update time in inquiries - #19

Merged
TsFreddie merged 1 commit into
asphyxia-core:masterfrom
Avimitin:dance-around-support
Aug 30, 2026
Merged

eamuse: include card update time in inquiries#19
TsFreddie merged 1 commit into
asphyxia-core:masterfrom
Avimitin:dance-around-support

Conversation

@Avimitin

Copy link
Copy Markdown
Contributor

Return a positive Unix timestamp from cardmng.inquire. DANCE aROUND treats a registered card response without this field as a temporarily busy reader.

@dannylin0711

Copy link
Copy Markdown
Contributor

lol, that's so weird

@Avimitin
Avimitin force-pushed the dance-around-support branch from 2f45693 to c5955bb Compare August 30, 2026 16:28
@Avimitin

Avimitin commented Aug 30, 2026

Copy link
Copy Markdown
Contributor Author

Sorry for AI slop, I didn't realize that this was intended only as a quick local fix. Searching the game's DLLs did not reveal enough evidence to determine the exact semantics of lastupdate. The field is required in a bound-card cardmng.inquire response because the protocol layer parses it, but the parsed value is never consumed by the game code. The game later sends another inquiry with update=1, so I cannot conclude that the field is never updated. Using Date.now() is clearly not appropriate. Reusing updatedAt may not be semantically exact either, but it is a minimal persistent fallback that works for this title because the returned value is not actually used.

@Avimitin
Avimitin force-pushed the dance-around-support branch from c5955bb to 15fecca Compare August 30, 2026 16:34
Return the lastupdate attribute required by clients such as DANCE aROUND using the card record's persisted NeDB updatedAt timestamp.

Co-authored-by: Codex <codex@openai.com>
@Avimitin
Avimitin force-pushed the dance-around-support branch from 15fecca to 403bd38 Compare August 30, 2026 16:35
@TsFreddie

Copy link
Copy Markdown
Contributor

as it stands currently it doesn't hurt to just have an extra field afaik.

@TsFreddie
TsFreddie merged commit 51a3327 into asphyxia-core:master Aug 30, 2026
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.

3 participants