Skip to content

Commit 526959f

Browse files
Daniel Baticaclaude
andcommitted
feat: pass opportunityId to assess-urls worker for precheck persistence
Include opportunityId in SQS message for assess-urls action to enable worker to fetch and update suggestion entities with precheck results, preventing data loss from async Lambda invocations. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
1 parent 285d3aa commit 526959f

1 file changed

Lines changed: 14 additions & 6 deletions

File tree

src/controllers/suggestions.js

Lines changed: 14 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -970,14 +970,22 @@ function SuggestionsController(ctx, sqs, env) {
970970
return badRequest(`Handler is not enabled for site ${site.getId()} autofix type ${opportunity.getType()}`);
971971
}
972972
const { AUTOFIX_JOBS_QUEUE: queueUrl } = env;
973-
// Intentionally omit opportunityId: worker uses context differently for URL-based assessments
974-
await sqs.sendMessage(queueUrl, {
973+
await sqs.sendMessage(
974+
queueUrl,
975+
{
976+
siteId,
977+
opportunityId,
978+
action: 'assess-urls',
979+
pages,
980+
...(precheckOnly === true && { precheckOnly: true }),
981+
},
982+
);
983+
return accepted({
984+
message: 'Assess-urls job queued',
975985
siteId,
976-
action: 'assess-urls',
977-
pages,
978-
...(precheckOnly === true && { precheckOnly: true }),
986+
opportunityId,
987+
pagesCount: pages.length,
979988
});
980-
return accepted({ message: 'Assess-urls job queued', siteId, pagesCount: pages.length });
981989
}
982990

983991
// suggestion-based flow (assess, fix, etc.)

0 commit comments

Comments
 (0)