fix(output): correct user-output consistency defects

[max-sixty]

An audit of wt's user-visible strings against the repo's writing-user-outputs skill. Six consistency corrections, each matching a rule the skill states — and, in most cases, a sibling message that already follows it:

• RefCreateConflict hint: dropped the cross-message pronoun ("it") and switched to the skill's "To X, run Y" command-hint form.
• "All shells already configured" and the version-check "Up to date": success ✓ → info ○ — both acknowledge existing state without changing anything.
• relocate summary: keyed the message type on whether anything was relocated, not on whether anything was skipped (the same outcome was rendered two ways).
• "Diagnostic saved": → success (a file was created) and the @-before-path convention.
• Removed a stray trailing period — the only single-line message in the codebase that had one.

--no-verify is deliberately untouched (handled in the canonical-paths PR). Snapshot tests updated.

This was written by Claude Code on behalf of Maximilian Roos

[worktrunk-bot]

codecov/patch is failing on src/commands/relocate.rs:714 — the bare return; line in show_summary's defensive early-return. The old form if relocated > 0 || skipped > 0 { … } collapsed the no-op into the implicit "fall off the end of the if" path; the new early-return is its own line and the existing tests never exercise show_summary(0, 0) (the sole caller at step/relocate.rs:153 runs after validated.is_empty() returned false, so at least one of relocated_count / skipped_count is always ≥ 1).

Simplest fix: restore the wrap form, keeping the new success-vs-info logic inside it:

pub fn show_summary(relocated: usize, skipped: usize) {
    if relocated > 0 || skipped > 0 {
        eprintln!();
        let plural = |n: usize| if n == 1 { "worktree" } else { "worktrees" };
        let msg = if skipped == 0 {
            format!("Relocated {relocated} {}", plural(relocated))
        } else {
            format!(
                "Relocated {relocated} {}, skipped {skipped} {}",
                plural(relocated),
                plural(skipped)
            )
        };
        // Success when worktrees moved; info when only skips (no change made).
        if relocated > 0 {
            eprintln!("{}", success_message(msg));
        } else {
            eprintln!("{}", info_message(msg));
        }
    }
}

Behavior is identical; the (0, 0) no-op falls out of the if instead of returning explicitly, so there's no isolated line to cover. Happy to push this if you want.

Dismissing my approval per the codecov/patch merge gate in CLAUDE.md.

codecov/patch failed — line 714 of relocate.rs uncovered