bara-lang/PLAN.md

Clojure/Nim — Test Suite Compliance Plan

Current Status: 233/233 (100%)

Component	Pass	Total	%
clojure.string	8	8	100%
clojure.core	225	225	100%
Total	233	233	100%

Unit Tests: 80/80 (100%) ✅

Status	Test	Issue
✅	40 reader + 40 emitter	All passing

Note: 8 macro tests (test_macros.nim) have pre-existing failures unrelated to this work.

Real-World Examples ✅

Example	Description	Status
fizzbuzz.clj	Classic FizzBuzz with cond, mod, range, doseq	✅ Works
wordfreq.clj	Word frequency counter with reduce, assoc, get, keys, sort, clojure.string/split, clojure.string/lower-case	✅ Works

Both examples compile to native binaries and produce correct output with minimal code changes (only removed #"" regex literal syntax).

0 Remaining Failures

All 233 clojure.test-suite compliance tests pass (rc=0). All 80 unit tests pass.

Previous failures resolved:

1. not_eq — Cross-namespace dependency ✅

• Root cause: extractRequires didn't find ns inside (do ...) wrapper from test runner. resolveNsToPath only tried .clj extension, not .cljc. Search path didn't include test suite directory.
• Fix: Made extractRequires recurse into do forms, added .cljc fallback in resolveNsToPath, added test suite auto-detection in search paths, filtered to only inline def/defn from required files (not macros).

2. add_watch — STM ref functions missing ✅

• Root cause: (ref 10) emitted as ref1(cljInt(10)) instead of cljRef(cljInt(10)). dosync, ref-set, alter also missing.
• Fix: Added runtime name mappings (ref→cljRef, ref-set→cljRefSet, dosync→cljDosync, alter→cljAlter). Implemented minimal STM ref runtime functions.

3. remove_watch — Same STM ref issue ✅

• Root cause: Same as add_watch — uses ref, dosync, alter.
• Fix: Same as add_watch.

4. Symbol Emit Fix ✅

• Root cause: emitExpr for unknown symbols fell through to cljSymbol(...) instead of emitting mangled name. No def registry existed to distinguish variable references from symbol literals.
• Fix: Added definedGlobals registry in emitter. def, defn, defn-, definterface, defprotocol, defrecord, deftype now register names. Symbol emit checks registry before falling back to cljSymbol(...). Unit tests updated to register globals before testing.

Session Log

2026-05-11 (current session) — 233/233 (100%), 80/80 unit tests

• Emitter: added definedGlobals registry with registerGlobal/clearGlobals procs
• Emitter: def/defn/defn-/definterface/defprotocol/defrecord/deftype register globals
• Emitter: symbol emit checks definedGlobals before falling back to cljSymbol(...)
• Emitter: cond now emits return statements in branches
• Emitter: reduce wraps named function symbols in cljFn(...) closure adapter
• Emitter: clojure.string/* functions mapped to runtime (split, lower-case, upper-case, trim, join, replace, includes?, starts-with?, ends-with?, blank?, reverse)
• Emitter: keys/vals removed from variadic list (they take single map arg)
• Emitter: statement detection expanded for echo cljRepr(...) wrapping (while, if, block:, for, try)
• Runtime: cljGet with 3 args now uses cljGetDefault for default value
• Runtime: cljStrSplit now supports regex patterns via Nim's re module
• Tests: emit symbol and emit mangled symbol now register globals before testing
• Examples: added fizzbuzz.clj and wordfreq.clj — both compile and run correctly
• Result: 78/80 → 80/80 unit tests