Skip to content

Add working directory context to dry-run compile command - #26

Open
ahogappa wants to merge 1 commit into
masterfrom
feature/improve-dry-run-command-display
Open

Add working directory context to dry-run compile command#26
ahogappa wants to merge 1 commit into
masterfrom
feature/improve-dry-run-command-display

Conversation

@ahogappa

@ahogappa ahogappa commented Jan 25, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add cd <work_dir> && prefix to compile command shown with dry-run option
  • Allow users to copy and paste the command directly for execution
  • Properly escape paths with special characters using Shellwords.escape

Test plan

  • All existing tests pass

🤖 Generated with Claude Code

Include cd command in dry-run output so users can copy and paste
the command directly. The work_dir path is escaped using Shellwords
to handle paths with spaces or special characters.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jan 25, 2026

Copy link
Copy Markdown

Walkthrough

The dry-run output in lib/kompo/tasks/packing.rb now prefixes executed commands with a change directory instruction, displaying "cd [work_dir] && [command]" instead of just the command, preserving working directory context in dry-run messages.

Changes

Cohort / File(s) Summary
Dry-run output formatting
lib/kompo/tasks/packing.rb
Modified both dry-run branches to prefix command output with change directory instruction (cd [work_dir] && ...) for working directory context

Possibly Related PRs

Poem

🐰 cd into the burrow, the path shines bright,
Dry-runs now show where we venture right,
With working directories spelled out clear,
A rabbit navigates without fear! 🌿✨


Estimated code review effort: 🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding working directory context to dry-run compile command output.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

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

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant