More protoc-gen-kaja tests & fixes by wham · Pull Request #253 · wham/kaja

wham · 2026-02-24T06:03:00Z

🎬 Demo

Video

Home

Call

Compiler

New Project

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…tput Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…om options Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…rder Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…pped in array Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…d in array Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…strings Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…umber order Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…-number order Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…on order Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…ration Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…ient.ts Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…ter differences Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…ons cast differences Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…camelCase Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…SDoc in grpc-client Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…SDoc in generic-server Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…eprecation Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

… types Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…tegy Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…ate status to DONE

tom-newotny and others added 30 commits February 21, 2026 13:24

Added test for WKT-typed custom field options (Duration/Timestamp)

b6b47ed

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Fixed WKT custom option codegen and WKT file filtering

7b818f4

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Added test for unquoted hyphenated json_name keys in custom option ou…

ff73daf

…tput Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Quoted custom option property keys containing non-identifier characters

2389631

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Added test for vertical tab in custom option string escaping

717dab0

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Added comprehensive JS string escaping for control characters in cust…

ee077ad

…om options Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Added test for integer map key ordering in custom options

0926a8d

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Sorted integer map keys in custom options to match JS Object.keys() o…

2993948

…rder Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Added test for single-element repeated field in custom option not wra…

3bc7a12

…pped in array Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Wrapped single-element repeated fields in arrays for custom options

8107fbd

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Added test for U+2028 LINE SEPARATOR escaping in custom option strings

7b3fe62

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Escaped U+2028 and U+2029 in JS string literals

d684872

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Added test for single-element repeated top-level extension not wrappe…

5b90b7a

…d in array Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Wrapped single-element repeated extension fields in arrays

5ba1673

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Updated RALPH.md with repeated extension fix notes

f56acbb

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Added test for unescaped string map keys in custom options

0d8748c

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Escaped special characters in string map keys for custom options

8c78fe5

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Added test for U+0085 NEXT LINE escaping in custom option strings

3fab7d1

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Fixed C1 control character and BOM escaping in JS string literals

da0e5f4

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Added test for null byte followed by digit escaping in custom option …

5362454

…strings Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Fixed null byte + digit escaping to use \x00 instead of \0

cfa4b0d

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Added test for cross-file extension ordering in custom options

574b031

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Fixed custom option ordering to use registry order instead of field n…

a90b9b5

…umber order Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Added test for custom option message field declaration order vs field…

c7707de

…-number order Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Fixed custom option field order to match message descriptor declarati…

e41bf60

…on order Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Added test for DEL character over-escaping in custom option strings

61fae1f

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Fixed DEL character escaping — C1 range starts at 0x80 not 0x7F

626da62

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Added test for uppercase hex digits in unicode escapes

5aed91a

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Updated NELSON notes with BOM/hex casing findings

6ab6684

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Fixed uppercase hex in Unicode escape sequences for BOM character

0fd53b0

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

tom-newotny and others added 27 commits February 22, 2026 06:13

Added test for ts.client = NO_CLIENT not suppressing client file gene…

d96b7a4

…ration Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Added ts.client = NO_CLIENT support to suppress client file generation

1c4a04f

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Added test for GRPC1_CLIENT generating .grpc-client.ts instead of .cl…

85c9aa0

…ient.ts Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Implemented gRPC client file generation for GRPC1_CLIENT option

65e5ae5

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Added test for GRPC1_CLIENT server-streaming optional metadata parame…

2012188

…ter differences Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Fixed gRPC client server streaming method signatures

70b7388

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Updated RALPH.md with test 276 fix notes

3155146

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Added test for GRPC1_CLIENT bidi streaming optional metadata and opti…

c50531d

…ons cast differences Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Fixed gRPC client bidi streaming optional metadata and options casting

3b7254c

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Added test for grpc-server definition name with underscored service name

2083004

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Fixed gRPC server definition name to lowercase first char instead of …

e623329

…camelCase Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Added test for GRPC1_CLIENT client-streaming import order mismatch

4f14de2

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Fixed gRPC client streaming import order

9ba9bd8

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Added test for service deprecation incorrectly propagated to method J…

1cdcc87

…SDoc in grpc-client Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Fixed gRPC client method deprecation not to cascade from service

69864a2

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Added test for service deprecation incorrectly propagated to method J…

7617022

…SDoc in generic-server Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Fixed generic server method @deprecated not inheriting from service d…

11d7e9f

…eprecation Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Added test for grpc-client import ordering with swapped cross-package…

9d2e703

… types Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Fixed gRPC client import ordering to use forward+prepend strategy

70c7703

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Added grpc-server cross-method import ordering test (283)

c6052f9

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Fixed gRPC server import ordering to use forward-iterate+prepend stra…

d494e67

…tegy Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Added test for cross-package type alias import ordering in main file

d66d32b

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Fixed service-only import ordering when file has messages

917c2c7

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Added test for .client.ts including GRPC1_CLIENT-only services

3fbc21b

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Filtered non-generic-client services from .client.ts generation

1a6bd74

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Filter non-generic-client services from .client.ts generation and upd…

8004124

…ate status to DONE

Remove status file from protoc-gen-kaja

fc6464b

wham marked this pull request as ready for review February 24, 2026 06:07

wham merged commit 911235e into main Feb 24, 2026
2 checks passed

wham deleted the wham/l2 branch February 24, 2026 06:07

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

More protoc-gen-kaja tests & fixes#253

More protoc-gen-kaja tests & fixes#253
wham merged 102 commits intomainfrom
wham/l2

wham commented Feb 24, 2026 •

edited by github-actions bot

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

wham commented Feb 24, 2026 • edited by github-actions bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Video

Home

Call

Compiler

New Project

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

wham commented Feb 24, 2026 •

edited by github-actions bot

Loading