Skip to content

chore: maximize repo quality — tests, security, DX, release config#2

Merged
kevinnft merged 1 commit into
mainfrom
chore/repo-maximization
May 12, 2026
Merged

chore: maximize repo quality — tests, security, DX, release config#2
kevinnft merged 1 commit into
mainfrom
chore/repo-maximization

Conversation

@kevinnft
Copy link
Copy Markdown
Owner

Improvements

🧪 Tests (First!)

  • error.rs: 4 unit tests for AppError variants
  • models/mod.rs: Serialization test template
  • CI now runs cargo test on every PR

🔒 Security

  • CSP hardening: tauri.conf.json now restricts connect-src, script-src, img-src
  • No more csp: null — blocks XSS and unauthorized connections

📦 Release Config

  • Bundle targets: deb, appimage, msi, nsis
  • Proper NSIS installer config with language selector
  • Linux .deb dependencies properly declared

🛠️ Developer Experience

  • .editorconfig: consistent formatting across Rust/TS/MD
  • rustfmt.toml: 100 char width, Unix newlines, import reorder
  • clippy.toml: disallow unwrap()/expect() — enforces proper error handling

Summary

6 files changed, 109 insertions — all improvements, zero breaking changes.

Improvements:
- Rust tests: error.rs (4 unit tests for AppError variants)
- Models: mod.rs (serialization test template)
- tauri.conf.json: CSP hardening (restrict sources), proper bundle targets (deb, appimage, msi, nsis)
- .editorconfig: consistent formatting across Rust/TS/MD
- rustfmt.toml: Rust formatting rules
- clippy.toml: disallow unwrap()/expect() patterns
@kevinnft kevinnft merged commit 130bdd0 into main May 12, 2026
0 of 2 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