Skip to content

Mirror installer scaffold: blocking-detection env-gate + test:run fix - #19

Merged
syntaxwanderer merged 1 commit into
masterfrom
develop
Jul 22, 2026
Merged

Mirror installer scaffold: blocking-detection env-gate + test:run fix#19
syntaxwanderer merged 1 commit into
masterfrom
develop

Conversation

@syntaxwanderer

Copy link
Copy Markdown
Member

Summary

Scaffold mirror sync from installer SSoT (update:scaffold:rebuild): env-gated Swoole blocking-I/O detection + test:run wrapper fix + regenerated scaffold-manifest.json.

Checks

  • ScaffoldSourceParityTest 3/3 green; RuntimeScaffoldConsistencyTest 10/10 green (run from semitexa.dev root).

What to Review

  • Manifest previous_sha256 growth only; no engine code changes.

🤖 Generated with Claude Code

update:scaffold:rebuild sync from installer SSoT: env-gated Swoole
blocking-I/O detection in docker-compose.yml / docker-compose.test.yml /
.env.default and the test:run wrapper fix, plus regenerated
scaffold-manifest.json (previous_sha256 grown). Keeps
ScaffoldSourceParityTest green.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@coderabbitai

coderabbitai Bot commented Jul 22, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@syntaxwanderer, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 7 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 1c1e2cd1-78e3-4065-ace5-3ebf98755bb9

📥 Commits

Reviewing files that changed from the base of the PR and between ed229b3 and 3c060ba.

📒 Files selected for processing (5)
  • resources/scaffold-manifest.json
  • resources/scaffold/.env.default
  • resources/scaffold/bin/semitexa
  • resources/scaffold/docker-compose.test.yml
  • resources/scaffold/docker-compose.yml
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch develop

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.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the scaffolded Docker/CLI runtime wiring to support opt-in Swoole blocking-I/O detection (via environment variables) and ensures bin/semitexa test:run does not bypass the phpunit service command when invoking PHPUnit, then regenerates the scaffold manifest to record the new hashes.

Changes:

  • Update scaffold docker-compose.yml and docker-compose.test.yml to run PHP with env-gated swoole.blocking_detection and swoole.blocking_threshold ini flags (Off by default).
  • Fix resources/scaffold/bin/semitexa test:run to invoke PHPUnit via php -d ... vendor/bin/phpunit so docker-compose.test.yml’s command behavior is preserved even with an explicit compose run command.
  • Add default env entries for blocking detection and update resources/scaffold-manifest.json with the new generated_at and file hashes.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
resources/scaffold/docker-compose.yml Adds env-gated Swoole blocking-I/O detection ini flags to the app service startup command.
resources/scaffold/docker-compose.test.yml Applies the same env-gated ini flags to the phpunit service command.
resources/scaffold/bin/semitexa Updates test:run to avoid overriding the phpunit service command without reapplying the ini flags.
resources/scaffold/.env.default Introduces default .env keys for blocking detection and threshold.
resources/scaffold-manifest.json Regenerates manifest metadata/hashes to reflect the scaffold updates.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@syntaxwanderer
syntaxwanderer merged commit 91956db into master Jul 22, 2026
2 checks passed
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.

2 participants