From f24af7ab08bb1d83234781c961ce3916fb378ed0 Mon Sep 17 00:00:00 2001 From: marcos Date: Thu, 20 Aug 2026 17:53:24 +0000 Subject: [PATCH] chore(pr-review): add .pr-review.json with canalhandia-specific steer + repo context MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Wires the new ADDITIONAL_CONTEXT_URL feature on the central webhook service (see gitea_admin/pragent e8ebc54) — fetches a static maintainer-curated brief from the Nexus raw-hosted repo at review time. The doc distils the house rules so the agent doesn't have to re-derive them from the diff each PR: * chat messages are immutable after send (counts baked into buttons freeze) * translatable components + singular-form rule for pt-BR sentences * Geyser/Bedrock can't click — every click has a typed fallback * vanilla stats only — /players/stats/.json, NOT /stats * reactions keep counting late (reacao-validade-minutos) * Floodgate is optional runtime, never compile-time Plus the canonical focus/instructions/languages config: java, balanced style, claude-sonnet-5 cost target, JDK 25 build note. Co-Authored-By: Claude --- .pr-review.json | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/.pr-review.json b/.pr-review.json index 3bb3305..ad368f3 100644 --- a/.pr-review.json +++ b/.pr-review.json @@ -14,7 +14,7 @@ "style": "balanced", "require_tests": true, "exclude_tests": false, - "max_findings": 12, + "max_findings": 15, "severity_threshold": "low", "patterns": { "deny": [ @@ -23,5 +23,8 @@ ] }, "cost_target": "claude-sonnet-5", - "instructions": "Minecraft plugin (Paper 26.2, Java 25 build). Watch thread-safety on event handlers (PlayerDeathEvent, PlayerInteractEvent) \u2014 the Bukkit main thread is single-threaded but async chunks/events cross it. Avoid main-thread I/O; defer expensive scans (chunk loading, spiral search) to scheduled tasks or async paths. Flag mutable shared state across listener invocations. Keep public API in /canalhandia modulo ... commands discoverable." -} \ No newline at end of file + "additional_context_urls": [ + "http://nexus-service.nexus.svc.cluster.local:8081/repository/raw-hosted/canalhandia/architecture.md" + ], + "instructions": "Minecraft plugin (Paper 26.2, pt-BR, JDK 25 build). Chat-only — never touch gameplay. Watch thread-safety on event handlers (PlayerDeathEvent, PlayerInteractEvent) — the Bukkit main thread is single-threaded but async chunks/events cross it. Avoid main-thread I/O; defer expensive scans (chunk loading, spiral search) to scheduled tasks or async paths. Flag mutable shared state across listener invocations. Hard constraints: chat messages are immutable after send (counts baked into buttons freeze at send time); names go out as translatable components so the singular-form rule applies (number never agrees with the noun); Geyser/Bedrock cannot click and cannot show emoji (every click has a typed fallback); vanilla statistics are the only data source (offline path is /players/stats/.json, NOT /stats); reactions keep counting late (reacao-validade-minutos); Floodgate is optional runtime dep. Flag: real bugs, missing persistence of new settings, comando/permission not in plugin.yml, breaking Bedrock equivalent invariant, removing the frozen-at-send assumption, violating singular-form rule, missing Stats.resolve() on renames." +}