chore: add timeout to charger, vehicle, dump

[premultiply]

Fixes #24101 (comment)

[andig]

Kann rein?

[sourcery-ai]

Hey there - I've reviewed your changes - here's some feedback:

• Extract the repeated timeout flag addition and retrieval logic into a shared helper to reduce duplication across commands.
• Consider handling the error returned by cmd.Flags().GetDuration instead of ignoring it to avoid masking potential flag parsing issues.

Prompt for AI Agents

Please address the comments from this code review:

## Overall Comments
- Extract the repeated timeout flag addition and retrieval logic into a shared helper to reduce duplication across commands.
- Consider handling the error returned by cmd.Flags().GetDuration instead of ignoring it to avoid masking potential flag parsing issues.

## Individual Comments

### Comment 1
<location> `cmd/dump.go:95-96` </location>
<code_context>
 	}

-	d := dumper{len: 2}
+	timeout, _ := cmd.Flags().GetDuration(flagTimeout)
+	d := dumper{len: 2, timeout: timeout}

 	d.Header("config", "=")
</code_context>

<issue_to_address>
**issue (bug_risk):** Handle potential errors from GetDuration to avoid silent failures.

Explicitly handle errors from GetDuration to prevent unexpected behavior when the flag value is malformed. Provide user feedback if parsing fails.
</issue_to_address>

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}