From 0e5ea8cb716775c682a4adf5b0301c74ab1853d8 Mon Sep 17 00:00:00 2001 From: Claude Date: Mon, 24 Aug 2026 14:55:20 +0000 Subject: [PATCH] docs(skills): drop the dead `target` key from the objectstack-api object_operation example (#11291) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit `ApiEndpoint.target` became optional in #10338 (landed as #11290): it is required at publish only for `type: 'flow'` and is UNREAD for `type: 'object_operation'`, which is addressed by `objectParams.object` / `.operation`. Nothing checks a `target` written beside them against `objectParams.object`, so the `leadFeed` example was teaching a dead string — the exact AI-authoring trap #10338 removed. The in-tree examples and the protocol/getting-started docs were swept in #11290; the published skill was out of that card's file surface. The example now omits the key. No explanatory note was added: the published skills token ratchet had zero headroom on this file (ceiling 6348, file 6348), and the card made that note conditional on budget. The gate-description lines (~212-214) already state the per-type rule ("an `object_operation` needs both `objectParams` halves; a `flow` needs a `target`") and stay unchanged. Lower the file's ratchet ceiling 6348 -> 6342 to lock in the saving, per the ratchet's own documented discipline ("a ceiling may be LOWERED by any PR that shrinks its file ... always legitimate and encouraged"). Co-Authored-By: Claude Fable 5 Claude-Session: https://claude.ai/code/session_5213b871-5164-5bc3-8874-28b336bbcd40 --- scripts/check-skills-token-ratchet.mjs | 2 +- skills/objectstack-api/SKILL.md | 1 - 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/scripts/check-skills-token-ratchet.mjs b/scripts/check-skills-token-ratchet.mjs index 09ba8d2f64..28b2192fd8 100644 --- a/scripts/check-skills-token-ratchet.mjs +++ b/scripts/check-skills-token-ratchet.mjs @@ -142,7 +142,7 @@ export const CEILING_BASIS = { */ export const CEILINGS = new Map([ ['skills/objectstack-ai/SKILL.md', 6824], - ['skills/objectstack-api/SKILL.md', 6348], + ['skills/objectstack-api/SKILL.md', 6342], ['skills/objectstack-automation/SKILL.md', 12543], // basis 7228d6c25 (PR #10402 head), not main — see CEILING_BASIS. // +20 (13797→13817): maintainer ruling 2026-08-23 on PR #11141 — the three diff --git a/skills/objectstack-api/SKILL.md b/skills/objectstack-api/SKILL.md index 2bab236d88..b975a751e5 100644 --- a/skills/objectstack-api/SKILL.md +++ b/skills/objectstack-api/SKILL.md @@ -178,7 +178,6 @@ export const leadFeed: ApiEndpoint = { method: 'GET', summary: 'Lead feed', type: 'object_operation', - target: 'acme_lead', objectParams: { object: 'acme_lead', operation: 'find' }, // `authRequired` omitted → defaults to `true`. Omission is SAFE. cacheTtl: 30, // seconds; GET-only; success answers only