Dev - #111
Merged
Merged
Conversation
- Add getAllBackendGatewayUrls() and getBackendPeerId() helpers to ipfs.config - Configure ipfs1 peer ID for WSS connection - Implement direct API upload to backend nodes via /api/v0/add - Add backend connection maintenance with auto-reconnect - Create NostrPinPublisher service for broadcasting pin requests - Integrate NostrPinPublisher into WalletGate on authentication - Add peer connection/disconnection logging for debugging The browser Helia node now: 1. Uploads content directly to all configured backend nodes 2. Publishes pin request events to Nostr for network-wide pinning 3. Maintains persistent WSS connection to backend for bitswap 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
- Add IPNS publishing with proper PeerId-based names
- Add IPNS resolution via DHT routing with unmarshalIPNSRecord
- Implement syncFromIpns() for startup sync with remote discovery
- Add version-based merge: remote > local → import, local > remote → publish
- Add fetchRemoteContent() and importRemoteData() helpers
- Migrate storage keys from old ipns-{hex} format to new PeerId format
- Track IPNS sequence numbers for monotonic record updates
- Add CID comparison before IPNS publish to avoid unnecessary DHT writes
- Handle interrupted syncs by always verifying remote on startup
- Fallback to syncNow() when remote fetch fails
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Phase 1 critical fixes for IPFS/IPNS sync reliability: - Add tombstones to TXF format to prevent zombie token resurrection - Implement SyncCoordinator with BroadcastChannel for tab coordination - Add leader election to ensure only one tab syncs at a time - Create retry utility with exponential backoff for IPFS operations - Track pending IPNS publishes for retry on failure - Update importRemoteData to handle tombstones and remove deleted tokens - Add deviceId to TxfMeta for conflict resolution New files: - SyncCoordinator.ts: Tab coordination via BroadcastChannel - src/utils/retry.ts: Exponential backoff retry utility 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Add hybrid IPNS publishing strategy for improved reliability: - Primary: HTTP POST to Kubo backend via /api/v0/routing/put - Uses allow-offline=true for fast response - 30s timeout with proper error handling - Fallback: Fire-and-forget browser DHT via Helia - Runs async in background, doesn't block sync - 60s background timeout Both paths publish the same signed IPNS record, providing redundancy while keeping sync operations fast. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
- Add IpnsUtils.ts for deriving IPNS names from private keys without full IpfsStorageService initialization - Add IpnsNametagFetcher.ts for lightweight HTTP gateway fetching of nametag data using DAG-JSON format - Update CreateWalletFlow.tsx to derive 10 addresses and async fetch nametags from IPNS in parallel during address selection - Add WalletRepository.saveNametagForAddress() to persist IPNS-fetched nametags to localStorage before page reload This enables users importing wallets in fresh browsers to see their existing nametags in the address selection dropdown. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Reduce console clutter by only logging connect/disconnect events for bootstrap peers (custom IPFS nodes and libp2p.io bootstrap nodes). Non-bootstrap DHT peer churn is silently ignored. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Add alert message when clicking recovery phrase button for wallets imported from file. Explains that mnemonic cannot be derived from master key (one-way BIP39 derivation). 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Configure actual peer IDs for unicity-ipfs2 through unicity-ipfs5, enabling browser DHT connections to all 5 Unicity IPFS nodes. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
…ct detection - Add resolveIpnsProgressively() to query all 5 gateways in parallel - Compare IPNS record sequence numbers to detect conflicts - Auto-merge when late-arriving response has higher sequence - Add tab-aware IPNS polling (45-75s interval with jitter) - Pause polling when tab is hidden to save resources - Increase timeouts for cold DHT nodes (10s initial, 25s per-gateway) - Fix IPFS auto-sync race condition in useWallet hook 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add connection gater to restrict libp2p to bootstrap peers only - Reduce fallback from 5 public peers to 1 emergency fallback - Reduce maxConnections from 50 to 10 - Remove timestamp from TXF metadata for CID stability - Fix token ID comparison: use genesis token ID instead of UUID - Simplify conflict resolution: local wins on version tie This significantly reduces browser traffic by preventing connections to random DHT-discovered peers while ensuring no unnecessary IPNS publishes when wallet content hasn't changed. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
- Add Unicity-based verification before accepting tombstones from remote - Archive tokens on every addToken() call for recovery purposes - Use SDK's isTokenStateSpent() for proper spent status verification - Load trust base from local ServiceProvider instead of HTTP fetch - Handle missing tokens (absent from remote without tombstone) - Restore unspent tokens from archive if tombstone is invalid - Add null-safe handling for spent token logging - Add integrity verification after sync operations This prevents token loss when remote devices have stale tombstones or skip versions during IPNS sync. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Critical fixes for IPNS sync: - Add pendingSync flag to ensure tokens added during sync are synced after - Fix version comparison to prioritize committed over pending transactions - Add IPNS sequence fix: use max(local, remote) + 1 to prevent overwrites Token validation improvements: - Add isPendingTransactionSubmittable() to detect dead pending transactions - Add validateSplitTokens() to verify split token burn was committed - Add ValidationAction type (ACCEPT, RETRY_LATER, DISCARD_FORK) New TokenBackupService for encrypted local backups: - AES-256-GCM encryption with PBKDF2 key derivation - Backup status monitoring with sync timestamp tracking - Support for file download and localStorage backup WalletRepository additions: - Add archived token storage for token history preservation - Add forked token tracking for conflict resolution 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
IPNS polling now also checks for CID differences when sequence numbers are equal. This fixes a race condition where two devices publish with the same sequence number but different content - the polling would incorrectly report "no updates" because it only checked sequence > local. Now triggers sync when: - Remote sequence > local sequence (existing behavior), OR - Remote CID differs from local CID at same/higher sequence (new) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Documents current architecture, completed fixes, and 6 real-time sync options for future implementation: - IPNS over PubSub (native IPFS) - Custom PubSub channel - WebSocket from backend - Server-Sent Events (SSE) - Nostr-based notifications - Hybrid approach (recommended) Includes implementation phases, metrics, and testing checklist. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Merges dev branch features: - Convert onboarding modals to fullscreen views - Replace L1 inline send form with modal - Add restore method selection modal - Integrate wallet import with L3 onboarding flow - Add CI/CD workflow (build, lint, test) - Add mixpanel analytics and KaTeX math rendering - Various UI improvements Preserves IPFS branch features: - IPNS nametag fetching during wallet import - Wallet clearing before create/import operations - Tombstone, archived, and forked token loading - All IPNS sync improvements 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Continue IPNS sync when tab is inactive but at slower rate - Active tab: 45-75 second polling interval - Inactive tab: 4-4.5 minute polling interval (reduces battery/resource usage) - Visibility changes trigger immediate interval adjustment 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
When restoring a wallet, the address selection screen was using the L1
wallet's private key (from scanned addresses) to derive the IPNS name
for nametag lookup. However, nametags are published using the L3 identity
key from UnifiedKeyManager.
The L1 wallet may have addresses from different BIP32 paths (e.g.,
m/84'/1'/0'/0/0) while the L3 identity always uses m/44'/0'/0'/0/{index}.
Different paths = different private keys = different IPNS names.
This fix ensures IPNS lookup uses l3Identity.privateKey (from
UnifiedKeyManager) instead of addr.privateKey (from L1 wallet scan),
so cross-browser nametag discovery works correctly.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add proper types to NametagFetchResult.data interface - Remove unused catch variable 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Previously, wallets with only a nametag (no tokens) would not sync to IPFS because: 1. Early exit condition only checked token count, ignoring nametag 2. localDiffersFromRemote() only compared tokens, not nametags Now: - Early exit skips only if no tokens AND no nametag - localDiffersFromRemote() returns true if local nametag differs from remote This ensures nametags sync reliably for cross-device discovery. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add JSON wallet format v1.0 with AES-256 encryption (PBKDF2 100k iterations) - Export includes mnemonic phrase, master key, chain code, and verification address - Import supports both encrypted and unencrypted JSON files - Mnemonic-based imports restore wallet directly without scan modal - Reset address index on import to prevent stale index from creating extra addresses - Update SaveWalletModal to JSON-only export format - Update ImportWalletModal to accept .json files - Add exportToJSON/downloadJSON methods to UnifiedKeyManager - Add JSON import support to L3 onboarding flow 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- JSON files with mnemonic no longer show "How many addresses to scan?" - Derive L3 identity from UnifiedKeyManager instead of requiring privateKey in wallet - Reset address index to 0 and save wallet before address selection - Apply fix to both L1 ImportWalletModal and L3 CreateWalletFlow 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Allow pasting entire 12-word recovery phrase into any input field. Words are automatically split and distributed across all fields. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add useAddressNametags hook to fetch nametags from IPNS - Uses sequential array position for L3 identity derivation (matching CreateWalletFlow) - Show nametags with @ prefix in selected address and dropdown - Display loading spinner while resolving IPNS names - Update individual addresses as IPNS results arrive 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add IPNS nametag fetch when identity exists but no local nametag - Show loading state while checking for existing Unicity ID - Auto-proceed to wallet if nametag found in IPNS - Saves fetched nametag to localStorage for persistence 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
L1 Wallet Dropdown: - Simplify useAddressNametags hook with single combined effect - Use refs to prevent duplicate fetches in React Strict Mode - Immediately fetch nametags for new addresses (+ button) - Continuous polling with backoff for addresses without nametags Complete Setup Screen: - Keep "Continue Setup" button visible while checking IPNS - Show loading indicator below button during nametag check - Auto-proceed to wallet if existing nametag found 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Resolved conflict in useWallet.ts by keeping both: - IPFS storage initialization (our branch) - Registry query (from main) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
When IPNS records expire (24h TTL) but browser has local CID, the sync would not republish IPNS because the CID hadn't changed. This left users unable to sync across devices. Changes: - Add forceIpnsPublish option to syncNow() for recovery scenarios - Detect ipnsNeedsRecovery when IPNS resolution fails but local CID exists - Force IPNS republish in all sync paths when recovery is needed - Prevent early return in conflict check when IPNS recovery required Also improves CreateWalletFlow: - Add processing status messages during nametag minting - Verify IPNS is resolvable after sync with retry logic - Ensure nametag propagates to IPFS before completing setup 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
When switching between wallet addresses, IpfsStorageService was using cached Ed25519 keys from a previously selected address, causing nametags to be synced to the wrong IPNS name. This resulted in silent data loss for addresses 0 and 1 when address 2 was initialized first. Changes: - Add currentIdentityAddress field to track which identity is initialized - Modify ensureInitialized() to detect identity changes before early return - Clear cached cryptographic keys when identity changes, forcing re-derivation - Keep Helia instance alive to avoid expensive reinitialization - Add logging for identity address during IPFS initialization 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
In goToAddressSelection(), the code was using the sequential loop index for L3 derivation instead of the L1 address's actual index. This caused nametags to be mismatched when change addresses were present in the list. Changes: - Filter out change addresses (isChange=true) since they don't have L3 identities - Use addr.index instead of sequential loop index for L3 derivation - This ensures nametag lookup matches the IPNS prefetch from scan window Example fix: - Before: [Address#0, Change#0, Address#1] → L3[0,1,2] → wrong nametags - After: [Address#0, Address#1] → L3[0,1] → correct nametags 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Thread isChange parameter through L3 derivation chain so external and change addresses derive DIFFERENT L3 identities (different BIP32 chain): - External #N derives L3 from m/44'/0'/0'/0/N - Change #N derives L3 from m/44'/0'/0'/1/N Changes: - UnifiedKeyManager.deriveAddress(index, isChange) - pass isChange to BIP32 - IdentityManager.deriveIdentityFromUnifiedWallet(index, mnemonic, isChange) - useAddressNametags: track isChange, pass to L3 derivation - scan.ts: pass isChange to getCachedL3Info for correct L3 lookup - CreateWalletFlow: show ALL addresses (not just external), add Change badge - MainWalletView: add Change badge, sort addresses (external first) Each L1 address now has its own unique L3 identity and can have its own independent nametag. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Fix session key mismatch in scan.ts that prevented IPNS lookups (changed "scan-session" to "user-pin-1234" for consistent key derivation) - Display discovered nametags in WalletScanModal with purple badge - Remove debug logging from IpnsNametagFetcher - Add IpfsStorageService.resetInstance() to allow identity switching - Reset IPFS service when user selects different address in CreateWalletFlow 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Allow users to load scanned addresses without waiting for IPNS nametag resolution (which can take up to 30s). The button now appears as soon as L1 balance scanning finishes, while Unicity ID resolution continues in the background with status "Resolving Unicity IDs...". - Add l1ScanComplete flag to ScanProgress interface - Signal completion after main loop, before IPNS await - Update WalletScanModal to show button when l1ScanComplete is true 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
IPNS records were expiring after 24 hours because the validity timestamp embedded in the signed record was too short. The pinner was republishing expired records to the DHT, but DHT peers reject records with past validity. Changed IPNS_LIFETIME from 24 hours to 99 years to ensure records remain valid for extended periods. Users need to open their wallet once to publish fresh IPNS records with the new validity period. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
…L1 scan Two issues fixed: 1. Stop Scan button now immediately updates UI by setting isScanning=false 2. Load Selected button stays visible after L1 scan completes, even when background IPNS callbacks fire (uses latching l1Complete state) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add large overlay spinner on token/asset list during initial IPFS sync - Add small spinner next to balance during any sync operation - Add sync:state-changed event for real-time React state updates - Track isInitialSyncing flag separately from isSyncing for better UX - Fix isEnabled always returning false by removing env var check (useWallet starts IPFS service unconditionally) - Add hasSyncStarted ref to prevent premature initialSyncComplete 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Reflects the decentralized P2P nature of IPFS storage - fog computing concept vs centralized cloud infrastructure. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Race gateway path (~30ms) and routing API (~5s) in parallel for IPNS resolution. Whichever method responds first provides the result. Changes: - Add resolveIpnsViaGatewayPath() for fast gateway path resolution - Modify resolveIpnsProgressively() to race both methods per gateway - Cache content from gateway path to avoid re-fetch in syncFromIpns() - Update IpnsNametagFetcher to race both methods with Promise.any() - Add gatewayPathTimeoutMs config (5s) to ipfs.config.ts - Add TODO for future IPNS archiving service in kubo docker Performance improvement: - First IPNS resolution: ~5s -> ~30ms - Sequence accuracy maintained via routing API catchup 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Change tokens from split operations were being lost during IPFS sync because the SDK's toJSON() output was missing required TXF fields. Changes: - Add TXF structure validation in saveChangeTokenToWallet - Normalize SDK token JSON with required fields (version, transactions, nametags, _integrity) before saving - Add enhanced logging for TXF validation failures in tokenToTxf - Log token ID when saving change tokens for debugging Root cause: tokenToTxf() validation rejected tokens without genesis/state fields, causing them to be skipped during IPFS sync and sanity checks. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
IPFS is now the primary source of truth for token storage. When a token is received via Nostr: 1. Token is saved to localStorage first 2. IPFS sync is awaited before marking the event as processed 3. If sync is already in progress, wait for it to complete then sync again 4. Only mark event as processed after IPFS sync succeeds This ensures tokens are never lost - if IPFS sync fails, the Nostr event remains unprocessed and will be redelivered on next connect. Changes: - Add waitForSyncCompletion() helper that listens for storage:completed event - Modify handleSubscriptionEvent to await IPFS sync before markEventAsProcessed - Handle "Sync already in progress" by waiting and retrying - 60-second timeout prevents infinite waiting 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Implements the Outbox pattern to prevent token loss during transfers: - OutboxRepository: localStorage persistence of pending transfers - OutboxTypes: Entry interfaces, status enum, and lifecycle utilities - OutboxRecoveryService: Startup recovery + periodic retry every 60s - TokenSplitExecutor: Outbox tracking for split operations - TxfSerializer/IpfsStorageService: Outbox entries in IPFS sync - useWallet: Recovery effect with cleanup on unmount Features: - Exponential backoff (30s base, 1h max) between retries - No age limit for pending entries (users may close app for days) - 10 consecutive failures before marking as FAILED - COMPLETED entries cleaned up after 24h 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
L1 SDK improvements: - Enhanced network.ts with better WebSocket handling and RPC reliability - Added addressHelpers.ts for address utility functions - Extended types.ts with additional wallet interfaces - Improved unifiedWalletBridge.ts for better L1/L3 integration Wallet infrastructure: - Refactored useAddressNametags hook for cleaner nametag resolution - Updated IdentityManager with improved key derivation flow - Enhanced UnifiedKeyManager for cross-layer key management - Added checkTokensForAddress to WalletRepository UI/UX: - Updated L1WalletView with improved scanning behavior - Minor fixes in SeedPhraseModal and L3WalletView Documentation: - Updated CLAUDE.md with latest project context 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add isNametagAvailable method to NametagService to check if nametag is already minted - Add checkNametagAvailability to useWallet hook - Validate nametag availability before minting (prevents duplicate nametags) - Add input validation: only lowercase letters, numbers, and special chars (_-+.) - Improve user experience with immediate feedback on nametag conflicts 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Resolved conflicts by integrating both branches' changes: - Kept IPFS storage initialization and recovery service from @cryptohog/ipfs - Added nametag validation (checkNametagAvailability) from dev - Integrated input validation (lowercase, allowed chars) from dev - Preserved detailed logging and error handling from @cryptohog/ipfs Changes from dev: - Nametag availability check before minting - Input validation for nametag (only a-z, 0-9, _-+.) - Updated activities.ts (Viktor greeting) - Enhanced markdown.tsx parsing 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
The CreateWalletFlow uses UnifiedKeyManager directly for wallet restoration, so the restoreWallet mutation from useWallet hook is not needed. This fixes TypeScript unused variable warning. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Feature/welcome modal
feat: IPFS/IPNS sync with multi-node upload and cross-browser nametag discovery
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
4 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.
No description provided.