Skip to content

fix(ci): derive version-copy floor from single source of truth - #6374

Merged
JSONbored merged 1 commit into
JSONbored:mainfrom
nghetienhiep:fix/issue-6292
Jul 16, 2026
Merged

fix(ci): derive version-copy floor from single source of truth#6374
JSONbored merged 1 commit into
JSONbored:mainfrom
nghetienhiep:fix/issue-6292

Conversation

@nghetienhiep

Copy link
Copy Markdown
Contributor

fix(ci): derive version-copy floor from single source of truth

The deterministic half of check-ui-mcp-version-copy.mjs hardcoded v0.2 /
0.2.x / 0.2.0 as the stale-version and minimum-supported-floor literals. The
real floor has since moved to 0.5.0 (MCP_MINIMUM_SUPPORTED_VERSION), so the
text scan was aimed several majors out of date and could no longer catch
current drift — its only live defense was the best-effort npm-registry diff.

Read the floor from MCP_MINIMUM_SUPPORTED_VERSION in
apps/loopover-ui/src/lib/mcp-package.ts (the same module the known-latest
check already reads) and build the three stale-version matchers from it, so a
future floor bump can't silently leave this check frozen again. This surfaces
the stale 0.5.0 examples in packages/loopover-mcp/README.md, which are updated
to the current 3.0.0 package version.

Adds a unit test asserting the script reads the floor dynamically from the
shipped constant and that the matchers flag the stale forms while leaving the
current version and the explicit minimum-supported line alone.

Closes #6292

@nghetienhiep
nghetienhiep requested a review from JSONbored as a code owner July 16, 2026 05:02
@superagent-security

Copy link
Copy Markdown
Contributor

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

@codecov

codecov Bot commented Jul 16, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.60%. Comparing base (fe1b96a) to head (bf0d847).
⚠️ Report is 13 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #6374   +/-   ##
=======================================
  Coverage   95.60%   95.60%           
=======================================
  Files         599      601    +2     
  Lines       47301    47310    +9     
  Branches    15046    15050    +4     
=======================================
+ Hits        45223    45232    +9     
  Misses       1291     1291           
  Partials      787      787           
Flag Coverage Δ
shard-1 44.01% <ø> (-0.02%) ⬇️
shard-2 36.65% <ø> (-0.01%) ⬇️
shard-3 32.44% <ø> (-0.15%) ⬇️
shard-4 34.53% <ø> (-0.18%) ⬇️
shard-5 31.03% <ø> (-0.55%) ⬇️
shard-6 45.27% <ø> (+0.33%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

@loopover-orb loopover-orb Bot added the gittensor:bug Gittensor-scored bug fix — scores a 0.05x multiplier. label Jul 16, 2026
@nghetienhiep
nghetienhiep force-pushed the fix/issue-6292 branch 2 times, most recently from 9e98527 to 9467e57 Compare July 16, 2026 05:27
@nghetienhiep
nghetienhiep force-pushed the fix/issue-6292 branch 2 times, most recently from a3025bd to 7676b81 Compare July 16, 2026 05:41
@nghetienhiep
nghetienhiep force-pushed the fix/issue-6292 branch 3 times, most recently from 0e491c8 to e36dfcf Compare July 16, 2026 06:09
@nghetienhiep
nghetienhiep force-pushed the fix/issue-6292 branch 2 times, most recently from 1307d60 to 77d9ed0 Compare July 16, 2026 06:16
@nghetienhiep
nghetienhiep force-pushed the fix/issue-6292 branch 4 times, most recently from ea2fdc8 to bf0d847 Compare July 16, 2026 06:23
@nghetienhiep
nghetienhiep force-pushed the fix/issue-6292 branch 3 times, most recently from 2300789 to c87b1fb Compare July 16, 2026 06:41
The deterministic half of check-ui-mcp-version-copy.mjs hardcoded v0.2 /
0.2.x / 0.2.0 as the stale-version and minimum-supported-floor literals. The
real floor has since moved to 0.5.0 (MCP_MINIMUM_SUPPORTED_VERSION), so the
text scan was aimed several majors out of date and could no longer catch
current drift — its only live defense was the best-effort npm-registry diff.

Read the floor from MCP_MINIMUM_SUPPORTED_VERSION in
apps/loopover-ui/src/lib/mcp-package.ts (the same module the known-latest
check already reads) and build the three stale-version matchers from it, so a
future floor bump can't silently leave this check frozen again. This surfaces
the stale 0.5.0 examples in packages/loopover-mcp/README.md, which are updated
to the current 3.0.0 package version.

Adds a unit test asserting the script reads the floor dynamically from the
shipped constant and that the matchers flag the stale forms while leaving the
current version and the explicit minimum-supported line alone.

Closes JSONbored#6292
@JSONbored
JSONbored merged commit e449499 into JSONbored:main Jul 16, 2026
5 checks passed
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.

fix(ci): check-ui-mcp-version-copy.mjs's hardcoded stale-version literals are frozen 3 majors behind the real minimum floor

2 participants