No files are written: mkdir@1.0.0 broke mkdir-promise as such writeFile processor broke

[dherges]

mkdirp@1.0.0 changed implementation from callback to promisified api

• changelog: https://github.com/isaacs/node-mkdirp/blob/master/CHANGELOG.md#10
• change: https://github.com/isaacs/node-mkdirp/commit/ed0ae94cf6c02380f4b3b45df39abcf74c62f956#diff-168726dbe96b3ce427e7fedce31bb0bc

---

mkdir-promise has "mkdirp": "*" dependency: https://github.com/ahmadnassri/mkdirp-promise/blob/master/package.json#L37

---

As such, the writeFile stopped working because the Promise never resolves.

https://github.com/angular/dgeni-packages/blob/master/base/services/writeFile.js#L1

---

I fixed it locally in a project workspace by overriding the factory with a custom implementation:

import * as mkdirp from 'mkdirp';
import * as fs from 'fs';
import * as path from 'canonical-path';

/**
 * mkdirp module changed implementation from callback to promise in 1.0.0
 *
 * As such, mkdir-promise stopped working.
 *
 * @link https://github.com/isaacs/node-mkdirp/blob/master/CHANGELOG.md
 */
export default function writeFile() {

  return function(file, content) {

    return new Promise((resolve, reject) => {
        mkdirp(path.dirname(file), undefined, (err, made) => {
          if (err === null) {
            resolve(made);
          } else {
            reject(err);
          }
        });
      }).then(() => new Promise((resolve, reject) => {
        fs.writeFile(file, content, function (err) {
          if (err) {
            reject(err);
          }
          resolve();
        });
      }));
  }
}

[dherges]

Thoughts...just thoughs...

1. Please take this issue to twitter 🤣
2. Ideally, mkdir-promise should mkdirp instanceof Promise ? mkdir : promisify(mkdirp)... 🤔
3. It could be worth adding a few promising lines of code for a Promise that's actually a Promise 🐆

[petebacondarwin]

How about we just update dgeni-packages to using mkdirp@1.0 and stop using mkdirp-promise? Since, AFAICT, the new version already returns a promise, right?

[dherges]

Sounds good to me!

[petebacondarwin]

I just got hit by this when updating AIO 🙀 So I better get on and fix it!

[petebacondarwin]

Thanks for raising this @dherges - it must have been quite a pain to debug what was going on!

[dherges]

you're welcome!

I had this thought to modify the dgeni code to work with different versions of the dependencies, and that would have required several "if/else" checks...

Just updating the dependencies (mkdirp >= 1.0.0) sounds very good to me!