Skip to content

feat: add project endpoints to OpenAPI spec - #394

Merged
gabor-boros merged 2 commits into
opcotech:mainfrom
apoorvdarshan:feat/297-add-project-endpoints
Aug 6, 2026
Merged

gabor-boros merged 2 commits into
opcotech:mainfrom
apoorvdarshan:feat/297-add-project-endpoints

Conversation

@apoorvdarshan

@apoorvdarshan apoorvdarshan commented Feb 17, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Add Project CRUD endpoints (GET/POST /v1/namespaces/{id}/projects, GET/PATCH/DELETE /v1/projects/{id}) to the OpenAPI specification with full Project schema, request bodies, and OAuth scopes
  • Regenerate server code and frontend client code
  • Add skeleton ProjectController wired into the HTTP server

Closes #297

Test plan

  • oapi-codegen completes without errors
  • go build ./... compiles successfully
  • Frontend client generation completes without errors

Developer Certificate of Origin

  • I have read and understood the Developer Certificate of Origin and added Signed-off-by: Apoorv Darshan to the commit trail.
@boring-cyborg

boring-cyborg Bot commented Feb 17, 2026

Copy link
Copy Markdown

Thanks for opening this pull request! 🥳
Please check out our contributing guide.
If you have any questions, please ask in the #development channel in our
Slack or
Discord.
Happy coding! 🚀

@vercel

vercel Bot commented Feb 17, 2026

Copy link
Copy Markdown

@apoorvdarshan is attempting to deploy a commit to the Opcotech Team on Vercel.

A member of the Team first needs to authorize it.

@gabor-boros

Copy link
Copy Markdown
Member

Hey @apoorvdarshan, thanks for the PR! I'm not sure how it went completely unnoticed from my side. I'm going to review it in the morning!

@gabor-boros

Copy link
Copy Markdown
Member

@apoorvdarshan It looks good to me! Could you please add Signed-off-by: <YOUR NAME> where where is your name, and "accept" the DCO? After that (and a rebase), I could merge the PR right away. 🙌

@apoorvdarshan
apoorvdarshan force-pushed the feat/297-add-project-endpoints branch from ec0115b to 12b3b9a Compare August 6, 2026 07:12
@apoorvdarshan

Copy link
Copy Markdown
Contributor Author

Added the DCO sign-off to the commit, accepted the DCO checklist in the PR description, and confirmed the branch is current with upstream main. Local non-container tests pass; Docker-backed integration suites require a running Docker daemon, so CI will provide that verification.

@gabor-boros

Copy link
Copy Markdown
Member

Let me resolve some conflicts I made and I'm going to merge. Thank you again for the contribution, @apoorvdarshan!

Add Project CRUD endpoints, schema, request bodies, and OAuth scopes
to the OpenAPI specification. Regenerate server and frontend client
code, and add skeleton ProjectController wired into the HTTP server.

Signed-off-by: Apoorv Darshan <ad13dtu@gmail.com>
@gabor-boros
gabor-boros force-pushed the feat/297-add-project-endpoints branch from 12b3b9a to 99d1d97 Compare August 6, 2026 07:17
Signed-off-by: Apoorv Darshan <ad13dtu@gmail.com>
@apoorvdarshan

Copy link
Copy Markdown
Contributor Author

Fixed the new Backend Lint failure by marking the intentionally unused parameters in the not-yet-implemented controller stubs as _. The focused HTTP transport tests and go vet pass locally. The remaining Vercel failures are authorization checks for the Opcotech deployment account and cannot be satisfied from a contributor fork.

@codecov

codecov Bot commented Aug 6, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 19 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
internal/transport/http/project.go 0.00% 17 Missing ⚠️
internal/transport/http/server.go 0.00% 2 Missing ⚠️
Files with missing lines Coverage Δ
internal/transport/http/server.go 0.00% <0.00%> (ø)
internal/transport/http/project.go 0.00% <0.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
@gabor-boros
gabor-boros merged commit abb2ac4 into opcotech:main Aug 6, 2026
21 of 24 checks passed
@boring-cyborg

boring-cyborg Bot commented Aug 6, 2026

Copy link
Copy Markdown

Awesome work, congrats on your first merged pull request! 🎉 🥳
Don't forget to join our
Slack or
Discord to stay in the loop!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment