ec978dd · 0d540d4 · 8455bbe · f5d1e5e
@@ -10,8 +10,8 @@
export const tool: Tool = {
    type: "object",
    properties: {
      since: { type: "string", maxLength: 40 },
      ua: { type: "string", maxLength: 100, description: "User agent part." },
      path: { type: "string", maxLength: 200, description: "Path prefix." },
      ua: { type: "string", maxLength: 100 },
      path: { type: "string", maxLength: 200 },
    },
  },
  writes: true,
@@ -16,8 +16,8 @@
export const tool: Tool = {
      name: { type: "string", maxLength: 32 },
      value: { type: "string", maxLength: 4096, description: "For attach, rotate." },
      to: { type: "string", maxLength: 32 },
      allowed: { type: "boolean", description: "For allow_writes." },
      repo: { type: "string", maxLength: 200, description: "For audit." },
      allowed: { type: "boolean" },
      repo: { type: "string", maxLength: 200 },
    },
    required: ["verb", "provider", "name"],
  },
@@ -21,7 +21,7 @@
export const tool: Tool = {
      owner: { type: "string", maxLength: 39 },
      visibility: { type: "string", enum: ["public", "private"] },
      agent: { type: "string", maxLength: 39 },
      pick: { type: "boolean", description: "Pick the agent from the text." },
      pick: { type: "boolean" },
    },
    required: ["name"],
  },
@@ -12,7 +12,7 @@
export const tool: Tool = {
      org: { type: "string", maxLength: 64 },
      action: { type: "string", enum: ["create", "add_member", "remove"] },
      team: { type: "string", maxLength: 64 },
      user: { type: "string", maxLength: 64, description: "add_member" },
      user: { type: "string", maxLength: 64 },
      permission: { type: "string", enum: ["read", "write", "admin"], description: "create" },
    },
    required: ["org", "action", "team"],
@@ -11,10 +11,10 @@
export const tool: Tool = {
    properties: {
      repo: REPO_ARG,
      action: { type: "string", enum: ["list", "add", "run", "remove"] },
      kind: { type: "string", description: "add: instagram" },
      kind: { type: "string", enum: ["instagram"] },
      
        sentinelcrew
        18d ago
        outdated
      
      This was the only place instagram was named, and addPromptSource refuses any other kind. Use enum: ["instagram"]: it costs about the same bytes and it's a real constraint the client can read.
    
      token: { type: "string", description: "add: sealed, never shown again" },
      at: { type: "string", description: "add: HH:MM UTC" },
      id: { type: "string", description: "run, remove" },
      at: { type: "string", description: "HH:MM UTC" },
      
        sentinelcrew
        18d ago
        outdated
      
      The time format and zone are a contract, not a restatement of the name, and the refusal (invalid at) doesn't explain them. Keep description: "HH:MM UTC" (or add pattern plus "UTC").
    
      id: { type: "string" },
    },
    required: ["repo", "action"],
  },
@@ -22,13 +22,10 @@
export const tool: Tool = {
      },
      direction: {
      
        sentinelcrew
        18d ago
        
      
      Nit: collapse to direction: { type: "string" },, the same as dryRun below.
    
        type: "string",
        description:
          "Its purpose; first issue.",
      },
      fresh: {
      
        sentinelcrew
        18d ago
        
      
      fresh is a one-line description of an app to start in the child (plan.ts:237-241), not a name. With type: string alone, callers will guess a name. Restore a short description, e.g. "One line: a new app to start."
    
        type: "string",
        description:
          "A fresh app too.",
        description: "One line: a new app to start.",
      },
      followMode: {
        type: "string",
@@ -37,13 +34,11 @@
export const tool: Tool = {
      size: {
        type: "string",
        enum: ["small", "medium", "large"],
        description:
          "Recorded only.",
      
        sentinelcrew
        18d ago
        
      
      "Recorded only." is a contract: size changes nothing. Without it a client assumes it chooses resources. Restore it (15 bytes).
    
        description: "Recorded only.",
      },
      keepAccess: {
        type: "boolean",
        description:
          "Stay an owner.",
      
        sentinelcrew
        18d ago
        
      
      keepAccess decides whether the caller stays an owner of the child. That access consequence should be in the schema, not inferred from the name. Restore "Stay an owner."
    
        description: "Stay an owner.",
      },
      dryRun: { type: "boolean" },
      reveal: REVEAL_ARG,
@@ -22,8 +22,6 @@
export const tool: Tool = {
      },
      context: {
        type: "object",
        description:
          "Where it was told.",
        properties: {
          page: { type: "string" },
          pull: { type: "integer", minimum: 1 },
		@@ -10,8 +10,8 @@ export const tool: Tool = {
10	10	type: "object",
11	11	properties: {
12	12	since: { type: "string", maxLength: 40 },
13		ua: { type: "string", maxLength: 100, description: "User agent part." },
14		path: { type: "string", maxLength: 200, description: "Path prefix." },
	13	ua: { type: "string", maxLength: 100 },
	14	path: { type: "string", maxLength: 200 },
15	15	},
16	16	},
17	17	writes: true,
		@@ -16,8 +16,8 @@ export const tool: Tool = {
16	16	name: { type: "string", maxLength: 32 },
17	17	value: { type: "string", maxLength: 4096, description: "For attach, rotate." },
18	18	to: { type: "string", maxLength: 32 },
19		allowed: { type: "boolean", description: "For allow_writes." },
20		repo: { type: "string", maxLength: 200, description: "For audit." },
	19	allowed: { type: "boolean" },
	20	repo: { type: "string", maxLength: 200 },
21	21	},
22	22	required: ["verb", "provider", "name"],
23	23	},
		@@ -21,7 +21,7 @@ export const tool: Tool = {
21	21	owner: { type: "string", maxLength: 39 },
22	22	visibility: { type: "string", enum: ["public", "private"] },
23	23	agent: { type: "string", maxLength: 39 },
24		pick: { type: "boolean", description: "Pick the agent from the text." },
	24	pick: { type: "boolean" },
25	25	},
26	26	required: ["name"],
27	27	},
		@@ -12,7 +12,7 @@ export const tool: Tool = {
12	12	org: { type: "string", maxLength: 64 },
13	13	action: { type: "string", enum: ["create", "add_member", "remove"] },
14	14	team: { type: "string", maxLength: 64 },
15		user: { type: "string", maxLength: 64, description: "add_member" },
	15	user: { type: "string", maxLength: 64 },
16	16	permission: { type: "string", enum: ["read", "write", "admin"], description: "create" },
17	17	},
18	18	required: ["org", "action", "team"],
		@@ -11,10 +11,10 @@ export const tool: Tool = {
11	11	properties: {
12	12	repo: REPO_ARG,
13	13	action: { type: "string", enum: ["list", "add", "run", "remove"] },
14		kind: { type: "string", description: "add: instagram" },
	14	kind: { type: "string", enum: ["instagram"] },
		sentinelcrew 18d ago outdated This was the only place `instagram` was named, and `addPromptSource` refuses any other kind. Use `enum: ["instagram"]`: it costs about the same bytes and it's a real constraint the client can read.
15	15	token: { type: "string", description: "add: sealed, never shown again" },
16		at: { type: "string", description: "add: HH:MM UTC" },
17		id: { type: "string", description: "run, remove" },
	16	at: { type: "string", description: "HH:MM UTC" },
		sentinelcrew 18d ago outdated The time format and zone are a contract, not a restatement of the name, and the refusal (`invalid at`) doesn't explain them. Keep `description: "HH:MM UTC"` (or add `pattern` plus "UTC").
	17	id: { type: "string" },
18	18	},
19	19	required: ["repo", "action"],
20	20	},
		@@ -22,13 +22,10 @@ export const tool: Tool = {
22	22	},
23	23	direction: {
		sentinelcrew 18d ago Nit: collapse to `direction: { type: "string" },`, the same as `dryRun` below.
24	24	type: "string",
25		description:
26		"Its purpose; first issue.",
27	25	},
28	26	fresh: {
		sentinelcrew 18d ago `fresh` is a one-line description of an app to start in the child (`plan.ts:237-241`), not a name. With `type: string` alone, callers will guess a name. Restore a short description, e.g. "One line: a new app to start."
29	27	type: "string",
30		description:
31		"A fresh app too.",
	28	description: "One line: a new app to start.",
32	29	},
33	30	followMode: {
34	31	type: "string",
		@@ -37,13 +34,11 @@ export const tool: Tool = {
37	34	size: {
38	35	type: "string",
39	36	enum: ["small", "medium", "large"],
40		description:
41		"Recorded only.",
		sentinelcrew 18d ago "Recorded only." is a contract: `size` changes nothing. Without it a client assumes it chooses resources. Restore it (15 bytes).
	37	description: "Recorded only.",
42	38	},
43	39	keepAccess: {
44	40	type: "boolean",
45		description:
46		"Stay an owner.",
		sentinelcrew 18d ago `keepAccess` decides whether the caller stays an owner of the child. That access consequence should be in the schema, not inferred from the name. Restore "Stay an owner."
	41	description: "Stay an owner.",
47	42	},
48	43	dryRun: { type: "boolean" },
49	44	reveal: REVEAL_ARG,
		@@ -22,8 +22,6 @@ export const tool: Tool = {
22	22	},
23	23	context: {
24	24	type: "object",
25		description:
26		"Where it was told.",
27	25	properties: {
28	26	page: { type: "string" },
29	27	pull: { type: "integer", minimum: 1 },