fix(nu): suppress error when ATUIN_HISTORY_ID is missing in pre_prompt hook

[AndrianBalanescu]

What

Fixes a crash in the nushell integration when ATUIN_HISTORY_ID is not set in the environment.

The _atuin_pre_prompt hook calls hide-env ATUIN_HISTORY_ID at the end of every prompt. If the env var doesn't exist (e.g. when using nushell overlays/subshells that don't capture it, like nu -e "overlay use ..."), this throws an unhandled env_variable_not_found error.

How

The fix mirrors what the script already does at line 15 — use the -i (ignore-if-missing) flag:

// Before
hide-env ATUIN_HISTORY_ID

// After
hide-env -i ATUIN_HISTORY_ID

This is the exact same pattern already used in the initialization section (line 15: hide-env -i ATUIN_HISTORY_ID), so it's consistent with the existing code. The reporter of #3520 also suggested this approach. (◕‿◕)

One file, one line changed. No behavior change for the normal case (var exists → it gets hidden). Only difference: missing var no longer crashes.

Closes #3520.

[github-actions]

Fossier: Manual Review Requested

@andrianbalanescu is a new contributor. A maintainer should review this PR before merging.

Score Breakdown

Total Score: 43.3/100 | Confidence: 100% | Outcome: REVIEW

Signal	Value	Score	Weight
account_age	3027	1.00	0.09
public_repos	16	0.80	0.05
contribution_history	34	0.17	0.05
follower_ratio	0.18	0.09	0.05
bot_signals	False	0.50	0.04
open_prs_elsewhere	8	0.00	0.09
closed_prs_elsewhere	5	0.50	0.10
merged_prs_elsewhere	0	0.20	0.08
prior_interaction	0	0.00	0.08
activity_velocity	1	1.00	0.08
pr_content	...	0.70	0.08
commit_email	no_email	0.50	0.04
pr_description	...	0.55	0.05
repo_stars	30474	0.30	0.04
org_membership	0	0.20	0.03
commit_verification	...	0.30	0.04
contributor_stars	4	0.08	0.04

[greptile-apps]

Greptile Summary

Adds the -i (ignore-if-missing) flag to the hide-env ATUIN_HISTORY_ID call in the _atuin_pre_prompt hook, preventing a crash when ATUIN_HISTORY_ID is absent (e.g. in nushell overlays/subshells).

• Mirrors the identical pattern already present at line 15 of the same file, making the two call sites consistent.

Confidence Score: 5/5

Safe to merge — minimal change with no behavior impact when the variable is present.

Single-flag addition to a shell script that directly fixes a crash for users running nushell overlays. Existing behavior when the variable is set is unchanged; the only effect is suppressing the error when it's absent.

No files require special attention.

Important Files Changed

Filename	Overview
crates/atuin/src/shell/atuin.nu	One-line fix: adds -i to hide-env ATUIN_HISTORY_ID in _atuin_pre_prompt, consistent with line 15 which already uses the same pattern.

_{Reviews (1): Last reviewed commit: "fix(nu): suppress error when ATUIN_HISTO..." | Re-trigger Greptile}

[ellie]

thank you!