Delegate StreamActions.refresh to Session

[seanpdoyle]

The bulk of the StreamActions.refresh implementation was reaching
through the global window.Turbo property, which itself was reaching
through either global variables or the Session.

This commit moves the implementation out of the StreamActions and into
a new Session.refresh(url, requestId) method. With that change, all
property access is encapsulated within the Session.

To support that change, this commit also introduces the
StreamElement.requestId property to read the [request-id] attribute.

[afcapel]

Makes sense, thanks @seanpdoyle 👍

[afcapel]

@seanpdoyle I just noticed that this commit introduced a circular dependency.

mbp 3.2.0 ~/Work/hotwired/turbo (ce4a2be...) yarn build
yarn run v1.22.19
$ rollup -c

src/index.js → dist/turbo.es2017-umd.js, dist/turbo.es2017-esm.js...
(!) Circular dependency
src/core/index.js -> src/core/streams/stream_actions.js -> src/core/index.js
created dist/turbo.es2017-umd.js, dist/turbo.es2017-esm.js in 345ms
✨  Done in 0.91s.

I'm going to temporarily revert the commit, but feel free to reopen a new PR without the circular dependency.

[seanpdoyle]

@afcapel I believe the circular dependency was re-introduced in response to https://github.com/hotwired/turbo/pull/1023/files#diff-dacdc6fe4d4b16718a923a226357e7750053bf3636e627ea06cff12345362137. I think that removing the StreamActions was important in making this change.

[seanpdoyle]

@afcapel I've confirmed locally that the circular dependency isn't mentioned during builds with this change:

diff --git a/src/core/index.js b/src/core/index.js
index b2ada17..27a2ad1 100644
--- a/src/core/index.js
+++ b/src/core/index.js
@@ -9,8 +9,6 @@ const session = new Session()
 const { cache, navigator } = session
 export { navigator, session, cache, PageRenderer, PageSnapshot, FrameRenderer, fetch }
 
-export { StreamActions } from "./streams/stream_actions"
-
 /**
  * Starts the main session.
  * This initialises any necessary observers such as those to monitor
diff --git a/src/index.js b/src/index.js
index 07e3e09..c3f2787 100644
--- a/src/index.js
+++ b/src/index.js
@@ -7,6 +7,7 @@ import * as Turbo from "./core"
 window.Turbo = Turbo
 Turbo.start()
 
+export { StreamActions } from "./core/streams/stream_actions"
 export * from "./core"
 export * from "./elements"
 export * from "./http"

[seanpdoyle]

I've opened #1049.

[afcapel]

Merged!