feat(community): add ASF roster and content pilots - #34
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
|
@codecov-ai-reviewer review |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
🟡 Changes recommended
Existing HStore search keywords are removed in both languages, causing a search regression.
Get a fresh assessment by requesting another Copilot review.
Pull request overview
Adds an offline ASF PMC/Committer roster, bilingual community rendering, search metadata, and documentation-format pilots.
Changes:
- Adds validated roster refresh, avatar handling, and artifact checks.
- Adds community member sections for HTML, print, and Markdown outputs.
- Adds bilingual search metadata and formatting examples with tests.
File summaries
| File | Description |
|---|---|
scripts/test_community_roster.py |
Roster and rendering tests |
scripts/fixtures/community_search_queries.json |
Search fixtures |
scripts/community_roster.py |
Roster generation and validation |
scripts/community_roster.md |
Roster maintenance documentation |
layouts/community/landing.md |
Markdown community rendering |
layouts/_partials/landing/sections/community-members.html |
Community section integration |
layouts/_partials/community/members.md |
Markdown member output |
layouts/_partials/community/members.html |
HTML member cards |
data/landing/community/en.yaml |
English member section |
data/landing/community/cn.yaml |
Chinese member section |
data/community/roster.json |
Checked-in ASF roster |
data/community/github-map.json |
Reviewed GitHub mappings |
assets/scss/community-members.scss |
Member card styling |
content/en/... |
English search metadata and content pilots |
content/cn/... |
Chinese search metadata and content pilots |
Review details
- Files reviewed: 39/39 changed files
- Comments generated: 2
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| search_keywords: | ||
| - HugeGraph HStore | ||
| - 分布式存储 | ||
| - server.port | ||
| - REST 端口 | ||
| - Store REST 端口 | ||
| search_boost: 1.5 | ||
| search_boost: 1.6 |
| search_keywords: | ||
| - HugeGraph HStore | ||
| - distributed storage | ||
| - server.port | ||
| - REST port | ||
| - Store REST port | ||
| search_boost: 1.5 | ||
| search_boost: 1.6 |
- surface Java client intent for client queries\n- surface graph visualization intent for Hubble queries\n- preserve existing titles and boost metadata
- accept CI Hugo build suffixes for 0.165.0 extended\n- keep the major and extended runtime contract strict\n- unblock current organization workflow validation
Before → after
Before, Community had no deterministic ASF member roster and the fixed latest search queries lacked enough factual summary context. After, the page renders the reviewed PMC/Committer roster with initials fallback, HTML/Print/Markdown parity, no visitor-side roster fetch, and bilingual search metadata/content pilots. GitHub identity mapping remains intentionally empty; no identity is inferred.
Integration boundary
oink-918at88f210ed0dbe2452a66f364f0438e4b21058a078.ac96442692f900929fb3868b0fcf468b085d826e2.Closes #468(Apache issue relationship will be carried into the Apache delivery PR).Current validation
Build and deploy siteis enabled again; current head run35276142973is in progress after the CI Hugo-version metadata compatibility fix.The integrated candidate has also passed the complete five-version production/staging aggregate, 49/49 current Chromium checks, 24 staging surface screenshots, and three independent review/re-review rounds. Those are integration evidence and will be repeated or reconciled after this PR's current CI completes.
The final Apache delivery description will include reviewed EN/CN desktop/mobile light/dark screenshots for navigation, version overflow, dynamic sidebar, Community, Download, and real Ask AI staging states. No screenshot in this organization PR is used as deployment proof.