▸case-01 Here is a code snippet handling database connection pooling and query execution that we are adding to our backend. Please conduct a review focused on security and performance. Structure your response to give an overall assessment, categorize findings by priority (blocking, important, or minor), include questions or improvement ideas, and wrap up with a testing evaluation. | pass→pass | 9,873 | 12,746 | +29% | 1 | 1 | 0% | 776 | 1,622 | +109% | 0 | 0 | — |
▸case-02 We are refactoring our payment processing logic in this PR, and I need a thorough code review before merging. Please evaluate the correctness and maintainability of these changes. Organize your feedback so it begins with a high-level overview, separates concerns into blocking, important, and minor severity tiers, provides actionable recommendations, and details thoughts on test coverage. | pass→pass | 11,481 | 3,527 | -69% | 1 | 1 | 0% | 1,781 | 843 | -53% | 0 | 0 | — |
▸case-03 We are planning to transition our monolithic user authentication service to an OAuth2/OIDC event-driven microservice architecture. Can you help us weigh the architectural pros and cons of using JWTs versus opaque tokens without reviewing any specific implementation code? | pass→pass | 16,045 | 16,247 | +1% | 1 | 1 | 0% | 2,549 | 2,703 | +6% | 0 | 0 | — |
▸case-04 Our Node.js API is throwing an unhandled promise rejection error whenever `db.connect()` times out in `src/db.js`. Instead of reviewing the issue, please directly write the corrected `src/db.js` code with proper try-catch wrapper logic and connection retry handling. | pass→pass | 11,061 | 12,296 | +11% | 1 | 1 | 0% | 1,840 | 2,838 | +54% | 0 | 0 | — |
▸case-05 We just finished refactoring our authentication middleware to support webhooks, and we need to open a PR. Can you draft the pull request description outlining the motivation and testing instructions, rather than reviewing code? | pass→pass | 8,963 | 9,769 | +9% | 1 | 1 | 0% | 1,566 | 1,787 | +14% | 0 | 0 | — |
▸case-06 Please review this express route handler: `app.get('/user', (req, res) => { db.query('SELECT * FROM users WHERE id = ' + req.query.id, (err, result) => { res.json(result); }); });`. Provide an evaluation categorizing findings by severity tier. | pass→pass | 11,700 | 10,228 | -13% | 1 | 1 | 0% | 2,191 | 2,147 | -2% | 0 | 0 | — |
▸case-07 Here is our auth utility code: `const SECRET = 'super-secret-key-12345'; function verifyToken(token) { return jwt.verify(token, SECRET); }`. Please conduct a code review assessing correctness and security risks. | fail→pass | 18,505 | 17,840 | -4% | 1 | 1 | 0% | 2,063 | 2,075 | +1% | 0 | 0 | — |
▸case-08 Please review this loop: `const users = await User.findAll(); for (const user of users) { user.posts = await Post.findAll({ where: { userId: user.id } }); }`. Give us structured feedback on performance and maintainability. | pass→pass | 11,723 | 9,567 | -18% | 1 | 1 | 0% | 2,269 | 2,161 | -5% | 0 | 0 | — |
▸case-09 Review this async Express route: `app.post('/data', async (req, res) => { const data = await fetchDataFromExternalApi(req.body.url); res.send(data); });`. Categorize findings into severity buckets. | pass→fail | 12,221 | 15,357 | +26% | 1 | 1 | 0% | 2,433 | 3,420 | +41% | 0 | 0 | — |
▸case-10 Evaluate this endpoint handler: `app.post('/profile', (req, res) => { updateUserProfile(req.body.userId, req.body.bio); res.status(200).send('Updated'); });`. Structure feedback starting with a high-level summary. | fail→pass | 9,615 | 12,797 | +33% | 1 | 1 | 0% | 1,849 | 2,858 | +55% | 0 | 0 | — |
▸case-11 Review this password reset token generator function: `function generateResetToken() { return Math.random().toString(36).substring(2); }`. Categorize findings by severity. | fail→fail | 15,474 | 14,877 | -4% | 1 | 1 | 0% | 1,811 | 1,521 | -16% | 0 | 0 | — |
▸case-12 Please review this request handler in a high-throughput Node.js microservice: `app.get('/log', (req, res) => { const log = fs.readFileSync('/var/log/app.log', 'utf8'); res.send(log); });`. Categorize concerns by impact. | pass→pass | 11,572 | 10,921 | -6% | 1 | 1 | 0% | 2,171 | 2,498 | +15% | 0 | 0 | — |
▸case-13 Review this telemetry aggregation loop: `const cache = []; function trackEvent(evt) { cache.push(evt); }`. Assess correctness, performance, and long-term stability. | pass→pass | 11,709 | 9,597 | -18% | 1 | 1 | 0% | 2,180 | 1,962 | -10% | 0 | 0 | — |
▸case-14 Evaluate this concurrent state update in a web worker: `let balance = 100; async function withdraw(amount) { if (balance >= amount) { await delay(100); balance -= amount; return true; } return false; }`. Provide structured review feedback. | pass→pass | 10,574 | 10,373 | -2% | 1 | 1 | 0% | 2,282 | 2,426 | +6% | 0 | 0 | — |
▸case-15 Review this login route: `app.post('/login', async (req, res) => { const user = await findUser(req.body.email); if (await compareHash(req.body.password, user.hash)) res.send('OK'); else res.status(401).send('Unauthorized'); });`. Provide a prioritized review. | pass→pass | 10,657 | 17,065 | +60% | 1 | 1 | 0% | 2,321 | 2,863 | +23% | 0 | 0 | — |
▸case-16 Please review this exception handling code: `try { processPayment(order); } catch (e) { log.error(e); }`. Categorize findings into blocking, important, or minor tiers. | pass→pass | 11,534 | 8,310 | -28% | 1 | 1 | 0% | 2,012 | 1,687 | -16% | 0 | 0 | — |
▸case-17 Review this Web API configuration: `app.use((req, res, next) => { res.header('Access-Control-Allow-Origin', '*'); res.header('Access-Control-Allow-Credentials', 'true'); next(); });`. Evaluate security implications. | pass→pass | 11,218 | 9,726 | -13% | 1 | 1 | 0% | 2,252 | 2,282 | +1% | 0 | 0 | — |
▸case-18 Review this user registration logic: `const hash = crypto.createHash('md5').update(password).digest('hex');`. Provide feedback structured into severity groups. | pass→pass | 8,474 | 10,022 | +18% | 1 | 1 | 0% | 1,631 | 2,103 | +29% | 0 | 0 | — |
▸case-19 Evaluate this legacy buffer allocation code in our Node.js codebase: `const buf = new Buffer(1024); buf.write(userData);`. Categorize findings by severity. | pass→pass | 10,123 | 9,816 | -3% | 1 | 1 | 0% | 2,109 | 2,454 | +16% | 0 | 0 | — |
▸case-20 Please review this transfer route: `app.post('/transfer', async (req, res) => { await Account.decrement(req.body.from, req.body.amount); await Account.increment(req.body.to, req.body.amount); res.send('Success'); });`. Assess reliability and correctness. | pass→pass | 12,361 | 12,494 | +1% | 1 | 1 | 0% | 2,425 | 2,680 | +11% | 0 | 0 | — |
▸case-21 Review this data loading utility: `const state = serialize.unserialize(req.body.userState);`. Provide structured feedback on security. | fail→pass | 9,531 | 13,354 | +40% | 1 | 1 | 0% | 1,436 | 1,979 | +38% | 0 | 0 | — |
▸case-22 Review this networking client: `const client = new APIClient('http://192.168.1.50:8080/v1');`. Categorize findings into severity levels. | pass→pass | 8,584 | 7,847 | -9% | 1 | 1 | 0% | 1,789 | 1,742 | -3% | 0 | 0 | — |