tailwindcss 3.2.3 → 3.2.4

package/CHANGELOG.md

@@ -9,6 +9,17 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 - Nothing yet!
 
+## [3.2.4] - 2022-11-11
+
+### Added
+
+- Add `blocklist` option to prevent generating unwanted CSS ([#9812](https://github.com/tailwindlabs/tailwindcss/pull/9812))
+
+### Fixed
+
+- Fix watching of files on Linux when renames are involved ([#9796](https://github.com/tailwindlabs/tailwindcss/pull/9796))
+- Make sure errors are always displayed when watching for changes ([#9810](https://github.com/tailwindlabs/tailwindcss/pull/9810))
+
 ## [3.2.3] - 2022-11-09
 
 ### Fixed
@@ -2110,7 +2121,8 @@ No release notes
 
 - Everything!
 
-[unreleased]: https://github.com/tailwindlabs/tailwindcss/compare/v3.2.3...HEAD
+[unreleased]: https://github.com/tailwindlabs/tailwindcss/compare/v3.2.4...HEAD
+[3.2.4]: https://github.com/tailwindlabs/tailwindcss/compare/v3.2.3...v3.2.4
 [3.2.3]: https://github.com/tailwindlabs/tailwindcss/compare/v3.2.2...v3.2.3
 [3.2.2]: https://github.com/tailwindlabs/tailwindcss/compare/v3.2.1...v3.2.2
 [3.2.1]: https://github.com/tailwindlabs/tailwindcss/compare/v3.2.0...v3.2.1

package/lib/cli/build/plugin.js

@@ -311,6 +311,17 @@ async function createProcessor(args, cliConfigPath) {
             let end = process.hrtime.bigint();
             console.error();
             console.error("Done in", (end - start) / BigInt(1e6) + "ms.");
+        }).then(()=>{}, (err)=>{
+            // TODO: If an initial build fails we can't easily pick up any PostCSS dependencies
+            // that were collected before the error occurred
+            // The result is not stored on the error so we have to store it externally
+            // and pull the messages off of it here somehow
+            // This results in a less than ideal DX because the watcher will not pick up
+            // changes to imported CSS if one of them caused an error during the initial build
+            // If you fix it and then save the main CSS file so there's no error
+            // The watcher will start watching the imported CSS files and will be
+            // resilient to future errors.
+            console.error(err);
         });
     }
     /**

package/lib/cli/build/watching.js

@@ -70,12 +70,13 @@ function createWatcher(args, { state , rebuild  }) {
    *
    * @param {*} file
    * @param {(() => Promise<string>) | null} content
+   * @param {boolean} skipPendingCheck
    * @returns {Promise<void>}
-   */ function recordChangedFile(file, content = null) {
+   */ function recordChangedFile(file, content = null, skipPendingCheck = false) {
         file = _path.default.resolve(file);
         // Applications like Vim/Neovim fire both rename and change events in succession for atomic writes
         // In that case rebuild has already been queued by rename, so can be skipped in change
-        if (pendingRebuilds.has(file)) {
+        if (pendingRebuilds.has(file) && !skipPendingCheck) {
             return Promise.resolve();
         }
         // Mark that a rebuild of this file is going to happen
@@ -152,8 +153,10 @@ function createWatcher(args, { state , rebuild  }) {
                     return;
                 }
                 // This will push the rebuild onto the chain
+                // We MUST skip the rebuild check here otherwise the rebuild will never happen on Linux
+                // This is because the order of events and timing is different on Linux
                 // @ts-ignore: TypeScript isn't picking up that content is a string here
-                await recordChangedFile(filePath, ()=>content);
+                await recordChangedFile(filePath, ()=>content, true);
             } catch  {
             // If reading the file fails, it's was probably a deleted temporary file
             // So we can ignore it and no rebuild is needed

package/lib/lib/setupContextUtils.js

@@ -1098,13 +1098,15 @@ function registerPlugins(plugins, context) {
     markInvalidUtilityCandidate(context, candidate);
 }
 function createContext(tailwindConfig, changedContent = [], root = _postcss.default.root()) {
+    var _blocklist;
     let context = {
         disposables: [],
         ruleCache: new Set(),
         candidateRuleCache: new Map(),
         classCache: new Map(),
         applyClassCache: new Map(),
-        notClassCache: new Set(),
+        // Seed the not class cache with the blocklist (which is only strings)
+        notClassCache: new Set((_blocklist = tailwindConfig.blocklist) !== null && _blocklist !== void 0 ? _blocklist : []),
         postCssNodeCache: new Map(),
         candidateRuleMap: new Map(),
         tailwindConfig,

package/lib/util/normalizeConfig.js

@@ -162,6 +162,20 @@ function normalizeConfig(config) {
         if (Array.isArray(purge === null || purge === void 0 ? void 0 : (ref = purge.options) === null || ref === void 0 ? void 0 : ref.safelist)) return purge.options.safelist;
         return [];
     })();
+    // Normalize the `blocklist`
+    config.blocklist = (()=>{
+        let { blocklist  } = config;
+        if (Array.isArray(blocklist)) {
+            if (blocklist.every((item)=>typeof item === "string")) {
+                return blocklist;
+            }
+            _log.default.warn("blocklist-invalid", [
+                "The `blocklist` option must be an array of strings.",
+                "https://tailwindcss.com/docs/content-configuration#discarding-classes"
+            ]);
+        }
+        return [];
+    })();
     // Normalize prefix option
     if (typeof config.prefix === "function") {
         _log.default.warn("prefix-function", [

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tailwindcss",
-  "version": "3.2.3",
+  "version": "3.2.4",
   "description": "A utility-first CSS framework for rapidly building custom user interfaces.",
   "license": "MIT",
   "main": "lib/index.js",

package/src/cli/build/plugin.js

@@ -364,6 +364,23 @@ export async function createProcessor(args, cliConfigPath) {
         console.error()
         console.error('Done in', (end - start) / BigInt(1e6) + 'ms.')
       })
+      .then(
+        () => {},
+        (err) => {
+          // TODO: If an initial build fails we can't easily pick up any PostCSS dependencies
+          // that were collected before the error occurred
+          // The result is not stored on the error so we have to store it externally
+          // and pull the messages off of it here somehow
+
+          // This results in a less than ideal DX because the watcher will not pick up
+          // changes to imported CSS if one of them caused an error during the initial build
+          // If you fix it and then save the main CSS file so there's no error
+          // The watcher will start watching the imported CSS files and will be
+          // resilient to future errors.
+
+          console.error(err)
+        }
+      )
   }
 
   /**

package/src/cli/build/watching.js

@@ -97,14 +97,15 @@ export function createWatcher(args, { state, rebuild }) {
    *
    * @param {*} file
    * @param {(() => Promise<string>) | null} content
+   * @param {boolean} skipPendingCheck
    * @returns {Promise<void>}
    */
-  function recordChangedFile(file, content = null) {
+  function recordChangedFile(file, content = null, skipPendingCheck = false) {
     file = path.resolve(file)
 
     // Applications like Vim/Neovim fire both rename and change events in succession for atomic writes
     // In that case rebuild has already been queued by rename, so can be skipped in change
-    if (pendingRebuilds.has(file)) {
+    if (pendingRebuilds.has(file) && !skipPendingCheck) {
       return Promise.resolve()
     }
 
@@ -198,8 +199,10 @@ export function createWatcher(args, { state, rebuild }) {
         }
 
         // This will push the rebuild onto the chain
+        // We MUST skip the rebuild check here otherwise the rebuild will never happen on Linux
+        // This is because the order of events and timing is different on Linux
         // @ts-ignore: TypeScript isn't picking up that content is a string here
-        await recordChangedFile(filePath, () => content)
+        await recordChangedFile(filePath, () => content, true)
       } catch {
         // If reading the file fails, it's was probably a deleted temporary file
         // So we can ignore it and no rebuild is needed

package/src/lib/setupContextUtils.js

@@ -1165,7 +1165,8 @@ export function createContext(tailwindConfig, changedContent = [], root = postcs
     candidateRuleCache: new Map(),
     classCache: new Map(),
     applyClassCache: new Map(),
-    notClassCache: new Set(),
+    // Seed the not class cache with the blocklist (which is only strings)
+    notClassCache: new Set(tailwindConfig.blocklist ?? []),
     postCssNodeCache: new Map(),
     candidateRuleMap: new Map(),
     tailwindConfig,

package/src/util/normalizeConfig.js

@@ -150,6 +150,24 @@ export function normalizeConfig(config) {
     return []
   })()
 
+  // Normalize the `blocklist`
+  config.blocklist = (() => {
+    let { blocklist } = config
+
+    if (Array.isArray(blocklist)) {
+      if (blocklist.every((item) => typeof item === 'string')) {
+        return blocklist
+      }
+
+      log.warn('blocklist-invalid', [
+        'The `blocklist` option must be an array of strings.',
+        'https://tailwindcss.com/docs/content-configuration#discarding-classes',
+      ])
+    }
+
+    return []
+  })()
+
   // Normalize prefix option
   if (typeof config.prefix === 'function') {
     log.warn('prefix-function', [