▸case-17 During a refactor of `OrderProcessor.cs` in `e-commerce`, a developer removes structured logging statements (`logger.LogInformation("Order created")`), arguing that log output is not part of the system's observable behavior. Is this assumption correct? | pass→pass | 11,201 | 6,155 | -45% | 1 | 1 | 0% | 1,866 | 1,059 | -43% | 0 | 0 | — |
▸case-22 In `order-service`, during a refactor of `OrderRepository.go`, a developer finds an old database migration file `004_create_orders.sql` with an inefficient index definition. They propose editing `004_create_orders.sql` directly in place to fix the index definition as part of the refactoring task. How should this proposal be handled? | pass→pass | 11,167 | 5,099 | -54% | 1 | 1 | 0% | 1,907 | 1,154 | -39% | 0 | 0 | — |
▸case-01 We need to restructure `PaymentProcessor.py` in the `acme-billing` repository, but it currently has zero test coverage. The developer wants to split the class into three sub-modules immediately and write unit tests for the new sub-modules once the split is finished. How should this task be structured? | pass→pass | 10,575 | 6,860 | -35% | 1 | 1 | 0% | 1,887 | 1,582 | -16% | 0 | 0 | — |
▸case-02 While restructuring `OrderService.ts` in our `shop-api` project, after successfully moving a helper method, should I automatically run `git checkout -b refactor/order-service` and execute `git commit -m "move helper"` to maintain clean history? | fail→pass | 9,471 | 10,690 | +13% | 1 | 1 | 0% | 1,557 | 794 | -49% | 0 | 0 | — |
▸case-03 While extracting a private helper in `UserRegistry.go` for `auth-service`, you uncover a bug where a nil pointer exception occurs if `tenant_id` is missing. The team lead suggests fixing the bug right now inside the extraction commit and updating the return value. How should this scenario be handled? | fail→pass | 10,109 | 6,456 | -36% | 1 | 1 | 0% | 1,792 | 1,369 | -24% | 0 | 0 | — |
▸case-04 In the `analytics-pipeline` repository, we renamed the function `calculate_user_retention` to `compute_user_retention`. We updated all direct Python import statements across the project. Is the refactoring step complete, or what additional searches must be conducted? | pass→pass | 11,617 | 9,960 | -14% | 1 | 1 | 0% | 2,035 | 2,104 | +3% | 0 | 0 | — |
▸case-05 When planning a refactor for `export_service.rb` in `data-exporter`, a developer claims that observable behavior is strictly limited to public Ruby class method signatures and return types. What additional aspects of observable behavior must be included? | pass→pass | 13,827 | 9,576 | -31% | 1 | 1 | 0% | 2,228 | 1,925 | -14% | 0 | 0 | — |
▸case-06 In `enterprise-core`, which has over 50,000 unit and integration tests, a developer renames a single private local variable in `InventoryManager.java`. Should they run the entire repository integration test suite immediately after this single rename? | fail→pass | 7,159 | 3,776 | -47% | 1 | 1 | 0% | 1,225 | 910 | -26% | 0 | 0 | — |
▸case-07 A developer working on `ReportBuilder.cs` in `bi-engine` wants to combine five class renames, two method extractions, three file moves, and deduplication into a single batch edit. What rule governs how structural changes should be executed? | pass→pass | 5,652 | 2,907 | -49% | 1 | 1 | 0% | 1,018 | 817 | -20% | 0 | 0 | — |
▸case-12 What final review check must be conducted on the git diff before declaring a refactor complete in `NotificationDispatcher.py`? | pass→pass | 7,796 | 2,037 | -74% | 1 | 1 | 0% | 1,312 | 646 | -51% | 0 | 0 | — |
▸case-08 While refactoring `AccountRepository.kt` in `bank-core`, we decide to rename the underlying database column from `acc_num` to `account_number`. Should this database schema migration be combined directly into the structural refactoring task? | pass→pass | 10,392 | 4,250 | -59% | 1 | 1 | 0% | 1,706 | 1,024 | -40% | 0 | 0 | — |
▸case-09 Prior to beginning a refactor in `checkout-api`, executing `git status` reveals several uncommitted file edits from another developer's debugging session. Should we run `git reset --hard` to clean the working tree before starting? | pass→pass | 7,439 | 3,447 | -54% | 1 | 1 | 0% | 1,256 | 871 | -31% | 0 | 0 | — |
▸case-10 We finished refactoring the rendering pipeline in `dashboard-ui`. Unit tests pass, but snapshot tests report minor HTML formatting diffs in the output golden files. Is it acceptable to conclude the refactor? | fail→pass | 9,936 | 7,802 | -21% | 1 | 1 | 0% | 1,720 | 1,477 | -14% | 0 | 0 | — |
▸case-11 After relocating `LegacyParser.cpp` to `Core/Parsers/FastParser.cpp` in `game-engine` and updating C++ header includes, what post-move verification step must be performed? | fail→pass | 6,681 | 4,537 | -32% | 1 | 1 | 0% | 1,128 | 1,104 | -2% | 0 | 0 | — |
▸case-13 In `shipping-calculator`, we created characterization tests to capture existing behavior before refactoring. Now that the refactor is finished and green, should those characterization tests be deleted? | fail→pass | 10,437 | 5,723 | -45% | 1 | 1 | 0% | 1,664 | 1,190 | -28% | 0 | 0 | — |
▸case-14 While refactoring `protobuf-services`, a developer directly modifies `UserProto.pb.go` to rename a field because re-generating code via `protoc` takes too long. How should generated-code boundaries be respected? | pass→pass | 10,030 | 6,441 | -36% | 1 | 1 | 0% | 1,787 | 1,354 | -24% | 0 | 0 | — |
▸case-15 In `warehouse-service`, two existing tests in `inventory_test.go` are failing due to outdated test fixtures. The developer wants to start refactoring `inventory.go` immediately and fix the test failures afterwards. How should this workflow be corrected? | pass→pass | 7,391 | 4,399 | -40% | 1 | 1 | 0% | 1,263 | 1,032 | -18% | 0 | 0 | — |
▸case-16 In `billing-service`, after extracting `calculate_tax` from `utils/math.ts` to `domain/tax.ts`, what steps must be taken to ensure all references are updated? | fail→pass | 9,971 | 6,066 | -39% | 1 | 1 | 0% | 1,683 | 1,328 | -21% | 0 | 0 | — |
▸case-18 After refactoring `ImageCompressor.cpp` in `media-sdk`, processing latency increases from 10ms to 90ms per image, but all functional assertions pass. Can this refactor be marked complete? | pass→pass | 10,269 | 5,618 | -45% | 1 | 1 | 0% | 1,737 | 1,243 | -28% | 0 | 0 | — |
▸case-19 While refactoring `UserService.java` in `user-management`, the developer changes a public method signature from `getUser(String id)` to `getUser(UUID id)`. No internal callers in the repo broke. Does this change qualify as a pure refactor? | pass→pass | 7,744 | 5,209 | -33% | 1 | 1 | 0% | 1,292 | 1,152 | -11% | 0 | 0 | — |
▸case-20 In `payment-gateway`, we are refactoring `TransactionHandler.java`. A developer suggests changing the default connection timeout from 30 seconds to 10 seconds in the same task to improve system responsiveness. Since both edits touch `TransactionHandler.java`, should this timeout adjustment be executed within the refactoring workflow? | pass→pass | 9,297 | 5,162 | -44% | 1 | 1 | 0% | 1,469 | 1,195 | -19% | 0 | 0 | — |
▸case-21 While refactoring `AuthService.py` in `identity-api`, a developer notices that error responses return `{ "error": "invalid_credentials" }` in snake_case, whereas the rest of the system uses camelCase (`{ "error": "invalidCredentials" }`). The developer wants to fix this field casing inconsistency directly inside the refactoring pull request. How should this field casing update be handled? | pass→pass | 9,245 | 5,169 | -44% | 1 | 1 | 0% | 1,601 | 1,234 | -23% | 0 | 0 | — |
▸case-23 When refactoring the configuration loader in `deploy-service`, a developer renames the environment variable `DATABASE_URL` to `DB_URI` across application code and Kubernetes deployment manifests simultaneously without a backward-compatible fallback. What risk does this pose during deployment? | pass→pass | 13,468 | 10,096 | -25% | 1 | 1 | 0% | 2,174 | 2,025 | -7% | 0 | 0 | — |
▸case-24 While refactoring `search_indexer.py` in `search-core`, you discover a typo in a comment in `recommendations.py`. Should you correct the typo in `recommendations.py` as part of your current refactor iteration? | pass→pass | 7,912 | 5,688 | -28% | 1 | 1 | 0% | 1,338 | 906 | -32% | 0 | 0 | — |