Skip to content

Conversation

@yungwine
Copy link
Collaborator

No description provided.

@yungwine yungwine requested a review from Copilot July 15, 2025 09:09
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

A series of updates to integrate a new “BTC Teleport” oracle module, including installation/uninstallation scripts, service configuration adjustments, and wiring the module into existing flows.

  • Add btc_teleport service stop/remove steps and related uninstall scripts
  • Extend mytoninstaller and mytonctrl to initialize, configure, and register the BTC Teleport module
  • Introduce new modules/btc_teleport.py, adjust systemd helper, and update various lifecycle hooks

Reviewed Changes

Copilot reviewed 16 out of 16 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
scripts/uninstall.sh Stop and remove btc_teleport service and cleanup its files
mytoninstaller/settings.py Enable “enable_btc_teleport” when restoring from backup
mytoninstaller/scripts/jsonrpcinstaller.sh Removed automatic restart of mtc-jsonrpc service
mytoninstaller/scripts/add2systemd.sh Added -w (working dir) flag and systemctl restart on install
mytoninstaller/mytoninstaller.py Fallback to global config when init_block is missing
mytonctrl/scripts/remove_btc_teleport.sh New script to remove BTC Teleport periphery and service
mytonctrl/scripts/btc_teleport*.sh New install scripts building and installing Rust/Go components
mytonctrl/mytonctrl.py Inject BTC Teleport console commands, upgrade hook, status print
mytoncore/mytoncore.py Import and auto-initialize BTC Teleport in validator mode
mytoncore/functions.py Add event handler and auto-vote loop for BTC Teleport proposals
modules/module.py Updated type annotation for local
modules/btc_teleport.py New module implementing BTC Teleport functionality
Comments suppressed due to low confidence (2)

modules/module.py:14

  • [nitpick] Annotating local as MyPyClass may be misleading if that class doesn’t match the actual interface. Consider using a protocol or removing the annotation to avoid confusion.
        self.local: MyPyClass = local

mytonctrl/mytonctrl.py:333

  • The Upgrade signature changed from (ton, args) to (local, ton, args). Existing calls may still pass two arguments, causing a mismatch. Verify and update all invocations.
def Upgrade(local, ton, args: list):

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@yungwine yungwine merged commit 862e9cc into ton-blockchain:dev Jul 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant