Skip to content

Harden Bazel PGP audit runfiles and cleanup - #1130

Draft
phlax with Copilot wants to merge 3 commits into
bazel-gpgfrom
copilot/bazel-gpg-yet-again
Draft

Harden Bazel PGP audit runfiles and cleanup#1130
phlax with Copilot wants to merge 3 commits into
bazel-gpgfrom
copilot/bazel-gpg-yet-again

Conversation

Copilot AI commented Sep 6, 2026

Copy link
Copy Markdown

Final cosmetic cleanup for the hermetic OpenPGP audit: remove ambiguous runfile resolution and ensure temporary reports are always cleaned up without changing audit behavior or security guarantees.

  • Bazel/Starlark
    • Consolidate audit_bzl into a single bzl_library dependency.
    • Remove the unnecessary _main workspace fallback.
  • Audit launchers
    • Restrict runfile resolution to $RUNFILES_DIR and launcher-adjacent runfiles.
    • Store reports under one trap-managed temporary directory.
  • Live audit
    • Build //pgp/audit:audit and execute its generated launcher directly with explicit runfiles.
  • Documentation
    • Clarify key digest action-key behavior, passphrase re-execution semantics, and hermetic jq usage.
bazel build //pgp/audit:audit
RUNFILES_DIR="$(bazel info bazel-bin)/pgp/audit/audit.sh.runfiles" \
  "$(bazel info bazel-bin)/pgp/audit/audit.sh" --aquery-json "$tmp/aquery.json"

phlax and others added 2 commits September 6, 2026 14:37
Signed-off-by: Ryan Northey <ryan@synca.io>
Co-authored-by: phlax <454682+phlax@users.noreply.github.com>
Copilot AI changed the title [WIP] Cleanup bazel/pgp/ for final audit pass Harden Bazel PGP audit runfiles and cleanup Sep 6, 2026
Copilot AI requested a review from phlax September 6, 2026 14:09
@phlax
phlax force-pushed the bazel-gpg branch 21 times, most recently from fcd89b3 to e7c2599 Compare September 7, 2026 09:40
@phlax
phlax force-pushed the bazel-gpg branch 8 times, most recently from 92a62c4 to 51d46ed Compare September 8, 2026 15:13
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.

2 participants