Skip to content

fix: preserve ignored signals at shell entry - #14

Merged
cataggar merged 1 commit into
kfrom
fix/issue-7-ignore-entry-traps
Aug 20, 2026
Merged

cataggar merged 1 commit into
kfrom
fix/issue-7-ignore-entry-traps

Conversation

@cataggar

@cataggar cataggar commented Aug 20, 2026

Copy link
Copy Markdown
Owner

Summary

  • silently decline handlers for signals captured as ignored at shell entry
  • preserve ignored dispositions across resets and cloned subshells without consulting later process dispositions
  • cover named and numeric trap specifications while leaving real-time signal parsing out of scope

Tests

  • cargo test --package brush-core traps::tests
  • cargo test --test brush-compat-tests -- 'Builtins: trap'
  • cargo fmt --all --check
  • cargo clippy --workspace --all-features --all-targets

Closes #7

Silently decline trap registration for signals captured as ignored at process entry while preserving the snapshot through resets and subshell clones.

Assisted-by: GitHub Copilot CLI:gpt-5.6-sol

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@cataggar
cataggar merged commit bad6fcd into k Aug 20, 2026
38 checks passed
@cataggar
cataggar deleted the fix/issue-7-ignore-entry-traps branch August 20, 2026 02:43
cataggar added a commit that referenced this pull request Aug 22, 2026
Silently decline trap registration for signals captured as ignored at process entry while preserving the snapshot through resets and subshell clones.

Assisted-by: GitHub Copilot CLI:gpt-5.6-sol

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.

Signals ignored at shell entry can still be trapped

1 participant