Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ES6 module no export Terminal #2878

Closed
elgs opened this issue Apr 27, 2020 · 29 comments
Closed

ES6 module no export Terminal #2878

elgs opened this issue Apr 27, 2020 · 29 comments
Assignees
Labels
area/build Regarding the build process type/enhancement Features or improvements to existing features
Milestone

Comments

@elgs
Copy link

elgs commented Apr 27, 2020

When I follow the doc to import Terminal as an ES6 module.

The recommended way to load xterm.js is via the ES6 module syntax:

import { Terminal } from 'xterm';

I got the following error. I opened the xterm.js and found it's an minified js file and it indeed didn't export Terminal.

The requested module '../../../node_modules/xterm/lib/xterm.js' does not provide an export named 'Terminal'

I'm not using any building tools. I just used plain ES6 import syntax.

@Tyriar
Copy link
Member

Tyriar commented Apr 27, 2020

I don't know the details here on what TS does but TS works for me. I guess this is requesting a native ESM target which has come up before #2486

@Tyriar Tyriar added the type/proposal A proposal that needs some discussion before proceeding label Apr 27, 2020
@elgs
Copy link
Author

elgs commented Apr 29, 2020

What I am wondering is I've seen a few other libraries claiming they are supporting ES6, but they are in the same situation like this, like this one highlightjs/highlight.js#2473. They use Rollup to build the library. I'm not familiar with Rollup.

My understanding of ES6 module is the library has some code syntax like:

export something = someFunction;

And the client is like:

import { something } from 'node_modules/someLib/someLib.js';

something();

I wonder if I misunderstood what ES6 is.

@elgs
Copy link
Author

elgs commented Apr 29, 2020

This is a big deal, because with the REAL ES6 support, the client can get rid of the dependency of build tools, though the build tools are awesome, but at least it's not necessary.

@Tyriar
Copy link
Member

Tyriar commented Apr 29, 2020

Yeah it would be nice to be on the latest and greatest but then you lose support for other better supported ways of importing. Eventually we'll move there but shipping a library that supports many different import types is tricky.

@elgs
Copy link
Author

elgs commented Apr 29, 2020

I understand that. That's why if you look at the lodash package, they released a separate package called lodash-es. That is the REAL ES6 support. But unfortunately the export syntax is illegal in the nodejs like module system.

You mentioned:

but then you lose support for other better supported ways of importing

Actually, no, you don't lose anything in browser. All modern build tools support the REAL ES6 syntax. What you lose is the support to build for nodejs users. But for packages like xterm, and highlightjs, why does it matter at all. Who use them in Nodejs.

@elgs
Copy link
Author

elgs commented Apr 29, 2020

lodash releases two separate packages for a wise reason, one for Nodejs and the other for browsers. Because the nature of lodash is that it is used in both Nodejs and browsers. In the old days, Javascript didn't have any module system in the browser, everything is bound to window, so that's why many build tools did many tricky things to translate the Nodejs module system to the browser window. That's understandable before REAL ES6 is usable in the browsers. Now ES6 is natively supported by browsers, packages like xterm and highlightjs should have no reason go around to the Nodejs module system.

@justinfagnani
Copy link

Is there any chance of moving to native modules in a major release? Every current major browser, and the past 3 major (soon to be 4 next moth) of Node, supports modules natively.

@Tyriar

then you lose support for other better supported ways of importing. Eventually we'll move there but shipping a library that supports many different import types is tricky.

I don't think there are better ways of importing than native modules. Every other way requires tools or userland loaders, and can be targeted with native modules and a tool anyway.

At this point I really don't think it'd even be worth it to publish two packages. Applications that support older browsers can really easily consume JS module dependencies and compile them to a different format. It's the default behavior for Rollup and Webpack.

One problem with not publishing standard JS modules is that you library is shut off from those that are using native modules. I'm trying to use a terminal widget for a project, but all my libraries are modules only, and there's no good way to import UMD from native modules.

@elgs
Copy link
Author

elgs commented Mar 8, 2021

As I learned more about the whole thing. Now I know this is kind of like a fake ES6 support. Webpack did the trick to allow ES6 like syntax to import Nodejs modules.

@TomzBench
Copy link

TomzBench commented Mar 9, 2021

@elgs

I made an es6 export version with a fork here:

https://github.com/TomzBench/xterm.js

It is a fork with a set of different tsconfig files + rollup to export an es module. Hopefully this works for you.

you can test it by importing @altronix/xterm

@elgs
Copy link
Author

elgs commented Mar 9, 2021

@TomzBench cool, thanks!

Can you please show an example assuming I don't have node or npm installed, which file should I import in the browser in order to use it?

@TomzBench
Copy link

I import like this

import {Terminal} from "@altronix/xterm"

@elgs
Copy link
Author

elgs commented Mar 9, 2021

Can you please tell me the absolute path of @altronix/xterm?

@TomzBench
Copy link

npm install @altronix/xterm

Then it will be inside your node modules. Are you trying to import from cdn? Or a script tag?

@elgs
Copy link
Author

elgs commented Mar 9, 2021

I am actually thinking about getting rid of the dependencies of node and rollup at all. I suddenly realized the source of this project is written in Typescript so it should be natively/immediately ready for Deno. And if we have for example an Angular project, we should be able to import the source code from this project without going through the rollup at all.

@TomzBench
Copy link

xtermjs typescript sources uses "paths" and referenced projects and outputs es5. So to have it build es6 output you can look at my fork and copy the tsconfig.es6.json files if you want to build it that way. You can copy the es6 xterm.js file and paste it where ever you need it if you don't want to use node.

@staff0rd
Copy link

I was getting this error with vite when using npm link & excluding xterm from vite's optimizeDeps config.

@TomzBench's fork fixes this.

@jogibear9988
Copy link

@Tyriar any news to real ES6 support?

@Tyriar
Copy link
Member

Tyriar commented Nov 19, 2021

Updates would be posted here

@walterjj
Copy link

@elgs

I made an es6 export version with a fork here:

https://github.com/TomzBench/xterm.js

It is a fork with a set of different tsconfig files + rollup to export an es module. Hopefully this works for you.

you can test it by importing @altronix/xterm

Good job! , Thankyou!

@Tyriar Tyriar added type/enhancement Features or improvements to existing features area/build Regarding the build process and removed type/proposal A proposal that needs some discussion before proceeding labels Dec 15, 2022
@Aurillium
Copy link

Is there any official progress or recommended solution on this issue?
This is still mentioned as the recommended way of importing the module on the website but it doesn't work.
The fork is also behind and isn't really a sustainable solution to the issue.

@jerch
Copy link
Member

jerch commented Jan 20, 2023

@Aurillium A proper move to full ESM support is quite involved, this mainly results from the fact, that TS has its own understanding of module resolution, and how far it allows to use ESM resolution beside the old nodejs resolution.

@Tyriar Could we do a lightweight transition during bundling, like creating an explicit ESM target for xterm.js and the addons? I currently do this in node-sixel with different tsconfig + webpack build targets, where the result is a proper ESM bundle. Downside - it is not properly reflected by node package exports, as this is more involved with the right entries in package.json, thus it will not work with pure nodejs ESM resolution yet. Still the browser would happily digest that as ESM.
I dont see yet a full move to ESM for xterm.js, as we would have to sort out, which parts should go into native ESM resolution, and which would always need some sort of prebundling. Furthermore cutting off the current nodejs module resolution will create many frictions on most embedding projects...

This is still mentioned as the recommended way of importing the module on the website but it doesn't work.

Currently the docs snippets are written for TS, not JS. Plus the idea, that your TS project is set up with nodejs module resolution. It just happens, that TS uses a similar syntax to JS-ESM, but in fact translates things to require(xy) calls underneath. Imho thats just a misunderstanding from the docs - note that the import syntax in the snippets does not contain '.js' endings on modules. TS uses that syntax way longer than ESM is out in the wild.

The fork is also behind and isn't really a sustainable solution to the issue.

Yes, for long term thats not a viable solution, as forks always tend to show that issue.

@TomzBench
Copy link

TomzBench commented Jan 20, 2023

I gave a quick try to resync with the master branch but i get some compile errors now. so i probably won't be able to fix them unless i need to use this package with vite again.

@johtso
Copy link

johtso commented Feb 2, 2023

I just tried to use xtermjs for the first time in a deno project, and immediately bumped into the The requested module 'npm:xterm@5.1.0' does not provide an export named 'Terminal' issue..

@Tyriar
Copy link
Member

Tyriar commented Feb 2, 2023

We're going to be forced to do this soon as VS Code is in the process of moving from AMD to ES6. I expect some progress on this in the next 2-6 months

@johtso
Copy link

johtso commented Feb 2, 2023

I managed to get things working by using the old version that @TomzBench kindly forked .. any tips on how I can import an addon? They have the same issue.

Edit:
Ah, just found @altronix/xterm-addon-fit

@Soesah
Copy link

Soesah commented Sep 4, 2023

Loading as import "xterm" and then simply using const terminal = new Terminal() works.

@thimpat
Copy link

thimpat commented Oct 1, 2023

The module doesn't seem to be ES6 compatible. I sent a temporary quick fix while waiting for the native export

import {Terminal, SearchAddon, WebLinksAddon, FitAddon} from "@thimpat/xterm";

@liudonghua123
Copy link

liudonghua123 commented Apr 1, 2024

I tried @altronix/xterm, but it's version 4.x and a little old, I forked and changed minimal code to output both esm and umd builds.

You can use it as import { Terminal } from "@liudonghua123/xterm";

OR

    <script type="importmap">
      {
        "imports": {
          "xterm": "https://unpkg.com/@liudonghua123/xterm@5.5.0/lib/xterm-esm.js"
        }
      }
    </script>

And some code also need to update, see also #4256 (comment).

@Tyriar Tyriar added this to the 5.6.0 milestone Jul 5, 2024
@Tyriar Tyriar self-assigned this Jul 5, 2024
@Tyriar
Copy link
Member

Tyriar commented Jul 7, 2024

Fixed with #5092! Anyone interested please test this out in the latest beta build

@Tyriar Tyriar closed this as completed Jul 7, 2024
@Tyriar Tyriar modified the milestones: 5.6.0, 6.0.0 Jul 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/build Regarding the build process type/enhancement Features or improvements to existing features
Projects
None yet
Development

No branches or pull requests