Local recording ends at Finish, before the learning review

Offen Anfängerfreundlich
#663 2 Kommentare 0 Reaktionen 1 zugewiesene Person Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

@richard81115 arbeitet bereits daran.

Seit 11.11.2026.

  • #679 von @richard81115 — offen

Bewertung

Schwierigkeit
1/5
Geschätzter Aufwand
Unter einer Stunde
Anfängerfreundlichkeit
84/100
Issue-Typ
Dokumentation
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
rust
Bereich
documentation

Rechercherichtung

Lies den Abschnitt „Local recordings“ in docs/task-mode.md und prüfe seine Beschreibung anhand des hier gemeldeten Verhaltens: Die Aufzeichnung endet, wenn der Versuch abgeschlossen ist, bevor die Lernbesprechung erscheint. Füge einen Satz hinzu, der erklärt, dass die Besprechung nicht enthalten ist und bei Bedarf separat aufgenommen werden muss. Erledigt, wenn die Dokumentation klar beschreibt, wo die Aufzeichnung endet und was Lernende tun können, um die Besprechung aufzunehmen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Observed behavior

The local task recorder stops as soon as the attempt is finished, so the downloaded WebM ends right after the "Finish and freeze this revision" dialog and never shows the learning review. docs/task-mode.md says that completion finalizes the recording, so the behavior matches the documentation.

Why it matters

The course assignment asks for a recording "from the rules page to the learning review after Finish". The built-in recorder cannot capture that last part, and a new capture cannot be started during the attempt.

Steps to reproduce

  1. Start the server with cargo run --release -- web --task-mode and open http://localhost:3000 in Chrome.
  2. Accept the rules, allow the microphone and camera, and calibrate.
  3. On the preparation page, start the local recording, choose this tab and share tab audio.
  4. Press Start, work on the task, press Finish and confirm.
  5. Open the downloaded WebM. Its last frame shows the workspace just after the dialog closes; the learning review that the page shows next is not in the file.

Cause

In web/task.js, attemptOver() calls stopMonitoring(), which calls localRecording.finish() and releases the camera and microphone. The result arrives later and renderReview() shows it, after the recorder has already saved the file.

Plan

The change I can send is a documentation one: a sentence in the "Local recordings" section of docs/task-mode.md stating that the recording ends when the attempt is finished and therefore does not include the learning review, so a learner who needs the review on video knows to capture it separately.

Recording through the review would be a larger change: for an attempt that ends with Finish, keep the tab capture running after monitoring stops, let the recorder drop its microphone input without reporting a lost microphone, and finalize in renderReview() once the review is in the page, with browser tests for both paths. I cannot commit to that before the assignment deadline, so I am leaving it as a direction rather than a promise.

Environment

codetrial 0.1.0 (5570fe9), built from source; server in WSL Ubuntu on Windows 11; Chrome 157.0.8103.39; default task rubik, language C.

Vorherrschende Sprache
Rust
Sterne
228
Forks
150
Ø Merge
1 T. 3 Std.
Gemergte PRs (30 T.)
226

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus sysprog21/codetrial

Alle Issues in sysprog21/codetrial

Ähnliche Issues

Weitere Issues zu Rust

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.