Skip to content

feat: rename the package to roadrunner/worker - #53

Merged
roxblnfk merged 1 commit into
3.xfrom
chore/rename-package
Oct 10, 2026
Merged

roxblnfk merged 1 commit into
3.xfrom
chore/rename-package

Conversation

@roxblnfk

@roxblnfk roxblnfk commented Oct 10, 2026 •

Copy link
Copy Markdown
Member

What was changed

  • The package is renamed to roadrunner/worker and replaces spiral/roadrunner-worker with self.version; namespaces stay unchanged.
  • Depends on roadrunner/goridge ^4.5; suggest points to roadrunner/cli.
  • Version::current() checks roadrunner/worker before the old name: the replaced entry in InstalledVersions has no pretty version.
  • composer.json metadata polished (description, keywords, authors, homepage, docs/source links); README and release-please config use the new name.

Why?

The roadrunner-php packages move to the roadrunner/ vendor; roadrunner/goridge 4.5.0 is already released the same way. The new name has to be registered on Packagist before release.

Checklist

  • Tested
    • composer validate --strict, testo (90 tests) and psalm pass locally
    • Unit tests added

Summary by CodeRabbit

  • Updates
    • The package is now listed as roadrunner/worker across installation guidance and package metadata. The previous package name is recognized as replaced.
    • The Goridge requirement is updated to roadrunner/goridge ^4.5, and the suggested CLI package is now roadrunner/cli.
    • Version detection now supports the new package name.

build: depend on roadrunner/* packages
chore: polish composer.json metadata

The package keeps its version line and replaces spiral/roadrunner-worker with self.version, so packages that still require spiral/roadrunner-worker are satisfied by roadrunner/worker and both names never get installed together. Version::current() looks up roadrunner/worker before the old name, because the replaced entry carries no pretty version. Namespaces stay unchanged.

Assisted-By: Claude Opus 5.5 <noreply@anthropic.com>
@roxblnfk
roxblnfk requested a review from a team October 10, 2026 12:29
@coderabbitai

coderabbitai Bot commented Oct 10, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 967ffce4-444b-415d-88da-72131d1d3a89

📥 Commits

Reviewing files that changed from the base of the PR and between cb816a5 and 8a9b21f.


📒 Files selected for processing (5)
  • .github/.release-please-config.json
  • README.md
  • composer.json
  • src/Version.php
  • tests/Unit/VersionTest.php

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.



📝 Walkthrough

Walkthrough

The package metadata, Release Please configuration, README installation references, and CLI references now use roadrunner names. Composer declares the former worker package as replaced and requires roadrunner/goridge ^4.5. Version detection and tests now include roadrunner/worker.

Changes

Worker Package Rename

Layer / File(s) Summary
Package metadata and references
composer.json, .github/.release-please-config.json, README.md
Composer metadata and package references use the roadrunner names. Composer declares the former worker package as replaced, updates the Goridge requirement to roadrunner/goridge ^4.5, and suggests roadrunner/cli.
Version detection and tests
src/Version.php, tests/Unit/VersionTest.php
Version::PACKAGE_NAMES includes roadrunner/worker. Tests cover that name alone and alongside the replaced package. The fallback test uses roadrunner/http.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Feature

Suggested reviewers: butschster


Merge Risk: ⚪ Minimal · up to 8a9b2

The package rename and dependency update show no concrete merge-blocking issue in the reviewed changes.

Pre-merge checks | Passed 4 | Failed 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 2 files. (3 skipped: 3 … Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely describes the main change: renaming the Composer package to roadrunner/worker.
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.

Full details: Docstring Coverage

Explanation

Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 2 files. (3 skipped: 3 unsupported.)


  • Fix all pre-merge checks with AI
✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Commit to this branch
  • Create a new PR


🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@roxblnfk
roxblnfk merged commit 14aba9c into 3.x Oct 10, 2026
15 checks passed
@roxblnfk
roxblnfk deleted the chore/rename-package branch October 10, 2026 12:50
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