Skip to content

(Draft) Add Pumpkin#1832

Open
clovertortoise wants to merge 7 commits intoCubeCoders:mainfrom
clovertortoise:pumpkin-template
Open

(Draft) Add Pumpkin#1832
clovertortoise wants to merge 7 commits intoCubeCoders:mainfrom
clovertortoise:pumpkin-template

Conversation

@clovertortoise
Copy link

First timer here.

I put together a template for Pumpkin, which is a highly performant Rust implementation of a Minecraft server.

To get the config parsing right, I referenced how beammp handles TOML files and grabbed a few other tricks from existing templates to hack this together into a working setup.

It successfully deploys and runs on Linux / ARM64.

Notes:

  • The Pumpkin server software itself is still in heavy active development, so some features are still a work in progress.
  • I haven't had a chance to test on Windows yet.

Copy link
Collaborator

@Greelan Greelan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice first attempt! It's looking very solid. I've suggested a number of tidies/improvements on a first pass

@clovertortoise
Copy link
Author

Thank you for the thorough review! I've gone ahead and implemented each change. Category structure been modified to closely match other MC templates and URLs are updated to point to the CC main branch.

@clovertortoise clovertortoise marked this pull request as ready for review February 25, 2026 19:52
Copilot AI review requested due to automatic review settings February 25, 2026 19:52
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request adds a new template for Pumpkin, a high-performance Rust-based Minecraft server implementation. The contributor has created a comprehensive template that supports both Java and Bedrock Edition clients, handles TOML configuration files, and includes platform-specific binary downloads for Linux (x86_64 and ARM64) and Windows.

Changes:

  • Added complete Pumpkin server template with extensive configuration options covering server settings, gameplay, networking, security, logging, and resource packs
  • Implemented TOML configuration parsing using patterns inspired by the beammp template
  • Created platform-specific update stages for downloading and setting up binaries

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 8 comments.

Show a summary per file
File Description
pumpkin.kvp Main template configuration defining server metadata, executable paths, console regex patterns, and AMP integration settings
pumpkinconfig.json Comprehensive configuration schema with 100+ settings organized into 9 subcategories for managing all aspects of the Pumpkin server
pumpkinmetaconfig.json TOML parsing configuration using dual regex patterns to handle both quoted strings and unquoted values (booleans, numbers, arrays)
pumpkinupdates.json Update stages for downloading platform-specific binaries and default configuration files from the CubeCoders repository
pumpkinConfig.toml Default main configuration file with core server settings
pumpkinFeatures.toml Default features configuration file with advanced settings organized in TOML sections
pumpkinports.json Port definitions for Java Edition (25565), Bedrock Edition (19132), and RCON (25575)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Greelan
Copy link
Collaborator

Greelan commented Mar 3, 2026

@clovertortoise is this finished yet? Has it been tested on Windows?

I note you haven't accepted AI's suggestion re double quotes for array values. Shouldn't at least Features.toml be made consistent?

@clovertortoise
Copy link
Author

@clovertortoise is this finished yet? Has it been tested on Windows?

I note you haven't accepted AI's suggestion re double quotes for array values. Shouldn't at least Features.toml be made consistent?

@Greelan Yes, this is ready to go. I’ve implemented the changes and verified that everything works correctly on both Linux and Windows.

Features.toml has also been updated to ensure consistency.

@clovertortoise clovertortoise requested a review from Greelan March 3, 2026 23:49
@Greelan
Copy link
Collaborator

Greelan commented Mar 4, 2026

No worries, I will give it a proper going-over.

Would it make sense to call the Server Port instead Java Port? To align with/distinguish from Bedrock Port?

@clovertortoise
Copy link
Author

No worries, I will give it a proper going-over.

Would it make sense to call the Server Port instead Java Port? To align with/distinguish from Bedrock Port?

Much appreciated! Good catch on the port alignment, I’ll update that now.

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.

3 participants