…ionals (#45) * feat: update landing page messaging for geographically mobile professionals - Add badge and core question "If I move to [country/city], how will I actually be standing financially?" - Update hero section to target EU free movers, expats, remote workers, consultants - Highlight key differentiators: tax-advisor depth, all countries in one place, free & community-maintained - Reframe features section as "Why This Beats Rough Cost-of-Living Tools" - Emphasize config-driven, agentic research approach - Update preset cards to "Popular Mobility Scenarios" - Update page metadata with new value proposition Closes #37 Co-authored-by: Pascal Bro <pascalwhoop@users.noreply.github.com> * fix: pass PM_AGENT_TOKEN as github_token for auto PR creation * fix: pass PM_AGENT_TOKEN as github_token in reviewer workflow * fix: increase reviewer max-turns to 30 and add allowed_tools * chore: set all agents to max-turns 100 * feat: reviewer auto-triggers /release-preview after approval * fix: deploy PR preview to per-PR worker name instead of fixed env * feat: salary mode toggle + destination rename (fixes #40, #42, #43) - Replace broken copy-all button with Same salary / Local salaries toggle - Same salary mode: gross syncs across all columns with live currency conversion - New columns auto-fill and re-convert when a country is selected - Tooltips on each mode explain when to use which - Rename "Add Country" → "Add Destination" throughout UI - Add e2e tests for salary mode toggle (tests/e2e/) - Configure Playwright on port 3938 to avoid dev server conflicts - Update developer skill with UI component and e2e testing conventions Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * fix: patch wrangler.toml name at deploy time for per-PR worker * chore: bump version to 0.2.4 [skip ci] * fix: use WRANGLER_CI_OVERRIDE_NAME to set per-PR worker name * fix: engine logic flaws and refactor module design - Fixed hardcoded German tax formula in generic bracket calculations. - Fixed bug in evaluateBracketTax where base_amount was added multiple times. - Centralized reference resolution logic in evaluators.ts and removed duplication. - Strongly typed evaluator functions and removed 'any' type casting. - Standardized custom function parameters and updated France config. - Added regression tests for all fixes. Closes #50, #51, #52, #53, #54 * fix: enable workers.dev subdomain for PR preview deployments Workers deployed via WRANGLER_CI_OVERRIDE_NAME had workers_dev disabled by default, causing 404s at the preview URL despite successful deploys. Two fixes: - Add workers_dev = true to wrangler.toml so wrangler enables it on deploy - Add explicit CF API call after deploy to ensure subdomain is enabled Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * chore: bump version to 0.2.5 [skip ci] * chore(deps): bump react-dom from 19.1.4 to 19.2.4 (#32) Bumps [react-dom](https://github.com/facebook/react/tree/HEAD/packages/react-dom) from 19.1.4 to 19.2.4. - [Release notes](https://github.com/facebook/react/releases) - [Changelog](https://github.com/facebook/react/blob/main/CHANGELOG.md) - [Commits](https://github.com/facebook/react/commits/v19.2.4/packages/react-dom) --- updated-dependencies: - dependency-name: react-dom dependency-version: 19.2.4 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chore(deps): bump @tanstack/react-query from 5.90.19 to 5.90.20 (#26) Bumps [@tanstack/react-query](https://github.com/TanStack/query/tree/HEAD/packages/react-query) from 5.90.19 to 5.90.20. - [Release notes](https://github.com/TanStack/query/releases) - [Changelog](https://github.com/TanStack/query/blob/main/packages/react-query/CHANGELOG.md) - [Commits](https://github.com/TanStack/query/commits/@tanstack/react-query@5.90.20/packages/react-query) --- updated-dependencies: - dependency-name: "@tanstack/react-query" dependency-version: 5.90.20 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * chore: bump version to 0.2.6 [skip ci] * feat: cost of living module (#38) (#49) * feat: add tax deduction framework (#27) - Extend DeductionNode schema with threshold, rate_limit, and phaseout support - Update deduction evaluator to handle threshold (above/below), cap, and phaseout - Apply input defaults in engine for optional inputs not provided by caller - Add NL 2025 deductions: mortgage interest, pension contributions, healthcare costs - Add DeductionManager UI component for managing optional deductions - Integrate DeductionManager into CountryColumn with correct number input parsing - Add deduction test vectors for NL 2025 (baseline, mortgage, pension, healthcare) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * feat: cost of living module (#38) - Add CostOfLiving type with rent, healthcare, food, mobility, travel fields - Add CostOfLivingSection collapsible UI component with per-field inputs - Show disposable income (net - annual living costs) in result summary when costs are set - Use disposable income for best-country comparison when any column has COL data - Integrate into CountryColumn below deductions, passes data through to ResultBreakdown Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: improve deduction & cost-of-living UX - Rename "Active Deductions" → "Tax Deductions" throughout - Add estimated tax saving display in deduction dialog (amount × marginal rate) - Fix mortgage_start_year default to 2025 (current tax year) - Show cost-of-living section expanded by default - Add section labels for Tax Deductions and Monthly Living Costs Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: exact deduction tax saving via engine + CHF prefix layout - Replace approximation with two real API calls: current result vs baseline (all deduction fields zeroed). Diff of net values = exact tax saving. - Extract calcRequest into useMemo in CountryColumn, shared by calculate() and DeductionManager so no request logic is duplicated. - Fix cost-of-living currency prefix overflow: replace absolute-positioned span+pl-7 with flex row, so "CHF" (and any multi-char symbol) renders correctly. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * docs: update add-new-country skill with deduction research guidance Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: NL mortgage rate cap correction + worktree test isolation - Add surtax nodes for NL 2025 mortgage rate cap (hypotheekrente aftrekbeperking) - Claws back excess benefit for top-bracket earners (49.5% capped at 37.48%) - Add test vector verifying 10k mortgage gives ~3748 saving not 4950 - Remove rate_limit from DeductionNode schema (was metadata-only, never enforced) - Exclude .worktrees and .cursor from vitest test discovery Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * fix: add input validation to prevent path traversal attacks - Add validation methods in ConfigLoader for country, year, and variant - Country: must be 2-letter lowercase ISO code (e.g., "nl", "de") - Year: must be 4-digit number (e.g., "2025") - Variant: must be alphanumeric with hyphens only (e.g., "30-ruling") - Apply validation in loadConfig, listYears, and listVariants - Add error handling in API route to return 400 for invalid inputs - Document ConfigLoader singleton pattern for security clarity This prevents attackers from using "../.." sequences to access arbitrary files on the filesystem. Co-authored-by: Pascal Bro <pascalwhoop@users.noreply.github.com> * feat: add committed settings.json to pre-approve CI agent bash commands Without a committed .claude/settings.json, Claude Code's permission prompt blocks gh pr review and gh issue comment in non-interactive CI mode, causing the reviewer agent to run but never post its output. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * Revert "feat: add committed settings.json to pre-approve CI agent bash commands" This reverts commit a9147641754b2b1e5fdffbc9a644bb4847032391. * fix: move allowed_tools to claude_args for v1.0 compatibility In claude-code-action v1.0, `allowed_tools` was renamed to `--allowedTools` passed via `claude_args`. The old `allowed_tools` parameter was a beta-era input that is no longer applied, causing Bash commands like `gh pr review` to hit approval prompts that silently fail in non-interactive CI mode. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: trigger claude review on /review comment instead of every push Replaced the automatic pull_request trigger with an issue_comment trigger so the review only runs when a maintainer explicitly comments /review on a PR. Avoids burning tokens on every branch push. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * fix: review feedback — COMPOUND_DEDUCTIONS, currency fallback, headers, magic number - Replace hardcoded COMPOUND_DEDUCTIONS map in UI with config-driven group field on secondary inputs; deduction manager derives groupings from def.group - Add group field to NumberInput schema type and InputDefinition API type - Extract top bracket threshold 76817 as named parameter top_bracket_threshold - Fix currency fallback in DeductionDialog: drop calcRequest.country fallback - Remove duplicate Tax Deductions inner label from DeductionManager - Default CostOfLivingSection to collapsed - Add inputDefs to useEffect dependency array in DeductionDialog Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com> Co-authored-by: Pascal Bro <pascalwhoop@users.noreply.github.com> * feat: replace [country/city] placeholder with animated city flip in hero Uses Aceternity UI LayoutTextFlip component to cycle through real city names (Amsterdam, Berlin, Zurich, London, etc.) instead of the static [country/city] placeholder text. Installs motion dependency. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com> Co-authored-by: Pascal Bro <pascalwhoop@users.noreply.github.com> Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

[Tech Debt] Remove any type casting in evaluators.ts #53

Description

Metadata

Metadata

Assignees

Labels

Projects

Milestone

Relationships

Development

Issue actions