Skip to content

Update Filecoin Wallet#1476

Merged
FrederikBolding merged 2 commits intomainfrom
update-filecoin-wallet-2
Apr 22, 2026
Merged

Update Filecoin Wallet#1476
FrederikBolding merged 2 commits intomainfrom
update-filecoin-wallet-2

Conversation

@Montoya
Copy link
Copy Markdown
Contributor

@Montoya Montoya commented Apr 22, 2026

This closes #1475


Note

Low Risk
Registry-only metadata/version bump; low risk aside from ensuring the new checksum/version is correct.

Overview
Updates the npm:filsnap entry in src/registry.json by changing the listed author to Filecoin Foundation (with updated website) and removing the snap’s top-level website metadata field.

Adds a new published version 1.10.3 with its checksum to the registry.

Reviewed by Cursor Bugbot for commit 209028d. Bugbot is set up for automated code reviews on this repo. Configure here.

@Montoya Montoya requested a review from a team as a code owner April 22, 2026 03:51
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit a7697ae. Configure here.

Comment thread src/registry.json
"name": "Filecoin Foundation",
"website": "https://fil.org/"
},
"website": "https://filsnap.dev",
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Metadata-level website field accidentally removed

Medium Severity

The metadata-level website field ("website": "https://filsnap.dev") was removed, which is distinct from the author website that was correctly updated to "https://fil.org/". The snap's description still says "visit the website to connect with MetaMask and start using the Snap," but there's no longer a website field to point users to. This looks like an accidental deletion during the author block update, since the removed line was adjacent to the modified author fields.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit a7697ae. Configure here.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes, intentional. Shut up bot.

@FrederikBolding FrederikBolding enabled auto-merge (squash) April 22, 2026 10:43
@FrederikBolding FrederikBolding merged commit 43df8cf into main Apr 22, 2026
22 checks passed
@FrederikBolding FrederikBolding deleted the update-filecoin-wallet-2 branch April 22, 2026 10:50
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.

[Update Snap] Filecoin Wallet

3 participants