Skip to content

Migrate lab website to React and add publication review pipeline - #30

Merged
advaitmb merged 10 commits into
masterfrom
codex/react-website-migration
Sep 9, 2026
Merged

advaitmb merged 10 commits into
masterfrom
codex/react-website-migration

Conversation

@advaitmb

@advaitmb advaitmb commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • replace the Jekyll site with a static Next.js/React export
  • apply the confirmed lab roster
  • add verified Scholar/OpenAlex identities and an initial review queue
  • add a repository publication-review skill and a repeatable collect/status/decide/check/publish pipeline
  • keep discovery, deployment, and automatic publication disabled pending review

Validation

  • 16 Node tests passed
  • 34 Python tests passed
  • production static build passed
  • 86 exported pages, legacy routes, local links, assets, and signup action checked

Review notes

No new publication has been approved or published. The 13 recent candidates remain pending. Production deployment remains manual.

@advaitmb

advaitmb commented Sep 9, 2026

Copy link
Copy Markdown
Contributor Author

Changes requested before merge

An independent review found five issues that should be addressed:

  1. P1 — Production deployment safety: Restrict the manual Pages deployment to the repository default branch. The current workflow deploys the selected workflow-dispatch ref, which can be an unmerged review branch.
  2. P2 — Legacy route: Preserve /allnews.html and require it in export validation.
  3. P2 — Conference metadata: Normalize OpenAlex conference proceedings as conference rather than other, with regression coverage, so filtering and BibTeX use inproceedings correctly.
  4. P2 — Join page: Render recruitment fallback or no-openings copy when all retained opportunities are closed or expired.
  5. P2 — BibTeX: Escape special characters while preserving intentional capitalization braces, with regression coverage using existing records.

Please also update the stale README and publication-policy statements about roster counts, author-registry status, and which PR contains the 12-paper batch.

All current automated checks pass, but these migration and deployment regressions should be resolved before the PR is marked ready or merged.

@advaitmb

advaitmb commented Sep 9, 2026

Copy link
Copy Markdown
Contributor Author

Addressed the independent review findings in commit 9a8871f:

  • restricted manual Pages deployment to the default branch
  • restored and validated the legacy /allnews.html route
  • classified OpenAlex conference sources as conference publications
  • restored recruitment fallback when all opportunities are closed or expired
  • escaped BibTeX special characters while preserving capitalization braces
  • refreshed the stale migration and publication-policy documentation

Regression coverage was added for each area. Local verification passed (18 JavaScript tests, 37 Python tests, 87-page export validation, and the all-closed opportunity skeleton scenario), and both GitHub check runs pass.

@advaitmb
advaitmb marked this pull request as ready for review September 9, 2026 20:02
@advaitmb
advaitmb merged commit dd04914 into master Sep 9, 2026
2 checks passed
@advaitmb
advaitmb deleted the codex/react-website-migration branch September 9, 2026 23:08
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.

1 participant