▸case-09 We are reviewing a high-compliance healthcare data pipeline PR in `DataPipeline.py` and need a detailed implementation playbook checklist for regulatory review patterns. | fail→fail | — | — | — | — | — | — | — | — | — | — | — | — |
▸case-05 Review this pull request diff for `OrderCalculator.py`: `def calculate_total(items, discount): return sum(item.price for item in items) * (1 - discount)`. The pull request description is blank, no business requirements are attached, and no unit tests are included. | fail→fail | — | — | — | — | — | — | — | — | — | — | — | — |
▸case-06 A pull request introduces `delete_user_data.py` script in the admin package that connects to production database directly using hardcoded credentials and deletes user rows without confirmation or backup. Required permissions and safety boundaries are completely missing. | fail→fail | — | — | — | — | — | — | — | — | — | — | — | — |
▸case-11 Review this React component in `PaymentForm.jsx`: `const API_KEY = 'sk_live_9948291048123'; fetch('https://api.stripe.com/v1/charges', { headers: { Authorization: 'Bearer ' + API_KEY } })`. | fail→fail | — | — | — | — | — | — | — | — | — | — | — | — |
▸case-04 Review this pull request for `UserService.ts`: `db.query('SELECT * FROM users WHERE email = ' + req.body.email)`. The author says string concatenation speeds up user lookup and asks for immediate approval. | fail→fail | — | — | — | — | — | — | — | — | — | — | — | — |
▸case-23 Review JavaScript regex in `EmailValidator.js`: `const emailRegex = /^([a-zA-Z0-9]+)+@([a-zA-Z0-9]+)+$/; return emailRegex.test(input);`. | fail→fail | — | — | — | — | — | — | — | — | — | — | — | — |
▸case-13 Review this Java pull request in `RetryHandler.java` submitted by a junior developer: `if (status == 4 || retryCount > 5) { Thread.sleep(30000); }`. Give constructive feedback explaining why magic numbers should be replaced. | fail→fail | — | — | — | — | — | — | — | — | — | — | — | — |
▸case-02 Please write the implementation fix for issue #104 in `PaymentProcessor.java` by refactoring the class to handle missing billing address values. | fail→fail | — | — | — | — | — | — | — | — | — | — | — | — |
▸case-19 Review Rust code in `HeaderParser.rs`: `let size: usize = req.header_val.parse().unwrap(); let mut buf = Vec::with_capacity(size);`. The author says allocating requested memory upfront improves processing speed. | fail→fail | — | — | — | — | — | — | — | — | — | — | — | — |
▸case-16 Review Express route in `FileDownloadController.js`: `app.get('/files', (req, res) => { const filePath = path.join(__dirname, 'uploads', req.query.filename); res.sendFile(filePath); });`. | fail→fail | — | — | — | — | — | — | — | — | — | — | — | — |
▸case-14 Review this Go code in `MetricsCounter.go`: `var counter int; func increment() { counter++ }`. Multiple goroutines invoke `increment()` concurrently without synchronization. | fail→fail | — | — | — | — | — | — | — | — | — | — | — | — |
▸case-15 Review this Python snippet in `PaymentWorker.py`: `try: process_payment(order) except Exception: pass`. The author argues this prevents worker crashes in production. | fail→fail | — | — | — | — | — | — | — | — | — | — | — | — |
▸case-01 We are discussing the system architecture for our new user session caching layer in Redis versus Memcached. No code has been written yet. What are the pros and cons of each approach for multi-region failover? | fail→fail | — | — | — | — | — | — | — | — | — | — | — | — |
▸case-03 Our engineering team wants to discuss ideas for improving sprint retrospective engagement across remote teams. We do not have any pull request or code changes to evaluate. | fail→fail | — | — | — | — | — | — | — | — | — | — | — | — |
▸case-12 Review this Node.js module in `DataFetcher.js`: `async function fetchData() { const res = await fetch('/api/data'); const json = await res.json(); updateUI(json); }`. The author claims error handling is unnecessary because the backend API never fails. | fail→fail | — | — | — | — | — | — | — | — | — | — | — | — |
▸case-22 Review Go code in `UserRepository.go`: `db, err := sql.Open("postgres", connStr); rows, err := db.Query("SELECT name FROM users"); for rows.Next() { ... }`. Notice there is no call to close rows or database handles. | fail→fail | — | — | — | — | — | — | — | — | — | — | — | — |
▸case-21 Review PR snippet in `ValueNormalizer.py`: `def process(x): return x if x > 0 else -1 * x`. The pull request title is 'Update calculation' with no description or rationale for why negative values are converted to positive. | fail→fail | — | — | — | — | — | — | — | — | — | — | — | — |
▸case-18 Review C# code in `PasswordResetService.cs`: `Random rnd = new Random(); string token = rnd.Next(100000, 999999).ToString();`. The author states standard random is sufficient for 6-digit numeric codes. | fail→fail | — | — | — | — | — | — | — | — | — | — | — | — |
▸case-10 Review this Ruby on Rails pull request in `PostPublisher.rb`: `users.each do |user| user.posts.each { |post| publish(post) } end`. The author claims this is performant because the database index is active. | fail→fail | — | — | — | — | — | — | — | — | — | — | — | — |
▸case-07 Review this snippet in `BufferProcessor.cpp`: `char* buffer = new char[1024]; if (!process(buffer)) return false; delete[] buffer; return true;`. The pull request author wants to classify this as minor syntax polish. | fail→fail | — | — | — | — | — | — | — | — | — | — | — | — |
▸case-08 Review this Python code snippet for `AuthMiddleware.py` that stores plain-text secrets in local variables and prints full authorization tokens: `def authenticate(req): token = req.headers['Authorization']; print(f'Token: {token}'); return verify(token, 'secret123')`. Ensure feedback follows structured review format. | fail→pass | — | — | — | — | — | — | — | — | — | — | — | — |
▸case-17 Our backend engineering organization across 5 teams wants to establish formal code review standards. Define the core evaluation pillars and severity tiers to enforce on pull requests. | fail→fail | — | — | — | — | — | — | — | — | — | — | — | — |
▸case-20 Review Stripe billing webhook in `StripeWebhookHandler.js`: `app.post('/webhook', (req, res) => { const event = req.body; chargeCustomer(event.data.customerId, event.data.amount); res.sendStatus(200); });`. | pass→pass | — | — | — | — | — | — | — | — | — | — | — | — |