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

Author: coderaiser

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

@@ -51,7 +51,13 @@ export const traverse = ({push, options}) => ({
                 continue;
 
             const keyPath = propPath.get('key');
+            const valuePath = propPath.get('value');
+            
             const keyIsString = keyPath.isStringLiteral();
+            const valueIsString = valuePath.isStringLiteral();
+            
+            if (valueIsString)
+                continue;
 
             if (computed && !keyIsString)
                 continue;
@@ -60,7 +66,6 @@ export const traverse = ({push, options}) => ({
                 continue;
 
             const {rename, ignore = []} = options;
-            const valuePath = propPath.get('value');
 
             const from = getName(valuePath);
             const to = getName(keyPath);

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

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

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

@@ -0,0 +1,3 @@
+const hello = {
+    world: 'world',
+};