Conversation
Adds a Timeline sub-tab to the Characters section in Show Config. Acts and scenes are displayed on the x-axis; characters with at least one spoken dialogue line occupy rows on the y-axis. Coloured bars span consecutive scenes where a character speaks, breaking at act boundaries. Reuses the existing timelineMixin (V2) / useTimeline composable (V3) and the /api/v1/show/character/stats endpoint — no backend changes required. Also adds an E2E smoke test for the Timeline tab and documents the feature. Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
cast/stats and character/stats were firing 5,137 individual SQL queries per request (~8s) due to nested lazy-loaded ORM traversal. Replace with selectinload chains that load all required data in ~8 bulk queries, and a single upfront set-query for revision-scoped cuts rather than one check per line part. Also fixes character list N+1 (per-character character_group and mic_allocation lazy loads) via selectinload on CharacterController, and removes Act.lines / Scene.lines from their schemas — both relationships loaded ~950 ORM rows per act/scene on every list request but were never read by either frontend client. Adds 11 behavioural count-verification tests for cast/stats and character/stats to document and guard the exact counting semantics. Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
github-actions
Bot
added
documentation
Client Test Results128 tests 128 ✅ 0s ⏱️ Results for commit c1a535c. ♻️ This comment has been updated with latest results. |
|
Client V3 Test Results23 tests 23 ✅ 0s ⏱️ Results for commit c1a535c. |
Playwright E2E Results (chromium)164 tests 164 ✅ 1m 39s ⏱️ Results for commit c1a535c. |
Python Test Results 1 files 1 suites 2m 1s ⏱️ Results for commit c1a535c. |
Playwright E2E Results (firefox)164 tests 164 ✅ 1m 41s ⏱️ Results for commit c1a535c. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.