Fix deprecation of "error" and "progress" fields in streaming responses

[thaJeztah]

• relates to pkg/jsonmessage: stop printing deprecated progressDetail, errorDetail, remove DisplayJSONMessagesToStream and Stream interface #49264
• relates to Refactor stream #2945
• relates to Do not require login unless 401 is received on push #1298

pkg/jsonmessage: JSONMessage: fix deprecation of ProgressMessage, ErrorMessage

• ErrorMessage was deprecated in 3043c26
  which was part of docker v0.6.0 / API v1.4
• ProgressMessage was deprecated in 597e0e6
  which was part of docker v0.7.1 / API v1.8

api: deprecation of "error" and "progress" fields in streaming responses

• error (ErrorMessage) was deprecated in 3043c26
  which was part of docker v0.6.0 / API v1.4
• progress (ProgressMessage) was deprecated in 597e0e6
  which was part of docker v0.7.1 / API v1.8

- Human readable description for the release notes

Go SDK: pkg/jsonmessage: JSONMessage: fix deprecation of ProgressMessage, ErrorMessage, which were deprecated in Docker v0.6.0 and v0.7.1 respectively.

API:

* Deprecated: The "error" and "progress" fields in streaming responses for
  endpoints that return a JSON progress response, such as `POST /images/create`,
  `POST /images/{name}/push`, and `POST /build` are deprecated. These fields
  were marked deprecated in API v1.4 (docker v0.6.0) and API v1.8 (docker v0.7.1)
  respectively, but still returned. These fields will be left empty or will
  be omitted in a future API version. Users should use the information in the
  `errorDetail` and `progressDetail` fields instead.

- A picture of a cute animal (not mandatory but encouraged)

[thaJeztah]

DERP; probably trailing whitespace in YAML; let me check

[thaJeztah]

OH! Nope; looks like we're generating some type from the swagger for one of these;

diff --git a/api/types/types_deprecated.go b/api/types/types_deprecated.go
index 76c04785cc..93e4336adc 100644
--- a/api/types/types_deprecated.go
+++ b/api/types/types_deprecated.go
@@ -9,7 +9,9 @@ import (
 	"github.com/docker/docker/api/types/storage"
 )
 
-// IDResponse Response to an API call that returns just an Id
+// IDResponse Response to an API call that returns just an Id.
+//
+// Deprecated: use either [container.CommitResponse] or [container.ExecCreateResponse]. It will be removed in the next release.
 type IDResponse = common.IDResponse
 
 // ContainerJSONBase contains response of Engine API GET "/containers/{name:.*}/json"

Please update api/swagger.yaml with any API changes, then 
run hack/generate-swagger-api.sh.

[thaJeztah]

Ignore me; probably just needs a rebase looking at that error

[thaJeztah]

Yay!

For those wanting to know; after fixing the first error... it was whitespace after all!

I ran hack/validate/yamllint, which (I found) doesn't validate the api/swagger.yaml (only the ones in docs/), and doesn't validate trailing whitespace.

So some follow-up PR incoming as well 😂