Walkthrough

OutlineExtractOptions is removed as a struct. Its two responsibilities are split into standalone pub(super) functions in options.rs. The show_empty_files flag is absorbed as a field on OutlineExtractors, which gains a try_from_arg constructor. stream_paths and its helpers drop the options parameter, and run_outline is simplified to a single constructor call.

Changes

Outline extraction options refactoring

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• ast-grep/ast-grep#2738: Introduced the original OutlineExtractOptions struct and stream_paths wiring that this PR refactors away.
• ast-grep/ast-grep#2748: Also reshapes OutlineExtractOptions/empty-file behavior plumbing across the same three files.
• ast-grep/ast-grep#2747: Refactors OutlineExtractors construction in extract.rs, changing how extractor options and filtering are passed in.

Poem

🐇 The options struct once carried weight,
Two fields bundled, passed around late.
Now show_empty_files lives inside,
And helpers split what struct did hide.
Less plumbing, cleaner is the way —
A tidy warren, hip-hooray! 🌿

❌ Failed checks (1 warning)

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

• X
• Mastodon
• Reddit
• LinkedIn

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.