Skip to content

[codex] add ast_canopy macro define extraction#373

Draft
isVoid wants to merge 3 commits into
mainfrom
codex/ast-canopy-macro-defines
Draft

[codex] add ast_canopy macro define extraction#373
isVoid wants to merge 3 commits into
mainfrom
codex/ast-canopy-macro-defines

Conversation

@isVoid

@isVoid isVoid commented Jun 4, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Adds macro_defines to ast_canopy parse results.
  • Exposes macro definitions through C++, pybind11, the Python Declarations dataclass, and the typing stub.
  • Extracts retained-file, object-like macro definitions from Clang preprocessor state without a second parser pass.
  • Adds tests for object-like macros, empty defines, skipped function-like macros, and retained include filtering.

Validation

  • pixi run pytest ast_canopy/tests/test_parse_macro.py
  • pixi run pytest ast_canopy/tests/test_parse_from_source.py

Notes

  • This branch was created from origin/main in a clean temporary worktree and includes only the ast_canopy macro changes.

@coderabbitai

coderabbitai Bot commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Enterprise

Run ID: d42d448b-ca6a-4f3c-9905-2d948f4b6a56

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/ast-canopy-macro-defines

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions

github-actions Bot commented Jun 4, 2026

Copy link
Copy Markdown
Doc Preview CI
🚀 View pre-built docs at
https://NVIDIA.github.io/numbast/pr-preview/pr-373/

Preview will be ready when GitHub Pages deployment finishes.

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