From ef32a5a6fe191cb7bce3037dc25171947db7e058 Mon Sep 17 00:00:00 2001 From: Bailey Nguyen Date: Tue, 22 Sep 2026 22:45:23 -0500 Subject: [PATCH] docs: record direct removal of legacy CLI flags --- .agents/skills/polyxml-codegen-workflow/SKILL.md | 6 +++--- docs/guides/compiler.md | 4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/.agents/skills/polyxml-codegen-workflow/SKILL.md b/.agents/skills/polyxml-codegen-workflow/SKILL.md index 8572db07..6f98d48d 100644 --- a/.agents/skills/polyxml-codegen-workflow/SKILL.md +++ b/.agents/skills/polyxml-codegen-workflow/SKILL.md @@ -277,11 +277,11 @@ derived types inherit their base's patterns at parse time): parsers can still return `None`; the CLI must reject invalid values before emitter defaults are applied. - New opt-ins belong in repeatable `--feature` and manifest `features` arrays. - The legacy hidden flags (`--zod`, `--source-gen`, `--record-kind`, `--rkyv`, + The former flags (`--zod`, `--source-gen`, `--record-kind`, `--rkyv`, `--builder`, `--codec`) and their manifest keys were deleted outright: clap rejects the flags as unexpected arguments, `#[serde(deny_unknown_fields)]` - rejects the keys, and the deprecation-warning loop no longer exists. Keep - CLI and manifest parity. + rejects the keys. No compatibility aliases or deprecation warnings are + planned (decision in #58). Keep CLI and manifest parity. - `--zero-copy` (and manifest `zero_copy`) stays a first-class bool because `--feature` cannot express `false`; it alone selects owned Rust output, and `--zero-copy=false --feature zero-copy` is still rejected as a contradiction. diff --git a/docs/guides/compiler.md b/docs/guides/compiler.md index 4ea4b6a8..861f480e 100644 --- a/docs/guides/compiler.md +++ b/docs/guides/compiler.md @@ -120,7 +120,7 @@ style = "pojo" features = ["builder", "direct-codec"] ``` -The legacy hidden flags `--zod`, `--source-gen`, `--record-kind`, `--rkyv`, +The former flags `--zod`, `--source-gen`, `--record-kind`, `--rkyv`, `--builder`, and `--codec`, along with their `polyxml.toml` counterparts, have been removed: clap rejects the flags outright and unknown manifest fields are errors, so the unified options above are the only spellings. For owned Rust @@ -301,7 +301,7 @@ polyxml completions fish > ~/.config/fish/completions/polyxml.fish Backend, style, and feature suggestions use the CLI's validation rules and the selected `--lang` (including aliases). With multiple languages, completion only suggests values accepted by every selected target. Without `--lang`, it uses the -Python default. Hidden deprecated flags are not suggested. Bash also completes +Python default. Removed flags are not suggested. Bash also completes comma-separated features; repeat `--feature` for portable completion across shells. ### CLI startup benchmark