bamorim / typed_ecto_schema / 252ffef746e3b2d4f86ff89ea9275798eee759a0

92%

push

github

fix: handle AST variable opts in macros (#53)

Problem: When field options are passed as variables in macro contexts,
they become AST variable references instead of literal keyword lists.
The original code called Keyword.drop/2 directly on opts, which failed
with FunctionClauseError when opts was an AST tuple.

Solution: Check if opts is a compile-time keyword list using
Keyword.keyword?/1. If yes, drop keys at compile time. If no (AST
variable reference), generate code to drop keys at runtime.

Changes:
- Modified TypedEctoSchema.SyntaxSugar.transform_expression/2 to handle
  both compile-time keyword lists and runtime AST variable references
- Added test case that reproduces the issue and verifies the fix
- Added CLAUDE.md with development guidelines

Fixes #52

2 of 2 new or added lines in 1 file covered. (100.0%)

59 of 64 relevant lines covered (92.19%)

27.11 hits per line