Skip to content

feat/apis: add algorithm-specific payload options - #3589

Draft
Matts966 wants to merge 7 commits into
mainfrom
feature/grpc/common-any-args
Draft

feat/apis: add algorithm-specific payload options#3589
Matts966 wants to merge 7 commits into
mainfrom
feature/grpc/common-any-args

Conversation

@Matts966

@Matts966 Matts966 commented Aug 5, 2026

Copy link
Copy Markdown
Member

Summary

  • extract the shared gRPC/protobuf interface changes from feature/agent/zvec-any-options
  • add algorithm-specific google.protobuf.Any options to the payload API
  • include regenerated Go, Rust, Swagger, and API documentation artifacts
  • exclude the zvec runtime implementation, Helm configuration, and CI changes

Verification

  • ran make proto/all
  • buf generate completed successfully
  • applied the repository proto/replace transformations in smaller batches because macOS xargs exceeded its command-line length limit

Stacked PR

feature/agent/zvec-any-options is based on this branch.

@coderabbitai

coderabbitai Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

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: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: c6c91365-b759-4e1b-b8aa-07d6b4032b7c

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

Comment @coderabbitai help to get the list of available commands.

@vdaas-ci

vdaas-ci commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

[CHATOPS:HELP] ChatOps commands.

  • 🙆‍♀️ /approve - approve
  • 🍱 /format - format codes and add licenses
  • /gen-test - generate test codes
  • 🏷️ /label - add labels
  • 🔚 2️⃣ 🔚 /label actions/e2e-v2 - run E2E deploy & integration test (tests/v2/e2e)

@Matts966

Matts966 commented Aug 7, 2026

Copy link
Copy Markdown
Member Author

/format

@vdaas-ci

vdaas-ci commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

[FORMAT] Updating license headers and formatting go codes triggered by Matts966.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Aug 7, 2026

Copy link
Copy Markdown

Deploying vald with  Cloudflare Pages  Cloudflare Pages

Latest commit: ee05196
Status: ✅  Deploy successful!
Preview URL: https://6c4ed128.vald.pages.dev
Branch Preview URL: https://feature-grpc-common-any-args.vald.pages.dev

View logs

@vdaas-ci

vdaas-ci commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

[FORMAT] Failed to format.

@Matts966

Copy link
Copy Markdown
Member Author

/format

@vdaas-ci

Copy link
Copy Markdown
Collaborator

[FORMAT] Updating license headers and formatting go codes triggered by Matts966.

@Matts966

Copy link
Copy Markdown
Member Author

/format

@vdaas-ci

Copy link
Copy Markdown
Collaborator

[FORMAT] Updating license headers and formatting go codes triggered by Matts966.

@vdaas-ci

Copy link
Copy Markdown
Collaborator

[FORMAT] Failed to format.

Signed-off-by: Vdaas CI <vald@vdaas.org>
@github-actions github-actions Bot added size/XL and removed size/XL labels Aug 12, 2026

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

golangci

🚫 [golangci] reported by reviewdog 🐶
Magic number: 11, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 12, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 13, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 14, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 15, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 16, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 17, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 18, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 19, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 20, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 21, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 22, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 23, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 24, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 25, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 26, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 27, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
Magic number: 28, in detected (mnd)


🚫 [golangci] reported by reviewdog 🐶
ST1006: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (staticcheck)

func (this *ZVec_DocumentOptions) EqualVT(that *ZVec_DocumentOptions) bool {


🚫 [golangci] reported by reviewdog 🐶
ST1006: receiver name should be a reflection of its identity; don't use generic names such as "this" or "self" (staticcheck)

func (this *ZVec_DocumentOptions) EqualMessageVT(thatMsg proto.Message) bool {


🚫 [golangci] reported by reviewdog 🐶
unnecessary conversion (unconvert)

binary.LittleEndian.PutUint32(dAtA[i:], uint32(f1))

@@ -22893,7 +24373,7 @@ func (m *Object_VectorRequest) UnmarshalVT(dAtA []byte) error {
return nil
}

func (m *Object_Distance) UnmarshalVT(dAtA []byte) error {
func (m *Remove_MultiRequest) UnmarshalVT(dAtA []byte) error {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci] reported by reviewdog 🐶
24376-24546 lines are duplicate of apis/grpc/v1/payload/payload_vtproto.pb.go:31703-31873 (dupl)

@@ -24468,7 +25664,7 @@ func (m *Object_Location) UnmarshalVT(dAtA []byte) error {
return nil
}

func (m *Object_StreamLocation) UnmarshalVT(dAtA []byte) error {
func (m *Object_Timestamp) UnmarshalVT(dAtA []byte) error {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci] reported by reviewdog 🐶
25667-25768 lines are duplicate of apis/grpc/v1/payload/payload_vtproto.pb.go:31072-31173 (dupl)

}
switch fieldNum {
case 1:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci] reported by reviewdog 🐶
26315-26355 lines are duplicate of apis/grpc/v1/payload/payload_vtproto.pb.go:26613-26653 (dupl)

}
switch fieldNum {
case 1:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci] reported by reviewdog 🐶
26613-26653 lines are duplicate of apis/grpc/v1/payload/payload_vtproto.pb.go:26901-26941 (dupl)

@@ -28046,7 +31069,7 @@ func (m *Info_ResourceStats) UnmarshalVT(dAtA []byte) error {
return nil
}

func (m *Info_CgroupStats) UnmarshalVT(dAtA []byte) error {
func (m *Info_ServicePort) UnmarshalVT(dAtA []byte) error {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci] reported by reviewdog 🐶
31072-31173 lines are duplicate of apis/grpc/v1/payload/payload_vtproto.pb.go:32097-32198 (dupl)

break
}
}
case 6:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci] reported by reviewdog 🐶
Magic number: 6, in detected (mnd)

break
}
}
case 7:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci] reported by reviewdog 🐶
Magic number: 7, in detected (mnd)

break
}
}
case 8:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci] reported by reviewdog 🐶
Magic number: 8, in detected (mnd)

break
}
}
case 9:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci] reported by reviewdog 🐶
Magic number: 9, in detected (mnd)

break
}
}
case 10:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci] reported by reviewdog 🐶
Magic number: 10, in detected (mnd)

@Matts966

Copy link
Copy Markdown
Member Author

/format

@vdaas-ci

Copy link
Copy Markdown
Collaborator

[FORMAT] Updating license headers and formatting go codes triggered by Matts966.

Signed-off-by: Vdaas CI <vald@vdaas.org>
@github-actions github-actions Bot added size/XL and removed size/XL labels Aug 12, 2026
break
}
}
case 11:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci] reported by reviewdog 🐶
Magic number: 11, in detected (mnd)

break
}
}
case 12:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci] reported by reviewdog 🐶
Magic number: 12, in detected (mnd)

break
}
}
case 13:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci] reported by reviewdog 🐶
Magic number: 13, in detected (mnd)

break
}
}
case 14:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci] reported by reviewdog 🐶
Magic number: 14, in detected (mnd)

break
}
}
case 15:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci] reported by reviewdog 🐶
Magic number: 15, in detected (mnd)

v = uint64(binary.LittleEndian.Uint64(dAtA[iNdEx:]))
iNdEx += 8
m.MeanEdgeLength = float64(math.Float64frombits(v))
case 24:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci] reported by reviewdog 🐶
Magic number: 24, in detected (mnd)

v = uint64(binary.LittleEndian.Uint64(dAtA[iNdEx:]))
iNdEx += 8
m.MeanEdgeLengthFor10Edges = float64(math.Float64frombits(v))
case 25:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci] reported by reviewdog 🐶
Magic number: 25, in detected (mnd)

v = uint64(binary.LittleEndian.Uint64(dAtA[iNdEx:]))
iNdEx += 8
m.MeanIndegreeDistanceFor10Edges = float64(math.Float64frombits(v))
case 26:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci] reported by reviewdog 🐶
Magic number: 26, in detected (mnd)

v = uint64(binary.LittleEndian.Uint64(dAtA[iNdEx:]))
iNdEx += 8
m.MeanNumberOfEdgesPerNode = float64(math.Float64frombits(v))
case 27:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci] reported by reviewdog 🐶
Magic number: 27, in detected (mnd)

v = uint64(binary.LittleEndian.Uint64(dAtA[iNdEx:]))
iNdEx += 8
m.C1Indegree = float64(math.Float64frombits(v))
case 28:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci] reported by reviewdog 🐶
Magic number: 28, in detected (mnd)

@github-actions github-actions Bot added size/XL and removed size/XL labels Aug 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants