Conversation
Realm Server Test Results 1 files ±0 1 suites ±0 14m 12s ⏱️ +5s Results for commit 9497d9e. ± Comparison against base commit e549179. This pull request removes 1 and adds 2 tests. Note that renamed tests count towards both.♻️ This comment has been updated with latest results. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 668297e252
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
Pull request overview
Adds a CI-produced realm index cache artifact and an opt-in dev workflow (INDEX_CACHE=true) to import that cached index into an environment-mode database to avoid slow first-time indexing.
Changes:
- Introduces a new CI job that boots realms, waits for readiness, and uploads a gzipped SQL dump of key index tables as an artifact.
- Adds a dev-mode import script to download the latest cache artifact from
main, optionally remap URLs forBOXEL_ENVIRONMENT, and import into the local DB. - Documents the new
INDEX_CACHEworkflow in the README.
Reviewed changes
Copilot reviewed 6 out of 7 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| scripts/import-cached-index.sh | New script to download CI artifact and import cached index into local Postgres (with URL remapping in env mode). |
| mise-tasks/lib/dev-common.sh | Hooks cache import into environment-mode dev bootstrap when INDEX_CACHE=true. |
| mise-tasks/ci/cache-index | New CI task to start services, wait for readiness, and dump index-related tables. |
| .github/workflows/ci.yaml | Adds cache-index job on main to run the cache-index task and upload artifacts/logs. |
| packages/realm-server/package.json | Adds wait-on dependency used by the new CI task. |
| pnpm-lock.yaml | Lockfile updates for wait-on and transitive dependencies. |
| README.md | Documents INDEX_CACHE usage and behavior. |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This reverts commit 5ab4d6a.
This adds a CI job on
mainthat indexes built-in realms and stores an SQL dump of them for use in development. AddingINDEX_CACHEwithmise run devor…dev-allopts into this feature. This facilitates reviewing PRs locally that need the catalog or other large realms, which otherwise take 20+ minutes to index for me.Here you see me running off an earlier commit (
5ab4d6a20, if you want to try it yourself) where the script to download the cache would fetch the artifact from this branch instead ofmain, where it doesn’t yet exist:screencast.2026-04-10.11-30-18.mp4
I had the
hostapplication running in another tab to remove the build time from the video.