emberjs / rfcs

RFCs for changes to Ember
https://rfcs.emberjs.com/
791 stars 408 forks source link

User-land template-escape into JS #805

Open NullVoxPopuli opened 2 years ago

NullVoxPopuli commented 2 years ago

Opening this issue to track ideas and exploration, and not so much to debate if we should or not (which is tiring, and we can talk about it on discord or in person/video call).

With First-Class Component templates merged, we can use the <template> syntax as a way of discussing strict-mode transforms in a concise, human-readable format. However, this RFC Issue assumes we have the following available in our hypothetical ember-source version:

This may accidentally come off as RFC-ish, but I just want to get my thoughts down real quick after having a chat with @jimschofield about some directions we can go in. It's def too early to submit an actual RFC on this idea -- need to do some exploration.

Presently, double curlies are defined as an escape to glimmer-S-Expressions, everything else is text or html -- which leaves a lot of room open to explore additional syntaxes while evaluating the utility of a concept.

For example, today in Ember, we don't have any concept of "Effects", yet they are highly needed in one of the bigger concepts missing from the switch to Octane (people formally used observers for this type of behavior).

In userland, as an addon author, it would be possible to transform this:

// some-component.gjs
<template>

  {() => console.log(@data)}

</template>

into a valid Ember template today:

const helperA = (data) => console.log(data);

<template>
  {{ (helperA @data) }}
</template>

Similarly, we can make some assumptions within our hypothetical transform to handle this in a class-based component, for example:

class Demo extends Component {
  @tracked searchText = '';

  // tho, what about concurrency, cancellation, dropping, etc? lots to figure out with something like this
  <template>
    {async () => {
      let { endpoint } = @searchProvider; // destructure named args
      let response = await fetch(`${endpoint}?query=${this.searchText}`);
      let json = await response.json();

      // set the auto-completion
      this.searchText = json.suggestions[0];
     }}
  </template>
}

would become:

import  { isDestroying, isDestroyed } from '@ember/destroyable';

const helperA = async (ctx, searchProvider) => {
  let { endpoint } = searchProvider; // destructure named args
  let response = await fetch(`${endpoint}?query=${ctx.searchText}`);
  let json = await response.json();

  // we can automatically protect people from destruction blunders
  if (isDestroying(ctx) || isDestroyed(ctx)) return;

  // set the auto-completion
  ctx.searchText = json.suggestions[0];
}

class Demo extends Component {
  @tracked searchText = '';

  <template>
    {{helperA this @searchProvider}}
  </template>
}

First off, why bother? This looks pretty close to JSX and implies a bunch of things which we might open ourselves up to if we aren't careful. But the motivation is that isn't a good way to implement "Effects" on a JS class.

Some options we have:

class MyComponent {
  @effect nameWasted = () => console.log(this.args.data);

  @effect(() => [this.args.data]
  nameNotNeeded(passedArg) {
    console.log(passedArg);
  }
}

Goals of an effect:

Problems with effects-in-classes

function Demo() {
  let [data] = useState();

  // runs every time data chages, due to the dependency list in the second argument
  useEffect(() => {
    console.log(data);
  }, [data])

  return <></>;
}

Some benefits of exploring this:

Outstanding things to figure out:

wagenet commented 2 years ago

What can be done to turn this into an actual RFC? I’d prefer for this repo not to be a place where good ideas go to sit indefinitely!

NullVoxPopuli commented 2 years ago

I think the next step is to (have someone) build this idea's proposed transform in user-land and have folks test it out.

Some of the syntax here (highlighting, tooling, etc) requires us to finish tooling support for the base <template> stuff (main things right now are eslint and template-lint, afaict).

wagenet commented 2 years ago

@NullVoxPopuli do you know of anyone interested in building the transform?

NullVoxPopuli commented 2 years ago

I know @lifeart has done most of the experimental transforms so far -- idk if they'd be interested in this one. My primary focus right now is limber.glimdown.com / glimdown in general / <template> / eslint rules / and maybe whatever else I need to meet my own goals.

I would totally experiment with this transform tho.