Commit b170a6d
fix: code review audit fixes — security, error handling, and repo hygiene
Addresses 12 of 20 items from @mrtwebdesign code review (P1-AUDIT-MATT.md):
Security & repo hygiene:
- Remove committed npx/playwright symlinks, add to .gitignore
- Remove stale BACKLOG-DEPRECATED.md and ROADMAP-DEPRECATED.md
- Add plaintext password warning to AGENTS.md sensitive data section
- Add 1MB Content-Length limit on MCP HTTP transport (413 response)
Error handling:
- Preserve error cause chain in withResourceError() (index.ts)
- Return structured error for missing WPCC binary instead of crash (wpcc.ts)
- Guard against undefined site name with actionable message (local-wp.ts)
Code quality:
- Add JSDoc to regex patterns in allowlist.ts, tmux.ts, wpcc.ts
- Fix POSIX compliance: &> to >/dev/null 2>&1 (wp-ajax-test/install.sh)
- Add timeout 10 wrap on WP-CLI call (theme-crash-loop.sh)
- Add file-existence guard for curl timeout (theme-crash-loop.sh)
- Bump MCP server README version 0.6.2 → 0.6.3
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>1 parent 7f75be8 commit b170a6d
15 files changed
Lines changed: 169 additions & 559 deletions
File tree
- PROJECT/1-INBOX
- experimental
- tools
- mcp-server
- src
- handlers
- security
- wp-ajax-test
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
70 | 70 | | |
71 | 71 | | |
72 | 72 | | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
461 | 461 | | |
462 | 462 | | |
463 | 463 | | |
464 | | - | |
| 464 | + | |
465 | 465 | | |
466 | 466 | | |
467 | 467 | | |
| |||
This file was deleted.
0 commit comments