incorporate review comments

Author: tkotthakota-adobe

.eslintrc.cjs

@@ -1,5 +1,5 @@
 /*
- * Copyright 2021 Adobe. All rights reserved.
+ * Copyright 2025 Adobe. All rights reserved.
  * This file is licensed to you under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License. You may obtain a copy
  * of the License at http://www.apache.org/licenses/LICENSE-2.0

CONTRIBUTING.md

@@ -1,4 +1,4 @@
-# Contributing to Project Franklin
+# Contributing to Sites Optimizer
 
 This project (like almost all of Project Franklin) is an Open Development project and welcomes contributions from everyone who finds it useful or lacking.
 

src/tasks/demo-url-processor/handler.js

@@ -42,9 +42,7 @@ export async function runDemoUrlProcessor(message, context) {
   try {
     // prepare demo url
     const demoUrl = prepareDemoUrl(siteUrl, organizationId, siteId);
-    let slackMessage = ':white_check_mark: Setup complete!';
-    await say(env, log, slackContext, slackMessage);
-    slackMessage = `:information_source: Access your demo environment here: ${demoUrl}`;
+    const slackMessage = `:white_check_mark: Setup complete! Access your demo environment here: ${demoUrl}`;
     await say(env, log, slackContext, slackMessage);
     log.info(`Setup complete! Access your demo environment here: ${demoUrl}`);
   } catch (error) {

src/tasks/disable-import-audit-processor/handler.js

@@ -36,7 +36,6 @@ export async function runDisableImportAuditProcessor(message, context) {
     importTypes,
     auditTypes,
   });
-  await say(env, log, slackContext, ':broom: *Disabling imports and audits*');
   try {
     // Database operations
     log.info('Starting database operations');
@@ -62,9 +61,7 @@ export async function runDisableImportAuditProcessor(message, context) {
     await configuration.save();
     log.info('Database changes saved successfully');
 
-    let slackMessage = `:white_check_mark: Disabled imports ${JSON.stringify(importTypes)} for site ${siteId}`;
-    await say(env, log, slackContext, slackMessage);
-    slackMessage = `:white_check_mark: Disabled audits ${JSON.stringify(auditTypes)} for site ${siteId}`;
+    const slackMessage = `:broom: *Disabled imports*: ${importTypes.join(', ')} and *audits*: ${auditTypes.join(', ')}`;
     await say(env, log, slackContext, slackMessage);
   } catch (error) {
     log.error('Error in disable import and audit processor:', {

src/tasks/opportunity-status-processor/handler.js

@@ -60,7 +60,6 @@ export async function runOpportunityStatusProcessor(message, context) {
     auditTypes,
   });
 
-  await say(env, log, slackContext, ':information_source: *Checking opportunity status*');
   try {
     // Get the site and its opportunities
     const site = await Site.findById(siteId);