whisper-money

Commit Graph

Author	SHA1	Message	Date
Víctor Falcón	fd67cf7c72	fix(automation): avoid rule preview n+1 (#431 ) ## Sentry issue - PHP-LARAVEL-2F: https://whisper-money.sentry.io/issues/122581787/ ## Root cause - Automation rule match preview eagerly loaded account, bank, category, and labels for every 500-transaction chunk, even for rules that only inspect description fields. - Sentry flagged repeated account/bank eager-load queries across chunks as an N+1 pattern on `/settings/automation-rules/{automationRule}/matches`. ## Fix - Detect variables used by an automation rule and eager load only relationships required for evaluation. - Keep label eager loading only when label-only skip logic needs it. - Avoid lazy loading unused relationships while preserving full data shape for rule evaluation. - Update the Sentry prompt to use the `sentry` CLI workflow. ## Verification - `vendor/bin/pint --dirty --format agent` - `php artisan test --compact tests/Feature/AutomationRuleApplicationTest.php` - `php artisan test --compact tests/Feature/AutomationRuleEvaluationTest.php tests/Feature/AutomationRuleApplicationTest.php`	2026-05-26 08:02:46 +02:00
Víctor Falcón	c929c1f7a5	chore: Add sentry issue slash command (#375 )	2026-05-10 17:43:39 +01:00
Víctor Falcón	c7cfa10117	fix: expose pi mcp extension as mcps.ts (#315 ) ## Summary - add project-local `mcps.ts` extension entry for Pi auto-discovery - move MCP bridge implementation into `.pi/extensions/mcps/main.ts` - ignore nested `node_modules/` directories so extension package installs do not dirty repo ## Testing - `node -e "const createJiti=require('/opt/homebrew/lib/node_modules/@mariozechner/pi-coding-agent/node_modules/@mariozechner/jiti/lib/jiti.cjs'); const jiti=createJiti(process.cwd()); const mod=jiti('./.pi/extensions/mcps.ts'); if (typeof mod.default !== "function") process.exit(1); console.log("ok");"`	2026-04-22 09:51:20 +01:00

Author

SHA1

Message

Date

Víctor Falcón

fd67cf7c72

fix(automation): avoid rule preview n+1 (#431 )

## Sentry issue
- PHP-LARAVEL-2F: https://whisper-money.sentry.io/issues/122581787/

## Root cause
- Automation rule match preview eagerly loaded account, bank, category,
and labels for every 500-transaction chunk, even for rules that only
inspect description fields.
- Sentry flagged repeated account/bank eager-load queries across chunks
as an N+1 pattern on
`/settings/automation-rules/{automationRule}/matches`.

## Fix
- Detect variables used by an automation rule and eager load only
relationships required for evaluation.
- Keep label eager loading only when label-only skip logic needs it.
- Avoid lazy loading unused relationships while preserving full data
shape for rule evaluation.
- Update the Sentry prompt to use the `sentry` CLI workflow.

## Verification
- `vendor/bin/pint --dirty --format agent`
- `php artisan test --compact
tests/Feature/AutomationRuleApplicationTest.php`
- `php artisan test --compact
tests/Feature/AutomationRuleEvaluationTest.php
tests/Feature/AutomationRuleApplicationTest.php`

2026-05-26 08:02:46 +02:00

Víctor Falcón

c929c1f7a5

chore: Add sentry issue slash command (#375 )

2026-05-10 17:43:39 +01:00

Víctor Falcón

c7cfa10117

fix: expose pi mcp extension as mcps.ts (#315 )

## Summary
- add project-local `mcps.ts` extension entry for Pi auto-discovery
- move MCP bridge implementation into `.pi/extensions/mcps/main.ts`
- ignore nested `node_modules/` directories so extension package
installs do not dirty repo

## Testing
- `node -e "const
createJiti=require('/opt/homebrew/lib/node_modules/@mariozechner/pi-coding-agent/node_modules/@mariozechner/jiti/lib/jiti.cjs');
const jiti=createJiti(process.cwd()); const
mod=jiti('./.pi/extensions/mcps.ts'); if (typeof mod.default !==
"function") process.exit(1); console.log("ok");"`

2026-04-22 09:51:20 +01:00

3 Commits