fix: prevent learning mode from collapsing read paths to $HOME

Files directly under ~ (e.g., ~/.gitignore, ~/.npmrc) were collapsed to the home directory, defeating sandboxing. Now keeps exact file paths when the parent directory would be $HOME.
docs: add Linux deny-by-default lessons to experience.md
2026-02-13 11:38:51 -06:00 · 2026-02-12 20:16:37 -06:00 · 2026-02-12 20:15:40 -06:00
22 changed files with 150 additions and 1005 deletions
--- a/.gitea/workflows/benchmark.yml
+++ b/.gitea/workflows/benchmark.yml
@@ -1,101 +0,0 @@
-name: Benchmarks
-
-on:
-  workflow_dispatch:
-    inputs:
-      min_runs:
-        description: "Minimum benchmark runs"
-        required: false
-        default: "30"
-      quick:
-        description: "Quick mode (fewer runs)"
-        required: false
-        default: "false"
-        type: boolean
-
-jobs:
-  benchmark-linux:
-    name: Benchmark (Linux)
-    runs-on: ubuntu-latest
-    steps:
-      - name: Checkout
-        uses: actions/checkout@v4
-
-      - name: Set up Go
-        uses: actions/setup-go@v5
-        with:
-          go-version-file: go.mod
-          cache: true
-
-      - name: Set up Python
-        uses: actions/setup-python@v5
-        with:
-          python-version: "3.12"
-
-      - name: Set up Node
-        uses: actions/setup-node@v4
-        with:
-          node-version: "20"
-
-      - name: Download dependencies
-        run: go mod download
-
-      - name: Install dependencies
-        run: |
-          sudo apt-get update
-          sudo apt-get install -y \
-            bubblewrap \
-            socat \
-            uidmap \
-            curl \
-            netcat-openbsd \
-            ripgrep \
-            hyperfine \
-            jq \
-            bc
-          # Configure subuid/subgid
-          echo "$(whoami):100000:65536" | sudo tee -a /etc/subuid
-          echo "$(whoami):100000:65536" | sudo tee -a /etc/subgid
-          sudo chmod u+s $(which bwrap)
-
-      - name: Install benchstat
-        run: go install golang.org/x/perf/cmd/benchstat@latest
-
-      - name: Build greywall
-        run: make build-ci
-
-      - name: Run Go microbenchmarks
-        run: |
-          mkdir -p benchmarks
-          go test -run=^$ -bench=. -benchmem -count=10 ./internal/sandbox/... | tee benchmarks/go-bench-linux.txt
-
-      - name: Run CLI benchmarks
-        run: |
-          MIN_RUNS="${{ github.event.inputs.min_runs || '30' }}"
-          QUICK="${{ github.event.inputs.quick || 'false' }}"
-
-          if [[ "$QUICK" == "true" ]]; then
-            ./scripts/benchmark.sh -q -o benchmarks
-          else
-            ./scripts/benchmark.sh -n "$MIN_RUNS" -o benchmarks
-          fi
-
-      - name: Upload benchmark results
-        uses: actions/upload-artifact@v4
-        with:
-          name: benchmark-results-linux
-          path: benchmarks/
-          retention-days: 30
-
-      - name: Display results
-        run: |
-          echo "=== Linux Benchmark Results ==="
-          echo ""
-
-          for f in benchmarks/*.md; do
-            [[ -f "$f" ]] && cat "$f"
-          done
-
-          echo ""
-          echo "=== Go Microbenchmarks ==="
-          grep -E '^Benchmark|^ok|^PASS' benchmarks/go-bench-linux.txt | head -50 || true
--- a/.gitea/workflows/main.yml
+++ b/.gitea/workflows/main.yml
@@ -1,115 +0,0 @@
-name: Build and test
-
-on:
-  push:
-    branches: [main]
-  pull_request:
-    branches: [main]
-
-jobs:
-  build:
-    name: Build
-    runs-on: ubuntu-latest
-    steps:
-      - name: Checkout
-        uses: actions/checkout@v4
-
-      - name: Set up Go
-        uses: actions/setup-go@v5
-        with:
-          go-version-file: go.mod
-          cache: true
-
-      - name: Download dependencies
-        run: go mod download
-
-      - name: Build
-        run: make build-ci
-
-  lint:
-    name: Lint
-    runs-on: ubuntu-latest
-    steps:
-      - name: Checkout
-        uses: actions/checkout@v4
-
-      - name: Set up Go
-        uses: actions/setup-go@v5
-        with:
-          go-version-file: go.mod
-          cache: true
-
-      - name: Download dependencies
-        run: go mod download
-
-      - name: Lint
-        uses: golangci/golangci-lint-action@v6
-        with:
-          install-mode: goinstall
-          version: v1.64.8
-
-  test-linux:
-    name: Test (Linux)
-    runs-on: ubuntu-latest
-    steps:
-      - name: Checkout
-        uses: actions/checkout@v4
-
-      - name: Set up Go
-        uses: actions/setup-go@v5
-        with:
-          go-version-file: go.mod
-          cache: true
-
-      - name: Set up Python
-        uses: actions/setup-python@v5
-        with:
-          python-version: "3.12"
-
-      - name: Set up Node
-        uses: actions/setup-node@v4
-        with:
-          node-version: "20"
-
-      - name: Download dependencies
-        run: go mod download
-
-      - name: Install Linux sandbox dependencies
-        run: |
-          sudo apt-get update
-          sudo apt-get install -y \
-            bubblewrap \
-            socat \
-            uidmap \
-            curl \
-            netcat-openbsd \
-            ripgrep
-          # Configure subuid/subgid for the runner user (required for unprivileged user namespaces)
-          echo "$(whoami):100000:65536" | sudo tee -a /etc/subuid
-          echo "$(whoami):100000:65536" | sudo tee -a /etc/subgid
-          # Make bwrap setuid so it can create namespaces as non-root user
-          sudo chmod u+s $(which bwrap)
-
-      - name: Verify sandbox dependencies
-        run: |
-          echo "=== Checking sandbox dependencies ==="
-          bwrap --version
-          socat -V | head -1
-          echo "User namespaces enabled: $(cat /proc/sys/kernel/unprivileged_userns_clone 2>/dev/null || echo 'check not available')"
-          echo "Kernel version: $(uname -r)"
-          echo "uidmap installed: $(which newuidmap 2>/dev/null && echo yes || echo no)"
-          echo "subuid configured: $(grep $(whoami) /etc/subuid 2>/dev/null || echo 'not configured')"
-          echo "bwrap setuid: $(ls -la $(which bwrap) | grep -q '^-rws' && echo yes || echo no)"
-          echo "=== Testing bwrap basic functionality ==="
-          bwrap --ro-bind / / -- /bin/echo "bwrap works!"
-          echo "=== Testing bwrap with user namespace ==="
-          bwrap --ro-bind / / --unshare-user --uid 0 --gid 0 -- /bin/echo "bwrap user namespace works!"
-
-      - name: Run unit and integration tests
-        run: make test-ci
-
-      - name: Build binary for smoke tests
-        run: make build-ci
-
-      - name: Run smoke tests
-        run: GREYWALL_TEST_NETWORK=1 ./scripts/smoke_test.sh ./greywall
--- a/.gitea/workflows/release.yml
+++ b/.gitea/workflows/release.yml
@@ -1,62 +0,0 @@
-name: Release
-
-on:
-  push:
-    tags:
-      - "v*"
-
-run-name: "Release ${{ github.ref_name }}"
-
-jobs:
-  goreleaser:
-    runs-on: ubuntu-latest
-    steps:
-      - name: Checkout
-        uses: actions/checkout@v4
-        with:
-          fetch-depth: 0
-
-      - name: Set up Go
-        uses: actions/setup-go@v5
-        with:
-          go-version-file: go.mod
-          cache: true
-
-      - name: Run GoReleaser
-        uses: goreleaser/goreleaser-action@v6
-        with:
-          distribution: goreleaser
-          version: "~> v2"
-          args: release --clean
-        env:
-          GITEA_TOKEN: ${{ secrets.GITEA_TOKEN }}
-          GORELEASER_FORCE_TOKEN: gitea
-
-  publish-version:
-    needs: [goreleaser]
-    runs-on: ubuntu-latest
-    steps:
-      - name: Checkout gh-pages
-        uses: actions/checkout@v4
-        with:
-          ref: gh-pages
-
-      - name: Update latest version
-        run: |
-          echo "${{ github.ref_name }}" > latest.txt
-
-          cat > latest.json << EOF
-          {
-            "version": "${{ github.ref_name }}",
-            "published_at": "$(date -u +%Y-%m-%dT%H:%M:%SZ)",
-            "url": "https://gitea.app.monadical.io/monadical/greywall/releases/tag/${{ github.ref_name }}"
-          }
-          EOF
-
-      - name: Commit and push to gh-pages
-        run: |
-          git config user.name "gitea-actions[bot]"
-          git config user.email "gitea-actions[bot]@noreply.gitea.app.monadical.io"
-          git add latest.txt latest.json
-          git commit -m "Update latest version to ${{ github.ref_name }}" || echo "No changes to commit"
-          git push origin gh-pages
--- a/.golangci.yml
+++ b/.golangci.yml
@@ -1,36 +1,8 @@
-version: "2"
 run:
+  timeout: 5m
  modules-download-mode: readonly
-linters:
-  default: none
-  enable:
-    - errcheck
-    - gocritic
-    - gosec
-    - govet
-    - ineffassign
-    - misspell
-    - revive
-    - staticcheck
-    - unused
-  settings:
-    gocritic:
-      disabled-checks:
-        - singleCaseSwitch
-    revive:
-      rules:
-        - name: exported
-          disabled: true
-  exclusions:
-    generated: lax
-    paths:
-      - third_party$
-      - builtin$
-      - examples$
-formatters:
-  enable:
-    - gofumpt
-  settings:
+
+linters-settings:
  gci:
    sections:
      - standard
@@ -39,11 +11,30 @@ formatters:
  gofmt:
    simplify: true
  goimports:
-      local-prefixes:
-        - gitea.app.monadical.io/monadical/greywall
-  exclusions:
-    generated: lax
-    paths:
-      - third_party$
-      - builtin$
-      - examples$
+    local-prefixes: gitea.app.monadical.io/monadical/greywall
+  gocritic:
+    disabled-checks:
+      - singleCaseSwitch
+  revive:
+    rules:
+      - name: exported
+        disabled: true
+
+linters:
+  enable-all: false
+  disable-all: true
+  enable:
+    - staticcheck
+    - errcheck
+    - gosimple
+    - govet
+    - unused
+    - ineffassign
+    - gosec
+    - gocritic
+    - revive
+    - gofumpt
+    - misspell
+
+issues:
+  exclude-use-default: false
--- a/.goreleaser.yaml
+++ b/.goreleaser.yaml
@@ -1,10 +1,5 @@
 version: 2

-gitea_urls:
-  api: https://gitea.app.monadical.io/api/v1
-  download: https://gitea.app.monadical.io
-  skip_tls_verify: false
-
 before:
  hooks:
    - go mod tidy
@@ -47,7 +42,7 @@ checksum:

 changelog:
  sort: asc
-  use: gitea
+  use: github
  format: "{{ .SHA }}: {{ .Message }}{{ with .AuthorUsername }} (@{{ . }}){{ end }}"
  filters:
    exclude:
@@ -81,7 +76,7 @@ changelog:
      order: 9999

 release:
-  gitea:
+  github:
    owner: monadical
    name: greywall
  draft: false
--- a/CLAUDE.md
+++ b/CLAUDE.md
@@ -54,7 +54,7 @@ scripts/               Smoke tests, benchmarks, release

 - **Language:** Go 1.25+
 - **Formatter:** `gofumpt` (enforced in CI)
- **Linter:** `golangci-lint` v2 (config in `.golangci.yml`)
+- **Linter:** `golangci-lint` v1.64.8 (config in `.golangci.yml`)
 - **Import order:** stdlib, third-party, local (`gitea.app.monadical.io/monadical/greywall`)
 - **Platform code:** build tags (`//go:build linux`, `//go:build darwin`) with `*_stub.go` for unsupported platforms
 - **Error handling:** custom error types (e.g., `CommandBlockedError`)
--- a/2
+++ b/2
@@ -70,7 +70,7 @@ build-darwin:
 install-lint-tools:
 	@echo "Installing linting tools..."
 	go install mvdan.cc/gofumpt@latest
-	go install github.com/golangci/golangci-lint/v2/cmd/golangci-lint@latest
+	go install github.com/golangci/golangci-lint/cmd/golangci-lint@latest
 	@echo "Linting tools installed"

 setup: deps install-lint-tools
--- a/README.md
+++ b/README.md
@@ -2,20 +2,20 @@

 **The sandboxing layer of the GreyHaven platform.**

-Greywall wraps commands in a sandbox that blocks network access by default and restricts filesystem operations. On Linux, it uses tun2socks for truly transparent proxying: all TCP/UDP traffic is captured at the kernel level via a TUN device and forwarded through an external SOCKS5 proxy. No application awareness needed.
+Greywall wraps commands in a sandbox that blocks network access by default and restricts filesystem operations. It is the core sandboxing component of the GreyHaven platform, providing defense-in-depth for running untrusted code.

 ```bash
-# Block all network access (default — no proxy running = no connectivity)
-greywall -- curl https://example.com
+# Block all network access (default)
+greywall curl https://example.com  # → 403 Forbidden

-# Route traffic through an external SOCKS5 proxy
-greywall --proxy socks5://localhost:1080 -- curl https://example.com
+# Allow specific domains
+greywall -t code npm install  # → uses 'code' template with npm/pypi/etc allowed

 # Block dangerous commands
 greywall -c "rm -rf /"  # → blocked by command deny rules
 ```

-Greywall also works as a permission manager for CLI agents. See [agents.md](./docs/agents.md) for integration with Claude Code, Codex, Gemini CLI, OpenCode, and others.
+Greywall also works as a permission manager for CLI agents. **Greywall works with popular coding agents like Claude Code, Codex, Gemini CLI, Cursor Agent, OpenCode, Factory (Droid) CLI, etc.** See [agents.md](./docs/agents.md) for more details.

 ## Install

@@ -39,123 +39,83 @@ go install gitea.app.monadical.io/monadical/greywall/cmd/greywall@latest
 ```bash
 git clone https://gitea.app.monadical.io/monadical/greywall
 cd greywall
-make setup && make build
+go build -o greywall ./cmd/greywall
 ```

 </details>

-**Linux dependencies:**
+**Additional requirements for Linux:**

- `bubblewrap` — container-free sandboxing (required)
- `socat` — network bridging (required)
-
-Check dependency status with `greywall --version`.
+- `bubblewrap` (for sandboxing)
+- `socat` (for network bridging)
+- `bpftrace` (optional, for filesystem violation visibility when monitoring with `-m`)

 ## Usage

-### Basic commands
+### Basic

 ```bash
-# Run with all network blocked (default)
-greywall -- curl https://example.com
+# Run command with all network blocked (no domains allowed by default)
+greywall curl https://example.com

 # Run with shell expansion
 greywall -c "echo hello && ls"

-# Route through a SOCKS5 proxy
-greywall --proxy socks5://localhost:1080 -- npm install
-
-# Expose a port for inbound connections (e.g., dev servers)
-greywall -p 3000 -c "npm run dev"
-
 # Enable debug logging
-greywall -d -- curl https://example.com
+greywall -d curl https://example.com

-# Monitor sandbox violations
-greywall -m -- npm install
+# Use a template
+greywall -t code -- claude  # Runs Claude Code using `code` template config

-# Show available Linux security features
-greywall --linux-features
+# Monitor mode (shows violations)
+greywall -m npm install

-# Show version and dependency status
-greywall --version
-```
-
-### Learning mode
-
-Greywall can trace a command's filesystem access and generate a config template automatically:
-
-```bash
-# Run in learning mode — traces file access via strace
-greywall --learning -- opencode
-
-# List generated templates
-greywall templates list
-
-# Show a template's content
-greywall templates show opencode
-
-# Next run auto-loads the learned template
-greywall -- opencode
+# Show all commands and options
+greywall --help
 ```

 ### Configuration

 Greywall reads from `~/.config/greywall/greywall.json` by default (or `~/Library/Application Support/greywall/greywall.json` on macOS).

-```jsonc
+```json
 {
-  // Route traffic through an external SOCKS5 proxy
-  "network": {
-    "proxyUrl": "socks5://localhost:1080",
-    "dnsAddr": "localhost:5353"
-  },
-  // Control filesystem access
-  "filesystem": {
-    "defaultDenyRead": true,
-    "allowRead": ["~/.config/myapp"],
-    "allowWrite": ["."],
-    "denyWrite": ["~/.ssh/**"],
-    "denyRead": ["~/.ssh/id_*", ".env"]
-  },
-  // Block dangerous commands
-  "command": {
-    "deny": ["git push", "npm publish"]
-  }
+  "extends": "code",
+  "network": { "allowedDomains": ["private.company.com"] },
+  "filesystem": { "allowWrite": ["."] },
+  "command": { "deny": ["git push", "npm publish"] }
 }
 ```

-Use `greywall --settings ./custom.json` to specify a different config file.
+Use `greywall --settings ./custom.json` to specify a different config.

-By default, traffic routes through the GreyProxy SOCKS5 proxy at `localhost:43052` with DNS via `localhost:43053`.
+### Import from Claude Code
+
+```bash
+greywall import --claude --save
+```

 ## Features

- **Transparent proxy** — All TCP/UDP traffic captured at the kernel level via tun2socks and routed through an external SOCKS5 proxy (Linux)
- **Network isolation** — All outbound blocked by default; traffic only flows when a proxy is available
- **Filesystem restrictions** — Deny-by-default read mode, controlled write paths, sensitive file protection
- **Learning mode** — Trace filesystem access with strace and auto-generate config templates
- **Command blocking** — Deny dangerous commands (`rm -rf /`, `git push`, `shutdown`, etc.)
- **SSH filtering** — Control which hosts and commands are allowed over SSH
- **Environment hardening** — Strips dangerous env vars (`LD_PRELOAD`, `DYLD_*`, etc.)
- **Violation monitoring** — Real-time logging of sandbox violations (`-m`)
- **Shell completions** — `greywall completion bash|zsh|fish|powershell`
- **Cross-platform** — Linux (bubblewrap + seccomp + Landlock + eBPF) and macOS (sandbox-exec)
+- **Network isolation** - All outbound blocked by default; allowlist domains via config
+- **Filesystem restrictions** - Control read/write access paths
+- **Command blocking** - Deny dangerous commands like `rm -rf /`, `git push`
+- **SSH Command Filtering** - Control which hosts and commands are allowed over SSH
+- **Built-in templates** - Pre-configured rulesets for common workflows
+- **Violation monitoring** - Real-time logging of blocked requests (`-m`)
+- **Cross-platform** - macOS (sandbox-exec) + Linux (bubblewrap)

-Greywall can also be used as a [Go package](docs/library.md).
+Greywall can be used as a Go package or CLI tool.

 ## Documentation

- [Documentation Index](docs/README.md)
+- [Index](/docs/README.md)
 - [Quickstart Guide](docs/quickstart.md)
- [Why Greywall](docs/why-greywall.md)
 - [Configuration Reference](docs/configuration.md)
 - [Security Model](docs/security-model.md)
 - [Architecture](ARCHITECTURE.md)
- [Linux Security Features](docs/linux-security-features.md)
- [AI Agent Integration](docs/agents.md)
 - [Library Usage (Go)](docs/library.md)
- [Troubleshooting](docs/troubleshooting.md)
+- [Examples](examples/)

 ## Attribution

--- a/cmd/greywall/main.go
+++ b/cmd/greywall/main.go
@@ -15,7 +15,6 @@ import (

 	"gitea.app.monadical.io/monadical/greywall/internal/config"
 	"gitea.app.monadical.io/monadical/greywall/internal/platform"
-	"gitea.app.monadical.io/monadical/greywall/internal/proxy"
 	"gitea.app.monadical.io/monadical/greywall/internal/sandbox"
 	"github.com/spf13/cobra"
 )
@@ -56,8 +55,8 @@ func main() {
 		Long: `greywall is a command-line tool that runs commands in a sandboxed environment
 with network and filesystem restrictions.

-By default, traffic is routed through the GreyProxy SOCKS5 proxy at localhost:43052
-with DNS via localhost:43053. Use --proxy and --dns to override, or configure in
+By default, traffic is routed through the GreyHaven SOCKS5 proxy at localhost:42051
+with DNS via localhost:42053. Use --proxy and --dns to override, or configure in
 your settings file at ~/.config/greywall/greywall.json (or ~/Library/Application Support/greywall/greywall.json on macOS).

 On Linux, greywall uses tun2socks for truly transparent proxying: all TCP/UDP traffic
@@ -99,8 +98,8 @@ Configuration file format:
 	rootCmd.Flags().BoolVarP(&debug, "debug", "d", false, "Enable debug logging")
 	rootCmd.Flags().BoolVarP(&monitor, "monitor", "m", false, "Monitor and log sandbox violations")
 	rootCmd.Flags().StringVarP(&settingsPath, "settings", "s", "", "Path to settings file (default: OS config directory)")
-	rootCmd.Flags().StringVar(&proxyURL, "proxy", "", "External SOCKS5 proxy URL (default: socks5://localhost:43052)")
-	rootCmd.Flags().StringVar(&dnsAddr, "dns", "", "DNS server address on host (default: localhost:43053)")
+	rootCmd.Flags().StringVar(&proxyURL, "proxy", "", "External SOCKS5 proxy URL (default: socks5://localhost:42052)")
+	rootCmd.Flags().StringVar(&dnsAddr, "dns", "", "DNS server address on host (default: localhost:42053)")
 	rootCmd.Flags().StringVarP(&cmdString, "c", "c", "", "Run command string directly (like sh -c)")
 	rootCmd.Flags().StringArrayVarP(&exposePorts, "port", "p", nil, "Expose port for inbound connections (can be used multiple times)")
 	rootCmd.Flags().BoolVarP(&showVersion, "version", "v", false, "Show version information")
@@ -112,8 +111,6 @@ Configuration file format:

 	rootCmd.AddCommand(newCompletionCmd(rootCmd))
 	rootCmd.AddCommand(newTemplatesCmd())
-	rootCmd.AddCommand(newCheckCmd())
-	rootCmd.AddCommand(newSetupCmd())

 	if err := rootCmd.Execute(); err != nil {
 		fmt.Fprintf(os.Stderr, "Error: %v\n", err)
@@ -124,7 +121,11 @@ Configuration file format:

 func runCommand(cmd *cobra.Command, args []string) error {
 	if showVersion {
-		fmt.Printf("greywall %s\n", version)
+		fmt.Printf("greywall - lightweight, container-free sandbox for running untrusted commands\n")
+		fmt.Printf("  Version: %s\n", version)
+		fmt.Printf("  Built:   %s\n", buildTime)
+		fmt.Printf("  Commit:  %s\n", gitCommit)
+		sandbox.PrintDependencyStatus()
 		return nil
 	}

@@ -202,20 +203,19 @@ func runCommand(cmd *cobra.Command, args []string) error {

 		if templatePath != "" {
 			learnedCfg, loadErr := config.Load(templatePath)
-			switch {
-			case loadErr != nil:
+			if loadErr != nil {
 				if debug {
 					fmt.Fprintf(os.Stderr, "[greywall] Warning: failed to load learned template: %v\n", loadErr)
 				}
-			case learnedCfg != nil:
+			} else if learnedCfg != nil {
 				cfg = config.Merge(cfg, learnedCfg)
 				if debug {
 					fmt.Fprintf(os.Stderr, "[greywall] Auto-loaded learned template for %q\n", templateLabel)
 				}
-			case templateName != "":
+			} else if templateName != "" {
 				// Explicit --template but file doesn't exist
 				return fmt.Errorf("learned template %q not found at %s\nRun: greywall templates list", templateName, templatePath)
-			case cmdName != "":
+			} else if cmdName != "" {
 				// No template found for this command - suggest creating one
 				fmt.Fprintf(os.Stderr, "[greywall] No learned template for %q. Run with --learning to create one.\n", cmdName)
 			}
@@ -230,18 +230,18 @@ func runCommand(cmd *cobra.Command, args []string) error {
 		cfg.Network.DnsAddr = dnsAddr
 	}

-	// GreyProxy defaults: when no proxy or DNS is configured (neither via CLI
-	// nor config file), use the standard GreyProxy ports.
+	// GreyHaven defaults: when no proxy or DNS is configured (neither via CLI
+	// nor config file), use the standard GreyHaven infrastructure ports.
 	if cfg.Network.ProxyURL == "" {
-		cfg.Network.ProxyURL = "socks5://localhost:43052"
+		cfg.Network.ProxyURL = "socks5://localhost:42052"
 		if debug {
-			fmt.Fprintf(os.Stderr, "[greywall] Defaulting proxy to socks5://localhost:43052\n")
+			fmt.Fprintf(os.Stderr, "[greywall] Defaulting proxy to socks5://localhost:42052\n")
 		}
 	}
 	if cfg.Network.DnsAddr == "" {
-		cfg.Network.DnsAddr = "localhost:43053"
+		cfg.Network.DnsAddr = "localhost:42053"
 		if debug {
-			fmt.Fprintf(os.Stderr, "[greywall] Defaulting DNS to localhost:43053\n")
+			fmt.Fprintf(os.Stderr, "[greywall] Defaulting DNS to localhost:42053\n")
 		}
 	}

@@ -411,89 +411,6 @@ func extractCommandName(args []string, cmdStr string) string {
 	return filepath.Base(name)
 }

-// newCheckCmd creates the check subcommand for diagnostics.
-func newCheckCmd() *cobra.Command {
-	return &cobra.Command{
-		Use:   "check",
-		Short: "Check greywall status, dependencies, and greyproxy connectivity",
-		Long: `Run diagnostics to check greywall readiness.
-
-Shows version information, platform dependencies, security features,
-and greyproxy installation/running status.`,
-		Args: cobra.NoArgs,
-		RunE: runCheck,
-	}
-}
-
-func runCheck(_ *cobra.Command, _ []string) error {
-	fmt.Printf("greywall - lightweight, container-free sandbox for running untrusted commands\n")
-	fmt.Printf("  Version: %s\n", version)
-	fmt.Printf("  Built:   %s\n", buildTime)
-	fmt.Printf("  Commit:  %s\n", gitCommit)
-
-	sandbox.PrintDependencyStatus()
-
-	fmt.Printf("\n  Greyproxy:\n")
-	status := proxy.Detect()
-	if status.Installed {
-		if status.Version != "" {
-			fmt.Printf("    ✓ installed (v%s) at %s\n", status.Version, status.Path)
-		} else {
-			fmt.Printf("    ✓ installed at %s\n", status.Path)
-		}
-		if status.Running {
-			fmt.Printf("    ✓ running (SOCKS5 :43052, DNS :43053, Dashboard :43080)\n")
-		} else {
-			fmt.Printf("    ✗ not running\n")
-			fmt.Printf("      Start with: greywall setup\n")
-		}
-	} else {
-		fmt.Printf("    ✗ not installed\n")
-		fmt.Printf("      Install with: greywall setup\n")
-	}
-
-	return nil
-}
-
-// newSetupCmd creates the setup subcommand for installing greyproxy.
-func newSetupCmd() *cobra.Command {
-	return &cobra.Command{
-		Use:   "setup",
-		Short: "Install and start greyproxy (network proxy for sandboxed commands)",
-		Long: `Downloads and installs greyproxy from GitHub releases.
-
-greyproxy provides SOCKS5 proxying and DNS resolution for sandboxed commands.
-The installer will:
-  1. Download the latest greyproxy release for your platform
-  2. Install the binary to ~/.local/bin/greyproxy
-  3. Register and start a systemd user service`,
-		Args: cobra.NoArgs,
-		RunE: runSetup,
-	}
-}
-
-func runSetup(_ *cobra.Command, _ []string) error {
-	status := proxy.Detect()
-
-	if status.Installed && status.Running {
-		fmt.Printf("greyproxy is already installed (v%s) and running.\n", status.Version)
-		fmt.Printf("Run 'greywall check' for full status.\n")
-		return nil
-	}
-
-	if status.Installed && !status.Running {
-		if err := proxy.Start(os.Stderr); err != nil {
-			return err
-		}
-		fmt.Printf("greyproxy started.\n")
-		return nil
-	}
-
-	return proxy.Install(proxy.InstallOptions{
-		Output: os.Stderr,
-	})
-}
-
 // newCompletionCmd creates the completion subcommand for shell completions.
 func newCompletionCmd(rootCmd *cobra.Command) *cobra.Command {
 	cmd := &cobra.Command{
@@ -586,7 +503,7 @@ Examples:
 		RunE: func(cmd *cobra.Command, args []string) error {
 			name := args[0]
 			templatePath := sandbox.LearnedTemplatePath(name)
-			data, err := os.ReadFile(templatePath) //nolint:gosec // user-specified template path - intentional
+			data, err := os.ReadFile(templatePath)
 			if err != nil {
 				if os.IsNotExist(err) {
 					return fmt.Errorf("template %q not found\nRun: greywall templates list", name)
--- a/internal/proxy/detect.go
+++ b/internal/proxy/detect.go
@@ -1,126 +0,0 @@
-// Package proxy provides greyproxy detection, installation, and lifecycle management.
-package proxy
-
-import (
-	"context"
-	"encoding/json"
-	"fmt"
-	"net/http"
-	"os/exec"
-	"regexp"
-	"strings"
-	"time"
-)
-
-const (
-	healthURL     = "http://localhost:43080/api/health"
-	healthTimeout = 2 * time.Second
-	cmdTimeout    = 5 * time.Second
-)
-
-// GreyproxyStatus holds the detected state of greyproxy.
-type GreyproxyStatus struct {
-	Installed  bool   // found via exec.LookPath
-	Path       string // full path from LookPath
-	Version    string // parsed version (e.g. "0.1.1")
-	Running    bool   // health endpoint responded with valid greyproxy response
-	RunningErr error  // error from the running check (for diagnostics)
-}
-
-// healthResponse is the expected JSON from GET /api/health.
-type healthResponse struct {
-	Service string         `json:"service"`
-	Version string         `json:"version"`
-	Status  string         `json:"status"`
-	Ports   map[string]int `json:"ports"`
-}
-
-var versionRegex = regexp.MustCompile(`^greyproxy\s+(\S+)`)
-
-// Detect checks greyproxy installation status, version, and whether it's running.
-// This function never returns an error; all detection failures are captured
-// in the GreyproxyStatus fields so the caller can present them diagnostically.
-func Detect() *GreyproxyStatus {
-	s := &GreyproxyStatus{}
-
-	// 1. Check if installed
-	s.Path, s.Installed = checkInstalled()
-
-	// 2. Check if running (via health endpoint)
-	running, ver, err := checkRunning()
-	s.Running = running
-	s.RunningErr = err
-	if running && ver != "" {
-		s.Version = ver
-	}
-
-	// 3. Version fallback: if installed but version not yet known, parse from CLI
-	if s.Installed && s.Version == "" {
-		s.Version, _ = checkVersion(s.Path)
-	}
-
-	return s
-}
-
-// checkInstalled uses exec.LookPath to find greyproxy on PATH.
-func checkInstalled() (path string, found bool) {
-	p, err := exec.LookPath("greyproxy")
-	if err != nil {
-		return "", false
-	}
-	return p, true
-}
-
-// checkVersion runs "greyproxy -V" and parses the output.
-// Expected format: "greyproxy 0.1.1 (go1.x linux/amd64)"
-func checkVersion(binaryPath string) (string, error) {
-	ctx, cancel := context.WithTimeout(context.Background(), cmdTimeout)
-	defer cancel()
-
-	out, err := exec.CommandContext(ctx, binaryPath, "-V").Output() //nolint:gosec // binaryPath comes from exec.LookPath
-	if err != nil {
-		return "", fmt.Errorf("failed to run greyproxy -V: %w", err)
-	}
-
-	matches := versionRegex.FindStringSubmatch(strings.TrimSpace(string(out)))
-	if len(matches) < 2 {
-		return "", fmt.Errorf("unexpected version output: %s", strings.TrimSpace(string(out)))
-	}
-	return matches[1], nil
-}
-
-// checkRunning hits GET http://localhost:43080/api/health and verifies
-// the response is from greyproxy (not some other service on that port).
-// Returns running status, version string from health response, and any error.
-func checkRunning() (bool, string, error) {
-	client := &http.Client{Timeout: healthTimeout}
-
-	ctx, cancel := context.WithTimeout(context.Background(), healthTimeout)
-	defer cancel()
-
-	req, err := http.NewRequestWithContext(ctx, http.MethodGet, healthURL, nil)
-	if err != nil {
-		return false, "", fmt.Errorf("failed to create request: %w", err)
-	}
-
-	resp, err := client.Do(req) //nolint:gosec // healthURL is a hardcoded localhost constant
-	if err != nil {
-		return false, "", fmt.Errorf("health check failed: %w", err)
-	}
-	defer func() { _ = resp.Body.Close() }()
-
-	if resp.StatusCode != http.StatusOK {
-		return false, "", fmt.Errorf("health check returned status %d", resp.StatusCode)
-	}
-
-	var health healthResponse
-	if err := json.NewDecoder(resp.Body).Decode(&health); err != nil {
-		return false, "", fmt.Errorf("failed to parse health response: %w", err)
-	}
-
-	if health.Service != "greyproxy" {
-		return false, "", fmt.Errorf("unexpected service: %q (expected greyproxy)", health.Service)
-	}
-
-	return true, health.Version, nil
-}
--- a/internal/proxy/install.go
+++ b/internal/proxy/install.go
@@ -1,258 +0,0 @@
-package proxy
-
-import (
-	"archive/tar"
-	"compress/gzip"
-	"context"
-	"encoding/json"
-	"fmt"
-	"io"
-	"net/http"
-	"os"
-	"os/exec"
-	"path/filepath"
-	"runtime"
-	"strings"
-	"time"
-)
-
-const (
-	githubOwner = "greyhavenhq"
-	githubRepo  = "greyproxy"
-	apiTimeout  = 15 * time.Second
-)
-
-// release represents a GitHub release.
-type release struct {
-	TagName string  `json:"tag_name"`
-	Assets  []asset `json:"assets"`
-}
-
-// asset represents a release asset.
-type asset struct {
-	Name               string `json:"name"`
-	BrowserDownloadURL string `json:"browser_download_url"`
-}
-
-// InstallOptions controls the greyproxy installation behavior.
-type InstallOptions struct {
-	Output io.Writer // progress output (typically os.Stderr)
-}
-
-// Install downloads the latest greyproxy release and runs "greyproxy install".
-func Install(opts InstallOptions) error {
-	if opts.Output == nil {
-		opts.Output = os.Stderr
-	}
-
-	// 1. Fetch latest release
-	_, _ = fmt.Fprintf(opts.Output, "Fetching latest greyproxy release...\n")
-	rel, err := fetchLatestRelease()
-	if err != nil {
-		return fmt.Errorf("failed to fetch latest release: %w", err)
-	}
-	ver := strings.TrimPrefix(rel.TagName, "v")
-	_, _ = fmt.Fprintf(opts.Output, "Latest version: %s\n", ver)
-
-	// 2. Find the correct asset for this platform
-	assetURL, assetName, err := resolveAssetURL(rel)
-	if err != nil {
-		return err
-	}
-	_, _ = fmt.Fprintf(opts.Output, "Downloading %s...\n", assetName)
-
-	// 3. Download to temp file
-	archivePath, err := downloadAsset(assetURL)
-	if err != nil {
-		return fmt.Errorf("download failed: %w", err)
-	}
-	defer func() { _ = os.Remove(archivePath) }()
-
-	// 4. Extract
-	_, _ = fmt.Fprintf(opts.Output, "Extracting...\n")
-	extractDir, err := extractTarGz(archivePath)
-	if err != nil {
-		return fmt.Errorf("extraction failed: %w", err)
-	}
-	defer func() { _ = os.RemoveAll(extractDir) }()
-
-	// 5. Find the greyproxy binary in extracted content
-	binaryPath := filepath.Join(extractDir, "greyproxy")
-	if _, err := os.Stat(binaryPath); err != nil {
-		return fmt.Errorf("greyproxy binary not found in archive")
-	}
-
-	// 6. Shell out to "greyproxy install"
-	_, _ = fmt.Fprintf(opts.Output, "\n")
-	if err := runGreyproxyInstall(binaryPath); err != nil {
-		return fmt.Errorf("greyproxy install failed: %w", err)
-	}
-
-	// 7. Verify
-	_, _ = fmt.Fprintf(opts.Output, "\nVerifying installation...\n")
-	status := Detect()
-	if status.Installed {
-		_, _ = fmt.Fprintf(opts.Output, "greyproxy %s installed at %s\n", status.Version, status.Path)
-		if status.Running {
-			_, _ = fmt.Fprintf(opts.Output, "greyproxy is running.\n")
-		}
-	} else {
-		_, _ = fmt.Fprintf(opts.Output, "Warning: greyproxy not found on PATH after install.\n")
-		_, _ = fmt.Fprintf(opts.Output, "Ensure ~/.local/bin is in your PATH.\n")
-	}
-
-	return nil
-}
-
-// fetchLatestRelease queries the GitHub API for the latest greyproxy release.
-func fetchLatestRelease() (*release, error) {
-	apiURL := fmt.Sprintf("https://api.github.com/repos/%s/%s/releases/latest", githubOwner, githubRepo)
-
-	client := &http.Client{Timeout: apiTimeout}
-
-	ctx, cancel := context.WithTimeout(context.Background(), apiTimeout)
-	defer cancel()
-
-	req, err := http.NewRequestWithContext(ctx, http.MethodGet, apiURL, nil)
-	if err != nil {
-		return nil, err
-	}
-	req.Header.Set("Accept", "application/vnd.github+json")
-	req.Header.Set("User-Agent", "greywall-setup")
-
-	resp, err := client.Do(req) //nolint:gosec // apiURL is built from hardcoded constants
-	if err != nil {
-		return nil, fmt.Errorf("GitHub API request failed: %w", err)
-	}
-	defer func() { _ = resp.Body.Close() }()
-
-	if resp.StatusCode != http.StatusOK {
-		return nil, fmt.Errorf("GitHub API returned status %d", resp.StatusCode)
-	}
-
-	var rel release
-	if err := json.NewDecoder(resp.Body).Decode(&rel); err != nil {
-		return nil, fmt.Errorf("failed to parse release response: %w", err)
-	}
-	return &rel, nil
-}
-
-// resolveAssetURL finds the correct asset download URL for the current OS/arch.
-func resolveAssetURL(rel *release) (downloadURL, name string, err error) {
-	ver := strings.TrimPrefix(rel.TagName, "v")
-	osName := runtime.GOOS
-	archName := runtime.GOARCH
-
-	expected := fmt.Sprintf("greyproxy_%s_%s_%s.tar.gz", ver, osName, archName)
-
-	for _, a := range rel.Assets {
-		if a.Name == expected {
-			return a.BrowserDownloadURL, a.Name, nil
-		}
-	}
-	return "", "", fmt.Errorf("no release asset found for %s/%s (expected: %s)", osName, archName, expected)
-}
-
-// downloadAsset downloads a URL to a temp file, returning its path.
-func downloadAsset(downloadURL string) (string, error) {
-	client := &http.Client{Timeout: 5 * time.Minute}
-
-	ctx, cancel := context.WithTimeout(context.Background(), 5*time.Minute)
-	defer cancel()
-
-	req, err := http.NewRequestWithContext(ctx, http.MethodGet, downloadURL, nil)
-	if err != nil {
-		return "", err
-	}
-
-	resp, err := client.Do(req) //nolint:gosec // downloadURL comes from GitHub API response
-	if err != nil {
-		return "", err
-	}
-	defer func() { _ = resp.Body.Close() }()
-
-	if resp.StatusCode != http.StatusOK {
-		return "", fmt.Errorf("download returned status %d", resp.StatusCode)
-	}
-
-	tmpFile, err := os.CreateTemp("", "greyproxy-*.tar.gz")
-	if err != nil {
-		return "", err
-	}
-
-	if _, err := io.Copy(tmpFile, resp.Body); err != nil {
-		_ = tmpFile.Close()
-		_ = os.Remove(tmpFile.Name()) //nolint:gosec // tmpFile.Name() is from os.CreateTemp, not user input
-		return "", err
-	}
-	_ = tmpFile.Close()
-
-	return tmpFile.Name(), nil
-}
-
-// extractTarGz extracts a .tar.gz archive to a temp directory, returning the dir path.
-func extractTarGz(archivePath string) (string, error) {
-	f, err := os.Open(archivePath) //nolint:gosec // archivePath is a temp file we created
-	if err != nil {
-		return "", err
-	}
-	defer func() { _ = f.Close() }()
-
-	gz, err := gzip.NewReader(f)
-	if err != nil {
-		return "", fmt.Errorf("failed to create gzip reader: %w", err)
-	}
-	defer func() { _ = gz.Close() }()
-
-	tmpDir, err := os.MkdirTemp("", "greyproxy-extract-*")
-	if err != nil {
-		return "", err
-	}
-
-	tr := tar.NewReader(gz)
-	for {
-		header, err := tr.Next()
-		if err == io.EOF {
-			break
-		}
-		if err != nil {
-			_ = os.RemoveAll(tmpDir)
-			return "", fmt.Errorf("tar read error: %w", err)
-		}
-
-		// Sanitize: only extract regular files with safe names
-		name := filepath.Base(header.Name)
-		if name == "." || name == ".." || strings.Contains(header.Name, "..") {
-			continue
-		}
-
-		target := filepath.Join(tmpDir, name) //nolint:gosec // name is sanitized via filepath.Base and path traversal check above
-
-		switch header.Typeflag {
-		case tar.TypeReg:
-			out, err := os.OpenFile(target, os.O_CREATE|os.O_WRONLY|os.O_TRUNC, os.FileMode(header.Mode)) //nolint:gosec // mode from tar header of trusted archive
-			if err != nil {
-				_ = os.RemoveAll(tmpDir)
-				return "", err
-			}
-			if _, err := io.Copy(out, io.LimitReader(tr, 256<<20)); err != nil { // 256 MB limit per file
-				_ = out.Close()
-				_ = os.RemoveAll(tmpDir)
-				return "", err
-			}
-			_ = out.Close()
-		}
-	}
-
-	return tmpDir, nil
-}
-
-// runGreyproxyInstall shells out to the extracted greyproxy binary with "install" arg.
-// Stdin/stdout/stderr are passed through so the interactive [y/N] prompt works.
-func runGreyproxyInstall(binaryPath string) error {
-	cmd := exec.Command(binaryPath, "install") //nolint:gosec // binaryPath is from our extracted archive
-	cmd.Stdin = os.Stdin
-	cmd.Stdout = os.Stdout
-	cmd.Stderr = os.Stderr
-	return cmd.Run()
-}
--- a/internal/proxy/start.go
+++ b/internal/proxy/start.go
@@ -1,30 +0,0 @@
-package proxy
-
-import (
-	"fmt"
-	"io"
-	"os"
-	"os/exec"
-)
-
-// Start runs "greyproxy service start" to start the greyproxy service.
-func Start(output io.Writer) error {
-	if output == nil {
-		output = os.Stderr
-	}
-
-	path, found := checkInstalled()
-	if !found {
-		return fmt.Errorf("greyproxy not found on PATH")
-	}
-
-	_, _ = fmt.Fprintf(output, "Starting greyproxy service...\n")
-	cmd := exec.Command(path, "service", "start") //nolint:gosec // path comes from exec.LookPath
-	cmd.Stdout = os.Stdout
-	cmd.Stderr = os.Stderr
-	if err := cmd.Run(); err != nil {
-		return fmt.Errorf("failed to start greyproxy service: %w", err)
-	}
-
-	return nil
-}
--- a/internal/sandbox/learning.go
+++ b/internal/sandbox/learning.go
@@ -166,11 +166,11 @@ func GenerateLearnedTemplate(straceLogPath, cmdName string, debug bool) (string,

 	// Save template
 	templatePath := LearnedTemplatePath(cmdName)
-	if err := os.MkdirAll(filepath.Dir(templatePath), 0o750); err != nil {
+	if err := os.MkdirAll(filepath.Dir(templatePath), 0o755); err != nil {
 		return "", fmt.Errorf("failed to create template directory: %w", err)
 	}

-	if err := os.WriteFile(templatePath, []byte(template), 0o600); err != nil {
+	if err := os.WriteFile(templatePath, []byte(template), 0o644); err != nil {
 		return "", fmt.Errorf("failed to write template: %w", err)
 	}

@@ -305,7 +305,11 @@ func isSensitivePath(path, home string) bool {

 	// Check GPG
 	gnupgDir := filepath.Join(home, ".gnupg")
-	return strings.HasPrefix(path, gnupgDir+"/")
+	if strings.HasPrefix(path, gnupgDir+"/") {
+		return true
+	}
+
+	return false
 }

 // getDangerousFilePatterns returns denyWrite entries for DangerousFiles.
--- a/internal/sandbox/learning_linux.go
+++ b/internal/sandbox/learning_linux.go
@@ -41,7 +41,7 @@ func ParseStraceLog(logPath string, debug bool) (*StraceResult, error) {
 	if err != nil {
 		return nil, fmt.Errorf("failed to open strace log: %w", err)
 	}
-	defer func() { _ = f.Close() }()
+	defer f.Close()

 	home, _ := os.UserHomeDir()
 	seenWrite := make(map[string]bool)
--- a/internal/sandbox/learning_linux_test.go
+++ b/internal/sandbox/learning_linux_test.go
@@ -127,7 +127,7 @@ func TestParseStraceLog(t *testing.T) {
 	}, "\n")

 	logFile := filepath.Join(t.TempDir(), "strace.log")
-	if err := os.WriteFile(logFile, []byte(logContent), 0o600); err != nil {
+	if err := os.WriteFile(logFile, []byte(logContent), 0o644); err != nil {
 		t.Fatal(err)
 	}

--- a/internal/sandbox/learning_test.go
+++ b/internal/sandbox/learning_test.go
@@ -70,7 +70,9 @@ func TestFindApplicationDirectory(t *testing.T) {

 func TestCollapsePaths(t *testing.T) {
 	// Temporarily override home for testing
-	t.Setenv("HOME", "/home/testuser")
+	origHome := os.Getenv("HOME")
+	os.Setenv("HOME", "/home/testuser")
+	defer os.Setenv("HOME", origHome)

 	tests := []struct {
 		name        string
@@ -317,7 +319,9 @@ func TestToTildePath(t *testing.T) {
 func TestListLearnedTemplates(t *testing.T) {
 	// Use a temp dir to isolate from real user config
 	tmpDir := t.TempDir()
-	t.Setenv("XDG_CONFIG_HOME", tmpDir)
+	origConfigDir := os.Getenv("XDG_CONFIG_HOME")
+	os.Setenv("XDG_CONFIG_HOME", tmpDir)
+	defer os.Setenv("XDG_CONFIG_HOME", origConfigDir)

 	// Initially empty
 	templates, err := ListLearnedTemplates()
@@ -330,18 +334,10 @@ func TestListLearnedTemplates(t *testing.T) {

 	// Create some templates
 	dir := LearnedTemplateDir()
-	if err := os.MkdirAll(dir, 0o750); err != nil {
-		t.Fatal(err)
-	}
-	if err := os.WriteFile(filepath.Join(dir, "opencode.json"), []byte("{}"), 0o600); err != nil {
-		t.Fatal(err)
-	}
-	if err := os.WriteFile(filepath.Join(dir, "myapp.json"), []byte("{}"), 0o600); err != nil {
-		t.Fatal(err)
-	}
-	if err := os.WriteFile(filepath.Join(dir, "notjson.txt"), []byte(""), 0o600); err != nil {
-		t.Fatal(err)
-	}
+	os.MkdirAll(dir, 0o755)
+	os.WriteFile(filepath.Join(dir, "opencode.json"), []byte("{}"), 0o644)
+	os.WriteFile(filepath.Join(dir, "myapp.json"), []byte("{}"), 0o644)
+	os.WriteFile(filepath.Join(dir, "notjson.txt"), []byte(""), 0o644) // should be ignored

 	templates, err = ListLearnedTemplates()
 	if err != nil {
@@ -419,7 +415,9 @@ func TestBuildTemplateNoAllowRead(t *testing.T) {
 func TestGenerateLearnedTemplate(t *testing.T) {
 	// Create a temp dir for templates
 	tmpDir := t.TempDir()
-	t.Setenv("XDG_CONFIG_HOME", tmpDir)
+	origConfigDir := os.Getenv("XDG_CONFIG_HOME")
+	os.Setenv("XDG_CONFIG_HOME", tmpDir)
+	defer os.Setenv("XDG_CONFIG_HOME", origConfigDir)

 	// Create a fake strace log
 	home, _ := os.UserHomeDir()
@@ -432,7 +430,7 @@ func TestGenerateLearnedTemplate(t *testing.T) {
 	}, "\n")

 	logFile := filepath.Join(tmpDir, "strace.log")
-	if err := os.WriteFile(logFile, []byte(logContent), 0o600); err != nil {
+	if err := os.WriteFile(logFile, []byte(logContent), 0o644); err != nil {
 		t.Fatal(err)
 	}

@@ -446,7 +444,7 @@ func TestGenerateLearnedTemplate(t *testing.T) {
 	}

 	// Read and verify template
-	data, err := os.ReadFile(templatePath) //nolint:gosec // reading test-generated template file
+	data, err := os.ReadFile(templatePath)
 	if err != nil {
 		t.Fatalf("failed to read template: %v", err)
 	}
--- a/internal/sandbox/linux.go
+++ b/internal/sandbox/linux.go
@@ -519,14 +519,8 @@ func buildDenyByDefaultMounts(cfg *config.Config, cwd string, debug bool) []stri
 			if fileExists(p) && canMountOver(p) &&
 				!strings.HasPrefix(p, "/dev/") && !strings.HasPrefix(p, "/proc/") && !boundPaths[p] {
 				boundPaths[p] = true
-				// Create intermediary dirs if needed.
-				// For files, only create dirs up to the parent to avoid
-				// creating a directory at the file's path.
-				dirTarget := p
-				if !isDirectory(p) {
-					dirTarget = filepath.Dir(p)
-				}
-				for _, dir := range intermediaryDirs("/", dirTarget) {
+				// Create intermediary dirs if needed
+				for _, dir := range intermediaryDirs("/", p) {
 					if !isSystemMountPoint(dir) {
 						args = append(args, "--dir", dir)
 					}
@@ -539,11 +533,7 @@ func buildDenyByDefaultMounts(cfg *config.Config, cwd string, debug bool) []stri
 			if !ContainsGlobChars(normalized) && fileExists(normalized) && canMountOver(normalized) &&
 				!strings.HasPrefix(normalized, "/dev/") && !strings.HasPrefix(normalized, "/proc/") && !boundPaths[normalized] {
 				boundPaths[normalized] = true
-				dirTarget := normalized
-				if !isDirectory(normalized) {
-					dirTarget = filepath.Dir(normalized)
-				}
-				for _, dir := range intermediaryDirs("/", dirTarget) {
+				for _, dir := range intermediaryDirs("/", normalized) {
 					if !isSystemMountPoint(dir) {
 						args = append(args, "--dir", dir)
 					}
@@ -564,7 +554,7 @@ func buildDenyByDefaultMounts(cfg *config.Config, cwd string, debug bool) []stri
 				if emptyFile == "" {
 					emptyFile = filepath.Join(os.TempDir(), "greywall", "empty")
 					_ = os.MkdirAll(filepath.Dir(emptyFile), 0o750)
-					_ = os.WriteFile(emptyFile, nil, 0o444) //nolint:gosec // intentionally world-readable empty file for bind-mount masking
+					_ = os.WriteFile(emptyFile, nil, 0o444)
 				}
 				args = append(args, "--ro-bind", emptyFile, p)
 				if debug {
@@ -681,28 +671,19 @@ func WrapCommandLinuxWithOptions(cfg *config.Config, command string, proxyBridge
 			bwrapArgs = append(bwrapArgs, "--bind", cwd, cwd)
 		}

-		// Make XDG_RUNTIME_DIR writable so dconf and other runtime services
-		// (Wayland, PulseAudio, D-Bus) work inside the sandbox.
-		// Writes to /run/ are already filtered out by the learning parser.
-		xdgRuntime := os.Getenv("XDG_RUNTIME_DIR")
-		if xdgRuntime != "" && fileExists(xdgRuntime) {
-			bwrapArgs = append(bwrapArgs, "--bind", xdgRuntime, xdgRuntime)
-		}
-
 	}

 	defaultDenyRead := cfg != nil && cfg.Filesystem.IsDefaultDenyRead()

-	switch {
-	case opts.Learning:
+	if opts.Learning {
 		// Skip defaultDenyRead logic in learning mode (already set up above)
-	case defaultDenyRead:
+	} else if defaultDenyRead {
 		// Deny-by-default mode: start with empty root, then whitelist system paths + CWD
 		if opts.Debug {
 			fmt.Fprintf(os.Stderr, "[greywall:linux] DefaultDenyRead mode enabled - tmpfs root with selective mounts\n")
 		}
 		bwrapArgs = append(bwrapArgs, buildDenyByDefaultMounts(cfg, cwd, opts.Debug)...)
-	default:
+	} else {
 		// Legacy mode: bind entire root filesystem read-only
 		bwrapArgs = append(bwrapArgs, "--ro-bind", "/", "/")
 	}
@@ -938,7 +919,7 @@ func WrapCommandLinuxWithOptions(cfg *config.Config, command string, proxyBridge
 					// Supported by glibc, Go 1.21+, c-ares, and most DNS resolver libraries.
 					_, _ = tmpResolv.WriteString("nameserver 1.1.1.1\nnameserver 8.8.8.8\noptions use-vc\n")
 				}
-				_ = tmpResolv.Close()
+				tmpResolv.Close()
 				dnsRelayResolvConf = tmpResolv.Name()
 				bwrapArgs = append(bwrapArgs, "--ro-bind", dnsRelayResolvConf, "/etc/resolv.conf")
 				if opts.Debug {
@@ -976,14 +957,6 @@ func WrapCommandLinuxWithOptions(cfg *config.Config, command string, proxyBridge
 		fmt.Fprintf(os.Stderr, "[greywall:linux] Skipping Landlock wrapper (running as library, not greywall CLI)\n")
 	}

-	// Bind-mount the greywall binary into the sandbox so the Landlock wrapper
-	// can re-execute it. Without this, running greywall from a directory that
-	// isn't the CWD (e.g., ~/bin/greywall from /home/user/project) would fail
-	// because the binary path doesn't exist inside the sandbox.
-	if useLandlockWrapper && greywallExePath != "" {
-		bwrapArgs = append(bwrapArgs, "--ro-bind", greywallExePath, greywallExePath)
-	}
-
 	bwrapArgs = append(bwrapArgs, "--", shellPath, "-c")

 	// Build the inner command that sets up tun2socks and runs the user command
@@ -1094,8 +1067,7 @@ sleep 0.3
 	// after the main command exits; the user can Ctrl+C to stop it.
 	// A SIGCHLD trap kills strace once its direct child exits, handling
 	// the common case of background daemons (LSP servers, watchers).
-	switch {
-	case opts.Learning && opts.StraceLogPath != "":
+	if opts.Learning && opts.StraceLogPath != "" {
 		innerScript.WriteString(fmt.Sprintf(`# Learning mode: trace filesystem access (foreground for terminal access)
 strace -f -qq -I2 -e trace=openat,open,creat,mkdir,mkdirat,unlinkat,renameat,renameat2,symlinkat,linkat -o %s -- %s
 GREYWALL_STRACE_EXIT=$?
@@ -1107,7 +1079,7 @@ exit $GREYWALL_STRACE_EXIT
 `,
 			ShellQuoteSingle(opts.StraceLogPath), command,
 		))
-	case useLandlockWrapper:
+	} else if useLandlockWrapper {
 		// Use Landlock wrapper if available
 		// Pass config via environment variable (serialized as JSON)
 		// This ensures allowWrite/denyWrite rules are properly applied
@@ -1128,7 +1100,7 @@ exit $GREYWALL_STRACE_EXIT

 		// Use exec to replace bash with the wrapper (which will exec the command)
 		innerScript.WriteString(fmt.Sprintf("exec %s\n", ShellQuote(wrapperArgs)))
-	default:
+	} else {
 		innerScript.WriteString(command)
 		innerScript.WriteString("\n")
 	}
--- a/internal/sandbox/linux_features.go
+++ b/internal/sandbox/linux_features.go
@@ -370,7 +370,7 @@ func suggestInstallCmd(features *LinuxFeatures) string {
 }

 func readSysctl(name string) string {
-	data, err := os.ReadFile("/proc/sys/" + name) //nolint:gosec // reading sysctl values - trusted kernel path
+	data, err := os.ReadFile("/proc/sys/" + name)
 	if err != nil {
 		return ""
 	}
--- a/internal/sandbox/linux_landlock.go
+++ b/internal/sandbox/linux_landlock.go
@@ -201,7 +201,7 @@ type LandlockRuleset struct {
 func NewLandlockRuleset(debug bool) (*LandlockRuleset, error) {
 	features := DetectLinuxFeatures()
 	if !features.CanUseLandlock() {
-		return nil, fmt.Errorf("landlock not available (kernel %d.%d, need 5.13+)",
+		return nil, fmt.Errorf("Landlock not available (kernel %d.%d, need 5.13+)",
 			features.KernelMajor, features.KernelMinor)
 	}

@@ -438,7 +438,7 @@ func (l *LandlockRuleset) addPathRule(path string, access uint64) error {
 // Apply applies the Landlock ruleset to the current process.
 func (l *LandlockRuleset) Apply() error {
 	if !l.initialized {
-		return fmt.Errorf("landlock ruleset not initialized")
+		return fmt.Errorf("Landlock ruleset not initialized")
 	}

 	// Set NO_NEW_PRIVS first (required for Landlock)
--- a/internal/sandbox/manager.go
+++ b/internal/sandbox/manager.go
@@ -78,7 +78,7 @@ func (m *Manager) Initialize() error {
 			bridge, err := NewProxyBridge(m.config.Network.ProxyURL, m.debug)
 			if err != nil {
 				if m.tun2socksPath != "" {
-					_ = os.Remove(m.tun2socksPath)
+					os.Remove(m.tun2socksPath)
 				}
 				return fmt.Errorf("failed to initialize proxy bridge: %w", err)
 			}
@@ -90,7 +90,7 @@ func (m *Manager) Initialize() error {
 				if err != nil {
 					m.proxyBridge.Cleanup()
 					if m.tun2socksPath != "" {
-						_ = os.Remove(m.tun2socksPath)
+						os.Remove(m.tun2socksPath)
 					}
 					return fmt.Errorf("failed to initialize DNS bridge: %w", err)
 				}
@@ -108,7 +108,7 @@ func (m *Manager) Initialize() error {
 					m.proxyBridge.Cleanup()
 				}
 				if m.tun2socksPath != "" {
-					_ = os.Remove(m.tun2socksPath)
+					os.Remove(m.tun2socksPath)
 				}
 				return fmt.Errorf("failed to initialize reverse bridge: %w", err)
 			}
@@ -166,7 +166,7 @@ func (m *Manager) wrapCommandLearning(command string) (string, error) {
 	if err != nil {
 		return "", fmt.Errorf("failed to create strace log file: %w", err)
 	}
-	_ = tmpFile.Close()
+	tmpFile.Close()
 	m.straceLogPath = tmpFile.Name()

 	m.logDebug("Strace log file: %s", m.straceLogPath)
@@ -193,7 +193,7 @@ func (m *Manager) GenerateLearnedTemplate(cmdName string) (string, error) {
 	}

 	// Clean up strace log since we've processed it
-	_ = os.Remove(m.straceLogPath)
+	os.Remove(m.straceLogPath)
 	m.straceLogPath = ""

 	return templatePath, nil
@@ -211,10 +211,10 @@ func (m *Manager) Cleanup() {
 		m.proxyBridge.Cleanup()
 	}
 	if m.tun2socksPath != "" {
-		_ = os.Remove(m.tun2socksPath)
+		os.Remove(m.tun2socksPath)
 	}
 	if m.straceLogPath != "" {
-		_ = os.Remove(m.straceLogPath)
+		os.Remove(m.straceLogPath)
 		m.straceLogPath = ""
 	}
 	m.logDebug("Sandbox manager cleaned up")
--- a/internal/sandbox/tun2socks_embed.go
+++ b/internal/sandbox/tun2socks_embed.go
@@ -38,14 +38,14 @@ func extractTun2Socks() (string, error) {
 	}

 	if _, err := tmpFile.Write(data); err != nil {
-		_ = tmpFile.Close()
-		_ = os.Remove(tmpFile.Name())
+		tmpFile.Close()
+		os.Remove(tmpFile.Name())
 		return "", fmt.Errorf("tun2socks: failed to write binary: %w", err)
 	}
-	_ = tmpFile.Close()
+	tmpFile.Close()

-	if err := os.Chmod(tmpFile.Name(), 0o755); err != nil { //nolint:gosec // executable binary needs execute permission
-		_ = os.Remove(tmpFile.Name())
+	if err := os.Chmod(tmpFile.Name(), 0o755); err != nil {
+		os.Remove(tmpFile.Name())
 		return "", fmt.Errorf("tun2socks: failed to make executable: %w", err)
 	}

--- a/scripts/release.sh
+++ b/scripts/release.sh
@@ -149,5 +149,5 @@ git push origin "$NEW_VERSION"

 echo ""
 info "✓ Released $NEW_VERSION"
-info "Gitea Actions will now build and publish the release."
+info "GitHub Actions will now build and publish the release."
 info "Watch progress at: https://gitea.app.monadical.io/monadical/greywall/actions"