source-academy / py-slang / 28742804235 / 1

78%

main: 78%

push

github

Renaming variables to match SICPy and remove unnecessary functions (#193)

* renaming

* Update docs/md/README_1.md

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

* gemini review responses

* format

* new failing test case corrected

* adding is_number, needed by SICP afterall

* addressed Shrey's review

* format

* getting test coverage back up

* fix(stepper): update preprocessing-error test expectations after rename

The is_int->is_integer rename (and map_linked_list->map) shifted the
fast-levenshtein "perhaps you meant" suggestions that two getSteps
preprocessing tests asserted verbatim, breaking CI:

- `missing` no longer suggests 'is_int' (renamed to is_integer, now out of
  suggestion range) -> the suggestion line is gone.
- `map` in §1 now suggests 'max' (map became a §2 list-library name) -> a
  suggestion line is added.

Re-derived both expected strings from the actual analyzer output.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

* Remove int/float/bool builtins (#184 Removals); restore len and §1 MISC docs

Per #184's Removals section, drop the int()/float()/bool() conversion builtins
from both evaluators, the tests and the LaTeX spec:
- src/stdlib/misc.ts: remove the three static builtins, plus the now-unused
  ControlItem/isFalsy/ValueError imports
- src/conductor/stepper/builtins.ts: remove the three entries, the now-dead
  truthy() helper, and their BUILTIN_MIN_ARGS / docstring / comment mentions
- docs/specs/python_misc.tex: drop int/float/bool (keep complex/str)
- tests: prune the int/float/bool cases; re-derive two levenshtein "perhaps you
  meant" suggestions that shifted now that `int` is gone (llist -> 'print')

Also fix two accidental doc regressions this branch introduced (flagged in
review, unrelated to #184):
- docs/lib/misc.js: restore the len() stub (it should not have moved at all)
- docs/specs/python_1.tex: restore \input python_misc (the MISC library i... (continued)

2460 of 3408 branches covered (72.18%)

Branch coverage included in aggregate %.

5884 of 7248 relevant lines covered (81.18%)

13817.08 hits per line