QuestionQ23

Eval, Testing, and Debugging

A teammate submitted a pull request that introduces a Claude-powered feature to your service. The code functions, but its prompt and model choice are hard-coded inline, it lacks error handling, and there are no tests for the integration. What would you request in code review?

Explanation

Prompt and model settings should be configurable, Claude API failures require explicit error handling for reliable service behavior, and integration tests are needed to validate the feature and prevent regressions. All three are necessary review changes before approval.

Community Discussion

No comments yet. Be the first to start the discussion!