Skip to content

Commit 3d72a49

Browse files
committed
Add TODO items
1 parent 5c749b5 commit 3d72a49

1 file changed

Lines changed: 9 additions & 1 deletion

File tree

docs/v5-todo.md

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,14 +38,22 @@
3838

3939
- [ ] Understand commit deletion behavior: does it cascade to regressions?
4040
Revisit whether commits should be deletable now that they can be ordinal-free.
41-
- [ ] Allow including an ordinal when making a data submission.
41+
- [ ] Allow including the ordinal (and commit_fields) in run submissions, and
42+
make it a hard error if any submitted commit_field (or ordinal) clashes with
43+
existing values on the commit. Currently commit_fields use first-write-wins
44+
silently, and ordinals can only be set via PATCH.
4245
- [ ] Understand whether filtering `/commits` by machine and other properties
4346
would simplify baseline selection and other queries. Audit calls to `/tests`,
4447
`/machines`, and `/runs` for similar simplification opportunities.
4548

4649
### General
4750

4851
- [ ] Add count endpoints (or a count mode) for commits, runs, machines, etc.
52+
- [ ] Enforce that all schema-defined fields (metrics, commit_fields, and
53+
machine_fields) have an explicit type. Machine fields currently have no `type`
54+
attribute and are always `String(256)`. Once typed, validate submitted values
55+
against declared types at all CRUD endpoints (run submission, PATCH commit,
56+
PATCH machine, etc.) — e.g. reject a string for an `integer` commit_field.
4957
- [ ] Audit sort orders across all API endpoints for consistency. For example,
5058
should `/machines/{name}/runs` allow sorting on commit order?
5159
- [ ] Understand whether the regression detection tool would benefit from a

0 commit comments

Comments
 (0)