chore: Expanding lll coverage to dag-graph

[yhakbar]

Description

Addressed lll findings in dag-graph.

TODOs

Read the Gruntwork contribution guidelines.

• I authored this code entirely myself
• I am submitting code based on open source software (e.g. MIT, MPL-2.0, Apache)
• I am adding or upgrading a dependency or adapted code and confirm it has a compatible open source license
• Update the docs.
• Run the relevant tests successfully, including pre-commit checks.
• Include release notes. If this PR is backward incompatible, include a migration guide.

Release Notes (draft)

Updated lll linter coverage to include dag-graph.

Summary by CodeRabbit

• Style

  • Reformatted command documentation and test fixtures for improved readability.

• Chores

  • Updated linter configuration to refine exclusion rules.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
terragrunt-docs	Ready	Preview, Comment	May 7, 2026 6:45pm

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 813d724e-80bb-4bef-b757-1dc6426bbefe

📥 Commits

Reviewing files that changed from the base of the PR and between a38c304 and 7c61950.

📒 Files selected for processing (3)

• .golangci.yml
• internal/cli/commands/dag/graph/cli.go
• internal/cli/commands/dag/graph/cli_test.go

---

📝 Walkthrough

Walkthrough

Linter configuration updated to exclude additional internal paths. CLI command Usage string and test fixture list reformatted across multiple lines for improved readability, with no functional changes.

Changes

Linter Configuration and CLI Formatting

Layer / File(s)	Summary
Linter Configuration .golangci.yml	The lll linter exclusion block adds a path-except regex pattern to exclude additional internal directory paths from line-length linting.
CLI Command and Test Formatting internal/cli/commands/dag/graph/cli.go, internal/cli/commands/dag/graph/cli_test.go	Command Usage string reformatted into multiple concatenated lines; BenchmarkRunGraphDependencies fixture list converted to multi-line struct literal format. Both changes are cosmetic and preserve all content.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

• gruntwork-io/terragrunt#5854: Both PRs modify the same .golangci.yml linter configuration file, specifically the lll path-except exclusion pattern.

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Description check	❓ Inconclusive	The description addresses the core purpose but omits several required template sections including issue reference and several TODO items.	Add the issue reference (Fixes #000) and include all required TODO checkboxes from the template, even if marked as not applicable.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: expanding the lll linter coverage to include dag-graph directories.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch chore/expanding-lll-coverage-to-dag-graph

---

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

The base branch was changed.