Rename EvolveX website presentation to RSIHub - #9
Conversation
|
Warning Review limit reached
Next review available in: 37 minutes You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?After more reviews become available, a review can be triggered using the 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 configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe site rebrands EvolveX as RSIHub, adds a bilingual project page, redirects the legacy route, updates homepage metadata and sitemap entries, and adds migration tests for links, assets, redirects, and legacy identifiers. ChangesRSIHub migration
Estimated code review effort: 4 (Complex) | ~45 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (5)
tests/test_rsihub_migration.py (2)
105-107: 🔒 Security & Privacy | 🔵 Trivial | 💤 Low valueTriage of the static analysis findings on this call.
The two command-injection findings are false positives. The argument list is a hardcoded literal,
shell=Falseis the default, and no external input reaches the call.Ruff
S607is valid but low risk here. The test invokesgitfromPATHby design. Add a targeted suppression so the rule stays enabled elsewhere.♻️ Suppress the partial-path rule
tracked = subprocess.check_output( - ["git", "ls-files", "-z"], cwd=ROOT + ["git", "ls-files", "-z"], cwd=ROOT # noqa: S607 ).decode("utf-8").split("\0")🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@tests/test_rsihub_migration.py` around lines 105 - 107, Add a targeted Ruff S607 suppression to the subprocess.check_output call that invokes the hardcoded git command, preserving the existing argument list and default shell=False behavior while leaving the rule enabled elsewhere.Source: Linters/SAST tools
50-59: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low valueThe asset assertion is brittle. Compare files only.
assets.iterdir()returns every entry, including subdirectories and dotfiles. If anyone adds a nested directory or a new asset, this exact-set equality fails even when the migration is correct. Ifrsihub/assetsdoes not exist,iterdir()raisesFileNotFoundErrorinstead of producing a clear assertion message.Assert that the expected files are present and restrict the listing to files.
♻️ Proposed change
- self.assertEqual({path.name for path in assets.iterdir()}, expected) + self.assertTrue(assets.is_dir(), f"missing asset directory: {assets}") + present = {path.name for path in assets.iterdir() if path.is_file()} + self.assertTrue(expected <= present, f"missing assets: {expected - present}")🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@tests/test_rsihub_migration.py` around lines 50 - 59, Update test_project_assets_moved_and_svg_titles_renamed to filter assets.iterdir() to regular files before comparing names, and assert the expected files are a subset of the discovered file names rather than requiring exact equality. Preserve a clear assertion when the assets directory is missing instead of allowing FileNotFoundError to escape.evolvex/index.html (1)
6-13: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low valueThe redirect is client-side only. Consider a server-level 301 and bilingual fallback text.
Two points on this compatibility page:
- A
meta refreshpluslocation.replaceis a client-side redirect. Search engines treat it as weaker than a 301. If the host supports redirect rules, prefer a 301 from/evolvex/to/rsihub/. If the host is GitHub Pages, the current approach is the available option.- The fallback sentence is English only. Every other page on the site provides
lang-enandlang-zhvariants. Users who reach the fallback with JavaScript disabled see English only.♻️ Bilingual fallback
- <p>RSIHub has moved to <a href="/rsihub/">its new project page</a>.</p> + <p lang="en">RSIHub has moved to <a href="/rsihub/">its new project page</a>.</p> + <p lang="zh-CN">RSIHub 已迁移至<a href="/rsihub/">新的项目页面</a>。</p>🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@evolvex/index.html` around lines 6 - 13, Configure a server-level permanent redirect from /evolvex/ to /rsihub/ when the hosting platform supports redirect rules, while retaining the existing client-side fallback for platforms such as GitHub Pages; update the fallback body in the page to provide both lang-en and lang-zh text, preserving the destination link in each language.rsihub/index.html (2)
471-489: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low valueAdd scope attributes to the benchmark table headers.
These tables are two-dimensional with a row-group axis. The column headers have no
scope, and therowspan="4"agent cells are data cells rather than headers. Screen readers cannot associate each score cell with both its method and its target agent.♻️ Proposed header semantics
- <th> + <th scope="col"> <span class="lang-en" lang="en">Target agent</span> <span class="lang-zh" lang="zh-CN">目标智能体</span> </th>- <td rowspan="4">MiniSWE Agent</td> + <th scope="rowgroup" rowspan="4">MiniSWE Agent</th>Also applies to: 557-575
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@rsihub/index.html` around lines 471 - 489, Add accessible header semantics to the benchmark tables around the visible thead and the corresponding section around the repeated table: add scope="col" to each column header, and convert the rowspan="4" target-agent cells into row headers with scope="rowgroup". Preserve the existing labels, spans, and table structure while ensuring score cells associate with both their column and agent row group.
185-194: 🚀 Performance & Scalability | 🔵 Trivial | 💤 Low valueAdd intrinsic dimensions to the SVG images.
The PNG images set
widthandheight, but the three SVG images do not. Browsers cannot reserve space before load, so the page can shift during layout. Addwidthandheightthat match each SVGviewBoxratio.♻️ Example for the lineage figure
<img src="./assets/evolve-lineage.svg" + width="1200" + height="600" alt="A baseline branches into evaluated candidates. The selected lineage rises through successive generations to a verified improvement, while unselected candidates remain visible as evidence." />Also applies to: 250-255, 456-462
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@rsihub/index.html` around lines 185 - 194, Add explicit width and height attributes to the SVG img elements, including the lineage image in the figure above and the SVG images at the other referenced locations, using dimensions that preserve each asset’s viewBox ratio. Leave the existing sources, alt text, and captions unchanged.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@tests/test_rsihub_migration.py`:
- Around line 96-119: Harden test_old_identity_is_absent_from_published_files so
the repository scan skips directories and missing working-tree paths, and
ignores any tracked file that cannot be decoded as UTF-8 instead of failing the
test. Consolidate forbidden to the minimal distinct identity values, retaining
the existing allowed test-file exception and image exclusions while preserving
assertions for readable text files.
---
Nitpick comments:
In `@evolvex/index.html`:
- Around line 6-13: Configure a server-level permanent redirect from /evolvex/
to /rsihub/ when the hosting platform supports redirect rules, while retaining
the existing client-side fallback for platforms such as GitHub Pages; update the
fallback body in the page to provide both lang-en and lang-zh text, preserving
the destination link in each language.
In `@rsihub/index.html`:
- Around line 471-489: Add accessible header semantics to the benchmark tables
around the visible thead and the corresponding section around the repeated
table: add scope="col" to each column header, and convert the rowspan="4"
target-agent cells into row headers with scope="rowgroup". Preserve the existing
labels, spans, and table structure while ensuring score cells associate with
both their column and agent row group.
- Around line 185-194: Add explicit width and height attributes to the SVG img
elements, including the lineage image in the figure above and the SVG images at
the other referenced locations, using dimensions that preserve each asset’s
viewBox ratio. Leave the existing sources, alt text, and captions unchanged.
In `@tests/test_rsihub_migration.py`:
- Around line 105-107: Add a targeted Ruff S607 suppression to the
subprocess.check_output call that invokes the hardcoded git command, preserving
the existing argument list and default shell=False behavior while leaving the
rule enabled elsewhere.
- Around line 50-59: Update test_project_assets_moved_and_svg_titles_renamed to
filter assets.iterdir() to regular files before comparing names, and assert the
expected files are a subset of the discovered file names rather than requiring
exact equality. Preserve a clear assertion when the assets directory is missing
instead of allowing FileNotFoundError to escape.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 09a2e7fe-d239-407c-af2f-50da7c18ee66
⛔ Files ignored due to path filters (5)
rsihub/assets/architecture.svgis excluded by!**/*.svgrsihub/assets/benchmark-results.svgis excluded by!**/*.svgrsihub/assets/evolve-lineage.svgis excluded by!**/*.svgrsihub/assets/paper-poster-lora-gen0.pngis excluded by!**/*.pngrsihub/assets/paper-poster-lora-gen2.pngis excluded by!**/*.png
📒 Files selected for processing (6)
.gitignoreevolvex/index.htmlindex.htmlrsihub/index.htmlsitemap.xmltests/test_rsihub_migration.py
There was a problem hiding this comment.
Pull request overview
Migrates the site’s featured project branding and canonical overview route from EvolveX to RSIHub, updating public-facing URLs/metadata while keeping the legacy /evolvex/ entry point as a compatibility redirect with a visible fallback.
Changes:
- Added a new canonical project overview page at
/rsihub/with updated SEO/social/JSON-LD metadata and refreshed asset titles. - Updated the homepage and sitemap to reference RSIHub as the canonical project and route.
- Replaced
/evolvex/content with a redirect + fallback and added regression tests to ensure the old identity/URLs are removed.
Reviewed changes
Copilot reviewed 5 out of 11 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
tests/test_rsihub_migration.py |
Adds unittest regression coverage for branding, URLs, assets, canonical metadata, sitemap, and redirect behavior. |
sitemap.xml |
Updates canonical project sitemap entry from /evolvex/ to /rsihub/ and bumps lastmod. |
rsihub/index.html |
Introduces the new RSIHub project overview page with updated links and structured metadata. |
rsihub/assets/evolve-lineage.svg |
Adds updated SVG with accessible title/desc aligned to RSIHub branding. |
rsihub/assets/benchmark-results.svg |
Renames SVG title to RSIHub for accessibility/SEO consistency. |
rsihub/assets/architecture.svg |
Renames SVG title to RSIHub for accessibility/SEO consistency. |
index.html |
Rebrands homepage metadata and featured-project links from EvolveX to RSIHub. |
evolvex/index.html |
Replaces legacy page with canonical/noindex redirect to /rsihub/ plus a visible fallback link. |
.gitignore |
Ignores Python __pycache__/ and .worktrees/ artifacts introduced by the new tests/workflow. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Summary
/evolvex/to/rsihub/simple-agent-lab/RSIHuband docs links to the case-sensitive/RSIHub//evolvex/as a compatibility redirect with a visible fallbackWhy
The project has been renamed to RSIHub. The website needs a consistent identity and links while preserving existing inbound links and search results for the previous overview URL.
This migration intentionally leaves the project's positioning, capabilities, architecture descriptions, and substantive copy unchanged.
Validation
env -u PYTHONPYCACHEPREFIX python3 -m unittest discover -s tests -v— 7 tests passedgit diff --check origin/main...HEAD— cleanSummary by CodeRabbit
New Features
Updates
Tests