Skip to content

chore: update dependencies - #112

Open
abnegate wants to merge 1 commit into
mainfrom
automation/dependencies-36373227308-1
Open

abnegate wants to merge 1 commit into
mainfrom
automation/dependencies-36373227308-1

Conversation

@abnegate

Copy link
Copy Markdown
Member

Automated weekly dependency update.

Dependency update report

Dependency Current Selected Reference Result
php:8.5-alpine sha256:4992c6fda82eadfb3b22dca3929188dc5831e9f44a3d42b3c8d36a460b4d5a80 sha256:93684051146ec037620855feb77f278090bde45ddc030801cd3f2a7685bc4deb — Updated
brotli 0.21.0 0.21.0 ced23f5b6f52 Current
imagick 3.8.1 3.8.1 70087bab33ea Current
lz4 0.7.1 0.7.1 065a57d8fe23 Current
maxminddb v1.14.0 v1.14.0 f3c92f68b3be Current
mongodb 2.5.2 2.5.3 3551aa91ec76 Updated
protobuf 5.36.2 5.36.2 f0a17c67de29 Current
redis 6.3.0 6.3.0 df4fab2de7fc Current
scrypt 2.0.2 2.0.2 5a14bc766423 Current
snappy 0.2.3 0.2.3 d31b77d63955 Current
swoole v6.2.2 v6.2.3 8b20cd39f8c1 Updated
xdebug 3.5.3 3.5.3 127bbcb98040 Current
yaml 2.3.0 2.3.0 c1f0d8ba5ef3 Current
zstd 0.18.0 0.18.0 c2593a4ce245 Current

Updates: 3

Dockerfile pins were updated.

@greptile-apps

greptile-apps Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 4/5

[High risk] Updates base container image and PHP extension versions.

The PR is not ready to merge because the Swoole update makes the container structure check fail.

Fix All in Claude CodeFindings

  1. P1 Swoole bump breaks image test ▶
Fix with agent prompt
### Issue 1
Dockerfile:30
The image now builds Swoole v6.2.3, but `tests.yaml` still requires `php --re swoole` to report version 6.2.2. The container structure workflow builds this image and runs that test, so a successful build will still fail the check and block the dependency update. This assertion mirrors a version pin rather than testing Swoole behavior; replace it with an observable compatibility check instead of changing the expected string to 6.2.3.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Summary

The PR updates the pinned PHP Alpine base image and the MongoDB and Swoole extension sources.

  • The Swoole bump conflicts with an unchanged image-test assertion, preventing the structure check from passing after a successful build.

Reviews (1) · Last reviewed commit: "chore: update dependencies"

Comment thread Dockerfile
PHP_SNAPPY_COMMIT="d31b77d63955dbbf1a302ca13c4795292f91d140" \
PHP_SWOOLE_VERSION="v6.2.2" \
PHP_SWOOLE_COMMIT="8e8c49915ca5f9dcb9ee654f9e336a9c88dd375e" \
PHP_SWOOLE_VERSION="v6.2.3" \

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Swoole bump breaks image test The image now builds Swoole v6.2.3, but tests.yaml still requires php --re swoole to report version 6.2.2. The container structure workflow builds this image and runs that test, so a successful build will still fail the check and block the dependency update. This assertion mirrors a version pin rather than testing Swoole behavior; replace it with an observable compatibility check instead of changing the expected string to 6.2.3.

Prompt To Fix With AI
This is a comment left during a code review.
Path: Dockerfile
Line: 30

Comment:
**Swoole bump breaks image test** The image now builds Swoole v6.2.3, but `tests.yaml` still requires `php --re swoole` to report version 6.2.2. The container structure workflow builds this image and runs that test, so a successful build will still fail the check and block the dependency update. This assertion mirrors a version pin rather than testing Swoole behavior; replace it with an observable compatibility check instead of changing the expected string to 6.2.3.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Fix in Claude Code Fix in Codex

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