Skip to content

fix(security): update docs after security hardening — repo is public, branch protection enabled, secret scanning on - #13

Merged
qmarcelle merged 1 commit into
mainfrom
fix/security-hardening-post-public
Jul 28, 2026
Merged

fix(security): update docs after security hardening — repo is public, branch protection enabled, secret scanning on#13
qmarcelle merged 1 commit into
mainfrom
fix/security-hardening-post-public

Conversation

@qmarcelle

Copy link
Copy Markdown
Contributor

Summary

Security hardening was applied via the GitHub API on 2026-07-28. This PR updates the committed documentation to match the actual state.

Changes

  • SECURITY.md: Removed the maintainer note about private vulnerability reporting not being live. It is now enabled.
  • docs/repository-settings.md: Updated to reflect:
    • Visibility is now public (was private)
    • Branch protection is enabled on main (was none)
    • Secret scanning + push protection are enabled
    • Private vulnerability reporting is enabled
    • Replaced the "what to enable when public" checklist with the current measured state
    • Updated the branch protection section with the actual configuration

Verification

gh api repos/workspacejson/standard/branches/main/protection --jq '{
  enforce_admins: .enforce_admins.enabled,
  pr_reviews: .required_pull_request_reviews.required_approving_review_count,
  conv_resolution: .required_conversation_resolution.enabled,
  force_pushes: .allow_force_pushes.enabled,
  deletions: .allow_deletions.enabled,
  status_checks: [.required_status_checks.contexts[]]
}'

Returns:

{"conv_resolution":true,"deletions":false,"enforce_admins":true,"force_pushes":false,"pr_reviews":1,"status_checks":["test (20)","test (22)","Four-path producer conformance"]}

Notes

These are documentation-only changes — the actual settings were applied via the GitHub API. The docs need to match so that drift is detectable.

… branch protection enabled, secret scanning on

- Remove SECURITY.md maintainer note: private vulnerability reporting is now enabled
- Update docs/repository-settings.md: visibility is public, branch protection is enabled,
  secret scanning + push protection are enabled, private vuln reporting is enabled
- Replace the 'what to enable when public' checklist with the current measured state
- Update the branch protection section to reflect what is actually configured
Copilot AI review requested due to automatic review settings July 28, 2026 17:45

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@qmarcelle qmarcelle self-assigned this Jul 28, 2026
@qmarcelle
qmarcelle merged commit 802ebda into main Jul 28, 2026
3 checks passed
@qmarcelle
qmarcelle deleted the fix/security-hardening-post-public branch July 28, 2026 18:03
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