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

Audit every module we load in brackets.js to see if we can remove any #720

Open
humphd opened this issue Apr 15, 2017 · 0 comments
Open

Comments

@humphd
Copy link

humphd commented Apr 15, 2017

I did this once before, but I notice some stuff being loaded that we probably don't need (e.g., node connection stuff), and would save us some download size. Here is what we are currently bundling into brackets.js:

/Users/dave/Sites/repos/brackets/dist/main.js
----------------
/Users/dave/Sites/repos/brackets/src/main.js
/Users/dave/Sites/repos/brackets/src/utils/Compatibility.js
/Users/dave/Sites/repos/brackets/src/bramble/thirdparty/MessageChannel/uuid.core.js
/Users/dave/Sites/repos/brackets/src/bramble/thirdparty/MessageChannel/message_channel.js
/Users/dave/Sites/repos/brackets/src/widgets/bootstrap-dropdown.js
/Users/dave/Sites/repos/brackets/src/widgets/bootstrap-modal.js
/Users/dave/Sites/repos/brackets/src/widgets/bootstrap-twipsy-mod.js
/Users/dave/Sites/repos/brackets/src/thirdparty/CodeMirror/lib/codemirror.js
/Users/dave/Sites/repos/brackets/src/thirdparty/CodeMirror/addon/edit/closebrackets.js
/Users/dave/Sites/repos/brackets/src/thirdparty/CodeMirror/addon/fold/xml-fold.js
/Users/dave/Sites/repos/brackets/src/thirdparty/CodeMirror/addon/edit/closetag.js
/Users/dave/Sites/repos/brackets/src/thirdparty/CodeMirror/addon/edit/matchbrackets.js
/Users/dave/Sites/repos/brackets/src/thirdparty/CodeMirror/addon/edit/matchtags.js
/Users/dave/Sites/repos/brackets/src/thirdparty/CodeMirror/addon/mode/multiplex.js
/Users/dave/Sites/repos/brackets/src/thirdparty/CodeMirror/addon/mode/overlay.js
/Users/dave/Sites/repos/brackets/src/thirdparty/CodeMirror/addon/mode/simple.js
/Users/dave/Sites/repos/brackets/src/thirdparty/CodeMirror/addon/scroll/scrollpastend.js
/Users/dave/Sites/repos/brackets/src/thirdparty/CodeMirror/addon/search/searchcursor.js
/Users/dave/Sites/repos/brackets/src/thirdparty/CodeMirror/addon/scroll/annotatescrollbar.js
/Users/dave/Sites/repos/brackets/src/thirdparty/CodeMirror/addon/search/matchesonscrollbar.js
/Users/dave/Sites/repos/brackets/src/thirdparty/CodeMirror/addon/search/match-highlighter.js
/Users/dave/Sites/repos/brackets/src/thirdparty/CodeMirror/addon/selection/active-line.js
/Users/dave/Sites/repos/brackets/src/thirdparty/CodeMirror/addon/selection/mark-selection.js
/Users/dave/Sites/repos/brackets/src/thirdparty/CodeMirror/keymap/sublime.js
/Users/dave/Sites/repos/brackets/src/thirdparty/CodeMirror/mode/meta.js
/Users/dave/Sites/repos/brackets/src/thirdparty/CodeMirror/mode/css/css.js
/Users/dave/Sites/repos/brackets/src/thirdparty/CodeMirror/mode/xml/xml.js
/Users/dave/Sites/repos/brackets/src/thirdparty/CodeMirror/mode/javascript/javascript.js
/Users/dave/Sites/repos/brackets/src/thirdparty/CodeMirror/mode/htmlmixed/htmlmixed.js
/Users/dave/Sites/repos/brackets/src/thirdparty/CodeMirror/mode/jsx/jsx.js
/Users/dave/Sites/repos/brackets/src/thirdparty/CodeMirror/mode/markdown/markdown.js
/Users/dave/Sites/repos/brackets/src/utils/AppInit.js
/Users/dave/Sites/repos/brackets/src/thirdparty/lodash.js
/Users/dave/Sites/repos/brackets/src/utils/EventDispatcher.js
/Users/dave/Sites/repos/brackets/src/utils/Async.js
/Users/dave/Sites/repos/brackets/src/utils/UrlParams.js
/Users/dave/Sites/repos/brackets/src/bramble/config/config.prod.js
/Users/dave/Sites/repos/brackets/src/command/CommandManager.js
/Users/dave/Sites/repos/brackets/src/utils/DeprecationWarning.js
/Users/dave/Sites/repos/brackets/src/command/Commands.js
/Users/dave/Sites/repos/brackets/src/utils/ShellAPI.js
/Users/dave/Sites/repos/brackets/src/utils/Global.js
/Users/dave/Sites/repos/brackets/src/filesystem/FileSystemError.js
/Users/dave/Sites/repos/brackets/src/utils/StringUtils.js
/Users/dave/Sites/repos/brackets/src/utils/PerfUtils.js
/Users/dave/Sites/repos/brackets/src/thirdparty/i18n/i18n.js
/Users/dave/Sites/repos/brackets/src/nls/strings.js
/Users/dave/Sites/repos/brackets/src/nls/root/strings.js
i18n!nls/strings
/Users/dave/Sites/repos/brackets/src/nls/urls.js
/Users/dave/Sites/repos/brackets/src/nls/root/urls.js
i18n!nls/urls
/Users/dave/Sites/repos/brackets/src/nls/strings-app.js
/Users/dave/Sites/repos/brackets/src/nls/root/strings-app.js
i18n!nls/strings-app
/Users/dave/Sites/repos/brackets/src/strings.js
/Users/dave/Sites/repos/brackets/src/file/FileUtils.js
text!language/languages.json
/Users/dave/Sites/repos/brackets/src/language/LanguageManager.js
/Users/dave/Sites/repos/brackets/src/widgets/DefaultDialogs.js
/Users/dave/Sites/repos/brackets/src/filesystem/WatchedRoot.js
/Users/dave/Sites/repos/brackets/src/filesystem/FileSystemEntry.js
/Users/dave/Sites/repos/brackets/src/filesystem/Directory.js
/Users/dave/Sites/repos/brackets/src/filesystem/impls/filer/lib/content.js
/Users/dave/Sites/repos/brackets/src/filesystem/impls/filer/lib/queue.js
/Users/dave/Sites/repos/brackets/src/bramble/ChannelUtils.js
/Users/dave/Sites/repos/brackets/src/filesystem/impls/filer/RemoteFiler.js
/Users/dave/Sites/repos/brackets/src/thirdparty/filer/dist/path.min.js
/Users/dave/Sites/repos/brackets/src/thirdparty/filer/dist/buffer.min.js
/Users/dave/Sites/repos/brackets/src/filesystem/impls/filer/FilerUtils.js
/Users/dave/Sites/repos/brackets/src/filesystem/impls/filer/BracketsFiler.js
text!filesystem/impls/filer/lib/default.html
/Users/dave/Sites/repos/brackets/src/filesystem/File.js
/Users/dave/Sites/repos/brackets/src/filesystem/FileIndex.js
/Users/dave/Sites/repos/brackets/src/filesystem/FileSystemStats.js
/Users/dave/Sites/repos/brackets/src/thirdparty/immutable.js
/Users/dave/Sites/repos/brackets/src/bramble/StartupState.js
/Users/dave/Sites/repos/brackets/src/filesystem/impls/filer/BlobUtils.js
/Users/dave/Sites/repos/brackets/src/filesystem/impls/filer/lib/async.js
/Users/dave/Sites/repos/brackets/src/filesystem/impls/filer/lib/CSSRewriter.js
/Users/dave/Sites/repos/brackets/src/filesystem/impls/filer/lib/HTMLRewriter.js
/Users/dave/Sites/repos/brackets/src/thirdparty/path-utils/path-utils.js
/Users/dave/Sites/repos/brackets/src/utils/ExtensionUtils.js
/Users/dave/Sites/repos/brackets/src/thirdparty/marked/marked.min.js
/Users/dave/Sites/repos/brackets/src/filesystem/impls/filer/lib/transforms.js
/Users/dave/Sites/repos/brackets/src/filesystem/impls/filer/lib/handlers.js
/Users/dave/Sites/repos/brackets/src/bramble/BrambleEvents.js
/Users/dave/Sites/repos/brackets/src/filesystem/impls/filer/FileSystemCache.js
/Users/dave/Sites/repos/brackets/src/filesystem/impls/filer/FilerFileSystem.js
/Users/dave/Sites/repos/brackets/src/filesystem/FileSystem.js
/Users/dave/Sites/repos/brackets/src/utils/KeyEvent.js
text!base-config/keyboard.json
/Users/dave/Sites/repos/brackets/src/command/KeyBindingManager.js
text!htmlContent/dialog-template.html
/Users/dave/Sites/repos/brackets/src/thirdparty/mustache/mustache.js
/Users/dave/Sites/repos/brackets/src/widgets/Dialogs.js
text!htmlContent/project-settings-dialog.html
/Users/dave/Sites/repos/brackets/src/preferences/PreferencesDialogs.js
/Users/dave/Sites/repos/brackets/src/thirdparty/globmatch.js
/Users/dave/Sites/repos/brackets/src/preferences/PreferencesBase.js
/Users/dave/Sites/repos/brackets/src/preferences/PreferencesImpl.js
/Users/dave/Sites/repos/brackets/src/preferences/PreferencesManager.js
/Users/dave/Sites/repos/brackets/src/utils/ViewUtils.js
/Users/dave/Sites/repos/brackets/src/utils/Resizer.js
/Users/dave/Sites/repos/brackets/src/view/WorkspaceManager.js
/Users/dave/Sites/repos/brackets/src/view/MainViewFactory.js
/Users/dave/Sites/repos/brackets/src/view/ViewStateManager.js
/Users/dave/Sites/repos/brackets/src/document/InMemoryFile.js
text!htmlContent/pane.html
/Users/dave/Sites/repos/brackets/src/view/Pane.js
/Users/dave/Sites/repos/brackets/src/view/MainViewManager.js
/Users/dave/Sites/repos/brackets/src/utils/AnimationUtils.js
/Users/dave/Sites/repos/brackets/src/widgets/PopUpManager.js
/Users/dave/Sites/repos/brackets/src/command/Menus.js
/Users/dave/Sites/repos/brackets/src/document/TextRange.js
/Users/dave/Sites/repos/brackets/src/utils/TokenUtils.js
/Users/dave/Sites/repos/brackets/src/utils/ValidationUtils.js
/Users/dave/Sites/repos/brackets/src/editor/Editor.js
/Users/dave/Sites/repos/brackets/src/editor/InlineWidget.js
/Users/dave/Sites/repos/brackets/src/editor/InlineTextEditor.js
/Users/dave/Sites/repos/brackets/src/editor/EditorManager.js
/Users/dave/Sites/repos/brackets/src/document/Document.js
/Users/dave/Sites/repos/brackets/src/document/DocumentManager.js
/Users/dave/Sites/repos/brackets/src/project/FileViewController.js
/Users/dave/Sites/repos/brackets/src/project/FileTreeViewModel.js
/Users/dave/Sites/repos/brackets/src/project/ProjectModel.js
/Users/dave/Sites/repos/brackets/src/thirdparty/react.js
/Users/dave/Sites/repos/brackets/src/thirdparty/react-dom.js
/Users/dave/Sites/repos/brackets/src/thirdparty/classnames.js
/Users/dave/Sites/repos/brackets/src/project/FileTreeView.js
/Users/dave/Sites/repos/brackets/src/command/DefaultMenus.js
/Users/dave/Sites/repos/brackets/src/project/ProjectManager.js
/Users/dave/Sites/repos/brackets/src/project/FileSyncManager.js
text!extensions/bramble-extensions.json
/Users/dave/Sites/repos/brackets/src/utils/BrambleExtensionLoader.js
/Users/dave/Sites/repos/brackets/src/utils/ExtensionLoader.js
/Users/dave/Sites/repos/brackets/src/thirdparty/jszip/dist/jszip.min.js
/Users/dave/Sites/repos/brackets/src/thirdparty/FileSaver.js
/Users/dave/Sites/repos/brackets/src/filesystem/impls/filer/ArchiveUtils.js
/Users/dave/Sites/repos/brackets/src/utils/DragAndDrop.js
/Users/dave/Sites/repos/brackets/src/utils/NativeApp.js
text!htmlContent/themes-settings.html
/Users/dave/Sites/repos/brackets/src/view/ThemeSettings.js
/Users/dave/Sites/repos/brackets/src/view/ViewCommandHandlers.js
text!htmlContent/main-view.html
/Users/dave/Sites/repos/brackets/src/document/ChangedDocumentTracker.js
/Users/dave/Sites/repos/brackets/src/language/JSUtils.js
text!htmlContent/working-set.html
/Users/dave/Sites/repos/brackets/src/project/WorkingSetView.js
/Users/dave/Sites/repos/brackets/src/search/FindUtils.js
/Users/dave/Sites/repos/brackets/src/utils/HealthLogger.js
/Users/dave/Sites/repos/brackets/src/LiveDevelopment/Inspector/Inspector.js
text!widgets/StatusBar.html
/Users/dave/Sites/repos/brackets/src/widgets/StatusBar.js
/Users/dave/Sites/repos/brackets/src/document/DocumentCommandHandlers.js
text!htmlContent/code-hint-list.html
/Users/dave/Sites/repos/brackets/src/editor/CodeHintList.js
/Users/dave/Sites/repos/brackets/src/editor/CodeHintManager.js
text!htmlContent/problems-panel.html
text!htmlContent/problems-panel-table.html
/Users/dave/Sites/repos/brackets/src/language/CodeInspection.js
/Users/dave/Sites/repos/brackets/src/language/HTMLUtils.js
/Users/dave/Sites/repos/brackets/src/language/CSSUtils.js
/Users/dave/Sites/repos/brackets/src/utils/DropdownEventHandler.js
/Users/dave/Sites/repos/brackets/src/widgets/DropdownButton.js
/Users/dave/Sites/repos/brackets/src/editor/MultiRangeInlineEditor.js
/Users/dave/Sites/repos/brackets/src/editor/CSSInlineEditor.js
/Users/dave/Sites/repos/brackets/src/project/WorkingSetSort.js
/Users/dave/Sites/repos/brackets/src/widgets/ModalBar.js
/Users/dave/Sites/repos/brackets/src/search/QuickSearchField.js
/Users/dave/Sites/repos/brackets/src/utils/StringMatch.js
/Users/dave/Sites/repos/brackets/src/search/QuickOpen.js
/Users/dave/Sites/repos/brackets/src/search/QuickOpenHelper.js
/Users/dave/Sites/repos/brackets/src/project/SidebarView.js
/Users/dave/Sites/repos/brackets/src/LiveDevelopment/LiveDevelopmentUtils.js
/Users/dave/Sites/repos/brackets/src/LiveDevelopment/LiveDevServerManager.js
text!LiveDevelopment/MultiBrowserImpl/protocol/remote/LiveDevProtocolRemote.js
text!LiveDevelopment/MultiBrowserImpl/protocol/remote/DocumentObserver.js
text!LiveDevelopment/Agents/RemoteFunctions.js
/Users/dave/Sites/repos/brackets/src/LiveDevelopment/MultiBrowserImpl/protocol/LiveDevProtocol.js
/Users/dave/Sites/repos/brackets/src/LiveDevelopment/MultiBrowserImpl/documents/LiveDocument.js
/Users/dave/Sites/repos/brackets/src/LiveDevelopment/MultiBrowserImpl/documents/LiveCSSDocument.js
/Users/dave/Sites/repos/brackets/src/language/HTMLDOMDiff.js
/Users/dave/Sites/repos/brackets/src/language/HTMLTokenizer.js
/Users/dave/Sites/repos/brackets/src/thirdparty/murmurhash3_gc.js
/Users/dave/Sites/repos/brackets/src/LiveDevelopment/MultiBrowserImpl/language/HTMLSimpleDOM.js
/Users/dave/Sites/repos/brackets/src/LiveDevelopment/MultiBrowserImpl/language/HTMLInstrumentation.js
/Users/dave/Sites/repos/brackets/src/LiveDevelopment/MultiBrowserImpl/documents/LiveHTMLDocument.js
/Users/dave/Sites/repos/brackets/src/LiveDevelopment/Servers/BaseServer.js
/Users/dave/Sites/repos/brackets/src/LiveDevelopment/LiveDevMultiBrowser.js
text!LiveDevelopment/main.less
/Users/dave/Sites/repos/brackets/src/LiveDevelopment/main.js
/Users/dave/Sites/repos/brackets/src/utils/ColorUtils.js
/Users/dave/Sites/repos/brackets/src/view/ThemeView.js
/Users/dave/Sites/repos/brackets/src/view/ThemeManager.js
/Users/dave/Sites/repos/brackets/src/language/XMLUtils.js
/Users/dave/Sites/repos/brackets/src/language/JSONUtils.js
text!htmlContent/copy-paste-dialog.html
/Users/dave/Sites/repos/brackets/src/editor/EditorCommandHandlers.js
/Users/dave/Sites/repos/brackets/src/editor/EditorOptionHandlers.js
/Users/dave/Sites/repos/brackets/src/editor/EditorStatusBar.js
text!htmlContent/image-view.html
/Users/dave/Sites/repos/brackets/src/thirdparty/Vibrant.js
/Users/dave/Sites/repos/brackets/src/editor/ImageViewer.js
text!htmlContent/edit-filter-dialog.html
text!htmlContent/filter-name.html
/Users/dave/Sites/repos/brackets/src/search/FileFilters.js
text!htmlContent/findreplace-bar.html
/Users/dave/Sites/repos/brackets/src/search/FindBar.js
/Users/dave/Sites/repos/brackets/src/search/SearchModel.js
/Users/dave/Sites/repos/brackets/src/utils/NodeConnection.js
/Users/dave/Sites/repos/brackets/src/utils/NodeDomain.js
/Users/dave/Sites/repos/brackets/src/search/FindInFiles.js
text!htmlContent/search-panel.html
text!htmlContent/search-results.html
text!htmlContent/search-summary.html
/Users/dave/Sites/repos/brackets/src/search/SearchResultsView.js
/Users/dave/Sites/repos/brackets/src/search/FindInFilesUI.js
/Users/dave/Sites/repos/brackets/src/search/ScrollTrackMarkers.js
/Users/dave/Sites/repos/brackets/src/search/FindReplace.js
/Users/dave/Sites/repos/brackets/src/view/PanelManager.js
/Users/dave/Sites/repos/brackets/src/brackets.js

Some things to see if we can dump:

/Users/dave/Sites/repos/brackets/src/thirdparty/CodeMirror/addon/fold/xml-fold.js
/Users/dave/Sites/repos/brackets/src/utils/NativeApp.js
/Users/dave/Sites/repos/brackets/src/utils/NodeConnection.js
/Users/dave/Sites/repos/brackets/src/utils/NodeDomain.js
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

No branches or pull requests

1 participant