Marat Al

Document our reliance on the side effect

1

>I've put a few thoughts here. Also, when I first looked at this PR, I was confused about why the `ARTSRWebSocket` class, which also makes use of `+[NSRunLoop ARTSR_networkRunLoop]`, hadn't...

Investigate using native NSURLSession web socket instead of SocketRocket

2

Try to implement `NSURLSessionWebSocketTask` functionality for `ARTWebSocket` protocol. ┆Issue is synchronized with this [Jira Task](https://ably.atlassian.net/browse/ECO-4441) by [Unito](https://www.unito.io)

We should only keep public only files that reflect types described in the IDL. Everything else should be hidden from users of this library. F.e. `ARTDeltaCodec.h`, `ARTHttp.h` etc. Related issue...

Replace `ARTStats.intervalGranularity` with `ARTStats.unit`

1

As it was deprecated here https://github.com/ably/canonical-api-reference-prototyping/pull/22/commits/2c8640415d81e43b4b6d400e6863820ecf06123b ┆Issue is synchronized with this [Jira Task](https://ably.atlassian.net/browse/ECO-4417) by [Unito](https://www.unito.io)

https://docs.ably.com/client-lib-development-guide/features/#TS10 https://docs.ably.com/client-lib-development-guide/features/#TS13 Implement also TS10c ┆Issue is synchronized with this [Jira Task](https://ably.atlassian.net/browse/ECO-4415) by [Unito](https://www.unito.io)

It should be just `push` https://docs.ably.com/client-lib-development-guide/features/#TS12m ┆Issue is synchronized with this [Jira Task](https://ably.atlassian.net/browse/ECO-4411) by [Unito](https://www.unito.io)

Comment to `ARTRestChannelsProtocol` and `ARTRealtimeChannelsProtocol` states that: > // We copy this from the parent class and replace ChannelType by ARTRealtimeChannel * because > // Swift ignores Objective-C generics and...

As in case of `Accept` header in this comment https://github.com/ably/ably-cocoa/pull/1420#discussion_r917117828 I think it's a legacy code in the `_time` function. Error creation is handled for all requests in the `executeRequest:...`...

>I've just noticed we should probably also set the Accept header on the request, [again similarly to -[ARTRest time:]] I think it's a legacy code. `Accept` header is set later...

Jazzy stopped crashing after I've replaced `--framework-root .` with `--framework-root Source`. But it still struggled with error `'AblyDeltaCodec/AblyDeltaCodec.h' file not found`. Then there were some manipulations with the make command...