Fix ETL tool issues

[stdavis]

No description provided.

[Copilot]

Pull request overview

This PR tightens and clarifies the county-to-UTRANS ETL behavior by improving failure reporting, preserving invalid domain values in audit notes, and enforcing target field-length constraints to prevent invalid appends.

Changes:

• Add clearer error context when adding missing template fields fails, and validate target string lengths during normalization.
• Preserve invalid domain values by appending them to UTRANS_NOTES, and only copy them into destination fields when they fit.
• Standardize county boundary name matching to uppercase (with special cases) and log total CLI elapsed time.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
cli/src/utrans/etl_common.py	Adds detailed AddField failure context and optional target-length validation in normalization.
cli/src/utrans/etl_mappers.py	Preserves invalid domain values in notes and conditionally assigns them based on destination field length.
cli/src/utrans/etl.py	Updates normalization call signature, uppercases county boundary lookup values, and logs elapsed time.
cli/tests/test_etl_common.py	Adds tests for AddField error reporting, field-length enforcement, and _fits_field_length.
cli/tests/test_cli.py	Adds coverage for uppercase county boundary name normalization.
cli/PROFILES.md	Documents the updated invalid-domain preservation + field-length behavior.

Suppressed comments (1)

cli/src/utrans/etl.py:196

• If switching to perf_counter() for the start time, the elapsed calculation should also use perf_counter() to keep units and monotonic behavior consistent.

    elapsed = time.time() - start_time
    log(f"Time elapsed: {elapsed:.2f}s")

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.