files ending in `_index.js` have `index` replaced with an empty string

[mylons]

files ending in `_index.js` have `index` replaced with an empty string

[dhh]

It's happening to support that library/index.js can be referenced as library.js. But it's not intended that library_index.js is transformed. Please do explore a fix!

[mylons]

Ah, it does appear this is happening on purpose. I'm curious why? Should I rename my files?

importmap-rails/lib/importmap/map.rb

Line 144 in e0c1676

[ mapping.under, filename.to_s.remove(filename.extname).remove(/\/?index$/).presence ].compact.join("/")

Looking at the rest of the repo it's not obvious as to why a file ending in index should have that removed

[Caleb-T-Owens]

Breaking down the regex, it's scanning for:

• \/? ← Matches one or zero / characters
• index ← Matches the word index
• $ ← Matches the end of a line

This is to try and match the following examples:

• index.js
• folder/index.js

But because it looks for 0 or 1 /, it also picks up super_index.js

I'm going to put together a PR updating this regex to /(:\/|^)index$/ which broken down is:

• (: ← Start non-capturing group
  • \/ ← Matches the / character
  • | ← OR operator
  • ^ ← Matches the start of the string
• ) ← Close group
• index ← Matches the word index
• $ ← Matches the end of a line

Which will still match

• index.js
• folder/index.js
  but specifically won't match super_index.js because we're now require either the string to start or the character / right before the word index