Skip to content

Comments

Add community ability: Crypto SOL Wallet (Solana/Phantom)#108

Open
codeforstartups wants to merge 3 commits intoopenhome-dev:devfrom
codeforstartups:add-crypto-sol-wallet
Open

Add community ability: Crypto SOL Wallet (Solana/Phantom)#108
codeforstartups wants to merge 3 commits intoopenhome-dev:devfrom
codeforstartups:add-crypto-sol-wallet

Conversation

@codeforstartups
Copy link
Contributor

What does this Ability do?

Suggested Trigger Words

Type

  • New community Ability
  • Improvement to existing Ability
  • Bug fix
  • Documentation update

External APIs

  • No external APIs
  • Uses external API(s):

Testing

  • Tested in OpenHome Live Editor
  • All exit paths tested (said "stop", "exit", etc.)
  • Error scenarios tested (API down, bad input, etc.)

Checklist

  • Files are in community/my-ability-name/
  • main.py follows SDK pattern (extends MatchingCapability, has register_capability + call)
  • README.md included with description, suggested triggers, and setup
  • resume_normal_flow() called on every exit path
  • No print() — using editor_logging_handler
  • No hardcoded API keys — using placeholders
  • No blocked imports (redis, connection_manager, user_config)
  • No asyncio.sleep() or asyncio.create_task() — using session_tasks
  • Error handling on all external calls
  • Tested in OpenHome Live Editor

Anything else?

@codeforstartups codeforstartups requested a review from a team as a code owner February 21, 2026 07:37
@github-actions
Copy link
Contributor

github-actions bot commented Feb 21, 2026

🔀 Branch Merge Check

PR direction: add-crypto-sol-walletdev

Passedadd-crypto-sol-walletdev is a valid merge direction

@github-actions
Copy link
Contributor

github-actions bot commented Feb 21, 2026

❌ Ability Validation Failed

📋 Validating: community/crypto-sol-wallet
  ❌ raw open() is not allowed — use capability_worker file helpers (read_file, write_file, etc.) instead
  ❌ Missing register capability tag — add the following line to your class:
    #{{register capability}}
See: https://docs.openhome.com/how_to_build_an_ability
  ❌ 2 error(s) found

📚 How to fix

@github-actions github-actions bot added first-contribution First-time contributor community-ability Community-contributed ability and removed first-contribution First-time contributor labels Feb 21, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 21, 2026

🔍 Lint Results

__init__.py — Empty as expected

Files linted: community/crypto-sol-wallet/main.py

❌ Flake8 Errors (could not be auto-fixed)

community/crypto-sol-wallet/main.py:491:38: E999 SyntaxError: '(' was never closed

Fix the remaining issues and push again. The lint will re-run automatically.

@github-actions github-actions bot added the first-contribution First-time contributor label Feb 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-ability Community-contributed ability first-contribution First-time contributor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants