fix(serverless): log debug path from the HTTP request

[Ryank90]

Summary

• --debug path/status now come from the generated client's HTTPResponse (Request.URL.Path) instead of duplicated OpenAPI path literals in each wrapper.
• Addresses the nit on feat(builds): implement builds list and show methods #86 without rebasing the builds PR or the rest of the stack.
• Stacked on fix/env-redact-stack so logResponse already exists.

Test plan

• go test -race ./internal/api/serverless/
• make lint
• Confirm --debug still prints the request path after a list/show call

Made with Cursor

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. To trigger a review, include coderabbit-review in the PR description. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 8b2dae64-23d5-4e9b-b263-f4c194bb7d96

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

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.}

[Copilot]

Pull request overview

This pull request updates serverless client debug logging so the request path and status are derived from the generated OpenAPI client’s HTTPResponse (specifically Request.URL.Path), removing duplicated hard-coded path literals across wrapper methods and keeping debug output aligned with future OpenAPI regenerations.

Changes:

• Refactors logResponse to accept *http.Response and derive (path, status) via a helper.
• Updates serverless wrapper methods to pass resp.HTTPResponse into logResponse.
• Adds unit tests to validate path/status extraction behavior.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
internal/api/serverless/log.go	Changes logResponse signature to take *http.Response and adds responsePathStatus helper.
internal/api/serverless/log_test.go	Updates existing logging test and adds coverage for responsePathStatus.
internal/api/serverless/client.go	Switches debug logging calls to use resp.HTTPResponse instead of hard-coded paths.
internal/api/serverless/env.go	Switches debug logging calls to use resp.HTTPResponse instead of hard-coded paths.
internal/api/serverless/secrets.go	Switches debug logging calls to use resp.HTTPResponse instead of hard-coded paths (still nil body).

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.