[wataru86]

What

Removed exports-loader from devDependencies and dropped the loader rule from webpack.config.coffee.

How

• The webpack build no longer passes .js files through exports-loader.
  • The loader was used without options, so the only output it added to dist/bundle.js was the /*** EXPORTS FROM exports-loader ***/ comment (lines 50 and 688).
• dist/bundle.js was not regenerated.
  • On master, gulp build does not run on current Node.js with gulp 3 (see Make gulp build work with gulp 5 #14, which is stacked on Bump gulp from 3.9.1 to 5.0.1 #9 and not yet on master). The build can be checked once Bump gulp from 3.9.1 to 5.0.1 #9 is merged.
• No lockfile is committed in this repository, so package.json and webpack.config.coffee are the only files changed.

Why

#16 (exports-loader 0.6.4 → 5.0.0) fails at npm i with ERESOLVE, because exports-loader 5 has a peer dependency on webpack ^5 while this repository uses webpack 1.
It cannot be fixed within #16 while the build runs on webpack 1: exports-loader 5 calls this.getOptions(schema), which only exists in the webpack 5 loader API, and requires the exports option.

Merging #17 (webpack 1.15.0 → 5.112.0) first does not help.
It lets #16 pass npm i, but the build runs the webpack that gulp-webpack depends on (>=1.9.0 <2.0.0-0), where this.getOptions does not exist.
CI on #17 is green only because CI does not run the build.

The loader has been a no-op since it was added in 009f80d: exports-loader 0.6 only emits exports for names passed as options, and none were passed.
The dist/bundle.js built in that commit already contained only the comment.
Removing it is a smaller change than upgrading it, whether or not the build moves to webpack 5.

Once this is merged, Dependabot will close #16 because the dependency no longer exists.

Verification

• Run npm i and npm test.
  • The tests only load dist/parser.js, so they do not cover the webpack build.
• Load webpack.config.coffee with coffee-script/register and check that only the output setting remains.
• Not verified: running gulp build and diffing dist/bundle.js, for the reason above. Apart from the unrelated function wrapping noted in Make gulp build work with gulp 5 #14, the expected diff is the removal of the two comment lines and their surrounding blank lines.

Results: #18 (comment)

Refs

• Bump exports-loader from 0.6.4 to 5.0.0 #16
• Bump webpack from 1.15.0 to 5.112.0 #17
• Remove unused pegjs devDependency #13
  • Previous removal of an unused devDependency.

🤖 Generated with Claude Code

[wataru86]

Verification results (on c4e587d, Node.js 24)

• npm i and npm test: 12 passing.
• node -e "require('coffee-script/register'); console.log(JSON.stringify(require('./webpack.config')))":

  {"output":{"filename":"./dist/bundle.js"}}
  

• npm ls exports-loader: (empty)

🤖 This comment was generated with Claude Code