Merge pull request #10 from WendellXY/codex/view-status-filter

Add status-based filtering to view

Author: WendellXY

CHANGELOG.md

@@ -4,6 +4,14 @@ All notable changes to this project will be documented in this file.
 
 The format is based on Keep a Changelog and this project follows Semantic Versioning while APIs remain in 0.x evolution.
 
+## [Unreleased]
+
+### Added
+- CLI `view` status-based filtering with `--status`, `--keys-only`, and `--json` output modes for listing untranslated/review-needed entries.
+
+### Changed
+- `view --status` now supports strict metadata gating: `langcodec --strict view ... --status ...` requires explicit status metadata (v1 support: `.xcstrings`).
+
 ## [0.7.0] - 2026-02-17
 
 ### Added

README.md

@@ -62,6 +62,8 @@ This is a `0.7.0` release available on [crates.io](https://crates.io/crates/lang
 - Normalize and detect drift: `langcodec normalize -i 'locales/**/*.{strings,xml,csv,tsv,xcstrings}' --check`
 - Sync existing keys only: `langcodec sync --source A.xcstrings --target B.xcstrings --match-lang en`
 - View: `langcodec view -i strings.xml --full`
+- View filtered untranslated/review-needed keys: `langcodec view -i Localizable.xcstrings --status new,needs_review --keys-only`
+- View filtered results as JSON: `langcodec view -i Localizable.xcstrings --status new --lang fr --json`
 - Stats (JSON): `langcodec stats -i Localizable.xcstrings --json`
   - See full options: langcodec-cli/README.md#stats
   - Example output:
@@ -99,6 +101,7 @@ This is a `0.7.0` release available on [crates.io](https://crates.io/crates/lang
 - Normalize `--continue-on-error` processes all inputs and returns non-zero if any file fails.
 - Android path inference: `values/strings.xml` (no qualifier) defaults to English (`en`).
 - When converting to `.xcstrings`, if `source_language` or `version` metadata is missing, the CLI defaults them to `en` and `1.0` respectively (overridable via flags).
+- Strict status filtering note: `langcodec --strict view --status ...` requires explicit status metadata (supported in v1: `.xcstrings`).
 
 #### Plurals
 

langcodec-cli/README.md

@@ -63,10 +63,21 @@ CI-oriented options:
 
 ```sh
 langcodec view -i values/strings.xml --full
+langcodec view -i Localizable.xcstrings --status new,needs_review
+langcodec view -i Localizable.xcstrings --status new --lang fr --json
+langcodec view -i Localizable.xcstrings --status new,needs_review --keys-only
 ```
 
 Prints entries. Plurals are labeled with `Type: Plural` and show categories.
 
+View options:
+
+- `--status`: Filter by one or more statuses (`translated|needs_review|new|do_not_translate|stale`), comma-separated.
+- `--keys-only`: Print only keys in text mode (`lang<TAB>key` when `--lang` is not set).
+- `--json`: Output machine-readable JSON (`summary` + `entries` or `keys` payload).
+- `--lang`: Restrict results to a specific language before status filtering.
+- `--strict`: With `--status`, requires explicit status metadata (supported in v1: `.xcstrings`).
+
 ### stats
 
 ```sh

langcodec-cli/src/main.rs

@@ -20,7 +20,7 @@ use crate::merge::{ConflictStrategy, run_merge_command};
 use crate::normalize::{NormalizeCliOptions, run_normalize_command};
 use crate::sync::{SyncOptions, run_sync_command};
 use crate::validation::{ValidationContext, validate_context, validate_language_code};
-use crate::view::print_view;
+use crate::view::{ViewOptions, print_view, validate_status_filter};
 use clap::{CommandFactory, Parser, Subcommand};
 use clap_complete::{Shell, generate};
 
@@ -167,6 +167,18 @@ enum Commands {
         #[arg(long)]
         full: bool,
 
+        /// Filter entries by status (e.g. translated, needs_review, stale, new, do_not_translate)
+        #[arg(long)]
+        status: Option<String>,
+
+        /// Print keys only
+        #[arg(long, default_value_t = false)]
+        keys_only: bool,
+
+        /// Output JSON instead of human-readable text
+        #[arg(long, default_value_t = false)]
+        json: bool,
+
         /// Validate plural completeness against CLDR category sets
         #[arg(long, default_value_t = false)]
         check_plurals: bool,
@@ -323,6 +335,13 @@ fn is_custom_input_extension(input: &str) -> bool {
         || input.ends_with(".langcodec")
 }
 
+fn input_supports_explicit_status_metadata(input: &str) -> bool {
+    std::path::Path::new(input)
+        .extension()
+        .and_then(|ext| ext.to_str())
+        .is_some_and(|ext| ext.eq_ignore_ascii_case("xcstrings"))
+}
+
 fn load_codec_for_readonly_command(
     input: &str,
     lang: &Option<String>,
@@ -515,6 +534,9 @@ fn main() {
             input,
             lang,
             full,
+            status,
+            keys_only,
+            json,
             check_plurals,
         } => {
             // Create validation context
@@ -530,6 +552,18 @@ fn main() {
                 std::process::exit(1);
             }
 
+            if let Err(e) = validate_status_filter(&status) {
+                eprintln!("❌ {}", e);
+                std::process::exit(1);
+            }
+
+            if strict && status.is_some() && !input_supports_explicit_status_metadata(&input) {
+                eprintln!(
+                    "❌ Strict mode with --status requires explicit status metadata. Supported in v1: .xcstrings"
+                );
+                std::process::exit(1);
+            }
+
             let codec = match load_codec_for_readonly_command(&input, &lang, strict) {
                 Ok(codec) => codec,
                 Err(e) => {
@@ -538,11 +572,24 @@ fn main() {
                 }
             };
 
-            print_view(&codec, &lang, full);
+            let view_options = ViewOptions {
+                full,
+                status,
+                keys_only,
+                json,
+            };
+
+            print_view(&codec, &lang, &view_options);
 
             if check_plurals {
                 match codec.validate_plurals() {
-                    Ok(()) => println!("\n✅ Plural validation passed"),
+                    Ok(()) => {
+                        if json || keys_only {
+                            eprintln!("✅ Plural validation passed");
+                        } else {
+                            println!("\n✅ Plural validation passed");
+                        }
+                    }
                     Err(e) => {
                         eprintln!("\n❌ Plural validation failed: {}", e);
                         std::process::exit(2);