Migrate from shared/ to published penguin-libs packages#91
Open
PenguinzTech wants to merge 15 commits intomainfrom
Open
Migrate from shared/ to published penguin-libs packages#91PenguinzTech wants to merge 15 commits intomainfrom
PenguinzTech wants to merge 15 commits intomainfrom
Conversation
Replace template shared/ code (122 files, unused by services) with published PenguinTech packages: penguin-libs, penguin-licensing, penguintechinc-utils (Python), and @penguintechinc/react-libs (React). - Docs: Update shared.licensing imports to penguin_licensing equivalents - Deps: Add penguin-libs to 4 Python services and react-libs to 2 frontends - Python: Switch to sanitized get_logger(), use IsStrongPassword validator - React: Replace hand-rolled login forms with LoginPageBuilder, add AppConsoleVersion - Delete shared/ directory entirely (py_libs, go_libs, node_libs, react_libs, licensing, database) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add scripts/version/update-version.sh for managing vMajor.Minor.Patch.build format versions. Supports major/minor/patch bumps and build-only timestamp refresh (default when no argument given). Bump version from v1.0.0 to v1.1.0 for the penguin-libs migration release. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
All alerts resolved. Learn more about Socket for GitHub. This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored. |
Convert .claude/ standard files from inline content to symlinks pointing to the centralized standards repository, keeping them in sync with upstream template updates. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add Helm values-alpha/beta, Kustomize overlays (alpha/beta), manifests, and deploy-beta.sh script for consistent k8s deployment across all repos. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…alpha Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
…localhost.local Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.
Summary
shared/directory (122 files, ~22K lines) — template code unused by any service, replaced by published PenguinTech packagespenguin-libs,penguin-licensing,penguintechinc-utils(Python) and@penguintechinc/react-libs(React) across all servicesget_logger), password validation (IsStrongPassword),LoginPageBuilderfor both frontends,AppConsoleVersionfor console version displayscripts/version/update-version.shfor managingvMajor.Minor.Patch.buildversionsv1.0.0tov1.1.0Files modified (22 services/docs) + 122 deleted + 3 created
shared.licensingimports in INTEGRATIONS.md, AUTHENTICATION.md; remove shared/ CI path triggers from WORKFLOWS_EXTENDED.md@penguintechinc/react-libsto managerServer + webClient package.json; create.npmrcfor GitHub Packagesget_logger()in 4 services; replacelen(password) < 8withIsStrongPasswordvalidatorLoginPageBuilder; addAppConsoleVersionto both App.tsxshared/directory (py_libs, go_libs, node_libs, react_libs, licensing, database)scripts/version/update-version.sh, bump to v1.1.0Test plan
pip install -r requirements.txtsucceeds in each Python servicenpm installsucceeds in both frontends (requires GitHub Packages auth)LoginPageBuilderin both managerServer and webClientIsStrongPasswordAppConsoleVersionlogs version info to browser console on loadgrep -r "shared/" --include="*.py" --include="*.ts" --include="*.tsx"returns zero results./scripts/version/update-version.shcorrectly bumps build timestampdocker-compose buildsucceeds🤖 Generated with Claude Code