babel-plugin-flow-runtime references undefined type parameter names with assertions disabled

[jedwards1211]

This is a:

• Bug Report
• Feature Request
• Question
• Other

Which concerns:

• flow-runtime
• babel-plugin-flow-runtime
• flow-runtime-validators
• flow-runtime-mobx
• flow-config-parser
• The documentation website

---

Code

// @flow

function foo<X>() {
  let bar: X
}

.babelrc

{
  "presets": ["es2015"],
  "plugins": [
    "transform-decorators-legacy",
    ["flow-runtime", {"assert": false}]
  ]
}

What is the current behaviour?

---

X is undefined in the output code below at let _barType = X:

import t from "flow-runtime";
function foo() {
  let _barType = X,
      bar;
}
t.annotate(foo, t.function(_fn => {
  const X = _fn.typeParameter("X");

  return [];
}));

What is the expected behaviour?

---

Either babel-plugin-flow-runtime doesn't create _barType, or it inserts const X = t.typeParameter("X") above let _barType it despite the "assert": false in the plugin options.

Which package versions are you using?

├── babel-core@6.26.0 
├── babel-plugin-flow-runtime@0.15.0 
├── babel-preset-es2015@6.24.0 

---

IssueHunt Summary

Backers (Total: $40.00)

• issuehunt ($40.00)

Submitted pull Requests

• #285 Fix assert false type parameter variables
• #296 fix(plugin): avoid undefined type parameter references when assert=false

---

Become a backer now!

Or submit a pull request to get the deposits!

Tips

• Checkout the Issuehunt explorer to discover more funded issues.
• Need some help from other developers? Add your repositories on IssueHunt to raise funds.

[issuehunt-oss]

@issuehunt has funded $40.00 to this issue.

---

• Submit pull request via IssueHunt to receive this reward.
• Want to contribute? Chip in to this issue via IssueHunt.
• Checkout the IssueHunt Issue Explorer to see more funded issues.
• Need help from developers? Add your repository on IssueHunt to raise funds.

[selimeneserd]

Submitted a fix in PR #292: #292