Skip to content

Fix sub-threshold sessions leaving transcript and summary artifacts#4509

Open
ComputelessComputer wants to merge 2 commits intomainfrom
fix-hide-empty-transcript-and-summary-tabs
Open

Fix sub-threshold sessions leaving transcript and summary artifacts#4509
ComputelessComputer wants to merge 2 commits intomainfrom
fix-hide-empty-transcript-and-summary-tabs

Conversation

@ComputelessComputer
Copy link
Collaborator

@ComputelessComputer ComputelessComputer commented Mar 11, 2026

reasoning: when we really have nothing recorded nor jotted down, it should remain in a clean state. but currently we always get three tabs: summary, memos, and transcript.

  • Prevent empty Transcript and Summary tabs for sessions below the transcript threshold
  • Skip persisting transcript artifacts when captured content is too small
  • Reset invalid selected tabs to a valid fallback
  • Add regression tests

@netlify
Copy link

netlify bot commented Mar 11, 2026

Deploy Preview for hyprnote-storybook canceled.

Name Link
🔨 Latest commit a94d4e2
🔍 Latest deploy log https://app.netlify.com/projects/hyprnote-storybook/deploys/69ba2906e736d00008521fea

@netlify
Copy link

netlify bot commented Mar 11, 2026

Deploy Preview for hyprnote ready!

Name Link
🔨 Latest commit a94d4e2
🔍 Latest deploy log https://app.netlify.com/projects/hyprnote/deploys/69ba2906b6d7950008a0c150
😎 Deploy Preview https://deploy-preview-4509--hyprnote.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@ComputelessComputer ComputelessComputer force-pushed the fix-hide-empty-transcript-and-summary-tabs branch from ab584a3 to 5002f50 Compare March 11, 2026 08:32
@ComputelessComputer ComputelessComputer requested review from goranmoomin and removed request for yujonglee March 16, 2026 04:30
@goranmoomin goranmoomin force-pushed the fix-hide-empty-transcript-and-summary-tabs branch from 42584eb to cfaf3f4 Compare March 18, 2026 03:13
ComputelessComputer and others added 2 commits March 18, 2026 13:22
Add comprehensive fallback handling when selected tabs become
unavailable. Enhance transcript word counting with error handling
and improve type safety in eligibility checks.

- Add fallback to raw tab when transcript is no longer available
- Fallback to first enhanced note when selected note is missing
- Improve error handling in transcript word parsing with try-catch
- Replace hardcoded threshold with shared constant
- Add comprehensive test coverage for tab computation scenarios
- Strengthen type definitions for transcript word store interface
@goranmoomin goranmoomin force-pushed the fix-hide-empty-transcript-and-summary-tabs branch from cfaf3f4 to a94d4e2 Compare March 18, 2026 04:24
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.

2 participants