[PROD RELEASE] - AI & V2 engagements & Fixes by kkartunov · Pull Request #1728 · topcoder-platform/work-manager

kkartunov · 2026-02-12T16:29:04Z

No description provided.

…topcoder.atlassian.net/browse/PM-3347)

…s input in challenge & engagement forms

Engagements v2 changes

…kills PM-3786 ai assisted skills suggestions

…kills PM-3786 #time 30min build & lint fixes

dist/static/media/marked.7fa2a38c.cjs

github-actions · 2026-02-12T16:29:22Z

.circleci/config.yml

 defaults: &defaults
  docker:
-    - image: cimg/python:3.11.11-browsers
+    - image: cimg/python:3.12.12-browsers


[❗❗ correctness]
Upgrading the Python image from 3.11.11 to 3.12.12 may introduce compatibility issues with existing code or dependencies. Ensure that all dependencies and code are compatible with Python 3.12.12 and that thorough testing is conducted to verify this change.

github-actions · 2026-02-12T16:29:28Z

config/webpack.config.js

      // require.resolve('webpack/hot/dev-server'),
      isEnvDevelopment &&
-        require.resolve('react-dev-utils/webpackHotDevClient'),
+        path.resolve(__dirname, 'webpackHotDevClient'),


[❗❗ correctness]
The change from require.resolve('react-dev-utils/webpackHotDevClient') to path.resolve(__dirname, 'webpackHotDevClient') could lead to potential issues if the webpackHotDevClient file is not present in the expected directory. Ensure that this file is correctly located and that any differences in behavior between the resolved module and the local file are intentional.

github-actions · 2026-02-12T16:31:11Z

dist/976489525ddbc1ce.ttf

@@ -0,0 +1 @@
+module.exports = __webpack_public_path__ + "static/media/roboto-blackitalic.9fce9ccc.ttf";


[⚠️ maintainability]
Consider using path.join from the Node.js path module to construct file paths. This approach can improve cross-platform compatibility by ensuring the correct path separators are used.

github-actions · 2026-02-12T16:31:15Z

dist/9ff19a4f2cee2a22.eot

@@ -0,0 +1 @@
+module.exports = __webpack_public_path__ + "static/media/roboto-bolditalic.aa10b644.eot";


[💡 style]
The lack of a newline at the end of the file can cause issues with some tools that process files line by line. Consider adding a newline to improve compatibility and maintainability.

github-actions · 2026-02-12T16:31:18Z

dist/a103d3afb23be6d5.woff

@@ -0,0 +1 @@
+module.exports = __webpack_public_path__ + "static/media/roboto-thin.391f6caa.woff";


[💡 style]
Consider adding a newline at the end of the file. While this is not a correctness issue, it is a common convention that can help prevent issues with some tools and version control systems.

github-actions · 2026-02-12T16:31:23Z

dist/abe50b9e8f8dd1b9.ttf

@@ -0,0 +1 @@
+module.exports = __webpack_public_path__ + "static/media/roboto-mediumitalic.3eef81d5.ttf";


[⚠️ correctness]
The use of __webpack_public_path__ assumes that it is correctly set at runtime. Ensure that this variable is properly configured in all environments where this code will run to avoid potential path resolution issues.

github-actions · 2026-02-12T16:31:27Z

dist/ad4d04a37af8ada3.svg

@@ -0,0 +1 @@
+module.exports = __webpack_public_path__ + "static/media/roboto-lightitalic.baa43a84.svg";


[❗❗ correctness]
The use of __webpack_public_path__ assumes that it is correctly set up and available in the environment where this module is used. Ensure that this variable is properly configured to avoid runtime errors related to incorrect asset paths.

github-actions · 2026-02-12T16:31:32Z

dist/bf79b0288d0b9cb9.ttf

@@ -0,0 +1 @@
+module.exports = __webpack_public_path__ + "static/media/roboto-bold.a452a6e8.ttf";


[💡 style]
Consider adding a newline at the end of the file. While this is not a correctness issue, it is a common convention that can help avoid potential issues with some tools and version control systems.

github-actions · 2026-02-12T16:31:36Z

dist/c984e46df0e90c1f.eot

@@ -0,0 +1 @@
+module.exports = __webpack_public_path__ + "static/media/roboto-light.82fabc8a.eot";


[⚠️ correctness]
The use of __webpack_public_path__ assumes that it is correctly set up in the webpack configuration. Ensure that this variable is properly defined and managed to avoid potential issues with asset loading paths in different environments.

github-actions · 2026-02-12T16:31:36Z

dist/c984e46df0e90c1f.eot

@@ -0,0 +1 @@
+module.exports = __webpack_public_path__ + "static/media/roboto-light.82fabc8a.eot";


[💡 style]
Consider adding a newline at the end of the file. While this is a minor issue, it can help avoid unnecessary diffs in version control systems and is a common convention.

github-actions · 2026-02-12T16:31:40Z

dist/d2276a84aff4ac30.svg

@@ -0,0 +1 @@
+module.exports = __webpack_public_path__ + "static/media/roboto-bold.aff20cb4.svg";


[❗❗ correctness]
The use of __webpack_public_path__ can be risky if not properly configured, as it relies on the global state of the webpack runtime. Ensure that __webpack_public_path__ is set correctly in all environments to avoid loading assets from incorrect paths.

github-actions · 2026-02-12T16:31:41Z

dist/d2276a84aff4ac30.svg

@@ -0,0 +1 @@
+module.exports = __webpack_public_path__ + "static/media/roboto-bold.aff20cb4.svg";


[💡 style]
Consider adding a newline at the end of the file to comply with POSIX standards, which can help prevent issues with some tools and version control systems.

github-actions · 2026-02-12T16:31:45Z

dist/d44f7fb00f260066.ttf

@@ -0,0 +1 @@
+module.exports = __webpack_public_path__ + "static/media/roboto-medium.9f855913.ttf";


[💡 style]
Consider adding a newline at the end of the file. While this is not a correctness issue, it is a common convention that can help prevent potential issues with certain tools or when concatenating files.

github-actions · 2026-02-12T16:31:48Z

dist/e86e56fec2935cb2.svg

@@ -0,0 +1 @@
+module.exports = __webpack_public_path__ + "static/media/roboto-black.7e09c6e9.svg";


[💡 style]
Consider adding a newline at the end of the file. While this is not a correctness issue, it is a common convention that can help avoid potential issues with certain tools or when concatenating files.

github-actions · 2026-02-12T16:31:51Z

dist/eba69177dc437c4f.svg

@@ -0,0 +1 @@
+module.exports = __webpack_public_path__ + "static/media/roboto-thinitalic.a96ddaaf.svg";


[💡 style]
Consider adding a newline at the end of the file. While not technically incorrect, having a newline at the end of files is a convention that can prevent potential issues with some tools and version control systems.

github-actions · 2026-02-12T16:31:55Z

dist/f196a91cced815e5.svg

@@ -0,0 +1 @@
+module.exports = __webpack_public_path__ + "static/media/roboto-bolditalic.fd7a4235.svg";


[💡 style]
Consider adding a newline at the end of the file. While not technically incorrect, having a newline at the end of files is a common convention that can prevent potential issues with certain tools and version control systems.

github-actions · 2026-02-12T16:31:58Z

dist/f598273ad7e765f0.woff

@@ -0,0 +1 @@
+module.exports = __webpack_public_path__ + "static/media/roboto-medium.8b88b4ee.woff";


[💡 style]
Consider adding a newline at the end of the file. While this is not technically incorrect, it is a common convention that can prevent issues with certain tools or when concatenating files.

github-actions · 2026-02-12T16:32:03Z

dist/index.html

+    />
+    <meta name="theme-color" content="#000000" />
+    <title>Work Manager - Topcoder</title>
+  <script defer src="/static/js/runtime~main.js"></script><script defer src="/static/js/vendors-node_modules_pnpm_fortawesome_free-solid-svg-icons_5_15_4_node_modules_fortawesome_fr-d89314.js"></script><script defer src="/static/js/main.js"></script></head>


[💡 readability]
Consider splitting the <script> tags onto separate lines for improved readability and maintainability. This will make it easier to manage and update individual scripts in the future.

github-actions · 2026-02-12T20:40:03Z

src/actions/engagements.js

+  if (uncachedProjectIds.length) {
+    const projectNameEntries = await Promise.all(
+      uncachedProjectIds.map(async (projectId) => {
+        try {


[⚠️ maintainability]
Consider adding logging or handling for the error case in fetchProjectById. Currently, if an error occurs, it silently returns null, which might make debugging difficult.

github-actions · 2026-02-12T20:40:03Z

src/actions/engagements.js

+    const projectNameEntries = await Promise.all(
+      uncachedProjectIds.map(async (projectId) => {
+        try {
+          const project = await fetchProjectById(projectId)


[❗❗ correctness]
Ensure that fetchProjectById handles network errors and returns a consistent response. If it throws an error, it might cause the hydrateEngagementProjectNames function to fail unexpectedly.

github-actions · 2026-02-12T20:40:03Z

src/actions/engagements.js

+      })
+    )
+
+    projectNameEntries.forEach(([projectId, projectName]) => {


[⚠️ performance]
The projectNameCache is a global object that could grow indefinitely if the application runs for a long time or processes many projects. Consider implementing a cache eviction strategy to prevent potential memory issues.

github-actions · 2026-02-12T20:40:09Z

src/components/DateInput/index.js

@@ -61,6 +65,52 @@ const DateInput = forwardRef(({
    return valueAsDate.getTime() < minAsDate.getTime() ? minAsDate : valueAsDate
  }

+  const adjustPopoverToViewport = () => {


[❗❗ correctness]
The adjustPopoverToViewport function relies on window and document objects, which are not available in server-side rendering environments. Consider adding checks or guards to ensure this function is only called in client-side environments.

github-actions · 2026-02-12T20:40:09Z

src/components/DateInput/index.js

@@ -83,6 +133,7 @@ const DateInput = forwardRef(({
        }
        onChange(newValue)
      }}
+      onFocus={adjustPopoverToViewport}


[⚠️ performance]
Attaching adjustPopoverToViewport to the onFocus event might lead to performance issues if the function is computationally expensive or if focus events are triggered frequently. Consider evaluating the performance impact and optimizing if necessary.

github-actions · 2026-02-12T20:40:13Z

src/components/EngagementsList/EngagementsList.module.scss

@@ -164,7 +168,7 @@

  .filters {
    display: grid;
-    grid-template-columns: minmax(240px, 2fr) repeat(3, minmax(160px, 1fr));
+    grid-template-columns: minmax(240px, 2fr) repeat(2, minmax(160px, 1fr));


[⚠️ design]
The change from repeat(3, minmax(160px, 1fr)) to repeat(2, minmax(160px, 1fr)) reduces the number of columns in the grid layout. Ensure that this change aligns with the intended design and that it does not negatively impact the layout on different screen sizes.

github-actions · 2026-02-12T20:40:18Z

src/components/EngagementsList/index.js

+  return fallbackProjectId
+}
+
+const getEngagementProjectName = (engagement, fallbackProjectName = null) => {


[💡 maintainability]
The function getEngagementProjectName could be optimized by checking for engagement once at the beginning, reducing redundancy in the checks for engagement.projectName and engagement.project.name.

github-actions · 2026-02-12T20:40:18Z

src/components/EngagementsList/index.js

-  const [statusFilter, setStatusFilter] = useState(DEFAULT_STATUS_OPTION)
-  const [sortBy, setSortBy] = useState(SORT_OPTIONS[0])
-  const [sortOrder, setSortOrder] = useState(SORT_ORDER_OPTIONS[0])
+  const [searchProjectName, setSearchProjectName] = useState('')


[💡 readability]
Consider renaming searchProjectName to projectNameFilter for consistency with other filter state variables like statusFilter and visibilityFilter.

github-actions · 2026-02-12T20:40:18Z

src/components/EngagementsList/index.js

@@ -353,59 +265,55 @@ const EngagementsList = ({

              return (
                <tr key={engagement.id || engagement.title}>


[⚠️ correctness]
Using engagement.id || engagement.title as the key for the table rows could lead to duplicate keys if engagement.id is null and multiple engagements have the same title. Consider using a more unique identifier.

github-actions · 2026-02-12T20:40:24Z

src/containers/Tab/index.js

    if (!projectId) {
      return 0
    }
    if (pathname.includes(`/projects/${projectId}/engagements`)) {
-      return 2
+      return canViewEngagements ? 2 : 0


[❗❗ correctness]
The change here introduces a conditional return value based on canViewEngagements. Ensure that the logic for determining canViewEngagements is correctly implemented and tested, as this impacts the tab navigation logic.

github-actions · 2026-02-12T20:40:24Z

src/containers/Tab/index.js

    if (projectId) {
-      if (tab === 3 && !canViewAssets) {
+      if ((tab === 2 && !canViewEngagements) || (tab === 3 && !canViewAssets)) {


[❗❗ correctness]
The condition now checks both tab === 2 && !canViewEngagements and tab === 3 && !canViewAssets. Ensure that these conditions are correctly aligned with the intended tab access logic, as incorrect conditions could prevent users from accessing the correct tabs.

github-actions · 2026-02-12T20:40:24Z

src/containers/Tab/index.js

        return
      }
      if (tab === 1) {
        history.push(`/projects/${projectId}/challenges`)
        this.setState({ currentTab: 1 })
-      } else if (tab === 2) {
+      } else if (tab === 2 && canViewEngagements) {


[⚠️ security]
The added condition && canViewEngagements ensures that navigation to the engagements tab only occurs if the user has the appropriate permissions. Verify that this logic is consistent with the application's permission model.

github-actions · 2026-02-12T20:40:30Z

src/routes.js

@@ -124,7 +124,7 @@ class Routes extends React.Component {
    const isReadOnly = checkReadOnlyRoles(this.props.token)
    const isCopilot = checkCopilot(this.props.token)
    const isAdmin = checkAdmin(this.props.token)
-    const canManageEngagements = checkAdminOrPmOrTaskManager(this.props.token, null)
+    const canAccessEngagements = checkAdminOrTalentManager(this.props.token)



[⚠️ maintainability]
The variable canAccessEngagements is used multiple times to control access to routes. Consider extracting this logic into a separate function or utility to improve maintainability and reduce repetition.

github-actions · 2026-02-12T20:40:30Z

src/routes.js

-          {!canManageEngagements && (
-            <Route exact path='/projects/:projectId/engagements/new'
+          {canAccessEngagements && (
+            <Route exact path='/projects/:projectId/engagements/:engagementId/applications'


[⚠️ design]
The route for /projects/:projectId/engagements/:engagementId/applications is nested under canAccessEngagements, which seems to imply that only users with access can view applications. Ensure this is the intended behavior, as it might restrict access more than necessary.

github-actions · 2026-02-12T20:40:30Z

src/routes.js

-          {!canManageEngagements && (
-            <Route path='/projects/:projectId/engagements/:engagementId'
+          {!canAccessEngagements && (
+            <Route path='/projects/:projectId/engagements'


[❗❗ security]
The warning message for users without access to engagements has been updated to 'You need Admin or Talent Manager role to view engagements'. Ensure that this change aligns with the intended access control policy, as it might affect user roles and permissions.

fix(PM-3485): fixed columns and data mismatch in resources tab

github-actions · 2026-02-16T07:49:23Z

.gitignore

-test-automation/test-results
+test-automation/test-results
+
+dist


[💡 maintainability]
The entry dist is already included in the .gitignore under the # production section at line 15. This duplication is unnecessary and could lead to confusion. Consider removing the duplicate entry.

jmgasper and others added 24 commits February 6, 2026 12:11

Missing assign members section on checkpoint reviewer setup (https://…

52363ce

…topcoder.atlassian.net/browse/PM-3347)

Show assignment of checkpoint reviewers on edit challenge page

7655a19

v6 project API updates

d44d9ba

Revert this change to not break dev environment

4105e35

Turn this down to avoid timeouts

2618327

Fix projects API route from local back to something that works.

70fa065

Change chunk size to avoid timeouts

cc50fab

Change chunk size to avoid timeouts

9263eb6

PM-3786 #time 5h Base service for calling workflow api

bcbcc8e

PM-3786 #time 2h FE integration for ai-suggested skills for the skill…

4638c97

…s input in challenge & engagement forms

Engagements updates for v2

ca3b234

Further v2 engagement updates

2989706

Checkbox alignment

5aa2478

Merge branch 'develop' into chunk_size

7d1d959

Merge pull request #1726 from topcoder-platform/chunk_size

3d337d5

Engagements v2 changes

Build fix

281a998

Build fix

9613ac3

Build fix

928a24d

Build fix

ee9a63b

V2 engagement tweaks https://topcoder.atlassian.net/browse/PM-3699

198c4dd

Further alignment tweak

3a051ff

Merge pull request #1725 from topcoder-platform/PM-3786_ai-assisted-s…

fd0ecba

…kills PM-3786 ai assisted skills suggestions

PM-3786 #time 30min build & lint fixes

6feee77

Merge pull request #1727 from topcoder-platform/PM-3786_ai-assisted-s…

c091375

…kills PM-3786 #time 30min build & lint fixes

github-advanced-security bot found potential problems Feb 12, 2026

View reviewed changes

kkartunov requested review from jmgasper and vas3a and removed request for jmgasper February 12, 2026 16:29

github-actions bot reviewed Feb 12, 2026

View reviewed changes

hentrymartin and others added 3 commits February 12, 2026 20:48

PM-3485 #time 2h fixed columns and data mismatch in resources tab

052e0d5

Update gitignore

c3a74f4

QA and requested tweaks

500e9b1

github-actions bot reviewed Feb 12, 2026

View reviewed changes

Add some prompts to the dropdowns to tell them apart

aa60cbf

jmgasper approved these changes Feb 12, 2026

View reviewed changes

hentrymartin and others added 2 commits February 13, 2026 21:04

Merge pull request #1729 from topcoder-platform/pm-3485

cddfaf1

fix(PM-3485): fixed columns and data mismatch in resources tab

clean up dist

015a292

github-actions bot reviewed Feb 16, 2026

View reviewed changes

kkartunov changed the title ~~[PROD RELEASE]~~ [PROD RELEASE] - AI & V2 engagements & Fixes Feb 16, 2026

kkartunov merged commit 38c5eab into master Feb 16, 2026
9 checks passed

		@@ -0,0 +1 @@
		module.exports = __webpack_public_path__ + "static/media/roboto-blackitalic.9fce9ccc.ttf"; No newline at end of file

		@@ -353,59 +265,55 @@ const EngagementsList = ({

		return (
		<tr key={engagement.id \|\| engagement.title}>

Comments

Conversation

kkartunov commented Feb 12, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions bot Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions bot Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions bot Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions bot Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions bot Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions bot Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions bot Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions bot Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions bot Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions bot Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions bot Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions bot Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions bot Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions bot Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions bot Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions bot Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions bot Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions bot Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions bot Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions bot Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions bot Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions bot Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions bot Feb 12, 2026

Choose a reason for hiding this comment

Uh oh!

github-actions bot Feb 12, 2026

Choose a reason for hiding this comment