Skip to content

PNPM Audit#2283

Merged
prxt6529 merged 14 commits intomainfrom
pnpm-audit
Apr 17, 2026
Merged

PNPM Audit#2283
prxt6529 merged 14 commits intomainfrom
pnpm-audit

Conversation

@prxt6529
Copy link
Copy Markdown
Collaborator

@prxt6529 prxt6529 commented Apr 16, 2026

Summary by CodeRabbit

  • Chores

    • Updated many project dependencies to newer versions, moved a codegen tool to devDependencies, and added workspace-level overrides to pin transitive dependency versions; removed the manifest-level overrides block.
  • Tests

    • Made Jest explicitly ignore build/artifact folders during test discovery and added/updated unit tests for component rendering.
  • Style / UI

    • Adjusted layout/spacing and accordion/button styles, refreshed tooltip layout, replaced an inline owner display with avatar + profile link, and enabled locale-aware formatting for displayed card counts.

Signed-off-by: prxt6529 <prxt@6529.io>
Signed-off-by: prxt6529 <prxt@6529.io>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 16, 2026

Warning

Rate limit exceeded

@prxt6529 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 38 minutes and 19 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 38 minutes and 19 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 80b73c5a-0a9b-410c-9536-2f38256c8651

📥 Commits

Reviewing files that changed from the base of the PR and between 615823e and aa923b1.

📒 Files selected for processing (5)
  • app/layout.tsx
  • components/delegation/Delegation.module.scss
  • components/user/subscriptions/UserPageSubscriptionsBalance.tsx
  • standalone/standalone-memes-mint/src/app/layout.tsx
  • styles/swiper.scss
📝 Walkthrough

Walkthrough

This PR adds modulePathIgnorePatterns to Jest, bumps multiple dependencies and devDependencies (including Next.js, Jest toolchain, Axios, Lodash, FontAwesome), moves the OpenAPI generator CLI to devDependencies, adds pnpm workspace overrides, and includes UI, SCSS, and test updates.

Changes

Cohort / File(s) Summary
Jest configuration
jest.config.js
Added modulePathIgnorePatterns: ["<rootDir>/.next/"] to the exported Jest config.
Package manifest
package.json
Bumped many deps/devDeps (e.g., next 16.2.1→16.2.3, axios 1.13.6→1.15.0, lodash 4.17.23→4.18.1, @fortawesome/react-fontawesome 0.2.6→3.3.0). Moved @openapitools/openapi-generator-cli to devDependencies and removed the overrides block.
pnpm workspace
pnpm-workspace.yaml
Added top-level overrides to pin/substitute transitive dependency versions (react/react-dom, minimatch→brace-expansion, get-uri→basic-ftp, react-toggle react/react-dom, etc.).
UI / Delegation component
components/delegation/CollectionDelegation.tsx, components/delegation/Delegation.module.scss
Adjusted heading/row spacing and back-button markup; added .collectionDelegationBackRow and .collectionDelegationTitleRow; updated accordion CSS variables, background color, .backBtn layout, and padding tweaks.
NextGen token image
components/nextGen/collections/nextgenToken/NextGenTokenImage.tsx
Replaced inline owner info with ProfileAvatar + Link, computed handleOrWallet/profileHref/initial, converted tooltip layout from react-bootstrap grid to flex/Tailwind-style, removed react-bootstrap imports.
User subscriptions
components/user/subscriptions/UserPageSubscriptionsBalance.tsx
Format cards count with .toLocaleString() when balance > 0.
Swiper styles
styles/swiper.scss
Removed custom arrow glyphs and hidden ::before/::after pseudo-elements for navigation buttons.
Tests / Mocks
__tests__/components/CollectionDelegationComponent.test.tsx, __tests__/components/nextGen/NextGenTokenImage.test.tsx
Added DOM-ordering test for collection delegation header/button; updated Jest mock for @/components/user/utils/UserCICAndLevel to export UserCICAndLevelSize with SMALL and MEDIUM.

Sequence Diagram(s)

(Skipped — changes are configuration, dependency bumps, styling, tests, and small UI refactors without new multi-component sequential control flow.)

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

  • wip #2193: Adds pnpm workspace overrides and pins transitive dependency versions overlapping the pnpm-workspace.yaml changes.
  • Update React and Nextjs #1647: Updates package.json dependency versions including Next.js—related to the dependency bumps here.
  • icons ui changes #2131: Related changes to UserCICAndLevel/NextGen token rendering; ties to the Jest mock and NextGenTokenImage edits.

Suggested reviewers

  • simo6529
  • ragnep

Poem

🐰 I nudged the config, quiet and spry,

Ignoring .next/ as builds hop by,
Packages leapt up, pinned paths set just right,
Styles and tests tidy in the soft moonlight,
A small rabbit cheer for this version-bump night 🥕✨

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check ❓ Inconclusive The title 'PNPM Audit' is vague and does not clearly convey the specific changes made in this pull request. Consider using a more descriptive title that summarizes the main changes, such as 'Update dependencies and workspace configuration' or 'Bump Next.js and Jest to v30, update pnpm overrides'.
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch pnpm-audit

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

Signed-off-by: prxt6529 <prxt@6529.io>
Signed-off-by: prxt6529 <prxt@6529.io>
Signed-off-by: prxt6529 <prxt@6529.io>
Signed-off-by: prxt6529 <prxt@6529.io>
Signed-off-by: prxt6529 <prxt@6529.io>
Signed-off-by: prxt6529 <prxt@6529.io>
Signed-off-by: prxt6529 <prxt@6529.io>
Signed-off-by: prxt6529 <prxt@6529.io>
Signed-off-by: prxt6529 <prxt@6529.io>
Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
components/delegation/Delegation.module.scss (1)

48-49: Minor: consider reusing $almost-very-dark-grey for consistency.

rgb(30, 30, 30) is hardcoded for --bs-accordion-btn-bg and --bs-accordion-active-bg, but per styles/variables.scss this value is already defined as $almost-very-dark-grey. Using the variable keeps the palette centralized.

♻️ Proposed change
-  --bs-accordion-btn-bg: rgb(30, 30, 30);
-  --bs-accordion-active-bg: rgb(30, 30, 30);
+  --bs-accordion-btn-bg: #{variables.$almost-very-dark-grey};
+  --bs-accordion-active-bg: #{variables.$almost-very-dark-grey};
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@components/delegation/Delegation.module.scss` around lines 48 - 49, Replace
the hardcoded rgb(30, 30, 30) values for the CSS custom properties
--bs-accordion-btn-bg and --bs-accordion-active-bg with the SCSS variable
$almost-very-dark-grey to centralize the palette; locate the assignments to
--bs-accordion-btn-bg and --bs-accordion-active-bg in Delegation.module.scss and
swap the literal rgb(...) for the SCSS variable so the module uses
$almost-very-dark-grey.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@components/delegation/Delegation.module.scss`:
- Around line 48-49: Replace the hardcoded rgb(30, 30, 30) values for the CSS
custom properties --bs-accordion-btn-bg and --bs-accordion-active-bg with the
SCSS variable $almost-very-dark-grey to centralize the palette; locate the
assignments to --bs-accordion-btn-bg and --bs-accordion-active-bg in
Delegation.module.scss and swap the literal rgb(...) for the SCSS variable so
the module uses $almost-very-dark-grey.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: ea68b02f-d90c-4927-84ef-c24b4a1f5502

📥 Commits

Reviewing files that changed from the base of the PR and between ea66ef5 and 615823e.

📒 Files selected for processing (7)
  • __tests__/components/CollectionDelegationComponent.test.tsx
  • __tests__/components/nextGen/NextGenTokenImage.test.tsx
  • components/delegation/CollectionDelegation.tsx
  • components/delegation/Delegation.module.scss
  • components/nextGen/collections/nextgenToken/NextGenTokenImage.tsx
  • components/user/subscriptions/UserPageSubscriptionsBalance.tsx
  • styles/swiper.scss
✅ Files skipped from review due to trivial changes (3)
  • tests/components/CollectionDelegationComponent.test.tsx
  • components/user/subscriptions/UserPageSubscriptionsBalance.tsx
  • components/delegation/CollectionDelegation.tsx

Signed-off-by: prxt6529 <prxt@6529.io>
Signed-off-by: prxt6529 <prxt@6529.io>
Signed-off-by: prxt6529 <prxt@6529.io>
@sonarqubecloud
Copy link
Copy Markdown

@prxt6529 prxt6529 merged commit d8edcc2 into main Apr 17, 2026
8 checks passed
@coderabbitai coderabbitai Bot mentioned this pull request Apr 30, 2026
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.

2 participants