Skip to content

Highlight version changes in Changes sidebar - #59

Merged
ai merged 1 commit into
ai:mainfrom
oiahoon:fix/highlight-version-change
Jul 31, 2026
Merged

ai merged 1 commit into
ai:mainfrom
oiahoon:fix/highlight-version-change

Conversation

@oiahoon

@oiahoon oiahoon commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Closes #45

Summary

  • highlight the changed major, minor, or patch segment in the Changes sidebar
  • preserve non-semver formatting and avoid highlighting new dependencies
  • cover each update type in the Storybook state data

Testing

  • /bin/bash ./scripts/format.sh
  • pnpm test:js
  • pnpm test:css
  • pnpm test:types
  • pnpm test:audit
  • pnpm test:web
  • pnpm build-visual
  • Storybook Change, Change Dark, and New Dependency manual checks

Comment thread web/ui/version.svelte Outdated
let value = $derived(formatVersion(version))
let parts = $derived(
value.match(
/^([vV]?)(\d+)(?:\.(\d+))?(?:\.(\d+))?((?:-[\dA-Za-z.-]+)?(?:\+[\dA-Za-z.-]+)?)$/

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Can you move it to helper function in web/stores/change.ts? We try to keep UI components simple as possible.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Addressed in 89d1ee4. Version parsing now lives in web/stores/change.ts.

@ai

ai commented Jul 18, 2026

Copy link
Copy Markdown
Owner

Thanks! Can you show a screenshot of the fix?

@oiahoon
oiahoon force-pushed the fix/highlight-version-change branch from 342f347 to 89d1ee4 Compare July 30, 2026 16:36
@oiahoon

oiahoon commented Jul 30, 2026

Copy link
Copy Markdown
Contributor Author
image

@ai
ai merged commit 2fb6fe6 into ai:main Jul 31, 2026
3 checks passed
@ai

ai commented Jul 31, 2026

Copy link
Copy Markdown
Owner

Thanks!

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.

Highlight version change in Changes sidebar

2 participants