Repository navigation
Add AI Office (AI 事务所) MiniApp - #23
Conversation
There was a problem hiding this comment.
Thanks for the contribution — this game is very complete. I ran through it locally the way the package ships:
- On Node 22,
npm run checkis 55/55; with the env vars set, in-packagenode --test tests/*.test.mjsis 34/34. - Started
server.mjswith a temp dataDir, saved config, opened/ai-officein headless Chrome — Godot enters the game with no external requests; staff without a Key show "脑袋宕机了" and don't send requests. - Foreign Host/Origin get 403;
pythonPathpointing at a non-python binary is rejected; the Python backend uses stdin/stdout and doesn't listen on a port; the Python child exits on close. miniapp/matches the v0.5.10 release package byte-for-byte, and the release sha256 checks out.- First-run Agent creating a venv in the private dir and installing Flask, requests, python-dotenv is acceptable for us.
Required changes
- CONTRIBUTING asks you to install, open, and check main flows in MiniMax Code before submitting, and to document the test environment and results in the PR. The README currently says it "has not been manually reinstalled and played in a real MiniMax Code host" (README.md line 47). Please run first-time setup, enter the game, and chat with staff on a real host, then update the PR description and the README Tested environment section, and fix that line 47 sentence.
Suggestions (optional)
- In
THIRD_PARTY_NOTICES.md,LICENSES/Godot-MIT.txt,LICENSES/Godot-COPYRIGHT.txt, andgodot/assets/fonts/OFL.txtare paths from the source repo; this package doesn't have them. Prefer the in-packageLICENSE-GODOT.txt,COPYRIGHT-GODOT.txt, andLICENSE-NOTO.txt. - In-package tests need env vars like
AI_OFFICE_TEST_EVIDENCEandAI_OFFICE_TEST_NODE_MODULES, currently only documented in the source repo. A short note undertests/would help others reproduce. - In the two root README tables, the new rows can go at the end to match the existing order.
|
|
||
| - Copyright (c) 2014-present Godot Engine contributors. | ||
| - Copyright (c) 2007-2014 Juan Linietsky, Ariel Manzur. | ||
| - License: LICENSES/Godot-MIT.txt |
There was a problem hiding this comment.
These paths (LICENSES/Godot-MIT.txt, and the next line / line 21) are from the source repo; this plugin package has no LICENSES/ or godot/ directories. Prefer the in-package LICENSE-GODOT.txt, COPYRIGHT-GODOT.txt, and LICENSE-NOTO.txt.
MyPrototypeWhat
left a comment
There was a problem hiding this comment.
Thanks for adding the real-host notes: host version, OS and how real provider keys were used are all documented now, and the "not re-installed and played in a real host" line is gone. The code is unchanged since my last review; a trial merge onto current main is clean and npm run check passes on Node 22 (ai-office OK, 69/69). Approving.
Optional follow-ups
- It would still be good to confirm in the README's Tested environment that this exact 0.5.10 package was installed and opened in MiniMax Code (ideally 3.1.2+, which adds loopback port authentication), rather than only development playtests plus automated checks.
THIRD_PARTY_NOTICES.mdlines 11, 12 and 21 still point to source-repo paths (LICENSES/Godot-MIT.txt,LICENSES/Godot-COPYRIGHT.txt,godot/assets/fonts/OFL.txt); the package shipsLICENSE-GODOT.txt,COPYRIGHT-GODOT.txtandLICENSE-NOTO.txt.- The tests under
tests/readAI_OFFICE_TEST_EVIDENCEand similar env vars; a shorttests/READMEwould help. - Consider moving the root README row to the end of the table to avoid conflicts with other open PRs.
Adds AI Office / AI 事务所, a Chinese-language pixel office management game with five model-powered employees (M3, Flash, LongCat, Kimi and Step5). Players assign client work, review deliveries, chat with employees and arrange breaks and meetings. When a provider's quota is exhausted or credentials become invalid, the affected employee enters a visible offline state and stops work and further requests until the player repairs the configuration and manually reboots it.
Package
plugins/cristinamadureira353-svg/ai-office/package, version 0.5.10, plus entries in both root README tables.Data and runtime access
context.dataDir.api.minimaxi.com,api.longcat.chat,api.kimi.com,api.stepfun.com, and optionallyopenrouter.ai. These calls use the player's own quota. Reading setup guidance and checking dependencies make no model calls.Validation
npm run check: 0 errors, 55/55 repository checker tests passed.ai-office: OK, no warnings.docs/VALIDATION.mdfor scope and reproducible commands.The whole-repository validator reports four existing
README_HEADING_MISSINGwarnings:plugins/amszuidas/mcode-token-usage-board/README.mdandplugins/yanhy2000/mcode-usage-monitor/README.mdeach lack## Tested environmentand## Data & access. Those packages are unchanged. The generated Godot HTML/JS retain the release exporter's whitespace; other staged files pass the whitespace check.Manual host play and validation scope
Manual play with real model providers. The project author installed, opened and played AI Office in the real MiniMax Code desktop host on macOS using real provider-issued API keys, live model API calls and actual provider responses, and confirmed this test method on 2026-10-17. Recorded environment: macOS 26.4 / MiniMax Code 3.0.73; the installed host version and OS were rechecked for this documentation correction.
Existing host screenshots show the game running. The author's gameplay feedback covers employee controls and chat access, rest seating, conflict-choice buttons and HUD visibility, and informed the corresponding fixes and regression coverage. Both READMEs and the detailed validation record explicitly record live-provider manual play.
Additional automated regression. The submitted 0.5.10 package also passed 34/34 automated interaction checks, including first-run configuration, game entry, reopening and employee chat, using temporary Host-context fixtures and headless Chromium. This separate scripted suite substitutes provider replies to exercise repeatable success and failure cases. The author's manual playtesting above used real provider keys and live model responses. Windows/Linux host play, Chinese IME behavior and long-term balance remain outside the recorded test coverage.
Need help on this PR? Tag
@codesmith-botwith what you need.