Potential fixes for 2 code quality findings

[Jordonbc]

This PR applies 2/2 suggestions from code quality AI findings.

[github-actions]

Review Summary

The changes remove || true from test and fix commands, which is a positive change.

Quality Issues Fixed:

• Commands now properly propagate failures instead of silently passing
• CI/CD pipelines and developers will see actual test/type-check failures

No Bugs Introduced:

• This is purely behavioral cleanup, not functional code changes

Minor Suggestion:

The test recipe could potentially run all three commands even if one fails (using ; instead of &&), but the current approach is arguably better since it fails fast. If you want to see all failures regardless, you could change it to:

test:
  cd Frontend && npm exec tsc -- -p tsconfig.json --noEmit; \
  cd Frontend && npx vitest run; \
  cargo test --workspace

Otherwise, LGTM - the changes correctly address the code quality findings about masking failures.

opencode session  |  github run

[Jordonbc]

/oc can you fix the minor issue?

[opencode-agent]

The PR has already removed the || true from both test and fix commands. The current Justfile looks correct.

Are you referring to the optional suggestion in the review about running all three test commands even if one fails (using ; instead of the current fail-fast approach)? Or is there a different minor issue you'd like addressed?

opencode session  |  github run