Skip to content

Conversation

abhishekgarg18
Copy link
Contributor

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!

@abhishekgarg18 abhishekgarg18 requested a review from naydav October 16, 2025 09:45
@abhishekgarg18 abhishekgarg18 changed the title send each cwv url fetch from rum separately so that mystique can process it. send each cwv url fetched from rum separately so that mystique can process it. Oct 16, 2025
@github-actions
Copy link

This PR will trigger no release when merged.

// eslint-disable-next-line no-await-in-loop
await sqs.sendMessage(env.QUEUE_SPACECAT_TO_MYSTIQUE, sqsMessage);
log.info(`CWV opportunity sent to Mystique for auto-suggest - siteId: ${siteId}, opportunityId: ${opportunityId}`);
// Filter for URL-type entries only (skip groups) and ensure suggestions exist for them.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps we can move part of the logic into needsAutoSuggest, which decides whether to send an SQS message to auto-guidance.

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.

2 participants