Skip to content

fix(ui): widen the ui-kit consumer range past its 1.0.0 pre-release cap - #5904

Merged
JSONbored merged 1 commit into
mainfrom
fix/ui-kit-consumer-range-cap
Jul 14, 2026
Merged

fix(ui): widen the ui-kit consumer range past its 1.0.0 pre-release cap#5904
JSONbored merged 1 commit into
mainfrom
fix/ui-kit-consumer-range-cap

Conversation

@JSONbored

Copy link
Copy Markdown
Owner

Summary

  • apps/loopover-ui and apps/loopover-miner-ui both pinned "@loopover/ui-kit" to ">=0.1.0 <1.0.0", which excludes the ui-kit-v1.0.0 release-please PR's target version and permanently blocks its npm ci with Missing: @loopover/ui-kit@0.2.0 from lock file (confirmed live on chore(release): cut ui-kit v1.0.0 #5826).
  • Checked the release's own CHANGELOG first: the only BREAKING CHANGES entry for this 1.0.0 cut is the gittensory-*loopover-* rename, which both consuming apps have already fully completed (they already import @loopover/ui-kit under its new name). So there's no live incompatibility to guard against — the <1.0.0 ceiling is just stale.
  • Widened both to <2.0.0, preserving the existing below-next-major convention rather than removing the cap outright.
  • Also reconciles a separate, pre-existing drift npm install surfaced while regenerating the lockfile: package-lock.json's @loopover/mcp and @loopover/miner entries still required @loopover/engine at ^3.1.0 — a leftover from an earlier release-please regeneration cycle — while their actual package.json files (already merged to main via chore(release): cut mcp v2.0.0 #5735/in-flight chore(release): cut miner v3.0.0 #5736) declare ^1.0.0/^3.0.0. Harmless today only by coincidence (local engine happens to satisfy both), but worth reconciling since the file was already being touched.

Scope

  • The PR title follows type(scope): short summary Conventional Commit format.
  • This PR is focused and does not mix unrelated backend, UI, MCP, docs, dependency, and deploy changes.
  • This follows CONTRIBUTING.md and does not reintroduce GitHub Pages, VitePress, site/, or CNAME.
  • I linked a currently open issue this PR resolves — maintainer PR fixing a dependency-range defect discovered live on chore(release): cut ui-kit v1.0.0 #5826, no linked issue.

Validation

  • git diff --check
  • npm run ui:typecheck — clean, both apps.
  • npm run ui:lint — 0 errors (145/7 pre-existing warnings in untouched route files, unrelated to this change).
  • npm run ui:build — not run locally (full build is slow); ui:typecheck + ui:lint cover the actual surface touched (a dependency range + regenerated lockfile, no source changes). CI's own ui:build step will validate this.
  • npm run test:coverage — skipped; src/** untouched.
  • npm run test:workers / npm run build:mcp / npm run test:mcp-pack — skipped, unaffected by this change.
  • npm audit --audit-level=moderate — no new dependencies added, only a range widened and a lockfile drift reconciled.

If any required check was skipped, explain why:

  • No src/ or application code changed — only two dependency range strings and the lockfile regenerated to match. typecheck and lint (which both rebuild @loopover/ui-kit first) are the relevant checks and are clean; CI's own ui:build will provide the final confirmation.

Safety

  • No secrets, wallet details, hotkeys, coldkeys, user PATs, private keys, raw trust scores, private rankings, or private maintainer evidence are exposed.
  • Public GitHub text stays sanitized, low-noise, and does not imply compensation guarantees or optimization tactics.
  • Auth/cookie/CORS/GitHub App/Cloudflare/session changes include negative-path tests. — N/A.
  • API/OpenAPI/MCP behavior updated/tested. — N/A.
  • UI changes use live API data or real empty/error/loading states. — N/A, no UI behavior changed, only a dependency range.
  • Visible UI changes include a UI Evidence section. — N/A, no visible UI change.
  • Public docs/changelogs are updated where needed; changelogs are only edited for release-prep PRs. — this PR does not touch CHANGELOG.md.

Notes

  • Discovered while triaging why the ui-kit release-please PR (chore(release): cut ui-kit v1.0.0 #5826) failed Build UI preview artifact even though ui-kit itself has no @loopover/* dependencies of its own.

apps/loopover-ui and apps/loopover-miner-ui both pinned "@loopover/ui-kit"
to ">=0.1.0 <1.0.0", excluding the ui-kit-v1.0.0 release-please PR's target
version and permanently blocking its npm ci (confirmed live on #5826). The
1.0.0 cut's only breaking-change entry is the gittensory->loopover rename,
which both apps already completed, so there's no live incompatibility --
widen the ceiling to <2.0.0, preserving the existing below-next-major
convention. Verified with npm run ui:typecheck and ui:lint (0 errors).

Also reconciles package-lock.json's stale "@loopover/engine" requires
entries for @loopover/mcp and @loopover/miner (still ^3.1.0 from an
earlier release-please regeneration cycle) back to match their current
package.json ranges (^1.0.0 / ^3.0.0) -- npm install surfaced this while
regenerating the lockfile for the ui-kit range change above.
@superagent-security

Copy link
Copy Markdown
Contributor

Superagent didn't find any vulnerabilities or security issues in this PR.

@JSONbored
JSONbored merged commit 4042874 into main Jul 14, 2026
6 checks passed
@JSONbored
JSONbored deleted the fix/ui-kit-consumer-range-cap branch July 14, 2026 22:11
@loopover-orb loopover-orb Bot added the gittensor:bug Gittensor-scored bug fix — scores a 0.05x multiplier. label Jul 14, 2026
This was referenced Jul 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gittensor:bug Gittensor-scored bug fix — scores a 0.05x multiplier.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant