Skip to content

feat(web): display the installed EvalScope version - #1784

Merged
Yunnglin merged 2 commits into
modelscope:mainfrom
git-jxj:codex/1778-ui-version
Sep 29, 2026
Merged

Yunnglin merged 2 commits into
modelscope:mainfrom
git-jxj:codex/1778-ui-version

Conversation

@git-jxj

@git-jxj git-jxj commented Sep 28, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Return the installed Python package version from the existing /api/v1/config endpoint.
  • Show the backend version beside the EvalScope brand in the web console.
  • Share the config response between the version display and the existing output-directory initialization.

Closes #1778

Validation

  • python -m pytest tests/service/test_api_contracts.py -q
  • npm test (web: 470 tests)
  • npm run contracts:check
  • npm run build
  • npm run lint
  • make dev
  • make lint
  • make lint-imports
  • python -m pytest tests/cli/test_all.py::TestRun::test_ci_lite -v -s -p no:warnings

git-jxj and others added 2 commits September 28, 2026 16:04
Signed-off-by: git-jxj <65210887+git-jxj@users.noreply.github.com>
Reuse the design-system Badge (rounded-full, accent-dim bg) instead of a
hand-rolled bordered box so the version chip matches the rest of the UI.
Badge now forwards native span attributes so callers can set title.

@Yunnglin Yunnglin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Verified end-to-end: backend returns the package version from evalscope.version via /api/v1/config; generated contracts match the generator output (schema regenerated byte-identical, contracts:check clean); the AppConfigProvider lift shares a single config fetch between ScanProvider and TopNav with correct provider nesting; Python contract tests (16) and web suite (470, incl. new MainLayout test) pass; ruff/eslint/tsc/drift clean; CI all green on the latest head. The version badge now reuses the design-system Badge pill (rounded-full, accent-dim), confirmed in a real browser. Non-blocking nit for follow-up: the tooltip string is hardcoded English rather than routed through t().

@Yunnglin
Yunnglin merged commit e6226d8 into modelscope:main Sep 29, 2026
4 checks passed
@git-jxj
git-jxj deleted the codex/1778-ui-version branch September 29, 2026 06:54
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