Skip to content

chore(deps): update phpswoole/swoole docker tag to v6 - #1641

Closed
renovate-appwrite[bot] wants to merge 1 commit into
mainfrom
renovate/phpswoole-swoole-6.x
Closed

chore(deps): update phpswoole/swoole docker tag to v6#1641
renovate-appwrite[bot] wants to merge 1 commit into
mainfrom
renovate/phpswoole-swoole-6.x

Conversation

@renovate-appwrite

Copy link
Copy Markdown
Contributor

This PR contains the following updates:

Package Type Update Change
phpswoole/swoole final major 5.1.8-php8.3-alpine6.2.1-php8.3-alpine

Configuration

📅 Schedule: (UTC)

  • Branch creation
    • At any time (no schedule defined)
  • Automerge
    • At any time (no schedule defined)

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Mend Renovate.

@renovate-appwrite renovate-appwrite Bot added the dependencies Pull requests that update a dependency file label Jul 9, 2026
@greptile-apps

greptile-apps Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This automated Renovate PR bumps the phpswoole/swoole Docker base image used by the mock server from 5.1.8-php8.3-alpine to 6.2.1-php8.3-alpine, a major version upgrade.

  • Swoole 6 introduces known breaking changes (deprecated \\Swoole\\Constant entries removed, HTTP parser replaced with llhttp, threading model requiring ZTS mode), none of which appear to be directly exercised by the mock server's code — but the utopia-php/swoole 0.8.* wrapper library compatibility with Swoole 6 should be confirmed before merging.
  • The swoole/ide-helper dev dependency in mock-server/composer.json is still pinned to 5.1.8 and was not updated alongside this image bump, leaving IDE stubs pointing at the old API surface.

Confidence Score: 4/5

The Dockerfile change itself is minimal, but this is a major Swoole version jump and the mock server's CI tests should be run and pass before merging to confirm the utopia-php/swoole wrapper and WebSocket adapter remain fully functional on Swoole 6.

The mock server code does not directly use the Swoole APIs that were removed or restructured in v6, and the single changed line is straightforward. The main open question is whether utopia-php/swoole 0.8.* and utopia-php/websocket ^1.0 are tested and compatible with Swoole 6; without passing CI evidence this remains unverified. The swoole/ide-helper pin mismatch is a development tooling gap rather than a runtime issue.

mock-server/composer.json (not in the diff) — the swoole/ide-helper dev dependency should be updated to a Swoole 6-compatible version.

Important Files Changed

Filename Overview
mock-server/Dockerfile Bumps the Swoole base image from 5.1.8 to 6.2.1 (a major version jump); the change itself is a single line, but Swoole 6 has known breaking changes (removed constants, new HTTP parser, threading model) and the companion swoole/ide-helper dev dependency in composer.json is still pinned to 5.1.8

Fix All in Claude Code Fix All in Codex

Prompt To Fix All With AI
Fix the following 1 code review issue. Work through them one at a time, proposing concise fixes.

---

### Issue 1 of 1
mock-server/Dockerfile:17
**`swoole/ide-helper` dev dependency is now mismatched**

`mock-server/composer.json` pins `"swoole/ide-helper": "5.1.8"` but the runtime image is now Swoole 6.2.1. The IDE helper provides stubs that mirror the actual Swoole extension API, so running it against the wrong major version means IDE type-checking and autocompletion will reflect the old 5.x surface (including APIs removed in 6.x). The `composer.json` `"require-dev"` entry should be updated to a Swoole 6-compatible IDE helper version alongside this image bump.

Reviews (1): Last reviewed commit: "chore(deps): update phpswoole/swoole doc..." | Re-trigger Greptile

Comment thread mock-server/Dockerfile
`if [ "$TESTING" != "true" ]; then echo "--no-dev"; fi`

FROM phpswoole/swoole:5.1.8-php8.3-alpine as final
FROM phpswoole/swoole:6.2.1-php8.3-alpine as final

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P2 swoole/ide-helper dev dependency is now mismatched

mock-server/composer.json pins "swoole/ide-helper": "5.1.8" but the runtime image is now Swoole 6.2.1. The IDE helper provides stubs that mirror the actual Swoole extension API, so running it against the wrong major version means IDE type-checking and autocompletion will reflect the old 5.x surface (including APIs removed in 6.x). The composer.json "require-dev" entry should be updated to a Swoole 6-compatible IDE helper version alongside this image bump.

Prompt To Fix With AI
This is a comment left during a code review.
Path: mock-server/Dockerfile
Line: 17

Comment:
**`swoole/ide-helper` dev dependency is now mismatched**

`mock-server/composer.json` pins `"swoole/ide-helper": "5.1.8"` but the runtime image is now Swoole 6.2.1. The IDE helper provides stubs that mirror the actual Swoole extension API, so running it against the wrong major version means IDE type-checking and autocompletion will reflect the old 5.x surface (including APIs removed in 6.x). The `composer.json` `"require-dev"` entry should be updated to a Swoole 6-compatible IDE helper version alongside this image bump.

How can I resolve this? If you propose a fix, please make it concise.

Fix in Claude Code Fix in Codex

@renovate-appwrite

Copy link
Copy Markdown
Contributor Author

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.

@ChiragAgg5k

Copy link
Copy Markdown
Member

Closing: CI is failing / known incompatible with current SDK templates and tooling. Superseded by Renovate config guards on cursor/renovate-dashboard-debug.

@ChiragAgg5k ChiragAgg5k closed this Jul 9, 2026
@ChiragAgg5k
ChiragAgg5k deleted the renovate/phpswoole-swoole-6.x branch July 9, 2026 09:58
@renovate-appwrite

Copy link
Copy Markdown
Contributor Author

Renovate Ignore Notification

Because you closed this PR without merging, Renovate will ignore this update. You will not get PRs for any future 6.x releases. But if you manually upgrade to 6.x then Renovate will re-enable minor and patch updates automatically.

If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant