chore: moved theme plugins into monorepo

.gitignore

@@ -12,15 +12,18 @@ build-debug.log
 tsdoc-metadata.json
 .vscode
 
+# All packages
+build/
+dist/
+packages/*/test/**/*.d.ts
+packages/*/test/**/*.d.ts.map
+
 # Core
 packages/blockly/tests/compile/main_compressed.js
 packages/blockly/tests/compile/main_compressed.js.map
-packages/blockly/build/
-packages/blockly/dist/
 packages/blockly/temp/
 
 # Docs: 
 # Autogenerated reference docs, do not check in
 packages/docs/docs/reference/
 packages/docs/.docusaurus
-packages/docs/build/

.prettierignore

@@ -4,6 +4,13 @@
 eslint.config.mjs
 package.json
 
+# Package build aritifacts, etc.
+**/dist/*
+**/build/*
+**/node_modules/*
+**/package-lock.json
+**/CHANGELOG.md
+
 # Core: Build Artifacts
 packages/blockly/msg/*
 packages/blockly/build/*

.prettierrc.js

@@ -25,5 +25,16 @@ module.exports = {
         'plugins': ['prettier-plugin-organize-imports'],
       },
     },
+    {
+      files: ['packages/theme-*/**/*'],
+      options: {
+        // Some properties must be quoted to preserve closure compiler behavior.
+        // If at least one property in an object requires quotes, quote all 
+        // properties.
+        quoteProps: 'consistent',
+        // Don't add spaces around braces for object literals.
+        bracketSpacing: false,
+      },
+    },
   ],
 };

eslint.config.mjs

@@ -139,13 +139,15 @@ export default defineConfig(
   {
     // Note: there should be no other properties in this object
     ignores: [
-      // Build artifacts
+      // All Packages
+      'packages/*/node_modules/**',
+      'packages/**/build/',
+      'packages/**/dist/',
+      // Core Build artifacts
       'packages/blockly/msg/*',
-      'packages/blockly/build/*',
-      'packages/blockly/dist/*',
       'packages/blockly/typings/*',
       'packages/blockly/docs/*',
-      // Tests other than mocha unit tests
+      // Core Tests other than mocha unit tests
       'packages/blockly/tests/blocks/*',
       'packages/blockly/tests/themes/*',
       'packages/blockly/tests/compile/*',
@@ -155,8 +157,7 @@ export default defineConfig(
       'packages/blockly/tests/screenshot/*',
       'packages/blockly/tests/test_runner.js',
       'packages/blockly/tests/workspace_svg/*',
-      // Demos, scripts, misc
-      'packages/blockly/node_modules/*',
+      // Core Demos, scripts, misc
       'packages/blockly/generators/*',
       'packages/blockly/demos/*',
       'packages/blockly/appengine/*',
@@ -167,9 +168,7 @@ export default defineConfig(
       'packages/blockly/PULL_REQUEST_TEMPLATE.md',
       // Docs
       'packages/docs/docs/reference/**',
-      'packages/docs/build/**',
       'packages/docs/.docusaurus/**',
-      'packages/docs/node_modules/**',
     ],
   },
   jsdoc.configs['flat/recommended'],
@@ -353,6 +352,50 @@ export default defineConfig(
       'mdx/remark': 'off',
     },
   },
+  {
+    files: ['packages/theme-*/**'],
+    rules: {
+            'no-invalid-this': 'off',
+      // valid-jsdoc does not work properly for interface methods.
+      // https://github.com/eslint/eslint/issues/9978
+      'valid-jsdoc': 'off',
+      // https://github.com/gajus/eslint-plugin-jsdoc#eslint-plugin-jsdoc-rules
+      'require-jsdoc': 'off',
+      'jsdoc/newline-after-description': 'off',
+      // This should warn instead, but it's broken for long record type params.
+      'jsdoc/require-description-complete-sentence': 'off',
+      'jsdoc/require-returns': [
+        'error',
+        {
+          forceRequireReturn: false,
+        },
+      ],
+      'jsdoc/require-description': [
+        'warn',
+        {
+          // Don't require descriptions if these tags are present.
+          exemptedBy: ['inheritdoc', 'param', 'return', 'returns', 'type'],
+        },
+      ],
+      'jsdoc/check-tag-names': 'off',
+      'jsdoc/check-access': 'warn',
+      'jsdoc/check-types': 'off',
+      'jsdoc/check-values': 'off',
+      'jsdoc/reject-any-type': 'off',
+      'jsdoc/reject-function-type': 'off',
+      'jsdoc/require-jsdoc': [
+        'warn',
+        {
+          enableFixer: false,
+          require: {
+            FunctionDeclaration: true,
+            ClassDeclaration: true,
+            MethodDefinition: true,
+          },
+        },
+      ],
+    }
+  },
   ...tseslint.config(
     buildTSOverride({
       files: ['packages/blockly/**/*.ts', 'packages/blockly/**/*.tsx'],