Skip to content

Lastfm#93

Open
FranciscoG wants to merge 4 commits intomainfrom
lastfm
Open

Lastfm#93
FranciscoG wants to merge 4 commits intomainfrom
lastfm

Conversation

@FranciscoG
Copy link
Copy Markdown
Owner

This branch started off as one thing, turned into another, and then morphed into what is now

  • Most important: I'm updating dependencies to the latest
  • Continuing to add types via JSDoc
  • Upgrade min node version to 20
  • replace Chai/Mocha with Node's built-in test runners

I was going to originally fix lastfm but abandoning that

upgrade core runtime dependencies, including DubAPI, Firebase Admin, Sheets, and other packages
raise the Node.js engine requirement to 20 and update the lockfile for the new dependency set
migrate tests from Mocha/Chai to the built-in node:test runner and rename test files to the new pattern
update the commands build script to use the current marked package without require.extensions hacks
add MusicBrainz title matching and shared string normalization helpers for track metadata lookup
extend the Last.fm utility with track search support and tighten JSON/error handling
improve bot typings and JSDoc coverage for DubAPI extensions, song history, private settings, and database usage
clean up playlist/history/youtube handling with safer property access and async fetch-based request flow
adjust spreadsheet integration to the newer Sheets client and document the current auth limitation
apply general compatibility and code modernization cleanup across the touched bot modules
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