Merge pull request #2887 from sveltejs/folder-structure

Folder structure

.gitignore

@@ -3,15 +3,15 @@
 .nyc_output
 node_modules
 *.map
-/src/compile/internal-exports.ts
+/src/compiler/compile/internal-exports.ts
 /compiler.*js
 /index.*js
-/internal.*js
-/store.*js
-/easing.*js
-/motion.*js
-/transition.*js
-/animate.*js
+/internal
+/store
+/easing
+/motion
+/transition
+/animate
 /scratch/
 /coverage/
 /coverage.lcov/

animate.d.ts

@@ -1 +0,0 @@
-export * from './types/animate';

easing.d.ts

@@ -1 +0,0 @@
-export * from './types/easing';

index.d.ts

@@ -1 +0,0 @@
-export * from './types/index';

internal.d.ts

@@ -1 +0,0 @@
-export * from './types/internal';

motion.d.ts

@@ -1 +0,0 @@
-export * from './types/motion';

package.json

@@ -9,15 +9,16 @@
     "compiler.*",
     "register.js",
     "index.*",
-    "internal.*",
-    "store.*",
-    "animate.*",
-    "transition.*",
-    "easing.*",
-    "motion.*",
+    "internal",
+    "store",
+    "animate",
+    "transition",
+    "easing",
+    "motion",
     "svelte",
     "README.md"
   ],
+  "types": "types/runtime",
   "scripts": {
     "test": "mocha --opts mocha.opts",
     "test:unit": "mocha --require sucrase/register --recursive ./**/__test__.ts",
@@ -30,8 +31,9 @@
     "prepare": "npm run build && npm run tsd",
     "dev": "rollup -cw",
     "pretest": "npm run build",
-    "posttest": "agadoo internal.mjs",
-    "prepublishOnly": "export PUBLISH=true && npm test",
+    "posttest": "agadoo internal/index.mjs",
+    "prepublishOnly": "export PUBLISH=true && npm test && npm run create-stubs",
+    "create-stubs": "node scripts/create-stubs.js",
     "tsd": "tsc -p . --emitDeclarationOnly",
     "typecheck": "tsc -p . --noEmit"
   },

rollup.config.js

@@ -9,7 +9,7 @@ import pkg from './package.json';
 
 const is_publish = !!process.env.PUBLISH;
 
-const tsPlugin = is_publish
+const ts_plugin = is_publish
 	? typescript({
 		include: 'src/**',
 		typescript: require('typescript')
@@ -18,39 +18,61 @@ const tsPlugin = is_publish
 		transforms: ['typescript']
 	});
 
+const external = id => id.startsWith('svelte/');
+
 export default [
-	/* internal.[m]js */
+	/* runtime */
 	{
-		input: `src/internal/index.ts`,
+		input: `src/runtime/index.ts`,
 		output: [
 			{
-				file: `internal.mjs`,
+				file: `index.mjs`,
 				format: 'esm',
-				paths: id => id.startsWith('svelte/') && id.replace('svelte', '.')
+				paths: id => id.startsWith('svelte/') && `${id.replace('svelte', '.')}`
 			},
 			{
-				file: `internal.js`,
+				file: `index.js`,
 				format: 'cjs',
-				paths: id => id.startsWith('svelte/') && id.replace('svelte', '.')
+				paths: id => id.startsWith('svelte/') && `${id.replace('svelte', '.')}`
 			}
 		],
-		external: id => id.startsWith('svelte/'),
+		external,
+		plugins: [ts_plugin]
+	},
 
-		plugins: [
-			tsPlugin,
-			{
-				generateBundle(options, bundle) {
-					const mod = bundle['internal.mjs'];
-					if (mod) {
-						fs.writeFileSync('src/compile/internal-exports.ts', `// This file is automatically generated\nexport default new Set(${JSON.stringify(mod.exports)});`);
+	...fs.readdirSync('src/runtime')
+		.filter(dir => fs.statSync(`src/runtime/${dir}`).isDirectory())
+		.map(dir => ({
+			input: `src/runtime/${dir}/index.ts`,
+			output: [
+				{
+					file: `${dir}/index.mjs`,
+					format: 'esm',
+					paths: id => id.startsWith('svelte/') && `${id.replace('svelte', '..')}`
+				},
+				{
+					file: `${dir}/index.js`,
+					format: 'cjs',
+					paths: id => id.startsWith('svelte/') && `${id.replace('svelte', '..')}`
+				}
+			],
+			external,
+			plugins: [
+				ts_plugin,
+				dir === 'internal' && {
+					generateBundle(options, bundle) {
+						const mod = bundle['index.mjs'];
+						if (mod) {
+							fs.writeFileSync('src/compiler/compile/internal-exports.ts', `// This file is automatically generated\nexport default new Set(${JSON.stringify(mod.exports)});`);
+						}
 					}
 				}
-			}]
-	},
+			]
+		})),
 
 	/* compiler.js */
 	{
-		input: 'src/compiler.ts',
+		input: 'src/compiler/index.ts',
 		plugins: [
 			replace({
 				__VERSION__: pkg.version
@@ -60,7 +82,7 @@ export default [
 				include: ['node_modules/**']
 			}),
 			json(),
-			tsPlugin
+			ts_plugin
 		],
 		output: {
 			file: 'compiler.js',
@@ -71,47 +93,5 @@ export default [
 		external: is_publish
 			? []
 			: id => id === 'acorn' || id === 'magic-string' || id.startsWith('css-tree')
-	},
-
-	/* motion.mjs */
-	{
-		input: `src/motion/index.ts`,
-		output: [
-			{
-				file: `motion.mjs`,
-				format: 'esm',
-				paths: id => id.startsWith('svelte/') && id.replace('svelte', '.')
-			},
-			{
-				file: `motion.js`,
-				format: 'cjs',
-				paths: id => id.startsWith('svelte/') && id.replace('svelte', '.')
-			}
-		],
-		plugins: [
-			tsPlugin
-		],
-		external: id => id.startsWith('svelte/')
-	},
-
-	// everything else
-	...['index', 'easing', 'transition', 'animate', 'store'].map(name => ({
-		input: `src/${name}.ts`,
-		output: [
-			{
-				file: `${name}.mjs`,
-				format: 'esm',
-				paths: id => id.startsWith('svelte/') && id.replace('svelte', '.')
-			},
-			{
-				file: `${name}.js`,
-				format: 'cjs',
-				paths: id => id.startsWith('svelte/') && id.replace('svelte', '.')
-			}
-		],
-		plugins: [
-			tsPlugin
-		],
-		external: id => id.startsWith('svelte/')
-	}))
+	}
 ];

scripts/create-stubs.js

@@ -0,0 +1,12 @@
+const fs = require('fs');
+
+fs.readdirSync('src/runtime')
+	.filter(dir => fs.statSync(`src/runtime/${dir}`).isDirectory())
+	.forEach(dir => {
+		fs.writeFileSync(`${dir}/package.json`, JSON.stringify({
+			main: './index.js',
+			module: './index.mjs'
+		}, null, '  '));
+
+		fs.writeFileSync(`${dir}/index.d.ts`, `export * from '../types/runtime/${dir}/index.d.ts';`);
+	});

src/compiler/compile/index.ts

@@ -1,4 +1,4 @@
-import { assign } from '../internal/index';
+import { assign } from '../../runtime/internal/index';
 import Stats from '../Stats';
 import parse from '../parse/index';
 import render_dom from './render-dom/index';

src/compiler/compile/nodes/shared/AbstractBlock.ts

@@ -1,5 +1,5 @@
 import Block from '../../render-dom/Block';
-import Component from './../../Component';
+import Component from '../../Component';
 import Node from './Node';
 import { INode } from '../interfaces';
 

src/compiler/compile/nodes/shared/Node.ts

@@ -1,5 +1,5 @@
-import Attribute from './../Attribute';
-import Component from './../../Component';
+import Attribute from '../Attribute';
+import Component from '../../Component';
 import { INode } from '../interfaces';
 import Text from '../Text';
 

src/compiler/compile/render-dom/wrappers/Element/Binding.ts

@@ -1,5 +1,5 @@
 import Binding from '../../../nodes/Binding';
-import ElementWrapper from '.';
+import ElementWrapper from '../Element';
 import { dimensions } from '../../../../utils/patterns';
 import get_object from '../../../utils/get_object';
 import Block from '../../Block';

src/compiler/compile/render-dom/wrappers/Element/StyleAttribute.ts

@@ -1,7 +1,7 @@
 import Attribute from '../../../nodes/Attribute';
 import Block from '../../Block';
 import AttributeWrapper from './Attribute';
-import ElementWrapper from '.';
+import ElementWrapper from '../Element';
 import { stringify } from '../../../utils/stringify';
 import add_to_set from '../../../utils/add_to_set';
 import Expression from '../../../nodes/shared/Expression';