Skip to content

chore(security): re-enable Dependabot vulnerability alerts and schedule dependency audits #236

Closed

Description

@abruption

What happened

Security and stability hardening audit finding H09 / P3 (2026-09-28). Priority is an audit recommendation, not a claim of a production incident.

The repository currently has no continuous signal for newly disclosed vulnerabilities in its dependencies:

  • gh api repos/abruption/session-peer/vulnerability-alerts returns 404 (alerts disabled).
  • gh api repos/abruption/session-peer/dependabot/alerts returns 403 "Dependabot alerts are disabled for this repository".
  • security_and_analysis.dependabot_security_updates is disabled.

#189 (913e800) removed .github/dependabot.yml to stop version-update pull requests and stated that it "does not disable dependency vulnerability alerts". That statement is accurate for the file change, but alerts are nonetheless off in repository settings now. .github/requirements/build.txt still says "Dependabot updates this file".

pip-audit and npm audit --omit=dev run only when CI or a release runs: ci.yml triggers on push to main and pull_request with no schedule, and publish.yml audits at release time. A vulnerability disclosed between commits is not reported.

What you expected

  • Dependabot vulnerability alerts enabled, independently of the decision to keep version updates off.
  • A weekly scheduled audit (pip-audit over the relay,mcp runtime set and npm audit --omit=dev for control/) that reports without blocking unrelated PRs.
  • The stale build.txt comment corrected.

Environment

  • Baseline: published session-peer v1.0.2, commit 2046836822b88e53246de73851186598604cb5ec.
  • Reproduction: read-only GitHub REST API queries and source review; no settings were changed.
  • Audit environment: macOS arm64, Python 3.14.7.
  • No live message tests or production mutation.

Source evidence

Acceptance criteria

  • Vulnerability alerts enabled and confirmed via the API; the setting is documented in SECURITY.md or RELEASING.md.
  • Scheduled audit lives in a separate workflow (a new ci.yml job would be forced into release-gate), uses commit-pinned actions and permissions: contents: read, and is added to the pinning test.
  • build.txt comment reflects how pins are actually maintained.

Scope and limitations

Target: v1.0.3 hardening work. This issue does not authorize a release or operating-service changes; enabling alerts is an owner decision and does not reintroduce version-update PRs.

Related, not duplicates: #117 (release gates and audits), #189 (Dependabot version updates removed), #140 (a former Dependabot PR).

Checked

  • Searched existing issues for duplicates.
  • Removed credentials, conversation contents, private paths and real device/session identifiers.
  • Used isolated fixtures without weakening inbound permissions.

Activity

  1. added this to the v1.0.3 milestone on Sep 28, 2026
  2. abruption commented on Sep 28, 2026

    @abruption
    OwnerAuthor

    Dependabot vulnerability alerts are enabled again (2026-09-28): PUT /vulnerability-alerts returned 204 and GET now returns 204.

    Remaining acceptance items (deferred until pending release work lands, to avoid unrelated workflow churn):

    • Separate scheduled audit workflow with commit-pinned actions and permissions: contents: read, added to the pinning test.
    • Document the setting in SECURITY.md or RELEASING.md.
    • Update the build.txt comment to reflect how pins are maintained.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions