Conversation
What was broken In the read-only challenge view, the review section lived inside a disabled fieldset. That made the AI review tab non-interactive and hid AI review details behind the default human tab. The view also lacked the human and AI review summary shown in work manager. Root cause The disabled fieldset wrapped the entire challenge editor, including the review tab switcher. The new platform-ui review form only rendered editable tab panels and never surfaced a view-only summary. What was changed Moved the read-only review section outside the parent disabled fieldset while keeping the review panels inside their own disabled fieldset. Kept the tab switcher interactive in view mode and added a read-only review configuration summary for human reviewers, AI workflows, review flow, and estimated cost. Updated the challenge editor docs to describe the read-only review behavior. Any added/updated tests Added a ReviewersField read-only tab regression test. Added ReviewConfigurationSummary coverage for human and AI summary rendering. Ran yarn test:no-watch --runInBand; unrelated existing failures remain in src/apps/wallet-admin/src/home/tabs/payments/PaymentsListView.spec.tsx.
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
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
1 participant
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.
What was broken
In the read-only challenge view, the review section lived inside a disabled fieldset.
That made the AI review tab non-interactive and hid AI review details behind the default human tab.
The view also lacked the human and AI review summary shown in work manager.
Root cause
The disabled fieldset wrapped the entire challenge editor, including the review tab switcher.
The new platform-ui review form only rendered editable tab panels and never surfaced a view-only summary.
What was changed
Moved the read-only review section outside the parent disabled fieldset while keeping the review panels inside their own disabled fieldset.
Kept the tab switcher interactive in view mode and added a read-only review configuration summary for human reviewers, AI workflows, review flow, and estimated cost.
Updated the challenge editor docs to describe the read-only review behavior.
Any added/updated tests
Added a ReviewersField read-only tab regression test.
Added ReviewConfigurationSummary coverage for human and AI summary rendering.
Ran yarn test:no-watch --runInBand; unrelated existing failures remain in src/apps/wallet-admin/src/home/tabs/payments/PaymentsListView.spec.tsx.