postcss 8.0.7 → 8.1.1

package/CHANGELOG.md

@@ -1,6 +1,22 @@
 # Change Log
 This project adheres to [Semantic Versioning](http://semver.org/).
 
+## 8.1.1
+* Update funding link.
+
+## 8.1 “Duke Gemory”
+* Added `Once` and `OnceExit` events.
+* Fixed `Root` and `RootExit` events re-visiting.
+* Fixed node re-visiting on deep children changes.
+* Added docs for visitor API events.
+
+## 8.0.9
+* Replace prototype in PostCSS 7 nodes instead of recreating them.
+* Added missed `Transformer` to exported types (by Pierre-Marie Dartus).
+
+## 8.0.8
+* Fix `8.0.7` regression on PostCSS 7 nodes converting (by Adam Wathan).
+
 ## 8.0.7
 * Fixed compatibility issue with mixin AST with PostCSS 7 and 8 nodes.
 * Added migration guide translation to Chinese to the warning.
@@ -55,6 +71,9 @@ This project adheres to [Semantic Versioning](http://semver.org/).
 * Removed docs from npm package.
 * Replaced `chalk` to `colorette`.
 
+## 7.0.35
+* Add migration guide link to PostCSS 8 error text.
+
 ## 7.0.34
 * Fix compatibility with `postcss-scss` 2.
 

package/README.md

@@ -371,7 +371,7 @@ prefixer({ display: 'flex' }) //=> { display: ['-webkit-box', '-webkit-flex', '-
 
 ### Runners
 
-* **Grunt**: [`grunt-postcss`](https://github.com/nDmitry/grunt-postcss)
+* **Grunt**: [`@lodder/grunt-postcss`](https://github.com/C-Lodder/grunt-postcss)
 * **HTML**: [`posthtml-postcss`](https://github.com/posthtml/posthtml-postcss)
 * **Stylus**: [`poststylus`](https://github.com/seaneking/poststylus)
 * **Rollup**: [`rollup-plugin-postcss`](https://github.com/egoist/rollup-plugin-postcss)

package/lib/at-rule.d.ts

@@ -46,7 +46,7 @@ export interface AtRuleProps extends ContainerProps {
  * Represents an at-rule.
  *
  * ```js
- * Root (root, { AtRule }) {
+ * Once (root, { AtRule }) {
  *   let media = new AtRule({ name: 'media', params: 'print' })
  *   media.append(…)
  *   root.append(media)

package/lib/comment.d.ts

@@ -26,7 +26,7 @@ export interface CommentProps extends NodeProps {
  * Represents a comment between declarations or statements (rule and at-rules).
  *
  * ```js
- * Root (root, { Comment }) {
+ * Once (root, { Comment }) {
  *   let note = new Comment({ text: 'Note: …' })
  *   root.append(note)
  * }

package/lib/container.js

@@ -1,6 +1,6 @@
 'use strict'
 
-let { isClean, isComplete } = require('./symbols')
+let { isClean } = require('./symbols')
 let Declaration = require('./declaration')
 let Comment = require('./comment')
 let Node = require('./node')
@@ -17,7 +17,6 @@ function cleanSource (nodes) {
 
 function markDirtyUp (node) {
   node[isClean] = false
-  node[isComplete] = false
   if (node.proxyOf.nodes) {
     for (let i of node.proxyOf.nodes) {
       markDirtyUp(i)
@@ -26,29 +25,22 @@ function markDirtyUp (node) {
 }
 
 // istanbul ignore next
-function rebuild (node, parent) {
-  let fix
+function rebuild (node) {
   if (node.type === 'atrule') {
-    fix = new AtRule()
+    Object.setPrototypeOf(node, AtRule.prototype)
   } else if (node.type === 'rule') {
-    fix = new Rule()
+    Object.setPrototypeOf(node, Rule.prototype)
   } else if (node.type === 'decl') {
-    fix = new Declaration()
+    Object.setPrototypeOf(node, Declaration.prototype)
   } else if (node.type === 'comment') {
-    fix = new Comment()
+    Object.setPrototypeOf(node, Comment.prototype)
   }
 
-  for (let i in node) {
-    if (i === 'nodes') {
-      fix.nodes = node.nodes.map(j => rebuild(j, fix))
-    } else if (i === 'parent' && parent) {
-      fix.parent = parent
-    } else if (Object.prototype.hasOwnProperty.call(i)) {
-      fix[i] = node[i]
-    }
+  if (node.nodes) {
+    node.nodes.forEach(child => {
+      rebuild(child)
+    })
   }
-
-  return fix
 }
 
 class Container extends Node {
@@ -344,7 +336,7 @@ class Container extends Node {
 
     let processed = nodes.map(i => {
       // istanbul ignore next
-      if (typeof i.markDirty !== 'function') i = rebuild(i)
+      if (typeof i.markDirty !== 'function') rebuild(i)
       if (i.parent) i.parent.removeChild(i)
       if (i[isClean]) markDirtyUp(i)
       if (typeof i.raws.before === 'undefined') {

package/lib/declaration.d.ts

@@ -36,7 +36,7 @@ export interface DeclarationProps {
  * Represents a CSS declaration.
  *
  * ```js
- * Root (root, { Declaration }) {
+ * Once (root, { Declaration }) {
  *   let color = new Declaration({ prop: 'color', value: 'black' })
  *   root.append(color)
  * }

package/lib/lazy-result.js

@@ -1,6 +1,6 @@
 'use strict'
 
-let { isComplete, isClean } = require('./symbols')
+let { isClean } = require('./symbols')
 let MapGenerator = require('./map-generator')
 let stringify = require('./stringify')
 let warnOnce = require('./warn-once')
@@ -50,11 +50,9 @@ function getEvents (node) {
 
 function toStack (node) {
   let events
-  if (node[isClean]) {
-    events = node.append ? [CHILDREN] : []
+  if (node.type === 'root') {
+    events = ['Root', CHILDREN, 'RootExit']
   } else {
-    node[isComplete] = true
-    node[isClean] = true
     events = getEvents(node)
   }
 
@@ -70,7 +68,6 @@ function toStack (node) {
 
 function cleanMarks (node) {
   node[isClean] = false
-  node[isComplete] = false
   if (node.nodes) node.nodes.forEach(i => cleanMarks(i))
   return node
 }
@@ -202,13 +199,12 @@ class LazyResult {
     this.prepareVisitors()
     if (this.hasListener) {
       let root = this.result.root
-      while (!root[isComplete]) {
-        root[isComplete] = true
+      while (!root[isClean]) {
         root[isClean] = true
         this.walkSync(root)
       }
-      if (this.listeners.RootExit) {
-        this.visitSync(this.listeners.RootExit, root)
+      if (this.listeners.OnceExit) {
+        this.visitSync(this.listeners.OnceExit, root)
       }
     }
 
@@ -237,29 +233,22 @@ class LazyResult {
   }
 
   walkSync (node) {
-    node.each(child => {
-      if (child[isComplete]) return
-      child[isComplete] = true
-
-      if (child[isClean]) {
-        if (child.nodes) this.walkSync(child)
+    node[isClean] = true
+    let events = getEvents(node)
+    for (let event of events) {
+      if (event === CHILDREN) {
+        if (node.nodes) {
+          node.each(child => {
+            if (!child[isClean]) this.walkSync(child)
+          })
+        }
       } else {
-        child[isClean] = true
-        let events = getEvents(child)
-        for (let event of events) {
-          if (event === CHILDREN) {
-            if (child.nodes) {
-              this.walkSync(child)
-            }
-          } else {
-            let visitors = this.listeners[event]
-            if (visitors) {
-              if (this.visitSync(visitors, child.toProxy())) return
-            }
-          }
+        let visitors = this.listeners[event]
+        if (visitors) {
+          if (this.visitSync(visitors, node.toProxy())) return
         }
       }
-    })
+    }
   }
 
   visitSync (visitors, node) {
@@ -281,8 +270,8 @@ class LazyResult {
   runOnRoot (plugin) {
     this.result.lastPlugin = plugin
     try {
-      if (typeof plugin === 'object' && plugin.Root) {
-        return plugin.Root(this.result.root, this.helpers)
+      if (typeof plugin === 'object' && plugin.Once) {
+        return plugin.Once(this.result.root, this.helpers)
       } else if (typeof plugin === 'function') {
         return plugin(this.result.root, this.result)
       }
@@ -349,8 +338,7 @@ class LazyResult {
     this.prepareVisitors()
     if (this.hasListener) {
       let root = this.result.root
-      while (!root[isComplete]) {
-        root[isComplete] = true
+      while (!root[isClean]) {
         root[isClean] = true
         let stack = [toStack(root)]
         while (stack.length > 0) {
@@ -366,8 +354,8 @@ class LazyResult {
         }
       }
 
-      if (this.listeners.RootExit) {
-        for (let [plugin, visitor] of this.listeners.RootExit) {
+      if (this.listeners.OnceExit) {
+        for (let [plugin, visitor] of this.listeners.OnceExit) {
           this.result.lastPlugin = plugin
           try {
             await visitor(root, this.helpers)
@@ -391,6 +379,7 @@ class LazyResult {
     for (let plugin of this.plugins) {
       if (typeof plugin === 'object') {
         for (let type of [
+          'Root',
           'Declaration',
           'Rule',
           'AtRule',
@@ -399,7 +388,8 @@ class LazyResult {
           'RuleExit',
           'AtRuleExit',
           'CommentExit',
-          'RootExit'
+          'RootExit',
+          'OnceExit'
         ]) {
           if (typeof plugin[type] === 'object') {
             for (let filter in plugin[type]) {
@@ -451,7 +441,8 @@ class LazyResult {
       let child
       while ((child = node.nodes[node.indexes[iterator]])) {
         node.indexes[iterator] += 1
-        if (!child[isComplete]) {
+        if (!child[isClean]) {
+          child[isClean] = true
           stack.push(toStack(child))
           return
         }
@@ -466,7 +457,7 @@ class LazyResult {
       visit.eventIndex += 1
       if (event === CHILDREN) {
         if (node.nodes && node.nodes.length) {
-          node[isComplete] = true
+          node[isClean] = true
           visit.iterator = node.getIterator()
         }
         return

package/lib/list.d.ts

@@ -4,7 +4,7 @@ export type List = {
    * `border-radius`, and other shorthand properties).
    *
    * ```js
-   * Root (root, { list }) {
+   * Once (root, { list }) {
    *   list.space('1px calc(10% + 1px)') //=> ['1px', 'calc(10% + 1px)']
    * }
    * ```
@@ -19,7 +19,7 @@ export type List = {
    * and `background` properties).
    *
    * ```js
-   * Root (root, { list }) {
+   * Once (root, { list }) {
    *   list.comma('black, linear-gradient(white, black)')
    *   //=> ['black', 'linear-gradient(white, black)']
    * }

package/lib/node.js

@@ -1,8 +1,8 @@
 'use strict'
 
-let { isClean, isComplete } = require('./symbols')
 let CssSyntaxError = require('./css-syntax-error')
 let Stringifier = require('./stringifier')
+let { isClean } = require('./symbols')
 let stringify = require('./stringify')
 
 function cloneNode (obj, parent) {
@@ -35,7 +35,6 @@ function cloneNode (obj, parent) {
 class Node {
   constructor (defaults = {}) {
     this.raws = {}
-    this[isComplete] = false
     this[isClean] = false
 
     for (let name in defaults) {
@@ -276,17 +275,13 @@ class Node {
   markDirty () {
     if (this[isClean]) {
       this[isClean] = false
-      this.markIncomplete()
+      let next = this
+      while ((next = next.parent)) {
+        next[isClean] = false
+      }
     }
   }
 
-  markIncomplete () {
-    let next = this
-    do {
-      next[isComplete] = false
-    } while ((next = next.parent))
-  }
-
   get proxyOf () {
     return this
   }

package/lib/postcss.d.ts

@@ -70,18 +70,96 @@ type CommentProcessor = (
 ) => Promise<void> | void
 
 interface Processors {
+  /**
+   * Will be called on `Root` node once.
+   */
+  Once?: RootProcessor
+
+  /**
+   * Will be called on `Root` node once, when all children will be processed.
+   */
+  OnceExit?: RootProcessor
+
+  /**
+   * Will be called on `Root` node.
+   *
+   * Will be called again on children changes.
+   */
   Root?: RootProcessor
+
+  /**
+   * Will be called on `Root` node, when all children will be processed.
+   *
+   * Will be called again on children changes.
+   */
   RootExit?: RootProcessor
+
+  /**
+   * Will be called on all `Declaration` nodes after listeners
+   * for `Declaration` event.
+   *
+   * Will be called again on node or children changes.
+   */
   Declaration?: DeclarationProcessor | { [prop: string]: DeclarationProcessor }
+
+  /**
+   * Will be called on all `Declaration` nodes.
+   *
+   * Will be called again on node or children changes.
+   */
   DeclarationExit?:
     | DeclarationProcessor
     | { [prop: string]: DeclarationProcessor }
+
+  /**
+   * Will be called on all `Rule` nodes.
+   *
+   * Will be called again on node or children changes.
+   */
   Rule?: RuleProcessor
+
+  /**
+   * Will be called on all `Rule` nodes, when all children will be processed.
+   *
+   * Will be called again on node or children changes.
+   */
   RuleExit?: RuleProcessor
+
+  /**
+   * Will be called on all`AtRule` nodes.
+   *
+   * Will be called again on node or children changes.
+   */
   AtRule?: AtRuleProcessor | { [name: string]: AtRuleProcessor }
+
+  /**
+   * Will be called on all `AtRule` nodes, when all children will be processed.
+   *
+   * Will be called again on node or children changes.
+   */
   AtRuleExit?: AtRuleProcessor | { [name: string]: AtRuleProcessor }
+
+  /**
+   * Will be called on all `Comment` nodes.
+   *
+   * Will be called again on node or children changes.
+   */
   Comment?: CommentProcessor
+
+  /**
+   * Will be called on all `Comment` nodes after listeners
+   * for `Comment` event.
+   *
+   * Will be called again on node or children changes.
+   */
   CommentExit?: CommentProcessor
+
+  /**
+   * Will be called when all other listeners processed the document.
+   *
+   * This listener will not be called again.
+   */
+  Exit?: RootProcessor
 }
 
 export interface Plugin extends Processors {
@@ -94,7 +172,7 @@ export interface PluginCreator<PluginOptions> {
   postcss: true
 }
 
-interface Transformer extends TransformCallback {
+export interface Transformer extends TransformCallback {
   postcssPlugin: string
   postcssVersion: string
 }

package/lib/processor.d.ts

@@ -2,6 +2,7 @@ import {
   AcceptedPlugin,
   Plugin,
   ProcessOptions,
+  Transformer,
   TransformCallback
 } from './postcss.js'
 import LazyResult from './lazy-result.js'

package/lib/processor.js

@@ -5,7 +5,7 @@ let Root = require('./root')
 
 class Processor {
   constructor (plugins = []) {
-    this.version = '8.0.7'
+    this.version = '8.1.1'
     this.plugins = this.normalize(plugins)
   }
 

package/lib/rule.d.ts

@@ -46,7 +46,7 @@ export interface RuleProps extends ContainerProps {
  * Represents a CSS rule: a selector followed by a declaration block.
  *
  * ```js
- * Root (root, { Rule }) {
+ * Once (root, { Rule }) {
  *   let a = new Rule({ selector: 'a' })
  *   a.append(…)
  *   root.append(a)

package/lib/symbols.js

@@ -1,6 +1,5 @@
 'use strict'
 
 module.exports = {
-  isComplete: Symbol('isComplete'),
   isClean: Symbol('isClean')
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "postcss",
-  "version": "8.0.7",
+  "version": "8.1.1",
   "description": "Tool for transforming styles with JS plugins",
   "engines": {
     "node": "^10 || ^12 || >=14"
@@ -25,8 +25,8 @@
     "transpiler"
   ],
   "funding": {
-    "type": "tidelift",
-    "url": "https://tidelift.com/funding/github/npm/postcss"
+    "type": "opencollective",
+    "url": "https://opencollective.com/postcss/"
   },
   "author": "Andrey Sitnik <andrey@sitnik.ru>",
   "license": "MIT",