fix objects in logs

[bordalix]

Summary by CodeRabbit

• Chores

  • Updated dependency resolution strategy for the SDK package.
  • Improved internal logging mechanisms.

• Improvements

  • Enhanced wallet management with automatic virtual transaction output renewal to maintain wallet state.

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

The PR switches the SDK dependency to a local file reference and integrates VtxoManager into the wallet provider to renew vtxos after wallet initialization, while updating logging calls in the ASP module from a custom logger to native console methods.

Changes

Cohort / File(s)	Summary
Dependency Management package.json	Updated @arkade-os/sdk from version 0.3.1-alpha.4 to local file reference file:../ts-sdk
Logging Updates src/lib/asp.ts	Replaced consoleLog import with consoleError and changed settleVtxos to use native console.log instead of the custom logger
Vtxo Manager Integration src/providers/wallet.tsx	Added VtxoManager import and instantiated it with the service worker wallet; calls renewVtxos() after wallet status ping setup with error suppression

Sequence Diagram

sequenceDiagram
    participant App
    participant WalletProvider
    participant ServiceWorker
    participant VtxoManager

    App->>WalletProvider: Initialize wallet provider
    WalletProvider->>ServiceWorker: Set up wallet and status ping
    WalletProvider->>VtxoManager: Instantiate with SW wallet
    
    rect rgba(76, 175, 80, 0.1)
        Note over VtxoManager: New flow
        VtxoManager->>VtxoManager: renewVtxos()
        opt Error occurs
            VtxoManager->>VtxoManager: Suppress error silently
        end
    end
    
    WalletProvider->>App: Wallet ready

Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• fix vtxos reload after settle in coin control #160: Adds complementary vtxo handling features including getVtxos and reloadWallet integration that work alongside this PR's VtxoManager renewal functionality.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix_objects_in_logs

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3ca0a3e and 58e3846.

⛔ Files ignored due to path filters (1)

• pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml

📒 Files selected for processing (3)

• package.json (1 hunks)
• src/lib/asp.ts (2 hunks)
• src/providers/wallet.tsx (2 hunks)

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[cloudflare-workers-and-pages]

Deploying wallet-signet with    Cloudflare Pages

Latest commit:	58e3846
Status:	🚫  Build failed.

View logs

[cloudflare-workers-and-pages]

Deploying wallet-mutinynet with    Cloudflare Pages

Latest commit:	58e3846
Status:	🚫  Build failed.

View logs