[PROD RELEASE] - AI Review/Updates & Fixes

.circleci/config.yml

@@ -160,7 +160,7 @@ workflows:
           context: org-global
           filters: &filters-dev
             branches:
-              only: ["develop", "pm-2917", "points", "pm-3270", "permissions-hotfix"]
+              only: ["develop", "pm-2917", "points", "pm-3270", "projects-api-v6", "PM-3813_ai-reviewer-configs"]
 
       # Production builds are exectuted only on tagged commits to the
       # master branch.

.github/workflows/trivy.yaml

@@ -18,7 +18,7 @@ jobs:
         uses: actions/checkout@v4
 
       - name: Run Trivy scanner in repo mode
-        uses: aquasecurity/trivy-action@0.33.1
+        uses: aquasecurity/trivy-action@0.35.0
         with:
           scan-type: "fs"
           ignore-unfixed: true

config/constants/development.js

@@ -22,6 +22,7 @@ module.exports = {
   TC_FINANCE_API_URL: process.env.TC_FINANCE_API_URL || `${API_V6}/finance`,
   TC_AI_API_BASE_URL: process.env.TC_AI_API_BASE_URL || `${API_V6}/ai`,
   TC_AI_SKILLS_EXTRACTION_WORKFLOW_ID: process.env.TC_AI_SKILLS_EXTRACTION_WORKFLOW_ID || 'skillExtractionWorkflow',
+  TC_AI_AUTOWRITE_WORKFLOW_ID: process.env.TC_AI_AUTOWRITE_WORKFLOW_ID || 'jdAutowriteWorkflow',
   CHALLENGE_DEFAULT_REVIEWERS_URL: `${DEV_API_HOSTNAME}/v6/challenge/default-reviewers`,
   CHALLENGE_API_VERSION: '1.1.0',
   CHALLENGE_TIMELINE_TEMPLATES_URL: `${DEV_API_HOSTNAME}/v6/timeline-templates`,

config/constants/local.js

@@ -36,6 +36,7 @@ module.exports = {
   ENGAGEMENTS_ROOT_API_URL: `${LOCAL_CHALLENGE_API}/engagements`,
   APPLICATIONS_API_URL: `${LOCAL_CHALLENGE_API}/engagements/applications`,
   TC_FINANCE_API_URL: process.env.TC_FINANCE_API_URL || 'http://localhost:3009/v6/finance',
+  TC_AI_AUTOWRITE_WORKFLOW_ID: process.env.TC_AI_AUTOWRITE_WORKFLOW_ID || 'jdAutowriteWorkflow',
   CHALLENGE_DEFAULT_REVIEWERS_URL: `${LOCAL_CHALLENGE_API.replace(/\/v6$/, '')}/v6/challenge/default-reviewers`,
   CHALLENGE_API_VERSION: '1.1.0',
   CHALLENGE_TIMELINE_TEMPLATES_URL: `${LOCAL_CHALLENGE_API}/timeline-templates`,

config/constants/production.js

@@ -21,6 +21,7 @@ module.exports = {
   TC_FINANCE_API_URL: process.env.TC_FINANCE_API_URL || `${API_V6}/finance`,
   TC_AI_API_BASE_URL: process.env.TC_AI_API_BASE_URL || `${API_V6}/ai`,
   TC_AI_SKILLS_EXTRACTION_WORKFLOW_ID: process.env.TC_AI_SKILLS_EXTRACTION_WORKFLOW_ID || 'skillExtractionWorkflow',
+  TC_AI_AUTOWRITE_WORKFLOW_ID: process.env.TC_AI_AUTOWRITE_WORKFLOW_ID || 'jdAutowriteWorkflow',
   CHALLENGE_DEFAULT_REVIEWERS_URL: `${PROD_API_HOSTNAME}/v6/challenge/default-reviewers`,
   CHALLENGE_API_VERSION: '1.1.0',
   CHALLENGE_TIMELINE_TEMPLATES_URL: `${PROD_API_HOSTNAME}/v6/timeline-templates`,
@@ -37,9 +38,9 @@ module.exports = {
   RESOURCE_ROLES_API_URL: `${PROD_API_HOSTNAME}/v6/resource-roles`,
   SUBMISSIONS_API_URL: `${PROD_API_HOSTNAME}/v6/submissions`,
   REVIEW_TYPE_API_URL: `${PROD_API_HOSTNAME}/v6/reviewTypes`,
-  REVIEWS_API_URL: `${PROD_API_HOSTNAME}/v6/reviews`, 
+  REVIEWS_API_URL: `${PROD_API_HOSTNAME}/v6/reviews`,
   SCORECARDS_API_URL: `${PROD_API_HOSTNAME}/v6/scorecards`,
-  WORKFLOWS_API_URL: `${PROD_API_HOSTNAME}/v6/workflows`, 
+  WORKFLOWS_API_URL: `${PROD_API_HOSTNAME}/v6/workflows`,
   SUBMISSION_REVIEW_APP_URL: `https://submission-review.${DOMAIN}/challenges`,
   STUDIO_URL: `https://studio.${DOMAIN}`,
   CONNECT_APP_URL: `https://connect.${DOMAIN}`,

src/actions/engagements.js

@@ -4,7 +4,6 @@ import {
   fetchEngagement,
   createEngagement as createEngagementAPI,
   updateEngagement as updateEngagementAPI,
-  patchEngagement,
   deleteEngagement as deleteEngagementAPI
 } from '../services/engagements'
 import { fetchSkillsByIds } from '../services/skills'
@@ -345,7 +344,10 @@ export function updateEngagementDetails (engagementId, engagementDetails, projec
 }
 
 /**
- * Partially updates engagement details
+ * Partially updates engagement details.
+ *
+ * The engagements API currently accepts partial engagement payloads through the
+ * `PUT :id` update route rather than a dedicated `PATCH :id` handler.
  * @param {String|Number} engagementId
  * @param {Object} partialDetails
  * @param {String|Number} projectId
@@ -358,7 +360,7 @@ export function partiallyUpdateEngagementDetails (engagementId, partialDetails,
     })
 
     try {
-      const response = await patchEngagement(engagementId, partialDetails)
+      const response = await updateEngagementAPI(engagementId, partialDetails)
       const [hydratedEngagement] = await hydrateEngagementSkills([_.get(response, 'data', {})])
       const updatedDetails = normalizeEngagement(hydratedEngagement || {})
       return dispatch({

src/actions/payments.js

@@ -19,6 +19,7 @@ const ENGAGEMENT_PAYMENT_STATUS = 'ON_HOLD_ADMIN'
  * @param {String} remarks
  * @param {String|Number} agreementRate
  * @param {String|Number} amount
+ * @param {String|Number} hoursWorked
  * @param {String|Number} billingAccountId
  * @param {String} paymentStatus
  */
@@ -30,6 +31,7 @@ export function createMemberPayment (
   remarks,
   agreementRate,
   amount,
+  hoursWorked,
   billingAccountId,
   paymentStatus = ENGAGEMENT_PAYMENT_STATUS
 ) {
@@ -39,6 +41,7 @@ export function createMemberPayment (
     })
 
     const parsedAmount = Number(amount)
+    const parsedHoursWorked = Number(hoursWorked)
     const trimmedTitle = typeof paymentTitle === 'string'
       ? paymentTitle.trim()
       : (paymentTitle != null ? String(paymentTitle) : '')
@@ -60,6 +63,9 @@ export function createMemberPayment (
       description: trimmedTitle,
       externalId: String(assignmentId),
       attributes,
+      ...(Number.isFinite(parsedHoursWorked) && parsedHoursWorked > 0
+        ? { hoursWorked: parsedHoursWorked }
+        : {}),
       ...(paymentStatus ? { status: paymentStatus } : {}),
       details: [
         {