Skip to content

Enhances send elements to supported status#93

Merged
johnnyt merged 2 commits intomainfrom
send-action-updates
Sep 2, 2025
Merged

Enhances send elements to supported status#93
johnnyt merged 2 commits intomainfrom
send-action-updates

Conversation

@johnnyt
Copy link
Member

@johnnyt johnnyt commented Sep 2, 2025

Implements content data processing improvements:

  • Adds better error handling with LogManager warnings
  • Adds JSON serialization for complex map values using Jason
  • Preserves :undefined atoms for invalid expressions
  • Adds String.trim() for direct content text

Implements parameter processing improvements:

  • Adds enhanced parameter name validation with regex checks
  • Adds better handling of complex parameter values with normalization
  • Adds comprehensive error reporting with structured metadata
  • Adds support for preserving :undefined values from failed evaluations

Updates feature detection:

  • Moves send_content_elements from :partial to :supported
  • Moves send_param_elements from :partial to :supported

🤖 Generated with Claude Code

Implements content data processing improvements:
- Adds better error handling with LogManager warnings
- Adds JSON serialization for complex map values using Jason
- Preserves :undefined atoms for invalid expressions
- Adds String.trim() for direct content text

Implements parameter processing improvements:
- Adds enhanced parameter name validation with regex checks
- Adds better handling of complex parameter values with normalization
- Adds comprehensive error reporting with structured metadata
- Adds support for preserving :undefined values from failed evaluations

Updates feature detection:
- Moves send_content_elements from :partial to :supported
- Moves send_param_elements from :partial to :supported

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@codecov
Copy link

codecov bot commented Sep 2, 2025

Codecov Report

❌ Patch coverage is 68.75000% with 15 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
lib/statifier/actions/send_action.ex 68.75% 15 Missing ⚠️
Files with missing lines Coverage Δ
lib/statifier/feature_detector.ex 94.02% <ø> (ø)
lib/statifier/actions/send_action.ex 77.52% <68.75%> (-12.89%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Improves code readability and performance by converting cond statements to pattern matching:

- build_event_data: Converts cond to pattern matching on content/params/namelist presence
- build_content_data: Converts cond to pattern matching on expr/content fields
- validate_param_name: Converts cond to pattern matching with guards for parameter validation
- get_param_value: Converts cond to pattern matching on expr/location fields

Additional improvements:
- Extracts process_single_param helper to reduce function nesting depth (fixes Credo issue)
- Adds underscore prefixes to unused variables to eliminate compiler warnings
- Maintains all existing functionality and test compatibility

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@johnnyt johnnyt merged commit 2e76e06 into main Sep 2, 2025
8 checks passed
@johnnyt johnnyt deleted the send-action-updates branch September 2, 2025 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant