Skip to content

Release v0.10.4#130

Merged
toasterbook88 merged 1 commit into
mainfrom
release/v0.10.4
May 21, 2026
Merged

Release v0.10.4#130
toasterbook88 merged 1 commit into
mainfrom
release/v0.10.4

Conversation

@toasterbook88
Copy link
Copy Markdown
Owner

Version bump and documentation refresh for v0.10.4.\n\n- Bump version to 0.10.4\n- Refresh docs/current-state.md\n- Update summary golden files for version bump\n\nAll quality gates pass.

- Bump version to 0.10.4

- Refresh docs/current-state.md for v0.10.4

- Update summary golden files for version bump

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@toasterbook88 toasterbook88 enabled auto-merge (squash) May 21, 2026 03:54
@toasterbook88 toasterbook88 merged commit 36a2241 into main May 21, 2026
8 checks passed
@toasterbook88 toasterbook88 deleted the release/v0.10.4 branch May 21, 2026 03:56
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the project version from 0.10.3 to 0.10.4 across the changelog, build information, documentation, and test data. Review feedback identifies a potential duplication of the version definition in the daemon package, which contradicts the intended single source of truth in the buildinfo package. Additionally, a minor decrease in total test coverage was noted in the documentation updates and should be reviewed to ensure no critical paths are left untested.


// Version is the single source of truth for the AXIS release string.
const Version = "0.10.3"
const Version = "0.10.4"
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.

high

The comment on line 3 states that Version is the "single source of truth", yet internal/daemon/handlers.go (line 83) uses an unqualified Version identifier without importing the buildinfo package. This suggests a duplicate version definition exists within the daemon package, which can lead to inconsistent version reporting (e.g., the API reporting an old version while the CLI reports the new one). Please ensure all packages use buildinfo.Version.

Comment thread docs/current-state.md
- `coverage gate passed: internal/mcp 88.7% >= 35.0%`
- `coverage gate passed: internal/ui 94.0% >= 80.0%`
- `coverage gate passed: total 73.8% >= 45.0%`
- `coverage gate passed: total 73.0% >= 45.0%`
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.

medium

The total test coverage has decreased from 73.8% to 73.0%. While this still satisfies the 45% coverage gate, a regression in coverage during a release cycle should be reviewed to ensure no critical paths are left untested.

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