Skip to content

Expand AST using Macro.Env API #293

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 239 commits into from
Oct 5, 2024
Merged

Expand AST using Macro.Env API #293

merged 239 commits into from
Oct 5, 2024

Conversation

@lukaszsamson
Copy link
Collaborator

@lukaszsamson lukaszsamson commented Jul 18, 2024

This PR replaces MetadataBuilder implementation with a mini compiler that is close to what elixir compiler does when traversing AST. It expands special forms and macros (if possible).
Key differences from elixir compiler:

  • It is designed to be error tolerant. In most places where elixir will throw off the compilation it attempt to continue
  • It does not do any tracing
  • It doesn't evaluate code. It means that it is not able to handle dynamic code and quoted defs
  • it is not able to expand local macros. This would require evaluating and compiling the code
  • it intercepts a few Kernel, Record macros (defmodule, def, defguard, defdelegate, @, defrecord, defprotocol, defimpl), attempts to traverse the code and extract info

@lukaszsamson
squash
c6bcf8f
@lukaszsamson
wip
3c391bc
@lukaszsamson
behaviours
24abad6
@lukaszsamson
defoverridable
8db10ba
@lukaszsamson
do not raise on unknown local
48a925f
@lukaszsamson
variables
fd4a4f3
@lukaszsamson
variables
2e46073
@lukaszsamson
more tests
7460210 
super resolve
@lukaszsamson
module attribute
6e50e3d
@lukaszsamson
defdelegate
eceb36f
@lukaszsamson
calls
42aeb0d
@lukaszsamson
fix bug in bind_quoted
6e99c46
@lukaszsamson
some support for unquote fragments
ae9f28c
@lukaszsamson
typespecs
d3c9ad3
@lukaszsamson
structs
4fd9305
@lukaszsamson
doc and meta
1cf683e
@lukaszsamson
Merge branch 'macro-envy' into expand-checkpoint
d0cdf68
@lukaszsamson
records
4b5fb8d
@lukaszsamson
protocols
05f8a6c
@lukaszsamson
reduce debug amount
d9103a4
@lukaszsamson
attribute type inference
895cdca
@lukaszsamson
first step
caf4270
@lukaszsamson
simplify var handling
199b196
@lukaszsamson
Merge branch 'master' into vars-simplification
ec5a4ba 
# Conflicts:
#	lib/elixir_sense/core/metadata_builder.ex
#	lib/elixir_sense/core/state.ex
#	test/elixir_sense/core/metadata_builder_test.exs
@lukaszsamson
fix scopes
61c37f6
@lukaszsamson
expand local structs
d02daf5 
recover from errors
@lukaszsamson
recover from map key errors
f6f1806
@lukaszsamson
error tolerant opts
2e4bc72
@lukaszsamson
recover from incomplete code with naked when
320eef8
@lukaszsamson
rescue errors on captures
ff4a89c
@lukaszsamson
exclude some tests on < 1.15
65f1798
@lukaszsamson
fix crash on < 1.14
fa78f3a
@lukaszsamson
exclude more tests
20003df
@lukaszsamson
exclude and fix tests on < 1.15
f0bad12
@lukaszsamson
finer excludes
534cb0f
@lukaszsamson
exclude tests on < 1.14
26d4e87
@lukaszsamson
simplify current_env
67b22e2
@lukaszsamson
format
986781b
@lukaszsamson
Revert "simplify current_env"
d640145 
This reverts commit 67b22e2.

# Conflicts:
#	lib/elixir_sense/core/metadata.ex
@lukaszsamson
optimize boolean
46546b2
@lukaszsamson
prevent type narrowing from leaking to outer scope
f91f258
@lukaszsamson
fix warnings
057a578
@lukaszsamson
disable test
749adcd
@lukaszsamson
reverse conditions
44a4987
@lukaszsamson
capture TODOs addressed
1ea8683
@lukaszsamson
work on TODOs
4eda1d5
@lukaszsamson
fix tests
a479232
@lukaszsamson
split modules
7322fb4
@lukaszsamson
fix small issues
d0d33d5
@lukaszsamson
split out infos from state
c085d6e
@lukaszsamson
separate compiler state
1a04eaa
@lukaszsamson
fix warnings
4348d7f
@lukaszsamson
rename
4626ff9
@lukaszsamson
fix invalid API usage
4ef2bc0
@lukaszsamson
exclude test
5f2a694
@lukaszsamson
exclude tests
ef46958
@lukaszsamson
hack for < 1.15
f2b6172
@lukaszsamson lukaszsamson changed the title [WIP] Expand AST using Macro.Env API Expand AST using Macro.Env API Oct 5, 2024
@lukaszsamson lukaszsamson merged commit d1ebaad into master Oct 5, 2024
35 checks passed
@josevalim josevalim mentioned this pull request Oct 7, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@josevalim josevalim josevalim left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@lukaszsamson @josevalim