Removes all compiles/non-source artifacts

[datengraben]

Ziel des PRs: Alle JS/CSS Artefakte die via Grunt kompiliert/gebaut werden entfernt.

• Explizite Dateien der Abhängigkeiten zu Moment.js und Select2 aus Repo entfernt und über NPM gemanaged

• Entfernt ungenutztes daterangepicker.css (und js) aus dem Repo und aus der Plugin.php

• Entfernt alle JS/CSS Build-Artefakte aus dem Repo. Diese existieren nur zur Build-Zeit und werden nicht mehr eingecheckt. ( Das hat @hansmorb ja durch das bauen des Plugin.zip gelöst)

• (OPTIONAL) Neuer Grunt clean task um fertige Build-Artefakte zu entfernen. (Diese sind aber auch schon in der .gitignore aufgelistet und werden damit gar nicht erst gestaged).

TODO:

• Test: Vergleichen der plugin ZIP Datei und der darin enthaltenen assets.

[hansmorb]

Megacool! Frage mich nur, warum die entsprechende Action nicht daraus sofort eine ZIP baut. Basiert der branch vielleicht auf einem alten Master?

[github-actions]

Download built plugin zip
Test Plugin in Wordpress Playground

[codecov]

Codecov Report

❌ Patch coverage is 0% with 24 lines in your changes missing coverage. Please review.
✅ Project coverage is 53.99%. Comparing base (63695a5) to head (c693cf9).

Files with missing lines	Patch %	Lines
src/Plugin.php	0.00%	24 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #1828      +/-   ##
============================================
- Coverage     54.08%   53.99%   -0.09%     
- Complexity     2670     2671       +1     
============================================
  Files            98       98              
  Lines         11708    11727      +19     
============================================
  Hits           6332     6332              
- Misses         5376     5395      +19     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[datengraben]

Megacool! Frage mich nur, warum die entsprechende Action nicht daraus sofort eine ZIP baut. Basiert der branch vielleicht auf einem alten Master?

@hansmorb hat funktioniert. Dummerweise haben wir im alten Release Prozess (2.10.3) noch nicht minified, dort sind die JS-Sources noch unverändert ausgelifert. Das wäre gut vorher nochmal umzustellen auf diesem Branch, dann können wir einfacher vergleichen.

[hansmorb]

Können wir in dem Zug vielleicht auch gleich die sass Dateien nicht mehr mit ausliefern und nur die minified js Versionen?

[datengraben]

@hansmorb klar das macht Sinn. Am Fri, May 16, 2025 at 03:55:01AM -0700 schrieb Hans Morbach:

…

hansmorb left a comment (wielebenwir/commonsbooking#1828) Können wir in dem Zug vielleicht auch gleich die sass Dateien nicht mehr mit ausliefern und nur die minified js Versionen? -- Reply to this email directly or view it on GitHub: #1828 (comment) You are receiving this because you authored the thread. Message ID: ***@***.***>

[hansmorb]

Brauchen wir das wirklich in der repo? Mal für einen sanity check nach dem PR ist das okay aber ansonsten haben wir ja die E2E Tests.

[hansmorb]

@hansmorb klar das macht Sinn. Am Fri, May 16, 2025 at 03:55:01AM -0700 schrieb Hans Morbach:
…
hansmorb left a comment (wielebenwir/commonsbooking#1828) Können wir in dem Zug vielleicht auch gleich die sass Dateien nicht mehr mit ausliefern und nur die minified js Versionen? -- Reply to this email directly or view it on GitHub: #1828 (comment) You are receiving this because you authored the thread. Message ID: @.***>

ACHTUNG: Wenn nur die minified Dateien ausgeliefert werden, schlagen u.U die E2E Tests fehl. Grund ist, dass diese WP_DEBUG aktiviert haben, was die nicht-minified Dateien erwartet. Aus dem Grund würde ich sagen lassen wir das so und liefern beide aus.

[datengraben]

Grundsätzlich hast du recht. Die E2E Test sind funktionale (wenn auch teilweise eher nur Smoke-) Tests. Grobe oder größere Änderungen an den kompilierten CSS Artefakten würden die sicher abfangen. Entweder für jemand paranoides oder jemand der nicht nachdenken will wäre das Skript zumindest eine Option :D Ich fand es intuitiv, weil 100% Sicherheit. Am Tue, Feb 10, 2026 at 03:16:41AM -0800 schrieb Hans Morbach:

…

@hansmorb commented on this pull request. > @@ -0,0 +1,70 @@ +#!/bin/bash Brauchen wir das wirklich in der repo? Mal für einen sanity check nach dem PR ist das okay aber ansonsten haben wir ja die E2E Tests. -- Reply to this email directly or view it on GitHub: #1828 (review) You are receiving this because you authored the thread. Message ID: ***@***.***>

[datengraben]

@hansmorb rebased!

But I think the npm dependency does not need to be in the non-dev dependencies.

[datengraben]

@hansmorb The e2e action is failing because of the slight difference between the "Ran grunt" step (runs => grunt default) we have in the compile-plugin action the "npm run dist" (runs => grunt dist).

If I look at the tasks, there is just slight difference between them.

	// Register tasks
	grunt.registerTask('default', [
		'dart-sass:adminDev',
		'dart-sass:publicDev',
		'dart-sass:globalDev',
		'dart-sass:themes',
		'uglify:dev',
		'uglify:dist',
		'babel',
		'copy',
		'node_versions',
	]);
	grunt.registerTask('dev', [
		'dart-sass:adminDev',
		'dart-sass:publicDev',
		'dart-sass:globalDev',
		'dart-sass:themes',
		'uglify:dev',
		'babel',
		'copy',
		'node_versions',
		'watch',
	]);
	grunt.registerTask('dist', [
		'clean',
		'dart-sass:admin',
		'dart-sass:public',
		'dart-sass:global',
		'dart-sass:themes',
		'babel',
		'uglify:dist',
		'copy',
		'node_versions',
	]);

I will change the dist to default in this branch for testing purpose.

[datengraben]

@hansmorb I think we have now reduced js depdencies untracked by our package manager to 2 (jquery-overscroll, leaflet-messagebox). I've added comments about it in the code. But maybe we should document that elsewhere too.

[datengraben]

@hansmorb ready. I've also removed the ran-grunt commit action, since don't need this anymore. Ofc double check :)

[hansmorb]

Looks good so far, I will get around to properly testing it next week. There is a function called
registerScriptsAndStyles() in the Plugin.php which also has $base = COMMONSBOOKING_PLUGIN_ASSETS_URL . 'packaged/';

Basically all the dependencies from the map are loaded here, but maybe we can move the dependency loading we have from dependencies of the packaged folder here? Then we have them all in one place.

[datengraben]

Sounds good to remove redundant code, but I have to say, I do not understand which place do you mean exactly. There is only one occurence of `packaged/` so there is no redundancy for this. Which redundancy do you mean? Am Fri, Feb 13, 2026 at 05:17:55AM -0800 schrieb Hans Morbach:

…

hansmorb left a comment (wielebenwir/commonsbooking#1828) Looks good so far, I will get around to properly testing it next week. There is a function called registerScriptsAndStyles() in the Plugin.php which also has $base = COMMONSBOOKING_PLUGIN_ASSETS_URL . 'packaged/'; Basically all the dependencies from the map are loaded here, but maybe we can move the dependency loading we have from dependencies of the packaged folder here? Then we have them all in one place. -- Reply to this email directly or view it on GitHub: #1828 (comment) You are receiving this because you were assigned. Message ID: ***@***.***>

[datengraben]

@hansmorb

As soon as this pr is merged, we can merge #1687 and then apply the formatter on the codebase.
Then all source files have propper formatting/linting rules.

[hansmorb]

• Not so important but we could just move that to registerScriptsAndStyles()

Sounds good to remove redundant code, but I have to say, I do not understand which place do you mean exactly. There is only one occurence of packaged/ so there is no redundancy for this. Which redundancy do you mean? Am Fri, Feb 13, 2026 at 05:17:55AM -0800 schrieb Hans Morbach:
…
hansmorb left a comment (wielebenwir/commonsbooking#1828) Looks good so far, I will get around to properly testing it next week. There is a function called registerScriptsAndStyles() in the Plugin.php which also has $base = COMMONSBOOKING_PLUGIN_ASSETS_URL . 'packaged/'; Basically all the dependencies from the map are loaded here, but maybe we can move the dependency loading we have from dependencies of the packaged folder here? Then we have them all in one place. -- Reply to this email directly or view it on GitHub: #1828 (comment) You are receiving this because you were assigned. Message ID: @.***>

[hansmorb]

Ich hab das so umgesetzt wie ich es meinte. Aktuell entsteht bei mir, wenn ich jetzt grunt laufen lasse noch eine global.css und global.css.map , die gab es vorher noch nicht. Kannst du das nachvollziehen woher das kommt?