feat: tab completion bell chime and sound driver cleanup#207
Merged
Conversation
Add tab completion that includes JS builtins (Promise, JSON, Math, etc.), registered native functions (exec, cd, pwd, etc.), and user-defined globals alongside PATH executables. Also fix arrow keys (Up/Down/Left/Right), Home, End, and Delete by generating VT100 escape sequences in the ARM64 virtio-input keyboard handler — these keycodes were previously silently dropped. Co-Authored-By: Ryan Breen <ryanbreen@gmail.com> Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…cleanup Audio support: - VirtIO sound drivers for PCI (x86_64) and MMIO (ARM64) - Audio syscalls (AudioInit 420, AudioWrite 421) wired into both architectures - libbreenix audio module with init/write wrappers - Tones demo program generating PCM sine waves - QEMU coreaudio backend configured in run.sh Syscall unification: - Remove ARM64-only syscall/io.rs; ARM64 now shares handlers.rs with x86_64 - cfg-gate x86_64-specific code (fork, exec, keyboard) inside handlers.rs - Replace Cpu::halt_with_interrupts() and Cpu::without_interrupts() with arch-agnostic wrappers (arch_halt_with_interrupts, arch_without_interrupts) - Fix copy_from_user to use validate_user_buffer for broader address validation - Update ARM64 dispatch table to route I/O syscalls through shared handlers ELF loader cleanup: - Convert per-page serial_println! noise to log::trace! (invisible at default log levels); summary messages downgraded to log::debug! Co-Authored-By: Ryan Breen <ryanbreen@gmail.com> Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add terminal bell to bsh that plays a short 880 Hz chime on ambiguous or empty tab completions. Uses lazy audio init and fixed-point sine synthesis. Also removes debug logging from the MMIO sound driver and fixes VirtIO sound format/rate enum values (S16=5, RATE_44100=6). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Test plan
🤖 Generated with Claude Code