Parcel: HMR not working with PostHTML Modules

[w-biggs]

🐛 bug report

When using PostHTML Modules for templating, modifying a partial html file doesn't trigger HMR as it should.

🎛 Configuration (.babelrc, package.json, cli command)

My .posthtmlrc file:

{
  "plugins": {
    "posthtml-modules": {
      "root": "./web"
    },
    "posthtml-expressions": {
      "root": "./web"
    }
  }
}

and my package.json's devDependencies:

"devDependencies": {
  "@babel/core": "^7.5.4",
  "mocha": "^6.1.4",
  "parcel-bundler": "^1.12.3",
  "posthtml-expressions": "^1.1.1",
  "posthtml-modules": "^0.4.2",
  "sass": "^1.22.3"
}

🤔 Expected Behavior

Editing a partial html file included as a PostHTML module should trigger HMR.

😯 Current Behavior

Nothing happens when the partial html file is edited.

💁 Possible Solution

Maybe Parcel isn't following PostHTML modules' paths or something? I'm not sure.

🔦 Context

Just trying to utilize HMR when developing - it makes my life easier.

💻 Code Sample

https://github.com/w-biggs/1212 is the repo; https://github.com/w-biggs/1212/blob/master/web/index.html is the entry HTML file.

🌍 Your Environment

| Software | Version(s) |
| ---------------- | ---------- |
| Parcel | 1.12.3 |
| posthtml-modules | 0.4.2 |
| Node | 11.14.0 |
| npm/Yarn | 6.9.2 |
| Operating System | Ubuntu 18.04.1 LTS (WSL) |

[pugson]

Experiencing this issue as well 😥️

I made a hacky workaround that updates index.html whenever a change is detected in one of the modules.

[github-actions[bot]]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 14 days if no further activity occurs.

[sscovil]

FWIW this is still an issue, I'm experiencing it as well.

[StraightOuttaCrompton]

Why has this been marked as closed? Seems like the issue hasn't been fixed.

[BigPackie]

Or you can just update the index.html to trigger a change in the module if you use the module from index.html.

_Very important_: using dash/hyphen - inside a module as a name for a local variable binding like {{my-variable}} breaks the code.

[vasanthdeveloper]

Yup, the issue still exists in 2.0.0-nightly.469 and I can confirm that.