anti/DECNET
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(web/session): subscribe to metadata/playing/idle/errored/reset/seeked too

Browse Source 
The original short subscribe list missed 'metadata' — which is the
one that carries the parsed duration + theme + marker info AFTER
_initializeDriver (the step that actually parses the cast). Without
it we only saw 'ready' (= UI mounted, parse not yet run) and jumped
to conclusions about the parser.

Add the full lifecycle set so the next repro pins which step the
player is actually getting stuck at.
This commit is contained in:
anti
2026-04-24 10:28:28 -04:00
parent b5c6b8a073
commit 442413870d
1 changed files with 8 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
decnet_web/src/components/SessionDrawer.tsx
View File

@@ -162,7 +162,14 @@ const SessionDrawer: React.FC<SessionDrawerProps> = ({ decky, sid, fields, onClo
// sync try/catch above and lands as an unhandled rejection. // sync try/catch above and lands as an unhandled rejection.
// Hook every lifecycle event so we can see which state it // Hook every lifecycle event so we can see which state it
// actually ends up in ("loading" / "ended" / "errored" / etc). // actually ends up in ("loading" / "ended" / "errored" / etc).
for (const evt of ['ready', 'play', 'pause', 'ended', 'error', 'errored', 'loading']) { // metadata is the one that carries the parsed duration, fires
// AFTER _initializeDriver (which does the actual cast parse).
// playing / idle tell us whether the timer ever advanced.
const events_to_hook = [
'ready', 'metadata', 'play', 'playing', 'pause', 'idle',
'ended', 'error', 'errored', 'loading', 'reset', 'seeked',
];
for (const evt of events_to_hook) {
try { try {
p.addEventListener?.(evt, (...args: unknown[]) => p.addEventListener?.(evt, (...args: unknown[]) =>
console.debug(`asciinema-player event: ${evt}`, ...args), console.debug(`asciinema-player event: ${evt}`, ...args),