Branch: refs/heads/claude/cool-tesla-je8lo7 Home: https://github.com/xwiki/xwiki-dev-llm Commit: e3243c3ed428f0347b99be8e79b56b6c38197f87 https://github.com/xwiki/xwiki-dev-llm/commit/e3243c3ed428f0347b99be8e79b56b... Author: Vincent Massol <[email protected]> Date: 2026-08-11 (Tue, 11 Aug 2026) Changed paths: M xwiki/okf/sonarqube/index.md M xwiki/okf/sonarqube/simplification-rules.md M xwiki/skills/xwiki-fix-sonarqube-issue/SKILL.md Log Message: ----------- Deny S2629 and route it to conventions/logging.md instead The S2629 section added in the previous commit was wrong: it advised deleting a redundant toString() from a parameterized log call, which contradicts conventions/logging.md ("Never silently remove an explicit toString()"). In XWiki a log argument is stored as an object and XStream-serialized into the job log, so the eager String is usually a deliberate snapshot. * Move S2629 out of the simplification family and into the index denylist, with a pointer to conventions/logging.md. * Tell the skill that a convention-owned rule's knowledge lives in okf/conventions/, not okf/sonarqube/, so the pointer gets followed. Co-Authored-By: Vincent Massol <[email protected]> To unsubscribe from these emails, change your notification settings at https://github.com/xwiki/xwiki-dev-llm/settings/notifications