Add decimal and payment_card strategies; per-strategy option docs#62
Merged
Conversation
- New anonymization strategies: decimal (min/max/scale/as_string) and payment_card (length 13-19, Luhn-valid digits, quoted). - Add scale field to AnonymizerSpec; export luhn_valid for reuse. - README: replace flat strategy table with per-strategy option docs and cross-cutting domain/as_string section; update examples. - Serialize cwd-based settings tests to avoid parallel flakiness. - AGENTS.md: point new-strategy checklist at README subsections. - docs/configuration.md and .dumplingconf.example: reflect new strategies. Co-authored-by: Andy Babic <ababic@users.noreply.github.com>
Co-authored-by: Andy Babic <ababic@users.noreply.github.com>
- blank: quoted empty string; NULL source stays NULL
- empty_array / empty_object: unquoted [] / {}; NULL source stays NULL
- Reject domain like other constant-output strategies
- JSON path tests verify [] / {} coerce to proper JSON types
- README: document strategies and mention in cheap-vs-realistic guidance
Co-authored-by: Andy Babic <ababic@users.noreply.github.com>
@v2 tracks the moving v2 branch; v2.9.0+ runs on Node 24 with recent dependency updates, which avoids older action bundles that can emit Node punycode (DEP0040) deprecation noise in CI logs. Co-authored-by: Andy Babic <ababic@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR bundles strategy/docs work and CI hardening.
Strategies & docs (earlier commits)
decimal,payment_card,blank,empty_array,empty_objectstrategies, README per-strategy docs, tests, etc.CI: rust-cache pin (latest)
Swatinem/rust-cache@v2follows the movingv2ref. The action’s changelog notes v2.9.0 moved the runtime to Node 24 and refreshed dependencies; v2.9.1 fixes a hash regression. Pinning toSwatinem/rust-cache@v2.9.1in all workflows reduces the chance of pulling an older action bundle that triggers Node’s built-inpunycodeDEP0040 deprecation noise (or worse if something treats stderr as fatal).If the warning persists after this pin, it is almost certainly from another step or from a transitive dependency still using
punycodeon Node 22+; the warning is usually harmless unlessNODE_OPTIONS=--throw-deprecationis set.Testing
Rust:
cargo fmt,cargo clippy --all-targets --all-features,cargo test --all-targets --all-features.Slack Thread