@battlegrid/mcp-server 31.2.66 → 31.2.68

package/README.md

@@ -24,6 +24,84 @@ Seeing package `31.x` alongside handshake `battlegrid@33.x` — the package **be
 
 **What this changes for you:** nothing about how you call anything. Upgrading the package no longer waits on a server deploy, and a server deploy no longer strands you on a package that names the wrong contract — reconnect and the announcement follows. **Contract breaking-change notes are no longer keyed to package versions**, since a contract move is no longer a release here; the v11-and-earlier notes below are kept as history, and the live vocabulary is always discovery.
 
+## Contract history — v87 (a fork's create resolves its source again)
+
+**Breaking at v87: a fork's create can now be refused where it committed.** v87 is the contract on top of
+v86. No schema, description or tool changes.
+
+### Narrowed acceptance
+
+A create draft that `fork_strategy` opened is committed by `commit_strategy_draft` with
+`expectedRevision: null`, or by the builder's Create. That create now resolves the strategy it copied
+again. It uses only the source id and revision in the draft's `ORIGIN`, by the same rule the fork applied:
+
+- **A source that moved past the forked revision** refuses the create with `CONFLICT` and **no
+  `details.nextAct`**. Reading the draft again cannot cure it; the message says to discard the draft and
+  fork again. `get_strategy_draft` and `stage_strategy_draft` report the same refusal in the draft's
+  diagnostics.
+- **A source you can no longer see** — archived, or neither yours nor SYSTEM — answers `NOT_FOUND`.
+
+### Widened acceptance
+
+- **Neither `fork_strategy` nor the create reads the source's stored revision snapshot.** A strategy whose
+  stored snapshot predates the current shape — every SYSTEM template among them — now forks and creates
+  from its live state, where both were refused.
+
+## Contract history — v86 (a refusal hint belongs to the operation that can act on it)
+
+**Breaking at v86: refusals on most tools lose `details.nextAct`, two refusals change code, and one
+draft read is reshaped.** v86 is the contract on top of v85.
+
+### Narrowed refusal details
+
+- **`details.nextAct` follows the refusal's meaning.** `DRAFT_VERSION_MOVED` and `DRAFT_AXIS_CONTESTED`
+  still carry `get_draft` from every tool. A revision `CONFLICT` and the in-flight commit refusal
+  (`get_draft`) and a validation refusal (`stage`) now carry a hint only from `stage_<kind>_draft` and
+  `commit_<kind>_draft` of the strategy and agent kinds.
+- **Removed from every other tool's validation refusal and revision `CONFLICT`**: `submit_market_grid`,
+  `fork_strategy`, `archive_strategy`, `restore_strategy`, and the arena and radar commit, pause, resume
+  and delete tools, and from the malformed-cursor refusal of `list_agent_drafts` and
+  `list_strategy_drafts`. The arena and radar `commit_*_draft` stale-revision refusals lose `get_draft`
+  too. A client that branched on `nextAct` from those tools reads the code instead.
+
+### Changed error codes
+
+- **The strategy active-quota refusal is `FORBIDDEN`**, as the agent slot quota is, where it was
+  `VALIDATION_ERROR` with `nextAct: 'stage'`: on `commit_strategy_draft` and `restore_strategy`.
+- **A create naming an id that is already the caller's strategy or agent is `DRAFT_VERSION_MOVED`**,
+  carrying the caller's draft version at that id, where it was `NOT_FOUND`. An agent create naming a
+  SYSTEM agent is `FORBIDDEN`, where it was `NOT_FOUND`.
+- **The moved-draft refusals end "Read the draft again and show the player what it now holds before
+  committing."**, where they said to commit what it now holds. Read the draft again and show the player
+  before committing; never commit the re-read draft on your own.
+
+### Reshaped output
+
+- **`get_agent_draft.impact` is a union on `operation`**: `{ operation: 'CREATE', capital }` for a create
+  draft, where it was `null`, and `{ operation: 'UPDATE', deployedPresetCount, openPositionCount,
+  radarArmedCoinCount, capital, rebind }` for an edit draft, where the counts sat at the top level. A
+  create draft's `capital` reads its trading configuration — drafted, or the platform's seed — over the
+  bound strategy's band, as the commit's capital check reads it.
+
+### Widened read
+
+- **`get_strategy` with `includeInactive: true`** serves your own strategy in any state, and otherwise
+  the visible set the default read serves — every SYSTEM strategy among it — where it answered
+  `NOT_FOUND` for anything but an owned PRIVATE strategy. Another user's PRIVATE strategy is still
+  `NOT_FOUND`.
+
+### Behaviour behind unchanged schemas
+
+- **The stage and commit descriptions** state the draft content cap (256,000 UTF-8 bytes) and its
+  refusal, what a commit reaches — it closes no position, and the one refusal an open position can cause
+  — and that a moved draft is shown to the player before any commit.
+- **A stage's `DRAFT_VERSION_MOVED`** fires only for a version the draft never reached — read from a draft
+  since committed or discarded. A stage naming an older version than the draft holds is composed over the
+  newer draft.
+
+A strict client re-reads the `get_agent_draft` output schema; list the tools again after the server
+deploys.
+
 ## Contract history — v85 (agents and strategies change only through their drafts)
 
 **Breaking at v85: seven tools are retired, and a surviving tool refuses an input it accepted at v84.**

package/SKILL.md

@@ -68,8 +68,9 @@ than about authoring:
   is `null` for an entity not created yet. Commit only after the player has seen the diff and impact
   and said yes. `discard_<kind>_draft({ request: { <id>, draftVersion } })` is the same fence.
 - **The proxy forwards numbers and `null` unchanged.** It never fills in a version, never retries a
-  refused call and never rebuilds a request: a `DRAFT_VERSION_MOVED`, `DRAFT_AXIS_CONTESTED` or
-  revision `CONFLICT` refusal carries `details.nextAct`, and the next act is the caller's.
+  refused call and never rebuilds a request. A `DRAFT_VERSION_MOVED` or `DRAFT_AXIS_CONTESTED` refusal
+  carries `details.nextAct` from every tool, and a revision `CONFLICT` or validation refusal carries it
+  from a stage or commit; the next act is the caller's.
 
 `fork_strategy` copies a revision into a new create draft and creates nothing until that draft is
 committed. An agent binds to a strategy through its draft's `STRATEGY_BINDING` axis, on a create or
@@ -98,9 +99,9 @@ The `play-market-grid` prompt (discover via `prompts/list`) provides a guided en
 | `API key must start with "bg_live_"` | Invalid key format | Generate a new key at battlegrid.trade → Profile → MCP |
 | Authentication failed (401/403) | Key revoked/rotated | Generate a new key and **restart** the proxy (keys read once at startup) |
 | `"account" parameter is required` | Multi-account call missing `account` | Add the outer `account`; keep `request` unchanged |
-| `DRAFT_VERSION_MOVED` | The draft changed after you read it, or no draft is held (`details.draftVersion` null) | Call `get_<kind>_draft` again and commit what it now holds |
+| `DRAFT_VERSION_MOVED` | The draft changed after you read it, or no draft is held (`details.draftVersion` null) | Call `get_<kind>_draft` again, show the player what it now holds, and commit only on their word |
 | `DRAFT_AXIS_CONTESTED` | The player changed `details.contestedAxes` in their open form after the version you staged against | Read the draft again and propose against what they now have |
-| `CONFLICT` on a commit | The live revision moved after the draft read | Read the draft again; commit at the new `committedRevision` |
+| `CONFLICT` on a commit | The live revision moved after the draft read | Read the draft again, show the player the new diff and impact, and commit at the new `committedRevision` only on their word |
 | Required at allocation Off | A rule flags `required` on a signal weighted `0` (contract 34) | Read `details.inertRequiredSignalIds`; per signal either raise `allocation` or set `required: false` |
 | Method not found | Calling a retired/unknown tool | Re-run `tools/list`; stage, read and commit through the entity's draft |
 | `Wager scope required` | `mcp:wager` not enabled | Enable Server-Signed Wagers in Profile → MCP |

package/dist/index.d.ts

@@ -49,7 +49,7 @@ import { type Implementation, type Prompt, type Resource } from '@modelcontextpr
  * being asked. Move it for a change to THIS package — a proxy fix, a dependency bump, a docs
  * correction. Never move it to track the server.
  */
-export declare const PACKAGE_VERSION = "31.2.66";
+export declare const PACKAGE_VERSION = "31.2.68";
 export declare const DEFAULT_URL = "https://mcp.battlegrid.trade/mcp";
 export interface EnvConfig {
     apiKeys: string[];

package/dist/index.js

@@ -52,7 +52,7 @@ import { ListToolsRequestSchema, CallToolRequestSchema, ListPromptsRequestSchema
  * being asked. Move it for a change to THIS package — a proxy fix, a dependency bump, a docs
  * correction. Never move it to track the server.
  */
-export const PACKAGE_VERSION = '31.2.66';
+export const PACKAGE_VERSION = '31.2.68';
 export const DEFAULT_URL = 'https://mcp.battlegrid.trade/mcp';
 const MAX_RETRIES = 3;
 const RETRY_DELAYS_MS = [2000, 4000, 8000];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@battlegrid/mcp-server",
-  "version": "31.2.66",
+  "version": "31.2.68",
   "description": "BattleGrid MCP server — play crypto prediction games from AI agents",
   "type": "module",
   "main": "dist/index.js",

package/skills/EXPORT.json

@@ -1,12 +1,12 @@
 {
   "generator": "battlegrid-app/server/scripts/export-mcp-skills.mjs",
-  "contractVersion": "85.0.0",
+  "contractVersion": "87.0.0",
   "files": {
-    "battlegrid-agent-management/SKILL.md": "708a37732e822184f99e58fc96b87780e61155e792482fcd56ae56aa7064cd5b",
+    "battlegrid-agent-management/SKILL.md": "b2f04d76480de2f9fdbd00f8cb25610d2b72fcaa4f87ca57a09bd9ffc78fdbba",
     "battlegrid-arena-play/SKILL.md": "4cfaa92a8d0dcaf477b227abd774e7aa8642c185d72108dfd401be0cf8aecd7f",
     "battlegrid-market-analysis/SKILL.md": "3a9943de9f482a8230fb2983efd2f2d8b8c17d49c242e49b68e7243c20c3aa7b",
     "battlegrid-radar-deployment/SKILL.md": "17301fa66f4b128154435462ffa967efcb3d016df671b03c485d0f8b840fc891",
-    "battlegrid-strategy-authoring/SKILL.md": "4c708dc90029e0b991c4c154d9c74a366a29c2d3a827b78936f2f344df3efeb4",
+    "battlegrid-strategy-authoring/SKILL.md": "a5066740d7d57f095545fd812b0228a8eef6e3b08c87047596a0e1639ae4971d",
     "battlegrid-strategy-doctor/SKILL.md": "14f20bff9e2b5a59d11634fe3615b1298d672d5e2eba06d3dc8698b571b3bee3",
     "battlegrid-strategy-examples/SKILL.md": "3c11942f7327e00208d48a7a256faf83d80faf568bd0181b707657bace3c1ef9",
     "battlegrid-strategy-examples/references/playbooks.md": "63aa07f3d55f97e53b2bf6401528807ab23bd093a4bbc37e598158249db24a59",

package/skills/battlegrid-agent-management/SKILL.md

@@ -88,11 +88,14 @@ Create creates it**. Staging creates nothing and spends no slot.
 
 **Then read it, confirm it, and commit it.** `get_agent_draft` serves the draft, its
 `draftVersion`, a `committedRevision` of `null` and the diagnostics: a missing or invalid axis is
-an error there, named, before anything is spent. A create draft has no impact. Then one
-confirmation naming **the strategy, the model, and the budget posture** — the capital ceiling and
-stops the trading configuration will carry, or that it will take the platform seed — and on the
-player's explicit pick call `commit_agent_draft` with the `agentId`, that read's `draftVersion` and
-`expectedRevision: null`. Where the player has the create screen open, their own Create is the
+an error there, named, before anything is spent. Its impact is `operation: CREATE` with the
+create's capital reading: the smallest order its configuration — drafted, or the platform seed —
+places, against the order floor, and whether that is `feasible`. Then one confirmation naming **the
+strategy, the model, and the budget posture** — the capital ceiling and stops the trading
+configuration will carry, or that it will take the platform seed, with the smallest order against
+the floor — and on the player's explicit pick call `commit_agent_draft` with the `agentId`, that
+read's `draftVersion` and `expectedRevision: null`. A reading with `feasible: false` is a create the
+commit will refuse: say so, and stage a larger ceiling or a looser configuration first. Where the player has the create screen open, their own Create is the
 other way to commit it.
 
 A create spends an agent slot against the player's rank quota, decided by the commit itself: a

package/skills/battlegrid-strategy-authoring/SKILL.md

@@ -205,8 +205,8 @@ diagnostics you have not explained to them.
 - **`DRAFT_AXIS_CONTESTED`** names the axes the player's own hand changed after the version you
   named. Their edit is newer than your proposal: read the draft, tell them so, and propose again
   only when they ask. Never re-send the same values.
-- **`DRAFT_VERSION_MOVED`** — the version you named is not the draft's. Read it again and propose
-  against what the player now has.
+- **`DRAFT_VERSION_MOVED`** — the version you named is one the draft never reached: it was read from
+  a draft since committed or discarded. Read it again and propose against what the player now has.
 
 Never raise the number to get past either refusal.
 
@@ -335,9 +335,9 @@ offered, and nothing else is.
 
 ## When the server refuses
 
-Each refusal is a specific typed code, and a draft refusal names its next act in
-`details.nextAct` — `get_draft` or `stage`. Read it and take the cheapest correct step — never
-retry the same call blindly.
+Each refusal is a specific typed code, and a refusal of a draft's stage or commit names its next
+act in `details.nextAct` — `get_draft` or `stage`; a moved or contested draft names `get_draft`
+from any tool. Read it and take the cheapest correct step — never retry the same call blindly.
 
 - **`DRAFT_VERSION_MOVED`** (`nextAct: get_draft`) — the draft changed after your read: the
   player's form saved an edit, another surface staged, or a human-paced review outlived it. Read
@@ -353,8 +353,8 @@ retry the same call blindly.
 - **A validation error** (`nextAct: stage`) — the error the diagnostics named, with its own
   `field` and `details`. Nothing was written. Return to step 4: fix the axis, stage, read, and
   confirm again.
-- **A refusal with no `nextAct`** — the strategy quota, a name already in use, `REPAIR_REQUIRED`,
-  viability, or a `CONFLICT` whose `details.reason` is `RADAR_DEPLOYMENT_MOVED`. The draft is
+- **A refusal with no `nextAct`** — the strategy quota (`FORBIDDEN`), a name already in use,
+  `REPAIR_REQUIRED`, viability, or a `CONFLICT` whose `details.reason` is `RADAR_DEPLOYMENT_MOVED`. The draft is
   untouched. Clear the cause with the player — a rename is an IDENTITY stage, which makes a new
   version to read and confirm — then commit the draft they confirmed. A moved radar deployment is
   a race, not a decision: read and commit again.