chore: Expanding lll coverage to queue

[yhakbar]

Description

Addressed lll findings in queue.

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 queue.

Summary by CodeRabbit

• Chores

  • Updated linter configuration for path exclusions.

• Style

  • Reformatted comments and test assertions for improved readability.

Note: This release contains internal code quality improvements with no user-facing changes.

[coderabbitai]

📝 Walkthrough

Walkthrough

The PR updates linter configuration to exclude internal/queue/ paths from the lll linter, and reformats comments and test assertions in the queue module for improved readability. All changes are cosmetic with no logic modifications.

Changes

Cohort / File(s)	Summary
Linter Configuration .golangci.yml	Updated lll linter path-except regex to exclude internal/queue/ paths from line length checks.
Queue Module Formatting internal/queue/queue.go, internal/queue/queue_test.go	Rewrapped comment documentation and test assertion arguments across multiple lines for readability without altering logic or behavior.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 25.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: expanding the lll linter configuration to include the queue package by adding internal/queue/ to exclusions.
Description check	✅ Passed	The description covers the main change (addressing lll findings in queue), includes checked TODOs, and provides release notes as required by the template.
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-queue

---

_{Review rate limit: 3/5 reviews remaining, refill in 12 minutes and 48 seconds.}

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

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
terragrunt-docs	Ready	Preview, Comment	Apr 29, 2026 5:20pm

The base branch was changed.

[coderabbitai]

🧹 Nitpick comments (1)

.golangci.yml (1)

115-117: Consider refactoring the path-except regex for maintainability.

The path-except regex on line 117 is extremely long and contains many directory patterns in a single expression, which can be challenging to maintain. Consider splitting this into multiple exclusion rules or using a more structured format (e.g., a list of path patterns).

💡 Example refactor approach

While golangci-lint v2 doesn't directly support arrays for path-except, you could potentially split this into multiple linter exclusion rules for better readability:

      - linters:
          - lll
-       path-except: '^(internal/awshelper/|internal/cas/|...)'
+       path-except: '^(internal/awshelper/|internal/cas/|internal/cli/commands/(backend/(delete|migrate)|catalog/tui/command|exec|find|help|list|stack)/)'
+     - linters:
+         - lll
+       path-except: '^(internal/cloner/|internal/configbridge/|internal/engine/|internal/errorconfig/)'

However, this is a low-priority enhancement and doesn't need to block this PR.

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In @.golangci.yml around lines 115 - 117, The long single path-except regex in
.golangci.yml (the path-except entry) is hard to maintain; split it into
multiple, smaller exclusions by creating separate linter entries or multiple
path-except rules that each match a subset of directories (e.g., group by
internal/* subsystems like internal/runner, internal/stacks, internal/tf,
pkg/log, etc.), or convert to several anchored regexes with clearer names so
each pattern is shorter and easier to edit; update the linter configuration to
apply these multiple exclusions in place of the single giant regex.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In @.golangci.yml:
- Around line 115-117: The long single path-except regex in .golangci.yml (the
path-except entry) is hard to maintain; split it into multiple, smaller
exclusions by creating separate linter entries or multiple path-except rules
that each match a subset of directories (e.g., group by internal/* subsystems
like internal/runner, internal/stacks, internal/tf, pkg/log, etc.), or convert
to several anchored regexes with clearer names so each pattern is shorter and
easier to edit; update the linter configuration to apply these multiple
exclusions in place of the single giant regex.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 8bcd4234-b568-4086-b432-17c9a0da1d83

📥 Commits

Reviewing files that changed from the base of the PR and between d27064f and d2956f7.

📒 Files selected for processing (3)

• .golangci.yml
• internal/queue/queue.go
• internal/queue/queue_test.go