challenge additions to platform ui for ai arena#1552
Open
jpeg22 wants to merge 4 commits intotopcoder-platform:ai-arena-managerfrom
Open
challenge additions to platform ui for ai arena#1552jpeg22 wants to merge 4 commits intotopcoder-platform:ai-arena-managerfrom
jpeg22 wants to merge 4 commits intotopcoder-platform:ai-arena-managerfrom
Conversation
jmgasper
requested changes
Mar 26, 2026
Collaborator
jmgasper
left a comment
There was a problem hiding this comment.
I can't run this because the craco.config.js file update broke the build:
platform-ui ai_arena_march_2026 ❯ yarn start
yarn run v1.22.22
$ bash start.sh
$ craco start --mode ${LOGICAL_ENV:-dev}
/home/jmgasper/Documents/Git/v6/platform-ui/craco.config.js:68
SyntaxError: Unexpected end of input
at wrapSafe (node:internal/modules/cjs/loader:1512:18)
at Module._compile (node:internal/modules/cjs/loader:1534:20)
at Object..js (node:internal/modules/cjs/loader:1699:10)
at Module.load (node:internal/modules/cjs/loader:1313:32)
at Function._load (node:internal/modules/cjs/loader:1123:12)
at TracingChannel.traceSync (node:diagnostics_channel:322:14)
at wrapModuleLoad (node:internal/modules/cjs/loader:217:24)
at Module.require (node:internal/modules/cjs/loader:1335:12)
at module.exports (/home/jmgasper/Documents/Git/v6/platform-ui/node_modules/import-fresh/index.js:33:91)
at loadJs (/home/jmgasper/Documents/Git/v6/platform-ui/node_modules/cosmiconfig/dist/loaders.js:16:18)
Node.js v22.13.0
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
I got really confused by the way this change was first shown in Devin last commit so I'm committing it as recommended in the github UI Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com>
jmgasper
reviewed
Mar 26, 2026
Collaborator
jmgasper
left a comment
There was a problem hiding this comment.
Still not working:
platform-ui ai_arena_march_2026 ❯ yarn start
yarn run v1.22.22
$ bash start.sh
$ craco start --mode ${LOGICAL_ENV:-dev}
/home/jmgasper/Documents/Git/v6/platform-ui/craco.config.js:50
webpack: {
^
SyntaxError: Unexpected token ':'
at wrapSafe (node:internal/modules/cjs/loader:1512:18)
at Module._compile (node:internal/modules/cjs/loader:1534:20)
at Object..js (node:internal/modules/cjs/loader:1699:10)
at Module.load (node:internal/modules/cjs/loader:1313:32)
at Function._load (node:internal/modules/cjs/loader:1123:12)
at TracingChannel.traceSync (node:diagnostics_channel:322:14)
at wrapModuleLoad (node:internal/modules/cjs/loader:217:24)
at Module.require (node:internal/modules/cjs/loader:1335:12)
at module.exports (/home/jmgasper/Documents/Git/v6/platform-ui/node_modules/import-fresh/index.js:33:91)
at loadJs (/home/jmgasper/Documents/Git/v6/platform-ui/node_modules/cosmiconfig/dist/loaders.js:16:18)
Node.js v22.13.0
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
Collaborator
Author
|
I’m gonna have to look at it. I think the Devin comments are not right for
this file.
…On Thu, Mar 26, 2026 at 2:04 AM Justin Gasper ***@***.***> wrote:
***@***.**** commented on this pull request.
Still not working:
platform-ui ai_arena_march_2026 ❯ yarn start
yarn run v1.22.22
$ bash start.sh
$ craco start --mode ${LOGICAL_ENV:-dev}
/home/jmgasper/Documents/Git/v6/platform-ui/craco.config.js:50
webpack: {
^
SyntaxError: Unexpected token ':'
at wrapSafe (node:internal/modules/cjs/loader:1512:18)
at Module._compile (node:internal/modules/cjs/loader:1534:20)
at Object..js (node:internal/modules/cjs/loader:1699:10)
at Module.load (node:internal/modules/cjs/loader:1313:32)
at Function._load (node:internal/modules/cjs/loader:1123:12)
at TracingChannel.traceSync (node:diagnostics_channel:322:14)
at wrapModuleLoad (node:internal/modules/cjs/loader:217:24)
at Module.require (node:internal/modules/cjs/loader:1335:12)
at module.exports (/home/jmgasper/Documents/Git/v6/platform-ui/node_modules/import-fresh/index.js:33:91)
at loadJs (/home/jmgasper/Documents/Git/v6/platform-ui/node_modules/cosmiconfig/dist/loaders.js:16:18)
Node.js v22.13.0
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
—
Reply to this email directly, view it on GitHub
<#1552?email_source=notifications&email_token=AXO56SM7RINDPPRODJ7KEP34STB55A5CNFSNUABKM5UWIORPF5TWS5BNNB2WEL2QOVWGYUTFOF2WK43UKJSXM2LFO4XTIMBRGE3DKNBVG44KM4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2L24DSL5ZGK5TJMV3V63TPORUWM2LDMF2GS33OONPWG3DJMNVQ#pullrequestreview-4011654578>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AXO56SP76YVI6QP6MQHNFQL4STB55AVCNFSM6AAAAACW63CSJ2VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHM2DAMJRGY2TINJXHA>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Collaborator
Author
|
I did
…On Thu, Mar 26, 2026 at 6:33 AM Kiril Kartunov ***@***.***> wrote:
***@***.**** requested changes on this pull request.
@jpeg22 <https://github.com/jpeg22> please fix the issues outlined by
Devin & Copilot. Do not just mark them as resolved but fix or post comments
with reasoning.
—
Reply to this email directly, view it on GitHub
<#1552?email_source=notifications&email_token=AXO56SJOUKTHT4O2WVTDZI34SUBRLA5CNFSNUABKM5UWIORPF5TWS5BNNB2WEL2QOVWGYUTFOF2WK43UKJSXM2LFO4XTIMBRGI4TMOJXGE32M4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJPLQOJPXEZLWNFSXOX3ON52GSZTJMNQXI2LPNZZV6Y3MNFRWW#pullrequestreview-4012969717>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AXO56SIZRQMNT45WO7OTJOT4SUBRLAVCNFSM6AAAAACW63CSJ2VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHM2DAMJSHE3DSNZRG4>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Collaborator
Author
|
On this particular issue, Devin believed we should have a v6 path element
and it suggested a commit that didn’t seem to have correct syntax so I made
my own change. Tha appears to have been missing a } so when Devin flagged
it again, I committed Devin’s suggestion, but that also had a syntax error,
so I’ve manually updated it.
On Thu, Mar 26, 2026 at 11:43 AM Adam Meikle ***@***.***>
wrote:
… I did
On Thu, Mar 26, 2026 at 6:33 AM Kiril Kartunov ***@***.***>
wrote:
> ***@***.**** requested changes on this pull request.
>
> @jpeg22 <https://github.com/jpeg22> please fix the issues outlined by
> Devin & Copilot. Do not just mark them as resolved but fix or post comments
> with reasoning.
>
> —
> Reply to this email directly, view it on GitHub
> <#1552?email_source=notifications&email_token=AXO56SJOUKTHT4O2WVTDZI34SUBRLA5CNFSNUABKM5UWIORPF5TWS5BNNB2WEL2QOVWGYUTFOF2WK43UKJSXM2LFO4XTIMBRGI4TMOJXGE32M4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJPLQOJPXEZLWNFSXOX3ON52GSZTJMNQXI2LPNZZV6Y3MNFRWW#pullrequestreview-4012969717>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AXO56SIZRQMNT45WO7OTJOT4SUBRLAVCNFSM6AAAAACW63CSJ2VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHM2DAMJSHE3DSNZRG4>
> .
> You are receiving this because you were mentioned.Message ID:
> ***@***.***>
>
|
Collaborator
Author
|
Can't add comments to individual Devin recs (No access to GitHub's topcoder platform through Devin with my handle), so here they are:
Investigating lazy loading comment and comment about the v6 path in craco |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Related JIRA Ticket:
https://topcoder.atlassian.net/browse/
What's in this PR?
These are the management pages for the AI arena problem library, tournament manager, and tournament launcher.
It looks like reports is a recent update I din't have in this branch (see last file)