Skip to content

Ruby CLI: Improve and flesh out herb analyze command#1259

Merged
marcoroth merged 1 commit into
mainfrom
herb-analyze-cli-rework
Feb 28, 2026
Merged

Ruby CLI: Improve and flesh out herb analyze command#1259
marcoroth merged 1 commit into
mainfrom
herb-analyze-cli-rework

Conversation

@marcoroth

@marcoroth marcoroth commented Feb 28, 2026

Copy link
Copy Markdown
Owner

This pull request reworks, improves and fleshes out the herb analyze in the Ruby CLI.

We removed the interactive mode in favor of simpler . to print progress. The interactive mode in its previous state was kind of broken and glitchy when the list of failed files didn't fit on a single screen height.

Instead of generating the analyze log file, we no show richer, and more relevant errors right in the output of the analyze command, that should make it easier to see the state of a project. Additionally, we visually updated the output to match the output of the herb-lint CLI more closely.

We also added a new report subcommand that you can use to generate a copy-able markdown report that can be used to report issues about Herb on GitHub.

More importantly, we restructured the way the analyze command works by trying the more closely group the failed files so it's easier to follow why something is failing and at which stage. Since we also have multiple parser and engine options, we try to fallback to less-strict options to see if that makes some of the errors go away.

The analyze subcommand also doesn't need an argument now, it automatically assumes . (current directory). It now also accepts single files, not just paths to directories.

CleanShot 2026-02-28 at 06 53 12@2x CleanShot 2026-02-28 at 06 55 35@2x

Resolves #507
Resolves #941

@marcoroth marcoroth added this to the v1.0.0 milestone Feb 28, 2026
@marcoroth marcoroth force-pushed the herb-analyze-cli-rework branch 2 times, most recently from 5e2f643 to 0316f17 Compare February 28, 2026 06:33
@pkg-pr-new

pkg-pr-new Bot commented Feb 28, 2026

Copy link
Copy Markdown
npx https://pkg.pr.new/@herb-tools/formatter@1259
npx https://pkg.pr.new/@herb-tools/language-server@1259
npx https://pkg.pr.new/@herb-tools/linter@1259

commit: 1e9757e

@github-actions

github-actions Bot commented Feb 28, 2026

Copy link
Copy Markdown

🌿 Interactive Playground and Documentation Preview

A preview deployment has been built for this pull request. Try out the changes live in the interactive playground:


🌱 Grown from commit 1e9757e


✅ Preview deployment has been cleaned up.

@marcoroth marcoroth merged commit 6c30eb5 into main Feb 28, 2026
32 checks passed
@marcoroth marcoroth deleted the herb-analyze-cli-rework branch February 28, 2026 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ruby: Better CLI flags for analyze Make Herb report output path configurable and ignore it in Git

1 participant