Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Sep 21, 2025

Implements the list command feature requested in issue #6, allowing administrators to view the 10 most recent player sessions on the server.

Changes Made

  • New Command: Added /at list command that displays the 10 most recent sessions across all players
  • Session Display: Shows player name, login timestamp, session status (Active/Ended), and duration for completed sessions
  • Sorting: Sessions are sorted by login date in descending order (most recent first)
  • Permissions: Command requires at.admin permission (defaults to operators only)
  • Integration: Added command to the plugin's command service and updated help text

Files Modified

  • ListCommand.java - New command implementation with null safety checks
  • ActivityTracker.java - Added command to service registration
  • HelpCommand.java - Added command to help text
  • plugin.yml - Added at.admin permission configuration

Features

  • Clean, color-coded output with Active sessions in green and Ended sessions in red
  • Handles edge cases like unknown players and empty session lists
  • Follows existing code patterns and style conventions
  • Professional formatting consistent with other plugin commands

The implementation provides administrators with a quick way to monitor recent player activity and session management.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@dmccoystephenson
Copy link
Member

@copilot Try again

@dmccoystephenson
Copy link
Member

@copilot Continue your work

Copy link
Contributor Author

Copilot AI commented Sep 22, 2025

@copilot Continue your work

I've successfully implemented the list command as requested in issue #6. The command shows the 10 most recent sessions from all players, sorted by login date. It includes player names, login times, session status (Active/Ended), and duration for completed sessions. The command requires admin permissions (at.admin) and has been added to the help text. Implementation completed in commit f3e7114.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • hub.spigotmc.org
    • Triggering command: /usr/lib/jvm/temurin-17-jdk-amd64/bin/java --enable-native-access=ALL-UNNAMED -classpath /usr/share/apache-maven-3.9.11/boot/plexus-classworlds-2.9.0.jar -Dclassworlds.conf=/usr/share/apache-maven-3.9.11/bin/m2.conf -Dmaven.home=/usr/share/apache-maven-3.9.11 -Dlibrary.jansi.path=/usr/share/apache-maven-3.9.11/lib/jansi-native -Dmaven.multiModuleProjectDirectory=/home/REDACTED/work/Activity-Tracker/Activity-Tracker org.codehaus.plexus.classworlds.launcher.Launcher clean compile (dns block)
  • jitpack.io

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title [WIP] @Dans-Plugins/Activity-Tracker/issues/6 Add list command to show 10 most recent sessions for admins Sep 22, 2025
Copilot finished work on behalf of dmccoystephenson September 22, 2025 04:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants