Skip to content

Improve error handling#68

Merged
robertluo merged 7 commits intomasterfrom
improve-error-handling
Jan 19, 2026
Merged

Improve error handling#68
robertluo merged 7 commits intomasterfrom
improve-error-handling

Conversation

@robertluo
Copy link
Owner

No description provided.

robertluo and others added 7 commits January 18, 2026 15:52
- Add circular dependency detection with clear error messages showing cycle path
- Add better error context for function wrapper failures (key, available keys)
- Add comprehensive Gotchas section to doc/concepts.adoc
- Make CloseableValue implement java.io.Closeable for with-open support
- Remove unnecessary #?(:clj) from defmacro fw/fnk - macros work in CLJS
- Rename doc/rational.adoc to doc/rationale.adoc (typo fix)
- Add doc/improvements.adoc tracking planned improvements
- Update WARP.md with known issues and gotchas
- Improve docstrings for fun-map, fw, fnk, closeable, life-cycle-map

Co-Authored-By: Warp <agent@warp.dev>
Google Closure Compiler in CLJS 1.12.x requires Java 21.

Co-Authored-By: Warp <agent@warp.dev>
Co-Authored-By: Warp <agent@warp.dev>
…tchas

- Fix valAt/lookup nil/false handling in both CLJ and CLJS
  Previously (get m :key :not-found) returned :not-found when value was nil/false
- Add IKVReduce implementation for better reduce-kv performance  
- Add ILookup to CLJS TransientDelegatedMap
- Document hash/equals gotchas in concepts.adoc
- Add tests for nil/false value handling
- Fix typo: iterface -> interface

Co-Authored-By: Warp <agent@warp.dev>
- Fix: -unwrap for Object/nil now throws instead of returning ex-info
- Fix: CachedWrapper race condition - move staleness check inside swap!
- Add: TracedWrapper print-method for consistency
- Cleanup: Remove dead CLJS raw-keys code path
- Cleanup: Standardize docstrings across wrapper constructors

Co-Authored-By: Warp <agent@warp.dev>
@robertluo robertluo merged commit 7a6322a into master Jan 19, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant