chore: Addressing #6019 feedback

[yhakbar]

Description

Addresses feedback from #6019

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.
• Update the changelog in the docs.
• Run the relevant tests successfully, including pre-commit checks.
• This change is backwards compatible.
• If this change is not forwards compatible (e.g. a new feature), it is gated behind a feature flag.

Summary by CodeRabbit

• Improvements

  • Enhanced cache context propagation in CLI provider cache server operations for more reliable cache management.

• Tests

  • Added test coverage for cache context injection in command execution.

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
terragrunt-docs	Ready	Preview, Comment	May 4, 2026 2:16pm

[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: 63d6f1ad-2de7-4e16-afd3-5b4810813423

📥 Commits

Reviewing files that changed from the base of the PR and between e20663b and 0d6c500.

📒 Files selected for processing (2)

• internal/cli/commands/commands.go
• internal/cli/commands/run_action_test.go

---

📝 Walkthrough

Walkthrough

The runAction function is renamed and exported as RunAction. The telemetry wrapper is updated to call the exported function. The provider cache server initialization now receives the run-scoped action context instead of the base context, propagating cache context through the server lifecycle.

Changes

RunAction Context Propagation

Layer / File(s)	Summary
API Export & Context Wiring internal/cli/commands/commands.go (lines 242–304)	runAction is renamed to exported RunAction. Provider cache server methods (server.Listen and server.Run) now receive actionCtx instead of base ctx, propagating run-scoped cache context into the server lifecycle.
Caller Integration internal/cli/commands/commands.go (lines 155–160)	WrapWithTelemetry updated to invoke RunAction instead of the internal runAction.
Test Coverage internal/cli/commands/run_action_test.go (lines 20–43)	New test TestRunActionInstallsRunScopedCache verifies that RunAction correctly injects cache context values (RunCmdCacheContextKey and RepoRootCacheContextKey) into the action callback's context.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~15 minutes

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description is largely incomplete. It references issue #6019 but lacks concrete details about what changes were made, why they were made, and what the feedback addressed.	Expand the description section to explain the specific changes: exporting RunAction, propagating actionCtx to server lifecycle, adding cache context tests, and how these changes address the feedback from #6019.
Title check	❓ Inconclusive	The title refers to addressing feedback from issue #6019, but does not clearly summarize the specific technical changes (exporting RunAction function, adding cache context propagation, and new test).	Consider using a more specific title like 'Export RunAction and propagate actionCtx to cache server' that describes the actual code changes.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
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/addressing-6019-feedback

Warning

Review ran into problems

🔥 Problems

Timed out fetching pipeline failures after 30000ms

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

• Generate code and open pull requests
• Plan features and break down work
• Investigate incidents and troubleshoot customer tickets together
• Automate recurring tasks and respond to alerts with triggers
• Summarize progress and report instantly

Built for teams:

• Shared memory across your entire org—no repeating context
• Per-thread sandboxes to safely plan and execute work
• Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started

---

_{Review rate limit: 4/5 reviews remaining, refill in 12 minutes.}

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