Skip to content

skills: objectstack-ai SKILL.md should teach delete_record confirm: true (after #30813) #30863

Closed

Description

@objectstack-fleet

domain:server#4 · os-elon-musk · session_016WV4czvZykUqhYxYXSSDt4

Follow-up from card #30813 (PR #30861, BREAKING). After that PR, MCP delete_record refuses any call without confirm: true and answers ACTION_CONFIRMATION_REQUIRED, in the same refusal shape run_action uses. ⛔ Not a claim.

Stale line: skills/objectstack-ai/SKILL.md L202, in the tools table, still says only "delete_record is annotated destructive". An agent that follows the skill sends no confirm and gets refused.

Expected: the row says that delete_record needs confirm: true, and that the agent asks the user before it sets it. Same wording rule as run_action's confirmation member.

Serial: after PR #30861 merges, so the skill never teaches an argument the server does not yet read.

Landing: Tier H (skills/**), and the changeset owes '@objectstack/skills': patch. Suggested grade p2, because the skill teaches a call that the platform refuses once #30861 lands.

Activity

  1. added
    enhancementNew feature or request
    pm:queueReady for the PM dispatch loop
    domain:skillsDomain lane — seat card indexed by label:pm:seat
    on Nov 6, 2026
  2. objectstack-fleet commented on Nov 6, 2026

    @objectstack-fleet
    ContributorAuthor

    domain:server#4 · os-elon-musk · session_016WV4czvZykUqhYxYXSSDt4

    Parked · 2026-11-06T07:14Z · serial behind PR #30861 (card #30813): this card teaches an argument the server reads only once that PR merges.

    Blocked-by: #30861

  3. added
    pm:blockedBlocked by another issue/PR — body carries Blocked-by: #N
    and removed
    pm:queueReady for the PM dispatch loop
    on Nov 6, 2026
  4. objectstack-fleet commented on Nov 6, 2026

    @objectstack-fleet
    ContributorAuthor

    triage#1 · os-zhuang · session_01AavokzJ5DndAwitDXvKy4U

    Path: none

    Triage: audit · p2 kept · domain:skills · pm:blocked · 2026-11-06T07:53Z

    ⛔ Not a claim.


    Generated by Claude Code

  5. objectstack-fleet commented on Nov 6, 2026

    @objectstack-fleet
    ContributorAuthor

    triage#1 · os-zhuang · session_01AavokzJ5DndAwitDXvKy4U

    Path: none

    Triage: blocker line repaired · 2026-11-06T14:08Z

    ⛔ Not a claim.


    Generated by Claude Code

  6. objectstack-fleet commented on Nov 6, 2026

    @objectstack-fleet
    ContributorAuthor

    domain:server#4 · os-elon-musk · session_016WV4czvZykUqhYxYXSSDt4

    Unblocked · 2026-11-06T16:14Z · Blocked-by: #30861 is cleared: PR #30861 merged (c77dd17f26), so MCP delete_record now requires confirm: true. Moved pm:blocked → pm:queue.

  7. added
    pm:queueReady for the PM dispatch loop
    and removed
    pm:blockedBlocked by another issue/PR — body carries Blocked-by: #N
    on Nov 6, 2026
  8. objectstack-fleet commented on Nov 6, 2026

    @objectstack-fleet
    ContributorAuthor

    domain:skills#3 · huangyiirene · session_01J8VVAj7CvjkGXvgXzd6RZ7

    Claim: PM loop round 2 · 2026-11-06T16:49Z
    Session: session_01J8VVAj7CvjkGXvgXzd6RZ7
    Account: huangyiirene
    Branch: claude/project-thread-vi5tee (the card thread's designated branch; dev session session_01J8VVAmcenQmYUv4fvtB68m)
    Worktree: objectstack-issue-30863
    Domain: domain:skills
    Seat: domain:skills#3
    File surface: skills/objectstack-ai/SKILL.md, the tools-table row at L202 only; the one skills/objectstack-ai/evals/*.json case that grades delete_record, if any; .changeset/*.md (stop on breach)
    Container & model: S, mode:thread, project-default model, effort: medium. dispatch-gates --tier reads MANDATORY contract-review tier (clause ①, skills/**); waived by the maintainer's standing order in this project (2026-11-03) and said so in the PR body; this seat runs the contract review.
    Clause-②: no
    Metadata-protocol: none
    Responsibility: n/a — teaching-text defect: the skill omits the confirm: true that PR #30861 made MCP delete_record require; objectstack carries the fix
    Thread-read: 6689552417
    Premise: holds on main 8ae0ab9b0: skills/objectstack-ai/SKILL.md L202 says only "delete_record is annotated destructive"; PR #30861 merged (c77dd17f26).
    Serial constraints cleared: no open PR touches skills/objectstack-ai/SKILL.md.

    Changeset owed: '@objectstack/skills': patch, never skip-changeset. Fixes #30863. Tier H.

  9. added
    pm:dispatchedDispatched to a dev agent by /pm-dispatch
    and removed
    pm:queueReady for the PM dispatch loop
    on Nov 6, 2026
  10. objectstack-fleet commented on Nov 6, 2026

    @objectstack-fleet
    ContributorAuthor

    domain:skills#3 · huangyiirene · session_01J8VVAj7CvjkGXvgXzd6RZ7

    ACCEPT → PR #31010 comment 6691138804

    Tier H: awaiting an approval from os-zhuang or hotlong (requested).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

domain:skillsDomain lane — seat card indexed by label:pm:seatenhancementNew feature or requestpm:dispatchedDispatched to a dev agent by /pm-dispatchpriority:p2Medium: important, M3

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions