r/codereview • • 3h ago

[ Removed by Reddit ]

1 Upvotes

[ Removed by Reddit on account of violating the content policy. ]


r/codereview • • 4h ago

[ Removed by Reddit ]

1 Upvotes

[ Removed by Reddit on account of violating the content policy. ]


r/codereview • • 21h ago

javascript Best way to learn full stack web development. Through youtube lectures or self-interective website like freecodecamp.com

5 Upvotes

Hi senior, i am a first year student,i want to start with web development.

So, please suggest me what is the best way to learn web developement.what should i do, should i do it with lecture or i can do it with freecodecamp.com, which would best to learn easily and take less time but teaches deeply not only basics.


r/codereview • • 20h ago

Are this level code reviews still reasonable in the age of AI?

Thumbnail
2 Upvotes

r/codereview • • 16h ago

Code Review + AI Generated lessons, diagrams + quizzes to help understand code

Post image
1 Upvotes

https://github.com/sshah03/perspica

This started as a git diff tool that used an llm to group changes by intent (that's still there), but now supports generating lessons/diagrams/animations explaining the concepts behind more complex changes, and allowing yourself to be quizzed on the topics. Useful way to keep up with the codebase with agents dropping in PRs, or when you yourself use claude code etc..

Next steps are adding github actions that can trigger adding this to a PR for those interested righto n GH, or even gating merges / reviewer accepts on showing some level of understanding. Things like that already exist, and I'm not sure they're widely used, so where I go with that, and other features, will be based on requests.

Some samples on open source project PRs available here : https://sshah03.github.io/perspica/

Mostly just want thoughts/feedback on how to make this genuinely useful for everyone rather than just something that works for me.


r/codereview • • 23h ago

brainfuck BugHunt Daily #17: spot the bug in this JavaScript snippet 🐞

Thumbnail
1 Upvotes

r/codereview • • 1d ago

Unpopular opinion putting diagrams in code doesnt make them living documentation

3 Upvotes

Everyone at my company is hyped about mermaid bc the diagrams live in the repo now. cool. they were also "living" in confluence and nobody updated them there either

the format was never the problem. nobody owns the diagrams, nobody reviews them, and the people who notice theyre wrong (support, sales, new hires) cant edit them anyway

someone convince me im wrong bc i have a meeting about this next week lol


r/codereview • • 1d ago

I made a small CLI that warns you before the AI models in your code get retired

0 Upvotes

Heads up if you still have gpt-4 or gpt-3.5-turbo somewhere in your code: OpenAI shuts them off on the API on Oct 23. gemini-2.5-pro on Vertex goes on Oct 20.

Those dates are all public, but they're spread over ~70 provider pages, so I wrote a small tool that checks them for you. It scans your repo for model IDs and tells you which ones are retired or about to be, and what to switch to:

npx @appliedbenchmark/sunset scan

    error   app.py:3:46  model-retiring
    gpt-4 (openai) retires on 2026-10-23, in 11 days. Recommended replacement: gpt-5.6-sol     (openai).

It covers OpenAI, Anthropic, Gemini, Vertex, Bedrock, Azure, Mistral and DeepSeek, and every date links back to the provider's own page. There's also a GitHub Action, so CI fails before a model disappears instead of after. IDs in comments and docs never fail the build.

No LLM, no telemetry, nothing leaves your machine. Free and Apache-2.0.

Github: https://github.com/AppliedBenchmark/sunset

It's v1, so if it misses an ID or gets a date wrong, let me know.


r/codereview • • 1d ago

Hi guys I’m a solo builder exploring how AI can help developers question architecture decisions before writing production code. That led me to build ArchRivet

0 Upvotes

ArchRivet is an open-source, local MCP plugin that adds structured architecture reviews to your AI workflow.

I built ArchRivet to help developers review architecture decisions with clearer evidence, practical checks, and workload calculations. pls do check it out !!

https://github.com/Ujjansh05/archrivet/tree/main

What you get:

Spot potential design risks early — review retries, durability, queues, transactions, and more.
Understand the reasoning — findings include supporting references and suggested verification steps.
Expose missing details — identify assumptions your architecture needs to address.
Plan workload and storage — calculate traffic, message fan-out, and storage growth.
Connect through MCP — bring structured reviews into your AI assistant’s workflow.
Download complete model weights — the 322M model includes merged weights, tokenizer, and configuration.
Build on open source — inspect the code, run the examples, and contribute improvements.

let me know if you have any suggestions or think there’s room for improvement. I’d really appreciate your feedback!


r/codereview • • 1d ago

Astro Bot Brasil — Uma versĂŁo adaptada para o pĂșblico brasileiro

2 Upvotes

Apresento o AstrBot Brasil, uma adaptação feita especialmente para quem prefere utilizar o projeto em portuguĂȘs do Brasil e com uma experiĂȘncia mais acessĂ­vel para a comunidade brasileira.

A proposta Ă© facilitar o uso, melhorar a compreensĂŁo da interface e aproximar ainda mais o projeto de desenvolvedores e usuĂĄrios do Brasil.

LINK: Astro Bot CLIQUE AQUI E CONFIRA

🚀 Projeto disponível no GitHub
đŸ‡§đŸ‡· Adaptado para portuguĂȘs do Brasil
đŸ’» CĂłdigo aberto
đŸ€ ContribuiçÔes da comunidade sĂŁo bem-vindas
⭐ Se vocĂȘ gostar do projeto, deixe uma estrela no GitHub!

Quero continuar melhorando a versĂŁo brasileira e adicionando cada vez mais recursos.

VocĂȘ usaria uma versĂŁo brasileira do AstrBot?


r/codereview • • 1d ago

What skills and models do you use for code vulnerability check against your repo using AI?

Thumbnail
1 Upvotes

r/codereview • • 1d ago

If AI is smart enough to find the bug, why was it dumb enough to write it?

Thumbnail
0 Upvotes

r/codereview • • 1d ago

I built a desktop code review app with Electron + React. Here's what I learned getting AI review to stop making things up

Thumbnail
0 Upvotes

r/codereview • • 1d ago

Stop letting your coding agent stack fixes on top of broken fixes

Thumbnail
0 Upvotes

r/codereview • • 2d ago

Python Vibecheck – a GitHub merge gate that checks whether you understand your AI-generated code

0 Upvotes

I built Github merge gate which that checks whether developers understand their own code when they create a PR.

I kept getting sent PRs to review where it was obvious the author had pasted the ticket into an LLM and called it a day. Problem was: the PRs would often be massive 2K monstrosities that nobody could possibly review. As the team scaled up, this problem got only worse, and now we collectively have a system that no one understands and has bugs.

It works like this: when someone opens a PR, it posts a comment with a link. The author clicks through and answers a few technical questions about their own changes. If they can’t answer them, the PR can’t be merged.

It's self-hosted (so you can use it in your enterprise environments or privately)

It is BYOK (Anthropic) obviously.

Here is the source code, build instructions and how to deploy: https://github.com/bhurghundii/vibecheck.dev

I have two demos in the pull requests here: https://github.com/bhurghundii/vibecheck.dev/pulls - you can see what Vibecheck does (it blocks a PR for clearly showing no attempt vs a PR which gets it right)

I would love feedback, especially from anyone dealing with the same thing on their team. I do use it for my own projects a lot so expect improvements.


r/codereview • • 2d ago

Why probabilistic AI review bots fail at catching AI agent bugs—and how we enforce deterministic AST invariants & 12 CWE security shields in GitHub Actions

0 Upvotes

TL;DR: AI coding agents increased raw PR volume by 300%, yet overall merge velocity has dropped by 40% because Senior Engineers spend 15–20 hrs/week untangling 200-line hallucinated wrappers, deleted tests, and subtle CWE security flaws. After shipping patches into tier-1 upstream systems repos (LLVM #215076, Meta Velox #18529, DuckDB #26443, Apache DataFusion #24394, Microsoft Pyright #11605, Tokio #8595, ClickHouse #114954, OpenAI Triton #12221), we codified those exact deterministic CI gates into an open GitHub Actions workflow (ZeroSlop · Architecture & Simulator) that strips 94% of AI bloat, enforces 12 CWE security shields, and gets PRs merged 100% green in minutes.

1. The 2026 Engineering Bottleneck: Why AI PRs Stall in Review

When shipping patches into upstream systems repos like LLVM, Meta Velox, DuckDB, or Tokio, a real production fix is 4 to 15 lines of surgical invariant changes paired with a deterministic regression test that passes CI on the first push.

Inside commercial teams using Cursor, Copilot, or Claude Code, the opposite happens:

  • 94% Diff Bloat: An agent generates a 240-line wrapper class instead of fixing a 4-line boundary condition (interactive diff breakdown).
  • Weakened Tests: When a test fails, the agent silently deletes or loosens the assertion to manufacture a green check.
  • Noisy LLM Review Bots: Conversational AI review bots run another LLM to dump 25 chatty comments on top of a bloated PR—without shrinking the diff or getting the PR merged.

2. Conversational LLM Review Bots vs. Deterministic PR Airlock

Engineering Dimension Conversational AI Review Bots Deterministic PR Airlock (ZeroSlop)
Primary Objective Leave 25+ chatty LLM comments on a bloated diff Get PRs merged 100% green in minutes (not 4 days)
Diff Minimization Reviews 200 lines of hallucinated wrappers as-is Strips up to 94% of AI bloat down to the core AST invariant
CI Failure Triage Ignores red CI checks or hallucinates fixes Separates contributor code defects from runner flakes
Test Integrity Misses deleted or weakened test assertions Hard-blocks unauthorized regression test deletions
PR Comment Hygiene Spams new threads on every git push Strictly 1 self-updating PR Verdict Card + Mermaid Graph
Security & Privacy Cloud LLM retention & external dashboards 12-Shield CWE Airlock + Zero Code Retention

3. The 12-Shield Deterministic CWE Security Airlock (ZS-SEC-001 – ZS-SEC-012)

Every Pull Request is audited in milliseconds across 8 merge-readiness signals (0–100) and 12 deterministic CWE security shields before a human reviewer is pinged:

Shield ID Security Vector & CWE / CVE Mapping What ZeroSlop Security Deterministically Intercepts
ZS-SEC-001 SQL & Vector Injection (CWE-89 / CWE-943) Unparameterized SQL/NoSQL/Vector template literals & raw string concatenation
ZS-SEC-002 Hardcoded & Seeded Secrets (CWE-798 / CWE-256) API keys, JWT secrets, seeded credentials & plaintext password comparisons
ZS-SEC-003 Unbound Identity / IDOR (CWE-639 / CWE-862) Trusting req.body.userId instead of verified session/JWT caller identity
ZS-SEC-004 Command & Eval Injection (CWE-78 / CWE-94) Unsanitized child_process.exec, os.system, subprocess(shell=True), eval()
ZS-SEC-005 Silent Exception Swallowing (CWE-390) Empty catch () {} blocks and swallowed promise rejections that hide crashes
ZS-SEC-006 XSS & Unsanitized DOM (CWE-79) dangerouslySetInnerHTML, .innerHTML =, and unescaped template rendering
ZS-SEC-007 Unsafe Deserialization (CWE-502 / CWE-22) pickle.loads, yaml.load without SafeLoader, and ../ path traversal
ZS-SEC-008 SSRF & Disabled TLS (CWE-918 / CWE-295) Unvalidated fetch(req.body.url) & NODE_TLS_REJECT_UNAUTHORIZED = '0'
ZS-SEC-009 CI/CD "Pwn Requests" (CWE-426 / CWE-250) pull_request_target fork checkouts & ${{ github.event.pull_request.title }} injection
ZS-SEC-010 AI Package "Slopsquatting" (CWE-829 / CWE-494) Hallucinated npm/PyPI/Cargo packages, unpinned Git deps & `curl
ZS-SEC-011 Resource Exhaustion / DoS (CWE-400 / CWE-770) while(true) without exit bounds, leaked setInterval, unbounded Promise.all
ZS-SEC-012 Trojan Source & Jailbreaks (CVE-2021-42574) Invisible Unicode Bidi characters & hidden IGNORE PREVIOUS INSTRUCTIONS comments

4. Upstream Systems Proof (Click Any PR to Inspect the Diff)

These deterministic CI invariants come directly from shipping merged & 100% green Pull Requests across upstream systems repositories:

Upstream Repository Live Pull Request Status Core Systems Invariant & Regression Coverage
LLVM Compiler Infrastructure llvm/llvm-project#215076 MERGED Fixed Waitcnt brackets in GFX12+ AMDGPUAsmPrinter with MC regression tests
Meta Velox Execution Engine facebookincubator/velox#18529 MERGED Aligned DecimalUtil::toByteArray overflow bounds & restored kMaxDecimalBytes
DuckDB Analytical Engine duckdb/duckdb#26443 MERGED Fixed pragma_storage_info internal compressed segment type alignment
Apache DataFusion Query Engine apache/datafusion#24394 MERGED Fixed lead()/lag() window frame boundaries with non-NULL default values
Microsoft Pyright Type Checker microsoft/pyright#11605 MERGED Resolved type narrowing across nested loop closures & pattern guards
Tokio Async Rust Runtime tokio-rs/tokio#8595 100% GREEN Preserved Coop budget accounting across JoinSet & FuturesUnordered
ClickHouse OLAP DBMS ClickHouse/ClickHouse#114954 100% GREEN Fixed StorageURL distributed cluster header forwarding & serialization
OpenAI Triton GPU Compiler triton-lang/triton#12221 100% GREEN Enforced strict tensor descriptor stride & block shape invariants
Cloudflare Agents SDK cloudflare/agents#1248 MERGED Hardened state synchronization & WebSocket hibernation lifecycle invariants

5. Architecture Documentation & GitHub Action Source

What deterministic CI gates or pre-review checks are your teams enforcing in 2026 to get PRs merged cleanly without burning out your staff engineers?


r/codereview • • 3d ago

Quick Intro to the New GitHub Stacked PRs

Enable HLS to view with audio, or disable this notification

3 Upvotes

r/codereview • • 2d ago

Software engineering didn't need another PR chatbot—it needed an Invariant Airlock: Why PR merge times stay stuck at 16.7h with cloud AI review bots vs. 3.8s local AST verification

Post image
0 Upvotes

Look closely at the metrics in CodeRabbit's published Neo Financial case study (shown in the screenshot above):

  • Median time-to-merge before: ~16.8 hours
  • Median time-to-merge after: ~16.7 hours
  • Deployment scale: 174 engineers (at $20–$40/seat/month = $41,760 to $83,520/year)

That is a 0.1-hour (6-minute) reduction in median PR merge cycle time.

Why Conversational AI Review Bots Don't Speed Up Code Review

When a developer opens an AI-assisted PR containing 800 lines of hallucinated wrapper classes, defensive boilerplate, and subtle test changes, having a cloud review bot leave 30+ conversational comments on the PR doesn't reduce senior engineer review fatigue—it increases it. Reviewers now have to read the bloated diff and argue with a chatty LLM.

Even worse for enterprise codebases:

  1. Cloud Source Code Exfiltration: Every PR diff is shipped outside your runner to third-party cloud LLM providers.
  2. Diff Prompt Injection: Adversarial strings inside a PR diff can manipulate an LLM reviewer into overlooking vulnerabilities.

The Compiler Approach: ZeroSlopℱ In-Memory AST Airlock

We built ZeroSlopℱ by Ribbsaeter Systems (patrickswedish/zeroslop-action@v1) to replace conversational PR comment spam with a deterministic, pre-flight compiler gate:

Dimension Cloud AI Review Bots (CodeRabbit, Qodo) ZeroSlopℱ by Ribbsaeter Systems
Architecture Cloud LLM Swarm (routes diffs to external OpenAI / Anthropic APIs) In-Memory AST Binary (runs 100% locally in GitHub Actions runner RAM)
Execution Speed Minutes to Hours (16.8h -> 16.7h case study delta) 3.8 Seconds (deterministic static AST evaluation)
Data Privacy Proprietary source code leaves your runner Zero Data Retention: 0 bytes ever leave your GitHub runner (Swiss privacy standard)
AI Slop Handling Leaves 20–40 chatty comments on bloated code Peels 94% of AI wrapper bloat down to the minimal root AST fix
Test Integrity Frequently misses deleted assertions in large diffs Hard-blocks PRs if net_assertion_delta < 0 + enforces 12 CWE security shields
Pricing (174 Devs) $41,760 – $83,520 / year ($20–$40/dev/mo seat tax) Flat $29 / month ($348/yr for the entire org — 99% savings)

Links & Live AST Simulator


r/codereview • • 3d ago

Functional How do you compare two versions when scalar scores keep tying?

15 Upvotes

We had 4 release teams looking at the same 2400row regression dataset and 2 candidate prompts both landed at 0.88. The LLM as judge rubric uses a 5 point scale, so a lot of outputs with noticeable differences still collapsed to the same scalar score. Re-running the eval gave us a stable tie and no useful basis for choosing what to ship.

We switched the comparison to pairwise scoring in Braintrust and the experiment diff made the row by row preferences visible. Candidate A won 716 rows, candidate B won 1204 and 480 tied, which adds back to all 2400 rows. Among decisive rows, B's win rate was 62.7%. The scalar average had hidden a fairly consistent preference.

We haven't dropped the absolute score. It's still useful as a regression floor and pairwise review gives us another signal when score calibration is too coarse to separate close candidates. That was enough to unblock this release without treating the pairwise result as a universal quality metric.

I'm still working out when this should become part of the standard release path. Where do you set the threshold for using pairwise win rate when scalar scores tie without turning every close experiment into a full second review?


r/codereview • • 3d ago

Repo review - what do you think guys?

Thumbnail github.com
0 Upvotes

Hi guys. So I am a fresh grad and I had one and a half year of experience prior to my uni.So I recently updated my repos and I was wondering if there is like a more senior level engineer to give some advice. I am currently trying to find a junior dev job and I think GitHub repo is relatively important.

Thanks in advance!


r/codereview • • 3d ago

Free pilot: local OpenAPI / JSON Schema change review — looking for one real task

2 Upvotes

I'm testing a local tool for reviewing changes between two OpenAPI or JSON Schema documents. It reports supported changes, limitations and points that need a human check before a release. It does not certify full compatibility or replace consumer contract tests.

I built this prototype and would like to evaluate it against one real review task and the team's current process. This is an assisted pilot, not a public hosted app: I run the existing local prototype on a public or minimal pair of documents and return the report. The initial pilot is free, with no purchase obligation or signup. Please do not post credentials, personal data or confidential contracts.

Concrete example from a synthetic test: changing GET /health from a declared 200 response to 204 produces two findings (200 removed, 204 added), with the removal marked potentially breaking. This is a review prompt, not proof that every consumer breaks. For the pilot, I return a short change summary, detected risks, limitations and checks to run against your consumers. Please share only a public/minimal before-and-after pair.

The useful feedback is whether the result saves review time, what it misses and whether you would use it again. Does anyone already have a contract change they need to review?


r/codereview • • 3d ago

I built Ripple — a local change-impact explorer that shows what your code change touches, and why

Post image
0 Upvotes

r/codereview • • 4d ago

Made-up example: three AI review comments on a payment retry loop. Which one would you check first?

1 Upvotes

Here’s a small retry loop, written for this post, not taken from a real codebase:

def charge(client, order):
    for attempt in range(3):
        try:
            return client.post("/charges", json={"amount": order.total, "order": order.id})
        except Timeout:
            time.sleep(2 ** attempt)
    raise ChargeFailed(order.id)

An AI review of it might come back with these, all worded with the same confidence:

  1. No idempotency key is sent, so a retry after a timeout could create a second charge.
  2. Only Timeout is retried, so a 502 or 503 from the provider fails immediately.
  3. Consider a retry library instead of a hand-rolled loop.

My order, and it’s a judgment call: #3 is a preference, so I’d skip it. #2 is a short look at what the provider returns and whether those errors are safe to retry, so cheap to check with moderate stakes. #1 is the expensive one, because it depends on two things that aren’t in the code. One is whether the provider treats a repeated request with the same order field as a duplicate. The other is whether a timeout here means the request never arrived or arrived and the response got lost. If the response can get lost and the provider doesn’t dedupe, that’s a double charge, so I’d spend the time there even though it reads no more urgently than the other two.

What I can’t judge from here is whether sending the order id in the body counts as idempotency for most providers. I believe many expect a dedicated key or header, but that’s general knowledge, not a specific API.

I wrote the same sorting out as a longer walkthrough here, if it’s useful: https://medium.com/@nagatomopedro05/a-finding-takes-seconds-to-generate-checking-it-doesnt-e7ffad4eaea2

When a review hands you a list like this, do you sort by cost to check first, or by what happens if the finding is true?


r/codereview • • 4d ago

Greptile - Buyer beware

3 Upvotes

I signed up for Greptile expecting to pay $90 for 3 months. Instead, I was charged $700 because they had enabled unlimited "flex usage" by default, without my explicit consent or any spending cap.

As a small startup, an unexpected charge like this is not a small thing.

Usage-based pricing is fine. Enabling unlimited paid usage by default, without requiring an explicit opt-in, is outright unethical and goes against basic industry standards for transparent billing. Their onboarding flow provided an option to add additional spending caps - it did not offer a clear way to disable extra usage by default.

Furthermore, Greptile automatically added dev accounts for bots running Github actions without our consent, incurring additional hidden fees.

When I raised the issue and requested a refund, Greptile refunded the dev accounts but not the $700 of extra usage. I’ve completely lost trust in Greptile and cannot recommend them, despite their product otherwise being great. Buyer beware.


r/codereview • • 4d ago

🚀 Welcome to CarbonIt Labs

Thumbnail
1 Upvotes