Skip to content

Commit 80d9ec4

Browse files
[docs-infra] Improve export into sandbox package.json (#46044)
1 parent 55ce1c1 commit 80d9ec4

File tree

4 files changed

+20
-24
lines changed

4 files changed

+20
-24
lines changed

docs/src/modules/sandbox/CodeSandbox.test.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@ describe('CodeSandbox', () => {
2929
expect(result.files).to.deep.equal({
3030
'package.json': {
3131
content: {
32+
private: true,
3233
description:
3334
'https://github.com/mui/material-ui/blob/v5.7.0/docs/data/material/components/buttons/BasicButtons.js',
3435
dependencies: {
@@ -120,6 +121,7 @@ ReactDOM.createRoot(document.querySelector("#root")).render(
120121
expect(result.files).to.deep.equal({
121122
'package.json': {
122123
content: {
124+
private: true,
123125
description:
124126
'https://github.com/mui/material-ui/blob/v5.7.0/docs/data/material/components/buttons/BasicButtons.tsx',
125127
dependencies: {
@@ -136,7 +138,6 @@ ReactDOM.createRoot(document.querySelector("#root")).render(
136138
'@types/react': 'latest',
137139
'@types/react-dom': 'latest',
138140
},
139-
main: 'index.tsx',
140141
scripts: {
141142
build: 'react-scripts build',
142143
eject: 'react-scripts eject',

docs/src/modules/sandbox/CodeSandbox.ts

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -80,6 +80,7 @@ function createReactApp(demoData: DemoData) {
8080

8181
files['package.json'] = {
8282
content: {
83+
private: true,
8384
description,
8485
dependencies,
8586
devDependencies,
@@ -89,9 +90,6 @@ function createReactApp(demoData: DemoData) {
8990
test: 'react-scripts test',
9091
eject: 'react-scripts eject',
9192
},
92-
...(demoData.codeVariant === 'TS' && {
93-
main: 'index.tsx',
94-
}),
9593
},
9694
};
9795

@@ -174,6 +172,7 @@ ReactDOM.createRoot(document.querySelector("#root")${type}).render(
174172

175173
files['package.json'] = {
176174
content: {
175+
private: true,
177176
description,
178177
dependencies,
179178
devDependencies,
@@ -183,9 +182,6 @@ ReactDOM.createRoot(document.querySelector("#root")${type}).render(
183182
test: 'react-scripts test',
184183
eject: 'react-scripts eject',
185184
},
186-
...(templateData.codeVariant === 'TS' && {
187-
main: 'index.tsx',
188-
}),
189185
},
190186
};
191187

@@ -262,6 +258,7 @@ ReactDOM.createRoot(document.querySelector("#root")${type}).render(
262258

263259
files['package.json'] = {
264260
content: {
261+
private: true,
265262
description,
266263
dependencies,
267264
devDependencies,
@@ -271,9 +268,6 @@ ReactDOM.createRoot(document.querySelector("#root")${type}).render(
271268
test: 'react-scripts test',
272269
eject: 'react-scripts eject',
273270
},
274-
...(templateData.codeVariant === 'TS' && {
275-
main: 'index.tsx',
276-
}),
277271
},
278272
};
279273

docs/src/modules/sandbox/StackBlitz.test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -56,8 +56,8 @@ describe('StackBlitz', () => {
5656
</body>
5757
</html>`,
5858
'package.json': `{
59-
"name": "mui-demo",
6059
"private": true,
60+
"description": "https://github.com/mui/material-ui/blob/v5.7.0/docs/data/material/components/buttons/BasicButtons.js",
6161
"type": "module",
6262
"scripts": {
6363
"dev": "vite",
@@ -166,8 +166,8 @@ export default defineConfig({
166166
</body>
167167
</html>`,
168168
'package.json': `{
169-
"name": "mui-demo",
170169
"private": true,
170+
"description": "https://github.com/mui/material-ui/blob/v5.7.0/docs/data/material/components/buttons/BasicButtons.tsx",
171171
"type": "module",
172172
"scripts": {
173173
"dev": "vite",

docs/src/modules/sandbox/StackBlitz.ts

Lines changed: 13 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -47,8 +47,9 @@ function openStackBlitz({
4747
*/
4848
function createViteFiles(
4949
demoData: DemoData,
50-
dependencies: Record<string, string> = {},
51-
devDependencies: Record<string, string> = {},
50+
dependencies: Record<string, string>,
51+
devDependencies: Record<string, string>,
52+
description: string,
5253
): Record<string, string> {
5354
const ext = getFileExtension(demoData.codeVariant);
5455
return {
@@ -64,8 +65,8 @@ export default defineConfig({
6465
'index.html': CRA.getHtml({ ...demoData, main: `/src/index.${ext}` }),
6566
'package.json': JSON.stringify(
6667
{
67-
name: 'mui-demo',
6868
private: true,
69+
description,
6970
type: 'module',
7071
scripts: {
7172
dev: 'vite',
@@ -115,7 +116,7 @@ export default defineConfig({
115116
}
116117

117118
/**
118-
* Create a Material Template for StackBlitz using the SDK and Vite
119+
* Create a Material Template for StackBlitz using the SDK and Vite.
119120
*/
120121
function createJoyTemplate(templateData: {
121122
title: string;
@@ -139,7 +140,7 @@ function createJoyTemplate(templateData: {
139140
});
140141

141142
// Create base Vite files with dependencies
142-
const viteFiles = createViteFiles(demoData, dependencies, devDependencies);
143+
const viteFiles = createViteFiles(demoData, dependencies, devDependencies, description);
143144

144145
// Restructure template files to be under src/
145146
const templateSourceFiles = templateData.files
@@ -183,7 +184,7 @@ ReactDOM.createRoot(document.querySelector("#root")${type}).render(
183184
});
184185
return this;
185186
},
186-
openStackBlitz: (initialFile: string = `src/App`) => {
187+
openStackBlitz: (initialFile: string = 'src/App') => {
187188
openStackBlitz({
188189
title,
189190
description,
@@ -194,7 +195,7 @@ ReactDOM.createRoot(document.querySelector("#root")${type}).render(
194195
};
195196
}
196197
/**
197-
* Create a Material Template for StackBlitz using the SDK and Vite
198+
* Create a Material Template for StackBlitz using the SDK and Vite.
198199
*/
199200
function createMaterialTemplate(templateData: {
200201
title: string;
@@ -218,7 +219,7 @@ function createMaterialTemplate(templateData: {
218219
});
219220

220221
// Create base Vite files with dependencies
221-
const viteFiles = createViteFiles(demoData, dependencies, devDependencies);
222+
const viteFiles = createViteFiles(demoData, dependencies, devDependencies, description);
222223

223224
// Restructure template files to be under src/
224225
const templateSourceFiles = templateData.files
@@ -260,7 +261,7 @@ ReactDOM.createRoot(document.getElementById('root')${templateData.codeVariant ==
260261
});
261262
return this;
262263
},
263-
openStackBlitz: (initialFile: string = `src/App`) => {
264+
openStackBlitz: (initialFile: string = 'src/App') => {
264265
openStackBlitz({
265266
title,
266267
description,
@@ -272,8 +273,8 @@ ReactDOM.createRoot(document.getElementById('root')${templateData.codeVariant ==
272273
}
273274

274275
/**
275-
* Create a React App for StackBlitz using the SDK and Vite
276-
* This maintains similar structure to the original createReactApp but uses Vite
276+
* Create a React App for StackBlitz using the SDK and Vite.
277+
* This maintains similar structure to the original createReactApp but uses Vite.
277278
*/
278279
function createReactApp(demoData: DemoData) {
279280
const ext = getFileExtension(demoData.codeVariant);
@@ -285,7 +286,7 @@ function createReactApp(demoData: DemoData) {
285286
devDeps: VITE_DEV_DEPENDENCIES,
286287
});
287288

288-
const viteFiles = createViteFiles(demoData, dependencies, devDependencies);
289+
const viteFiles = createViteFiles(demoData, dependencies, devDependencies, description);
289290

290291
const demoFiles: Record<string, string> = {
291292
[`src/Demo.${ext}`]: flattenRelativeImports(demoData.raw),

0 commit comments

Comments
 (0)