Skip to content

feat(ui): project section grpc to REST API migration#6412

Draft
Marvin9 wants to merge 13 commits into
mainfrom
Marvin9/project-grpc-rest-migration
Draft

feat(ui): project section grpc to REST API migration#6412
Marvin9 wants to merge 13 commits into
mainfrom
Marvin9/project-grpc-rest-migration

Conversation

@Marvin9

@Marvin9 Marvin9 commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Description

Closes #6282

Checklist

Eligibility

  • Linked to an existing issue with no blocking labels (kind/proposal, needs discussion, needs research, maintainer only, area/security, size/large, size/x-large, size/xx-large).
  • Changes documentation only.
  • Changes ten lines or fewer.

Quality

  • Adds or updates corresponding tests.
  • Adds or updates corresponding documentation.

AI Use Disclosure

This PR was written:

  • By a human without AI assistance.
  • By a human with AI assistance. A human has reviewed every line prior to opening the PR.
  • By an AI with human supervision. A human has reviewed every line prior to opening the PR.
  • Entirely by an AI. No human has reviewed this prior to opening the PR.

Sign-Off

All commits:

  • Are signed off by their author (git commit -s) (required)
  • Are cryptographically signed (git commit -S) (encouraged)

Marvin9 and others added 8 commits May 26, 2026 15:44
Signed-off-by: Mayursinh Sarvaiya <marvinduff97@gmail.com>
Signed-off-by: Mayursinh Sarvaiya <marvinduff97@gmail.com>
Signed-off-by: Mayursinh Sarvaiya <marvinduff97@gmail.com>
Signed-off-by: Mayursinh Sarvaiya <marvinduff97@gmail.com>
…EST API

Rewrites the Watcher class to use SSE-based REST endpoints instead of
ConnectRPC streams, adds watchPromotions support, and migrates
stage-freight, artifact-selector-utils, argocd-link, and promotions
components from v1alpha1 protobuf types to v2 REST models. Fixes null
safety for last-promotion dates and uses per-type React Flow node
constants instead of the generic CUSTOM_NODE.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: Mayursinh Sarvaiya <marvinduff97@gmail.com>
Signed-off-by: Mayursinh Sarvaiya <marvinduff97@gmail.com>
Signed-off-by: Mayursinh Sarvaiya <marvinduff97@gmail.com>
@netlify

netlify Bot commented Jun 5, 2026

Copy link
Copy Markdown

Deploy Preview for docs-kargo-io ready!

Name Link
🔨 Latest commit 82e91a7
🔍 Latest deploy log https://app.netlify.com/projects/docs-kargo-io/deploys/6a294bb15ba13f0009a49c59
😎 Deploy Preview https://deploy-preview-6412.docs.kargo.io
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@kargo-governance-bot kargo-governance-bot Bot added area/ui Affects the UI kind/refactor Non-functional changes to implementation details needs/priority Priority has not yet been determined; a good signal that maintainers aren't fully committed labels Jun 5, 2026
@codecov

codecov Bot commented Jun 5, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 58.34%. Comparing base (5e02f7e) to head (82e91a7).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #6412   +/-   ##
=======================================
  Coverage   58.34%   58.34%           
=======================================
  Files         500      500           
  Lines       42015    42013    -2     
=======================================
- Hits        24515    24514    -1     
+ Misses      16029    16027    -2     
- Partials     1471     1472    +1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

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

Marvin9 and others added 5 commits June 9, 2026 18:21
Replaces monolithic Watcher with per-resource REST watch hooks
(stages, warehouses, promotions, freight) and updates all consumers
to use the new hooks and query cache API.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: Mayursinh Sarvaiya <marvinduff97@gmail.com>
Signed-off-by: Mayursinh Sarvaiya <marvinduff97@gmail.com>
Signed-off-by: Mayursinh Sarvaiya <marvinduff97@gmail.com>
@Marvin9

Marvin9 commented Jun 10, 2026

Copy link
Copy Markdown
Contributor Author

Remaining:

  • Refactor watch endpoints for performance (right now they are same as they were pre-performance improvement)
  • Analysis run module
  • Testing each and every affected change and cases

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

Labels

area/ui Affects the UI kind/refactor Non-functional changes to implementation details needs/priority Priority has not yet been determined; a good signal that maintainers aren't fully committed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

UI: API migration to REST - project

1 participant