Skip to content

fix: ci constantly failing because of @acutal-app/api#13317

Merged
AbdulTheActivePiecer merged 3 commits into
mainfrom
fix/ci-constant-fail
May 20, 2026
Merged

fix: ci constantly failing because of @acutal-app/api#13317
AbdulTheActivePiecer merged 3 commits into
mainfrom
fix/ci-constant-fail

Conversation

@AbdulTheActivePiecer
Copy link
Copy Markdown
Collaborator

No description provided.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 20, 2026

Confidence Score: 5/5

Safe to merge — the code change is a well-placed defensive guard, and the dependency upgrade resolves a CI breakage with no observable functional regression in the piece logic.

The only code change is a correct early-exit guard that improves input handling. The major-version dependency bump was already discussed in a prior review thread and brings no changes to the called API surface (api.importTransactions, api.shutdown, initializeAndDownloadBudget) that would break the piece at runtime.

No files require special attention beyond what was already raised in prior review threads about the major version bump.

Important Files Changed

Filename Overview
packages/pieces/community/actualbudget/src/lib/actions/import-transactions.ts Adds an early Array.isArray guard before initializing the budget API, correctly preventing unnecessary network calls on bad input
packages/pieces/community/actualbudget/package.json Bumps @actual-app/api from 25.3.1 to 26.5.2; crosses a major version boundary and pulls in a new @actual-app/core sub-package with significant new transitive dependencies
bun.lock Lock file updated to reflect the new @actual-app/api@26.5.2 resolution and all its transitive dependencies

Reviews (3): Last reviewed commit: "Merge branch 'main' into fix/ci-constant..." | Re-trigger Greptile

Comment thread packages/pieces/community/actualbudget/package.json
@AbdulTheActivePiecer AbdulTheActivePiecer merged commit 0fe48da into main May 20, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant