Skip to content

chore: onboard CrystalLattice to the fleet catalog - #28

Merged
veillette merged 1 commit into
mainfrom
claude/three-ui-gaps-58uiom
Aug 7, 2026
Merged

chore: onboard CrystalLattice to the fleet catalog#28
veillette merged 1 commit into
mainfrom
claude/three-ui-gaps-58uiom

Conversation

@veillette

@veillette veillette commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Description

Brings OpenPhysics/CrystalLattice — a five-screen crystal structure sim — into the fleet catalog, and regenerates the landing page from it.

  • structure/repos.json — new row, inserted alphabetically between CarnotHeatEngine and DopplerEffect
  • screenshots/CrystalLattice.png — ground-truth capture of screen 1, taken with this repo's own scripts/screenshot.mjs so it goes through the sim's ScreenshotGenerator like every other sim's
  • docs/assets/CrystalLattice.webp + docs/index.html — regenerated by scripts/generate-pages-index.sh

The docs/index.html diff is only the new card plus the two counts (17 → 18 originals, 31 → 32 total); no other card moved.

Onboarding puts the sim in the weekly compliance fan-out, so a companion change on the sim's own claude/three-ui-gaps-58uiom branch fixes the README-outline failure that surfaced — it led with ## Screens, which the six-section outline does not allow, and check-repo-compliance.sh has a specific message for exactly that case. With that branch checked out, scripts/check-repo-compliance.sh passes against it, leaving only the two standing warnings (documented hardcoded colours, and gh unavailable locally). That branch has no PR open yet, so on main as it stands today the sim would fail the compliance gate.

Type of change

  • New feature
  • Documentation update

Catalog + landing page content; no script or workflow changes.

Related issues

None.

Testing

  • scripts/check-repos-catalog.sh — Catalog OK (41 repos, schemaVersion 1.1.0), all structural invariants pass
  • npx bats tests/ — 69/69 pass, including the pages-index golden-page test
  • scripts/check-repo-compliance.sh /path/to/CrystalLattice — passes (against the branch noted above)
  • npm run lint / check / build — not applicable to this repo

How to test

  1. scripts/check-repos-catalog.sh — expect Catalog OK (41 repos, …)
  2. npx bats tests/ — expect 69 passing
  3. Open docs/index.html and confirm the Crystal Lattice card renders with its thumbnail, links to https://openphysics.github.io/CrystalLattice, and that the footer reads 32 live simulations

Checklist

  • Self-review completed
  • Documentation updated if needed
  • No unrelated changes included

Note

The card links to https://openphysics.github.io/CrystalLattice, which currently 404s — that repo's Deploy workflow has never run (zero runs; no push-event run of any workflow exists there, though PR-event runs do). Its Pages source likely still needs setting to GitHub Actions in repo settings. The card will start resolving once that is done and a deploy lands; nothing in this PR depends on it.

Adds the catalog row, the ground-truth screenshot, and the regenerated
landing page. The screenshot is a screen-1 capture through the sim's own
ScreenshotGenerator, same as the rest of the fleet.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Sa4YLoQJ4SwMSEkGxCcpg3
@cursor

cursor Bot commented Aug 7, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@veillette
veillette merged commit ba15de5 into main Aug 7, 2026
1 check passed
@veillette
veillette deleted the claude/three-ui-gaps-58uiom branch August 7, 2026 01:45
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