I agree this seems broken.This now the case in the map returned by declarations. I don't think this agrees with the specification and I believe it should be fixed. Invocation and lookup must be consistent.

The simple name of a constructor should be the identifier without the class or the constant symbol for the empty string if it is a default constructor.

This will break existing uses, but I think we can still do that as we revise the mirror library to its final form.

cc @rmacnak-google.
Set owner to @gbracha.
Added Accepted label.

This comment was originally written by @Andersmholmgren

Similarly you end up in the unfortunate state where you can look up a constructor but then not instantiate off it

import 'dart:mirrors';

class Foo {
  Foo.build();
}

main() {
  final cm = reflectClass(Foo);

  Symbol ctrName;

  cm.declarations.forEach((k, v) {
    if (v is MethodMirror && v.isConstructor) {
      ctrName = k;
    }
  });

  final i = cm.newInstance(ctrName, []);
}

Closing in favor of #16075 since ClassMirrors.constructors no longer exists and the linked issue covers ClassMirrors.declarations.