Code Review

Use this skill when reviewing SynapseML changes.

Steps

1. Inspect diff: git --no-pager diff --stat && git --no-pager diff
2. Run Scala style: sbt scalastyle test:scalastyle
3. Run Python format check: black --check --extend-exclude 'docs/' .
4. Run targeted tests for touched code.
5. Apply the checklists below to every changed file.
6. Report only concrete issues with file paths and fixes.

Security Checklist

Apply when changes touch serialization, I/O, network, or authentication code.

Deserialization (CWE-502)

• No raw ObjectInputStream.readObject() — use SafeObjectInputStream with an allowlist