Skip to content

[Fix] Track frontend lockfile + migrate to public @penguintechinc/react-libs - #159

Open
PenguinzTech wants to merge 3 commits into
mainfrom
fix/webui-frontend-lockfile
Open

[Fix] Track frontend lockfile + migrate to public @penguintechinc/react-libs#159
PenguinzTech wants to merge 3 commits into
mainfrom
fix/webui-frontend-lockfile

Conversation

@PenguinzTech

Copy link
Copy Markdown
Contributor

Summary

Two frontend fixes for the hub module web UI (admin/hub_module/frontend):

  1. Track the frontend lockfile (fix(frontend): regenerate package-lock.json and enable its tracking) — package-lock.json was previously gitignored/untracked. This regenerates it and updates .gitignore so the lockfile is committed, giving reproducible, hash-verified installs (npm ci) in CI and Docker builds.
  2. Migrate to the public shared library (feat(frontend): migrate @penguin/react_libs to public @penguintechinc/react-libs) — swaps the old @penguin/react_libs dependency for the public npm package @penguintechinc/react-libs (v1.3.4) and updates all component imports across the admin/superadmin/public pages.

Changes

  • admin/hub_module/frontend/package.json — dependency swapped to public @penguintechinc/react-libs
  • admin/hub_module/frontend/package-lock.json — regenerated, now tracked
  • .gitignore — stop ignoring the frontend lockfile
  • Import updates across ~14 frontend .jsx pages/components

Why this matters

The desktop client depends on this landing — it consumes the same public @penguintechinc/react-libs package, and a tracked, reproducible lockfile is a prerequisite for its build. This unblocks that downstream work.

Notes

  • Frontend-only change; no backend/API surface touched.
  • No secrets introduced.

PenguinzTech and others added 2 commits July 27, 2026 18:45
The frontend's package-lock.json was out of sync with package.json
(@penguin/react_libs version mismatch: lock had 1.0.0, manifest
required 1.1.0). Regenerated lockfile via npm install and removed
package-lock.json from .gitignore to enable proper tracking per
frontend-react.md standards (exact npm versions must be committed).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…/react-libs

Replaced local file:// dependency (@penguin/react_libs from shared/react_libs)
with the official published package @penguintechinc/react-libs@1.3.4 on public
npm. Updated all 15 import sites across the frontend (FormModalBuilder usage).

No API divergence — public package's FormModalBuilder export is compatible.
Both npm ci and production build pass with no changes to component usage.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@PenguinzTech PenguinzTech self-assigned this Jul 28, 2026
@PenguinzTech
PenguinzTech requested a review from Tim-Machine July 28, 2026 01:29
- vite 6.2.6 → 6.4.3: fixes 8 high-severity path traversal/file read vulnerabilities
  (GHSA-859w-5945-r5v3, GHSA-g4jq-h2w9-997c, GHSA-jqfw-vq24-v9c3,
   GHSA-93m4-6634-74q7, GHSA-4w7w-66w2-5vf9, GHSA-p9ff-h696-f583,
   GHSA-v6wh-96g9-6wx3, GHSA-fx2h-pf6j-xcff)

- axios 1.7.9 → 1.18.1: fixes 5 high-severity SSRF/DoS/auth bypass vulnerabilities
  (GHSA-jr5f-v2jv-69x6, GHSA-4hjh-wcwx-xvwj, GHSA-3p68-rc4w-qgx5,
   GHSA-w9j2-pvgh-6h63, GHSA-pmwg-cvhr-8vh7)

- postcss 8.4.49 → 8.5.25: security updates

- Pin all dependencies to exact versions (no ^/~) per frontend-react.md

npm audit after: 12 high-severity remaining (mostly react-router 7.x CVEs across all
versions 7.0.0-7.18.2; no stable fix available without major version bump).
npm ci: OK. npm run build: OK. All production build passes.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@socket-security

Copy link
Copy Markdown

Caution

Review the following alerts detected in dependencies.

According to your organization's Security Policy, you must resolve all "Block" alerts before proceeding. It is recommended to resolve "Warn" alerts too. Learn more about Socket for GitHub.

Action Severity Alert  (click "▶" to expand/collapse)
Block High
Obfuscated code: npm eslint-plugin-react is 90.0% likely obfuscated

Confidence: 0.90

Location: Package overview

From: admin/hub_module/frontend/package-lock.jsonnpm/eslint-plugin-react@7.37.3

ℹ Read more on: This package | This alert | What is obfuscated code?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev.

Suggestion: Packages should not obfuscate their code. Consider not using packages with obfuscated code.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/eslint-plugin-react@7.37.3. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

Warn Low
Unmaintained: npm normalize-range was last published 11 years ago

Last Publish: 9/13/2015, 5:20:04 AM

From: admin/hub_module/frontend/package-lock.jsonnpm/autoprefixer@10.4.20npm/normalize-range@0.1.2

ℹ Read more on: This package | This alert | What are unmaintained packages?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev.

Suggestion: Package should publish periodic maintenance releases if they are maintained, or deprecate if they have no intention in further maintenance.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/normalize-range@0.1.2. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

View full report

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