Skip to content

Commit cbe9d5d

Browse files
knhn1004claude
andcommitted
fix(ci): drop committed index.json so the freshness check stops fighting itself
The build stamps index.json with a fresh `generated_at` every run, so the "is the file up to date" check could never pass. The pages workflow already rebuilds the index before uploading the site artifact, so the committed copy was redundant anyway. Now: index is generated at deploy time only, CI just validates schema + smoke-tests the build. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 4e3d789 commit cbe9d5d

3 files changed

Lines changed: 6 additions & 60 deletions

File tree

.github/workflows/ci.yml

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -18,12 +18,6 @@ jobs:
1818
- name: Validate rules against schema
1919
working-directory: tools
2020
run: bun run validate
21-
- name: Build index and check it is up to date
21+
- name: Build index (smoke test only)
2222
working-directory: tools
23-
run: |
24-
bun run build-index
25-
if ! git diff --quiet -- ../site/data/index.json; then
26-
echo "::error::site/data/index.json is stale. Run 'bun run build-index' in tools/ and commit the result."
27-
git --no-pager diff -- ../site/data/index.json
28-
exit 1
29-
fi
23+
run: bun run build-index

.gitignore

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,3 +2,7 @@ node_modules/
22
.DS_Store
33
*.log
44
dist/
5+
6+
# Generated at deploy time by the pages workflow. The Pages site fetches
7+
# data/index.json from the deployed artifact, not from the repo.
8+
site/data/index.json

site/data/index.json

Lines changed: 0 additions & 52 deletions
This file was deleted.

0 commit comments

Comments
 (0)