39 per language sync (#53)

* Add language filter option for subtitle actions

* Add missing ID to build-and-push step in Docker workflow

Author: mateoradman

.github/workflows/docker.yml

@@ -51,6 +51,7 @@ jobs:
             type=raw,value=latest,enable={{is_default_branch}}
 
       - name: Build and push Docker image
+        id: build-and-push
         uses: docker/build-push-action@v5
         with:
           context: .

README.md

@@ -200,6 +200,7 @@ Options:
       --offset <OFFSET>  Skip N records (ignored if ids are specified) [default: skip none] [default: 0]
       --limit <LIMIT>    Limit to N records (ignored if ids are specified) [default: unlimited]
       --skip-processed   Skip previously processed items (uses local database to track processed subtitles)
+  -l, --language <LANGUAGE>  Filter subtitles by language code (e.g., en, es, fr)
   -h, --help             Print help
 ```
 
@@ -226,6 +227,7 @@ Options:
       --offset <OFFSET>  Skip N records (ignored if ids are specified) [default: skip none] [default: 0]
       --limit <LIMIT>    Limit to N records (ignored if ids are specified) [default: unlimited]
       --skip-processed   Skip previously processed items (uses local database to track processed subtitles)
+  -l, --language <LANGUAGE>  Filter subtitles by language code (e.g., en, es, fr)
   -h, --help             Print help
 ```
 
@@ -244,3 +246,30 @@ Options:
   -g                   Use Golden-Section search [default: false]
   -h, --help           Print help
 ```
+
+## Usage Examples
+
+### Sync all English subtitles for movies
+
+```bash
+bb --config config.json movies --language en sync
+```
+
+### Perform OCR fixes only on Spanish subtitles for TV shows
+
+```bash
+bb --config config.json tv-shows --language es ocr-fixes
+```
+
+### Sync specific movies by ID with language filter
+
+```bash
+bb --config config.json movies --ids 123,456,789 --language en sync
+```
+
+### Remove hearing impaired tags from French subtitles with skip-processed
+
+```bash
+bb --config config.json movies --language fr --skip-processed remove-hearing-impaired
+```
+

src/actions.rs

@@ -27,6 +27,7 @@ pub struct Action {
     pub offset: u32,
     pub limit: Option<u32>,
     pub skip_processed: bool,
+    pub language_code: Option<String>,
     pub pb: ProgressBar,
     pub db_conn: Arc<Mutex<Connection>>,
     pub is_tty: bool,
@@ -52,6 +53,7 @@ impl Action {
             ids: Vec::new(),
             offset: 0,
             skip_processed: false,
+            language_code: None,
             limit: None,
             pb,
             db_conn,
@@ -89,6 +91,15 @@ impl Action {
         }
     }
 
+    /// Check if subtitle matches the language filter (if specified)
+    fn matches_language_filter(&self, subtitle_code: Option<&String>) -> bool {
+        match (&self.language_code, subtitle_code) {
+            (Some(lang_code), Some(subtitle_code)) => lang_code == subtitle_code,
+            (Some(_), None) => false,
+            (None, _) => true,
+        }
+    }
+
     async fn get_all<T>(&self, url: Url) -> Result<PaginatedResponse<T>, Box<dyn std::error::Error>>
     where
         T: DeserializeOwned,
@@ -147,6 +158,10 @@ impl Action {
                 continue;
             }
 
+            if !self.matches_language_filter(subtitle.audio_language_item.code2.as_ref()) {
+                continue;
+            }
+
             let msg = format!(
                 "Performing action {} on {} subtitle of episode {}",
                 self.action.to_string(),
@@ -200,6 +215,10 @@ impl Action {
                 continue;
             }
 
+            if !self.matches_language_filter(subtitle.audio_language_item.code2.as_ref()) {
+                continue;
+            }
+
             let msg = format!(
                 "Performing action {} on {} subtitle of movie {}",
                 self.action.to_string(),

src/cli.rs

@@ -59,6 +59,9 @@ pub struct CommonArgs {
     /// Must have all subtitles processed to be skipped.
     #[arg(long, default_value_t = false, required = false)]
     skip_processed: bool,
+    /// Filter subtitles by language code (e.g., en, es, fr)
+    #[arg(long, short = 'l')]
+    language: Option<String>,
     /// List available actions
     #[command(subcommand)]
     subcommand: ActionCommands,
@@ -105,6 +108,7 @@ impl Commands {
                 action.limit = c.limit;
                 action.offset = c.offset;
                 action.skip_processed = c.skip_processed;
+                action.language_code = c.language;
                 action.movies().await
             }
             Commands::TVShows(c) => {
@@ -113,6 +117,7 @@ impl Commands {
                 action.limit = c.limit;
                 action.offset = c.offset;
                 action.skip_processed = c.skip_processed;
+                action.language_code = c.language;
                 action.tv_shows().await
             }
         }