done
The done action marks issues as fixed or skipped. This updates Sheriff’s database and advances your progress.
Request
Section titled “Request”{ "action": "done", "fps": ["88d32cab35478753", "ab1c2d3e12345678"], "status": "fixed"}Parameters
Section titled “Parameters”| Parameter | Type | Required | Description |
|---|---|---|---|
fps | array | Yes | Array of fingerprints to mark |
status | string | Yes | "fixed" or "skip" (the server also accepts aliases "fix"/"f", "skipped"/"s", but schema-validating clients may only accept the canonical values) |
Response
Section titled “Response”{ "marked": 2, "prog": {"total": 136, "fixed": 115, "skip": 2, "rem": 19}}Response Fields
Section titled “Response Fields”| Field | Description |
|---|---|
marked | Number of issues marked |
prog | Updated progress summary (total, fixed, skip, rem) |
Status Options
Section titled “Status Options”Use when you’ve resolved the issue in the code:
{"action": "done", "fps": ["88d32cab35478753"], "status": "fixed"}Use for false positives or issues you choose not to fix:
{"action": "done", "fps": ["ab1c2d3e12345678"], "status": "skip"}Examples
Section titled “Examples”Mark Single Issue Fixed
Section titled “Mark Single Issue Fixed”{ "action": "done", "fps": ["88d32cab35478753"], "status": "fixed"}Mark Multiple Issues Fixed
Section titled “Mark Multiple Issues Fixed”{ "action": "done", "fps": ["88d32cab35478753", "ab1c2d3e12345678", "f9e8d7c6a1b2c3d4"], "status": "fixed"}Skip False Positive
Section titled “Skip False Positive”{ "action": "done", "fps": ["xyz1234567890abc"], "status": "skip"}Best Practices
Section titled “Best Practices”Fix All, Then Mark All
Section titled “Fix All, Then Mark All”Fix all issues in a file before calling done:
next→ Get 3 issues inService.java- Edit
Service.javato fix all 3 donewith all 3 fingerprints
Use Full Fingerprints
Section titled “Use Full Fingerprints”Always use the full fingerprint string returned by the next action. Sheriff performs exact matching on fingerprints:
{"fps": ["88d32cab35478753"]} ✓ Full fingerprint{"fps": ["88d32cab"]} ✗ Truncated — won't matchError Cases
Section titled “Error Cases”| Error Code | Cause | Solution |
|---|---|---|
SARIF_NOT_LOADED | No SARIF file has been loaded | Call load first |
INVALID_STATUS | Status is not fixed or skip | Use "fixed" or "skip" |
INVALID_FINGERPRINT | Fingerprints are missing or none exist | Provide valid fingerprints from next response |
DB_ERROR | Database error occurred | Check .sheriff/ directory permissions |
Undoing Marks
Section titled “Undoing Marks”Made a mistake? Use the reopen action to undo:
{"action": "reopen", "fps": ["88d32cab35478753"]}