Reviewed 3 of 3 files at r1.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @oncilla)

a discussion (no related file):
I know it's not related to this, but can you please also add that infra.ResponseWriterFromContext return false when h.request.Context() is nil?

go/lib/common/defs_test.go, line 31 at r1 (raw file):

		"typed nil":    (*A)(nil),
		"typed ptr":    &A{},
		"typed struct": A{},

Nit: you can't have an untyped struct / ptr, so might just call them ptr / struct.

Reviewable status: 3 of 6 files reviewed, 1 unresolved discussion (waiting on @scrye)

a discussion (no related file):

Done.
Will reflect in the commit message when merging.

Reviewed 4 of 4 files at r2.
Reviewable status: complete! all files reviewed, all discussions resolved