GH#876: Better idiot-resistance for wrong-ZIP downloads#878
Conversation
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 52 minutes and 45 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughBoth README.md and ultimate-multisite.php have been updated to address incomplete plugin installations caused by users downloading the source ZIP from GitHub's Code button instead of using the Releases page. The README now includes installation guidance, while the plugin's error notice has been enhanced with clearer messaging and actionable links. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
🔨 Build Complete - Ready for Testing!📦 Download Build Artifact (Recommended)Download the zip build, upload to WordPress and test:
🌐 Test in WordPress Playground (Very Experimental)Click the link below to instantly test this PR in your browser - no installation needed! Login credentials: |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
🧹 Nitpick comments (1)
README.md (1)
50-50: Standardize the Releases URL owner slug across docs and runtime notices.Line 50 uses
Multisite-Ultimate, while the admin notice usesUltimate-Multisite(Line 108 inultimate-multisite.php). Please pick one canonical URL to avoid redirect dependence and future link drift.Proposed doc tweak
-> **Important:** Do not use GitHub's green **"Code → Download ZIP"** button on this page. That archive is the raw source code and is missing the compiled dependencies — installing it will produce errors such as `Failed opening required .../vendor/autoload_packages.php`. Always download from the [**Releases page**](https://github.com/Multisite-Ultimate/ultimate-multisite/releases) instead. +> **Important:** Do not use GitHub's green **"Code → Download ZIP"** button on this page. That archive is the raw source code and is missing the compiled dependencies — installing it will produce errors such as `Failed opening required .../vendor/autoload_packages.php`. Always download from the [**Releases page**](https://github.com/Ultimate-Multisite/ultimate-multisite/releases) instead.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@README.md` at line 50, The README and the admin notice in ultimate-multisite.php use inconsistent GitHub owner slugs ("Multisite-Ultimate" vs "Ultimate-Multisite"); choose one canonical slug and update all occurrences to match it (update the README.md line containing the Releases page URL and the admin notice string in ultimate-multisite.php), ensuring the Releases URL is identical in both places to avoid redirects and future link drift.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@README.md`:
- Line 50: The README and the admin notice in ultimate-multisite.php use
inconsistent GitHub owner slugs ("Multisite-Ultimate" vs "Ultimate-Multisite");
choose one canonical slug and update all occurrences to match it (update the
README.md line containing the Releases page URL and the admin notice string in
ultimate-multisite.php), ensuring the Releases URL is identical in both places
to avoid redirects and future link drift.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: fea5b70c-fa02-4a08-aa7b-2dc16bbacd78
📒 Files selected for processing (2)
README.mdultimate-multisite.php
|
Performance Test Results Performance test results for 0be55b9 are in 🛎️! Note: the numbers in parentheses show the difference to the previous (baseline) test run. Differences below 2% or 0.5 in absolute values are not shown. URL:
|
- README: add prominent callout warning against the GitHub 'Code > Download ZIP' button; directs users to the Releases page instead. - Main plugin file: fix broken link in incomplete-install admin notice (was pointing at the old superdav42/wp-multisite-waas repo); update error messages to mention the Releases page and include a direct link to it alongside the developer docs; add a bold heading to the notice for immediate visual scanability.
superdav42
force-pushed
the
feature/auto-20260415-151236-gh876
branch
from
8699f08 to
e16e2fd
Compare
|
Completed via PR #878.
aidevops.sh v3.8.43 plugin for OpenCode v1.4.6 with claude-sonnet-4-6 spent 31m and 6,832 tokens on this as a headless worker. Solved in 8m. |
🔨 Build Complete - Ready for Testing!📦 Download Build Artifact (Recommended)Download the zip build, upload to WordPress and test:
🌐 Test in WordPress Playground (Very Experimental)Click the link below to instantly test this PR in your browser - no installation needed! Login credentials: |
Merge SummaryPR #878 merged (squash) — resolves #876. Changes delivered
Verification
aidevops.sh v3.8.43 plugin for OpenCode v1.4.6 with claude-sonnet-4-6 Solved in 1h 9m. |
1 participant
Summary
Resolves #876
Users who click GitHub's green "Code → Download ZIP" button get the raw source archive (no compiled vendor dependencies) and hit cryptic errors. Two improvements:
README.md
Added a prominent blockquote callout directly above the Installation section:
ultimate-multisite.php— incomplete-install admin noticesuperdav42/wp-multisite-waas(the previous repo) — updated toUltimate-Multisite/ultimate-multisite.Runtime Testing
Risk: Low — documentation change + error-message text/URL update. No logic changes. Verified:
git diffshows only the two expected files changed.--standard=WordPress): violation count reduced 52 → 48 (net improvement; pre-existing project config has broken sniff references unrelated to this PR).aidevops.sh v3.8.43 plugin for OpenCode v1.4.6 with claude-sonnet-4-6 spent 5m and 16,539 tokens on this as a headless worker.
Summary by CodeRabbit
Documentation
Bug Fixes