bug PR-6.2-dev Pending

User tests: Successful: Unsuccessful:

avatar joomdonation
joomdonation
17 May 2026

Pull Request resolves #47755 .

  • I read the Generative AI policy and my contribution is either not created with the help of AI or is compatible with the policy and GNU/GPL 2 or later.

Summary of Changes

Currently, run npm ci on Windows produce some errors. This PR fixes that error. Honestly, I do not understand the script, this fix is provided completely by Copilot. I just makes this PR because I am using Windows and confirm that the fix worked.

Testing Instructions

  • Run npm ci on Windows for Joomla 6.2-dev branch
  • Apply patch

Actual result BEFORE applying this Pull Request

There are some errors

Expected result AFTER applying this Pull Request

No errors anymore.

Link to documentations

Please select:

  • Documentation link for guide.joomla.org:

  • No documentation changes for guide.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar joomdonation joomdonation - open - 17 May 2026
avatar joomdonation joomdonation - change - 17 May 2026
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 17 May 2026
Category JavaScript Repository
avatar joomdonation joomdonation - change - 17 May 2026
The description was changed
avatar joomdonation joomdonation - edited - 17 May 2026
avatar richard67 richard67 - change - 17 May 2026
The description was changed
avatar richard67 richard67 - edited - 17 May 2026
avatar Hackwar Hackwar - test_item - 17 May 2026 - Tested successfully
avatar Hackwar
Hackwar - comment - 17 May 2026

I have tested this item ✅ successfully on d02d361

Works fine for me on Windows 11


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47789.

avatar Hackwar
Hackwar - comment - 17 May 2026

I have tested this item ✅ successfully on d02d361

Works fine for me on Windows 11


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47789.

avatar chmst chmst - test_item - 17 May 2026 - Tested successfully
avatar chmst
chmst - comment - 17 May 2026

I have tested this item ✅ successfully on d02d361


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47789.

avatar chmst
chmst - comment - 17 May 2026

I have tested this item ✅ successfully on d02d361


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47789.

avatar joomdonation joomdonation - change - 17 May 2026
Status Pending Ready to Commit
avatar joomdonation
joomdonation - comment - 17 May 2026

RTC. Thanks


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47789.

avatar joomdonation
joomdonation - comment - 17 May 2026

RTC. Thanks


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47789.

avatar CSGoat0
CSGoat0 - comment - 17 May 2026

Didn't work for me, windows 10.
I will try again and update if it works.

avatar joomdonation
joomdonation - comment - 17 May 2026

Thanks @CSGoat0

avatar joomdonation
joomdonation - comment - 17 May 2026

Thanks @CSGoat0 . I am using Windows 11

avatar MacJoom
MacJoom - comment - 17 May 2026

Didn't work for me, windows 10. I will try again and update if it works.

Can you check your node version? Should not be a Windows 10 problem, but we won't go down that rabbithole

avatar MacJoom
MacJoom - comment - 17 May 2026

Didn't work for me, windows 10. I will try again and update if it works.

@CSGoat0 Can you check your node version? Should not be a Windows 10 problem, but we won't go down that rabbithole

avatar CSGoat0
CSGoat0 - comment - 17 May 2026

Didn't work for me, windows 10. I will try again and update if it works.

Can you check your node version? Should not be a Windows 10 problem, but we won't go down that rabbithole

Tried again still didn't work.

Node version: 22.16.0
NPM version: 11.14.1

avatar CSGoat0
CSGoat0 - comment - 17 May 2026

Didn't work for me, windows 10. I will try again and update if it works.

Can you check your node version? Should not be a Windows 10 problem, but we won't go down that rabbithole

Tried again still didn't work.

Node version: 22.16.0
NPM version: 11.14.1
Windows 10.

avatar joomdonation
joomdonation - comment - 17 May 2026

@CSGoat0 In my case, I have node v24.13.0. npm 11.7.0 . So the reason could be because your node version is old

avatar CSGoat0
CSGoat0 - comment - 17 May 2026

@CSGoat0 In my case, I have node v24.13.0. npm 11.7.0 . So the reason could be because your node version is old

I have tried again with Node version v24.15.0 and unfortunately it didn't work.
I have also tried on a fresh Joomla 6.2 branch, and it didn't work.

Maybe if someone else have Windows 10, they can confirm.
But making it works for 11 is good though.

avatar brianteeman
brianteeman - comment - 17 May 2026

When testing please make sure that you are testing without anything in the media folder

avatar brianteeman brianteeman - test_item - 17 May 2026 - Tested unsuccessfully
avatar brianteeman
brianteeman - comment - 17 May 2026

I have tested this item 🔴 unsuccessfully on d02d361


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47789.

avatar brianteeman
brianteeman - comment - 17 May 2026

I have tested this item 🔴 unsuccessfully on d02d361


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47789.

avatar brianteeman
brianteeman - comment - 17 May 2026
Failed Task [system::js]
Trace: Error: Processing failed for "D:\repos\j61\media_source\system\js\fields\joomla-field-color-slider.es6.js".
    at file:///D:/repos/j61/build/build-modules-js/javascript/js-handle.mjs:157:11
    at async Promise.all (index 23) {
  [cause]: Error: ENOENT: no such file or directory, mkdir 'D:\repos\j61\media\system\js\fields'
      at async mkdir (node:internal/fs/promises:859:10)
      at async writeOutputFile (file:///D:/repos/j61/node_modules/rollup/dist/es/shared/node-entry.js:23616:5)
      at async Queue.work (file:///D:/repos/j61/node_modules/rollup/dist/es/shared/node-entry.js:22681:32) {
    errno: -4058,
    code: 'ENOENT',
    syscall: 'mkdir',
    path: 'D:\\repos\\j61\\media\\system\\js\\fields'
  }
}
    at file:///D:/repos/j61/build/build-modules-js/builder/builder-factory.mjs:74:21

PS D:\repos\j61> node -v
v24.15.0
PS D:\repos\j61> npm -v
11.14.1

windows 11

avatar heelc29 heelc29 - test_item - 17 May 2026 - Tested unsuccessfully
avatar heelc29
heelc29 - comment - 17 May 2026

I have tested this item 🔴 unsuccessfully on d02d361

npm ci works now👍(tested with win11; npm 11.14.1; node 22.22.2)

but uri of codemirror files in joomla.asset.json is wrong -> codemirror not working
image


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47789.
avatar heelc29
heelc29 - comment - 17 May 2026

I have tested this item 🔴 unsuccessfully on d02d361

npm ci works now👍(tested with win11; npm 11.14.1; node 22.22.2)

but uri of codemirror files in joomla.asset.json is wrong -> codemirror not working
image


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47789.
avatar richard67 richard67 - change - 17 May 2026
Status Ready to Commit Pending
Labels Added: bug PR-6.2-dev
avatar richard67
richard67 - comment - 17 May 2026

Back to pending due to new, unsuccessful human tests.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47789.

avatar richard67
richard67 - comment - 17 May 2026

Back to pending due to new, unsuccessful human tests.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47789.

avatar LadySolveig
LadySolveig - comment - 17 May 2026

@heelc29 would this resolve the issue? I'm opening a PR against @joomdonation branch, but maybe you could test it out beforehand @heelc29 @CSGoat0 ? Unfortunately, I can't.

media_source/plg_editors_codemirror/builder.mjs

image
uri: modulePath.replace('.js', '.min.js').replace(/\\/g, '/'),
avatar LadySolveig
LadySolveig - comment - 17 May 2026

@heelc29 would this resolve the issue? I'm opening a PR against @joomdonation branch, but maybe you could test it out beforehand @heelc29 @CSGoat0 ? Unfortunately, I can't.

media_source/plg_editors_codemirror/builder.mjs

image
uri: modulePath.replace('.js', '.min.js').replace(/\\/g, '/'),
avatar LadySolveig
LadySolveig - comment - 17 May 2026

@heelc29 would this resolve the issue? I'm opened a PR (joomdonation#6) against @joomdonation branch, but maybe you could test it out beforehand @heelc29 @CSGoat0 ? Unfortunately, I can't.

media_source/plg_editors_codemirror/builder.mjs

image
uri: modulePath.replace('.js', '.min.js').replace(/\\/g, '/'),
avatar LadySolveig
LadySolveig - comment - 17 May 2026

@heelc29 would this resolve the issue? I opened a PR (joomdonation#6) against @joomdonation branch, but maybe you could test it out beforehand @heelc29 @CSGoat0 ? Unfortunately, I can't.

media_source/plg_editors_codemirror/builder.mjs

image
uri: modulePath.replace('.js', '.min.js').replace(/\\/g, '/'),
avatar CSGoat0
CSGoat0 - comment - 17 May 2026

This is an update:

  • My original error was on a new Joomla 6 clone; it was composer error that is not related to this PR. This is another issue that have just shown up. I tried a lot to run npm ci and npm install on a new cloned joomla 6.2
    git clone -b 6.2-dev https://github.com/YourForkLinkOrJoomlaMainLink and every time i get you should run composer install first even after running composer install multiple times and clone multiple times in new different folders.

  • When I tried this PR on my working version of 6.2, the npm ci worked correctly with no errors.

  • I can confirm that this PR break the codemirror files and it doesn't work correctly on the site just as @heelc29 have said.

  • I couldn't replicate ENOENT issue reported by @brianteeman, I ran npm ci on 2 scenarios:
    • When there is no media folder at all
    • When there is an empty media folder
    both scenarios, the npm ci command ran correctly with no issues except the codemirror issue.

Summary:

  • This PR solves the issue it should solve
  • This PR breaks the codemirror
  • There is another npm issue on a new 6.2 clone if someone can confirm
  • I couldn't replicate ENOENT issue
avatar CSGoat0
CSGoat0 - comment - 17 May 2026

This is an update:

  • My original error was on a new Joomla 6 clone; it was composer error that is not related to this PR. This is another issue that have just shown up. I tried a lot to run npm ci and npm install on a new cloned joomla 6.2
    git clone -b 6.2-dev https://github.com/YourForkLinkOrJoomlaMainLink and every time i get you should run composer install first even after running composer install multiple times and clone multiple times in new different folders.

  • When I tried this PR on my working version of 6.2, the npm ci worked correctly with no errors.

  • I can confirm that this PR break the codemirror files and it doesn't work correctly on the site just as @heelc29 have said.

  • I couldn't replicate ENOENT issue reported by @brianteeman, I ran npm ci on 2 scenarios:
    • When there is no media folder at all
    • When there is an empty media folder
    both scenarios, the npm ci command ran correctly with no issues except the codemirror issue.

Summary:

  • This PR solves the issue it should solve
  • This PR breaks the codemirror
  • There is another npm and composer issue on a new 6.2 clone if someone can confirm
  • I couldn't replicate ENOENT issue
avatar CSGoat0
CSGoat0 - comment - 17 May 2026

This is an update:

  • My original error was on a new Joomla 6 clone; it was composer error that is not related to this PR. This is another issue that have just shown up. I tried a lot to run npm ci and npm install on a new cloned joomla 6.2
    git clone -b 6.2-dev https://github.com/YourForkLinkOrJoomlaMainLink and every time i get you should run composer install first even after running composer install multiple times and clone multiple times in new different folders.

  • When I tried this PR on my working version of 6.2, the npm ci worked correctly with no errors.

  • I can confirm that this PR break the codemirror files and it doesn't work correctly on the site just as @heelc29 have said.

  • I couldn't replicate ENOENT issue reported by @brianteeman, I ran npm ci on 2 scenarios:
    • When there is no media folder at all
    • When there is an empty media folder
    both scenarios, the npm ci command ran correctly with no issues except the codemirror issue.

Summary:

  • This PR solves the issue it should solve
  • This PR breaks the codemirror
  • There is another npm and composer issue on a new 6.2 clone if someone can confirm
  • I couldn't replicate ENOENT issue

Notes:

This update is based on a meeting with @LadySolveig, Thanks to her.

avatar CSGoat0
CSGoat0 - comment - 17 May 2026

@LadySolveig I have tried this line uri: modulePath.replace('.js', '.min.js').replace(/\\/g, '/'), It replaces every \\ with / but the codemirror still broken somehow.

avatar CSGoat0
CSGoat0 - comment - 17 May 2026

I found it.
Along with this line suggested line uri: modulePath.replace('.js', '.min.js').replace(/\\/g, '/'), you need to find a way to add the version correctly in plg_editors_codemirror. When i added this missing line manually there, the codemirror worked correctly.
image

avatar brianteeman
brianteeman - comment - 17 May 2026

i have re-run the test about 20 times and I cannot repeat the error I got before. weird but I guess its one of those things.

I can confirm the codemirror slash issue reported by @heelc29

I tried the fix from @LadySolveig and that resolved the slash issue

I tested codemirror inside Joomla and it appeared to run correctly

avatar MacJoom
MacJoom - comment - 17 May 2026

@joomdonation can you include the fix from @CSGoat0 / @LadySolveig ? - Thank you all - i hope this is the final fix...

avatar CSGoat0
CSGoat0 - comment - 17 May 2026

i have re-run the test about 20 times and I cannot repeat the error I got before. weird but I guess its one of those things.

I can confirm the codemirror slash issue reported by @heelc29

I tried the fix from @LadySolveig and that resolved the slash issue

I tested codemirror inside Joomla and it appeared to run correctly

Codemirror will be missing both styling and numbered lines. It won't look like this photo:

image

If you try to add the version in plg_editors_codemirror/builder.mjs you will get the correct codemirror.

avatar brianteeman
brianteeman - comment - 17 May 2026

@CSGoat0 codemirror loads just fine for me after the slash fix
image
image

avatar CSGoat0
CSGoat0 - comment - 17 May 2026

@CSGoat0 codemirror loads just fine for me after the slash fix

Could you try it on normal articles?
When i go to System -> Global Configuration -> Site -> Default Editor and set it to code mirror, it is no loaded correctly on articles for me.

avatar CSGoat0
CSGoat0 - comment - 17 May 2026

@CSGoat0 codemirror loads just fine for me after the slash fix

Could you try it on normal articles?
When i go to System -> Global Configuration -> Site -> Default Editor and set it to code mirror, it is not loaded correctly on articles for me.

avatar brianteeman
brianteeman - comment - 17 May 2026

@CSGoat0 no problem there either
image

avatar CSGoat0
CSGoat0 - comment - 17 May 2026

@CSGoat0 no problem there either
I tried it again; this is what I get. it's different from the other one here comment

image
avatar CSGoat0
CSGoat0 - comment - 17 May 2026

@CSGoat0 no problem there either

I tried it again; this is what I get. it's different from the other one here comment

image
avatar CSGoat0
CSGoat0 - comment - 17 May 2026

@CSGoat0 no problem there either

I tried it again; this is what I get. it's different from the other one here comment

image

The code mirror file at media -> plg_editors_codemirror -> joomla.asset.json are fixed, but it doesn't work.

image
avatar CSGoat0
CSGoat0 - comment - 17 May 2026

@CSGoat0 no problem there either

I tried it again; this is what I get. it's different from the other one here comment

image

The code mirror file at media -> plg_editors_codemirror -> joomla.asset.json is fixed, but it doesn't work.

image
avatar joomdonation
joomdonation - comment - 18 May 2026

I talked with @LadySolveig, she will open a new PR to replace this one in the morning. She understands the script, so it will be easier for her to make any additional changes if needed. Thanks you all for testing and help.

avatar joomdonation joomdonation - change - 18 May 2026
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2026-05-18 00:30:49
Closed_By joomdonation
avatar joomdonation joomdonation - close - 18 May 2026

Add a Comment

Login with GitHub to post a comment