The keyword 'interface' is reserved (Note that you need plugins to import files that are not JavaScript)

[ulyssesdotcodes]

Getting this error. It seems to crop up on some other repos, but I'm not sure what's causing it yet. Reporting and will investigate myself a little later today.

Steps to reproduce:

1. Fork repo using template
2. Clone repo into .obsidian/plugins directory
3. Run npm i
4. Run npm run dev

Expected
5. It compiles into a dev environment

Actual
5. It errors (copied below)

Thanks for the amazing product, looking forward to developing some plugins for it!

rollup v2.51.1
bundles main.ts → ...
[!] Error: The keyword 'interface' is reserved (Note that you need plugins to import files that are not JavaScript)
\\jouly\home\Obsidian\home\.obsidian\plugins\node-red-obsidian\main.ts (3:0)
1: import { App, Modal, Notice, Plugin, PluginSettingTab, Setting } from 'obsidian';
2:
3: interface NodeREDPluginSettings {
   ^
4:   nodeRedUrl: string;
5: }
Error: The keyword 'interface' is reserved (Note that you need plugins to import files that are not JavaScript)
    at error (U:\Obsidian\home\.obsidian\plugins\node-red-obsidian\node_modules\rollup\dist\shared\rollup.js:7917:30)
    at Module.error (U:\Obsidian\home\.obsidian\plugins\node-red-obsidian\node_modules\rollup\dist\shared\rollup.js:9859:16)
    at Module.tryParse (U:\Obsidian\home\.obsidian\plugins\node-red-obsidian\node_modules\rollup\dist\shared\rollup.js:10258:25)
    at Module.setSource (U:\Obsidian\home\.obsidian\plugins\node-red-obsidian\node_modules\rollup\dist\shared\rollup.js:10161:24)
    at ModuleLoader.addModuleSource (U:\Obsidian\home\.obsidian\plugins\node-red-obsidian\node_modules\rollup\dist\shared\rollup.js:19466:20)
    at async ModuleLoader.fetchModule (U:\Obsidian\home\.obsidian\plugins\node-red-obsidian\node_modules\rollup\dist\shared\rollup.js:19522:9)
    at async Promise.all (index 0)


[2021-06-10 09:37:59] waiting for changes...

[ulyssesdotcodes]

Related to rollup/plugins#892

[ulyssesdotcodes]

Removing nodeResolve({browser: true}) seems to fix the issue. As does making it point in such a way that it doesn't resolve node_modules.

[jimboc1]

Thanks for posting a solution as it worked for me too. Not sure of the consequences but I'll handle that when I come to it.

[ryanpcmcquen]

Can we remove nodeResolve from the default? The current state is broken, which really hurts the usability of this template.

[lishid]

I will review the sample template soon!

[lishid]

I have updated this template to use ESBuild instead of Rollup. Anyone interested in giving it a spin?

[lishid]

I've seen a few plugins using the new sample so I'm assuming everything is good to go!