Merge pull request #9 from WendellXY/codex/normalize-subcommand

Normalize Subcommand

Author: WendellXY

.gitignore

@@ -2,4 +2,7 @@ target
 Cargo.lock
 
 # Editor Configurations
-.vscode
+.vscode
+
+# Local planning artifacts
+docs/plans/*

CHANGELOG.md

@@ -10,10 +10,12 @@ The format is based on Keep a Changelog and this project follows Semantic Versio
 - Global `--strict` mode in CLI workflows and strict-read controls in library input options.
 - CLI `diff` command with human-readable and JSON report output.
 - Plural validation enhancements and richer stats reporting.
+- CLI `normalize` command support across Apple `.strings`, `.xcstrings`, Android `strings.xml`, CSV, and TSV formats, including drift detection with `--check`.
 
 ### Changed
 - Ongoing improvements to conversion coverage across Apple, Android, CSV, and TSV formats.
 - Documentation and contributor guidance updates.
+- Normalize workflow documentation, including `--dry-run`, `--no-placeholders`, `--key-style`, multi-input `--output` constraints, and `--continue-on-error` behavior.
 
 ### Fixed
 - Multiple parser/writer correctness and conversion edge-case fixes covered by expanded tests.

README.md

@@ -3,7 +3,7 @@
 Universal localization toolkit: library + CLI for Apple/Android/CSV/TSV.
 
 - Library crate (`langcodec`): parse, write, convert, merge with a unified model
-- CLI crate (`langcodec-cli`): convert, diff, merge, sync, view, stats, debug, edit
+- CLI crate (`langcodec-cli`): convert, diff, merge, sync, view, stats, debug, edit, normalize
 
 ---
 
@@ -59,6 +59,7 @@ This is a `0.7.0` release available on [crates.io](https://crates.io/crates/lang
 - Convert: `langcodec convert -i input.strings -o strings.xml`
 - Diff: `langcodec diff --source A.xcstrings --target B.xcstrings --json`
 - Edit (add/update/remove): `langcodec edit set -i 'locales/**/*.strings' -k welcome -v "Hello"` (use `--dry-run` to preview)
+- 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`
 - Stats (JSON): `langcodec stats -i Localizable.xcstrings --json`
@@ -92,6 +93,10 @@ This is a `0.7.0` release available on [crates.io](https://crates.io/crates/lang
 - The convert command also supports custom JSON/YAML input formats.
 - The CLI will error if you try to merge files of different formats.
 - Edit supports multiple inputs and glob patterns. When multiple inputs are provided, edits are applied in-place and `--output` is not allowed.
+- Normalize supports `.strings`, Android `strings.xml`, `.csv`, `.tsv`, and `.xcstrings`.
+- Normalize options: `--check` (fail on drift), `--dry-run` (preview only), `--no-placeholders` (skip placeholder canonicalization), `--key-style` (`none|snake|kebab|camel`).
+- Normalize multi-input constraint: `--output` is single-input only; pair multi-input workflows with in-place writes.
+- 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).
 

langcodec-cli/README.md

@@ -3,7 +3,7 @@
 Universal CLI for converting, inspecting, merging, and editing localization files.
 
 - Formats: Apple `.strings`, `.xcstrings`, Android `strings.xml`, CSV, TSV
-- Commands: convert, diff, merge, sync, view, stats, debug, edit
+- Commands: convert, diff, merge, sync, view, stats, debug, edit, normalize
 
 ## Install
 
@@ -121,6 +121,39 @@ Options:
 
 Supported formats: .strings, .xml (Android), .xcstrings, .csv, .tsv. Custom JSON/YAML/.langcodec edit is currently not enabled.
 
+### normalize
+
+Normalize localization files in-place (or to `--output` in single-input mode).
+
+```sh
+# Normalize in-place
+langcodec normalize -i en.lproj/Localizable.strings
+
+# CI drift check (non-zero if any file would change)
+langcodec normalize -i 'locales/**/*.{strings,xml,csv,tsv,xcstrings}' --check
+
+# Preview without writing
+langcodec normalize -i values/strings.xml --dry-run
+
+# Disable placeholder normalization and rename keys to snake_case
+langcodec normalize -i Localizable.xcstrings --no-placeholders --key-style snake
+
+# Keep processing remaining files and summarize failures at the end
+langcodec normalize -i a.strings -i b.csv -i c.tsv --continue-on-error
+```
+
+Options and behavior:
+
+- --check: Detects normalization drift and exits non-zero when a file would change.
+- --dry-run: Prints what would change and exits without writing files.
+- --no-placeholders: Skips placeholder canonicalization (for example `%@` → `%s`).
+- --key-style: Renames keys during normalization. Values: `none` (default), `snake`, `kebab`, `camel`.
+- --output/-o: Single-input mode only. If multiple inputs are provided, `--output` is rejected.
+- --continue-on-error: Continues processing all matched inputs, prints a summary, and exits non-zero if any file failed.
+- --inputs/-i: One or more files, including quoted glob patterns.
+
+Supported normalize formats: `.strings`, Android `strings.xml`, `.csv`, `.tsv`, `.xcstrings`.
+
 ## Notes
 
 - Android plurals `<plurals>` are supported.

langcodec-cli/src/main.rs

@@ -4,6 +4,7 @@ mod diff;
 mod edit;
 mod formats;
 mod merge;
+mod normalize;
 mod path_glob;
 mod stats;
 mod sync;
@@ -16,6 +17,7 @@ use crate::debug::run_debug_command;
 use crate::diff::{DiffOptions, run_diff_command};
 use crate::edit::{EditSetOptions, run_edit_set_command};
 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;
@@ -196,6 +198,37 @@ enum Commands {
         version: Option<String>,
     },
 
+    /// Normalize localization files.
+    Normalize {
+        /// The input files to normalize (supports glob patterns). Quote patterns to avoid shell expansion.
+        #[arg(short, long, required = true, num_args = 1.., help = "Input files. Supports glob patterns. Quote patterns to avoid slow shell-side expansion (e.g., '/path/**/*/Localizable.strings').")]
+        inputs: Vec<String>,
+
+        /// Optional output file (single-file mode only). If omitted, writes in-place.
+        #[arg(short, long)]
+        output: Option<String>,
+
+        /// Preview changes without writing
+        #[arg(long, default_value_t = false)]
+        dry_run: bool,
+
+        /// Exit non-zero if normalization would change the file
+        #[arg(long, default_value_t = false)]
+        check: bool,
+
+        /// Disable placeholder normalization
+        #[arg(long, default_value_t = false)]
+        no_placeholders: bool,
+
+        /// Key renaming style: none|snake|kebab|camel
+        #[arg(long, default_value = "none")]
+        key_style: String,
+
+        /// Continue processing remaining files when a file fails
+        #[arg(long, default_value_t = false)]
+        continue_on_error: bool,
+    },
+
     /// Show translation coverage and per-status counts.
     Stats {
         /// The input file to analyze
@@ -595,6 +628,30 @@ fn main() {
             cmd = cmd.bin_name("langcodec");
             generate(shell, &mut cmd, "langcodec", &mut std::io::stdout());
         }
+        Commands::Normalize {
+            inputs,
+            output,
+            dry_run,
+            check,
+            no_placeholders,
+            key_style,
+            continue_on_error,
+        } => {
+            let opts = NormalizeCliOptions {
+                inputs,
+                output,
+                dry_run,
+                check,
+                no_placeholders,
+                key_style,
+                continue_on_error,
+                strict,
+            };
+            if let Err(e) = run_normalize_command(opts) {
+                eprintln!("❌ Normalize failed: {}", e);
+                std::process::exit(1);
+            }
+        }
         Commands::Stats { input, lang, json } => {
             // Validate
             let mut context = ValidationContext::new().with_input_file(input.clone());