Skip to content

chore(deps): pin @remoteoss/json-schema-form to the kit's required floor - #1554

Merged
remotecom merged 5 commits into
mainfrom
chore/pin-json-schema-form-to-kit-requirement
Oct 15, 2026
Merged

remotecom merged 5 commits into
mainfrom
chore/pin-json-schema-form-to-kit-requirement

Conversation

@gabrielseco

@gabrielseco gabrielseco commented Oct 13, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

@remoteoss/json-schema-form is now pinned exactly, and CI fails when the pin falls out of step with what the form kit requires.

Why

The kit moved to ^1.2.19 (#1547) while package.json still said ^1.2.18, and nothing flagged it. The existing check only counted installed copies, so a stale range slipped through. The pin plus the new check makes the drift fail loudly and tells you the version to set.

What changed

Toggle details
  • package.json / lockfile: @remoteoss/json-schema-form ^1.2.18 → 1.2.18.
  • scripts/check-single-schema-form.ts: after the single-copy check, compares the declared version with the floor of the range declared by the installed @remoteoss/remote-json-schema-form-kit. Mismatch exits 1 and prints the version to set.
  • Renovate keeps json-schema-form disabled (chore(renovate): stop bumping @remoteoss/json-schema-form on its own #1546), so the pin moves with the kit by hand, guided by the check.
  • Not covered: example/ has its own lockfile, which the check doesn't inspect.
  • chore(deps): update dependency @remoteoss/remote-json-schema-form-kit to v1.1.4 #1547 (kit 1.1.4, which requires ^1.2.19) will fail this check until its pin is bumped to 1.2.19. That is the check working as intended.
  • No public API change.

Screenshots

N/A

Related Resources

Testing

  • Pin at 1.2.18 with kit 1.1.2 (^1.2.18): check passes.

  • Pin at 1.2.17: check fails and prints Set "@remoteoss/json-schema-form": "1.2.18".

  • Tested against the example/ app in a browser

  • Feature flag: N/A

🤖 Generated with Claude Code


Note

Low Risk
Dependency version pins and a CI guard script only; no runtime or public API changes.

Overview
Pins @remoteoss/json-schema-form to an exact version (1.2.19) instead of a caret range, and bumps @remoteoss/remote-json-schema-form-kit to 1.1.4 in the root and example lockfiles so the pin matches what the kit requires.

Extends scripts/check-single-schema-form.ts (run via check-exports / CI): after ensuring only one installed copy of the form package, it compares the declared pin in root package.json to the minimum version implied by the installed kit’s dependency range. A mismatch fails the script with instructions to set the correct pin. The example app’s kit devDependency is updated but the check still only inspects the root package.

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

Declare @remoteoss/json-schema-form exactly and make
check-single-schema-form fail unless the declared version equals the
floor of the range the installed kit requires, printing the version
to set. Range drift (^1.2.18 vs the kit's ^1.2.19) previously went
undetected because the check only counted installed copies.

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

github-actions Bot commented Oct 13, 2026 •

Copy link
Copy Markdown
Contributor

📦 Bundle Size Report

Metric Current Previous Change Status
Total (gzip) 230.76 kB 230.76 kB 0 B (0%) 🟢
Total (raw) 642.57 kB 642.57 kB 0 B (0%) 🟢
CSS (gzip) 21.95 kB 21.95 kB 0 B (0%) 🟢
CSS (raw) 114.51 kB 114.51 kB 0 B (0%) 🟢

Size Limits

  • ✅ Total gzipped: 230.76 kB / 350 kB (65.9%)
  • ✅ Total raw: 642.57 kB / 850 kB (75.6%)
  • ✅ CSS gzipped: 21.95 kB / 25 kB (87.8%)

Largest Files (Top 5)

  1. index.esm-wIkXrqU6.js - 11.4 kB (0 B (0%))
  2. styles.css - 10.97 kB (0 B (0%))
  3. index.css - 10.97 kB (0 B (0%))
  4. internals-BOZceErx.js - 6.13 kB (0 B (0%))
  5. hooks-ThU7CDLI.js - 5.85 kB (0 B (0%))
View All Files (291 total)
File Size (gzip) Change
index.esm-wIkXrqU6.js 11.4 kB 0 B (0%)
styles.css 10.97 kB 0 B (0%)
index.css 10.97 kB 0 B (0%)
internals-BOZceErx.js 6.13 kB 0 B (0%)
hooks-ThU7CDLI.js 5.85 kB 0 B (0%)
sdk.gen-BRPiw4M1.js 5.58 kB 0 B (0%)
index.js 5.51 kB 0 B (0%)
flows/Onboarding/hooks.js 4.67 kB 0 B (0%)
utils-CUgvJHl4.js 4.07 kB 0 B (0%)
FieldSetField-D0HdSLTs.js 4.04 kB 0 B (0%)

✅ Bundle size check passed

@github-actions

github-actions Bot commented Oct 13, 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-i0sx6fmik-remotecom.vercel.app
Latest Commit:19eb713

Deployed with vercel-action

@github-actions

github-actions Bot commented Oct 13, 2026 •

Copy link
Copy Markdown
Contributor

Deploy preview for remote-flows ready!

Project:remote-flows
Status: ✅  Deploy successful!
Preview URL:https://remote-flows-fnt8375tz-remotecom.vercel.app
Latest Commit:19eb713

Deployed with vercel-action

@github-actions

github-actions Bot commented Oct 13, 2026 •

Copy link
Copy Markdown
Contributor

📊 Coverage Report

⚪ Coverage unchanged

Metric Current Previous Change Status
Lines 89.59% 89.59% 0% ⚪
Statements 89.47% 89.47% 0% ⚪
Functions 88.48% 88.48% 0% ⚪
Branches 81.24% 81.24% 0% ⚪

Detailed Breakdown

Lines Coverage
  • Covered: 5397 / 6024
  • Coverage: 89.59%
  • Change: 0% (0 lines)
Statements Coverage
  • Covered: 5462 / 6105
  • Coverage: 89.47%
  • Change: 0% (0 statements)
Functions Coverage
  • Covered: 1428 / 1614
  • Coverage: 88.48%
  • Change: 0% (0 functions)
Branches Coverage
  • Covered: 3351 / 4125
  • Coverage: 81.24%
  • Change: 0% (0 branches)

✅ Coverage check passed

gabrielseco and others added 3 commits October 13, 2026 18:38
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
…-form-to-kit-requirement

# Conflicts:
#	example/package-lock.json
…-form-to-kit-requirement

# Conflicts:
#	example/package-lock.json
#	example/package.json
@remotecom
remotecom merged commit 4e4fb3e into main Oct 15, 2026
16 checks passed
@remotecom
remotecom deleted the chore/pin-json-schema-form-to-kit-requirement branch October 15, 2026 14:02
@gabrielseco gabrielseco mentioned this pull request Oct 16, 2026
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.

3 participants