Skip to content

Add ChordPro lyrics support - #152

Draft
RedyAu wants to merge 1 commit into
mainfrom
feat/chordpro-support
Draft

RedyAu wants to merge 1 commit into
mainfrom
feat/chordpro-support

Conversation

@RedyAu

@RedyAu RedyAu commented Apr 5, 2026

Copy link
Copy Markdown
Member

Summary

  • add a new dart_chordpro parser package in the Reformatus org and consume it as a pinned git dependency
  • extend the app lyrics parsing layer with format-agnostic verse, line, comment, and chord segment primitives
  • add ChordPro support for inline chords, verse/chorus/bridge blocks, chorus recall, comments, and page breaks while keeping OpenSong rendering on the same UI path

Companion repo

Validation

  • dart analyze in dart_chordpro
  • dart test in dart_chordpro
  • flutter analyze lib/data/song/lyrics/format.dart lib/data/song/lyrics/parser.dart lib/services/song/verse_tag_pretty.dart lib/ui/song/lyrics/view.dart test/data/lyrics_parser_test.dart test/data/song_from_bank_api_json_test.dart
  • flutter test test/data

Notes

  • full-project flutter analyze still reports existing warnings in lib/services/app_links/cue_compression.dart, lib/services/cue/compression.dart, and test/router/imperative_url_reflection_test.dart; this PR does not add new analyzer issues

@RedyAu
RedyAu marked this pull request as draft April 18, 2026 15:21
@RedyAu RedyAu self-assigned this Apr 18, 2026
@RedyAu
RedyAu force-pushed the feat/chordpro-support branch 2 times, most recently from d42c155 to cabc028 Compare April 19, 2026 09:49
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.

1 participant