Copilot commented on code in PR #12649:
URL: https://github.com/apache/apisix/pull/12649#discussion_r3077974589
##########
apisix/plugins/grpc-transcode/proto.lua:
##########
@@ -60,6 +149,7 @@ local function compile_proto_text(content)
end
compiled[proto_fake_file].index = index
+ compiled.message_index = build_message_index(compiled[proto_fake_file])
Review Comment:
PR description mentions adding a `fetch_proto_array_names` function, but the
implementation actually builds `proto.message_index` and relies on
`set_default_array` + descriptor metadata to enforce JSON array encoding.
Please update the PR description (or rename references) to reflect the current
approach so reviewers/users aren't looking for a function that doesn't exist.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
