Updates per @JamesHenry's code review

Author: weirdpattern

lib/ast-converter.js

@@ -1187,10 +1187,27 @@ module.exports = function(ast, extra) {
                         id: null,
                         params: node.parameters.map(function(param) {
                             var convertedParam = convertChild(param);
-                            convertedParam.decorators = (param.decorators) ? param.decorators.map(function(d) {
+                            var decorators = (param.decorators) ? param.decorators.map(function(d) {
                                 return convertChild(d.expression);
                             }) : [];
-                            return convertedParam;
+
+                            if (param.modifiers) {
+                                return {
+                                    type: "TSParameterProperty",
+                                    range: [param.getStart(), param.end],
+                                    loc: getLoc(param, ast),
+                                    accessibility: getTSNodeAccessibility(param),
+                                    isReadonly: param.modifiers.some(function(modifier) {
+                                        return modifier.kind === SyntaxKind.ReadonlyKeyword;
+                                    }),
+                                    parameter: convertedParam,
+                                    decorators: decorators
+                                };
+                            }
+
+                            return assign(convertedParam, {
+                                decorators: decorators
+                            });
                         }),
                         generator: false,
                         expression: false,

tests/fixtures/typescript/basics/class-with-private-parameter-properties.result.js

@@ -0,0 +1,6 @@
+class Foo {
+  constructor(private firstName: string,
+              private readonly lastName: string,
+              private age: number = 30,
+              private readonly student: boolean = false) {}
+}

tests/fixtures/typescript/basics/class-with-private-parameter-properties.src.ts

@@ -0,0 +1,6 @@
+class Foo {
+  constructor(protected firstName: string,
+              protected readonly lastName: string,
+              protected age: number = 30,
+              protected readonly student: boolean = false) {}
+}

tests/fixtures/typescript/basics/class-with-protected-parameter-properties.result.js

@@ -0,0 +1,6 @@
+class Foo {
+  constructor(public firstName: string,
+              public readonly lastName: string,
+              public age: number = 30,
+              public readonly student: boolean = false) {}
+}