Skip to content

refactor(app): share Beats track duration formatter - #2126

Merged
cursor[bot] merged 3 commits into
mainfrom
cursor/dedupe-music-duration-c596
Oct 23, 2026
Merged

cursor[bot] merged 3 commits into
mainfrom
cursor/dedupe-music-duration-c596

Conversation

@ucguy4u

@ucguy4u ucguy4u commented Oct 23, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add formatMusicTrackDuration for Beats player labels (m:ss).
  • Replace three identical private _formatDuration helpers in music_screen.dart with the shared function.

Test plan

  • cd app && flutter test test/features/music/presentation/music_duration_format_test.dart
Open in Web Open in Cursor 

@github-actions

Copy link
Copy Markdown
Contributor

Plugin Module Size Gate

Policy: modules over 3 MB must be delivered as plugins; plugin modules over 5 MB must document cache management.

Module Size Type Status
No changed Dart/Flutter modules detected n/a n/a OK

@github-actions

Copy link
Copy Markdown
Contributor

🚀 PR Quick Check Summary

Check Status Description
PR Validation ❌ failure Title format, docs, bundled model guardrail
Code Quality ❌ failure Analyze, formatting
Core Tests ✅ success Core package unit tests
Airo Mind Shell ✅ success Mind flavour analyze, tests, entrypoint compile

💡 Note: Full app tests, coverage reports, and security scans run on merge to main.

View Details

@ucguy4u ucguy4u added the docs-not-needed PR intentionally ships without a docs/wiki update label Oct 23, 2026 — with Cursor
@cursor
cursor Bot force-pushed the cursor/dedupe-music-duration-c596 branch from 06dff6d to b33f00f Compare October 23, 2026 21:34
@github-actions

Copy link
Copy Markdown
Contributor

🚀 PR Quick Check Summary

Check Status Description
PR Validation ✅ success Title format, docs, bundled model guardrail
Code Quality ❌ failure Analyze, formatting
Core Tests ✅ success Core package unit tests
Airo Mind Shell ✅ success Mind flavour analyze, tests, entrypoint compile

💡 Note: Full app tests, coverage reports, and security scans run on merge to main.

View Details

@github-actions

Copy link
Copy Markdown
Contributor

🚀 PR Quick Check Summary

Check Status Description
PR Validation ✅ success Title format, docs, bundled model guardrail
Code Quality ❌ failure Analyze, formatting
Core Tests ✅ success Core package unit tests
Airo Mind Shell ✅ success Mind flavour analyze, tests, entrypoint compile

💡 Note: Full app tests, coverage reports, and security scans run on merge to main.

View Details

@github-actions

Copy link
Copy Markdown
Contributor

🚀 PR Quick Check Summary

Check Status Description
PR Validation ✅ success Title format, docs, bundled model guardrail
Code Quality ✅ success Analyze, formatting
Core Tests ✅ success Core package unit tests
Airo Mind Shell ✅ success Mind flavour analyze, tests, entrypoint compile

💡 Note: Full app tests, coverage reports, and security scans run on merge to main.

View Details

cursoragent and others added 3 commits October 23, 2026 22:00
Extract formatMusicTrackDuration and use it from music_screen instead of
three identical private helpers.

docs-not-needed: internal dedupe only; no user-facing behavior change.

Co-authored-by: Uday Chauhan <chauhan.s.uday26@gmail.com>
Co-authored-by: Uday Chauhan <chauhan.s.uday26@gmail.com>
Co-authored-by: Uday Chauhan <chauhan.s.uday26@gmail.com>
@cursor
cursor Bot force-pushed the cursor/dedupe-music-duration-c596 branch from ecdae80 to 5b89735 Compare October 23, 2026 22:00
@github-actions

Copy link
Copy Markdown
Contributor

🚀 PR Quick Check Summary

Check Status Description
PR Validation ✅ success Title format, docs, bundled model guardrail
Code Quality ✅ success Analyze, formatting
Core Tests ✅ success Core package unit tests
Airo Mind Shell ✅ success Mind flavour analyze, tests, entrypoint compile

💡 Note: Full app tests, coverage reports, and security scans run on merge to main.

View Details

@cursor
cursor Bot merged commit 31c932e into main Oct 23, 2026
20 checks passed
@cursor
cursor Bot deleted the cursor/dedupe-music-duration-c596 branch October 23, 2026 22:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs-not-needed PR intentionally ships without a docs/wiki update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants