Naiyuan Qing 31ec36146f refactor(skills): split createSkillWithFiles for tx composition + add workspace find-or-create query ...

Two soft blockers cleared so create-from-template (next commit) can
fold N skill creates and the agent + binding writes into one outer
transaction:

1. createSkillWithFiles used to Begin/Commit its own tx. Caller
   composition was impossible — N invocations meant N separate
   transactions and no atomicity over the whole materialise step.
   Pull the body into createSkillWithFilesInTx(ctx, qtx, input); the
   original function becomes a thin wrapper that manages its own tx
   for standalone callers. Existing call sites: zero behaviour change.

2. Add GetSkillByWorkspaceAndName sqlc query — workspace skill lookup
   by name, anchored to UNIQUE(workspace_id, name) from migration
   008. Lets the template materialiser implement find-or-create:
   reuse the workspace's existing skill row when a template
   references the same name, rather than crashing on the unique
   constraint or polluting the workspace with `<name>-2` clones.

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

2026-05-13 16:45:04 +08:00

..

agent

fix(agent): stop filtering Pi extension tools via hardcoded --tools allowlist (#2379) (#2381)

2026-05-11 16:11:32 +08:00

db

refactor(skills): split createSkillWithFiles for tx composition + add workspace find-or-create query

2026-05-13 16:45:04 +08:00

protocol

fix(chat): collapse chat-done flicker via inline cache write (#2509)

2026-05-13 15:27:44 +08:00

redact

…