fusionjs/fusion-plugin-universal-events

Migrated to https://github.com/fusionjs/fusionjs

FUSION.JS

Modern framework for fast, powerful React apps

What is it?

fu·sion — noun

The process or result of joining two or more things together to form a single entity.

Fusion.js, Uber’s open source universal web framework, represents the fusion of the client and the server. It's geared for server-side rendering out of the box, and its plugin-driven architecture allows for complex frontend and backend logic to be encapsulated in a single plugin:

import App from 'fusion-react';
import Router from 'fusion-plugin-react-router';

export default () => {
  const app = new App(<div>...</div>);

  /*
  One line of code sets up everything you need for routing:
  - Server rendering
  - React Providers on both server and browser
  - Bundle splitting integration
  - Hot module reloading support
  */
  app.register(Router);

  return app;
}

We initially built Fusion.js to make our own websites easier to maintain, but were so impressed with the benefits that we decided to offer it to the community as an open source project!

Try it out

If you're interested in giving Fusion.js a shot, Overview and Core Concepts are great places to start.

Contributing

This is a monorepo of all open source Fusion.js packages maintained using Yarn v2. Take a look at CONTRIBUTING.md for info on how to develop in this repo.

License

MIT

fusion-plugin-universal-events's Issues

Batch events emitted from `EventEmitter.of(ctx)`

It is useful to batch events emitted on the server to
allow plugins to add mappers that are dependent on data
coming from middleware.

Fix bug where mappers were run before batch was flushed

Native fetch throws errors

Type of issue

Bug

Description

If you follow the getting started guide and register fetch via the window,

// ...
  __BROWSER__ && app.register(FetchToken, window.fetch)

this will fail in chrome with “Uncaught TypeError: Illegal invocation”, as the fetch is later executed during flush in the context of current object.

For this to work properly, it must be executed in the context of window, https://stackoverflow.com/a/10743608/1403990

A naive change is to modify the constructor of the UniversalEmitter:

this.fetch = fetch; // from
this.fetch = fetch.bind(window); // to

but i'm not sure how this would play out in older browsers, different fetch implementations, etc.

One other thing. This bug, is unreported to devs as the try catch surrounding the call to the fetch, swallows the error.

 try {
      const res = await this.fetch('/_events', {
        method: 'POST',
        headers: {
          'Content-Type': 'application/json',
        },
        body: JSON.stringify({items}),
      });

      if (!res.ok) {
        // sending failed so put the logs back into storage
        this.storage.addToStart(...items);
      }
    } catch (e) {
      // There should be some noise here during development
     // that this branch is being executed...
      this.storage.addToStart(...items);
    }

Add Token dependencies to readme

Problem/Rationale

Documentation regarding Fusion API is out of date given recent changes to leverage new Dependency Injection architecture.

Solution/Change/Deliverable

Update documentation

Pass string to createToken for better error messages

Passing objects to createToken results in poor error messages. We should
pass a string instead.

Ensure global mappers and handlers are called for events emitted from ctx scoped emitter

Allow server-side global emission with ctx

Example:
https://github.com/fusionjs/fusion-plugin-browser-performance-emitter/blob/54c21ae45a99a2719608873d13585b467ef668dd/src/server.js#L28-L31

  const emitter = EventEmitter.of();
  emitter.on('browser-performance-emitter:stats:browser-only', (e, ctx) => {
    emitter.emit('browser-performance-emitter:stats', mapPerfEvent(e), ctx);
  });

Currently, ctx is not passed in handleEvent() when emits from a global emitter service, which prevents plugin listeners on server-side to re-emit browser events with ctx

emit(type, payload, ctx) was the contract agreed upon, therefore we should still support it until a proper alternative.

Migrate fusion-plugin-universal-events to DI

Migrate to new fusion-tokens api

Use navigator.sendBeacon when available

It appears navigator.sendBeacon is fairly well supported across browsers: https://caniuse.com/#feat=beacon

Perhaps this API should be used when possible.

Fix bug with server side event batching

We need to handle the case where events are
emitted after the batch is flushed for a given
request.

Action Required: Fix Renovate Configuration

There is an error with this repository's Renovate configuration that needs to be fixed. As a precaution, Renovate will stop PRs until it is resolved.

Error type: Preset name not found within published preset config (monorepo:angularmaterial). Note: this is a nested preset so please contact the preset author if you are unable to fix it yourself.

Add support for cleanup function

Fix flow types

Update dependencies

Migrate fusion-plugin-universal-events to createPlugin

Add an initial implementation of the universal events plugin

Test issue

Support * event type for handlers and mappers

It can be useful to map or handle all events. For example,
an event logger that logs all events. We can support this
through an interface such as:

events.on('*', payload => {});
events.map('*', payload => {});

Use Buildkite and Docker for CI

Test issue