Skip to main content
Glama
Flyozzzz

OMP Tandem

by Flyozzzz
OverviewSchemaSourceRelated ServersDiscussionsAdmin
Python
Local

tandem_review_run

Run a read-only code review scenario with automatic stage coordination: capture a snapshot, perform an independent review, and optionally compare author input, tracking progress via run_id.

Instructions

Run one read-only review scenario without manually coordinating its native turns.

start: supply ReviewRequest and a stable request_key for this logical request. Reusing the key returns the same run; a different payload conflicts. Code captures the snapshot, runs independent think review, then at most one comparison if author material was supplied and the independent report succeeded. Total budget defaults to 600 seconds, including startup, both stages and questions. It never edits files or runs supplied test commands. status/reply/cancel: use the returned run_id, never start again to wait. Replies require the current question_id. Full stage answers, findings, applicability and peer-only usage are assembled by code. Claims are not automatically accepted or applied. Missing context needs an explicit new capture (context_paths), not hidden live reads.

Input Schema

TableJSON Schema
NameRequiredDescriptionDefault
actionYes
answerNo
run_idNo
compareNo
requestNo
executionNo
question_idNo
request_keyNo
wait_secondsNo
budget_secondsNo

Output Schema

TableJSON Schema
NameRequiredDescriptionDefault

No arguments

Schema Changelog

Changes observed during successful MCP inspections.

  1. Changed1 schema field changedv3.5.0
    • changedInput schema / properties / request / anyOf
      Previous value: -[
-  {
-    "properties": {
-      "author_proposal": {
-        "default": "",
-        "maxLength": 200000,
-        "type": "string"
-      },
-      "author_rationale": {
-        "default": "",
-        "maxLength": 200000,
-        "type": "string"
-      },
-      "base": {
-        "default": "HEAD",
-        "maxLength": 200,
-        "minLength": 1,
-        "type": "string"
-      },
-      "checks": {
-        "items": {
-          "properties": {
-            "code_fingerprint": {
-              "anyOf": [
-                {
-                  "pattern": "^[0-9a-f]{64}$",
-                  "type": "string"
-                },
-                {
-                  "type": "null"
-                }
-              ],
-              "default": null
-            },
-            "command": {
-              "anyOf": [
-                {
-                  "maxLength": 4000,
-                  "type": "string"
-                },
-                {
-                  "type": "null"
-                }
-              ],
-              "default": null
-            },
-            "name": {
-              "maxLength": 200,
-              "minLength": 1,
-              "type": "string"
-            },
-            "output": {
-              "maxLength": 1048576,
-              "type": "string"
-            },
-            "source": {
-              "default": "coordinator supplied",
-              "maxLength": 1000,
-              "type": "string"
-            }
-          },
-          "required": [
-            "name",
-            "output"
-          ],
-          "type": "object"
-        },
-        "maxItems": 32,
-        "type": "array"
-      },
-      "context_paths": {
-        "description": "Explicit additional saved context; staged reviews use index bytes, never live files.",
-        "items": {
-          "type": "string"
-        },
-        "maxItems": 256,
-        "type": "array"
-      },
-      "criteria": {
-        "items": {
-          "type": "string"
-        },
-        "maxItems": 100,
-        "type": "array"
-      },
-      "external_boundaries": {
-        "items": {
-          "type": "string"
-        },
-        "maxItems": 100,
-        "type": "array"
-      },
-      "paths": {
-        "anyOf": [
-          {
-            "items": {
-              "type": "string"
-            },
-            "maxItems": 256,
-            "minItems": 1,
-            "type": "array"
-          },
-          {
-            "type": "null"
-          }
-        ],
-        "default": null
-      },
-      "requirements": {
-        "maxLength": 200000,
-        "minLength": 1,
-        "type": "string"
-      },
-      "source": {
-        "default": "worktree",
-        "description": "Select worktree bytes or Git index bytes compared with base. Staged capture never reads live files.",
-        "enum": [
-          "worktree",
-          "staged"
-        ],
-        "type": "string"
-      }
-    },
-    "required": [
-      "requirements"
-    ],
-    "type": "object"
-  },
-  {
-    "type": "null"
-  }
-]New value: +[
+  {
+    "properties": {
+      "author_proposal": {
+        "default": "",
+        "maxLength": 200000,
+        "type": "string"
+      },
+      "author_rationale": {
+        "default": "",
+        "maxLength": 200000,
+        "type": "string"
+      },
+      "base": {
+        "default": "HEAD",
+        "maxLength": 200,
+        "minLength": 1,
+        "type": "string"
+      },
+      "checks": {
+        "items": {
+          "properties": {
+            "code_fingerprint": {
+              "anyOf": [
+                {
+                  "pattern": "^[0-9a-f]{64}$",
+                  "type": "string"
+                },
+                {
+                  "type": "null"
+                }
+              ],
+              "default": null
+            },
+            "command": {
+              "anyOf": [
+                {
+                  "maxLength": 4000,
+                  "type": "string"
+                },
+                {
+                  "type": "null"
+                }
+              ],
+              "default": null
+            },
+            "name": {
+              "maxLength": 200,
+              "minLength": 1,
+              "type": "string"
+            },
+            "output": {
+              "maxLength": 1048576,
+              "type": "string"
+            },
+            "source": {
+              "default": "coordinator supplied",
+              "maxLength": 1000,
+              "type": "string"
+            }
+          },
+          "required": [
+            "name",
+            "output"
+          ],
+          "type": "object"
+        },
+        "maxItems": 32,
+        "type": "array"
+      },
+      "commit": {
+        "anyOf": [
+          {
+            "maxLength": 200,
+            "minLength": 1,
+            "type": "string"
+          },
+          {
+            "type": "null"
+          }
+        ],
+        "default": null,
+        "description": "Required with source=commit: the exact committed snapshot to review against base."
+      },
+      "context_paths": {
+        "description": "Explicit additional saved context; staged reviews use index bytes, never live files.",
+        "items": {
+          "type": "string"
+        },
+        "maxItems": 256,
+        "type": "array"
+      },
+      "criteria": {
+        "items": {
+          "type": "string"
+        },
+        "maxItems": 100,
+        "type": "array"
+      },
+      "external_boundaries": {
+        "items": {
+          "type": "string"
+        },
+        "maxItems": 100,
+        "type": "array"
+      },
+      "paths": {
+        "anyOf": [
+          {
+            "items": {
+              "type": "string"
+            },
+            "maxItems": 256,
+            "minItems": 1,
+            "type": "array"
+          },
+          {
+            "type": "null"
+          }
+        ],
+        "default": null
+      },
+      "requirements": {
+        "maxLength": 200000,
+        "minLength": 1,
+        "type": "string"
+      },
+      "source": {
+        "default": "worktree",
+        "description": "Select worktree bytes, Git index bytes, or the tree of an existing commit compared with base. Staged and commit captures never read live files.",
+        "enum": [
+          "worktree",
+          "staged",
+          "commit"
+        ],
+        "type": "string"
+      }
+    },
+    "required": [
+      "requirements"
+    ],
+    "type": "object"
+  },
+  {
+    "type": "null"
+  }
+]
  2. Addedv3.3.0

TDQS

A4.1/5.0
Behavior5/5

Does the description disclose side effects, auth requirements, rate limits, or destructive behavior?

With no annotations provided, the description carries the full burden and does so well: read-only, never edits files or runs supplied test commands, runs an independent review and at most one comparison, default budget, code-assembled outputs, and claims are not automatically accepted. This is unusually rich behavioral disclosure.

Agents need to know what a tool does to the world before calling it. Descriptions should go beyond structured annotations to explain consequences.

Conciseness4/5

Is the description appropriately sized, front-loaded, and free of redundancy?

Front-loads the core purpose and then organizes action-specific guidance under start and status/reply/cancel. The text is dense but mostly earns its place; a few clauses are opaque but not wasteful enough to damage clarity.

Shorter descriptions cost fewer tokens and are easier for agents to parse. Every sentence should earn its place.

Completeness4/5

Given the tool's complexity, does the description cover enough for an agent to succeed on first attempt?

For a complex 10-parameter orchestrator with an output schema, the description covers the workflow, budget behavior, context capture rules, and claim handling. It does not fully document every parameter or distinguish sibling tools, but it gives enough to invoke the tool correctly in most cases.

Complex tools with many parameters or behaviors need more documentation. Simple tools need less. This dimension scales expectations accordingly.

Parameters3/5

Does the description clarify parameter syntax, constraints, interactions, or defaults beyond what the schema provides?

Schema description coverage is 0% across 10 parameters, so the description must compensate. It explains action, request_key, request/ReviewRequest, run_id, question_id, and budget_seconds, but leaves answer, compare, execution, wait_seconds, and most nested request fields unexplained, so coverage is only partial.

Input schemas describe structure but not intent. Descriptions should explain non-obvious parameter relationships and valid value ranges.

Purpose4/5

Does the description clearly state what the tool does and how it differs from similar tools?

States a specific verb and resource: 'Run one read-only review scenario.' The action enum is explained, but the description never explicitly distinguishes this multiplexer from siblings such as tandem_start, tandem_review, tandem_reply, or tandem_cancel, so an agent must infer the boundary.

Agents choose between tools based on descriptions. A clear purpose with a specific verb and resource helps agents select the right tool.

Usage Guidelines4/5

Does the description explain when to use this tool, when not to, or what alternatives exist?

Provides clear action-level guidance: start needs a ReviewRequest and stable request_key; status/reply/cancel use the returned run_id; replies need the current question_id; never start again to wait. It does not name alternative tools or state when not to use this tool, but the action usage is well covered.

Agents often have multiple tools that could apply. Explicit usage guidance like "use X instead of Y when Z" prevents misuse.

Deploy Server

Other Tools