Improve meeting dashboard and analysis context - #19
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates Parley’s live meeting experience by making the analysis UI more “outline-first,” improving analysis prompt context (including bounded transcript overlap), and preserving session context via a stored snapshot that can be exported alongside the full transcript.
Changes:
- Reworks analysis prompting/state to support a topic outline with discussion points, bounded transcript overlap, and less provider-forcing (no fixed temperature / reasoning directives).
- Snapshots meeting context at session creation (with legacy fallback) and adds a “context + transcript” export mode.
- Refreshes the frontend workbench layout (discussion outline + panels) and adds an export-choice dialog.
Reviewed changes
Copilot reviewed 20 out of 20 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| README.md | Updates documentation to reflect the new analysis interval behavior, outline-based analysis state, and export options. |
| meeting_service.go | Adds context snapshot usage for analysis context and introduces transcript export via a shared markdown export helper. |
| meeting_service_test.go | Adds coverage to ensure analysis context prefers the session snapshot over the active profile. |
| internal/store/store.go | Updates default analysis timeout to 60s and adds a new sessions column for context snapshot JSON. |
| internal/store/sessions.go | Introduces ContextSnapshot, persists it on session creation, and hydrates it in session bundles. |
| internal/store/sessions_test.go | Updates session creation calls and validates context snapshot round-tripping. |
| internal/llm/client.go | Removes forced temperature from chat completion requests. |
| internal/llm/client_test.go | Asserts the client does not send temperature (and still does not send max_tokens). |
| internal/export/export.go | Enhances notes export summary fallback and adds a context+transcript markdown renderer. |
| internal/export/export_test.go | Adds tests for derived outline summary and transcript export content/exclusions. |
| internal/analysis/engine.go | Adds per-topic discussion points, bounded transcript overlap, expanded prior state, and updated prompting rules. |
| internal/analysis/engine_test.go | Updates/extends tests for prompting changes, topic rollover details, and overlap separation. |
| frontend/src/components/ui/scroll-area.tsx | Allows passing a viewport ref to enable scroll-follow behavior. |
| frontend/src/components/SettingsDialog.tsx | Updates analysis timeout defaults to 60s. |
| frontend/src/components/SessionsDialog.tsx | Changes export interaction to trigger a centralized export flow (no per-row spinner). |
| frontend/src/components/ExportDialog.tsx | Adds a new dialog to choose between notes export and context+transcript export. |
| frontend/src/components/AnalysisPanels.tsx | Rebuilds the workbench panels around an auto-following “Discussion outline” and updated panel layout. |
| frontend/src/App.tsx | Integrates export dialog and updates analysis state handling for points/assertions/suggestions. |
| frontend/bindings/github.com/tomvokac/parley/meetingservice.ts | Adds binding for ExportTranscriptMarkdown. |
| frontend/bindings/github.com/tomvokac/parley/internal/analysis/models.ts | Updates topic model to include points. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Collaborator
Author
|
@copilot resolve the merge conflicts in this pull request |
Contributor
Resolved by merging |
Copilot stopped work on behalf of
vokac-nri due to an error
July 16, 2026 15:42
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
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.
Summary
Verification
go test -count=1 ./internal/analysis ./internal/llm ./internal/export ./internal/store .npx tsc --noEmitnpm run build