Merge branch 'develop'

Lands the synthesized 8-section project reference at the repo root so team
members can understand ScoreKeeper without reading the whole codebase.
Cross-checked against live source: in-memory HashMap<UUID,Integer> scores,
five /score-* commands, /score-archive + persistence as unimplemented stubs.
Sourced from completed children t_2b93fd59 / t_1f4982b3 / t_f5d60eac /
t_710fadae.

.gitignore

@@ -1,3 +1,5 @@
 .gradle/
 build/
 target/
+.worktrees/
+bin/

.settings/org.eclipse.core.resources.prefs

@@ -1,5 +0,0 @@
-#Sun Mar 04 01:03:26 PST 2012
-eclipse.preferences.version=1
-encoding//src/main/java=UTF-8
-encoding//src/main/resources=UTF-8
-encoding/<project>=UTF-8

.settings/org.eclipse.jdt.apt.core.prefs

@@ -1,2 +0,0 @@
-eclipse.preferences.version=1
-org.eclipse.jdt.apt.aptEnabled=false

.settings/org.eclipse.jdt.core.prefs

@@ -1,10 +0,0 @@
-eclipse.preferences.version=1
-org.eclipse.jdt.core.classpath.outputOverlappingAnotherSource=ignore
-org.eclipse.jdt.core.compiler.codegen.targetPlatform=1.8
-org.eclipse.jdt.core.compiler.compliance=1.8
-org.eclipse.jdt.core.compiler.problem.enablePreviewFeatures=disabled
-org.eclipse.jdt.core.compiler.problem.forbiddenReference=warning
-org.eclipse.jdt.core.compiler.problem.reportPreviewFeatures=ignore
-org.eclipse.jdt.core.compiler.processAnnotations=disabled
-org.eclipse.jdt.core.compiler.release=disabled
-org.eclipse.jdt.core.compiler.source=1.8

.settings/org.eclipse.m2e.core.prefs

@@ -1,5 +0,0 @@
-#Sun Feb 19 04:11:11 PST 2012
-activeProfiles=
-eclipse.preferences.version=1
-resolveWorkspaceProjects=true
-version=1

.vscode/settings.json

@@ -2,5 +2,9 @@
   "[java]": {
     "editor.formatOnSave": true
   },
-    "java.configuration.updateBuildConfiguration": "interactive"
+  "java.configuration.updateBuildConfiguration": "interactive",
+  "java.compile.nullAnalysis.mode": "automatic",
+  "chat.tools.terminal.autoApprove": {
+    "printf": true
+  }
 }

CONTRIBUTING.md

@@ -3,44 +3,154 @@
 
 Thank you for your interest in contributing to the ScoreKeeper plugin! This guide will help you set up your development environment and follow best practices for working with the project.
 
-## Development Environment Setup
+# Getting Started
+Pick a folder to hold your minecraft server and this project I'm going to use `$HOME/Projects`. My Minecraft server will be in `$HOME/Projects/MCServer` and this project will be in `$HOME/Projects/ScoreKeeper`.
 
-To develop and test the ScoreKeeper plugin, you will need:
-- A Paper Minecraft server (for plugin deployment and testing)
-- The official Minecraft Java Edition client (for connecting to your test server)
+Setup our project folder.
 
-### 1. Setting up the Paper Server (macOS/Linux/WSL)
-1. Download the latest Paper server jar from [https://papermc.io/downloads](https://papermc.io/downloads).
-2. Create a directory for your test server and place the Paper jar inside.
-3. Set the following environment variables in your shell profile (e.g., `.bashrc`, `.zshrc`, or manually in your terminal):
+**Linux, macOS, and WSL**
 ```bash
-   export MINECRAFT_SERVER_PATH=/path/to/your/server
-   export MINECRAFT_SERVER_JAR=paper-<version>.jar  # Can be relative to MINECRAFT_SERVER_PATH or absolute
+PROJECTS_DIR="$HOME/Projects"
+MINECRAFT_SERVER_PATH="$PROJECTS_DIR/MCServer"
+# Set this to the latest download url on the Paper downloads page [https://papermc.io/downloads](https://papermc.io/downloads) and update the jar name.
+MINECRAFT_SERVER_DOWNLOAD_URI="https://fill-data.papermc.io/v1/objects/fb73c7e310215016955617ab957022d9e1d47aeba206df3a98c5ecb43756527c/paper-1.21.8-25.jar"
+MINECRAFT_SERVER_JAR="paper-1.21.8-25.jar"
+mkdir -p "$MINECRAFT_SERVER_PATH"
+# Keep this shell open we will use these variables later.
 ```
-4. Use the provided scripts to manage your server:
-   - To start (or restart) the server: `tools/bash/start-server.sh`
-   - To stop the server: `tools/bash/stop-server.sh`
-   These scripts use the environment variables above to locate and manage your server.
-5. On first run, accept the EULA by editing `eula.txt` and setting `eula=true` in your server directory.
-6. Place your built plugin jar in the `plugins/` directory.
-7. Use the start script again to restart the server and load the plugin.
 
-### 2. Setting up the Minecraft Client
+**Windows**
+```powershell
+$projectsDir="$HOME\Projects"
+$minecraftServerDir="$projectsDir\MCServer"
+# Set this to the latest download url on the Paper downloads page [https://papermc.io/downloads](https://papermc.io/downloads) and update the jar name.
+$minecraftServerDownloadUri="https://fill-data.papermc.io/v1/objects/fb73c7e310215016955617ab957022d9e1d47aeba206df3a98c5ecb43756527c/paper-1.21.8-25.jar"
+$minecraftServerJar="paper-1.21.8-25.jar"
+New-Item -Type Directory $projectsDir
+New-Item -Type Directory $minecraftServerDir
+# Keep this shell open we will use these variables later.
+```
+
+## Install the JDK
+**Linux / WSL**
+```bash
+apt install openjdk-21-jdk
+```
+
+**macOS**
+```zsh
+brew install --cask microsoft-openjdk@21
+```
+
+**Windows**
+From Powershell
+```powershell
+winget install --id Microsoft.OpenJDK.21 --source winget
+```
+
+## Setup a PaperMC Server
+**Linux, macOS, and WSL**
+```bash
+# You're probably still here, but if not go to the server dir.
+cd "$MINECRAFT_SERVER_PATH"
+curl -O "$MINECRAFT_SERVER_DOWNLOAD_URI"
+java -jar "$MINECRAFT_SERVER_JAR"
+nano eula.txt # Change eula=false to eula=true, then save and exit.
+java -jar "$MINECRAFT_SERVER_JAR"
+```
+
+**Windows**
+```powershell
+# You're probably still here, but if not go to the server dir.
+Set-Location $minecraftServerDir
+Invoke-WebRequest -Uri $minecraftServerDownloadUri -OutFile (Split-Path $minecraftServerDownloadUri -Leaf)
+java -jar $minecraftServerJar
+notepad eula.txt
+# Change eula=false to eula=true, then save and close the file.
+java -jar $minecraftServerJar
+```
+
+If you get windows security popups about java with the little java dude allow them. It's because the server is listening on the network. Now you should have the server running and be dropped at a prompt like `> `. Type `stop` and hit enter to kill the server. You can also try CTRL + C.
+
+## Get the Code
+**Linux, macOS, and WSL**
+```bash
+cd "$PROJECTS_DIR"
+git clone https://github.com/headhunter45/ScoreKeeper.git
+code ScoreKeeper
+```
+
+```powershell
+Set-Location $projectsDir
+git clone https://github.com/headhunter45/ScoreKeeper.git
+# Then to open VSCode
+code ScoreKeeper
+```
+
+## Building the Plugin
+This should download the preferred version of our build tool (gradle) and use it to build the project. Among other intermediate files it will create `build/libs/ScoreKeeper-${Version}.jar`. This is the plugin jar that you copy to the minecraft server to install the plugin. From now on you will use the utility scripts in `tools\powershell\*.ps1` or `tools/bash/*.sh` to build and deploy the plugin to our test server.
+
+**Linux, macOS, and WSL**
+```bash
+./gradlew build
+```
+
+**Windows**
+```powershell
+.\gradlew.bat build
+```
+
+## Using the Utility Scripts
+Our scripts depend on two environment variables. You can set them each time you open a shell or set them once for every shell your user opens. The variables are `MINECRAFT_SERVER_PATH` and `MINECRAFT_SERVER_JAR`. They are used by the scripts to know where the server is so they can control it and deploy the plugin.
+
+**Linux, macOS, and WSL**
+
+To set them for the current session run these commands. To set for every new session add these commands to your `.bashrc` or `.zshrc` file.
+```bash
+MINECRAFT_SERVER_PATH="$HOME/Projects/MCServer"
+MINECRAFT_SERVER_JAR="paper-1.21.8-25.jar"
+```
+
+**Windows**
+
+To set them for a single session run these commands.
+```powershell
+$env:MINECRAFT_SERVER_PATH = "$HOME\Projects\MCServer"
+$env:MINECRAFT_SERVER_JAR = "paper-1.21.8-25.jar"
+```
+
+If you want them available in all of your powershell sessions you can set them permanently like this.
+```powershell
+[Environment]::SetEnvironmentVariable("MINECRAFT_SERVER_PATH", "$HOME\Projects\MCServer", "User")
+[Environment]::SetEnvironmentVariable("MINECRAFT_SERVER_JAR", "paper-1.21.8-25.jar", "User")
+```
+
+### Start-Server.ps1 and start-server.sh
+Run `tools\powershell\Start-Server.ps1` or `tools/bash/start-server.sh` to start the server and open a GUI to manage it.
+
+### Stop-Server.ps1 and stop-server.sh
+Run `tools\powershell\Stop-Server.ps1` or `tools/bash/stop-server.sh` to stop the server. You can also safely close the GUI.
+
+### Build-Plugin.ps1 and build-plugin.sh
+Run `tools\powershell\Build-Plugin.ps1` or `tools/bash/build-plugin.sh` to build the plugin. This builds the plugin in `build/libs` as a jar named `ScoreKeeper-${VERSION}.jar`. To install the plugin manually you can copy this to `$env:MINECRAFT_SERVER_PATH\plugins` or `$MINECRAFT_SERVER_PATH/plugins` and restart the server.
+
+### Deploy-Plugin.ps1 and deploy-plugin.sh
+Run `tools\powershell\Deploy-Plugin.ps1` or `tools/bash/deploy-plugin.sh` to copy the plugin to the server plugins directory. This checks if the built plugin is up to date with the source code. If it is, then the script copies the plugin to the plugins directory to install it. To use the new plugin restart the server.
+
+# Setting up the Minecraft Client
 - **macOS / Linux:**
   - Download and install the official Minecraft Launcher from [minecraft.net](https://www.minecraft.net/en-us/download).
   - Log in with your Mojang/Microsoft account and launch the Java Edition client.
-- **WSL (Windows Subsystem for Linux):**
+- **Windows / WSL (Windows Subsystem for Linux):**
   - For best performance, run the Minecraft client directly under Windows, not inside WSL.
   - Download and install the official Minecraft Launcher for Windows from [minecraft.net](https://www.minecraft.net/en-us/download).
   - Use WSL for server and plugin development, and connect to your test server from the Windows client.
 
-### 3. Connecting the Client to Your Server
+# Connecting the Client to Your Server
 - Start your Paper server (see above).
 - In the Minecraft client, add a new server with the address:
   - `localhost` (if running both client and server on the same machine)
   - Or use your machine's IP address if connecting across devices.
 
-### 4. Notes
-- You can automate server startup and plugin deployment with bash scripts for faster testing.
-- For automated testing or CI, run the Paper server in a headless Linux environment (including WSL, Docker, or CI runners).
+# Notes
 - Always use the provided scripts and environment variables for consistent server management.

Project.md

@@ -0,0 +1,280 @@
+# ScoreKeeper — Project Reference
+
+> **For a developer joining the team cold.** Read top-to-bottom in ~8 minutes;
+> you'll know where to look and what's real vs. planned on day one.
+
+---
+
+## 1. What is ScoreKeeper?
+
+ScoreKeeper is a **Paper Minecraft plugin** (Java 21, Bukkit/Adventure APIs) that manages player
+score tracking. It exposes five in-game/console commands (`/score-get`, `/score-add`,
+`/score-subtract`, `/score-reset`, `/score-archive`) for manual score manipulation — no events,
+no timers, no automation.
+
+**State of the project:** live in-memory scores work today; the \"high-score table\" advertised
+in the README name and `/score-archive` description is **not implemented**. Persistence, ranking,
+and scoring formulas are all **future scope**. See §4 for the gap between *what exists* and
+*what is planned*.
+
+---
+
+## 2. Tech Stack & Build
+
+| Item | Value |
+|------|-------|
+| Language | Java 21 (Gradle auto-downloads toolchain) |
+| Runtime target | Paper 1.21.7 (Bukkit + Adventure APIs) |
+| Build tool | Gradle 8.14.3 (`gradlew`) |
+| Linting | Spotless (`googleJavaFormat()` + license header from `config/license-header.txt`) |
+| Release | `net.researchgate.release` plugin; tags `v$version`; rejects snapshot deps except paper-api |
+| External deps | **None at runtime.** `paper-api:1.21.7-R0.1-SNAPSHOT` is `compileOnly` only. |
+| CI/release entry point | `./gradlew spotlessCheck build` then `./gradlew release` |
+
+### Build quickly
+
+```bash
+./gradlew build          # compiles, runs tests (none yet)
+./gradlew spotlessApply  # format to project style
+./gradlew assemble       # produces jar in build/libs/
+```
+
+Drop the resulting jar onto a Paper server's `plugins/` directory.
+
+---
+
+## 3. Architecture Overview
+
+### Directory layout
+
+```
+├── src/main/java/com/majinnaibu/minecraft/plugins/scorekeeper/
+│   ├── ScoreKeeperPlugin.java        ← entry point, onEnable/onDisable, score CRUD core
+│   └── commands/
+│       ├── ScoreGetCommand.java      → /score-get [player]
+│       ├── ScoreAddCommand.java      → /score-add [player] <amount>
+│       ├── ScoreSubtractCommand.java → /score-subtract [player] <amount>
+│       ├── ScoreResetCommand.java    → /score-reset [player]
+│       └── ScoreArchiveCommand.java  → /score-archive [player] (stub)
+├── src/main/resources/
+│   └── plugin.yml                    ← command manifest + main-class declare
+├── tools/bash/ & tools/powershell/  ← dev helper scripts
+├── config/license-header.txt         ← Spotless license header
+├── build.gradle                      ← Gradle build config (above)
+├── gradle.properties                 ← version, group coordinates
+├── CONTRIBUTING.md                   ← dev env setup
+└── README.md                         ← user-facing command reference + notes
+```
+
+### Data flow (text diagram)
+
+```
+ADMIN / CONSOLE / RCON ──▶ types a /score-* command
+                              │
+                              ▼
+                 ┌──────────────────────┐
+                 │ 5× Score*Command.java│   executors parse args,
+                 │ resolves target      │   resolvePlayerExact(name) (online-only)
+                 └──────────┬───────────┘
+                            │ calls
+                            ▼
+              ┌──────────────────────────────┐
+              │  ScoreKeeperPlugin            │
+              │                                │
+              │  addScore / subtractScore     │  read-modify-write
+              │  resetScore / setScore        │
+              │  getScore (read)              │
+              │  archiveScore (STUB — no-op) │
+              └──────────┬───────────────────┘
+                         │
+                         ▼
+               HashMap<UUID, Integer>   ← in RAM only
+                 _playerScores          ← ScoreKeeperPlugin:38
+```
+
+**Key architectural facts:**
+
+- **Single class owns everything.** `ScoreKeeperPlugin` holds the scores map, score CRUD
+  methods, logging helpers (logWarning/logInfo/logError), and the Adventure component builder
+  for chat color. There is no service layer or boundary separation.
+- **No event listeners.** Zero `@EventHandler`/`Listener` registrations across the codebase.
+  Game events (join, death, kill) do not affect scores. Only manual commands change state.
+- **No scheduler or tick logic.** No periodic tasks, no countdowns, no automated scoring.
+- **No permission nodes.** Every command is available to every sender (player, console, RCON).
+
+### File: ScoreKeeperPlugin.java (`ScoreKeeperPlugin`)
+
+- `main` class in `plugin.yml` (Bukkit plugin entry point)
+- `HashMap<UUID, Integer> _playerScores` at line 38 — the sole score store
+- `onEnable()` (line 52): wires five command executors, logs \"load not implemented\" warning
+- `onDisable()` (line 46): logs \"save not implemented\" warning — scores lost on shutdown
+- Score CRUD methods: `addScore`, `subtractScore`, `resetScore`, `setScore`, `getScore`,
+  `archiveScore` (lines 67–91, 76–86, 72–74)
+- Private helpers: `getPlayerScore(Player)` (get-or-create at 0), `setPlayerScore(Player,int)`
+
+---
+
+## 4. Domain Model & Scoring Logic
+
+### 4.1 What exists today (REAL — in code)
+
+| Aspect | Detail | Source |
+|--------|--------|--------|
+| Score model | Single `int` per player, keyed by UUID | `ScoreKeeperPlugin:38` |
+| Storage container | `HashMap<UUID, Integer>` on plugin instance | same file |
+| Default value | `0` — lazy-created on first map access | `getPlayerScore:100-106` |
+| Live only | **No persistence.** Scores erased on server restart | `onEnable:59`, `onDisable:47` |
+| Scoring direction | Any integer (negatives allowed, no floor) | add/subtract are raw `+`/`-` |
+| Recording method | Manual commands only — `/score-add`, `/score-subtract` | command executors |
+| Auto-scoring | **None** — no events, no timers | proven by grep across src/ |
+| Permissions | **None declared.** All commands open to all senders | `plugin.yml`, no permission guard in code |
+
+### 4.2 Open Score Lifecycle (what exists + what is planned)
+
+```
+Stage  1. First access — getPlayerScore() auto-creates key at 0   [REAL]
+         2. Admin runs /score-add player N or /score-subtract       [REAL]
+         3. Player accumulates points over the session             [REAL]
+         4. Read via /score-get (read-only, lazily registers)      [REAL]
+         5. Server restart — scores LOST on shutdown               [REAL]
+         6. Intended: /score-archive freezes score → table        [GAP ✓ not built]
+         7. Intended: high-score table display command            [GAP ✓ not built]
+```
+
+### 4.3 The "high-score table" — status
+
+| Feature | Status | Details |
+|---------|--------|---------|
+| `/score-archive` | **Stub** | Command prints \"archive command unimplemented\"; `archiveScore()` method only logs, never writes a table or resets the player's score |
+| Persistence (save) | **Not built** | `onDisable()` is a TODO stub — map discarded at shutdown |
+| Persistence (load) | **Not built** | `onEnable()` is a TODO stub — map always starts empty `{}` |
+| Sorting / ranking | **Not coded** | No sort, no tie-breaking, no entry cap, no decay logic exists |
+| Scoring formulas | **Not coded** | Scores are plain integer accumulators (`Σ(adds) − Σ(subtracts)`) |
+
+**Design decisions to be made (none answered by code today):**
+
+- Storage format for the table (YAML per Bukkit convention; JSON? SQLite?)
+- What an entry looks like (name + score + timestamp? name is not stored with score today)
+- Sort order and tie-breaking strategy
+- Max entries / leaderboard cap
+- Whether `/score-archive` also resets the live score (README says it does)
+
+---
+
+## 5. Plugin Integration & Reference Table
+
+### 5.1 Command → Handler → Effect on score data
+
+| Trigger | Usage | Handler | Effect |
+|---------|-------|---------|--------|
+| `/score-get [player]` | Self or other | `ScoreGetCommand.java:38-84`, delegated to `getScore → getPlayerScore` | **Read-only.** Returns integer. Lazily creates entry at `0` if unseen. |
+| `/score-add [player] <N>` | Self (omit name) or target others | `ScoreAddCommand.java` | `score += N`. Amount must parse as int. No direction validation (negative N still adds). |
+| `/score-subtract [player] <N>` | Same | `ScoreSubtractCommand.java` | `score -= N`. No minimum clamping; negatives freely produced. |
+| `/score-reset [player]` | Same | `ScoreResetCommand.java` | `score = 0`. Key created at `0` if absent. |
+| `/score-archive [player]` | Same | `ScoreArchiveCommand.java` | **No-op.** Prints \"archive command unimplemented\". Does NOT call the `archiveScore()` method. |
+
+### 5.2 Shared behavior details
+
+Every executor follows this pattern:
+
+1. Parse arguments — if `split.length == 1` and sender is a player, target = self (RCON/console
+   requires an explicit `<playerName>` or prints usage).
+2. Resolve target player via `server.getPlayerExact(name)` — **exact, case-sensitive, online-only.**
+3. Call the corresponding `ScoreKeeperPlugin` method.
+4. Echo color-coded result; return `true`.
+
+### 5.3 Inter-plugin / public API surface
+
+`ScoreKeeperPlugin` exposes these public methods that other plugins *could* call if they hold a reference
+(but there is **no formal service registration**):
+
+| Method | Visibility | Called by commands? | Notes |
+|--------|------------|---------------------|-------|
+| `addScore(Player, int)` | `public` | Yes (`/score-add`) | Read-modify-write on `_playerScores` |
+| `subtractScore(Player, int)` | `public` | Yes (`/score-subtract`) | Same pattern |
+| `getScore(Player)` | `public` | Yes (`/score-get`) | Wrapper around `getPlayerScore` |
+| `setScore(Player, int)` | `public` | **No** | Internal write path only; not hooked to any command |
+| `resetScore(Player)` | `public` | Yes (`/score-reset`) | Sets to `0` |
+| `archiveScore(Player)` | `public` | **No** | Only logs a warning; never called by the archive command |
+
+### 5.4 Concurrency note
+
+The map is a plain `HashMap`. Safe because Paper dispatches commands on the server's single main
+thread — but it is **not safe for off-thread use**. Any future event-driven scoring that runs
+asynchronously could corrupt state via non-atomic read-modify-write.
+
+---
+
+## 6. Configuration Reference
+
+| File | Format | Purpose |
+|------|--------|---------|
+| `plugin.yml` | YAML (Bukkit manifest) | Declares main class, api-version, five commands + descriptions/usage strings |
+| `build.gradle` | Gradle Kotlin (Groovy DSL) | Dependencies, task config, release/spotless settings |
+| `gradle.properties` | Properties | Project version and group coordinates (for Maven publishing) |
+| **No `config.yml`** | — | ScoreKeeper has **no player-editable configuration**. |
+| **No `permissions:` block** | — | No permission nodes declared; all commands are open. Plans say \"permissions coming after archive works.\" |
+
+---
+
+## 7. Running & Testing Locally
+
+### Build
+
+```bash
+./gradlew assemble           # produces ScoreKeeper.jar in build/libs/
+```
+
+### Run locally (Paper server)
+
+1. Download Paper 1.21.7 from `https://papermc.io`
+2. Copy the built jar into `plugins/`
+3. Start the server, verify onEnable logs:
+   ```
+   [ScoreKeeper] ScoreKeeper version X.Y.Z is enabled.
+   [ScoreKeeper] Unable to load scores from file. This feature is not implemented yet.
+   ```
+
+### Commands (in-game or via RCON/console)
+
+| Test scenario | Command | Expected output |
+|--------------|---------|-----------------|
+| Check own score | `/score-get` | \"Your score is 0\" (creates entry at 0) |
+| Add points to self | `/score-add 10` | \"You gained 10 points! Now have 10.\" |
+| Subtract from self | `/score-subtract 5` | \"You lost 5 points! Now have 5.\" |
+| Check another player | `/score-get PlayerName` | \"PlayerName's score is N.\" (must be online) |
+| Reset own score | `/score-reset` | \"Your score has been reset to 0.\" |
+| Archive (stub) | `/score-archive` | \"archive command unimplemented\" |
+
+### Tests
+
+- **No unit/integration tests exist yet.** `./gradlew test` runs an empty suite. Consider
+  adding tests for `ScoreKeeperPlugin`'s private `Player` score state mocking in a future task.
+
+---
+
+## 8. Where to Look First
+
+| I want to understand… | Go to |
+|----------------------|-------|
+| The entire live data model | `ScoreKeeperPlugin:38` — one `HashMap<UUID, Integer>` field |
+| How scores change (write path) | `addScore(/:67-70)`, `subtractScore(/:88-91)`, `setPlayerScore(/:108-110)` |
+| How a player is looked up | `getPlayerExact(name)` online only — see any `*Command.java` line ~50 |
+| First-time-player behavior | `getPlayerScore(/:100-106)` — auto-inserts `0` on first access |
+| High-score / archive table | **Does not exist.** See `archiveScore(/:72-74)` stub; design decisions in §4.3 |
+| Persistence (save/load) | `onDisable(/:47)`, `onEnable(/:59)` — both TODO, nothing writes to disk |
+| What's declared/intended but not wired | `plugin.yml:18-20` (`/score-archive`), `README.md:11` and README notes |
+| Command implementations | `commands/Score*Command.java` (all 5 handlers) |
+| Build/runtime config | `build.gradle`, `plugin.yml`, `gradle.properties` |
+| Dev env / CONTRIBUTING | `CONTRIBUTING.md` |
+---
+
+## Appendix: Risk Summary
+
+| # | Issue | Impact | Section |
+|---|-------|--------|---------|
+| 1 | **Data loss on restart** — no save/load implemented | Every server boot wipes all scores (§4.1) | §4.1 |
+| 2 | **No permissions** — any player can self-add points | Integrity of scoring is unenforceable today (§5.2) | §5.2 |
+| 3 | **Race condition on add/subtract** — non-atomic read-modify-write on plain `HashMap` | Corrupt state if future event-driven scoring runs async (§5.4) | §5.4 |
+| 4 | **No `int` overflow safety** — Java wrapping semantics apply | Undetectable score corruption near ±2.1 billion | §4.1 |
+| 5 | **Name lookups online-only & case-sensitive** | Can't target offline players; \"Alice\" ≠ \"alice\" (§5.2) | §5.2 |

ProjectDescription.json

@@ -0,0 +1,23 @@
+{
+  "id": "scorekeeper",
+  "title": "ScoreKeeper",
+  "description": "Paper Minecraft plugin for tracking and managing player scores.",
+  "docId": "ScoreKeeper",
+  "image": {
+    "url": "https://image.pollinations.ai/prompt/pixel%20art%20Minecraft%20server%20scoreboard%20with%20a%20glowing%20diamond%20and%20number%20counters%2C%20clean%20square%20game%20plugin%20thumbnail%2C%20no%20text",
+    "alt": "Pixel art scoreboard with a diamond for the ScoreKeeper Minecraft plugin",
+    "prompt": "Pixel art Minecraft server scoreboard with a glowing diamond and number counters, clean square game plugin thumbnail, no text",
+    "width": 1024,
+    "height": 1024,
+    "backgroundClassName": "bg-surface"
+  },
+  "url": "/projects/scorekeeper",
+  "tags": [],
+  "source": {
+    "type": "",
+    "url": ""
+  },
+  "feedback": {
+    "url": ""
+  }
+}

ProjectDescription.md

@@ -0,0 +1,16 @@
+# ScoreKeeper
+
+ScoreKeeper is a Paper Minecraft plugin for tracking player scores during server activities, events, or games. It keeps scores per player in memory and exposes commands for viewing, changing, resetting, and archiving scores.
+
+## Details
+
+- Built for Paper API 1.21.7 with Java 21 and Gradle.
+- Registers `/score-get`, `/score-add`, `/score-subtract`, `/score-reset`, and `/score-archive`.
+- Commands use the executing player when a player name is omitted.
+- Scores are currently held in memory and are not persisted when the server stops.
+- Score archiving is planned, but the current implementation reports that archiving is unavailable.
+- The project is packaged as a Java plugin with a `plugin.yml` descriptor.
+
+## Image
+
+No project image asset is included in the repository. The JSON description includes a prompt-backed image URL for a future project thumbnail.

README.md

@@ -19,3 +19,5 @@ NOTES:
 
 See [CONTRIBUTING.md](CONTRIBUTING.md) for development environment setup instructions.
 
+Tooling note:
+* Gradle is configured with Java Toolchains and will automatically download a Java 21 JDK when needed during builds.

build.gradle

@@ -6,7 +6,6 @@ plugins {
 }
 
 group = 'com.majinnaibu.minecraft.plugins'
-version = '0.2.1-SNAPSHOT'
 description = "ScoreKeeper"
 
 tasks.withType(JavaCompile).configureEach {
@@ -44,12 +43,14 @@ publishing {
 }
 
 release {
+//    buildTasks.addAll(['spotlessCheck', 'build'])
+    buildTasks = ['spotlessCheck', 'build']
     failOnSnapshotDependencies = true
     ignoredSnapshotDependencies = [
         "io.papermc.paper:paper-api"
     ]
     revertOnFail = true
-    tagTemplate = '$version'
+    tagTemplate = 'v$version'
     versionPropertyFile = 'gradle.properties'
 }
 

docs/domain-score-tracking.md

@@ -0,0 +1,365 @@
+# ScoreKeeper — Score-Tracking & "High-Score Table" Domain Analysis
+
+> **Scope of this document:** the domain logic that tracks player scores and (intended) the
+> high-score table. Written for a new team member who must understand the scoring system
+> *without opening the source*. Every claim below is traced to a file/line so it can be
+> re-verified.
+>
+> **Bottom line up front (read this first):** ScoreKeeper currently tracks **live scores
+> only, in memory, for the running server session**. It has **no persistence** and **no
+> high-score table**. The high-score table is *described as a planned feature* in the
+> README and is where the `/score-archive` command is *intended* to land scores, but
+> **that code path is an unimplemented stub today.** There is no sorting, no tie-breaking,
+> no entry cap, no decay, and no scoring formula/weighting anywhere in the codebase.
+> This is a greenfield/early-stage domain — the write-up documents the *real* state and
+> flags the gap, because documenting features that don't exist as if they did would mislead
+> the next engineer.
+
+---
+
+## 0. How to read the rest of this doc against the task's four questions
+
+The task asks for (1) the score-entry data model + persistence format, (2) how scores are
+recorded, (3) the high-score-table algorithm, and (4) scoring formulas/weighting.
+For each question the answer is: **the mechanism that exists vs. the mechanism the project
+intends**, so readers never conflate the two.
+
+| # | Question | Short answer |
+|---|----------|--------------|
+| 1 | Score-entry data model & persistence | Live score = `UUID -> int` in a `HashMap`. **No persistence** (load/save are `TODO` stubs). |
+| 2 | How scores are recorded | **Manual, event-free:** the *administrator* runs a command. No game events, no timers. |
+| 3 | High-score-table algorithm | **Not implemented.** No sort/tie-break/cap/decay. Only an aspirational `/score-archive`. |
+| 4 | Scoring formulas / weighting | **None.** Scores are plain integer accumulators; no weighting, no decay, no bonuses. |
+
+---
+
+## 1. The score-entry data model & persistence  *(task Q1)*
+
+### 1.1 What a "score entry" actually is (current reality)
+
+The entire live-score domain is **one field** in the plugin's entry-point class:
+
+- **File:** `src/main/java/com/majinnaibu/minecraft/plugins/scorekeeper/ScoreKeeperPlugin.java:38`
+- **Declaration:** `private final HashMap<UUID, Integer> _playerScores = new HashMap<UUID, Integer>();`
+
+That is the whole model:
+
+| Aspect | Value |
+|--------|-------|
+| Record shape | A single `Integer` (the score), keyed by a player's `UUID`. |
+| Key type | `java.util.UUID` — the player's `UniqueId`, *not* the name string. |
+| Value type | `int` (Java `Integer`), autoboxed into the map. |
+| Container | one `java.util.HashMap` on the plugin instance. |
+| Uniqueness | one entry per `UUID`. A player can have exactly one live score at a time. |
+| Default for a new player | `0` — see §3.1 ("first-time player" edge case). |
+
+There is **no `Score`/`ScoreEntry`/`PlayerStats` value class**, no timestamp, no
+name stored alongside the score (name is looked up from the live server only — see §2.3),
+and no secondary index.
+
+### 1.2 Persistence format: **none exists**
+
+Persistence is *intended* but *not built*. Three concrete stubs prove this:
+
+1. **On disable (server stop):** `ScoreKeeperPlugin.java:46-49` `onDisable()` logs
+   *"Unable to save scores to file. This feature is not implemented yet."* — the map
+   is **discarded** on shutdown; nothing is written.
+2. **On enable (server start):** `ScoreKeeperPlugin.java:59-60` `onEnable()` logs
+   *"Unable to load scores from file. This feature is not implemented yet."* — the map
+   always starts **empty** `{}` every server start.
+3. **Archive:** `ScoreKeeperPlugin.java:72-74` `archiveScore(Player)` logs
+   *"Unable to archive score for <name>."* and returns — the "save to high-score table"
+   action is a no-op.
+
+So there is **no SQLite, no JSON, no YAML, no flat file, no config.yml**. The only file on
+disk that concerns scores is `plugin.yml` (the command manifest — it lists the commands but
+stores no scores). A grep of the whole tree for `sql / json / yaml / saveResource /
+YamlConfiguration / File / Files. / Gson / Jackson` finds **zero hits** other than the
+TODO/log strings.
+
+**Persistence format today = "in RAM only, lost on server restart."**
+
+> Implication for a new engineer: any "who's the top player" question answered *across
+> restarts* is impossible today. Scores reset to the map being empty on every server boot.
+
+---
+
+## 2. How scores are recorded  *(task Q2)*
+
+### 2.1 The recording model is **manual and command-driven — no events, no timers**
+
+Scores move only when an administrator (or console/RCON) **types a command**. There is:
+
+- **No** `implements Listener`, no `@EventHandler`, no `registerEvents(...)`, no
+  `getServer().getPluginManager().registerEvents(...)`.
+- **No** scheduler: no `getScheduler()`, no `runTask`, no `Timer`.
+- **No** automatic hook to game events (e.g. a player killing a mob or reaching a goal
+  awards points automatically — *nothing like that exists*).
+
+A grep for `EventListener|@EventHandler|Listener|Scheduler|runTask` across `src/` returns
+**zero**.
+
+So "scoring" happens **on demand**, by a person, via one of the four mutating commands:
+
+| Command | Effect on the map | Source |
+|---------|-------------------|--------|
+| `/score-add [player] <amount>` | `score += amount` | `ScoreKeeperPlugin.addScore` `:67-70` |
+| `/score-subtract [player] <amount>` | `score -= amount` | `ScoreKeeperPlugin.subtractScore` `:88-91` |
+| `/score-reset [player]` | `score = 0` | `ScoreKeeperPlugin.resetScore` `:80-82` |
+| `/score-archive [player]` | **intended** to snapshot to a high-score table and reset to 0; **actually a no-op** | `ScoreKeeperPlugin.archiveScore` `:72-74` |
+
+`/score-get [player]` is read-only (returns the score, §2.4).
+
+### 2.2 The add/subtract primitives (the only real "write" path)
+
+Both funnel through one private setter:
+
+```
+addScore(player, n)    ->  old = getPlayerScore(player); setPlayerScore(player, old + n)   :67-70
+subtractScore(player,n)->  old = getPlayerScore(player); setPlayerScore(player, old - n)   :88-91
+setScore(player, s)    ->  setPlayerScore(player, s)                                       :84-86
+resetScore(player)     ->  setPlayerScore(player, 0)                                        :80-82
+setPlayerScore(p, v)   ->  _playerScores.put(p.getUniqueId(), v)                            :108-110
+```
+
+Notes a new engineer must internalize:
+
+- **Read-modify-write.** `addScore`/`subtractScore` read the current value, compute, write
+  back in two map operations. (There is no concurrency control — see §5 risks.)
+- **`int` math.** Values are Java `int`; a score can go **negative** (there is no floor at
+  0 — `/score-subtract 50` on a 10-score yields `-40`), and can overflow `int` only after
+  `~2.1e9` points, which is not a practical concern.
+- **No validation of the direction.** The only validation is *syntactic*: the amount must
+  parse as an integer (see §2.3).
+
+### 2.3 Command surface (who/what is allowed to record a score)
+
+Every command executor lives in `src/main/java/.../scorekeeper/commands/` and shares one
+shape: parse args, resolve the target player, call the `ScoreKeeperPlugin` method, return
+`true` (consumed). Two behavioral details matter:
+
+- **Self-target default:** if a *player* runs a command and omits the name, they target
+  *themselves*: `targetPlayer = (Player) sender`. (e.g. `ScoreGetCommand.java:44`,
+  `ScoreAddCommand.java:48`, `ScoreResetCommand.java:47-49`.)
+- **RCON/console path:** if the sender is **not** a player (console/RCON), a player name
+  is **required** — omitting it prints usage instead of acting
+  (`ScoreGetCommand.java:46-48`, `ScoreAddCommand.java:45-47`,
+  `ScoreResetCommand.java:44-46`).
+- **Name resolution:** `getServer().getPlayerExact(name)` — **exact, case-sensitive**
+  lookup by the *currently online* name (`ScoreGetCommand.java:50`,
+  `ScoreAddCommand.java:57`, `ScoreResetCommand.java:50`). A name that is not online
+  yields `null` → "Can't find a player with that name". Note the *map key is a UUID*, but
+  *lookup by name only works while that player is online* — you cannot change an offline
+  player's score by name today.
+- **No permission nodes anywhere.** `plugin.yml` declares no `permission:`/`permissions:`
+  and the code checks none. The README's note ("the commands aside from get are intended
+  for admins … Permissions support is coming") is **intent, not enforcement** — a vanilla
+  player can run `/score-add` today.
+
+### 2.4 Reading a score (`/score-get`)
+
+`ScoreGetCommand.java:38-84` → `ScoreKeeperPlugin.getScore(player)` → `getPlayerScore(player)`
+(`ScoreKeeperPlugin.java:76-78, 100-106`). It returns the live integer (0 for a known-but-
+unwritten player, which also *materializes* a 0 entry, see §3.1). Output formatting
+("Your score is N" vs. "PLAYER's score is N") depends only on whether the sender is the
+target or not — it carries no ranking or table.
+
+---
+
+## 3. The high-score-table algorithm  *(task Q3)*
+
+### 3.1 What exists: **nothing. It is aspirational.**
+
+There is **no high-score table, no ranking structure, no sort, no tie-break rule, no max
+entry count, and no decay/rotation** anywhere in the codebase. The task's premise ("the
+high-score table") reflects the **product intent** in the README, not the **code**. Concretely:
+
+- The README (`README.md:11`) says `/score-archive` *"Saves the player's score to a
+  'High Scores' table and resets their current score to 0."*
+- The manifest (`plugin.yml:18-20`) declares `/score-archive` with that very description.
+- But the command is a stub: `ScoreArchiveCommand.java:37-39` (`onCommand` body) replies
+   *"archive command unimplemented"*, and the underlying `archiveScore`
+  (`ScoreKeeperPlugin.java:72-74`) only logs and returns — it neither writes a table nor
+  resets anything.
+
+Therefore the "algorithm" is: **no-op.** A new engineer asked to "implement the high-score
+table" is starting from zero; the design questions in §3.2 are **open**, not answered by code.
+
+### 3.2 Open design questions the table must settle (no code today decides these)
+
+Since no implementation exists, these are *decisions to be made*, recorded here so the
+implementation task doesn't reinvent the debate:
+
+- **Storage/persistence format.** The load/save TODOs (`:47`, `:59`) say "to file";
+  `plugin.yml`/Bukkit convention favors YAML (`YamlConfiguration`), but JSON/SQLite are all
+  viable. **Unchosen.** Persisting to `data/` via `getDataFolder()` is the idiomatic Bukkit
+  path.
+- **What an entry is.** Today a live score is just `{uuid -> int}`. A high-score entry
+  likely needs **player name + score + (timestamp?) + (date earned?)**. Name is *not*
+  currently stored with the score, so a durable table must capture the name at archive
+  time (a UUID-only entry can't render a leaderboard without the name being resolvable).
+- **Sorting / ordering.** Almost certainly **descending by score**.
+- **Tie-breaking.** Undefined. Candidate rules: by *score only*; by *earliest* archived
+  first (stable, time-ordered); by *name* (alphabetical); or "first to reach that score."
+- **Max entries / cap.** Undefined. A leaderboard needs a cap (e.g. top 10 / top 30);
+  today there is no cap because there is no table.
+- **Decay / rotation / time window.** None, and none described in docs — so the safe
+  default is **"scores are permanent, no decay."**
+- **Reset semantics on archive.** README promises archive *resets to 0* *and* records.
+  `ScoreArchiveCommand` must decide: does archiving also reset? Does it allow multi-entries
+  (a player appearing more than once) or one row per player? All open.
+
+### 3.3 Full lifecycle of a score: creation → (intended) display on the high-score table
+
+This is the acceptance-criterion walk-through. Each stage is marked **[REAL]** (in code now)
+or **[GAP]** (intended, not built).
+
+1. **Player joins / acts** — a score *doesn't* exist yet. First interaction with the map is
+   lazy (see stage 4a). **[REAL]**
+2. **Admin records a score** — `admin: /score-add Alice 10`. Parsed in `ScoreAddCommand`,
+   routed to `addScore`. The first time this runs for Alice, `getPlayerScore` **creates**
+   her entry at 0 and returns it; then `10` is written. **[REAL]**
+3. **Score accumulates** — later `/score-add Alice 5` → 15; `/score-subtract Alice 2` → 13;
+   `/score-reset Alice` → 0. All in the in-memory map, live only. **[REAL]**
+4. **Read** — `/score-get Alice` → "Alice's score is 13". No ranking shown. **[REAL]**
+5. **Server restart** — `onDisable` *cannot* save (stub), `onEnable` *cannot* load (stub);
+   the map is **lost**. Alice's 13 is gone; every score starts empty again. **[REAL] —
+   this is the current data-loss reality.**
+6. **Intended: archive to the high-score table** — `admin: /score-archive Alice` should
+   freeze Alice's 13 into a durable, sorted leaderboard and reset her live score to 0, so
+   it survives restarts. **[GAP]** — currently returns "archive command unimplemented".
+7. **Intended: display on the high-score table** — a command (not yet built) renders the
+   capped, sorted list. **[GAP]** — no such command, no structure to render.
+
+**Edge cases (explicitly, per the acceptance criteria):**
+
+- **First-time player / first write.** A player has no map entry until `getPlayerScore` is
+  called, which **inserts `0` on first access** (`ScoreKeeperPlugin.java:100-106`). So a
+  first `/score-add X 10` yields **10, not** `10 - 0` ambiguity, and a first `/score-subtract
+  X 3` yields **−3** (no minimum-0 clamp). A first `/score-get X` returns **0** and
+  *materializes* the entry at 0. This `get-or-create-0` behavior is the de-facto "first-time
+  player" rule. **[REAL]**
+- **Ties.** **Undefined** because no table exists. When the table is built, decide a
+  tie-break (§3.2). No code today can observe or sort ties — `HashMap` gives no order. **[GAP]**
+- **Table full.** **Undefined** because no cap exists. When a cap is introduced, decide
+  what happens to the displaced entry: drop it, or "rotate" it into overflow. No code
+  today enforces a maximum. **[GAP]**
+- **Offline / unknown name.** `getPlayerExact(name)` returns `null` for anyone not
+  currently online or any misspelling → the command prints "Can't find a player with that
+  name" and changes nothing. You **cannot** archive or reset an *offline* player by name
+  today; by-UUID writes (if a future table stores names+UUIDs) would fix that. **[REAL]**
+- **Negative / large scores.** No clamp; negatives allowed, `int` overflow only near
+  ~2.1 billion. **[REAL]**
+- **Double-spawn / relogin.** Keyed by `UUID`, **not name**, so a name change or two
+  players sharing a name can't collide on the *live* map; but a *name* is never stored,
+  so a future table that only persists the UUID can later fail to render a name.
+
+---
+
+## 4. Scoring formulas & weighting  *(task Q4)*
+
+**None.** There is:
+
+- No formula, no multiplier, no combo/bonus/streak logic.
+- No weighting between action types.
+- No time-based decay or "points per game" normalization.
+
+Scores are **pure integer accumulators**: `score = Σ(adds) − Σ(subtracts)` over the session,
+resettable to 0 by `/score-reset` (intended: on `/score-archive`). The only "arithmetic" in
+the code is the `+`/`−` in `addScore`/`subtractScore` and the `0` floor written by
+`resetScore`. `int` semantics apply (wrapping overflow; no checked arithmetic). Any
+formula/weighting is a **future-design** decision, not encoded today.
+
+---
+
+## 5. Data-flow diagram (text)
+
+```
+   ADMINISTRATOR / CONSOLE / RCON        (no game events, no scheduler, no auto-scoring)
+        │  types a command
+        ▼
+  ┌─────────────────────────────┐
+  │  command executors          │  commands/Score*Command.java
+  │  parse args + resolve player│  getPlayerExact(name)  ── must be ONLINE (name→UUID)
+  │  (self-target if player)    │  no permission checks
+  └──────────────┬──────────────┘
+                 │ calls
+                 ▼
+  ┌───────────────────────────────────────────────┐
+  │  ScoreKeeperPlugin (entry point; plugin.yml main)│
+  │                                                 │
+  │  addScore / subtractScore / resetScore / setScore │  read-modify-write on...
+  │  getScore (read)               archiveScore (STUB)│
+  └──────────────┬───────────────────────┬──────────┘
+                 │ read-modify-write       │ (intended: snapshot→table, reset 0)
+                 ▼                        ▼
+   live map: HashMap<UUID, int>            ╳ NOT IMPLEMENTED
+   "score entry = one int per UUID"        high-score table:
+   default 0, no clamp, in RAM only       ─ no structure, no sort,
+        │                                  ─ no persistence,
+        ├─ READ: /score-get → "N"          ─ no cap/decay
+        │
+        └─ LIFECYCLE:
+            onEnable  :  load from file  → TODO (stays {} ; lost on restart)
+            onDisable :  save to file    → TODO (map discarded at shutdown)
+
+   PERSISTENCE:  none  │  SCORE FORMULA: none  │  EVENTS/TIMERS: none  │
+   PERMISSIONS: none  │  HIGH-SCORE TABLE: aspirational stub only
+```
+
+---
+
+## 6. Quick reference — "where to look first"
+
+| I want to understand… | Go to |
+|-----------------------|-------|
+| The entire live data model | `ScoreKeeperPlugin.java:38` (one `HashMap<UUID,Integer>`) |
+| How a score changes | `addScore`/`subtractScore`/`resetScore`/`setScore` `:67-91` |
+| How a score is read | `getScore` `:76-78` → `getPlayerScore` `:100-106` (note get-or-create-0) |
+| The "high-score table" | **Doesn't exist** — see `archiveScore` stub `:72-74`; design open in §3.2 |
+| Persistence (save/load) | `onDisable` `:46-49`, `onEnable` `:59-60` — both `TODO`, not built |
+| Command behaviors | `commands/Score*Command.java` (self-target default; RCON requires a name; exact online-name lookup) |
+| What's declared/intended but not wired | `plugin.yml:18-20` (`/score-archive`), `README.md:11`, `README.md` notes |
+| Build/runtime target | `build.gradle` (paper-api 1.21.7, Java 21), `plugin.yml` (`api-version: 1.21`) |
+
+---
+
+## 7. Risks a new engineer should know (correctness gaps in the *current* live domain)
+
+These are real, not future-work — flagging because the doc's job is to prevent a new
+developer from assuming the system is more mature than it is:
+
+1. **Data loss on restart.** No save/load (`:47`, `:59`). Every server boot wipes all
+   scores. Any "leaderboard across sessions" is broken by design.
+2. **No permissions.** Anyplayer can self-add points. README says this is by intent
+   ("coming after I get archive to work") but it is not enforced.
+3. **Race condition on add/subtract.** `read → compute → write` is not atomic and the
+   `HashMap` is not thread-safe; concurrent command executions could lose an update.
+   (Low likelihood today since commands are serial on one thread, but it's a latent bug for
+   any async/event-driven scoring that gets added.)
+4. **No minimum-0 / no validation of amount sign.** Subtracts produce negatives freely.
+5. **Name lookups are online-only & exact.** Can't target offline players by name;
+   case-sensitive; "Alice" ≠ "alice".
+6. **No high-score table.** The flagship advertised feature (`/score-archive` → "High
+   Scores" table) is a no-op; a new engineer must not expect ranking/persistence to work.
+
+---
+
+## 8. Summary for the synthesizer (`Project.md` § "domain model & scoring logic")
+
+Distilled, code-grounded, and honest:
+
+- **Model:** one `int` per player `UUID`, held in `ScoreKeeperPlugin._playerScores`
+  (`HashMap`), default `0`.
+- **Recording:** manual, admin-driven commands; **no events/timers**; no permission guard.
+- **Persistence:** **none** — load/save are `TODO`; scores are lost on restart.
+- **High-score table:** **does not exist** — `/score-archive` is a stub; sorting/tie-break/
+  cap/decay are **undecided design items** (listed in §3.2).
+- **Formula/weighting:** **none** — plain `int` accumulation.
+- **Lifecycle:** create-on-first-access (0) → add/subtract/reset (live only) → read via
+  `/score-get` → *intended* archive+reset + table display **not built** → lost on restart.
+- **Edge cases:** first-time/first-write ⇒ 0-or-delta, no floor; ties/table-full ⇒ undefined
+  (no table yet); offline/unknown name ⇒ "can't find", unchanged.
+
+Everything below the "high-score table" line of the README is **future work**, and the
+write-up's job is to make that boundary explicit so the next team member isn't misled.

docs/plugin-integration-and-event-wiring.md

@@ -0,0 +1,166 @@
+# ScoreKeeper — Plugin Integration & Event/Command Reference
+
+> Analysis for the ScoreKeeper codebase. Scope: how the plugin hooks into the
+> Minecraft (Paper) server runtime — event listeners, command registrations,
+> scheduled/tick logic, and inter-plugin/API surface — with a table mapping each
+> trigger → handler class → effect on score data.
+>
+> Platform: **Paper** (`io.papermc.paper:paper-api:1.21.7-R0.1-SNAPSHOT`), Java 21,
+> Gradle. Main class: `com.majinnaibu.minecraft.plugins.scorekeeper.ScoreKeeperPlugin`.
+
+---
+
+## 1. TL;DR for a new developer
+
+ScoreKeeper is almost entirely **command-driven**. It registers **no Bukkit event
+listeners**, declares **no `@EventHandler` methods, no `Listener` implementations,
+and calls `getPluginManager().registerEvents(...)` nowhere**. It also schedules
+**no timed tasks** (no `BukkitScheduler`/`BukkitRunnable`/tick logic).
+
+The only things that touch score state are **five console/chat commands**
+(`score-get`, `score-add`, `score-subtract`, `score-reset`, `score-archive`).
+Score state lives in an **in-memory `HashMap<UUID, Integer>`** and is **not
+persisted** — `onEnable`/`onDisable` have `TODO` stubs, so data is lost on restart.
+
+There is a **`highscore` / high-score table in the project name and the task
+prompt, but no such command or logic exists yet** — it is unimplemented.
+
+---
+
+## 2. Event listeners (Bukkit/Adventure)
+
+**None.** The plugin listens to no server events. Confirmed by absence of any of:
+`@EventHandler`, `implements Listener`, `registerEvents(`, `PlayerJoinEvent`,
+`PlayerQuitEvent`, `PlayerDeathEvent`, or any other event import/registration.
+
+| Server event | Handler | Effect on score data |
+|---|---|---|
+| PlayerJoin | *none* | none |
+| PlayerQuit | *none* | none |
+| PlayerDeath | *none* | none |
+| (all other Bukkit events) | *none* | none |
+| Custom/plugin-injected events | *none* | none |
+
+**Implication:** nothing about a player's score changes as a result of gameplay
+events (joining, quitting, dying, scoring points in-game, etc.). The only way
+score data changes is through the commands in §3.
+
+---
+
+## 3. Command registrations & permissions
+
+Commands are declared in `src/main/resources/plugin.yml` (no `aliases`, no
+`permissions:` block, and no per-command `permission:` — so **every command is
+available to every sender with no permission gating**). Each is wired to an
+executor in `ScoreKeeperPlugin.onEnable()`.
+
+Argument-resolution idiom shared by every handler:
+`boolean rcon = !(sender instanceof Player);` — an RCON caller has no self
+context, so the "target = self" shortcut is unavailable and a `<playerName>`
+must be supplied.
+
+`<playerName>` is resolved with `server.getPlayerExact(name)` — an **exact,
+case-sensitive lookup of currently-online players only**. Score operations
+therefore apply to **online players**, and an unknown/offline name yields a
+"Can't find a player with that name" error (except where noted).
+
+### Reference table — command → handler class → effect on score data
+
+| Trigger (command) | Usage | Handler class | Delegates to | Effect on score data |
+|---|---|---|---|---|
+| `score-get` | `/score-get [player]` | `ScoreGetCommand` | `ScoreKeeperPlugin.getScore(Player)` → `getPlayerScore` | **Read only.** Returns the player's current score. Side effect: lazily registers the player at `0` if unseen. No persistent change. |
+| `score-add` | `/score-add [player] <amount>` | `ScoreAddCommand` | `ScoreKeeperPlugin.addScore(Player,int)` | `score = old + amount` (integer `amount` in `[1..]`). |
+| `score-subtract` | `/score-subtract [player] <amount>` | `ScoreSubtractCommand` | `ScoreKeeperPlugin.subtractScore(Player,int)` | `score = old - amount` (can go negative). |
+| `score-reset` | `/score-reset [player]` | `ScoreResetCommand` | `ScoreKeeperPlugin.resetScore(Player)` | `score = 0` (writes 0; key created if absent). |
+| `score-archive` | `/score-archive [player]` | `ScoreArchiveCommand` | *nothing* | **No state change.** Handler prints `"archive command unimplemented"`. It does **not** call `ScoreKeeperPlugin.archiveScore(...)` — see §5. |
+
+### Command behaviour details
+
+- **Sender vs. target.** Single-arg form means "operate on the invoking player"
+  (`split.length == 1` → `targetPlayer = sender`). Two-arg form means
+  `<playerName> <amount>` (add/subtract) or `<playerName>` (get/reset). For an
+  RCON sender, the single-arg form is treated as missing a target and prints
+  usage instead.
+- **Amount parsing.** `score-add`/`score-subtract` require an integer
+  `amount`; a non-integer prints `"amount must be an integer"`.
+- **Error/usage messaging.** `echoError` (red) and `echoUsage` (colour-coded)
+  differ between player and RCON call sites (`/score-add` vs `score-add`).
+  All handlers `return true` (command handled).
+- **Permissions.** None declared → no operator/permission requirement; any
+  player or RCON can run them.
+
+---
+
+## 4. Scheduled tasks / tick-based logic
+
+**None.** Confirmed by absence of `getScheduler()`, `runTask`,
+`BukkitRunnable`, `BukkitTask`, `scheduleSync*`, or any periodic/repeating
+logic. ScoreKeeper runs no background or tick-driven work.
+
+---
+
+## 5. Inter-plugin dependencies & API exposure
+
+- **Declared dependencies:** plugin.yml has no `depend`, `softdepend`, `load`,
+  or `load-before` → **no declared inter-plugin coupling**.
+- **Build dependency:** the only third-party dependency is
+  `io.papermc.paper:paper-api` (marked `compileOnly`, and explicitly ignored as a
+  snapshot by the release plugin). No other plugin/API is referenced.
+- **Public API surface.** `ScoreKeeperPlugin` is a plain `JavaPlugin` exposing
+  `public` methods that other plugins *could* call if they hold a reference:
+  - `void addScore(Player, int)`
+  - `void subtractScore(Player, int)`
+  - `int getScore(Player)`
+  - `void setScore(Player, int)` — **public but used by no command** (internal
+    write path; the only writer not reachable via a command).
+  - `void resetScore(Player)`
+  - `void archiveScore(Player)` — **public but called by nothing**; it only logs
+    a warning. The `/score-archive` command does **not** invoke it.
+  - `void sendMessage(CommandSender, Component)`, `logInfo/logWarning/logError`
+- **No formal service registration.** There is no `registerService`/`asService`
+  (ServiceLoader), no dedicated API artifact, and no `api:` block in plugin.yml.
+  Exposure is by **public method surface only** — informal and not discoverable
+  by other plugins.
+- **Persistence (unimplemented).** `onEnable` logs a warning that load-from-file
+  is unimplemented; `onDisable` logs that save-to-file is unimplemented. The
+  `HashMap<UUID,Integer>` is the sole state store and is **forgotten on restart**.
+
+---
+
+## 6. Lifecycle
+
+| Hook | What it does |
+|---|---|
+| `onEnable()` | Wires the five `score-*` command executors; logs the "load not implemented" warning and an enable log line. |
+| `onDisable()` | Logs the "save not implemented" warning. **All in-memory scores are lost on shutdown.** |
+
+---
+
+## 7. State model (quick reference)
+
+- **Store:** `private HashMap<UUID,Integer> _playerScores` on `ScoreKeeperPlugin`
+  (one map field for the whole plugin).
+- **Keying:** by `Player.getUniqueId()` (UUID), so scores are per-player and
+  survive name changes *within a single server run*; lost on restart.
+- **Lazy init:** any read (including `get`/`score-get`) auto-creates the key at `0`.
+- **Writers:** `addScore`, `subtractScore`, `resetScore`, `setScore`, all funneled
+  through `setPlayerScore(player, value)`.
+- **Concurrency:** plain `HashMap`; safe today only because command dispatch
+  runs on the server's main thread. Not thread-safe for off-thread use.
+
+---
+
+## 8. "Which server trigger causes which state change" — summary
+
+| Trigger | State change |
+|---|---|
+| `/score-add [player] <amount>` | `_playerScores[uuid] += amount` |
+| `/score-subtract [player] <amount>` | `_playerScores[uuid] -= amount` |
+| `/score-reset [player]` | `_playerScores[uuid] = 0` |
+| `/score-get [player]` | none (reads; lazily registers at 0) |
+| `/score-archive [player]` | none (not implemented) |
+| Any Minecraft/Bukkit event (join/quit/death/etc.) | **none — no listeners exist** |
+| Server tick / scheduled task | **none — no scheduler exists** |
+| Server enable | no data change (registration + log only) |
+| Server disable | in-memory scores discarded (no persistence) |
+| `addScore`/`subtractScore`/`setScore`/`archiveScore` via the public API | same table where called; only reachable by code holding a plugin reference, not by in-game events or commands (except add/subtract/reset which do have commands) |

gradle.properties

@@ -1,2 +1,3 @@
-version=0.2.1
+version=0.2.2-SNAPSHOT
+org.gradle.java.installations.auto-download=true
 

gradle/wrapper/gradle-wrapper.properties

@@ -1,5 +1,7 @@
 distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
-distributionUrl=https\://services.gradle.org/distributions/gradle-8.7-bin.zip
+distributionUrl=https\://services.gradle.org/distributions/gradle-8.14.3-bin.zip
+networkTimeout=10000
+validateDistributionUrl=true
 zipStoreBase=GRADLE_USER_HOME
 zipStorePath=wrapper/dists

gradlew

@@ -1,78 +1,129 @@
-#!/usr/bin/env sh
+#!/bin/sh
+
+#
+# Copyright © 2015-2021 the original authors.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#      https://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+# SPDX-License-Identifier: Apache-2.0
+#
 
 ##############################################################################
-##
-##  Gradle start up script for UN*X
-##
+#
+#   Gradle start up script for POSIX generated by Gradle.
+#
+#   Important for running:
+#
+#   (1) You need a POSIX-compliant shell to run this script. If your /bin/sh is
+#       noncompliant, but you have some other compliant shell such as ksh or
+#       bash, then to run this script, type that shell name before the whole
+#       command line, like:
+#
+#           ksh Gradle
+#
+#       Busybox and similar reduced shells will NOT work, because this script
+#       requires all of these POSIX shell features:
+#         * functions;
+#         * expansions «$var», «${var}», «${var:-default}», «${var+SET}»,
+#           «${var#prefix}», «${var%suffix}», and «$( cmd )»;
+#         * compound commands having a testable exit status, especially «case»;
+#         * various built-in commands including «command», «set», and «ulimit».
+#
+#   Important for patching:
+#
+#   (2) This script targets any POSIX shell, so it avoids extensions provided
+#       by Bash, Ksh, etc; in particular arrays are avoided.
+#
+#       The "traditional" practice of packing multiple parameters into a
+#       space-separated string is a well documented source of bugs and security
+#       problems, so this is (mostly) avoided, by progressively accumulating
+#       options in "$@", and eventually passing that to Java.
+#
+#       Where the inherited environment variables (DEFAULT_JVM_OPTS, JAVA_OPTS,
+#       and GRADLE_OPTS) rely on word-splitting, this is performed explicitly;
+#       see the in-line comments for details.
+#
+#       There are tweaks for specific operating systems such as AIX, CygWin,
+#       Darwin, MinGW, and NonStop.
+#
+#   (3) This script is generated from the Groovy template
+#       https://github.com/gradle/gradle/blob/HEAD/platforms/jvm/plugins-application/src/main/resources/org/gradle/api/internal/plugins/unixStartScript.txt
+#       within the Gradle project.
+#
+#       You can find Gradle at https://github.com/gradle/gradle/.
+#
 ##############################################################################
 
 # Attempt to set APP_HOME
+
 # Resolve links: $0 may be a link
-PRG="$0"
-# Need this for relative symlinks.
-while [ -h "$PRG" ] ; do
-    ls=`ls -ld "$PRG"`
-    link=`expr "$ls" : '.*-> \(.*\)$'`
-    if expr "$link" : '/.*' > /dev/null; then
-        PRG="$link"
-    else
-        PRG=`dirname "$PRG"`"/$link"
-    fi
+app_path=$0
+
+# Need this for daisy-chained symlinks.
+while
+    APP_HOME=${app_path%"${app_path##*/}"}  # leaves a trailing /; empty if no leading path
+    [ -h "$app_path" ]
+do
+    ls=$( ls -ld "$app_path" )
+    link=${ls#*' -> '}
+    case $link in             #(
+      /*)   app_path=$link ;; #(
+      *)    app_path=$APP_HOME$link ;;
+    esac
 done
-SAVED="`pwd`"
-cd "`dirname \"$PRG\"`/" >/dev/null
-APP_HOME="`pwd -P`"
-cd "$SAVED" >/dev/null
 
-APP_NAME="Gradle"
-APP_BASE_NAME=`basename "$0"`
-
-# Add default JVM options here. You can also use JAVA_OPTS and GRADLE_OPTS to pass JVM options to this script.
-DEFAULT_JVM_OPTS=""
+# This is normally unused
+# shellcheck disable=SC2034
+APP_BASE_NAME=${0##*/}
+# Discard cd standard output in case $CDPATH is set (https://github.com/gradle/gradle/issues/25036)
+APP_HOME=$( cd -P "${APP_HOME:-./}" > /dev/null && printf '%s\n' "$PWD" ) || exit
 
 # Use the maximum available, or set MAX_FD != -1 to use that value.
-MAX_FD="maximum"
+MAX_FD=maximum
 
 warn () {
     echo "$*"
-}
+} >&2
 
 die () {
     echo
     echo "$*"
     echo
     exit 1
-}
+} >&2
 
 # OS specific support (must be 'true' or 'false').
 cygwin=false
 msys=false
 darwin=false
 nonstop=false
-case "`uname`" in
-  CYGWIN* )
-    cygwin=true
-    ;;
-  Darwin* )
-    darwin=true
-    ;;
-  MINGW* )
-    msys=true
-    ;;
-  NONSTOP* )
-    nonstop=true
-    ;;
+case "$( uname )" in                #(
+  CYGWIN* )         cygwin=true  ;; #(
+  Darwin* )         darwin=true  ;; #(
+  MSYS* | MINGW* )  msys=true    ;; #(
+  NONSTOP* )        nonstop=true ;;
 esac
 
-CLASSPATH=$APP_HOME/gradle/wrapper/gradle-wrapper.jar
+CLASSPATH="\\\"\\\""
+
 
 # Determine the Java command to use to start the JVM.
 if [ -n "$JAVA_HOME" ] ; then
     if [ -x "$JAVA_HOME/jre/sh/java" ] ; then
         # IBM's JDK on AIX uses strange locations for the executables
-        JAVACMD="$JAVA_HOME/jre/sh/java"
+        JAVACMD=$JAVA_HOME/jre/sh/java
     else
-        JAVACMD="$JAVA_HOME/bin/java"
+        JAVACMD=$JAVA_HOME/bin/java
     fi
     if [ ! -x "$JAVACMD" ] ; then
         die "ERROR: JAVA_HOME is set to an invalid directory: $JAVA_HOME
@@ -81,96 +132,120 @@ Please set the JAVA_HOME variable in your environment to match the
 location of your Java installation."
     fi
 else
-    JAVACMD="java"
-    which java >/dev/null 2>&1 || die "ERROR: JAVA_HOME is not set and no 'java' command could be found in your PATH.
+    JAVACMD=java
+    if ! command -v java >/dev/null 2>&1
+    then
+        die "ERROR: JAVA_HOME is not set and no 'java' command could be found in your PATH.
 
 Please set the JAVA_HOME variable in your environment to match the
 location of your Java installation."
     fi
+fi
 
 # Increase the maximum file descriptors if we can.
-if [ "$cygwin" = "false" -a "$darwin" = "false" -a "$nonstop" = "false" ] ; then
-    MAX_FD_LIMIT=`ulimit -H -n`
-    if [ $? -eq 0 ] ; then
-        if [ "$MAX_FD" = "maximum" -o "$MAX_FD" = "max" ] ; then
-            MAX_FD="$MAX_FD_LIMIT"
-        fi
-        ulimit -n $MAX_FD
-        if [ $? -ne 0 ] ; then
-            warn "Could not set maximum file descriptor limit: $MAX_FD"
-        fi
-    else
-        warn "Could not query maximum file descriptor limit: $MAX_FD_LIMIT"
-    fi
-fi
-
-# For Darwin, add options to specify how the application appears in the dock
-if $darwin; then
-    GRADLE_OPTS="$GRADLE_OPTS \"-Xdock:name=$APP_NAME\" \"-Xdock:icon=$APP_HOME/media/gradle.icns\""
-fi
-
-# For Cygwin, switch paths to Windows format before running java
-if $cygwin ; then
-    APP_HOME=`cygpath --path --mixed "$APP_HOME"`
-    CLASSPATH=`cygpath --path --mixed "$CLASSPATH"`
-    JAVACMD=`cygpath --unix "$JAVACMD"`
-
-    # We build the pattern for arguments to be converted via cygpath
-    ROOTDIRSRAW=`find -L / -maxdepth 1 -mindepth 1 -type d 2>/dev/null`
-    SEP=""
-    for dir in $ROOTDIRSRAW ; do
-        ROOTDIRS="$ROOTDIRS$SEP$dir"
-        SEP="|"
-    done
-    OURCYGPATTERN="(^($ROOTDIRS))"
-    # Add a user-defined pattern to the cygpath arguments
-    if [ "$GRADLE_CYGPATTERN" != "" ] ; then
-        OURCYGPATTERN="$OURCYGPATTERN|($GRADLE_CYGPATTERN)"
-    fi
-    # Now convert the arguments - kludge to limit ourselves to /bin/sh
-    i=0
-    for arg in "$@" ; do
-        CHECK=`echo "$arg"|egrep -c "$OURCYGPATTERN" -`
-        CHECK2=`echo "$arg"|egrep -c "^-"`                                 ### Determine if an option
-
-        if [ $CHECK -ne 0 ] && [ $CHECK2 -eq 0 ] ; then                    ### Added a condition
-            eval `echo args$i`=`cygpath --path --ignore --mixed "$arg"`
-        else
-            eval `echo args$i`="\"$arg\""
-        fi
-        i=$((i+1))
-    done
-    case $i in
-        (0) set -- ;;
-        (1) set -- "$args0" ;;
-        (2) set -- "$args0" "$args1" ;;
-        (3) set -- "$args0" "$args1" "$args2" ;;
-        (4) set -- "$args0" "$args1" "$args2" "$args3" ;;
-        (5) set -- "$args0" "$args1" "$args2" "$args3" "$args4" ;;
-        (6) set -- "$args0" "$args1" "$args2" "$args3" "$args4" "$args5" ;;
-        (7) set -- "$args0" "$args1" "$args2" "$args3" "$args4" "$args5" "$args6" ;;
-        (8) set -- "$args0" "$args1" "$args2" "$args3" "$args4" "$args5" "$args6" "$args7" ;;
-        (9) set -- "$args0" "$args1" "$args2" "$args3" "$args4" "$args5" "$args6" "$args7" "$args8" ;;
+if ! "$cygwin" && ! "$darwin" && ! "$nonstop" ; then
+    case $MAX_FD in #(
+      max*)
+        # In POSIX sh, ulimit -H is undefined. That's why the result is checked to see if it worked.
+        # shellcheck disable=SC2039,SC3045
+        MAX_FD=$( ulimit -H -n ) ||
+            warn "Could not query maximum file descriptor limit"
+    esac
+    case $MAX_FD in  #(
+      '' | soft) :;; #(
+      *)
+        # In POSIX sh, ulimit -n is undefined. That's why the result is checked to see if it worked.
+        # shellcheck disable=SC2039,SC3045
+        ulimit -n "$MAX_FD" ||
+            warn "Could not set maximum file descriptor limit to $MAX_FD"
     esac
 fi
 
-# Escape application args
-save () {
-    for i do printf %s\\n "$i" | sed "s/'/'\\\\''/g;1s/^/'/;\$s/\$/' \\\\/" ; done
-    echo " "
-}
-APP_ARGS=$(save "$@")
+# Collect all arguments for the java command, stacking in reverse order:
+#   * args from the command line
+#   * the main class name
+#   * -classpath
+#   * -D...appname settings
+#   * --module-path (only if needed)
+#   * DEFAULT_JVM_OPTS, JAVA_OPTS, and GRADLE_OPTS environment variables.
 
-# Collect all arguments for the java command, following the shell quoting and substitution rules
-if $JAVACMD --add-opens java.base/java.lang=ALL-UNNAMED -version ; then
-    DEFAULT_JVM_OPTS="--add-opens java.base/java.lang=ALL-UNNAMED $DEFAULT_JVM_OPTS"
+# For Cygwin or MSYS, switch paths to Windows format before running java
+if "$cygwin" || "$msys" ; then
+    APP_HOME=$( cygpath --path --mixed "$APP_HOME" )
+    CLASSPATH=$( cygpath --path --mixed "$CLASSPATH" )
+
+    JAVACMD=$( cygpath --unix "$JAVACMD" )
+
+    # Now convert the arguments - kludge to limit ourselves to /bin/sh
+    for arg do
+        if
+            case $arg in                                #(
+              -*)   false ;;                            # don't mess with options #(
+              /?*)  t=${arg#/} t=/${t%%/*}              # looks like a POSIX filepath
+                    [ -e "$t" ] ;;                      #(
+              *)    false ;;
+            esac
+        then
+            arg=$( cygpath --path --ignore --mixed "$arg" )
+        fi
+        # Roll the args list around exactly as many times as the number of
+        # args, so each arg winds up back in the position where it started, but
+        # possibly modified.
+        #
+        # NB: a `for` loop captures its iteration list before it begins, so
+        # changing the positional parameters here affects neither the number of
+        # iterations, nor the values presented in `arg`.
+        shift                   # remove old arg
+        set -- "$@" "$arg"      # push replacement arg
+    done
 fi
 
-eval set -- $DEFAULT_JVM_OPTS $JAVA_OPTS $GRADLE_OPTS "\"-Dorg.gradle.appname=$APP_BASE_NAME\"" -classpath "\"$CLASSPATH\"" org.gradle.wrapper.GradleWrapperMain "$APP_ARGS"
 
-# by default we should be in the correct project dir, but when run from Finder on Mac, the cwd is wrong
-if [ "$(uname)" = "Darwin" ] && [ "$HOME" = "$PWD" ]; then
-  cd "$(dirname "$0")"
+# Add default JVM options here. You can also use JAVA_OPTS and GRADLE_OPTS to pass JVM options to this script.
+DEFAULT_JVM_OPTS='"-Xmx64m" "-Xms64m"'
+
+# Collect all arguments for the java command:
+#   * DEFAULT_JVM_OPTS, JAVA_OPTS, and optsEnvironmentVar are not allowed to contain shell fragments,
+#     and any embedded shellness will be escaped.
+#   * For example: A user cannot expect ${Hostname} to be expanded, as it is an environment variable and will be
+#     treated as '${Hostname}' itself on the command line.
+
+set -- \
+        "-Dorg.gradle.appname=$APP_BASE_NAME" \
+        -classpath "$CLASSPATH" \
+        -jar "$APP_HOME/gradle/wrapper/gradle-wrapper.jar" \
+        "$@"
+
+# Stop when "xargs" is not available.
+if ! command -v xargs >/dev/null 2>&1
+then
+    die "xargs is not available"
 fi
 
+# Use "xargs" to parse quoted args.
+#
+# With -n1 it outputs one arg per line, with the quotes and backslashes removed.
+#
+# In Bash we could simply go:
+#
+#   readarray ARGS < <( xargs -n1 <<<"$var" ) &&
+#   set -- "${ARGS[@]}" "$@"
+#
+# but POSIX shell has neither arrays nor command substitution, so instead we
+# post-process each arg (as a line of input to sed) to backslash-escape any
+# character that might be a shell metacharacter, then use eval to reverse
+# that process (while maintaining the separation between arguments), and wrap
+# the whole thing up as a single "set" statement.
+#
+# This will of course break if any of these variables contains a newline or
+# an unmatched quote.
+#
+
+eval "set -- $(
+        printf '%s\n' "$DEFAULT_JVM_OPTS $JAVA_OPTS $GRADLE_OPTS" |
+        xargs -n1 |
+        sed ' s~[^-[:alnum:]+,./:=@_]~\\&~g; ' |
+        tr '\n' ' '
+    )" '"$@"'
+
 exec "$JAVACMD" "$@"

gradlew.bat

@@ -1,3 +1,21 @@
+@rem
+@rem Copyright 2015 the original author or authors.
+@rem
+@rem Licensed under the Apache License, Version 2.0 (the "License");
+@rem you may not use this file except in compliance with the License.
+@rem You may obtain a copy of the License at
+@rem
+@rem      https://www.apache.org/licenses/LICENSE-2.0
+@rem
+@rem Unless required by applicable law or agreed to in writing, software
+@rem distributed under the License is distributed on an "AS IS" BASIS,
+@rem WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+@rem See the License for the specific language governing permissions and
+@rem limitations under the License.
+@rem
+@rem SPDX-License-Identifier: Apache-2.0
+@rem
+
 @if "%DEBUG%"=="" @echo off
 @rem ##########################################################################
 @rem
@@ -10,24 +28,28 @@ if "%OS%"=="Windows_NT" setlocal
 
 set DIRNAME=%~dp0
 if "%DIRNAME%"=="" set DIRNAME=.
+@rem This is normally unused
 set APP_BASE_NAME=%~n0
 set APP_HOME=%DIRNAME%
 
+@rem Resolve any "." and ".." in APP_HOME to make it shorter.
+for %%i in ("%APP_HOME%") do set APP_HOME=%%~fi
+
 @rem Add default JVM options here. You can also use JAVA_OPTS and GRADLE_OPTS to pass JVM options to this script.
-set DEFAULT_JVM_OPTS=
+set DEFAULT_JVM_OPTS="-Xmx64m" "-Xms64m"
 
 @rem Find java.exe
 if defined JAVA_HOME goto findJavaFromJavaHome
 
 set JAVA_EXE=java.exe
 %JAVA_EXE% -version >NUL 2>&1
-if "%ERRORLEVEL%" == "0" goto init
+if %ERRORLEVEL% equ 0 goto execute
 
-echo.
-echo ERROR: JAVA_HOME is not set and no 'java' command could be found in your PATH.
-echo.
-echo Please set the JAVA_HOME variable in your environment to match the
-echo location of your Java installation.
+echo. 1>&2
+echo ERROR: JAVA_HOME is not set and no 'java' command could be found in your PATH. 1>&2
+echo. 1>&2
+echo Please set the JAVA_HOME variable in your environment to match the 1>&2
+echo location of your Java installation. 1>&2
 
 goto fail
 
@@ -35,48 +57,36 @@ goto fail
 set JAVA_HOME=%JAVA_HOME:"=%
 set JAVA_EXE=%JAVA_HOME%/bin/java.exe
 
-if exist "%JAVA_EXE%" goto init
+if exist "%JAVA_EXE%" goto execute
 
-echo.
-echo ERROR: JAVA_HOME is set to an invalid directory: %JAVA_HOME%
-echo.
-echo Please set the JAVA_HOME variable in your environment to match the
-echo location of your Java installation.
+echo. 1>&2
+echo ERROR: JAVA_HOME is set to an invalid directory: %JAVA_HOME% 1>&2
+echo. 1>&2
+echo Please set the JAVA_HOME variable in your environment to match the 1>&2
+echo location of your Java installation. 1>&2
 
 goto fail
 
-:init
-@rem Get command-line arguments, handling Windows variants
-
-if not "%OS%" == "Windows_NT" goto win9xME_args
-
-:win9xME_args
-@rem Slurp the command line arguments.
-set CMD_LINE_ARGS=
-set _SKIP=2
-
-:win9xME_args_slurp
-if "x%~1" == "x" goto execute
-
-set CMD_LINE_ARGS=%*
-
 :execute
 @rem Setup the command line
 
-set CLASSPATH=%APP_HOME%\gradle\wrapper\gradle-wrapper.jar
+set CLASSPATH=
+
 
 @rem Execute Gradle
-"%JAVA_EXE%" %DEFAULT_JVM_OPTS% %JAVA_OPTS% %GRADLE_OPTS% "-Dorg.gradle.appname=%APP_BASE_NAME%" -classpath "%CLASSPATH%" org.gradle.wrapper.GradleWrapperMain %CMD_LINE_ARGS%
+"%JAVA_EXE%" %DEFAULT_JVM_OPTS% %JAVA_OPTS% %GRADLE_OPTS% "-Dorg.gradle.appname=%APP_BASE_NAME%" -classpath "%CLASSPATH%" -jar "%APP_HOME%\gradle\wrapper\gradle-wrapper.jar" %*
 
 :end
 @rem End local scope for the variables with windows NT shell
-if "%ERRORLEVEL%"=="0" goto mainEnd
+if %ERRORLEVEL% equ 0 goto mainEnd
 
 :fail
 rem Set variable GRADLE_EXIT_CONSOLE if you need the _script_ return code instead of
 rem the _cmd.exe /c_ return code!
-if  not "" == "%GRADLE_EXIT_CONSOLE%" exit 1
-exit /b 1
+set EXIT_CODE=%ERRORLEVEL%
+if %EXIT_CODE% equ 0 set EXIT_CODE=1
+if not ""=="%GRADLE_EXIT_CONSOLE%" exit %EXIT_CODE%
+exit /b %EXIT_CODE%
 
 :mainEnd
 if "%OS%"=="Windows_NT" endlocal

settings.gradle

@@ -1 +1,5 @@
+plugins {
+	id 'org.gradle.toolchains.foojay-resolver-convention' version '0.9.0'
+}
+
 rootProject.name = 'ScoreKeeper'

tools/powershell/Build-Plugin.ps1

@@ -0,0 +1,20 @@
+#!/usr/bin/env pwsh
+
+# Set strict mode for better error handling
+Set-StrictMode -Version Latest
+$ErrorActionPreference = "Stop"
+
+# Find project root (two directories above this script)
+$ScriptDir = Split-Path -Parent $MyInvocation.MyCommand.Path
+$ProjectRoot = Split-Path -Parent (Split-Path -Parent $ScriptDir)
+
+# Save current location and change to project root
+Push-Location $ProjectRoot
+
+try {
+    # Build the project using Gradle
+    & ".\gradlew.bat" build
+} finally {
+    # Always restore the original location
+    Pop-Location
+}

tools/powershell/Deploy-Plugin.ps1

@@ -0,0 +1,49 @@
+#!/usr/bin/env pwsh
+
+# Set strict mode for better error handling
+Set-StrictMode -Version Latest
+$ErrorActionPreference = "Stop"
+
+# Check required environment variable
+if (-not $env:MINECRAFT_SERVER_PATH) {
+    Write-Error "Error: MINECRAFT_SERVER_PATH environment variable is not set."
+    exit 1
+}
+
+# Find project root (two directories above this script)
+$ScriptDir = Split-Path -Parent $MyInvocation.MyCommand.Path
+$ProjectRoot = Split-Path -Parent (Split-Path -Parent $ScriptDir)
+
+# Find the most recent plugin jar file
+$LibsPath = Join-Path $ProjectRoot "build\libs"
+$PluginJar = Get-ChildItem -Path "$LibsPath\*.jar" -ErrorAction SilentlyContinue | 
+    Sort-Object LastWriteTime -Descending | 
+    Select-Object -First 1
+
+if (-not $PluginJar) {
+    Write-Error "Error: No plugin jar found in $LibsPath. Build the plugin first."
+    exit 1
+}
+
+# Optional: Warn if jar is older than any source file
+$SourcePaths = @(
+    (Join-Path $ProjectRoot "src\main\java"),
+    (Join-Path $ProjectRoot "src\main\resources")
+)
+
+$NewerSourceFiles = Get-ChildItem -Path $SourcePaths -Recurse -File -ErrorAction SilentlyContinue | 
+    Where-Object { $_.LastWriteTime -gt $PluginJar.LastWriteTime }
+
+if ($NewerSourceFiles) {
+    Write-Host "Warning: The built plugin jar is older than some source files. Consider rebuilding." -ForegroundColor Yellow
+}
+
+# Create plugins directory if it doesn't exist
+$PluginsPath = Join-Path $env:MINECRAFT_SERVER_PATH "plugins"
+if (-not (Test-Path $PluginsPath)) {
+    New-Item -ItemType Directory -Path $PluginsPath -Force | Out-Null
+}
+
+# Copy the plugin jar to the plugins directory
+Copy-Item -Path $PluginJar.FullName -Destination $PluginsPath -Force
+Write-Host "Deployed $($PluginJar.Name) to $PluginsPath\"

tools/powershell/Start-Server.ps1

@@ -0,0 +1,34 @@
+#!/usr/bin/env pwsh
+
+# Set strict mode for better error handling
+Set-StrictMode -Version Latest
+$ErrorActionPreference = "Stop"
+
+# Check required environment variables
+if (-not $env:MINECRAFT_SERVER_PATH -or -not $env:MINECRAFT_SERVER_JAR) {
+    Write-Error "Error: MINECRAFT_SERVER_PATH and MINECRAFT_SERVER_JAR must be set."
+    exit 1
+}
+
+Push-Location $env:MINECRAFT_SERVER_PATH
+
+# Find running server process (java with the server jar)
+$ServerProcess = Get-Process -Name "java" -ErrorAction SilentlyContinue | Where-Object { 
+    $_.CommandLine -like "*$($env:MINECRAFT_SERVER_JAR)*" 
+}
+
+if ($ServerProcess) {
+    Write-Host "Minecraft server is running (PID: $($ServerProcess.Id)). Stopping it..."
+    Stop-Process -Id $ServerProcess.Id -Force
+    Start-Sleep -Seconds 5
+} else {
+    Write-Host "Minecraft server is not running."
+}
+
+# Start the server in a new window
+try {
+    $ServerJob = Start-Process -FilePath "java" -ArgumentList "-jar", $env:MINECRAFT_SERVER_JAR -PassThru
+    Write-Host "Minecraft server started in new window with PID: $($ServerJob.Id)"
+} finally {
+    Pop-Location
+}

tools/powershell/Stop-Server.ps1

@@ -0,0 +1,25 @@
+#!/usr/bin/env pwsh
+
+# Set strict mode for better error handling
+Set-StrictMode -Version Latest
+$ErrorActionPreference = "Stop"
+
+# Check required environment variables
+if (-not $env:MINECRAFT_SERVER_PATH -or -not $env:MINECRAFT_SERVER_JAR) {
+    Write-Error "Error: MINECRAFT_SERVER_PATH and MINECRAFT_SERVER_JAR must be set."
+    exit 1
+}
+
+# Find running server process (java with the server jar)
+$ServerProcess = Get-Process -Name "java" -ErrorAction SilentlyContinue | Where-Object { 
+    $_.CommandLine -like "*$($env:MINECRAFT_SERVER_JAR)*" 
+}
+
+if ($ServerProcess) {
+    Write-Host "Minecraft server is running (PID: $($ServerProcess.Id)). Stopping it..."
+    Stop-Process -Id $ServerProcess.Id -Force
+    Start-Sleep -Seconds 5
+    Write-Host "Minecraft server stopped."
+} else {
+    Write-Host "Minecraft server is not running."
+}