Skip to content

chore(example): bump MYS contract_details schema version to 5 - #1561

Merged
gabrielseco merged 2 commits into
mainfrom
chore/bump-mys-contract-details-v5
Oct 14, 2026
Merged

gabrielseco merged 2 commits into
mainfrom
chore/bump-mys-contract-details-v5

Conversation

@gabrielseco

@gabrielseco gabrielseco commented Oct 14, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

The example app now uses Malaysia's latest contract details form (v5). The MYS schema changelog now covers v3–v5.

Why

  • MYS was still pinned to v2 while the backend is on v5.
  • Pinning the example to latest makes us run the form partners will get when they upgrade.

What changed

Toggle details
  • jsonSchemaVersions.ts: MYS contract_details 2 → 5.
  • schemaVersions.ts: unchanged, because it already lists MYS v1–v5.
  • Docs: MYS.md gains v3–v5 and the README version matrix row moves to v5. Tiger has no x-rmt-schema-changelog for MYS, so every entry is inferred from the schema diffs (v5 is tiger's current root schema, v3–v4 are frozen snapshots).
    • v3: reserve messaging and titled fieldset on annual_gross_salary; jsf v1 engine.
    • v4: extreme-salary acknowledgement, flag-gated.
    • v5: contract duration moves to the shared contract_duration component (unlimited maximum); standalone contract_duration_type / contract_end_date definitions removed.
  • Nothing is marked breaking beyond the v5 contract duration field restructure, which could affect payloads relying on the old standalone fields.
  • No public API change.

Screenshots

N/A

Related Resources

  • Tiger: apps/tiger/priv/json_schemas/contract_details/snapshots/MYS/{3,4}.json and the current MYS.json

Testing

  • Tested against the example/ app in a browser
  • npm run schema-canary -- --country=MYS --checks=pinned,latest,submit against sandbox after the bump (pinned v5, latest, submit v5): all pass.

🤖 Generated with Claude Code


Note

Low Risk
Documentation and example-app pinning only; no library API changes, though v5 contract-duration field restructuring matters for anyone migrating off older MYS schemas.

Overview
Pins the example onboarding flow to Malaysia contract_details v5 (was v2) so local testing matches the current backend schema.

Docs now list MYS as v5 in the country matrix and add changelog entries for v3–v5: JSON Schema Form v1 and high-salary reserve messaging (v3), extreme-salary acknowledgement when pay ≥ ~USD 1M equivalent (v4), and contract duration consolidated into the shared contract_duration component with standalone contract_duration_type / contract_end_date removed (v5). Integrators upgrading to v5 should expect contract-duration payload shape changes; v3–v4 are additive and flag-gated.

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

gabrielseco and others added 2 commits October 14, 2026 05:43
Pin Malaysia to contract_details v5 in the example app and document v3-v5
in the MYS schema changelog and the version matrix.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Deploy preview for adp-cost-calculator ready!

Project:adp-cost-calculator
Status: ✅  Deploy successful!
Preview URL:https://adp-cost-calculator-e8u0df0j3-remotecom.vercel.app
Latest Commit:4641c15

Deployed with vercel-action

@github-actions

github-actions Bot commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Deploy preview for remote-flows ready!

Project:remote-flows
Status: ✅  Deploy successful!
Preview URL:https://remote-flows-dz4hy81e6-remotecom.vercel.app
Latest Commit:4641c15

Deployed with vercel-action

@github-actions

Copy link
Copy Markdown
Contributor

📊 Coverage Report

⚪ Coverage unchanged

Metric Current Previous Change Status
Lines 89.58% 89.58% 0% ⚪
Statements 89.46% 89.46% 0% ⚪
Functions 88.47% 88.47% 0% ⚪
Branches 81.21% 81.21% 0% ⚪

Detailed Breakdown

Lines Coverage
  • Covered: 5393 / 6020
  • Coverage: 89.58%
  • Change: 0% (0 lines)
Statements Coverage
  • Covered: 5458 / 6101
  • Coverage: 89.46%
  • Change: 0% (0 statements)
Functions Coverage
  • Covered: 1427 / 1613
  • Coverage: 88.47%
  • Change: 0% (0 functions)
Branches Coverage
  • Covered: 3345 / 4119
  • Coverage: 81.21%
  • Change: 0% (0 branches)

✅ Coverage check passed

@gabrielseco
gabrielseco merged commit 0502db8 into main Oct 14, 2026
14 checks passed
@gabrielseco
gabrielseco deleted the chore/bump-mys-contract-details-v5 branch October 14, 2026 16:29
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.

2 participants