Conversation
Co-authored-by: dmccoystephenson <21204351+dmccoystephenson@users.noreply.github.com> Agent-Logs-Url: https://github.com/Dans-Plugins/AlternateAccountFinder/sessions/29b4f607-dfc0-4056-ac3e-29b977552b3a
There was a problem hiding this comment.
Pull request overview
Aligns the repository with DPC conventions by standardizing project documentation and adding baseline CI/release automation and Copilot guidance.
Changes:
- Restructures
README.mdand adds new convention-aligned docs (USER_GUIDE.md,COMMANDS.md,CONFIG.md,CHANGELOG.md,CONTRIBUTING.md). - Adds GitHub Actions workflows for build and release publishing.
- Adds repository-specific Copilot instructions.
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
| README.md | Reordered/expanded README sections; adds links to in-repo docs; adds testing/dev/release pointers |
| CONTRIBUTING.md | Adds contribution guide based on DPC template |
| USER_GUIDE.md | Adds getting-started guide and common command scenarios |
| COMMANDS.md | Adds command reference for /aaf subcommands |
| CONFIG.md | Documents config.yml keys with types/defaults/examples |
| CHANGELOG.md | Seeds Keep a Changelog formatted changelog |
| .github/workflows/build.yml | Adds CI build on push/PR to main/develop |
| .github/workflows/release.yml | Adds release build + attachment upload workflow |
| .github/copilot-instructions.md | Adds Copilot guidance on stack, layout, and workflow |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
.github/copilot-instructions.md
Outdated
| - `logins/` – Business logic and repository for login/IP tracking | ||
| - `notifications/` – Notification service integrations (Mailboxes, RPKit, message) | ||
| - `src/main/resources/` – `plugin.yml`, `config.yml`, and database migration scripts | ||
| - `src/test/java/` – Unit tests |
There was a problem hiding this comment.
This lists src/test/java/ as the location for unit tests, but that directory doesn't exist in the current repo. Consider removing this line or adding the test directory (with at least a starter test) so the instructions match reality.
| - `src/test/java/` – Unit tests | |
| - `src/test/java/` – (intended) unit test sources; create this directory when adding tests |
.github/workflows/release.yml
Outdated
| - name: Upload JAR to release | ||
| uses: softprops/action-gh-release@v2 | ||
| with: | ||
| files: build/libs/*.jar |
There was a problem hiding this comment.
files: build/libs/*.jar will upload every JAR produced by the build (likely both the plain JAR and the shaded *-all.jar). If only the runnable shaded artifact should be released, narrow this glob (e.g., to build/libs/*-all.jar) or explicitly select the intended artifact to avoid confusing releases.
| files: build/libs/*.jar | |
| files: build/libs/*-all.jar |
CONTRIBUTING.md
Outdated
| Run the unit tests with: | ||
|
|
||
| Linux: `./gradlew clean test` | ||
| Windows: `.\gradlew.bat clean test` |
There was a problem hiding this comment.
This guide instructs contributors to run "unit tests", but the repository currently has no test sources (src/test/) and no JUnit (or other test framework) dependency configured in build.gradle. Either add a real test setup or reword this section to reflect the current state (e.g., ./gradlew clean build as a verification build).
| Run the unit tests with: | |
| Linux: `./gradlew clean test` | |
| Windows: `.\gradlew.bat clean test` | |
| Run a verification build with: | |
| Linux: `./gradlew clean build` | |
| Windows: `.\gradlew.bat clean build` |
| ## Testing | ||
|
|
||
| ### Unit Tests | ||
|
|
||
| Linux: | ||
|
|
||
| ``` | ||
| ./gradlew clean test | ||
| ``` | ||
|
|
||
| Windows: | ||
|
|
||
| ``` | ||
| .\gradlew.bat clean test | ||
| ``` | ||
|
|
||
| If you see `BUILD SUCCESSFUL`, the tests have passed. | ||
|
|
There was a problem hiding this comment.
The README documents "Unit Tests" and instructs running ./gradlew clean test, but the repo currently has no src/test/ directory and build.gradle has no JUnit (or other test) dependencies configured. Either add/enable an actual test setup (e.g., add JUnit 5 dependencies + test sources) or adjust this section to avoid implying unit tests exist.
README.md
Outdated
| ### Test Server with Plugin Hot-Reloading | ||
|
|
||
| A Docker-based test server is available for development. | ||
|
|
||
| #### Setup |
There was a problem hiding this comment.
This section claims a "Test Server with Plugin Hot-Reloading", but the current compose.yml/Dockerfile only build the image (including building/copying the plugin once) and don't mount the built JAR into the container for live reload. Either remove the hot-reloading claim or update the Docker setup to actually support it (e.g., volume-mount build/libs/*-all.jar into plugins/).
README.md
Outdated
| - [Planned Improvements](https://github.com/dmccoystephenson/AlternateAccountFinder/issues?q=is%3Aopen+is%3Aissue+label%3Aenhancement) | ||
| Please fill out a bug report [here](https://github.com/Dans-Plugins/AlternateAccountFinder/issues/new). | ||
|
|
||
| - [Known Bugs](https://github.com/Dans-Plugins/AlternateAccountFinder/issues?q=is%3Aopen+is%3Aissue+label%3Abug) |
There was a problem hiding this comment.
The "Known Bugs" link is listed under "Support" here, but it’s also listed again later under the "Roadmap" section. Consider keeping it in only one place (or replacing this with a short note that the issue tracker is used for bugs) to avoid duplicated/out-of-sync links.
| - [Known Bugs](https://github.com/Dans-Plugins/AlternateAccountFinder/issues?q=is%3Aopen+is%3Aissue+label%3Abug) |
README.md
Outdated
| ``` | ||
|
|
||
| This requires [Docker](https://docs.docker.com/get-docker/) and [Docker Compose](https://docs.docker.com/compose/install/) to be installed. | ||
| ## Authors and Acknowledgement |
There was a problem hiding this comment.
Heading uses "Acknowledgement" (singular). For grammar/consistency with common OSS docs, this should typically be "Acknowledgements".
| ## Authors and Acknowledgement | |
| ## Authors and Acknowledgements |
.github/copilot-instructions.md
Outdated
| - Language: Java | ||
| - Build tool: Gradle (Groovy DSL) with Shadow plugin for fat JARs | ||
| - Target platform: Spigot / Paper (API version 1.17+) | ||
| - Test framework: JUnit 5 |
There was a problem hiding this comment.
The instructions state the test framework is JUnit 5, but build.gradle currently has no JUnit (or other test framework) dependency configured. Either add the test dependencies/configuration or remove this to avoid misleading guidance.
| - Test framework: JUnit 5 | |
| - Test framework: Not yet configured (prefer JUnit 5; add dependencies in build.gradle before writing tests) |
|
@copilot address comments |
Co-authored-by: dmccoystephenson <21204351+dmccoystephenson@users.noreply.github.com> Agent-Logs-Url: https://github.com/Dans-Plugins/AlternateAccountFinder/sessions/c11eabe6-ccb7-44e7-b3e6-1293455f1276
Done in commit
|
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| | Permission | Default | Description | | ||
| |--------------|---------|--------------------------------------------------| | ||
| | `aaf.accounts` | op | Allows viewing all accounts for an IP address | | ||
| | `aaf.ips` | op | Allows viewing all IPs for a player | | ||
| | `aaf.alts` | op | Allows viewing suspected alt accounts for a player | |
There was a problem hiding this comment.
The permissions table markdown is malformed (rows start with ||), which will render as an extra empty column (or not render as a table in some viewers). Use a single leading | per row (and consistent trailing |) so the table renders correctly.
|
|
||
| ### Language Files | ||
|
|
||
| Update `src/main/resources/lang/` for any user-facing string changes. |
There was a problem hiding this comment.
This section references src/main/resources/lang/ for user-facing strings, but that directory doesn't exist in the repo right now. Either update this guidance to match the current codebase (e.g., point at the current message/notification classes), or add the lang/ directory and wire it up before documenting it here.
| Update `src/main/resources/lang/` for any user-facing string changes. | |
| Update the appropriate message/notification classes or resource files for any user-facing string changes, keeping them centralized for easier localization. |
Brings the repo into compliance with the DPC conventions by adding all required documentation files, CI/CD workflows, and Copilot instructions.
Documentation
README.md– Restructured to the required section order; Usage now links to in-repo docs; added Testing, Development, and Changelog sectionsCONTRIBUTING.md– Created from DPC template (Thank You → Links → Requirements → Getting Started → Issues/Milestones → Making Changes → Testing → Questions)USER_GUIDE.md– Prerequisites, first steps, common scenarios for all three sub-commands, permissions tableCOMMANDS.md– Full reference for/aaf accounts,/aaf ips,/aaf altsCONFIG.md– Everyconfig.ymlkey documented with type, default, description, and YAML exampleCHANGELOG.md– Keep a Changelog format; seeds a[2.0.0]entry and an[Unreleased]sectionCI/CD
.github/workflows/build.yml– Triggers on push/PR tomainanddevelop; JDK 17 +./gradlew clean build; scoped topermissions: contents: read.github/workflows/release.yml– Triggers onrelease: created; builds and attachesbuild/libs/*.jarto the release viasoftprops/action-gh-release@v2Copilot
.github/copilot-instructions.md– Technology stack, source layout, coding conventions, and DPC contribution workflowOriginal prompt
⚡ Quickly spin up Copilot coding agent tasks from anywhere on your macOS or Windows machine with Raycast.