Skip to content

Fix review findings and quality gates - #144

Merged
zerox80 merged 2 commits into
mainfrom
codex/fix-review-findings
Jun 28, 2026
Merged

Fix review findings and quality gates#144
zerox80 merged 2 commits into
mainfrom
codex/fix-review-findings

Conversation

@zerox80

@zerox80 zerox80 commented Jun 28, 2026

Copy link
Copy Markdown
Owner

Summary

  • fix guest comment rate limiting by storing and querying a stable rate_limit_key
  • make admin tutorial comments use authenticated claims instead of trusting author payloads
  • align admin upload body limits with 10MB uploads plus multipart overhead
  • clean up clippy/eslint findings, duplicate form data, dynamic icon rendering, and test harness setup

Verification

  • cargo fmt --manifest-path backend/Cargo.toml
  • cargo clippy --manifest-path backend/Cargo.toml --all-targets -- -D warnings
  • cargo test --manifest-path backend/Cargo.toml -j 1
  • npm.cmd run lint
  • npm.cmd test -- --run --pool forks --maxWorkers=1
  • npm.cmd run build
  • git diff --check

@zerox80
zerox80 marked this pull request as ready for review June 28, 2026 07:07
@zerox80
zerox80 merged commit f6e1ec8 into main Jun 28, 2026
3 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.

1 participant