fix: trim repeat-dict metadata before validation

[njzjz-bot]

Summary

• reproduce the behavior described in trim_pattern="_*" not supported for repeat dict Argument #70 on the current master branch
• trim keys matching trim_pattern at repeat-dict containers before iterating item values
• skip non-string Python mapping keys during pattern matching
• preserve existing item-level trimming and add regression tests for scalar metadata and integer item keys

Fixes #70

Validation

• PYTHONPATH=. python -m pytest -q — 60 passed, 4 skipped
• PYTHONPATH=. python -m unittest discover -q — 63 tests passed, 4 skipped
• ruff check dargs/dargs.py tests/test_normalizer.py — passed
• python -m compileall -q dargs tests — passed

Coding agent: Codex
Codex version: codex-cli 0.151.0
Model: gpt-5.6-sol
Reasoning effort: xhigh

[coderabbitai]

Important

• 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 642953d0-7af9-4701-af52-e0d338960bee

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.89%. Comparing base (c2c0a83) to head (037aa3c).

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #166      +/-   ##
==========================================
+ Coverage   78.84%   78.89%   +0.04%     
==========================================
  Files           9        9              
  Lines         884      886       +2     
==========================================
+ Hits          697      699       +2     
  Misses        187      187              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[njzjz-bot]

Review summary

Three independent subagent reviews were performed for this PR:

• Logic review: the Issue trim_pattern="_*" not supported for repeat dict Argument #70 fix correctly trims repeat-dict container metadata before item traversal; no blocking correctness findings.
• Compatibility review: identified a P2 edge case where non-string Python mapping keys (for example, integer keys) were passed to the trim regex. This was fixed in commit f973619 by matching only string keys.
• Test review: confirmed the main regression and recommended broader edge coverage; regression tests now cover scalar container metadata and non-string repeat-dict keys.

Validation is green:

• PYTHONPATH=. python -m pytest -q: 60 passed, 4 skipped
• PYTHONPATH=. python -m unittest discover -q: 63 passed, 4 skipped
• Ruff and compileall passed
• Build, type check, pre-commit, Codecov, and Read the Docs checks passed

Coding agent: Codex
Codex version: codex-cli 0.151.0
Model: gpt-5.6-sol
Reasoning effort: xhigh