Skip to content

feat: add migration version admin handler#1219

Merged
ferhatelmas merged 2 commits into
masterfrom
ferhat/migration-admin-handler
Jul 9, 2026
Merged

feat: add migration version admin handler#1219
ferhatelmas merged 2 commits into
masterfrom
ferhat/migration-admin-handler

Conversation

@ferhatelmas

Copy link
Copy Markdown
Member

What kind of change does this PR introduce?

feature

What is the current behavior?

Branching needs to know about migration version of the different stacks.

What is the new behavior?

Add a new admin endpoint (/migration-version) that returns version in the shape of {"migrationVersion":"some-version"}.

Additional context

Exclude new endpoint from logging.

Copilot AI review requested due to automatic review settings July 9, 2026 08:55
@ferhatelmas ferhatelmas requested a review from a team as a code owner July 9, 2026 08:55

Copilot AI 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.

Pull request overview

Adds a new admin endpoint for retrieving the current stack’s migration version so external systems (e.g., branching) can discover it at runtime.

Changes:

  • Added GET /migration-version returning { migrationVersion: <latest local migration name> }.
  • Excluded /migration-version from request logging in the admin app.
  • Added a unit test validating the new endpoint response.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
src/admin-app.ts Adds /migration-version route and excludes it from admin request logging.
src/admin-app.test.ts Mocks migration lookup and asserts /migration-version returns the expected payload.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/admin-app.ts Outdated
Comment thread src/admin-app.test.ts
@coveralls

coveralls commented Jul 9, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 29012269724

Coverage increased (+0.04%) to 79.271%

Details

  • Coverage increased (+0.04%) from the base build.
  • Patch coverage: 4 of 4 lines across 1 file are fully covered (100%).
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 12596
Covered Lines: 10431
Line Coverage: 82.81%
Relevant Branches: 7299
Covered Branches: 5340
Branch Coverage: 73.16%
Branches in Coverage %: Yes
Coverage Strength: 423.63 hits per line

💛 - Coveralls

Signed-off-by: ferhat elmas <elmas.ferhat@gmail.com>
Signed-off-by: ferhat elmas <elmas.ferhat@gmail.com>

Copilot AI 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.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

@ferhatelmas ferhatelmas merged commit ede2757 into master Jul 9, 2026
27 checks passed
@ferhatelmas ferhatelmas deleted the ferhat/migration-admin-handler branch July 9, 2026 11:19
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.

4 participants