C++ API by mtmatt · Pull Request #203 · CSIE-Challenge/Challenge2025

mtmatt · 2025-07-25T09:00:06Z

新增給 C++ 使用的 API。

…tournament

…t/final-tournament

littlecube8152 · 2025-07-25T10:34:27Z

 		result = game_self.score
 	else:
-		result = game_other.display_score
+		result = game_other.score


This is intentional. Please change it back.
If the behavior mismatches then something else is wrong.

littlecube8152 · 2025-07-25T10:37:31Z

+	# TODO: re-enable this after the tournament
 	if (
-		text.contains("start-process powershell --verb runAs")
+		false and text.contains("start-process powershell --verb runAs")
 		or text.contains("start-process powershell –verb runAs")
 	):
 		get_tree().change_scene_to_file("res://scenes/maps/water/storyline-triggered.tscn")


There are some workarounds and potential hacks or tweaks left by the final tournament that might be left unchecked. I suggest merging this PR after all the dust has settled.

littlecube8152 · 2025-07-25T10:46:48Z

+# Install libraries and headers
+install: $(STATIC_LIB) $(SHARED_LIB)
+	sudo mkdir -p /usr/local/lib
+	sudo mkdir -p /usr/local/include/game_api
+	sudo cp $(STATIC_LIB) $(SHARED_LIB) /usr/local/lib/
+	sudo cp $(HEADERS) /usr/local/include/game_api/
+	sudo ldconfig
+
+# Uninstall
+uninstall:
+	sudo rm -f /usr/local/lib/$(STATIC_LIB)
+	sudo rm -f /usr/local/lib/$(SHARED_LIB)
+	sudo rm -rf /usr/local/include/game_api
+	sudo ldconfig


Makefiles should not execute sudo. Ask the user to do that instead (sudo make install).

Actually, now I think about it, we should make it a header-only library so there will be no hassle compiling, installing, and building. And that is easier to use and maintain.

littlecube8152 · 2025-07-25T10:51:56Z

+        std::cout << "\n--- Test Summary ---" << std::endl;
+        std::cout << "✓ All major API categories tested!" << std::endl;
+        std::cout << "✓ Connection and authentication working" << std::endl;
+        std::cout << "✓ Game state information retrieval working" << std::endl;
+        std::cout << "✓ Enemy parsing fix successful" << std::endl;
+        std::cout << "✓ Tower management APIs functional" << std::endl;
+        std::cout << "✓ Unit and spell systems accessible" << std::endl;


How is this summary useful? It did not take into account of all previous testing results.
By the way, I do not think using Unicode characters in C++ examples will be great.

littlecube8152 · 2025-07-25T10:58:36Z

+    std::string toString() const {
+        return "(" + std::to_string(x) + ", " + std::to_string(y) + ")";
+    }


The Python equivalent __str__ is actually not used anywhere and not that handy.
I would rather overload those for basic_ostream operator<< or remove them.

littlecube8152 · 2025-07-25T11:02:16Z

+class GodotNull : public GodotVariant {
+public:
+    GodotNull() {}
+
+    std::vector<uint8_t> serialize() const override {
+        std::vector<uint8_t> result;
+        GodotSerializer::push_int32(result, static_cast<int32_t>(TypeCode::NULL_TYPE));
+        return result;
+    }
+
+    TypeCode getType() const override { return TypeCode::NULL_TYPE; }
+};
+
+class GodotBool : public GodotVariant {
+public:
+    GodotBool(bool v) : value(v) {}
+
+    std::vector<uint8_t> serialize() const override {
+        std::vector<uint8_t> result;
+        GodotSerializer::push_int32(result, static_cast<int32_t>(TypeCode::BOOL_TYPE));
+        GodotSerializer::push_int32(result, value ? 1 : 0);
+        return result;
+    }
+
+    TypeCode getType() const override { return TypeCode::BOOL_TYPE; }
+    bool getValue() const { return value; }
+
+private:
+    bool value;
+};
+
+class GodotInt : public GodotVariant {
+public:
+    GodotInt(int64_t v) : value(v) {}
+    GodotInt(int v) : value(static_cast<int64_t>(v)) {}
+
+    std::vector<uint8_t> serialize() const override {
+        std::vector<uint8_t> result;
+
+        uint32_t header = static_cast<uint32_t>(TypeCode::INT_TYPE);
+
+        // Check if value fits in 32-bit range
+        if (value > INT32_MAX || value < INT32_MIN) {
+            // 64-bit integer serialization
+            header |= HEADER_DATA_FLAG_64;
+            GodotSerializer::push_int32(result, static_cast<int32_t>(header));
+            GodotSerializer::push_int64(result, value);
+        } else {
+            // 32-bit integer serialization
+            GodotSerializer::push_int32(result, static_cast<int32_t>(header));
+            GodotSerializer::push_int32(result, static_cast<int32_t>(value));
+        }
+
+        return result;
+    }
+
+    TypeCode getType() const override { return TypeCode::INT_TYPE; }
+    int64_t getValue() const { return value; }
+
+private:
+    int64_t value;
+};
+
+class GodotFloat : public GodotVariant {
+public:
+    GodotFloat(double v) : value(v) {}
+    GodotFloat(float v) : value(static_cast<double>(v)) {}
+
+    std::vector<uint8_t> serialize() const override {
+        std::vector<uint8_t> result;
+
+        uint32_t header = static_cast<uint32_t>(TypeCode::FLOAT_TYPE);
+        float f = static_cast<float>(value);
+
+        if (static_cast<double>(f) != value) {
+            // Need 64-bit precision
+            header |= HEADER_DATA_FLAG_64;
+            GodotSerializer::push_int32(result, static_cast<int32_t>(header));
+            GodotSerializer::push_double(result, value);
+        } else {
+            // 32-bit precision is sufficient
+            GodotSerializer::push_int32(result, static_cast<int32_t>(header));
+            GodotSerializer::push_float(result, f);
+        }
+
+        return result;
+    }
+
+    TypeCode getType() const override { return TypeCode::FLOAT_TYPE; }
+    double getValue() const { return value; }
+    float getValueAsFloat() const { return static_cast<float>(value); }
+
+private:
+    double value;
+};
+
+class GodotString : public GodotVariant {
+public:
+    GodotString(const std::string& v) : value(v) {}
+    GodotString(const char* v) : value(v) {}
+
+    std::vector<uint8_t> serialize() const override {
+        std::vector<uint8_t> result;
+        GodotSerializer::push_int32(result, static_cast<int32_t>(TypeCode::STRING_TYPE));
+        GodotSerializer::push_string(result, value);
+        return result;
+    }
+
+    TypeCode getType() const override { return TypeCode::STRING_TYPE; }
+    const std::string& getValue() const { return value; }
+
+private:
+    std::string value;
+};


These types really look like unnecessary bloat and why don't we just use C++ built-in types?

littlecube8152 · 2025-07-25T11:07:47Z

+    CLIENT_ERR = 501
+};
+
+const std::unordered_map<int, std::string> StatusCodeToString = {


We need to discuss the naming conventions in C++. Nevertheless, this should be either all uppercases or lowercases with underscores.

mtmatt · 2025-07-25T14:26:42Z

I converted the PR to draft first. After dealing with the above problems, I will click ready again.

- The code is not working yet, but this is a step towards a header-only library.

HyperSoWeak and others added 24 commits July 18, 2025 01:25

feat: minecraft assets

bef991f

feat: tnt minecart blow up towers and enemies

63ae866

fix: queue_free enemy

4697666

fix: map

c0c5fa8

feat: generate final match config files

447035d

feat: new map (graph)

55d9054

feat: the final tie-breaker

049cb41

fix: disable forbidden features for the tournament

93c1d6b

fix: saving game results

2da616c

feat: output game result compatible with the scoreboard's format

7babccb

Merge remote-tracking branch 'origin/feat/map-graph' into feat/final-…

6a9582b

…tournament

Merge remote-tracking branch 'origin/feat/secret-map-reveal' into fea…

6885380

…t/final-tournament

chore: fake version number

0cdeca9

fix: final fix

7e976a8

fix: shy guy

fd1857b

chore: shy guy animation frame

247beba

hotfix: team10 fix loop logic, not tested

51b92f9

fix: team token mischanged

11fa055

Merge branch 'feat/minecraft' into feat/final-tournament

be52839

fix: slow down after leaving beacon aura

b38348e

fix: revert divide on exit area

1fa6a2e

feat: add cpp api

141b840

fix: stronger web socket for cpp and a better example

46e64ac

fix: return opponent scores

f6e8180

mtmatt requested a review from HyperSoWeak July 25, 2025 09:00

mtmatt and others added 2 commits July 25, 2025 17:03

Merge branch 'develop' into feat/cpp-api

696878f

fix: formater

e49a62e

littlecube8152 suggested changes Jul 25, 2025

View reviewed changes

littlecube8152 changed the title ~~Feat/cpp api~~ C++ API Jul 25, 2025

mtmatt marked this pull request as draft July 25, 2025 14:25

mtmatt added 2 commits July 26, 2025 07:39

fix: move cpp_api files to single header

03ad583

- The code is not working yet, but this is a step towards a header-only library.

refactor: move reference close to var name

a92ae80

mtmatt closed this Nov 21, 2025

PixelCat31415 deleted the feat/cpp-api branch March 9, 2026 13:17

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

C++ API#203

C++ API#203
mtmatt wants to merge 28 commits into
developfrom
feat/cpp-api

mtmatt commented Jul 25, 2025

Uh oh!

littlecube8152 Jul 25, 2025

Uh oh!

littlecube8152 Jul 25, 2025

Uh oh!

littlecube8152 Jul 25, 2025

Uh oh!

littlecube8152 Jul 25, 2025

Uh oh!

littlecube8152 Jul 25, 2025

Uh oh!

littlecube8152 Jul 25, 2025

Uh oh!

littlecube8152 Jul 25, 2025

Uh oh!

littlecube8152 Jul 25, 2025

Uh oh!

mtmatt commented Jul 25, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Uh oh!

Conversation

mtmatt commented Jul 25, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

mtmatt commented Jul 25, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants