- notifications - lists.xwiki.org

[xwiki/xwiki-dev-llm] c4388f: [Misc] Record that a functional test is a scenario...
by XWiki Notifications 27 Aug '26

27 Aug '26

Branch: refs/heads/master Home: https://github.com/xwiki/xwiki-dev-llm Commit: c4388f99cf42edcd47d6e9235de7525c0a421a19 https://github.com/xwiki/xwiki-dev-llm/commit/c4388f99cf42edcd47d6e9235de75… Author: Vincent Massol <vincent(a)massol.net> Date: 2026-08-27 (Thu, 27 Aug 2026) Changed paths: M .claude-plugin/marketplace.json M kimi.plugin.json M opencode.jsonc M xwiki/.claude-plugin/plugin.json M xwiki/okf/index.md M xwiki/okf/testing/strategy.md M xwiki/skills/xwiki-review/SKILL.md M xwiki/skills/xwiki-test-guidelines/SKILL.md Log Message: ----------- [Misc] Record that a functional test is a scenario with a single fixture * Add the scenario rule to okf/testing/strategy.md: a *IT pays a wiki start, a browser start and a page load per navigation, so the number of fixtures drives its runtime, not the number of assertions. Write as few @Test methods as the scenario allows, extend an existing method or *IT rather than rebuilding the same fixture, and note that @Order fixes execution order without sharing a fixture so it is not a substitute. Sourced from the dev wiki's functional-testing best practices, which the plugin only linked and never stated. * Add the matching procedural bullet to the xwiki-test-guidelines skill, so the rule applies when writing a test and not only when reviewing one. * Add the check to the xwiki-review Tests angle: a new @Test method or *IT class that rebuilds a fixture an existing one already builds is a finding. * Update the strategy summary and the OKF map entry, and bump the plugin version. Co-Authored-By: Claude Opus 5 (1M context) <noreply(a)anthropic.com> Commit: 63d3343a254b846586abcdc75652f160c676698a https://github.com/xwiki/xwiki-dev-llm/commit/63d3343a254b846586abcdc75652f… Author: Vincent Massol <vincent(a)massol.net> Date: 2026-08-27 (Thu, 27 Aug 2026) Changed paths: M xwiki/okf/index.md M xwiki/okf/testing/strategy.md M xwiki/skills/xwiki-test-guidelines/SKILL.md Log Message: ----------- [Misc] Pivot the scenario rule from method count to duplicate fixtures * Restated the rule as "no two @Test methods build the same fixture" instead of "as few methods as possible, one if you can". The defect the rule targets is a second method paying for a fixture an existing one already builds, not method count in itself, and the previous wording read as a push towards a single method however long it grew. * Added the counterweight: a method nobody can follow end to end, or one whose failure no longer says which behaviour broke, has been merged too far. A distinct fixture justifies a distinct method, a merely distinct assertion does not. * Mirrored the change in the xwiki-test-guidelines bullet, the strategy summary and the OKF map. Co-Authored-By: Claude Opus 5 (1M context) <noreply(a)anthropic.com> Commit: f2679a39ede9629ffd5aea87e0587668cc8fd8fc https://github.com/xwiki/xwiki-dev-llm/commit/f2679a39ede9629ffd5aea87e0587… Author: Vincent Massol <vincent(a)massol.net> Date: 2026-08-27 (Thu, 27 Aug 2026) Changed paths: M .claude-plugin/marketplace.json M kimi.plugin.json M opencode.jsonc M xwiki/.claude-plugin/plugin.json M xwiki/okf/index.md M xwiki/okf/testing/strategy.md M xwiki/skills/xwiki-review/SKILL.md M xwiki/skills/xwiki-test-guidelines/SKILL.md Log Message: ----------- [Misc] Record that a functional test is a scenario with a single fixture Compare: https://github.com/xwiki/xwiki-dev-llm/compare/707cdb63c1c7...f2679a39ede9 To unsubscribe from these emails, change your notification settings at https://github.com/xwiki/xwiki-dev-llm/settings/notifications

1 0

[jira@xwiki.org][Issue] Updates for XWIKI-23590: Inconsistent rounding of the "Save" button in page Preview mode
by XWiki Notifications 27 Aug '26

27 Aug '26

1 0

[jira@xwiki.org][Issue] Updates for XWIKI-23742: NotificationWatchUIX stylesheet selector too wide
by XWiki Notifications 27 Aug '26

27 Aug '26

1 0

[xwiki/xwiki-dev-llm] 63d334: [Misc] Pivot the scenario rule from method count t...
by XWiki Notifications 27 Aug '26

27 Aug '26

Branch: refs/heads/it-scenario-fixture-rule Home: https://github.com/xwiki/xwiki-dev-llm Commit: 63d3343a254b846586abcdc75652f160c676698a https://github.com/xwiki/xwiki-dev-llm/commit/63d3343a254b846586abcdc75652f… Author: Vincent Massol <vincent(a)massol.net> Date: 2026-08-27 (Thu, 27 Aug 2026) Changed paths: M xwiki/okf/index.md M xwiki/okf/testing/strategy.md M xwiki/skills/xwiki-test-guidelines/SKILL.md Log Message: ----------- [Misc] Pivot the scenario rule from method count to duplicate fixtures * Restated the rule as "no two @Test methods build the same fixture" instead of "as few methods as possible, one if you can". The defect the rule targets is a second method paying for a fixture an existing one already builds, not method count in itself, and the previous wording read as a push towards a single method however long it grew. * Added the counterweight: a method nobody can follow end to end, or one whose failure no longer says which behaviour broke, has been merged too far. A distinct fixture justifies a distinct method, a merely distinct assertion does not. * Mirrored the change in the xwiki-test-guidelines bullet, the strategy summary and the OKF map. Co-Authored-By: Claude Opus 5 (1M context) <noreply(a)anthropic.com> To unsubscribe from these emails, change your notification settings at https://github.com/xwiki/xwiki-dev-llm/settings/notifications

1 0

[xwiki/xwiki-dev-llm] c4388f: [Misc] Record that a functional test is a scenario...
by XWiki Notifications 27 Aug '26

27 Aug '26

Branch: refs/heads/it-scenario-fixture-rule Home: https://github.com/xwiki/xwiki-dev-llm Commit: c4388f99cf42edcd47d6e9235de7525c0a421a19 https://github.com/xwiki/xwiki-dev-llm/commit/c4388f99cf42edcd47d6e9235de75… Author: Vincent Massol <vincent(a)massol.net> Date: 2026-08-27 (Thu, 27 Aug 2026) Changed paths: M .claude-plugin/marketplace.json M kimi.plugin.json M opencode.jsonc M xwiki/.claude-plugin/plugin.json M xwiki/okf/index.md M xwiki/okf/testing/strategy.md M xwiki/skills/xwiki-review/SKILL.md M xwiki/skills/xwiki-test-guidelines/SKILL.md Log Message: ----------- [Misc] Record that a functional test is a scenario with a single fixture * Add the scenario rule to okf/testing/strategy.md: a *IT pays a wiki start, a browser start and a page load per navigation, so the number of fixtures drives its runtime, not the number of assertions. Write as few @Test methods as the scenario allows, extend an existing method or *IT rather than rebuilding the same fixture, and note that @Order fixes execution order without sharing a fixture so it is not a substitute. Sourced from the dev wiki's functional-testing best practices, which the plugin only linked and never stated. * Add the matching procedural bullet to the xwiki-test-guidelines skill, so the rule applies when writing a test and not only when reviewing one. * Add the check to the xwiki-review Tests angle: a new @Test method or *IT class that rebuilds a fixture an existing one already builds is a finding. * Update the strategy summary and the OKF map entry, and bump the plugin version. Co-Authored-By: Claude Opus 5 (1M context) <noreply(a)anthropic.com> To unsubscribe from these emails, change your notification settings at https://github.com/xwiki/xwiki-dev-llm/settings/notifications

1 0

[jira@xwiki.org][Issue] Updates for XWIKI-23742: NotificationWatchUIX stylesheet selector too wide
by XWiki Notifications 27 Aug '26

27 Aug '26

1 0

[xwiki/xwiki-platform] 7e94a5: XWIKI-23742: NotificationWatchUIX stylesheet selec...
by XWiki Notifications 27 Aug '26

27 Aug '26

Branch: refs/heads/XWIKI-23742 Home: https://github.com/xwiki/xwiki-platform Commit: 7e94a52185df159dfa2a11e899219787f59e019d https://github.com/xwiki/xwiki-platform/commit/7e94a52185df159dfa2a11e89921… Author: Vincent Massol <vincent(a)massol.net> Date: 2026-08-27 (Thu, 27 Aug 2026) Changed paths: M xwiki-platform-core/xwiki-platform-notifications/xwiki-platform-notifications-ui/src/main/resources/XWiki/Notifications/Code/NotificationsWatchUIX.xml Log Message: ----------- XWIKI-23742: NotificationWatchUIX stylesheet selector too wide * Scope the watch modal stylesheet selectors to #watchModal so that they no longer style unrelated Bootstrap panels on any page where the on-demand stylesheet is loaded. Co-Authored-By: Claude Opus 5 (1M context) <noreply(a)anthropic.com> To unsubscribe from these emails, change your notification settings at https://github.com/xwiki/xwiki-platform/settings/notifications

1 0

[xwiki/xwiki-rendering]
by XWiki Notifications 27 Aug '26

27 Aug '26

Branch: refs/heads/claude/peaceful-archimedes-b8qklo Home: https://github.com/xwiki/xwiki-rendering To unsubscribe from these emails, change your notification settings at https://github.com/xwiki/xwiki-rendering/settings/notifications

1 0

[xwiki/xwiki-rendering] 396ff5: [Misc] Replace an instanceof if/else chain with a ...
by XWiki Notifications 27 Aug '26

27 Aug '26

Branch: refs/heads/master Home: https://github.com/xwiki/xwiki-rendering Commit: 396ff5541fa3d77d65c81ccdcf0fe107c77e4120 https://github.com/xwiki/xwiki-rendering/commit/396ff5541fa3d77d65c81ccdcf0… Author: claude[bot] <209825114+claude[bot](a)users.noreply.github.com> Date: 2026-08-27 (Thu, 27 Aug 2026) Changed paths: M xwiki-rendering-transformations/xwiki-rendering-transformation-macro/src/main/java/org/xwiki/rendering/internal/macro/source/MacroContentSourceReferenceConverter.java Log Message: ----------- [Misc] Replace an instanceof if/else chain with a switch expression (SonarQube) (#415) * Fixes 1 SonarQube java:S6880 issue ("Replace the chain of if/else with a switch expression"), using Java 21 pattern matching for switch. * The branch order is unchanged and "case null" keeps the original null behaviour (a null value went to the else branch). Co-authored-by: Vincent Massol <vincent(a)massol.net> To unsubscribe from these emails, change your notification settings at https://github.com/xwiki/xwiki-rendering/settings/notifications

1 0

[xwiki/xwiki-dev-llm] 707cdb: [Misc] Drop the duplicated rule-key lists and stal...
by XWiki Notifications 27 Aug '26

27 Aug '26

Branch: refs/heads/master Home: https://github.com/xwiki/xwiki-dev-llm Commit: 707cdb63c1c72152a80abdc5954bd9f606d7e18d https://github.com/xwiki/xwiki-dev-llm/commit/707cdb63c1c72152a80abdc5954bd… Author: Vincent Massol <vincent(a)massol.net> Date: 2026-08-27 (Thu, 27 Aug 2026) Changed paths: M .claude-plugin/marketplace.json M kimi.plugin.json M opencode.jsonc M xwiki/.claude-plugin/plugin.json M xwiki/okf/index.md M xwiki/okf/sonarqube/constant-and-resource-rules.md M xwiki/okf/sonarqube/dead-code-rules.md M xwiki/okf/sonarqube/index.md M xwiki/okf/sonarqube/modernization-rules.md M xwiki/okf/sonarqube/simplification-rules.md M xwiki/okf/sonarqube/syntax-rules.md M xwiki/okf/sonarqube/test-code-rules.md Log Message: ----------- [Misc] Drop the duplicated rule-key lists and stale anecdotes from the sonarqube OKF * Collapse the rule-key lists to one authoritative copy: sonarqube/index.md's rule map. The three other copies had all drifted — okf/index.md's per-family bullets were missing 24 of the 69 keys (test-code-rules advertised 4 of 9, simplification 12 of 21), and each family file's frontmatter summary and "S1116 · S1124 · …" subtitle line were missing 14 between them. A stale list makes a rule read as undocumented and the fix get applied past the trap the file exists to describe. The summaries now describe the traps instead of enumerating keys, which is what a family file is chosen on. * Remove the "used to be on the denylist" notes for S3252 and S6355 and the same framing on S3252's own section. The current placement already says it, and okf/conventions/code-comments.md rules out justifying by history. * Generalise six one-off batch statistics (site counts, "18 of the 22", "one site in six") that sonarqube/index.md's own policy excludes as volatile. The durable signal — near-zero drops, near-100% drops, permanent drop — is kept. * Delete the S8924 "not yet established" note; its actionable half is already step 1 of that same list. * Verified before the edit that all 69 documented keys and the 33 denylist keys are still present and status=READY in SonarCloud's Java ruleset for the xwiki org, so nothing here documents a retired rule. Co-Authored-By: Claude Opus 5 (1M context) <noreply(a)anthropic.com> To unsubscribe from these emails, change your notification settings at https://github.com/xwiki/xwiki-dev-llm/settings/notifications

1 0