You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Adds a public Project docs index at /projects/ that catalogues live CoMPhy Lab documentation sites hosted at https://comphy-lab.org/{Repo-Name}/ (GitHub Pages for comphy-lab/* repos). This is an in-site catalogue, not a wiki engine.
Path chosen
/projects/ — free on the site, sits next to Research in the IA, and matches “project documentation sites” better than /wiki/ or /docs/.
What changed
_data/project-docs.yml — maintainable inventory (24 live sites verified HTTP 200 on 2026-09-20; excluded comphy-lab.github.io and Pages repos that 404: energy-budget-issues, When-does-a-drop-stop-bouncing).
projects/index.md + assets/css/projects-v2.css — alphabetical list with title + link + repo path chip.
Nav — header link on all layouts that carry the nav; footer Explore column; command palette “Go to Project Docs”.
Catalogue comphy-lab.org/{Repo-Name}/ Pages in _data/project-docs.yml,
render an alphabetical index, and link it from header, footer, and the
command palette. Fixes#40.
Co-authored-by: Vatsal Sanjay | CoMPhy Lab <vatsal.sanjay@comphy-lab.org>
Limit details: You’ve used all 5 included reviews currently available. Your 22 included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.
Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).
Reviewing files that changed from the base of the PR and between 78a1733 and f96db5f.
📒 Files selected for processing (6)
_data/project-docs.yml
projects/index.md
scripts/purge-website-cache.py
tests/browser-security.test.py
tests/command-data-actual.test.js
tests/purge-website-cache.test.py
No actionable comments were generated in the recent review. 🎉
ℹ️ Recent review info⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Essentials
Run ID: c87fb864-4537-43ac-ba4f-bf6b71111111
📥 Commits
Reviewing files that changed from the base of the PR and between ea50f83 and 78a1733.
📒 Files selected for processing (9)
_data/project-docs.yml
_includes/footer-v2.html
_layouts/default.html
_layouts/history.html
_layouts/join-us.html
_layouts/team.html
assets/css/projects-v2.css
assets/js/command-data.js
projects/index.md
Included review availability: 1 review is currently available. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.
📜 Recent review details⏰ Context from checks skipped due to timeout. (2)
The site now provides a /projects/ index for 25 project documentation sites. It loads project data from YAML, renders sorted entries, applies responsive styling, and adds navigation links across the site.
Changes
Project documentation
Layer / File(s)
Summary
Catalogue and project index _data/project-docs.yml, projects/index.md
Defines project repositories, titles, and notes. The index sorts the entries and renders documentation links, paths, and the live-site count.
Project index styling _layouts/default.html, assets/css/projects-v2.css
Loads page-specific CSS and styles the header, project rows, links, actions, footer, and responsive mobile layout.
Check skipped - CodeRabbit’s high-level summary is enabled.
Title check
✅ Passed
The title clearly summarises the main change: adding the /projects/ documentation index. It also identifies the affected site and linked issue.
Linked Issues check
✅ Passed
Issue #40 requests a page that provides access to CoMPhy Lab documentation, including the Viscoelastic3D site. projects/index.md creates /projects/, reads the project inventory, sorts entries, and…
Out of Scope Changes check
✅ Passed
The changes remain connected to issue #40. The data file supplies the catalogue, the CSS styles the new page, and the header, footer, history, team, join-us, and command-palette links provide navigati…
Docstring Coverage
✅ Passed
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
✨ Finishing Touches📝 Generate docstrings
Commit to this branch
Create a new PR
✨ Simplify code
Commit to this branch
Create a new PR
Comment @coderabbitai help to get the list of available commands.
Bugbot flagged that the new index skipped Cloudflare CSP matching,
release-manifest inclusion, and post-deploy cache purge. Add it to
OWNED_DIRECTORY_ROOTS / OWNED_EXACT_PATHS and cover it in policy tests.
Co-authored-by: Vatsal Sanjay | CoMPhy Lab <vatsal.sanjay@comphy-lab.org>
Cover the new projects command id, title, and target path so a future
route or ID change is caught by command-data-actual tests.
Co-authored-by: Vatsal Sanjay | CoMPhy Lab <vatsal.sanjay@comphy-lab.org>
The reason will be displayed to describe this comment to others. Learn more.
Copilot review overview
🔵 Needs a closer look
The project index has a moderate, two-vote case-sensitive sorting issue that must be fixed before approval.
Review effort: Lite Findings: None
Previously missed (1)
In code that hasn't changed since last review
Use case-insensitive sorting for the public index
projects/index.md:11
sort compares string values case-sensitively, so this does not implement the case-insensitive ordering promised in _data/project-docs.yml: for example, VE Drop Impact — Contact Line will sort before Vatsal Sanjay profile because E precedes lowercase a. Use Liquid's natural/case-insensitive sort so the public index remains alphabetically ordered.
The catalogue YAML promises case-insensitive alphabetical order, but
Liquid sort is case-sensitive. Use sort_natural so titles like
VE Drop Impact do not precede Vatsal Sanjay profile.
Co-authored-by: Vatsal Sanjay | CoMPhy Lab <vatsal.sanjay@comphy-lab.org>
@comphy-bot note: addressing Copilot body finding on catalogue sort — switched projects/index.md to sort_natural: "title" so ordering matches the case-insensitive promise in _data/project-docs.yml.
The reason will be displayed to describe this comment to others. Learn more.
Review: Add /projects/ documentation index for CoMPhy Pages (Fixes #40) (#115)
Summary
+296/−0 across 13 files. Adds a public /projects/ catalogue of live CoMPhy Pages sites, with header/footer/command-palette entry and owned-path registration so CSP, release-manifest, and purge include the new route. No P0/P1 on this head.
Peer findings on earlier commits (owned-path CSP/purge registration, command-palette regression coverage, case-insensitive catalogue sort) are fixed at 79217c43.
Findings
P2 — Duplicate generic action-link names
File: projects/index.md:43 Layer: Maintainability / a11y Impact: Every catalogue row exposes a second link whose accessible name is the identical Open docs ↗, so a screen-reader links list cannot tell which project it opens. Detail: The title link already uses {{ p.title }} and the same href="/{{ p.repo }}/". The sibling .projects-list__action link repeats that destination with generic text. Fix direction: Add aria-label="Open docs for {{ p.title }}" on the action link, or drop the duplicate action and keep the title as the sole link.
Missing Tests
None blocking. This diff already asserts the projects command-palette route, owned-path /projects/, and CSP starts_with(..., "/projects/").
Verification Steps
JEKYLL_ENV=production bundle exec jekyll build — /_site/projects/ renders 24 items in natural title order
Confirm /projects/ is in OWNED_DIRECTORY_ROOTS so scripts/browser-security.py Cloudflare expression includes starts_with(http.request.uri.path, "/projects/")
documentationWeb is a test site for the docs tooling and VatsalSy is a personal
profile page; neither is project documentation, so they do not belong on the
public /projects/ index.
# note Optional one-line context (shown muted under the title).
items:
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
comphy-bot
left a comment
Fixes #40
Summary
Adds a public Project docs index at
/projects/that catalogues live CoMPhy Lab documentation sites hosted athttps://comphy-lab.org/{Repo-Name}/(GitHub Pages forcomphy-lab/*repos). This is an in-site catalogue, not a wiki engine.Path chosen
/projects/— free on the site, sits next to Research in the IA, and matches “project documentation sites” better than/wiki/or/docs/.What changed
_data/project-docs.yml— maintainable inventory (24 live sites verified HTTP 200 on 2026-09-20; excludedcomphy-lab.github.ioand Pages repos that 404:energy-budget-issues,When-does-a-drop-stop-bouncing).projects/index.md+assets/css/projects-v2.css— alphabetical list with title + link + repo path chip.How to add a new entry
https://comphy-lab.org/{Repo-Name}/returns HTTP 200.{ repo, title, note? }to_data/project-docs.yml.Verification
JEKYLL_ENV=production bundle exec jekyll build—/_site/projects/renders 24 items.:4001/projects/returns HTTP 200; nav + ElastoFlow/Viscoelastic3Dlink present.npm test— 138 passed.Out of scope / follow-ups
/projects/to the search DB when [Enhancement]: While searching, search also the sub pages #41 lands; the command-palette nav entry is already present.Summary by CodeRabbit
/projects/, listing 25 live project documentation sites alphabetically.