feature: @putout/plugin-apply-shorthand-properties: exclude default

Author: coderaiser

packages/plugin-apply-shorthand-properties/lib/apply-shorthand-properties.js

@@ -9,6 +9,7 @@ const {
 const {
     isImportSpecifier,
     identifier,
+    isStringLiteral,
 } = types;
 
 export const report = () => `Use shorthand properties`;
@@ -38,7 +39,12 @@ export const traverse = ({push, options}) => ({
             });
     },
     '__object'(path) {
-        for (const propPath of path.get('properties')) {
+        const properties = path.get('properties');
+        
+        if (hasDefault(properties))
+            return;
+        
+        for (const propPath of properties) {
             const {computed, shorthand} = propPath.node;
 
             if (shorthand)
@@ -116,3 +122,12 @@ function getName(path) {
 
     return '';
 }
+
+function hasDefault(properties) {
+    for (const property of properties) {
+        if (isStringLiteral(property.node.value, {value: 'default'}))
+            return true;
+    }
+    
+    return false;
+}

packages/plugin-apply-shorthand-properties/test/apply-shorthand-properties.js

@@ -114,6 +114,11 @@ test('plugin-apply-shorthand-properties: no report: spread', (t) => {
     t.end();
 });
 
+test('plugin-apply-shorthand-properties: no report: default', (t) => {
+    t.noReport('default');
+    t.end();
+});
+
 testWithRemove('plugin-apply-shorthand-properties: transform with options: assign', (t) => {
     t.transformWithOptions('assign', {
         rename: true,

packages/plugin-apply-shorthand-properties/test/fixture/default.js

@@ -0,0 +1,5 @@
+const x = {
+    a: 'a',
+    b: 'b',
+    default: 'default'
+};