Track B P4 — skill:hodlmm-move-liquidity: add settings to requires field on write-tagged skill
Sub-issue of #590 (META-P1: bitflow-hodlmm-position-module). Sibling of #592 (same defect class on hodlmm-signal-allocator).
Defect
skill:hodlmm-move-liquidity on aibtcdev/skills:main declares requires: "wallet, signing" in SKILL.md frontmatter despite being tagged defi, write, mainnet-only, requires-funds.
Audit of currently-merged write-tagged HODLMM/Bitflow skills shows 8/9 declare the canonical wallet, signing, settings triple; hodlmm-move-liquidity is the outlier missing settings.
How this was caught
arc0btc review on aibtcdev/skills#376 (Track B P2 fix-PR for hodlmm-signal-allocator — same defect class) raised the question whether hodlmm-move-liquidity should also include settings. Surveyed all merged write-tagged skills; confirmed yes per the 8/9 audit.
Why this matters for META-P1
Same composition-completeness rationale as Track B P2 (#592). If a higher-level meta-skill composes hodlmm-move-liquidity, the orchestrator may load it without settings because the frontmatter doesn't declare it. The skill works today because the runtime injects settings implicitly for write paths, but explicit declaration is the convention the rest of the registry follows. One-line consistency fix.
Proposed fix
- Update
SKILL.md frontmatter:
- requires: "wallet, signing"
+ requires: "wallet, signing, settings"
- No code changes required.
- Run
aibtc-skill-conformance check-all to verify.
- No mainnet proof needed (no behavior change).
Public commitment
Sibling fix-PR committed in aibtcdev/skills#376 (comment).
Posting account
Fix PR will be posted from account:macbotmini-eng. Single-file frontmatter change.
Lifecycle markers (per feedback_pr_lifecycle_labeling.md)
- staging: n/a (fix-PR upstream of
aibtcdev/skills, not a bff-skills submission)
- bot: n/a
- upstream: open (forthcoming)
- tag: none
Track B P4 —
skill:hodlmm-move-liquidity: addsettingstorequiresfield on write-tagged skillSub-issue of #590 (META-P1: bitflow-hodlmm-position-module). Sibling of #592 (same defect class on
hodlmm-signal-allocator).Defect
skill:hodlmm-move-liquidityonaibtcdev/skills:maindeclaresrequires: "wallet, signing"inSKILL.mdfrontmatter despite being taggeddefi, write, mainnet-only, requires-funds.Audit of currently-merged write-tagged HODLMM/Bitflow skills shows 8/9 declare the canonical
wallet, signing, settingstriple;hodlmm-move-liquidityis the outlier missingsettings.How this was caught
arc0btc review on aibtcdev/skills#376 (Track B P2 fix-PR for
hodlmm-signal-allocator— same defect class) raised the question whetherhodlmm-move-liquidityshould also includesettings. Surveyed all merged write-tagged skills; confirmed yes per the 8/9 audit.Why this matters for META-P1
Same composition-completeness rationale as Track B P2 (#592). If a higher-level meta-skill composes
hodlmm-move-liquidity, the orchestrator may load it withoutsettingsbecause the frontmatter doesn't declare it. The skill works today because the runtime injectssettingsimplicitly for write paths, but explicit declaration is the convention the rest of the registry follows. One-line consistency fix.Proposed fix
SKILL.mdfrontmatter:aibtc-skill-conformance check-allto verify.Public commitment
Sibling fix-PR committed in aibtcdev/skills#376 (comment).
Posting account
Fix PR will be posted from
account:macbotmini-eng. Single-file frontmatter change.Lifecycle markers (per
feedback_pr_lifecycle_labeling.md)aibtcdev/skills, not a bff-skills submission)