Skip to content

chore(governance): add Dependabot + CODEOWNERS, sanitize archived IPs (#822) - #825

Merged
vybe merged 1 commit into
devfrom
feature/822-dependabot-codeowners
May 13, 2026
Merged

chore(governance): add Dependabot + CODEOWNERS, sanitize archived IPs (#822)#825
vybe merged 1 commit into
devfrom
feature/822-dependabot-codeowners

Conversation

@AndriiPasternak31

Copy link
Copy Markdown
Contributor

Fixes #822

Summary

Two governance additions plus a docs sanitization sweep:

  1. Dependabot (weekly) for github-actions, npm (frontend + mcp-server), and pip (backend). Keeps SHA-pinned actions from rotting and surfaces CVEs on the same cadence as PR security: Dependabot high/critical vulnerability remediation (umbrella) #793's prior dependency cleanup.
  2. CODEOWNERS covers .github/workflows/, .github/dependabot.yml, and .github/CODEOWNERS itself — the surfaces a hijacker would touch to remove the protections added in the companion SHA-pin PR. Branch-protection "require review from code owners" must be enabled in repo settings for this to block merges; on its own CODEOWNERS only auto-requests review.
  3. Archived plan doc referenced 100.64.0.1 as an example Tailscale IP — replaced with 203.0.113.1 (RFC 5737 TEST-NET-3, never-routable). CLAUDE.md forbids internal addresses in this public tree even in archived docs.

What changed

  • .github/dependabot.yml (new): weekly updates for 4 ecosystems
  • .github/CODEOWNERS (new): @AndriiPasternak31 ownership over CI/governance surfaces
  • docs/archive/plans/ssh-access-mcp-tool.md: 100.64.0.1 → 203.0.113.1

Follow-up required (admin)

Branch protection on main and dev needs "Require review from Code Owners" enabled for CODEOWNERS to actually gate merges. Without that, this PR only auto-requests review.

Test plan

  • Dependabot config is valid YAML — GitHub will validate on merge
  • CODEOWNERS syntax is @username form, recognized by GitHub
  • No live IPs remain in docs/ (grep'd for 100.64\., 10\., 192\.168\.)

Out of scope

This PR was previously bundled into #798 (circuit-breaker fix) — split out for isolated governance review.

🤖 Generated with Claude Code

Dependabot (weekly) for github-actions, npm (frontend + mcp-server), and
pip (backend). Keeps SHA-pinned actions from rotting and surfaces CVEs
on the same cadence as PR #793's prior dependency cleanup.

CODEOWNERS covers .github/workflows/, .github/dependabot.yml, and
.github/CODEOWNERS itself — the surfaces a hijacker would touch to
remove the protections added in the prior commit. Branch-protection
"require review from code owners" must be enabled in repo settings for
this to block merges; on its own CODEOWNERS only auto-requests review.

Archived plan doc referenced 100.64.0.1 as an example Tailscale IP —
replaced with 203.0.113.1 (RFC 5737 TEST-NET-3, never-routable). CLAUDE.md
forbids internal addresses in this public tree even in archived docs.

@vybe vybe left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Clean governance companion to #826. Dependabot cadence is correct, CODEOWNERS scopes to the right surfaces. Consider adding a core maintainer to CODEOWNERS and enabling 'Require review from Code Owners' in branch protection.

@vybe
vybe merged commit bba45f8 into dev May 13, 2026
19 checks passed
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