Custom Well-Known-Type support

[Dogacel]

Add plugin support to write custom well-known types.

I think this feature would make pbandk much more convenient for some use-cases. For example we would like to use java.time.Instant instead of wkt.Timestamp.

We have some custom types such as UUID we would like to convert it to java.util.UUID.

I am trying to understand how this works:

pbandk/protoc-gen-pbandk/lib/src/commonMain/kotlin/pbandk/gen/Main.kt

Lines 36 to 44 in 120f51b

	val needToGenerate = request.fileToGenerate.contains(protoFile.name)
	// Convert the file to our model
	val file = FileBuilder.buildFile(FileBuilder.Context(protoFile, params.let {
	// As a special case, if we're not generating it but it's a well-known type package, change it our known one
	if (needToGenerate || packageName != "google.protobuf") it
	else it + ("kotlin_package" to "pbandk.wkt")
	}))

How can we modify this bit to inject custom WKTs?

[garyp]

I've actually got a huge rewrite of pbandk internals in progress that would make this much simpler to implement (the rewrite is for #217). It's been slow going, but hopefully I can get that up for review sometime soon. With the rewrite in place, we'll have a foundation for defining custom adapters per-type (e.g. use kotlin.time.Duration instead of pbandk.wkt.Duration for all google.protobuf.Duration messages) and per-field (e.g. use java.util.UUID instead of kotlin.String for a specific string field that is used to hold a UUID). Supporting this use case is an explicit goal of the rewrite (along with its primary purpose of adding support for builders) as it's something we'd like to have at Streem too.

[Dogacel]

I've actually got a huge rewrite of pbandk internals in progress that would make this much simpler to implement (the rewrite is for #217). It's been slow going, but hopefully I can get that up for review sometime soon. With the rewrite in place, we'll have a foundation for defining custom adapters per-type (e.g. use kotlin.time.Duration instead of pbandk.wkt.Duration for all google.protobuf.Duration messages) and per-field (e.g. use java.util.UUID instead of kotlin.String for a specific string field that is used to hold a UUID). Supporting this use case is an explicit goal of the rewrite (along with its primary purpose of adding support for builders) as it's something we'd like to have at Streem too.

That sounds amazing, it is exactly what we need.

[Dogacel]

Hi @garyp. I don't see any releases since last year November. Are there any updates on the progress of Pb and K?