add conditio to now playing track - it does not have date yet (#1)

Author: landsman

.github/workflows/test.yml

@@ -1,6 +1,8 @@
 name: Check
 
-on: push
+on:
+  - push
+  - pull_request
 
 jobs:
   qa:

README.md

@@ -38,3 +38,8 @@ supabase start
 supabase function serve --no-verify-jwt
 ```
 
+**deploy modified function**
+
+```bash
+supabase functions deploy --project-ref xxx
+```

supabase/functions/fetch-scrobbles/index.ts

@@ -78,14 +78,16 @@ Deno.serve(async () => {
 
     console.log(`Fetching page ${page}/${totalPages}`);
 
-    const toInsert: Row[] = tracks.map((track) => ({
-      created_at: new Date().toISOString(),
-      listened_at: new Date(track.date.uts * 1000).toISOString(),
-      artist_name: track.artist.name,
-      track_name: track.name,
-      album_name: track.album["#text"],
-      lastfm_data: track,
-    }));
+    const toInsert: Row[] = tracks
+      .filter((track) => !(track["@attr"] && track["@attr"].nowplaying))
+      .map((track) => ({
+        created_at: new Date().toISOString(),
+        listened_at: new Date(track.date!.uts * 1000).toISOString(),
+        artist_name: track.artist.name,
+        track_name: track.name,
+        album_name: track.album["#text"],
+        lastfm_data: track,
+      }));
 
     const { error, message } = await table.save(toInsert);
     if (error) {

supabase/functions/fetch-scrobbles/lastfm.types.ts

@@ -12,7 +12,8 @@ interface Artist {
 
 interface Track {
   artist: Artist;
-  date: {
+  // date is present only when the track is not playing right now...
+  date?: {
     uts: number;
     "#text": string;
   };
@@ -26,6 +27,10 @@ interface Track {
     "#text": string;
   };
   loved: string;
+  // present only when it is playing, weird I know
+  "@attr"?: {
+    nowplaying: boolean;
+  };
 }
 
 interface Attributes {