trying to silence third-party errors with [declarations] seems to re-raise ignored errors instead

[timri]

I try to use flow in my project and suppress errors from dependencies.
My project uses react-native-firebase, when I run flow I get 3 errors inside node_modules/react-native-firebase.
I wanted to silence them and have extended the .flowconfig with

[declarations]
.*/node_modules/react-native-firebase/.*

But now I get new errors, which are normally silenced, for example:

Error ┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈ node_modules/react-native-firebase/dist/modules/firestore/DocumentReference.js.flow:54:53

Cannot call CollectionReference with parentPath bound to collectionPath because null [1] is incompatible with Path [2].

     node_modules/react-native-firebase/dist/modules/firestore/DocumentReference.js.flow
     51│   get parent(): CollectionReference {
     52│     const parentPath = this._documentPath.parent();
     53│     // $FlowExpectedError: parentPath can never be null
     54│     return new CollectionReference(this._firestore, parentPath);
     55│   }
     56│
     57│   get path(): string {

     node_modules/react-native-firebase/dist/modules/firestore/CollectionReference.js.flow
 [2] 29│   constructor(firestore: Firestore, collectionPath: Path) {

     node_modules/react-native-firebase/dist/modules/firestore/Path.js.flow
 [1] 36│   parent(): Path | null {

The 3 original errors are also still reported.

I am running flow 0.78.0

[vjpr]

Use [untyped].

[FezVrasta]

@vjpr [untyped] has a different purpose, it will ignore all the types of the matching modules and replace them with any. while [declarations] will use the module types, and ignore any error happening inside them

[TrySound]

@timri Do you have $FlowExpectedError defined in your flowconfig? Out of the box there is only $FlowFixMe

[FezVrasta]

@TrySound I got the same error in #7388 and I was using the default $FlowFixMe

[murrayju]

I'm having the same problem. The desired behavior is to use any type information provided in node_modules when possible, but to never report errors that occur within the node_modules. I only want to see errors that occur in my project's source.

The documentation reads like [declarations] should do exactly what I want, but it seems to do the opposite. Is there any way to write the configuration now to get the behavior that I want? It seems to me like this should be the default behavior. We should be encouraging projects to publish their flow type annotations in their npm packages, and by default leverage this information to improve typing in a project. But this should not result in seeing error output for a project in external dependencies that can't be fixed.

[mroch]

something's broken, trying to figure out what happened. indeed, errors within [declarations] should not be reported, and the annotations should be taken at face value.

[mroch]

I think it's only a problem inside cycles.

[abettadapur]

Any update on this issue? I am seeing this on 0.92

[steobrien]

Also seeing this problem on latest Flow, 0.94

[asazernik]

Still reproducing on 0.104.0.

@mroch - could you explain what you mean by "inside cycles"? I'm wondering if knowing what this means would help me work around the issue.