proto3 repeated field cause error Illegal wire type for field Message.Field

[ryanhex53]

syntax = "proto3";

package com.jiaojiaoapp.protobuftest;

message ProtobufTest {
    float num = 1;
    bytes payload = 2;
    repeated int32 scores = 3;
}

var builder = ProtoBuf.loadProtoFile(path.join(__dirname, "protobuftest.proto")),
    Message = builder.build("com.jiaojiaoapp.protobuftest");
    var message = new Message.ProtobufTest({
        num: 1,
        payload: new Buffer('helloworld')
    });
    message.scores.push(1);
    message.scores.push(2);

this will cause error

[Error: Illegal wire type for field Message.Field .com.jiaojiaoapp.protobuftest.ProtobufTest.scores: 2 (0 expected)]

[ryanhex53]

resolved after add [packed=true], no more errors.

syntax = "proto3";

package com.jiaojiaoapp.protobuftest;

message ProtobufTest {
    float num = 1;
    bytes payload = 2;
    repeated int32 scores = 3 [packed=true];
}

but i think should treat it as packed by default as the google docs says:

In proto3, repeated fields of scalar numeric types use packed encoding by default.

[tanis2000]

Yes, according to Google docs, when using syntax = "proto3" the default behavior should be to consider repeated fields as [packed=true]
Any chance of this being corrected?

[jdimond]

Related: #390

[codeka]

I think this is a dupe of #432.

[dcodeIO]

Closing this for now.

protobuf.js 6.0.0

Feel free to send a pull request if this is still a requirement.