Add documentation sections and fix Parallels Tools sudo elevation#6
Open
DAEMON-404 wants to merge 3 commits into
Open
Add documentation sections and fix Parallels Tools sudo elevation#6DAEMON-404 wants to merge 3 commits into
DAEMON-404 wants to merge 3 commits into
Conversation
Documents the opt-in pt-install flow (attach CD -> pt-install -> reboot), what the helper fixes automatically (noexec remount, kernel headers, libfuse2t64 shim), and the kernel-header abort recovery path. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01QZaFM9UGEhsfHQnrD7HD3m
… works The vendored copy shipped inside the package is mode 644 and voidwalker runs it as 'bash parallels-tools.sh'. The self-elevation re-exec'd 'sudo -E ... "$0"', which makes sudo treat the non-executable path as a command and die with 'command not found'. Re-exec through 'bash "$0"' so it works regardless of the file's exec bit. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01QZaFM9UGEhsfHQnrD7HD3m
Adds an overview of the three folded-in engines, an arsenal/categories section, the full BSPWM rice + zsh/tmux environment (deploy flags, what it installs, the 'not Starship' prompt note, login steps and a keymap table), a pimpmykali kali-fix section, pe shell-sugar/fzf widgets, and shodan/pt command cards. Adds kbd/pill/feature-list styles and the matching nav links. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01QZaFM9UGEhsfHQnrD7HD3m
There was a problem hiding this comment.
Pull request overview
This PR updates the project’s documentation landing page to describe the “three engines” concept (arsenal, pentest-env, kali hardening), adds a few small UI styles to support those new sections (keycaps/pills/feature lists), and fixes Parallels Tools helper self-elevation so it works even when the script isn’t marked executable (e.g., invoked via bash parallels-tools.sh).
Changes:
- Fixed Parallels Tools installer self-elevation by re-executing via
sudo … bash "$0"instead ofsudo … "$0". - Expanded
docs/index.htmlwith new “overview”, “arsenal”, “rice”, “kali fix”, and “parallels” sections and updated navigation labels/anchors. - Added CSS for
<kbd>, pill badges, and feature lists used by the new documentation content.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
voidwalker/integrations/assets/pentest-env/config/scripts/parallels-tools.sh |
Makes sudo self-elevation robust when the script lacks the executable bit by invoking via bash. |
docs/index.html |
Adds new documentation sections, navigation updates, and supporting CSS styles for the docs site. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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
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
This PR expands the documentation site with new sections covering the three core engines (arsenal, pentest environment, and Kali hardening), adds styling for keybindings and feature lists, and fixes a critical bug in the Parallels Tools installer where
sudo "$0"fails when the script lacks the executable bit.Key Changes
Documentation & Styling:
<kbd>elements, pill-style badges (.pills), and feature lists (.flist)pt)Bug Fix:
parallels-tools.shsudo elevation: changedexec sudo -E PT_MOUNT="$MOUNT_TARGET" "$0"toexec sudo -E PT_MOUNT="$MOUNT_TARGET" bash "$0"to handle cases where the script is invoked without the executable bit set (e.g., when vendored inside the voidwalker package and run viabash parallels-tools.sh)Implementation Details
sudo "$0"treats a non-executable path as a command name and fails with "command not found", whilesudo bash "$0"explicitly invokes the interpreter.seccontainers with.sec__head,.sec__sub, and supporting elementshttps://claude.ai/code/session_01QZaFM9UGEhsfHQnrD7HD3m