feat: add artist radio

dont create reports on http status errors

src/client/channel_rss.rs

@@ -25,9 +25,9 @@ impl RustyPipeQuery {
             .http_request_txt(self.client.inner.http.get(&url).build()?)
             .await
             .map_err(|e| match e {
-                Error::HttpStatus(404) => Error::Extraction(ExtractionError::ContentUnavailable(
-                    "Channel not found".into(),
-                )),
+                Error::HttpStatus(404, _) => Error::Extraction(
+                    ExtractionError::ContentUnavailable("Channel not found".into()),
+                ),
                 _ => e,
             })?;
 

src/client/mod.rs

@@ -538,7 +538,7 @@ impl RustyPipe {
         let status = res.status();
 
         if status.is_client_error() || status.is_server_error() {
-            Err(Error::HttpStatus(status.into()))
+            Err(Error::HttpStatus(status.into(), "none".into()))
         } else {
             Ok(res)
         }
@@ -1086,26 +1086,18 @@ impl RustyPipeQuery {
         };
 
         if status.is_client_error() || status.is_server_error() {
-            let status_code = status.as_u16();
+            let error_msg = serde_json::from_str::<response::ErrorResponse>(&resp_str)
+                .map(|r| r.error.message)
+                .unwrap_or_default();
 
             return match status {
                 StatusCode::NOT_FOUND => Err(Error::Extraction(
-                    ExtractionError::ContentUnavailable("404 Not found".into()),
+                    ExtractionError::ContentUnavailable(error_msg.into()),
                 )),
-                StatusCode::BAD_REQUEST => {
-                    let error_res = serde_json::from_str::<response::ErrorResponse>(&resp_str);
-                    Err(Error::Extraction(ExtractionError::BadRequest(
-                        error_res
-                            .map(|r| r.error.message)
-                            .unwrap_or_default()
-                            .into(),
-                    )))
-                }
-                _ => {
-                    let e = Error::HttpStatus(status_code);
-                    create_report(Level::ERR, Some(e.to_string()), vec![]);
-                    Err(e)
-                }
+                StatusCode::BAD_REQUEST => Err(Error::Extraction(ExtractionError::BadRequest(
+                    error_msg.into(),
+                ))),
+                _ => Err(Error::HttpStatus(status.as_u16(), error_msg.into())),
             };
         }
 

src/client/music_artist.rs

@@ -166,7 +166,7 @@ fn map_artist_page(
     lang: crate::param::Language,
     skip_extendables: bool,
 ) -> Result<MapResult<(MusicArtist, Vec<String>)>, ExtractionError> {
-    // dbg!(&self);
+    // dbg!(&res);
 
     let header = res.header.music_immersive_header_renderer;
 
@@ -272,6 +272,13 @@ fn map_artist_page(
             .map(|m| m.as_str().to_owned())
     });
 
+    let radio_id = header.start_radio_button.and_then(|b| {
+        b.button_renderer
+            .navigation_endpoint
+            .watch_endpoint
+            .and_then(|w| w.playlist_id)
+    });
+
     Ok(MapResult {
         c: (
             MusicArtist {
@@ -292,6 +299,7 @@ fn map_artist_page(
                 similar_artists: mapped.c.artists,
                 tracks_playlist_id,
                 videos_playlist_id,
+                radio_id,
             },
             album_page_params,
         ),

src/client/response/music_artist.rs

@@ -5,7 +5,7 @@ use crate::serializer::text::Text;
 
 use super::{
     music_item::{
-        Grid, ItemSection, MusicThumbnailRenderer, SimpleHeader, SingleColumnBrowseResult,
+        Button, Grid, ItemSection, MusicThumbnailRenderer, SimpleHeader, SingleColumnBrowseResult,
     },
     SectionList, Tab,
 };
@@ -41,6 +41,9 @@ pub(crate) struct MusicHeaderRenderer {
     #[serde(default)]
     #[serde_as(as = "DefaultOnError")]
     pub share_endpoint: Option<ShareEndpoint>,
+    #[serde(default)]
+    #[serde_as(as = "DefaultOnError")]
+    pub start_radio_button: Option<Button>,
 }
 
 #[derive(Debug, Deserialize)]

src/client/response/music_item.rs

@@ -320,14 +320,14 @@ pub(crate) struct MusicCarouselShelfHeader {
 #[derive(Debug, Deserialize)]
 #[serde(rename_all = "camelCase")]
 pub(crate) struct MusicCarouselShelfHeaderRenderer {
-    pub more_content_button: Option<MoreContentButton>,
+    pub more_content_button: Option<Button>,
     #[serde(default)]
     pub title: TextComponents,
 }
 
 #[derive(Debug, Deserialize)]
 #[serde(rename_all = "camelCase")]
-pub(crate) struct MoreContentButton {
+pub(crate) struct Button {
     pub button_renderer: ButtonRenderer,
 }
 

src/client/snapshots/rustypipe__client__music_artist__tests__map_music_artist_default.snap

@@ -3649,4 +3649,5 @@ MusicArtist(
   ],
   tracks_playlist_id: Some("OLAK5uy_mHzt_lZ0Vdnv3NAqvCLQw5CiYe0q96iBs"),
   videos_playlist_id: Some("OLAK5uy_k9FAxC8Xb8T0KqP5wFbn8KsP3VkUjb334"),
+  radio_id: Some("RDEM_Ktu-TilkxtLvmc9wX1MLQ"),
 )

src/client/snapshots/rustypipe__client__music_artist__tests__map_music_artist_no_artist.snap

@@ -152,4 +152,5 @@ MusicArtist(
   similar_artists: [],
   tracks_playlist_id: None,
   videos_playlist_id: None,
+  radio_id: None,
 )

src/client/snapshots/rustypipe__client__music_artist__tests__map_music_artist_no_cont.snap

@@ -1387,4 +1387,5 @@ MusicArtist(
   ],
   tracks_playlist_id: Some("OLAK5uy_mHzt_lZ0Vdnv3NAqvCLQw5CiYe0q96iBs"),
   videos_playlist_id: Some("OLAK5uy_k9FAxC8Xb8T0KqP5wFbn8KsP3VkUjb334"),
+  radio_id: Some("RDEM_Ktu-TilkxtLvmc9wX1MLQ"),
 )

src/client/snapshots/rustypipe__client__music_artist__tests__map_music_artist_no_more_albums.snap

@@ -966,4 +966,5 @@ MusicArtist(
   ],
   tracks_playlist_id: Some("OLAK5uy_m6843aeUO05cz_t1seql2dQ9eUgwyuOXI"),
   videos_playlist_id: None,
+  radio_id: Some("RDEMd8PZIv9CPsvvEDbo_pUD4w"),
 )

src/client/snapshots/rustypipe__client__music_artist__tests__map_music_artist_only_more_singles.snap

@@ -1885,4 +1885,5 @@ MusicArtist(
   ],
   tracks_playlist_id: Some("OLAK5uy_nGjQIyWU6PsJZdJst7WOJHVZ7N7DupP04"),
   videos_playlist_id: Some("OLAK5uy_mulXOv4f1FNKi1-0zgY3YouLDTXX2wLu4"),
+  radio_id: Some("RDEMzMxPuaGyofN40xcgHuZAbw"),
 )

src/client/snapshots/rustypipe__client__music_artist__tests__map_music_artist_only_singles.snap

@@ -626,4 +626,5 @@ MusicArtist(
   ],
   tracks_playlist_id: None,
   videos_playlist_id: Some("OLAK5uy_lmH3iVq6lqjsnLkBWzpvRTh0DidLzbU-I"),
+  radio_id: Some("RDEMYsk_DTFHAng1G7n5toi_oA"),
 )

src/error.rs

@@ -19,8 +19,8 @@ pub enum Error {
     #[error("http error: {0}")]
     Http(#[from] reqwest::Error),
     /// Erroneous HTTP status code received
-    #[error("http status code: {0}")]
-    HttpStatus(u16),
+    #[error("http status code: {0} message: {1}")]
+    HttpStatus(u16, Cow<'static, str>),
     /// Unspecified error
     #[error("error: {0}")]
     Other(Cow<'static, str>),

src/model/mod.rs

@@ -1111,6 +1111,8 @@ pub struct MusicArtist {
     pub tracks_playlist_id: Option<String>,
     /// ID of the playlist containging the artist's videos
     pub videos_playlist_id: Option<String>,
+    /// ID of the artist radio
+    pub radio_id: Option<String>,
 }
 
 /// YouTube Music search result

tests/snapshots/youtube__music_artist_basic.snap

@@ -316,4 +316,5 @@ MusicArtist(
   similar_artists: "[artists]",
   tracks_playlist_id: Some("OLAK5uy_n6aX-F_lCQxgyTIv4FJhp78bXV93b9NUM"),
   videos_playlist_id: Some("OLAK5uy_nrePwvOEzmO7SydszEFfCDu8gAJxKfFtw"),
+  radio_id: Some("RDEMdgjzN3Qrk_GD7BooQbkJ4A"),
 )

tests/snapshots/youtube__music_artist_basic_all.snap

@@ -661,4 +661,5 @@ MusicArtist(
   similar_artists: "[artists]",
   tracks_playlist_id: Some("OLAK5uy_n6aX-F_lCQxgyTIv4FJhp78bXV93b9NUM"),
   videos_playlist_id: Some("OLAK5uy_nrePwvOEzmO7SydszEFfCDu8gAJxKfFtw"),
+  radio_id: Some("RDEMdgjzN3Qrk_GD7BooQbkJ4A"),
 )

tests/snapshots/youtube__music_artist_no_artist.snap

@@ -15,4 +15,5 @@ MusicArtist(
   similar_artists: "[artists]",
   tracks_playlist_id: None,
   videos_playlist_id: None,
+  radio_id: None,
 )

tests/snapshots/youtube__music_artist_no_more_albums.snap

@@ -151,4 +151,5 @@ MusicArtist(
   similar_artists: "[artists]",
   tracks_playlist_id: Some("OLAK5uy_miHesZCUQY5S9EwqfoNP2tZR9nZ0NBAeU"),
   videos_playlist_id: Some("OLAK5uy_mqbgE6T9uvusUWrAxJGiImf4_P4dM7IvQ"),
+  radio_id: Some("RDEM7AbogW0cCnElSU0WYm1GqA"),
 )

tests/snapshots/youtube__music_artist_only_singles.snap

@@ -31,4 +31,5 @@ MusicArtist(
   similar_artists: "[artists]",
   tracks_playlist_id: None,
   videos_playlist_id: Some("OLAK5uy_lmH3iVq6lqjsnLkBWzpvRTh0DidLzbU-I"),
+  radio_id: Some("RDEMYsk_DTFHAng1G7n5toi_oA"),
 )

tests/youtube.rs

@@ -2031,7 +2031,34 @@ async fn music_radio_playlist_not_found() {
         .music_radio_playlist("PL5dDx681T4bR7ZF1IuWzOv1omlZZZZZZZ")
         .await;
 
-    // Currently this returns valid data
+    if let Err(err) = res {
+        assert!(
+            matches!(
+                err,
+                Error::Extraction(ExtractionError::ContentUnavailable(_))
+            ),
+            "got: {}",
+            err
+        );
+    }
+}
+
+#[tokio::test]
+async fn music_radio_artist() {
+    let rp = RustyPipe::builder().strict().build();
+    let tracks = rp
+        .query()
+        .music_radio("RDEM_Ktu-TilkxtLvmc9wX1MLQ")
+        .await
+        .unwrap();
+    assert_next_items(tracks, rp.query(), 20).await;
+}
+
+#[tokio::test]
+async fn music_radio_not_found() {
+    let rp = RustyPipe::builder().strict().build();
+    let res = rp.query().music_radio("RDEM_Ktu-TilkxtLvmc9wXZZZZ").await;
+
     if let Err(err) = res {
         assert!(
             matches!(