Skip to content

memcache - chore: defense - scaffold security docs - #105

Merged
jaredwray merged 3 commits into
mainfrom
cursor/defense-security-docs-2602
Aug 17, 2026
Merged

memcache - chore: defense - scaffold security docs#105
jaredwray merged 3 commits into
mainfrom
cursor/defense-security-docs-2602

Conversation

@jaredwray

@jaredwray jaredwray commented Aug 17, 2026

Copy link
Copy Markdown
Owner

Summary

Scaffold the public security policy and the defense-in-depth checklist so this repo can be hardened one item at a time against the latest catalog.

Status update

DEFENSE_IN_DEPTH.md: § 1 Security docs → (PR #105 pending)

Reconciled already-true items (no matching PR): packageManager pin (pnpm@11.5.2), no Dependabot config, workflow permissions: contents: read, no pull_request_target, accurate repository.url, Aikido GitHub app on builds, Socket GitHub app on PRs. Existing pnpm cooldown is 2 days (2880), not the catalog's 7-day baseline — that stays unchecked for a later PR.

Changes

  • Replace SECURITY.md with private-disclosure reporting plus a "How this repository is secured" summary (no control bullets until those sections land).
  • Add DEFENSE_IN_DEPTH.md from the current catalog (Profile: npm library · public).

Verification

  • SECURITY.md and DEFENSE_IN_DEPTH.md present and internally linked
  • Followed the Contributing and Code of Conduct guidelines
  • Tests for the changes have been added (for bug fixes/features) with 100% code coverage — docs-only; no code or test changes

Reference

defense-in-depth-nodejs § 1

Open in Web Open in Cursor 

cursoragent and others added 3 commits August 17, 2026 22:09
Replace the public security policy with private-disclosure reporting and add
the defense-in-depth checklist, reconciling items already true in this repo.

Co-authored-by: Jared Wray <me@jaredwray.com>
Co-authored-by: Jared Wray <me@jaredwray.com>
Co-authored-by: Jared Wray <me@jaredwray.com>
@codecov

codecov Bot commented Aug 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (589c733) to head (7bba6c5).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #105   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            8         8           
  Lines         1399      1399           
  Branches       303       308    +5     
=========================================
  Hits          1399      1399           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@jaredwray
jaredwray marked this pull request as ready for review August 17, 2026 22:15

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7bba6c59d8

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread DEFENSE_IN_DEPTH.md
Comment on lines +9 to +10
- [ ] `SECURITY.md` present — contact info + "How this repository is secured" summary (PR #105 pending)
- [ ] `DEFENSE_IN_DEPTH.md` present (this file) (PR #105 pending)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Mark the newly added security documents as present

When this commit lands, both checklist requirements are already satisfied by this same diff, but they remain unchecked and permanently labeled “PR #105 pending.” Because this file is presented as the repository's security-progress tracker, the merged document will incorrectly report that its own security policy and checklist are missing; mark these entries complete rather than recording transient PR state.

Useful? React with 👍 / 👎.

@jaredwray
jaredwray merged commit b8203e5 into main Aug 17, 2026
15 checks passed
@jaredwray
jaredwray deleted the cursor/defense-security-docs-2602 branch August 17, 2026 22:51
jaredwray added a commit that referenced this pull request Aug 17, 2026
* chore: defense - add CODEOWNERS for high-risk paths

Require @jaredwray review on /.github/, /.cursor/, /.devcontainer/, and
/scripts/. Reconcile § 1 security docs as merged in PR #105.

Co-authored-by: Jared Wray <me@jaredwray.com>

* chore: defense - record PR #106 on CODEOWNERS checklist

Co-authored-by: Jared Wray <me@jaredwray.com>

---------

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
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