Skip to content

feat(community): add ASF roster and content pilots - #34

Merged
imbajin merged 3 commits into
oink-918from
prep/oink-918-community
Sep 17, 2026
Merged

imbajin merged 3 commits into
oink-918from
prep/oink-918-community

Conversation

@imbajin

@imbajin imbajin commented Sep 17, 2026

Copy link
Copy Markdown

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

  • Base: oink-918 at 88f210ed0dbe2452a66f364f0438e4b21058a078.
  • Current head: ac96442692f900929fb3868b0fcf468b085d826e2.
  • Preserves PR-A's EN/CN LLMSFULL output and current base docs/routes; excludes handoff files and old PR-A history.
  • Related: Closes #468 (Apache issue relationship will be carried into the Apache delivery PR).

Current validation

  • 37 focused Community Python tests pass, including rendered HTML/Print/Markdown contracts.
  • 24 fixed EN/CN Lunr queries pass in real Chromium after factual Hubble and Clients descriptions were added; no title/body/boost inflation.
  • Community responsive/output matrix passes at 1440/900/390/320px, including dark mode and no visitor GitHub/Whimsy requests.
  • Standalone strict Hugo build and link checks pass.
  • Organization Build and deploy site is enabled again; current head run 35276142973 is 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.

- render deterministic ASF members across Community outputs
- preserve last-good roster with atomic validated refresh
- enrich bilingual search metadata and content components
- port product commits d88167d..bb27083 onto oink-918
- retain PR-A docs-root LLMSFULL output declarations
@coderabbitai

coderabbitai Bot commented Sep 17, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 4d735d5a-9363-4647-8fbb-8422a0d7a969

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

@github-actions

Copy link
Copy Markdown

@codecov-ai-reviewer review

@imbajin
imbajin marked this pull request as ready for review September 17, 2026 20:50
Copilot AI lite review requested due to automatic review settings September 17, 2026 20:50
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 17, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-17T21:18:14.967524Z 774389b New commits
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

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.

🟡 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.

Comment on lines 5 to +9
search_keywords:
- HugeGraph HStore
- 分布式存储
- server.port
- REST 端口
- Store REST 端口
search_boost: 1.5
search_boost: 1.6
Comment on lines 5 to +9
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
@imbajin
imbajin merged commit e6e4393 into oink-918 Sep 17, 2026
12 checks passed
@imbajin
imbajin deleted the prep/oink-918-community branch September 17, 2026 21:31
@imbajin

imbajin commented Sep 18, 2026

Copy link
Copy Markdown
Author

Follow-up is consolidated in #36 (currently open). It restores the existing EN/CN HStore search keywords (REST port / Store REST port and localized equivalents) and adds contract coverage. The original threads remain historical metadata on this merged PR; the fix is reviewed and tested in #36.

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