source-academy / frontend / 29313684357

42%

Build Type

push

github

Committed by web-flow

Commit Message

Fetch searchData.json instead of rewritedSearchData.json for SICP JS/SICPy search (#4095)

* Fetch searchData.json instead of rewritedSearchData.json

source-academy/sicp renamed its generated search-data file to fix an
embarrassing typo (rewritedSearchData.json -> searchData.json), see
source-academy/sicp#1301. Updated both fetch URLs (SICP JS and
SICPy) to match, and renamed the rewritedSearchData identifier to
searchData throughout the autocomplete search code for consistency.

Fixes #4094

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

* Guard against missing idToContentMap entries in sentenceSearch

Gemini review on this PR flagged that sentenceSearch accessed
idToContentMap[id] without optional chaining, unlike the sibling
sentenceAutoComplete which already guards this. Pre-existing issue
(predates this PR), but on a line already touched here, so fixing
it now rather than filing separately.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>

Coverage Stats

2735 of 7546 branches covered (36.24%)

Branch coverage included in aggregate %.

1 of 18 new or added lines in 3 files covered. (5.56%)

3 existing lines in 1 file now uncovered.

6439 of 14187 relevant lines covered (45.39%)

28.05 hits per line