This PR implements filtering to exclude songs containing "TODO" from the song collection, as there is no need to add incomplete songs to the database.

Changes Made

• Added song_contains_todo() function: Checks all song fields (name, melody, composer, arranger, lyrics, notes) for "TODO" content
• Modified extraction logic: Songs with TODO in any field are now filtered out before being added to songs.json
• Added comprehensive tests: 9 test cases covering all song fields and edge cases

Implementation Details

The filtering is implemented in extract_songs.py with minimal code changes:

def song_contains_todo(song: SongInfo) -> bool:
    """Check if a song contains 'TODO' in any of its fields."""
    fields_to_check = [
        song.name, song.melody, song.composer, 
        song.arranger, song.lyrics, song.notes
    ]
    
    for field in fields_to_check:
        if field and "TODO" in field:
            return True
    
    return False

The filter is applied in the main extraction loop:

if song.name != "Parse Error":
    # Filter out songs that contain TODO in any field
    if not song_contains_todo(song):
        songs.append(asdict(song))

Testing

• ✅ All unit tests pass (9/9)
• ✅ Integration testing confirms TODO songs are excluded
• ✅ Backwards compatibility maintained
• ✅ No syntax or compilation errors

The implementation is case-sensitive and safely handles None values in song fields.

Fixes #5.

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.