Skip to content

feat: make AST logging extraction opt-in via CLI toggle#1875

Open
ajithksr wants to merge 5 commits into
Graphify-Labs:v8from
ajithksr:feature/generic-ast-logging-extractor
Open

feat: make AST logging extraction opt-in via CLI toggle#1875
ajithksr wants to merge 5 commits into
Graphify-Labs:v8from
ajithksr:feature/generic-ast-logging-extractor

Conversation

@ajithksr

@ajithksr ajithksr commented Jul 14, 2026

Copy link
Copy Markdown

This PR makes the AST-based logging extraction feature an opt-in CLI toggle, improving the default performance and usability for users who do not need logging analysis.

Changes:

  • Refactored GenericLogExtractor to load configuration lazily based on environment variables.
    • Added --enable-logging and --logging-config PATH arguments to graphify extract and graphify update.
    • Updated inline CLI help and README.md to document the new flags.
    • Updated test suite to verify the disabled-by-default behavior and the opt-in extraction.
      All 128 tests pass successfully.

@ajithksr ajithksr changed the title Feature/generic ast logging extractor feat: make AST logging extraction opt-in via CLI toggle Jul 14, 2026
@ajithksr

Copy link
Copy Markdown
Author

Hello @safishamsi just checking in to see if you have bandwidth to review this PR. Let me know if you need any further context or changes to get this merged!" I intend to provide a featire where we can use graphify to extract all comment blocks from with in a function...with a traget to improve logging analysis in linux/android world.

@ajithksr
ajithksr force-pushed the feature/generic-ast-logging-extractor branch from 10f9055 to 56598f6 Compare July 24, 2026 03:17
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