Separate classes from client code

[jacobparis]

Vite is pickier than the old compiler about server/client code separation

• all exports other than Remix's are required to be client safe, and it will fail the build if server-only code is used in one
• non-exported functions at top level are okay as long as they're only used in loaders and actions, HOWEVER that doesn't seem to apply to classes, and having top level classes that import server code triggers Remix Vite to fail the build. This might be a Remix bug and I've asked them about it

So this PR just goes through the offending files with top level service classes and moves them into sibling files

routes/
- api.v1.endpoint.ts
+ api.v1.endpoint/
+   route.ts
+   Service.server.ts

Closes #

✅ Checklist

• I have followed every step in the contributing guide
• The PR title follows the convention.
• I ran and tested the code works

---

Testing

[Describe the steps you took to test this change]

---

Changelog

[Short description of what has changed]

---

Screenshots

[Screenshots]

💯

[changeset-bot]

⚠️ No Changeset found

Latest commit: 74ed6ee

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR