ReviewOS

also looking at this

pantry-pm/pantry

chore(deps): update all non-major dependencies

#185
Closed chrisbbreuer wants to merge buddy-bot/update-non-major-updates-1757488178247 into main
10 files +301 -78

Review threads live on the whole diff, not on one commit, so none are shown here - a thread's line means something in the branch's final form, and painting it into an intermediate step would put it on code it is not about.

.github/workflows/precompile-php.ymlmodified+26-2
Changes to .github/workflows/precompile-php.yml
@@ -167,7 +167,31 @@ jobs:
167167 run: |
168168 # macOS: Let Launchpad handle all dependencies
169169 echo "📦 macOS build will use Launchpad for all dependencies"
170 echo "🚀 No system package installation needed - Launchpad will provide everything"
170 echo "🚀 Installing PHP dependencies via Launchpad first..."
171
172 # Pre-install dependencies to ensure they're available
173 echo "🔧 Pre-installing dependencies via Launchpad..."
174 bun run launchpad install php --deps-only || echo "⚠️ Dependency installation had issues, continuing anyway"
175
176 # Debug: Check if ICU libraries are available
177 echo "🔍 Checking for ICU libraries..."
178 if [ -d "$HOME/.local/unicode.org" ]; then
179 echo "✅ ICU libraries found at: $HOME/.local/unicode.org"
180 ls -la "$HOME/.local/unicode.org/"
181 if [ -f "$HOME/.local/unicode.org"/*/lib/pkgconfig/icu-uc.pc ]; then
182 echo "✅ ICU pkgconfig files found"
183 find "$HOME/.local/unicode.org" -name "*.pc" -type f | head -5
184 else
185 echo "❌ ICU pkgconfig files not found"
186 fi
187 else
188 echo "❌ ICU libraries not found"
189 fi
190
191 # Debug: Check PATH and PKG_CONFIG_PATH
192 echo "🔍 Environment check:"
193 echo "PATH includes unicode.org: $(echo $PATH | grep -o unicode.org || echo 'NO')"
194 echo "PKG_CONFIG_PATH: ${PKG_CONFIG_PATH:-'NOT SET'}"
171195
172196 - name: Install build dependencies (Windows)
173197 if: matrix.platform == 'win32'
@@ -721,7 +745,7 @@ jobs:
721745 echo " - Manifest file size: $(stat -c%s manifest.json 2>/dev/null || stat -f%z manifest.json) bytes"
722746
723747 - name: Create GitHub Release
724 uses: softprops/action-gh-release@v1
748 uses: softprops/action-gh-release@v2.3.3
725749 with:
726750 tag_name: binaries-${{ github.run_number }}
727751 name: PHP Binaries - Build ${{ github.run_number }}
package.jsonmodified+12-19
Changes to package.json
@@ -17,43 +17,36 @@
1717 "keywords": ["homebrew", "pkgx", "bun", "package"],
1818 "scripts": {
1919 "build": "for dir in packages/*; do if [ -f \"$dir/package.json\" ]; then echo \"Building $dir\" && bun run --cwd $dir build; fi; done",
20 "fresh": "bunx rimraf node_modules/ bun.lock && bun i",
21 "test": "bun test",
2220 "lint": "bunx --bun eslint .",
2321 "lint:fix": "bunx --bun eslint . --fix",
24 "changelog": "bunx logsmith --verbose",
25 "changelog:generate": "bunx logsmith --output CHANGELOG.md",
26 "release": "bun run changelog:generate && bunx bumpx prompt --recursive",
27 "postinstall": "bunx git-hooks",
22 "fresh": "bunx rimraf node_modules/ bun.lock && bun i",
23 "changelog": "bunx --bun changelogen --output CHANGELOG.md",
24 "release": "bun run changelog && bunx --bun bumpp -r --all",
25 "test": "bun test",
2826 "dev:docs": "bun vitepress dev docs",
2927 "build:docs": "bun vitepress build docs",
3028 "preview:docs": "bun vitepress preview docs",
3129 "typecheck": "bun --bun tsc --noEmit"
3230 },
3331 "devDependencies": {
34 "@stacksjs/bumpx": "^0.1.84",
3532 "@stacksjs/docs": "^0.70.23",
3633 "@stacksjs/eslint-config": "^4.14.0-beta.3",
37 "@stacksjs/gitlint": "^0.1.5",
3834 "@stacksjs/launchpad": "workspace:*",
39 "@stacksjs/logsmith": "^0.1.18",
40 "@types/bun": "^1.2.22",
41 "buddy-bot": "^0.9.7",
42 "bun-git-hooks": "^0.2.19",
35 "@types/bun": "^1.2.20",
36 "buddy-bot": "^0.8.8",
37 "bumpp": "^10.2.3",
4338 "bun-plugin-dtsx": "0.21.12",
39 "changelogen": "^0.6.2",
40 "lint-staged": "^15.5.2",
41 "simple-git-hooks": "^2.13.1",
4442 "typescript": "^5.9.2"
4543 },
4644 "overrides": {
4745 "@stacksjs/dtsx": "0.9.5",
4846 "unconfig": "0.3.10"
4947 },
50 "git-hooks": {
51 "pre-commit": {
52 "staged-lint": {
53 "*.{js,ts,json,yaml,yml,md}": "bunx --bun eslint --fix"
54 }
55 },
56 "commit-msg": "bunx gitlint --edit .git/COMMIT_EDITMSG"
48 "lint-staged": {
49 "*.{js,ts}": "bunx --bun eslint . --fix"
5750 },
5851 "workspaces": [
5952 "packages/*"
packages/launchpad/package.jsonmodified+2-2
Changes to packages/launchpad/package.json
@@ -2,7 +2,7 @@
22 "name": "@stacksjs/launchpad",
33 "type": "module",
44 "version": "0.6.4",
5 "packageManager": "bun@1.2.21",
5 "packageManager": "bun",
66 "description": "Like Homebrew, but faster.",
77 "author": "Chris Breuer <chris@stacksjs.org>",
88 "license": "MIT",
@@ -64,7 +64,7 @@
6464 "typecheck": "bun --bun tsc --noEmit --skipLibCheck"
6565 },
6666 "dependencies": {
67 "bunfig": "^0.15.0",
67 "bunfig": "^0.11.1",
6868 "cac": "^6.7.14",
6969 "ts-pkgx": "^0.4.76"
7070 },