fix: fix build and add CLI to server image

[alexsapps]

go.work/go.work.sum requires cli to be present. CLI isn't really used by the server, but copying the CLI files in helps it build anyway. The alternative seemed to be to create a 2nd version of go.work in a dedicated directory just for this Dockerfile which doesn't seem worth it.

Since we have to copy the CLI files in anyway, may as well build it and make it available in the server image! Should be useful for running the CLI in EC2.

Summary by CodeRabbit

• New Features

  • Go-based CLI tool is now bundled into the Docker container and accessible directly from the system PATH.

• Chores

  • Updated dependencies to include enhanced testing capabilities and upgraded authentication and communication libraries for improved stability and platform compatibility.

[coderabbitai]

📝 Walkthrough

Walkthrough

The pull request updates the Docker build process to include a Go-based CLI binary alongside the existing API server, and upgrades Go module dependencies in the server configuration. The CLI is built during the container build stage and made available in the PATH via a symlink.

Changes

Cohort / File(s)	Summary
Docker Build Pipeline Dockerfile	Extends the build-api stage to compile a Go CLI binary (/adb) alongside the existing API server. The final Alpine stage copies both binaries into /app and creates a symlink for CLI accessibility via /usr/local/bin/adb.
Go Module Dependencies server/src/go.mod	Updates golang.org/x/oauth2 from v0.22.0 to v0.35.0, adds testcontainers dependencies (v0.41.0), and upgrades Google Cloud and gRPC-related packages to newer versions including gRPC v1.79.1 and Protobuf v1.36.11.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A CLI hops into the Docker nest,
Built alongside the API with the rest,
Dependencies dance to versions new,
Symlinks skip through the PATH, it's true!
All in one container, bundled so fine! 🥕

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly describes the main changes: fixing the build issue and adding the CLI to the server image, which are the primary objectives of the PR.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch alex/fix-build

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@Dockerfile`:
- Line 40: The RUN step creating a symlink with "ln -s /app/adb
/usr/local/bin/adb" is not idempotent and fails if /usr/local/bin/adb already
exists; update the Dockerfile RUN command that references the symlink creation
to use a forceful, idempotent approach (e.g., replace the plain "ln -s"
invocation with a force option like "-sf" or an existence check + rm before
linking) so repeated builds or differing base images won't error when creating
/usr/local/bin/adb.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: b7712b24-dabf-4519-9ab6-43c2e34a3c4b

📥 Commits

Reviewing files that changed from the base of the PR and between 80bf2b8 and 7db2d2f.

⛔ Files ignored due to path filters (2)

• go.work.sum is excluded by !**/*.sum
• server/src/go.sum is excluded by !**/*.sum

📒 Files selected for processing (2)

• Dockerfile
• server/src/go.mod