feat(bankr): migrate to CLI + add non-interactive login docs#109
Merged
feat(bankr): migrate to CLI + add non-interactive login docs#109
Conversation
- Remove bankr/scripts/ (curl-based submit/poll/cancel) — replaced by `bankr prompt`, `bankr status`, `bankr cancel` CLI commands - Update 8 scripts across endaoment, ens-primary-name, erc-8004, and veil to use `bankr prompt` instead of hardcoded script paths - Expand SKILL.md LLM Gateway section with model table, credits, OpenClaw setup, and Anthropic API mode
- Rename to OpenClaw Skills Library (formerly Clawdbot), update repo URL to BankrBot/openclaw-skills - Add 5 missing skills to table: clanker, onchainkit, endaoment, ens-primary-name, veil - Update directory tree to list all 13 skill directories
…skill-cli # Conflicts: # README.md
Add --url and --api-key flags documentation to SKILL.md for AI agents that cannot interact with terminal prompts. Update command reference table with the new login flag variants.
Document the new threadId parameter and CLI flags (--continue, --thread) that enable multi-turn conversations with the Bankr AI agent.
Remove /v1 suffix from gateway base URL and update API format docs to reflect automatic per-model configuration via bankr llm setup.
Contributor
|
@BugBot run |
Bugbot couldn't runSomething went wrong. Try again by commenting "Cursor review" or "bugbot run", or contact support (requestId: serverGenReqId_e93e6ef4-e270-4416-b1bd-e7841450b13f). |
…ent variables Restructure SKILL.md to present two integration paths (CLI and REST API), extract detailed LLM gateway docs into a dedicated reference file, and document environment variables and separate LLM key configuration.
Condense the skill description to ~400 chars (was 620), remove duplicated OpenClaw setup block, inline LLM key config instructions, and reduce verbosity across all four reference files.
Collaborator
Author
|
bugbot run |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
Collaborator
Author
Code reviewNo issues found. Checked for bugs and CLAUDE.md compliance. 🤖 Generated with Claude Code |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
curl/bash-based API calls withbankr prompt,bankr login, and other CLI commands throughout the skillbankr promptand direct REST API (/api/v1/llm/chat/completions) workflows--threadflag andX-Thread-IdheaderAffected Services
Test plan
bankr login --api-key bk_TESTworks with the documented flowbankr login --urlprints the correct dashboard URLNote
Low Risk
Documentation-heavy change with script wiring updates; main risk is breaking users who relied on removed
bankr/scripts/*.shpaths or older config locations.Overview
Updates the
bankrskill to prefer the Bankr CLI over the repo’s bash/curl wrappers: docs now center onbankr login(including non-interactive flags),bankr prompt, thread continuation, and CLI job controls, while also documenting a direct REST API alternative.Adds new documentation for the Bankr LLM Gateway (new
references/llm-gateway.md) and expands references to includethreadIdusage and CLI equivalents.Removes the legacy
bankr/scripts/*.shwrappers and updates other skills’ scripts (endaoment,ens-primary-name,erc-8004, and parts ofveil) to callbankr promptand/or require thebankrbinary (withveilkeeping a curl fallback via config). The top-levelREADME.mdis updated to list additional skills/providers and refresh the repo structure section.Written by Cursor Bugbot for commit 582c11a. This will update automatically on new commits. Configure here.