Skip to content

TKW: resolve repository audit findings#42

Merged
codebend3r merged 5 commits into
mainfrom
fix/audit-findings
Jul 12, 2026
Merged

TKW: resolve repository audit findings#42
codebend3r merged 5 commits into
mainfrom
fix/audit-findings

Conversation

@codebend3r

@codebend3r codebend3r commented Jul 12, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add repository-wide content integrity validation — duplicate slugs, filename/frontmatter mismatches, broken cross-collection references
  • Contain keyboard focus within the site menu by exposing it as a modal dialog
  • Replace the vulnerable content-parser dependencies and clear bun audit
  • Refresh the README to match the current stack, routes, and deployment

Content integrity

  • New lib/content-integrity.ts validates unique slugs, filename-vs-frontmatter slugs, and links across collections
  • Correct malformed and stale entity references
  • Add placeholder records for referenced-but-unwritten characters
  • Broken links now fail tests with source and field diagnostics

Accessibility

  • Expose the open SiteMenu drawer as a modal dialog
  • Loop forward and reverse tab focus within its controls

Security

  • Replace gray-matter / js-yaml 3 with the maintained yaml parser
  • Override Next's vulnerable transitive PostCSS pin
  • Retain frontmatter parsing coverage; bun audit --prod reports no vulnerabilities

Docs

  • Document the Bun workflow, routes, and project layout
  • Document content integrity and placeholder conventions
  • Explain static export and Netlify deployment

@netlify

netlify Bot commented Jul 12, 2026

Copy link
Copy Markdown

Deploy Preview for theknownworld ready!

Name Link
🔨 Latest commit cb7237e
🔍 Latest deploy log https://app.netlify.com/projects/theknownworld/deploys/6a541bb4b374900008061c29
😎 Deploy Preview https://deploy-preview-42--theknownworld.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

- correct malformed and stale entity references\n- add placeholder entries for referenced but unwritten characters\n- validate unique slugs, filename slugs, and cross-collection links
- expose the open drawer as a modal dialog\n- loop forward and reverse tab focus within its controls\n- cover dialog semantics and keyboard containment
- replace gray-matter and js-yaml 3 with the maintained yaml parser\n- override Next's vulnerable PostCSS pin\n- retain frontmatter parsing coverage and clear bun audit
- describe the Bun workflow, routes, and project layout\n- document content integrity and placeholder conventions\n- explain static export and Netlify deployment
@codebend3r
codebend3r force-pushed the fix/audit-findings branch from 3fac6a8 to ea6d6dc Compare July 12, 2026 22:46
@codebend3r
codebend3r marked this pull request as ready for review July 12, 2026 22:49
- refresh all 7 captures in `docs/screenshots/` at 1440px full-page
- reflects current `/`, `/houses`, house, character, and castle pages
@codebend3r
codebend3r merged commit b6863c8 into main Jul 12, 2026
5 checks passed
@codebend3r
codebend3r deleted the fix/audit-findings branch July 12, 2026 23:57
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.

1 participant