screepers / posis Goto Github PK

Currently there is no way to use 0 MOVE part creeps along with POSIS due to the lack of ability to specify an exact spawn point, or at least request access to one of the spawns. Depending on execution order, it is possible to never gain access to the desired spawn structure, since the spawn extension keeps taking it for queued creeps.

The best solution I see is to add an optional spawnId field to the spawnCreep options along with a conditionally optional directions array for specifying which direction the stationary creep should be spawned by. The implementation would be required to spawn these creeps from these exact spawns and directions when spawning this creeps. An error would be thrown if directions is supplied without a spawnId.

I can create the PR myself, but I figured I would open an issue for discussion first.

For example: (ignore the actual properties)

spawnCreep(opts: { pos: RoomPosition, bodyOptions: BodyPartConstant[][], memory?: any, priority: number });

Benefits:

• Named arguments (better chance of breaking if anything changes)
• Extensibility
• No stupid (undefined, undefined, "stuff") to skip some optional arguments
• Destructuring

Is there any particular reason why .startProcess() has to return a container object instead of just the Id? If the launching process needs the actual process, it could call .getProcessById() using the returned Id instead of forcing the container object creation on every call of .startProcess()

https://github.com/screepers/POSIS/blob/master/src/core/IPosisKernel.d.ts#L2

First one is ideal, second one is slightly less so, etc.

Host doesn't have to honor it, but convention is that sort order represents client preferences.

We should consider adding a function to cancel a previously ordered Creep.
This can be useful when the situation changed in an unpredictable way (e.g. a room is getting invaded or an "attack" order has been cancelled).

Proposed declaration:

/**
  * Cancel a previously ordered Creep (`spawnCreep`).
  * Returns `true` if it was cancelled successfully.
  */
cancelCreep(id: string): boolean;