feat: release 2.8.41

Author: jmrenouard

.agent/README.md

@@ -55,4 +55,4 @@ This directory contains the project's technical constitution, specialized skills
 
 
 ---
-*Generated automatically by `/doc-sync` on Mon Apr 13 22:12:00 CEST 2026*
+*Generated automatically by `/doc-sync` on Thu Apr 16 17:03:00 CEST 2026*

.agent/rules/03_execution_rules.md

@@ -71,6 +71,9 @@ To ensure quality and clarity in every development cycle, all non-trivial featur
 
 - **STRICT PROHIBITION:** No `git commit`, `git push`, or `git tag` without using `/git-flow` or an explicit user order.
 - **BRANCHING MANDATORY:** ALL developments, features, bug fixes, and interventions MUST be done in a dedicated Git branch separated from `master`. Committing directly to the `master` branch is strictly prohibited.
+- **RELEASE DELIVERY:** The release delivery process MUST always occur in a branch named `vX.XX.XX` (e.g., `v2.8.41`). Modifying the `main` branch directly is utterly prohibited.
+- **VERSION NUMBERS:** Changing version numbers automatically without an explicit user request is prohibited.
+- **RELEASE PROCESS:** The release process consists uniquely of pushing the version branch with the release notes as the content (commit message), *after* all verifications are successful.
 - **Conventional Commits:** Use `feat:`, `fix:`, `chore:`, `docs:`, `perf:`, `refactor:`, `style:`, `test:`, `ci:`. Breaking changes must be marked with `!` after type/scope or `BREAKING CHANGE:` in footer.
 - **Commit Validation:** Commits are automatically linted via `commitlint`. Non-compliant messages will be rejected by the pre-commit hook.
 - **History Documentation:** Use `npm run commit` to generate structured history.

.agent/rules/04_best_practices.md

@@ -82,7 +82,8 @@ Beyond hard constraints, following established patterns ensures code durability,
 
 - Release workflows (via `/git-flow`) MUST force push tags to the origin at each release to ensure synchronization with GitHub.
 - The `/git-flow` workflow MUST always be preceded by a successful `/release-preflight` execution.
-- Only the Release Manager is authorized to decide when to increment version numbers, or incrementing occurs automatically after a `git-flow` commit.
+- Version incrementing MUST only happen upon an explicit request. Automatic version bumping is strictly prohibited.
+- The release delivery process MUST be executed strictly on a `vX.XX.XX` branch. No modifications to the main branch are allowed during release.
 
 ### 13. Release Artifact Integrity
 

.agent/workflows/git-flow.md

@@ -8,7 +8,27 @@ category: tool
 
 This workflow MUST be orchestrated by the **Release Manager**.
 
-1. **Run Release Preflight Workflow**
+## 🧠 Constraints
+
+- **Branch Mandatory**: The release process MUST only be executed on a dedicated branch named `vX.XX.XX` (e.g., `v2.8.41`).
+- **No Main Modification**: Pushing to the `main` or `master` branch is strictly prohibited here.
+- **No Automatic Bumping**: Version numbers MUST NOT be automatically incremented after release unless explicitly requested by the user.
+
+## 🛠️ Implementation
+
+// turbo
+1. **Branch Verification**
+   - Verify the current Git branch matches the `vX.XX.XX` pattern.
+
+   ```bash
+   CURRENT_BRANCH=$(git rev-parse --abbrev-ref HEAD)
+   if [[ ! "$CURRENT_BRANCH" =~ ^v[0-9]+\.[0-9]+\.[0-9]+$ ]]; then
+     echo "ERROR: Release process must be executed on a vX.XX.XX branch."
+     exit 1
+   fi
+   ```
+
+2. **Run Release Preflight Workflow**
    - Execute the `/release-preflight` workflow to ensure full consistency and test passing.
    - **CRITICAL**: Do NOT proceed if `/release-preflight` fails.
 
@@ -18,8 +38,8 @@ This workflow MUST be orchestrated by the **Release Manager**.
    ```
 
    // turbo
-2. **Commit Current Changes**
-   - Commit all pending changes including `Changelog` updates for the current version.
+3. **Commit Release Notes**
+   - Commit all pending changes. The commit message MUST be strictly formatted as the release notes extracted from the `Changelog`.
 
    ```bash
    # Extract content between the first version header and the next one
@@ -30,47 +50,19 @@ This workflow MUST be orchestrated by the **Release Manager**.
    ```
 
    // turbo
-3. **Create Tag for Current Version with Changelog content**
-   - Extract the latest release notes and create an annotated tag.
+4. **Create Tag for Current Version**
+   - Create an annotated tag incorporating the release notes.
 
    ```bash
-   # Extract content between the first version header and the next one
-   TAG_MSG=$(awk "/^$CURRENT_VER/,/^([0-9]+\.[0-9]+\.[0-9]+)/ {if (\$0 !~ /^([0-9]+\.[0-9]+\.[0-9]+)/) print}" Changelog | sed '/^$/d')
-   git tag -a v$CURRENT_VER -m "Release $CURRENT_VER" -m "$TAG_MSG"
+   git tag -a v$CURRENT_VER -m "Release $CURRENT_VER" -m "$RELEASE_NOTES"
    ```
 
    // turbo
-4. **Push Branch and Tag**
-   - Push to the remote repository.
+5. **Push Branch and Tag**
+   - Push the current branch and the tag to the remote repository.
 
    ```bash
-   git push origin main
+   git push origin $CURRENT_BRANCH
    git push origin v$CURRENT_VER
    ```
 
-   // turbo
-5. **Post-Push: Increment Version for Next Cycle**
-   - Calculate the next patch version and update files.
-
-   ```bash
-   NEW_VER=$(echo $CURRENT_VER | awk -F. '{print $1"."$2"."($3+1)}')
-   echo $NEW_VER > CURRENT_VERSION.txt
-   # Update all version occurrences in mysqltuner.pl
-   perl -pi -e "s/\Q$CURRENT_VER\E/$NEW_VER/g" mysqltuner.pl
-   
-   DATE=$(date +%Y-%m-%d)
-   echo -e "$NEW_VER $DATE\n\n- \n" > tmp_changelog && cat Changelog >> tmp_changelog && mv tmp_changelog Changelog
-   ```
-
-   // turbo
-6. **Commit Version Bump**
-   - Commit the incremented version for the next development cycle.
-
-   ```bash
-   git add CURRENT_VERSION.txt mysqltuner.pl Changelog
-   npx commitlint --from=HEAD~1 # Or simply use npm run commit if not automated
-   git commit -m "chore: bump version to $NEW_VER"
-   git push origin main
-   ```
-
-   // turbo

Changelog

@@ -4,6 +4,7 @@
 
 - ci: enhance Quality Gate to strictly enforce zero-warning policy on GitHub Actions tests.
 - ci: implement dynamic CI test environment detection by wrapping configuration extraction.
+- ci: refactor GitHub Actions release and prerelease workflows to support dynamic versions and checksum generation.
 - fix: Restore compatibility with older Perl versions (by @jasongill).
 - feat: implement idiomatic Perl Boolean practices across the project (#34).
 - refactor: update CLI metadata to use `undef` as default for string/path options.

POTENTIAL_ISSUES

@@ -158,3 +158,5 @@ The following external commands are currently used via `execute_system_command`
   - Refactored `%CLI_METADATA` defaults from `'0'` to `undef` for string/path options.
   - Wrapped template reading logic in `get_template_model` to prevent `uninitialized value` warnings during `require`.
   - Verified 100% test stability (262 tests passed).
+- [x] **Zero-Warning Enforcement**: Fixed `removed_innodb_vars.t` mocking warnings and `mysql_80_modeling_checks` uninitialized values. Implemented strict Exit 1 policy on GitHub Actions CI.
+- [x] **Dynamic CI Discovery**: Created perl wrapper to dynamically parse `mysql_support.md` and `mariadb_support.md` allowing testing configurations to be uncoupled from the `Makefile`.

build/refactor_mocks.pl

@@ -0,0 +1,53 @@
+#!/usr/bin/env perl
+use strict;
+use warnings;
+use File::Slurp;
+
+my @files = glob("tests/*.t");
+foreach my $file (@files) {
+    next unless -f $file;
+    
+    my $content = read_file($file);
+    my $original = $content;
+    
+    # Only act if we see testing of mysqltuner (has myvar)
+    next unless $content =~ /\%main::myvar/ || $content =~ /\%myvar/;
+    
+    # 1. Require TestHelper safely at the top, after loading mysqltuner
+    if ($content =~ /(require [\'\"].*mysqltuner\.pl[\'\"];?)/) {
+        unless ($content =~ /MySQLTuner::TestHelper/) {
+            $content =~ s/(require [\'\"].*mysqltuner\.pl[\'\"];?)/$1\nrequire '.\/tests\/MySQLTuner\/TestHelper.pm';/s;
+        }
+    } elsif ($content =~ /(require \$script;?)/) {
+        unless ($content =~ /MySQLTuner::TestHelper/) {
+            $content =~ s/(require \$script;?)/$1\nrequire '.\/tests\/MySQLTuner\/TestHelper.pm';/s;
+        }
+    } else {
+        # Can't find require mysqltuner
+        unless ($content =~ /MySQLTuner::TestHelper/) {
+            $content =~ s/(use Test::More;.*?\n)/$1\nrequire '.\/tests\/MySQLTuner\/TestHelper.pm';\n/s;
+        }
+    }
+    
+    # 2. Modify assignments to preserve defaults
+    # Find `%main::myvar = (` and replace with `%main::myvar = ( %main::myvar,`
+    $content =~ s/(\%main::myvar\s*=\s*\()/$1 \%main::myvar, /g;
+    $content =~ s/(\%main::mystat\s*=\s*\()/$1 \%main::mystat, /g;
+    $content =~ s/(\%main::mycalc\s*=\s*\()/$1 \%main::mycalc, /g;
+
+    # 3. Add reset_state calls.
+    # Replace global %main::myvar with reset_state + local
+    $content =~ s/(\%main::myvar\s*=\s*\()/MySQLTuner::TestHelper::reset_state();\n    $1/g;
+    
+    # Handle files with local sub reset_state that mocks things.
+    if ($content =~ /sub reset_state \{/) {
+        # Strip their bodies or remove entirely
+        $content =~ s/sub reset_state \{.*?\n\}//ms;
+        $content =~ s/reset_state\(\);/MySQLTuner::TestHelper::reset_state();/g;
+    }
+    
+    if ($content ne $original) {
+        write_file($file, $content);
+        print "Updated $file\n";
+    }
+}

releases/v2.8.41.md

@@ -7,15 +7,16 @@
 ```text
 2.8.41 2026-04-13
 
-- fix: Restore compatibility with older Perl versions (by @jasongill).
+- chore: automated project maintenance and cleanup (extracted `RULES.md`, `MEMORY_DB.md`, `TESTS.md`).
 - feat: implement idiomatic Perl Boolean practices across the project (#34).
-- refactor: update CLI metadata to use `undef` as default for string/path options.
-- refactor: replace non-idiomatic `eq '0'`, `ne 0`, etc., with standard truthiness checks.
+- feat: add recommendation for `table_open_cache_instances` based on CPU cores (#480).
+- feat: improve syslog and systemd journal detection for error logs (#440).
+- feat: initialize `$mysqllogin` to avoid uninitialized value warnings (#490).
+- fix: Restore compatibility with older Perl versions (by @jasongill).
 - fix: wrap template loading in `get_template_model()` to avoid `uninitialized value` warnings during `require`.
 - fix: allow `--updateversion` to work on hosts without `mysql`/`mariadb` installed (#36).
 - fix: skip local SSL certificate warnings if they are in an inaccessible `datadir` (#33).
 - fix: correct false positives in `check_removed_innodb_variables` by distinguishing real server variables from internal ones (#32).
-- refactor: replace "master"/"slave" terminology with "source"/"replica" for cultural sensitivity (#888).
 - fix: improve join_buffer_size recommendation formatting in Variables to Adjust (#881).
 - fix: suppress MySQL client warning regarding 'DISABLED' boolean value for SSL (#887).
 - fix: correctly handle `--defaults-file` and `--defaults-extra-file` without dropping options (#605).
@@ -25,11 +26,16 @@
 - fix: prevent `AUTO_INCREMENT` capacity false positives for empty tables (#37).
 - fix: refactor InnoDB Redo Log Capacity logic to be workload-based and avoid false positives (#714, #737, #777).
 - fix: add guards against division by zero in calculations for improved stability (#435).
-- feat: add recommendation for `table_open_cache_instances` based on CPU cores (#480).
-- feat: improve syslog and systemd journal detection for error logs (#440).
-- feat: initialize `$mysqllogin` to avoid uninitialized value warnings (#490).
-- chore: automated project maintenance and cleanup (extracted `RULES.md`, `MEMORY_DB.md`, `TESTS.md`).
+- fix: add truthiness guards to `mysql_innodb` and `mysql_stats` subroutines.
+- fix: improve `which` logic for better container/minimal environment support.
+- fix: enhance login failure reporting with detailed output.
+- ci: enhance Quality Gate to strictly enforce zero-warning policy on GitHub Actions tests.
+- ci: implement dynamic CI test environment detection by wrapping configuration extraction.
+- ci: refactor GitHub Actions release and prerelease workflows to support dynamic versions and checksum generation.
 - ci: migrate maintenance script to GitHub Actions.
+- refactor: update CLI metadata to use `undef` as default for string/path options.
+- refactor: replace non-idiomatic `eq '0'`, `ne 0`, etc., with standard truthiness checks.
+- refactor: replace "master"/"slave" terminology with "source"/"replica" for cultural sensitivity (#888).
 - chore(deps): update docker/setup-buildx-action action to v4.
 - chore(deps): update docker/build-push-action action to v7.
 - chore(deps): update docker/login-action action to v4.
@@ -40,45 +46,56 @@
 
 | Metric | Current | Progress | Status |
 | :--- | :--- | :--- | :--- |
-| Total Indicators | 300+ | +0 | 🚀 |
+| Total Indicators | 12 | 0 | 🛡️ |
 | Efficiency Checks | 0 | 0 | 🛡️ |
-| Risk Detections | 0 | 0 | 🚀 |
-| Information Points | 0 | 0 | 🚀 |
+| Risk Detections | 2 | 0 | 🛡️ |
+| Information Points | 10 | 0 | 🛡️ |
+
+## 🛠️ Internal Commit History
+
+- ci: refactor Github release definitions with pre-releases and deliverables support (68f8d12)
+- ci: enforce zero-warning policy and dynamic test env (2eebcc4)
+- feat: release 2.8.41 final stabilization (e479ba7)
+- feat: release 2.8.41 final polish (e8d2751)
+- feat: release 2.8.41 final polish (bac36cb)
+- feat: release 2.8.41 (56b9dd7)
+- Merge pull request #41 from major/master (089a598)
+- Merge pull request #885 from major/renovate/docker-setup-buildx-action-4.x (d430513)
+- Merge pull request #886 from major/renovate/docker-build-push-action-7.x (9b7dd14)
+- Merge pull request #884 from major/renovate/docker-login-action-4.x (5d56ecb)
+- Merge pull request #894 from major/renovate/softprops-action-gh-release-3.x (9805873)
+- Merge pull request #882 from jasongill/master (d1f3808)
+- Merge pull request #895 from jmrenouard/master (2feb466)
+- Merge pull request #40 from major/master (10fcd37)
+- Merge branch 'master' into master (c13bb0b)
+- feat: release 2.8.40 (21d5873)
+- perltidy mysqltuner.pl (5653056)
+- chore(deps): update all non-major dependencies to v20.5.0 (#893) (290b646)
+- chore(deps): update dependency lodash to v4.18.1 [security] (#892) (c01e968)
+- chore: add automerge to Renovate config (633edcc)
+- Add CodeQL analysis workflow configuration (43a57c9)
+- Merge pull request #35 from C0RD/master (9474c70)
+- chore(deps): update all non-major dependencies to v20.5.0 (#893) (4520d11)
+- chore(deps): update softprops/action-gh-release action to v3 (75e9cee)
+- chore(deps): update dependency lodash to v4.18.1 [security] (#892) (e77d43c)
+- chore: add automerge to Renovate config (0470fb7)
+- Merge pull request #38 from jmrenouard/dependabot/npm_and_yarn/npm_and_yarn-b22b4dc46b (69caecf)
+- Merge pull request #39 from major/master (54dbffd)
+- chore(deps-dev): bump the npm_and_yarn group across 1 directory with 2 updates (955284a)
+- chore(deps): update docker/build-push-action action to v7 (a5a6234)
+- chore(deps): update docker/setup-buildx-action action to v4 (830bbc2)
+- chore(deps): update docker/login-action action to v4 (2ca8b8d)
+- Change default update check to disabled (938db61)
+- Remove r modifier from regex for compatability with older Perl versions (d6f7f97)
+- chore: bump version to 2.8.41 (3e54167)
+- Merge pull request #879 from jmrenouard/master (9b8617f)
 
 ## ⚙️ Technical Evolutions
 
-### 🛠️ Internal Commit History
-- Merge pull request #41 from major/master
-- feat: implement idiomatic Perl Boolean practices across the project (#34)
-- refactor: replace "master"/"slave" terminology with "source"/"replica" for cultural sensitivity (#888)
-- fix: allow --updateversion to work on hosts without mysql/mariadb installed (#36)
-- fix: skip local SSL certificate warnings if they are in an inaccessible datadir (#33)
-- fix: correct false positives in check_removed_innodb_variables (#32)
-- fix: improve join_buffer_size recommendation formatting in Variables to Adjust (#881)
-- fix: suppress MySQL client warning regarding 'DISABLED' boolean value for SSL (#887)
-- fix: correctly handle --defaults-file and --defaults-extra-file without dropping options (#605)
-- fix: restore Debian maintenance account automatic login by using idiomatic boolean checks (#896)
-- fix: include tmp_table_size in per-thread memory calculation (#864)
-- fix: add retry mechanism for initial SELECT VERSION() query (#782)
-- fix: prevent AUTO_INCREMENT capacity false positives for empty tables (#37)
-- fix: refactor InnoDB Redo Log Capacity logic to be workload-based and avoid false positives (#714, #737, #777)
-- fix: add guards against division by zero in calculations for improved stability (#435)
-- feat: add recommendation for table_open_cache_instances based on CPU cores (#480)
-- feat: improve syslog and systemd journal detection for error logs (#440)
-- feat: initialize $mysqllogin to avoid uninitialized value warnings (#490)
-- fix: add truthiness guards to mysql_innodb and mysql_stats subroutines
-- fix: improve 'which' logic for better container/minimal environment support
-- fix: enhance login failure reporting with detailed output
-- fix: wrap template loading in get_template_model() to avoid warnings during require
-- fix: Restore compatibility with older Perl versions
-- chore: automated project maintenance and cleanup (extracted RULES.md, MEMORY_DB.md, TESTS.md)
-- ci: migrate maintenance script to GitHub Actions
-- chore(deps): update docker/setup-buildx-action action to v4
-- chore(deps): update docker/build-push-action action to v7
-- chore(deps): update docker/login-action action to v4
-- chore(deps): update softprops/action-gh-release action to v3
+*Internal logic hardening (no interface or diagnostic changes).*
 
 ## ✅ Laboratory Verification Results
 
 - [x] Automated TDD suite passed.
 - [x] Multi-DB version laboratory execution validated.
+- [x] Performance indicator delta analysis completed.