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

fix(files): normalize paths #595

Merged
merged 11 commits into from
Sep 30, 2022
Merged

fix(files): normalize paths #595

merged 11 commits into from
Sep 30, 2022

Conversation

danilowoz
Copy link
Member

@danilowoz danilowoz commented Sep 30, 2022

Closes #353

Normalizes leading slash on paths looks necessary to make the file tree state and the ability to overwrite template files more reliable. I've noted many cases where a mistyping might lead to the wrong sandbox configuration, especially when this file is a package.json.

Features:

  • Introduces normalizePath function to ensure leading slash on paths;

Fixes:

  • Normalizes path on adding new files to the Sandpack context;
  • Make sure custom entry overwrite main value on package.json file;
  • Normalizes activeFile and visibleFiles: it used to return values that didn't match with the file tree;
  • activeFile and visibleFiles have been ignored when the content was an empty string;

@codesandbox
Copy link

codesandbox bot commented Sep 30, 2022

CodeSandbox logoCodeSandbox logo  Open in CodeSandbox Web Editor | VS Code | VS Code Insiders | Storybook

@codesandbox-ci
Copy link

codesandbox-ci bot commented Sep 30, 2022

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit c924df2:

Sandbox Source
Sandpack Configuration
sandpack-with-codemirror-vim Configuration

@github-actions
Copy link

github-actions bot commented Sep 30, 2022

Size changes

sandpack-react

Total base (gzip) Total current (gzip) +/-
168 kb 168 kb 🎉 -11 bytes (-0.01%)
Details
Dependency name / file Base Current +/-
0 522 kb 522 kb ✅ 0 byte
main 505 kb 506 kb ⚠️ +248 bytes (+0.05%)
@codesandbox/sandpack-react 86 kb 86 kb ⚠️ +252 bytes (+0.29%)
@code-hike/classer 1 kb 1 kb ✅ 0 byte
@codesandbox/sandpack-client 8 kb 8 kb ✅ 0 byte
@stitches/core 16 kb 16 kb ✅ 0 byte
@codemirror/state 35 kb 35 kb ✅ 0 byte
@codemirror/view 167 kb 167 kb ✅ 0 byte
@codemirror/closebrackets 6 kb 6 kb ✅ 0 byte
@codemirror/matchbrackets 4 kb 4 kb ✅ 0 byte
@codemirror/commands 24 kb 24 kb ✅ 0 byte
@codemirror/comment 5 kb 5 kb ✅ 0 byte
@codemirror/gutter 10 kb 10 kb ✅ 0 byte
@codemirror/highlight 12 kb 12 kb ✅ 0 byte
@codemirror/history 9 kb 9 kb ✅ 0 byte
@react-hook/intersection-observer 2 kb 2 kb ✅ 0 byte
@codemirror/lang-css 11 kb 11 kb ✅ 0 byte
@codemirror/lang-javascript 6 kb 6 kb ✅ 0 byte
@codemirror/lang-html 15 kb 15 kb ✅ 0 byte
clean-set 229 bytes 229 bytes ✅ 0 byte
@codemirror/text 13 kb 13 kb ✅ 0 byte
style-mod 3 kb 3 kb ✅ 0 byte
@codemirror/rangeset 16 kb 16 kb ✅ 0 byte
w3c-keyname 2 kb 2 kb ✅ 0 byte
@lezer/common 34 kb 34 kb ✅ 0 byte
@codemirror/language 19 kb 19 kb ✅ 0 byte
@react-hook/passive-layout-effect 191 bytes 191 bytes ✅ 0 byte
@lezer/css 9 kb 9 kb ✅ 0 byte
@lezer/html 12 kb 12 kb ✅ 0 byte
@lezer/javascript 54 kb 54 kb ✅ 0 byte
@codemirror/autocomplete 34 kb 34 kb ✅ 0 byte
@lezer/lr 31 kb 31 kb ✅ 0 byte
@codemirror/tooltip 14 kb 14 kb ✅ 0 byte
lodash.isequal 18 kb 18 kb ✅ 0 byte
lz-string 9 kb 9 kb ✅ 0 byte
react-devtools-inline 823 kb 823 kb ✅ 0 byte
codesandbox-import-utils 3 kb 3 kb ✅ 0 byte
package-build-stats 487 bytes 487 bytes ✅ 0 byte
intersection-observer 13 kb 13 kb ✅ 0 byte
react-is 2 kb 2 kb ✅ 0 byte

sandpack-client

Total base (gzip) Total current (gzip) +/-
7 kb 7 kb ⚠️ +68 bytes (+0.94%)
Details
Dependency name / file Base Current +/-
main 20 kb 21 kb ⚠️ +204 bytes (+0.98%)
@codesandbox/sandpack-client 8 kb 9 kb ⚠️ +177 bytes (+2.06%)
codesandbox-import-utils 3 kb 3 kb ✅ 0 byte
lodash.isequal 18 kb 18 kb ✅ 0 byte
package-build-stats 487 bytes 487 bytes ✅ 0 byte

@danilowoz danilowoz merged commit 71a2044 into main Sep 30, 2022
@danilowoz danilowoz deleted the draft/boring-germain branch September 30, 2022 12:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Normalize files paths
1 participant