Skip to content

fix: txt export now outputs plain text instead of markdown - #369

Merged
shouri123 merged 2 commits into
shouri123:mainfrom
nyxsky404:fix/txt-export-plain-text-format
Jun 2, 2026
Merged

fix: txt export now outputs plain text instead of markdown#369
shouri123 merged 2 commits into
shouri123:mainfrom
nyxsky404:fix/txt-export-plain-text-format

Conversation

@nyxsky404

@nyxsky404 nyxsky404 commented May 31, 2026

Copy link
Copy Markdown
Contributor

Description

The "Export as TXT" button was calling generateMarkdown() which produces markdown syntax (#, **, - [ ], _italics_). When opened in Notepad or TextEdit, these render as formatting noise rather than clean readable text.

Added a dedicated generatePlainText() function that produces properly formatted plain text.

Fixes #359

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

What Changed

  • src/dashboard.ts:
    • Added generatePlainText(state) function that outputs clean plain text with • bullets, indentation, and [done]/[ ] status markers
    • Updated the TXT export click handler to use generatePlainText() instead of generateMarkdown()

Example Output

Meeting Summary — May 31, 2026

Meeting ID: abc-defg-hij
Duration: 00:45:12
Sentiment: positive

Attendees
  • Alice
  • Bob

Summary
  Discussed Q4 roadmap and approved new tooling budget.

Action Items
  [done] Send proposal — Alice
  [ ] Review budget — Bob (due: June 5)

Key Decisions
  • Approved new tooling — CTO

How It Was Tested

  • npm run type-check — passes
  • npm run lint — passes
  • Verified TXT output contains no markdown syntax

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings

Copilot AI review requested due to automatic review settings May 31, 2026 12:59
@nyxsky404
nyxsky404 requested a review from shouri123 as a code owner May 31, 2026 12:59
@coderabbitai

coderabbitai Bot commented May 31, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

@nyxsky404, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 24 minutes and 19 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: af3d291f-8137-4476-b94b-29ddf4ae1fb0

📥 Commits

Reviewing files that changed from the base of the PR and between 34dd3ef and 241ea65.

📒 Files selected for processing (1)
  • src/dashboard.ts
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions Bot added gssoc Official GSSoC contribution issue gssoc:approved GSSoC: PR approved and scored labels May 31, 2026
@github-actions

github-actions Bot commented May 31, 2026

Copy link
Copy Markdown
Contributor

🚀 Thank You for Contributing to Late-Meet

Please ensure that:

  • the issue was assigned to you before opening this PR
  • the PR references the related issue
  • your changes follow repository contribution guidelines
  • the project builds successfully before submission

Unassigned, duplicate, or low-quality PRs may be closed.

Thank you for contributing 💙

@github-actions github-actions Bot added bug Something isn't working size/M type:design Type: Design / UI / UX (Optional bonus) type:docs Type: Documentation type:feature Type: New Feature labels May 31, 2026
@github-actions

github-actions Bot commented May 31, 2026

Copy link
Copy Markdown
Contributor

👋 Thank you @nyxsky404 for your contribution to Late-Meet!

✅ Verified: You are assigned to the linked issue #359.

Please review any automated suggestions or code review comments that may appear below! We will review your PR as soon as possible!

Please consider starring the repository ⭐ to show your support!

@github-actions github-actions Bot added the type:code Type: Code change label May 31, 2026
@nyxsky404 nyxsky404 changed the title fix: TXT export now outputs plain text instead of markdown fix: txt export now outputs plain text instead of markdown May 31, 2026
@shouri123

shouri123 commented Jun 1, 2026

Copy link
Copy Markdown
Owner

Warning: Pull Request Quality Audit Feedback

Hello @nyxsky404! Thank you for your contribution to Late-Meet!

Our automated PR checks have detected some issues with your PR. Please review the failed checks below and address them so we can get this merged:

Check Name Conclusion Log / Details Link
PR Title Lint failure View Workflow Logs

Please resolve these issues and push updates to your branch. The checks will automatically re-run. Let us know if you have any questions or need assistance!

@github-actions github-actions Bot removed the type:docs Type: Documentation label Jun 1, 2026
@shouri123 shouri123 added level:intermediate Difficulty level: Intermediate (35 pts) component:ai Component: AI/LLM integration labels Jun 1, 2026
@nyxsky404
fix: TXT export now outputs plain text instead of markdown
d781a56 
The 'Export as TXT' button was calling generateMarkdown() which produces
markdown syntax (# headers, **bold**, - [ ] checkboxes, _italics_). This
renders as formatting noise in plain text viewers.

Added a dedicated generatePlainText() function that produces clean plain
text with:
- Simple section headers (no # prefix)
- Bullet points using • character
- Indented content for readability
- [done]/[ ] for action item status
- Parenthesized metadata instead of markdown emphasis

Fixes shouri123#359
@nyxsky404
nyxsky404 force-pushed the fix/txt-export-plain-text-format branch from 7299c1f to d781a56 Compare June 1, 2026 16:29
@nyxsky404

nyxsky404 commented Jun 1, 2026

Copy link
Copy Markdown
Contributor Author

@shouri123 Fixed both issues:

  1. PR Title Lint — already renamed the title to start with a lowercase letter (was "TXT", now "txt")
  2. Out of date — rebased onto latest main

CI should be all green now. Thanks for flagging!

shouri123
shouri123 previously approved these changes Jun 1, 2026
View reviewed changes
@shouri123

shouri123 commented Jun 1, 2026

Copy link
Copy Markdown
Owner
image

fix this

123-Shouri-123
123-Shouri-123 previously approved these changes Jun 1, 2026
View reviewed changes
@nyxsky404
fix: remove nested template literal in generatePlainText (SonarCloud)
241ea65 
Extracted the ternary with nested template literal into a separate
variable to satisfy SonarCloud rule against nested template literals.
@sonarqubecloud

sonarqubecloud Bot commented Jun 1, 2026

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@nyxsky404

nyxsky404 commented Jun 1, 2026

Copy link
Copy Markdown
Contributor Author

@shouri123 Fixed the SonarCloud warning — refactored the nested template literal in generatePlainText() by extracting the ternary into a separate variable. Pushed in 241ea65. The PR title lint issue was already fixed earlier (renamed to lowercase).

@shouri123
shouri123 merged commit f88c5c0 into shouri123:main Jun 2, 2026
39 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@shouri123 shouri123 shouri123 approved these changes
@123-Shouri-123 123-Shouri-123 123-Shouri-123 approved these changes
Copilot code review Copilot Reviewing

Assignees

No one assigned

Labels

bug Something isn't working component:ai Component: AI/LLM integration gssoc:approved GSSoC: PR approved and scored gssoc Official GSSoC contribution issue level:intermediate Difficulty level: Intermediate (35 pts) size/M type:code Type: Code change type:design Type: Design / UI / UX (Optional bonus) type:feature Type: New Feature

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[BUG] TXT export uses generateMarkdown() — outputs markdown syntax instead of plain text

3 participants

@nyxsky404 @shouri123 @123-Shouri-123