fix: clarify error message when starting docker container

[unlair]

Error message was not indicating which Docker container failed to start, forcing the user to use a process of elimination to narrow down the problem.

What kind of change does this PR introduce?

Bug fix (updating error message).

What is the current behavior?

When an error is returned when starting a Docker container (e.g. #3593 the error message reads:
failed to start docker container: <some error>

What is the new behavior?

The error message reads:
failed to start docker container "<container name>": <some error>

[coveralls]

Pull Request Test Coverage Report for Build 21680522717

Details

• 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
• 7 unchanged lines in 2 files lost coverage.
• Overall coverage decreased (-0.02%) to 55.353%

---

Files with Coverage Reduction	New Missed Lines	%
internal/storage/rm/rm.go	2	80.61%
internal/gen/keys/keys.go	5	12.9%

Totals
Change from base Build 21669533478:	-0.02%
Covered Lines:	7114
Relevant Lines:	12852

---

💛 - Coveralls

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Central YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro

Cache: Disabled due to Reviews > Disable Cache setting

Disabled knowledge base sources:

• Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 97f8bfd and ab6464d.

📒 Files selected for processing (1)

• internal/utils/docker.go

🚧 Files skipped from review as they are similar to previous changes (1)

• internal/utils/docker.go

---

📝 Walkthrough

Summary by CodeRabbit

• Bug Fixes
  • Improved Docker container startup error messages to include the container name for better debugging clarity.

Walkthrough

The DockerStart function's error handling was modified to include the container name in the wrapped error message. The format string changed from "failed to start docker container: %w" to "failed to start docker container %q: %w", containerName, err. No other logic, control flow, or public signatures were changed.

🚥 Pre-merge checks | ✅ 3 | ❌ 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 (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the main change: clarifying an error message when docker container startup fails by adding the container name.
Merge Conflict Detection	✅ Passed	✅ No merge conflicts detected when merging into develop

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

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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