Skip to content

Update react monorepo - #4792

Open
red-hat-konflux[bot] wants to merge 1 commit into
masterfrom
konflux/mintmaker/master/react-monorepo
Open

Update react monorepo#4792
red-hat-konflux[bot] wants to merge 1 commit into
masterfrom
konflux/mintmaker/master/react-monorepo

Conversation

@red-hat-konflux

@red-hat-konflux red-hat-konflux Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

This PR contains the following updates:

Package Change Age Confidence
@types/react (source) 19.2.1719.2.18 age confidence
@types/react-dom (source) 19.2.319.2.4 age confidence

Configuration

📅 Schedule: (UTC)

  • Branch creation
    • At any time (no schedule defined)
  • Automerge
    • At any time (no schedule defined)

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.


  • If you want to rebase/retry this PR, check this box

To execute skipped test pipelines write comment /ok-to-test.


Documentation

Find out how to configure dependency updates in MintMaker documentation or see all available configuration options in Renovate documentation.

Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

The web application package manifest updates @types/react from 19.2.17 to 19.2.18 and @types/react-dom from 19.2.3 to 19.2.4.

Changes

React type updates

Layer / File(s) Summary
Update React type versions
admin-console/src/main/webapp/package.json
Updates the @types/react and @types/react-dom devDependency versions.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Suggested reviewers: g-duval

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly indicates a React-related dependency update, which matches the pull request objectives.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch konflux/mintmaker/master/react-monorepo

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@admin-console/src/main/webapp/package.json`:
- Around line 30-31: Regenerate admin-console/src/main/webapp/yarn.lock from the
updated `@types/react` and `@types/react-dom` versions declared in package.json,
ensuring their resolved versions and integrity checksums become 19.2.18 and
19.2.4 respectively. Commit the resulting lockfile changes without modifying
unrelated dependencies.
🪄 Autofix

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: fb59012c-91fe-4e70-ad4b-02ed254c1bf3

📥 Commits

Reviewing files that changed from the base of the PR and between 65529af and e00e63c.

📒 Files selected for processing (1)
  • admin-console/src/main/webapp/package.json

Comment on lines +30 to +31
"@types/react": "19.2.18",
"@types/react-dom": "19.2.4",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win

Update the lockfile with these dependency changes.

admin-console/src/main/webapp/yarn.lock still resolves @types/react to 19.2.17 and @types/react-dom to 19.2.3. This leaves the manifest and lockfile inconsistent. Yarn immutable installs can fail, or installs can continue using the old type packages.

Regenerate the lockfile and commit the updated entries and checksums.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@admin-console/src/main/webapp/package.json` around lines 30 - 31, Regenerate
admin-console/src/main/webapp/yarn.lock from the updated `@types/react` and
`@types/react-dom` versions declared in package.json, ensuring their resolved
versions and integrity checksums become 19.2.18 and 19.2.4 respectively. Commit
the resulting lockfile changes without modifying unrelated dependencies.

@larrymou9

Copy link
Copy Markdown
Collaborator

Performance Tests failed

PR #4792
Branch konflux/mintmaker/master/react-monorepo
SHA e00e63c598b8c04983174d9fa0d69355b8d84534
Build #106
Image tested quay.io/redhat-user-workloads/hcc-integrations-tenant/notifications/notifications-backend:on-pr-e00e63c598b8c04983174d9fa0d69355b8d84534

Tests run: InsightsNotificationswebhook_runner · InsightsNotificationsgw_runner · InsightsNotificationsemail_runner

📊 Performance Results

InsightsNotificationsemail_runner

Metric Result Value Min Max
notifications-engine-service.restarts.sum ✅ PASS 0.000 0.000 0.000
notifications-connector-email-service.restarts.sum ✅ PASS 0.000 0.000 0.000
SUMMARY.total_rps ✅ PASS 14.083 13.966 14.498
SUMMARY.median_response_time ✅ PASS 10.000 10.000 12.000
POST_notifications.avg_response_time ✅ PASS 28.566 25.972 30.650
SUMMARY.num_failures ✅ PASS 0.000 0.000 0.000
SUMMARY.fail_ratio ✅ PASS 0.000 0.000 0.000

InsightsNotificationswebhook_runner

Metric Result Value Min Max
notifications-engine-service.restarts.sum ✅ PASS 0.000 0.000 0.000
notifications-connector-webhook-service.restarts.sum ✅ PASS 0.000 0.000 0.000
results.created_at.duration_stats.mean ❌ FAIL 173.282 141.021 154.384
results.created_at.duration_stats.median ❌ FAIL 192.884 152.376 170.057
results.created_at.rps_stats.mean ❌ FAIL 3.155 3.240 3.678
results.created_at.rps_stats.median ❌ FAIL 2.613 2.900 3.200

InsightsNotificationsgw_runner

Metric Result Value Min Max
notifications-engine-service.restarts.sum ✅ PASS 0.000 0.000 0.000
notifications-gw-service.restarts.sum ✅ PASS 0.000 0.000 0.000
SUMMARY.total_rps ✅ PASS 14.356 14.105 14.599
SUMMARY.median_response_time ✅ PASS 10.000 10.000 12.000
POST_notifications.avg_response_time ✅ PASS 26.703 24.871 29.947
SUMMARY.num_failures ✅ PASS 0.000 0.000 0.000
SUMMARY.fail_ratio ✅ PASS 0.000 0.000 0.000

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant