Spotify improvements (#19420)

* Syntax improvements to spotify components

* Adjusting default page size to 50

* Fixing search method

* Removing remaining lodash calls

* Version bumps

* Adjusting response formats

* Fixing more response formats

* Fix

* fixing silly validation

* Adjusting retry delay

* Adjusting request for max 20 albums

* Fixing 'get recommendations'

Author: GTFalcao

components/factorial_api_keys/factorial_api_keys.app.mjs

@@ -8,4 +8,4 @@ export default {
       console.log(Object.keys(this.$auth));
     },
   },
-};
+};

components/flowii/flowii.app.mjs

@@ -8,4 +8,4 @@ export default {
       console.log(Object.keys(this.$auth));
     },
   },
-};
+};

components/hailey_hr/hailey_hr.app.mjs

@@ -8,4 +8,4 @@ export default {
       console.log(Object.keys(this.$auth));
     },
   },
-};
+};

components/nuvemshop/nuvemshop.app.mjs

@@ -8,4 +8,4 @@ export default {
       console.log(Object.keys(this.$auth));
     },
   },
-};
+};

components/simpleem/simpleem.app.mjs

@@ -8,4 +8,4 @@ export default {
       console.log(Object.keys(this.$auth));
     },
   },
-};
+};

components/spotify/actions/add-items-to-playlist/add-items-to-playlist.mjs

@@ -1,12 +1,10 @@
-import { axios } from "@pipedream/platform";
-import get from "lodash/get.js";
 import spotify from "../../spotify.app.mjs";
 
 export default {
   name: "Add Items to a Playlist",
   description: "Add one or more items to a user’s playlist. [See the docs here](https://developer.spotify.com/documentation/web-api/reference/#/operations/add-tracks-to-playlist).",
   key: "spotify-add-items-to-playlist",
-  version: "0.1.4",
+  version: "0.1.5",
   annotations: {
     destructiveHint: false,
     openWorldHint: true,
@@ -46,14 +44,15 @@ export default {
       uris: this.spotify.sanitizedArray(uris),
     };
 
-    const resp = await axios($, this.spotify._getAxiosParams({
+    const { data: resp } = await this.spotify._makeRequest({
+      $,
       method: "POST",
-      path: `/playlists/${get(playlistId, "value", playlistId)}/tracks`,
+      url: `/playlists/${playlistId.value ?? playlistId}/tracks`,
       data,
-    }));
+    });
 
     // eslint-disable-next-line multiline-ternary
-    $.export("$summary", `Successfully added ${data.uris.length} ${data.uris.length == 1 ? "item" : "items"} to "${get(playlistId, "label", playlistId)}"`);
+    $.export("$summary", `Successfully added ${data.uris.length} ${data.uris.length == 1 ? "item" : "items"} to "${playlistId.label ?? playlistId}"`);
 
     return resp;
   },

components/spotify/actions/create-playlist/create-playlist.mjs

@@ -1,11 +1,10 @@
-import { axios } from "@pipedream/platform";
 import spotify from "../../spotify.app.mjs";
 
 export default {
   name: "Create a Playlist",
   description: "Create a playlist for a Spotify user. The playlist will be empty until you add tracks. [See the docs here](https://developer.spotify.com/documentation/web-api/reference/#/operations/create-playlist).",
   key: "spotify-create-playlist",
-  version: "0.1.4",
+  version: "0.1.5",
   annotations: {
     destructiveHint: false,
     openWorldHint: true,
@@ -53,11 +52,12 @@ export default {
       collaborative: isCollaborative,
     };
 
-    const resp = await axios($, this.spotify._getAxiosParams({
+    const { data: resp } = await this.spotify._makeRequest({
+      $,
       method: "POST",
-      path: `/users/${this.spotify.$auth.oauth_uid}/playlists`,
+      url: `/users/${this.spotify.$auth.oauth_uid}/playlists`,
       data,
-    }));
+    });
 
     $.export("$summary", `Successfully created a new playlist, "${data.name}"`);
 

components/spotify/actions/get-album-tracks/get-album-tracks.mjs

@@ -1,13 +1,14 @@
-import { axios } from "@pipedream/platform";
 import spotify from "../../spotify.app.mjs";
-import { ITEM_TYPES } from "../../common/constants.mjs";
-const DEFAULT_LIMIT = 20;
+import {
+  ITEM_TYPES,
+  PAGE_SIZE,
+} from "../../common/constants.mjs";
 
 export default {
   name: "Get Album Tracks",
   description: "Get all tracks in an album. [See the docs here](https://developer.spotify.com/documentation/web-api/reference/get-an-albums-tracks)",
   key: "spotify-get-album-tracks",
-  version: "0.0.5",
+  version: "0.0.6",
   annotations: {
     destructiveHint: false,
     openWorldHint: true,
@@ -25,12 +26,11 @@ export default {
         query,
         page,
       }) {
-        const limit = DEFAULT_LIMIT;
         const albums = await this.spotify.getItems(
           ITEM_TYPES.ALBUM,
           query,
-          limit,
-          limit * page,
+          PAGE_SIZE,
+          PAGE_SIZE * page,
         );
         return {
           options: albums.map((album) => ({
@@ -43,19 +43,20 @@ export default {
   },
   async run({ $ }) {
     const params = {
-      limit: DEFAULT_LIMIT,
+      limit: PAGE_SIZE,
       offset: 0,
     };
     const tracks = [];
     let total = 0;
 
     do {
-      const { items } = await axios($, this.spotify._getAxiosParams({
-        path: `/albums/${this.albumId}/tracks`,
+      const { data } = await this.spotify._makeRequest({
+        $,
+        url: `/albums/${this.albumId}/tracks`,
         params,
-      }));
-      tracks.push(...items);
-      total = items.length;
+      });
+      tracks.push(...data.items);
+      total = data.items.length;
       params.offset += params.limit;
     } while (total === params.limit);
 

components/spotify/actions/get-all-tracks-by-artist/get-all-tracks-by-artist.mjs

@@ -1,11 +1,10 @@
-import get from "lodash/get.js";
 import spotify from "../../spotify.app.mjs";
 
 export default {
   name: "Get All Tracks by Artist",
   description: "Get Spotify tracks information related with an artist's. [see docs here](https://developer.spotify.com/documentation/web-api/reference/#/operations/get-multiple-albums).",
   key: "spotify-get-all-tracks-by-artist",
-  version: "0.1.4",
+  version: "0.1.5",
   annotations: {
     destructiveHint: false,
     openWorldHint: true,
@@ -34,16 +33,18 @@ export default {
     } = this;
 
     const chunksOfAlbumIds = await this.spotify.fetchChunksOfAlbumsIds({
+      $,
       artistId,
       market,
     });
 
     const tracks = await this.spotify.getAllTracksByChunksOfAlbumIds({
+      $,
       chunksOfAlbumIds,
       market,
     });
 
-    $.export("$summary", `Successfully fetched ${tracks.length} tracks for "${get(artistId, "label", artistId)}"`);
+    $.export("$summary", `Successfully fetched ${tracks.length} tracks for "${artistId.label ?? artistId}"`);
 
     return tracks;
   },

components/spotify/actions/get-artist-top-tracks/get-artist-top-tracks.mjs

@@ -1,12 +1,10 @@
-import { axios } from "@pipedream/platform";
-import get from "lodash/get.js";
 import spotify from "../../spotify.app.mjs";
 
 export default {
   name: "Get an Artist's Top Tracks",
-  description: "Get Spotify catalog information about an artist’s top tracks by country. [See the docs here](https://developer.spotify.com/documentation/web-api/reference/#/operations/get-an-artists-top-tracks).",
+  description: "Get Spotify catalog information about an artist's top tracks by country. [See the docs here](https://developer.spotify.com/documentation/web-api/reference/#/operations/get-an-artists-top-tracks).",
   key: "spotify-get-artist-top-tracks",
-  version: "0.1.4",
+  version: "0.1.5",
   annotations: {
     destructiveHint: false,
     openWorldHint: true,
@@ -34,16 +32,16 @@ export default {
       market,
     } = this;
 
-    const res = await axios($, this.spotify._getAxiosParams({
-      method: "GET",
-      path: `/artists/${get(artistId, "value", artistId)}/top-tracks`,
+    const { data } = await this.spotify._makeRequest({
+      $,
+      url: `/artists/${artistId.value ?? artistId}/top-tracks`,
       params: {
         market,
       },
-    }));
+    });
 
-    $.export("$summary", `Successfully fetched top tracks for "${get(artistId, "label", artistId)}"`);
+    $.export("$summary", `Successfully fetched top tracks for "${artistId.label ?? artistId}"`);
 
-    return get(res, "tracks", []);
+    return data?.tracks ?? [];
   },
 };