Fix colon in symbol/keyword literals and enable reader tests

Allow colon (:) inside symbol and keyword names. Previously colons
were unconditionally excluded from isSymbolChar(), causing 🔤def:ghi
to be tokenized as three tokens. Now colons are only special at token
boundaries (keyword literal start). Reader test assertions: 144 → 146.

Author: chaploud

.dev/memo.md

@@ -27,13 +27,13 @@ Phase 53: Hardening & pprint Tests — IN PROGRESS
 - [x] 53.2: Fix loop destructuring recur bug
 - [x] 53.3: Align distinct? to upstream (enabled by 53.2)
 - [x] 53.4: Fix BigDecimal exponent notation (1.0e+1M)
-- [ ] 53.5: Fix colon in symbol/keyword literals
+- [x] 53.5: Fix colon in symbol/keyword literals
 - [ ] 53.6: Port pprint tests (content-equivalent, CW-adapted)
 - [ ] 53.7: Full regression + update docs
 
 ## Current Task
 
-53.5: Fix colon in symbol/keyword literals
+53.6: Port pprint tests (content-equivalent, CW-adapted)
 
 ## Previous Task
 
@@ -43,7 +43,6 @@ Phase 52: Quality & Alignment (complete)
 ## Known Issues
 
 - apply on infinite lazy seq realizes eagerly (deferred — no tests need it)
-- Colon in symbol/keyword literals parsed as keyword delimiter
 - binding *ns* doesn't affect read-string for auto-resolved keywords
 - Regex capture groups/backreferences not supported
 

src/reader/tokenizer.zig

@@ -455,8 +455,9 @@ fn isTerminator(c: u8) bool {
 fn isSymbolChar(c: u8) bool {
     // # is valid inside symbols (gensym: foo#, tagged: my#tag)
     // but not at start (handled by next() dispatch).
-    // : is excluded — colons start keywords.
-    return !isTerminator(c) and c != ':';
+    // : is valid inside symbols/keywords (e.g. abc:def, :abc:def:ghi).
+    // Colons only start keywords at token boundaries (handled by next()).
+    return !isTerminator(c);
 }
 
 fn isWhitespace(c: u8) bool {

test/upstream/clojure/test_clojure/reader.clj

@@ -10,7 +10,7 @@
 
 ;; Upstream: clojure/test/clojure/test_clojure/reader.cljc
 ;; Upstream lines: 802
-;; CLJW markers: 29
+;; CLJW markers: 27
 
 (ns clojure.test-clojure.reader
   ;; CLJW: removed clojure.instant, clojure.walk, clojure.edn, test.generative,
@@ -22,12 +22,11 @@
 (deftest Symbols
   (is (= 'abc (symbol "abc")))
   (is (= '*+!-_? (symbol "*+!-_?")))
-  ;; CLJW: colon in symbol names parsed as keyword delimiter, skipped 2 tests
-  ;; (is (= 'abc:def:ghi (symbol "abc:def:ghi")))
+  (is (= 'abc:def:ghi (symbol "abc:def:ghi")))
   (is (= 'abc/def (symbol "abc" "def")))
   (is (= 'abc.def/ghi (symbol "abc.def" "ghi")))
   (is (= 'abc/def.ghi (symbol "abc" "def.ghi")))
-  ;; (is (= 'abc:def/ghi:jkl.mno (symbol "abc:def" "ghi:jkl.mno")))
+  (is (= 'abc:def/ghi:jkl.mno (symbol "abc:def" "ghi:jkl.mno")))
   ;; CLJW: symbol? instead of instance? clojure.lang.Symbol
   (is (symbol? 'alphabet)))
 
@@ -226,14 +225,12 @@
   (is (= :abc (keyword "abc")))
   (is (= :abc (keyword 'abc)))
   (is (= :*+!-_? (keyword "*+!-_?")))
-  ;; CLJW: colon in keyword literals parsed as keyword delimiter
-  ;; Use keyword fn to create and verify instead of literal
-  (is (= (keyword "abc:def:ghi") (keyword "abc:def:ghi")))
+  (is (= :abc:def:ghi (keyword "abc:def:ghi")))
   (is (= :abc/def (keyword "abc" "def")))
   (is (= :abc/def (keyword 'abc/def)))
   (is (= :abc.def/ghi (keyword "abc.def" "ghi")))
   (is (= :abc/def.ghi (keyword "abc" "def.ghi")))
-  (is (= (keyword "abc:def" "ghi:jkl.mno") (keyword "abc:def" "ghi:jkl.mno")))
+  (is (= :abc:def/ghi:jkl.mno (keyword "abc:def" "ghi:jkl.mno")))
   ;; CLJW: keyword? instead of instance? clojure.lang.Keyword
   (is (keyword? :alphabet)))