Skip to content

Make learning sessions durable, retryable, and safe across memory resets - #1

Merged
Jo2234 merged 2 commits into
mainfrom
codex/approved-audit-fixes-2026-09-06
Sep 7, 2026
Merged

Jo2234 merged 2 commits into
mainfrom
codex/approved-audit-fixes-2026-09-06

Conversation

@Jo2234

@Jo2234 Jo2234 commented Sep 6, 2026 •

Copy link
Copy Markdown
Owner

Learning progress can be lost when multiple clients save concurrently, duplicated when a quiz is retried, or restored by delayed callbacks after memory deletion. Use process-safe, reentrant locks with atomic file replacement, short reload/apply transactions, stable session IDs and reset epochs. Commit completion receipts with progress so repeated submissions return the committed result. Preserve each Gradio practice session's actual language when applying its transaction.

Voice calls now share an awaited, retryable finalizer across navigation, language changes and window close. Captured call identity prevents the selected language from changing where progress is saved; failed persistence retains the transcript/checkpoint. Delete All Memory clears managed checkpoints, speech cache and recovery files and invalidates pending work. Camera analysis skips unchanged samples before inference, with explicit-capture and maximum-age refresh. Include curriculum JSON in Python distributions and run renderer regressions in CI.

Validation: 133 Python tests passed across the full suite and the four separately rerun local-loopback cases; 11 actual-renderer VM scenarios and existing JavaScript checks, Ruff, and diff whitespace checks pass. Two new Gradio regressions failed before the language fix and pass afterward. The original implementation also validated built wheel/sdist and installed Spanish/French/Hindi curriculum loading; no heavy build was repeated for the callback fix. No real provider calls were used. Physical camera behavior and packaged native window lifecycle remain manual acceptance checks; locking targets POSIX macOS/Linux.

CI follow-up: declare the directly imported certifi dependency explicitly. A fresh isolated wheel install with OpenAI 3.8.0 resolves certifi and passes dependency checks, source/wheel imports, SSL context creation and all 133 tests.

@Jo2234
Jo2234 marked this pull request as ready for review September 7, 2026 02:41
@Jo2234
Jo2234 merged commit d923a6a into main Sep 7, 2026
4 checks passed
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