Add SimpleFin Sync as a verified community addon - #9
Conversation
…c49a32bc0a5aa5444)
|
Updated the listing to v1.3.0 (release, tag Only the six version-carrying fields changed — Highlights since v1.0.1:
419 tests, clean type-check. |
|
Bumped to v1.3.1 (release) — fixes a flaw in v1.3.0's |
Its `commit` still named the 1.8.2 commit and its `notes` still described the emoji-picker fix, so the listing would have advertised 1.9.0 while resolving to older source. This copy is known to drift from the one in the store PR (wealthfolio/wealthfolio-addons#9); both get updated, and the PR's is the authority. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Adds automatic categorization of Amazon charges from their order emails, and fixes two import bugs: a refused row could discard a whole account's batch, and a heal run reported phantom balance drift equal to a refused row's amount. Accounts that could not be verified now say "not checked" rather than showing as in sync. Release: https://github.com/Bubbles840/wealthfolio-simplefin-addon/releases/tag/v1.9.0
Also aligns source.tag, source.commit and changelogUrl with the released version — they had been left at v1.12.0 by the previous bump.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
The Verified Community tier this PR targeted no longer exists. Moves the entry under community/directory, drops the fields that only made sense when Wealthfolio built and hosted the artifact, and removes the cover art, which community listings do not use. Adds the commercial model, since SimpleFIN needs an account and no repository reveals that, plus the derived record read from the publisher's repository.
|
Thanks for this, and sorry it sat so long. I introduced the addon sandbox in 3.6 and reworked how community listings work before getting to the queue, so this needed converting rather than just merging. The short version: I've dropped the Verified Community tier. Building, hosting and standing behind other people's code isn't something I can do properly on my own, and a "verified" badge that isn't a real code audit misleads people. So there are two channels now. Mine, in the app. Everyone else's, in a directory that links to your repo, where people download and install with "Install from File". I've pushed the conversion to your branch:
You don't declare data handling any more. I read it from your manifest: the network permission plus One thing I do need from you. Your README describes Amazon categorisation running in the companion container, which reads a mailbox. That's outside the addon, so nothing in the manifest can show it, and the listing would understate what the full setup does. Could you add: "privacyUrl": "https://github.com/Bubbles840/wealthfolio-simplefin-addon/blob/main/PRIVACY.md",
"dataHandling": {
"leavesDevice": true,
"externalServices": [
{ "name": "Your mail provider, read by the companion container", "url": "https://..." }
]
}Correct that to whatever is actually true. I'd rather your listing say it than have me guess. Then post this as a comment and I'll merge: I have read POLICIES.md and confirm:
- [ ] I am the publisher of this addon, or authorised to act for them.
- [ ] I have the rights to the name and description, and grant Wealthfolio permission to display them.
- [ ] The repository is public and has a licence file.
- [ ] What I declared is accurate, and my manifest declares every host the addon reaches.
- [ ] There is no hidden data collection or undisclosed remote code.
- [ ] I own support, updates, security fixes, privacy, licensing, and commercial terms.
- [ ] The name doesn't imply the addon is official or endorsed.
- [ ] A listing is a link only. Wealthfolio doesn't host, audit, endorse, or support it, and may remove it. |
Adds privacyUrl and dataHandling, as asked for in the PR. Corrected to what the code actually does rather than filled in from the template: - SimpleFIN Bridge — required, supplies the transactions. - Telegram — only when the user configures notifications. - The user's own mail provider, over IMAP, read by the COMPANION container for the optional Amazon categorisation. This is the one the manifest can never show, since it happens outside the addon sandbox. leavesDevice is true: transactions come from SimpleFIN, and both of the optional integrations send or read data off-device. privacyUrl points at a new PRIVACY.md in the addon repo (Bubbles840/wealthfolio-simplefin-addon@be8e530), which spells out what each service receives, that bank credentials are never handled (SimpleFIN's setup token is exchanged once for an access URL held in Wealthfolio's encrypted secret storage), that the mailbox reader keeps only an order's total, date and Amazon's own category label, and that there is no telemetry. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Thanks for converting it rather than bouncing it back — and the reasoning for dropping the Verified tier makes sense to me. Standing behind other people's code is a real commitment and it's better not to imply one that isn't there. I've added the two pieces you asked for, corrected to what the setup actually does:
The privacy doc spells out what each service receives: bank credentials are never handled (SimpleFIN's setup token is exchanged once for an access URL, held in Wealthfolio's encrypted secret storage), the mailbox reader keeps only an order's total, date and Amazon's own category label — not message bodies — and there is no telemetry or analytics anywhere in the addon or the companion. Validated against the repo's own I have read POLICIES.md and confirm:
|
Submitting SimpleFin Sync for the verified community tier.
What it does
Imports bank and credit-card transactions from SimpleFin into Wealthfolio. SimpleFin is read-only, so the addon can never move money.
Beyond plain import:
TRANSFER_OUT/TRANSFER_IN, preventing in-transit transfers from inflating spending.TRANSFER_OUTrows are accounted for during drift calculations so in-flight transfers don't flag false balance drift warnings.Details
v1.0.1, commitda86f5c8b211f42111ef59ca8c2278c2e6462fe8tsc --noEmitcleanPermissions
Each is declared with a purpose in
manifest.json:accounts(getAll, create)activities(checkImport, import, search, getAll, create, update, saveMany)portfolio.getLatestValuationssecretsnetwork.requestallowedHostsnavigation.navigateui