Skip to content

fix(cwv-trends-audit): read endDate from auditContext.messageData#2327

Open
szergen wants to merge 5 commits intomainfrom
cwv-trends-audit-enddate-slack-fix
Open

fix(cwv-trends-audit): read endDate from auditContext.messageData#2327
szergen wants to merge 5 commits intomainfrom
cwv-trends-audit-enddate-slack-fix

Conversation

@szergen
Copy link
Copy Markdown
Contributor

@szergen szergen commented Apr 7, 2026

  • When triggered via the Slack bot, keyword args like endDate:2026-04-05 are serialized into message.data by the API service and surfaced as auditContext.messageData.endDate by the RunnerAudit framework.
  • The runner now checks auditContext.endDate ?? auditContext.messageData?.endDate so the provided date is used instead of defaulting to today.

Please ensure your pull request adheres to the following guidelines:

  • make sure to link the related issues in this description
  • when merging / squashing, make sure the fixed issue references are visible in the commits, for easy compilation of release notes
  • If data sources for any opportunity has been updated/added, please update the wiki for same opportunity.

Related Issues

Thanks for contributing!

aciobanasu and others added 3 commits April 6, 2026 15:59
…fallback

When triggered via the Slack bot, keyword args like endDate:2026-04-05
are serialized into message.data by the API service and surfaced as
auditContext.messageData.endDate by the RunnerAudit framework.
The runner now checks auditContext.endDate ?? auditContext.messageData?.endDate
so the provided date is used instead of defaulting to today.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 7, 2026

This PR will trigger a patch release when merged.

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@szergen szergen requested a review from iuliag April 7, 2026 14:13
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