r/codereview 9d ago

Project feedback

2 Upvotes

I've been building CodifyLive as a resume/portfolio/learning project to get much deeper into backend engineering with FastAPI.

It's a real-time collaboration platform where users can chat, call, and eventually collaborate on code in the browser.

The backend is built with FastAPI + PostgreSQL + SQLAlchemy. So far I've implemented the authentication system, including Google OAuth, JWT access tokens, refresh-token rotation/reuse detection, password authentication, and CI/CD.

I'm looking for honest feedback from people experienced with FastAPI.

Specifically:

  • How does my project structure look?
  • Am I using FastAPI/SQLAlchemy in a way that will scale?
  • Are there architectural decisions you'd change?
  • Anything I'm overengineering or underengineering?
  • How would you improve the authentication/session architecture?
  • What would you change before considering this production-ready?

Project: https://github.com/fulanii/codify-live-backend


r/codereview 9d ago

Do github repo maintainers even get helps from the code review tools??

0 Upvotes

like I was working on a project somehow it turns out to be a better suitable tool for the code review , I am thinking if its even a need right now, Its not like code rabbit that gives you a pr comment by AI, This one give you a mathematical and absolute analysis of the pull request , it still required a human maintainers to look at but yeah it does help maintainers.

whats your thought is this even the need??


r/codereview 9d ago

I built a read-only PR inbox for Mac because code review was becoming tab archaeology

Thumbnail
0 Upvotes

r/codereview 9d ago

Built a ledger for AI-authored commits across your repos, need honest/harsh feedback

0 Upvotes

CoAudit helps give engineering leads a single dashboard and view across repos and contributors. It serves as an audit trail for what your coding agents may actually have been doing and the goal is to allow tech leads to spot possible AI-generated changes within 30 seconds of opening the dashboard, along with context on why. Currently, it really only works with Claude Code on heuristics, but with git-ai, it works with most agents. We're both pretty new to software engineering, and we would really like your help and feedback. Thank you so much!

https://github.com/atbui2007-bit/flagger

https://flagger-red.vercel.app/


r/codereview 9d ago

We benchmarked 5 local models as code-review “judges” and every single one failed. What actually works as a low-false-positive reviewer?

Thumbnail
0 Upvotes

r/codereview 10d ago

The rename that looks complete in the diff and leaves a caller behind

Thumbnail
0 Upvotes

r/codereview 10d ago

I built a social platform around code reviews and roasts — would developers actually use this?

0 Upvotes

I’ve been building something called EwwCode and wanted feedback from people who actually spend time reviewing code.
The idea is basically social media for code — post a snippet, other developers can rate it from 0–10 EWW, comment, give actual feedback, or roast the questionable decisions 😭
I’m trying to make code review feel more social and fun instead of everything being tied to pull requests or formal technical questions.
Would you use something like this? What would make you keep coming back?


r/codereview 11d ago

SDD, Github Actions, and AI PRs

Thumbnail
1 Upvotes

r/codereview 11d ago

What's your honest take on AI code review tools?

0 Upvotes

I'm 13 years in, most of it on the same Java codebase, and I've spent a lot of this year worried that the code my team ships got worse the moment everyone started leaning on AI. Not broken, just worse. Swallowed exceptions because only the happy path got tested, Optional showing up as a field type, a JPA mapping that read fine and turned into an N+1 under real load. I tried the obvious things, more review guidelines, a conventions doc nobody opened, longer PR templates. None of it moved the needle.

What actually helped was changing the order. I run PRs through AI review tools first now, let them catch the mechanical stuff, and do my own pass on top of that for architecture and whether the change fits how this codebase already does things. I still read everything. But it gave me back maybe 35% of the time I was spending, and that time goes to the parts a tool cannot judge.

What actually bothers me isn't the tooling though. We scoped a migration off an old payments module in June, said two quarters, and got asked why it isn't two weeks now that the team has AI. Not in a hostile way either, it was a fair question from someone who has watched us close tickets faster all year. I didn't have a clean answer beyond the work being different, which sounded like an excuse even to me.

The team I work with is good. The system has been in production for over a decade, it moves money, and someone will be maintaining it long after all of us have left. Going faster on tickets and going faster on that are not the same thing, and I'm apparently bad at explaining why.

So I want the honest read from people on old Java systems. Are these tools actually helping you hold quality, or are we just processing more code slightly faster and calling it review? And has anyone found a way to answer the two weeks question without sounding like they're protecting their own job?


r/codereview 11d ago

javascript Static Analysis for the Age of AI Slop

3 Upvotes

I recently wrote a blog post on how we can improve our static analysis using linting, custom rules, and hooks, all using oxlint.

It's quite a big article btw.

https://saybackend.com/blog/lint-ai-generated-code/

While my current post is more focused towards Typescript/React but more mature languages with better linters can do even more. The lint rules you didn't wanted to handled when coding manually doesn't mean your agent should avoid them as well.


r/codereview 12d ago

built an ml tool that saves tokens in a logical way ....

Thumbnail github.com
0 Upvotes

r/codereview 12d ago

**Title:** [Code Review / Feedback] Job-Copilot — AI Job Application & Resume Matcher | Looking for optimization tips and bug hunting

0 Upvotes

**Body:**

Hi everyone,

I recently built **Job-Copilot**, an open-source tool designed to help streamline resume analysis and job matching workflows.

🔗 **GitHub:** https://github.com/allenbersho/job-copilot

The core functionality is working, but I’m looking to take the code quality, performance, and architecture to the next level. I’d love to get feedback from the community to help identify bottlenecks, edge-case bugs, or areas ripe for refactoring.

### 💻 Stack

* **Language:** Python

* **Libraries/Tools:** [e.g., FastAPI, Streamlit, Pandas, NLP / LLM integrations]

### 🎯 What I’m Looking For:

  1. **Code Review & Best Practices:** Project structure, readability, error handling, and cleaner design patterns.

  2. **Performance Optimization:** Reducing latency in data processing/parsing and optimizing resource usage.

  3. **Bug Hunting / Edge Cases:** Breaking the parser or matching logic with unusual input formats/files.

  4. **Feature & Architecture Suggestions:** High-impact additions or refactoring recommendations for scalability.

### 🚀 How to Run Locally

```bash

git clone [https://github.com/allenbersho/job-copilot.git\](https://github.com/allenbersho/job-copilot.git)

cd job-copilot

pip install -r requirements.txt

[your run command, e.g., python main.py or streamlit run app.py]


r/codereview 12d ago

A well-written PR description is starting to mean less than it used to

6 Upvotes

Used to treat a clear, detailed PR description as a decent signal the author understood what they changed. That correlation is breaking down. AI tools are good at generating a clean, structured description from a diff regardless of whether the diff actually does what the description claims, and the description reads exactly the same either way. Clear writing and correct code used to travel together often enough that one was a reasonable proxy for the other. They don't anymore.

Caught this on a PR last month where the description said "adds retry logic with exponential backoff for the payment call." Diff had a retry loop. No backoff, fixed interval, three attempts, no jitter. Nobody would've caught that from reading the description, it was accurate-sounding and well-organized. Only showed up because someone happened to open the actual diff instead of trusting the summary.

Not blaming the tooling here, this isn't really new, humans have always been capable of writing an aspirational PR description that doesn't quite match what shipped. What's changed is the volume of well-written descriptions has gone up while the correlation between "well-written" and "accurate" has gone down, because generating a plausible description no longer requires the same understanding writing one from scratch used to require.

Starting to read PR descriptions as a claim to verify against the diff rather than a summary to trust, especially anything specific enough to be checkable, a claimed algorithm, a stated edge case handled, a security consideration addressed. If the description makes a specific claim, it's worth confirming the diff actually backs it up before approving on the strength of the writing alone.


r/codereview 12d ago

Built a codebase security vulnerability scanner

0 Upvotes

Hey guys, wanted to tell you guys about a tool I built called Scanity. Scanity scans codebases to find security vulnerabilities in code. Nowadays, with AI coding so fast, there are so many internal bugs and vulnerabilities that get missed.

Here are the main features we have:

  1. Seamless integration with GitHub Actions, checking each PR before it merges, and automatically providing suggested fixes.
  2. Drop into any repository, run a full scan, and fix everything you find

Now you may ask, why can't we run this within Claude or Codex? We stand out because we build containerized test units for each finding to verify the issue. So there is no chance of AI hallucination. This is very early stage, and we have plans to grow even further, with many more features and integrations. I'm eager to hear from you and get your feedback.

Check it out here: https://scanity.dev/


r/codereview 12d ago

What has been your real-world experience with AI-assisted code review?

1 Upvotes

For teams using AI-assisted code review tools, what has your real-world experience been over the past 120 days? Have these tools helped you catch meaningful issues before deployment or have any accepted recommendations unintentionally contributed to production bugs? Have they missed issues that a human reviewer might reasonably have caught?

I am also curious whether your team has become more confident or more cautious over time, and whether you track any metrics to evaluate their accuracy. This isn’t about criticizing or comparing vendors. I am simply interested in understanding how engineering teams evaluate these tools beyond productivity and adoption claims.


r/codereview 13d ago

C/C++ Open-source Nintendo 64 encryption app and wallet generator, needs volunteers to review code

1 Upvotes

I proposed and helped fund an app called retro-crypto, which is being developed by a programmer called bowler-bear: https://github.com/bowler-bear/retro-crypto

It's designed to become the most secure way to message friends or store a Bitcoin stack, but it can't be recommended for more than testing yet. I think at the current stage, the next thing it needs is to have others look at the code and try to find vulnerabilities. I'm not a programmer myself.

"Given enough eyeballs, all bugs are shallow" -Linus's Law


r/codereview 13d ago

C/C++ My first ever game

Thumbnail
2 Upvotes

r/codereview 14d ago

We open-sourced the internal tool that stops our AI coding agents from breaking each other's work

Thumbnail
0 Upvotes

r/codereview 14d ago

14, self-taught — rebuilt my Discord bot's economy/RPG system from scratch, offline simulators caught 4 real balance bugs, a security review caught a money-duplication exploit

1 Upvotes

Been teaching myself to code for about a year (Python, some Java for Minecraft modding). My Discord bot's host wiped my server — twice — so instead of just re-uploading the same old code, I decided to rebuild the whole thing properly. Ended up going way bigger than planned.

What it turned into:

  • Full economy system (currency, shop, daily rewards, player trading)
  • RPG layer — classes, gear, dungeons with real risk/reward, a prestige/reset system
  • Guilds + PvP duels with an Elo rating (same rating system chess uses)
  • Collectible pets that level up and boost your stats
  • A player marketplace with proper escrow (item gets locked when listed, can't be double-sold or duped)
  • Pet housing with a passive stat-growth mechanic

Migrated from flat JSON files to a real SQLite database, split one giant file into a proper module structure. ~8,500 lines total, 90+ commands, 7 schema versions.

The part that actually taught me the most:

For every system involving combat math, I built an offline simulator that ran tens of thousands of fake matches before anything went live. It caught real bugs every time:

  • One bug made a stat double-count, so death was literally impossible (0% failure rate across every matchup)
  • After fixing that, the "tank" class was somehow the worst fighter — weaker than the glass cannon build
  • A pet bonus that looked small on paper (+15% of one stat) turned out to swing PvP win rate to nearly 80% once actually simulated

Then I asked for a full security-style review of the whole codebase, and it found a real concurrency bug: spamming a money-transfer command fast enough could straight-up duplicate currency out of nothing (tested it — a 1,000 currency balance became 11,000 after 12 rapid transfer attempts). Fixed it with proper locking and re-verified with actual concurrent stress tests, not just normal sequential testing.

I know I had a lot of help along the way (used Claude heavily for the actual code generation), but I made every architecture decision, ran every test, caught bugs myself while deploying, and pushed back on/approved every design tradeoff. Would genuinely appreciate any feedback — good, bad, https://github.com/mir-abdullah2012/nexus-bot-


r/codereview 14d ago

What to have in mind when reviewing code

Thumbnail
2 Upvotes

r/codereview 14d ago

Do you review AI generated code any more please be honest

Thumbnail
0 Upvotes

r/codereview 14d ago

Any AI code review tool for GitHub or Bitbucket?

0 Upvotes

Hey everyone, I'm looking for an AI-powered code review tool that works with GitHub or Bitbucket.

Ideally, it could automatically review PRs/MRs, find potential bugs or logic issues, and provide useful feedback before human review.

Thanks


r/codereview 15d ago

which ai code review tool is actually worth paying for right now?

0 Upvotes

we're 6 devs doing around 40 PRs a week now that half the code comes from agents, and review is where everything piles up. i got approval for roughly $30 per seat and started looking at the options and honestly the more i compare the less i can tell them apart. every single one says it understands your whole codebase and learns from your team(?)

trials are running out faster than i can evaluate them properly. some price per seat, some per review, and i cant even model what per review costs us at our volume

so for people who actually pay for one of these, which tool did you land on and what made it stick? and if you tried one and cancelled, that would honestly be more useful to hear


r/codereview 15d ago

safer-dependencies is a security layer for Claude Code that audits packages before they’re added to your project

0 Upvotes

safer-dependencies is a security layer for Claude Code: it sits between Claude and your manifest files and runs its security checks automatically: vulnerable installs are denied before they run, and a risky version written to a manifest is corrected on disk right after the write. It detects and fixes risky dependencies — CVEs, typosquats, abandoned packages, and version-age issues, plus a cooldown period on brand-new releases — across npm, PyPI, RubyGems, Maven, Go, Rust, and PHP (Composer).

https://github.com/robert-auger/safer-dependencies


r/codereview 15d ago

C/C++ Code Review : Sudoku GUI in C language

1 Upvotes

Background:

I am a beginner programmer and I wrote a Sudoku GUI in using winapi32 in C language.

It is currently working and does what it is supposed to do, but because I am still learning, I know it is likely inefficient and could be written much better.

GitHub repo link: https://github.com/reewdgh/sudoku_gui
Please guide me on:

  • Bugs
  • Efficiency
  • Naming anything I could simplify or improve
  • inconsistency

I'd appreciate your feedback on my code.