chore: Adjust all required and optional fields in API

[Rockyy174]

No description provided.

[sourcery-ai]

Hey - I've left some high level feedback:

• In OperatelyWeb.Api.Goals.update_target/2, the repeated update_attrs = if attrs[:...], do: ... blocks could be simplified using something like attrs |> Map.take([:name, :start_value, :target_value, :unit]) |> Enum.reduce(%{}, ...) or by composing a mapping function, which would be easier to extend and less error-prone.
• Now that the goal and project *_access_level inputs are required integers in both the API schema and TS types, consider introducing a shared enum or dedicated type for access levels instead of plain number/:integer to avoid magic values and make incorrect usage harder at compile time.

Prompt for AI Agents

Please address the comments from this code review:

## Overall Comments
- In `OperatelyWeb.Api.Goals.update_target/2`, the repeated `update_attrs = if attrs[:...], do: ...` blocks could be simplified using something like `attrs |> Map.take([:name, :start_value, :target_value, :unit]) |> Enum.reduce(%{}, ...)` or by composing a mapping function, which would be easier to extend and less error-prone.
- Now that the goal and project `*_access_level` inputs are required integers in both the API schema and TS types, consider introducing a shared enum or dedicated type for access levels instead of plain `number`/`:integer` to avoid magic values and make incorrect usage harder at compile time.

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}