PR #91

73 votes · 56 up · 17 down

View on GitHub
73
Total Votes
+56
Upvotes
-17
Downvotes
+56-17

Comments(3)

bpottleComment#91Rewrite it in Rust (for real)

Yep, and for the record, I think the PR should absolutely be downvoted to hell in favor of others if that's the consensus. I wrote it because it was fun for me regardless, whether or not it makes it through doesn't really factor in. Once that PR for daily merges passes, overhauls like this are going to become wildly impractical.

embedding-shapesComment#91Rewrite it in Rust (for real)

I thought https://github.com/skridlevsky/openchaos/pull/13 was bad with it's addition of 500 lines + countless of more lines via 3rd party dependencies, but this:

image

Holy moly. I realize most of it is coming from the .lock file, but that just means even more code to having to care about. Why are people obsessed with making things more complicated, instead of making things simpler? Do engineering (chaos or not, simple still matters), not whatever this cultcoding is.

openchaos-bot[bot]Comment#91Rewrite it in Rust (for real)

🤖 OpenChaos Bot

Summary: This PR introduces Rust-based backend functionality to the OpenChaos project, including server setup with axum, dependencies management via Cargo.lock, and gitignore entries.

Files changed: 5 (/.gitignore, /Cargo.lock, /Cargo.toml..etc)

Impact: High - Introduces a new backend language/framework which will likely have a large architectural impact to the OpenChaos project.

⚠️ Large PR - partial review

openchaos-bot

All Activity(79)

bpottleComment#91Rewrite it in Rust (for real)

Yep, and for the record, I think the PR should absolutely be downvoted to hell in favor of others if that's the consensus. I wrote it because it was fun for me regardless, whether or not it makes it through doesn't really factor in. Once that PR for daily merges passes, overhauls like this are going to become wildly impractical.

embedding-shapesComment#91Rewrite it in Rust (for real)

I thought https://github.com/skridlevsky/openchaos/pull/13 was bad with it's addition of 500 lines + countless of more lines via 3rd party dependencies, but this:

image

Holy moly. I realize most of it is coming from the .lock file, but that just means even more code to having to care about. Why are people obsessed with making things more complicated, instead of making things simpler? Do engineering (chaos or not, simple still matters), not whatever this cultcoding is.

openchaos-bot[bot]Comment#91Rewrite it in Rust (for real)

🤖 OpenChaos Bot

Summary: This PR introduces Rust-based backend functionality to the OpenChaos project, including server setup with axum, dependencies management via Cargo.lock, and gitignore entries.

Files changed: 5 (/.gitignore, /Cargo.lock, /Cargo.toml..etc)

Impact: High - Introduces a new backend language/framework which will likely have a large architectural impact to the OpenChaos project.

⚠️ Large PR - partial review

openchaos-bot

bpottlePR closed#91

Rewrite it in Rust (for real)

Ported the site from Next.js to Rust using Axum. The GitHub API client logic was reimplemented in src/github.rs using reqwest for async HTTP and futures for parallel reaction fetching.

The Axum server (src/main.rs) exposes an /api/pulls endpoint that replaces the original Next.js API route. It handles CORS, serves static files, and returns PR data with voting tallies.

tokio, tower-http, reqwest, serde

The Next.js frontend remains intact (src/app/, src/components/) but can now call the Rust backend.

image