Systems Review

After implementing, let's review for second-order effects:

Implementation Summary

• What was built? Brief description.

• What changed? Files modified.

• What's the interface? Input/output contracts.

Performance Review

Analyze performance implications:

• What's the time complexity?

• What happens at 10x scale?

• Where are potential bottlenecks?

• Are there N+1 query risks?

• Is caching appropriate?

Security Review

Check for security issues:

• Is input validated?

• Is authorization checked?

• Could this be exploited?

• Is sensitive data protected?

• Are there injection risks?

Downstream Effects

Consider blast radius:

• What breaks if this fails?

• What systems depend on this?

• Are there race conditions?

• Could this cause cascading failures?

Test Coverage

Verify testing:

• Is the happy path tested?

• Are error paths tested?

• Are edge cases covered?

• Is the interface contract verified?

Recommendations

Based on review, provide:

• Critical - Must fix before shipping

• High - Should fix soon

• Medium - Consider improving

• Low - Nice to have

Sign-off Checklist

• Decomposition was followed

• Interfaces are verified

• Error handling is complete

• Performance is acceptable

• Security is reviewed

• Tests are adequate