Skip to content

feat(ui): Add agent version compatibility check#52

Merged
nfebe merged 1 commit intomainfrom
feat/agent-version-check
Mar 22, 2026
Merged

feat(ui): Add agent version compatibility check#52
nfebe merged 1 commit intomainfrom
feat/agent-version-check

Conversation

@nfebe
Copy link
Copy Markdown
Contributor

@nfebe nfebe commented Mar 22, 2026

Validate agent version against min/max bounds on each health check. Show a warning banner when incompatible. Dev builds (suffix -dev, -alpha, -beta, -rc, etc.) show a dismissable warning. Non-dev incompatible versions cannot be dismissed.

@sourceant
Copy link
Copy Markdown

sourceant bot commented Mar 22, 2026

Code Review Summary

This PR introduces a robust agent version compatibility check. It validates the agent's version against defined constraints and provides a non-intrusive warning banner for development builds or incompatible versions.

🚀 Key Improvements

  • Implemented a centralized version comparison utility in src/utils/version.ts.
  • Added comprehensive unit tests for version logic in src/__tests__/version.test.ts.
  • Integrated compatibility state into the Pinia statsStore for global access.

💡 Minor Suggestions

  • Refine the version parsing to explicitly handle pre-release identifiers if they aren't caught by the dev-build regex.
  • Improve the user-facing message when the maximum version is exceeded.

Copy link
Copy Markdown

@sourceant sourceant bot left a comment

Choose a reason for hiding this comment

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

Review complete. No specific code suggestions were generated. See the overview comment for a summary.

@nfebe nfebe force-pushed the feat/agent-version-check branch from 3c50110 to d7bbabd Compare March 22, 2026 19:59
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages bot commented Mar 22, 2026

Deploying flatrun-ui with  Cloudflare Pages  Cloudflare Pages

Latest commit: 6866282
Status: ✅  Deploy successful!
Preview URL: https://6d2a1b7a.flatrun-ui.pages.dev
Branch Preview URL: https://feat-agent-version-check.flatrun-ui.pages.dev

View logs

Copy link
Copy Markdown

@sourceant sourceant bot left a comment

Choose a reason for hiding this comment

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

Review complete. See the overview comment for a summary.

Validate agent version against min/max bounds on each
health check. Show a warning banner when incompatible.
Dev builds (suffix -dev, -alpha, -beta, -rc, etc.) show
a dismissable warning. Non-dev incompatible versions
cannot be dismissed.

Signed-off-by: nfebe <fenn25.fn@gmail.com>
@nfebe nfebe force-pushed the feat/agent-version-check branch from d7bbabd to 6866282 Compare March 22, 2026 20:01
Copy link
Copy Markdown

@sourceant sourceant bot left a comment

Choose a reason for hiding this comment

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

Review complete. See the overview comment for a summary.

@nfebe nfebe merged commit 3218bdd into main Mar 22, 2026
5 checks passed
@nfebe nfebe deleted the feat/agent-version-check branch March 22, 2026 20:02
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.

1 participant