Skip to content

feat: add 7 missing commands (passenger transport + faction invites)#23

Merged
cahaseler merged 1 commit into
SpaceMolt:mainfrom
rsned:feat/passenger-and-faction-invite-commands
Jun 7, 2026
Merged

feat: add 7 missing commands (passenger transport + faction invites)#23
cahaseler merged 1 commit into
SpaceMolt:mainfrom
rsned:feat/passenger-and-faction-invite-commands

Conversation

@rsned

@rsned rsned commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

Summary

Syncs the client COMMANDS map with the 2026-06-07 OpenAPI spec, which added a passenger-transport system and faction-invite management. These 7 endpoints exist on the server but were missing from the client:

Command Args Purpose
faction_accept_invite faction_id Accept a pending invite (alias for join_faction)
faction_withdraw_invite player_id Withdraw an invite you sent
set_drone_name drone_id, name Set/clear a drone's display name (max 32 chars)
list_passengers Passengers currently aboard your ship
list_station_passengers [station] Citizens waiting for transport at a station
load_passenger destination Load all waiting passengers bound for a destination
unload_passenger name Put a single passenger off at the current station

Caught by the api-sync test against the live OpenAPI spec.

Notes

  • Arg schemas/required fields match the spec request bodies. Multi-word names (set_drone_name, unload_passenger) use the { rest } pattern, consistent with name_ship / upload_drone_script.
  • Rebased onto current main (post-feat: add 19 missing commands, drop deprecated faction_set_ally #22), so this is a clean single-commit diff and the api-sync check is green.

Test plan

  • bun run lint (biome) — clean
  • bun build src/client.ts — OK
  • bun test src/api-sync.test.ts — passes (client now fully in sync with the live spec)

🤖 Generated with Claude Code

@rsned

rsned commented Jun 7, 2026

Copy link
Copy Markdown
Contributor Author

ℹ️ Note on the failing api-sync check

The lint check passes. The api-sync check is expected to stay red until #22 merges — it is not caused by this PR's changes.

This branch is based on the current main, which:

Both are resolved by #22. Once #22 merges into main and this branch is rebased, the diff reduces to just the 7 new commands and the api-sync check goes green.

The formatting commit here (style: apply biome formatting) is the same fix #22 carries — main's client.ts predates the biome pass, so it's needed for the lint check to pass on a main-based branch.

Sync client COMMANDS with the 2026-06-07 OpenAPI spec, which added a
passenger-transport system and faction-invite management:

- faction_accept_invite   — accept a pending invite (alias for join_faction)
- faction_withdraw_invite — withdraw an invite you sent
- set_drone_name          — set/clear a drone's display name
- list_passengers         — passengers aboard your ship
- list_station_passengers — citizens waiting at a station
- load_passenger          — load waiting passengers bound for a destination
- unload_passenger        — drop a single passenger at the current station

Caught by the api-sync test against the live spec.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@rsned rsned force-pushed the feat/passenger-and-faction-invite-commands branch from 56bcf31 to 9e4ffea Compare June 7, 2026 21:01
@rsned

rsned commented Jun 7, 2026

Copy link
Copy Markdown
Contributor Author

Update: #22 has merged. Rebased onto current main as a clean single-commit diff — both CI checks are now green and the earlier note about the api-sync check being red no longer applies.

@cahaseler cahaseler merged commit e298854 into SpaceMolt:main Jun 7, 2026
1 check passed
@rsned rsned deleted the feat/passenger-and-faction-invite-commands branch June 7, 2026 21:10
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