JavaScriptExpert
diff --git a/‎src/api/action.ts‎
Lines changed: 2 additions & 1 deletion b/‎src/api/action.ts‎
Lines changed: 2 additions & 1 deletion
diff --git a/‎test/base/action.js‎
Lines changed: 13 additions & 0 deletions b/‎test/base/action.js‎
Lines changed: 13 additions & 0 deletions
@@ -55,10 +55,11 @@ export var action: IActionFactory = function action(arg1, arg2?, arg3?, arg4?):
     // @action fn() {}
     if (arg4 === true) {
         // apply to instance immediately
-        arg1[arg2] = createAction(name, arg3.value)
+        arg1[arg2] = createAction(arg1.name || arg2, arg3.value)
     } else {
         return namedActionDecorator(arg2).apply(null, arguments)
     }
+    toString
 } as any
 
 action.bound = boundActionDecorator as any
 
@@ -455,3 +455,16 @@ test("bound actions bind", () => {
     d()
     d2()
 })
+
+test("Fix #1367", () => {
+    const x = mobx.extendObservable(
+        {},
+        {
+            method() {}
+        },
+        {
+            method: mobx.action
+        }
+    )
+    expect(mobx.isAction(x.method)).toBe(true)
+})