Skip to content

fix: enforce semver for tool versionName#118

Merged
dupontgu merged 1 commit into
mainfrom
fix/semver-versionname
Jul 22, 2026
Merged

fix: enforce semver for tool versionName#118
dupontgu merged 1 commit into
mainfrom
fix/semver-versionname

Conversation

@aconanlai

Copy link
Copy Markdown
Contributor

No description provided.

@aconanlai
aconanlai requested a review from dupontgu July 21, 2026 19:45

@dupontgu dupontgu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

one thing that would be nice to have here - we should validate semver in LightSdkPlugin.applyToolMetadata and quickly fail the build if it's off

Edit: you did that!! sorry I missed

@dupontgu
dupontgu merged commit d502bca into main Jul 22, 2026
1 check passed
github-actions Bot pushed a commit to tyleryancey/light-tools that referenced this pull request Jul 23, 2026
Upstream changes:

d502bca Merge pull request lightphone#118 from lightphone/fix/semver-versionname
cd621b7 Merge pull request lightphone#117 from lightphone/builder-ci
c0cf471 fix: enforce semver for tool versionName
29911b2 filter on tags
809c0d8 add circleci config
5124ab6 Merge pull request lightphone#112 from lightphone/fix/icons
92b3b20 Merge pull request lightphone#97 from lightphone/fix/weather-tool-search
bb87390 use light api proxy
9cd076f delete redundant icons, fix ugly ones
eb5b6a6 fix: return list of location search results in weather tool
tyleryancey added a commit to tyleryancey/light-tools that referenced this pull request Jul 23, 2026
Upstream changes:

464b414 Merge upstream lightphone/light-sdk main
c43875c ci(sync): merge from upstream instead of ff-only, add chess, summarize syncs
d502bca Merge pull request lightphone#118 from lightphone/fix/semver-versionname
cd621b7 Merge pull request lightphone#117 from lightphone/builder-ci
c0cf471 fix: enforce semver for tool versionName
29911b2 filter on tags
809c0d8 add circleci config
5124ab6 Merge pull request lightphone#112 from lightphone/fix/icons
92b3b20 Merge pull request lightphone#97 from lightphone/fix/weather-tool-search
bb87390 use light api proxy
9cd076f delete redundant icons, fix ugly ones
eb5b6a6 fix: return list of location search results in weather tool

Conflict resolution: kept chess's lighttool.toml identity (permissions = [])
and bumped versionName to semver 0.1.0 for the new builder enforcement;
kept deletion of the sample HomeScreen.kt that chess had replaced.

Co-Authored-By: Claude Fable 5 <[email protected]>
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