[vesync] Add auth V2 support by dag81 · Pull Request #20236 · openhab/openhab-addons

dag81 · 2026-02-16T01:08:06Z

The changes are currently being soak tested but the new flows have been proven by multiple users in different data centre's in the linked issue thread below. (Soak testing primarily for token refresh if it does happen). Opening now for review while the soak test's run.

The main adjustments here relate to: #19127

This adjusts:

Corrects a error when dispose is called, as .clear on the hashmap now appears to cause errors, it is re-referenced to a new HashMap instance to ensure a empty one is referenced without errors.
Authorization handling with VeSync servers to support a newer scheme, that at some point new accounts could only use. Old accounts do actually appear to support the new schema on testing with my account, so the upgrade should be backwards compatible for V2 based devices at least.
The URL/s utilised so the relevant cloud service is targeted based on the data extracted from the authorization handshakes.
A few other minor adjustments for Static Code Analysis bits that were flagged, since the last time this binding was updated, to remove them.

[veSync] Initial upload of prototype POC code before cleanup Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Non-op cleanup's from checks Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Constant cleanups Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Refactored to remove inner class with cleanups Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Bugfix previous constants update Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Cleanup commented out lines Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Cleanups - removal of unit test and cleanup Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] SCA cleanups Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Token refresh support Signed-off-by: David Goodyear <david.goodyear@gmail.com>

Copilot

Pull request overview

Adds VeSync “Auth V2” login support (incl. region-aware endpoint selection) to restore/maintain connectivity for newer VeSync accounts and data centers, alongside some minor SCA-driven cleanup.

Changes:

Introduce Auth V2 login handshake (token request → authorize-code login → optional region redirect) and persist the selected server base URL in the user session.
Route subsequent API calls via the selected regional server URL and add a token-refresh retry on token-related errors.
Minor robustness/SCA updates (Locale-stable toLowerCase, timer calculation using Instant, hashmap dispose behavior).

Reviewed changes

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

Show a summary per file

File	Description
bundles/org.openhab.binding.vesync/src/main/test/org/openhab/binding/vesync/internal/handler/requests/VesyncAuthenticatedRequestTest.java	Renames the public test class (currently breaks Java filename/classname contract).
bundles/org.openhab.binding.vesync/src/main/java/org/openhab/binding/vesync/internal/handlers/VeSyncDeviceAirPurifierHandler.java	Locale-stable lowercasing; removes unnecessary suppression.
bundles/org.openhab.binding.vesync/src/main/java/org/openhab/binding/vesync/internal/handlers/VeSyncDeviceAirHumidifierHandler.java	Locale-stable lowercasing; switches auto-off time calc to `Instant`.
bundles/org.openhab.binding.vesync/src/main/java/org/openhab/binding/vesync/internal/handlers/VeSyncBridgeHandler.java	Adds token-expiry detection and re-login retry wrapper for authorized calls.
bundles/org.openhab.binding.vesync/src/main/java/org/openhab/binding/vesync/internal/handlers/VeSyncBaseDeviceHandler.java	Switches request URL building to relative paths (base URL now comes from session/server selection).
bundles/org.openhab.binding.vesync/src/main/java/org/openhab/binding/vesync/internal/dto/responses/VeSyncUserSession.java	Adds `serverUrl` to session; adjusts session fields/toString.
bundles/org.openhab.binding.vesync/src/main/java/org/openhab/binding/vesync/internal/dto/responses/VeSyncAuthTokenResponse.java	New DTO for the auth-token step response.
bundles/org.openhab.binding.vesync/src/main/java/org/openhab/binding/vesync/internal/dto/responses/VeSyncAuthLoginWithAuthorizeCodeVeSyncResponse.java	New DTO for the authorize-code login response (incl. region redirect data).
bundles/org.openhab.binding.vesync/src/main/java/org/openhab/binding/vesync/internal/dto/requests/VeSyncRequest.java	Refactors to extend new base request DTO (currently introduces duplicate `httpMethod` field).
bundles/org.openhab.binding.vesync/src/main/java/org/openhab/binding/vesync/internal/dto/requests/VeSyncProtocolConstants.java	Adds US/EU server addresses; converts endpoints to relative paths.
bundles/org.openhab.binding.vesync/src/main/java/org/openhab/binding/vesync/internal/dto/requests/VeSyncBaseRequest.java	New common request base (acceptLanguage/traceId/method/httpMethod).
bundles/org.openhab.binding.vesync/src/main/java/org/openhab/binding/vesync/internal/dto/requests/VeSyncAuthTokenRequest.java	New request DTO for the auth-token step.
bundles/org.openhab.binding.vesync/src/main/java/org/openhab/binding/vesync/internal/dto/requests/VeSyncAuthLoginWithAuthorizeCodeVeSyncRegionChange.java	New DTO for region-change authorize-code login.
bundles/org.openhab.binding.vesync/src/main/java/org/openhab/binding/vesync/internal/dto/requests/VeSyncAuthLoginWithAuthorizeCodeVeSync.java	New DTO for authorize-code login.
bundles/org.openhab.binding.vesync/src/main/java/org/openhab/binding/vesync/internal/api/VeSyncV2ApiHelper.java	Replaces legacy login with Auth V2 helper; prefixes requests with the session’s selected server URL; fixes dispose map handling.
bundles/org.openhab.binding.vesync/src/main/java/org/openhab/binding/vesync/internal/api/LoginAuthV2Helper.java	New helper implementing the Auth V2 sequence + cross-region redirect handling (needs null-safety hardening).

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

[veSync] Copilot suggestion minor mods Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] LoginAuthV2Helper.java NPE Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Refactored to remove inner class with cleanups Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Token refresh correction and NPE handling updates based on co-pilot feedback Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Further NPE bits from copilot flagged bits Signed-off-by: David Goodyear <david.goodyear@gmail.com>

dag81 · 2026-02-17T22:47:42Z

Hi @psmedley I worked through each of the bits co-pilot raised. I resolved so I could track but the changes are from commit 4e9da4f onwards, along with one bug I spotted while re-reading. Currently soak testing here on our main server now - but would be good to open up for human review when there is time please.

psmedley · 2026-02-18T08:02:25Z

Hi @psmedley I worked through each of the bits co-pilot raised. I resolved so I could track but the changes are from commit 4e9da4f onwards, along with one bug I spotted while re-reading. Currently soak testing here on our main server now - but would be good to open up for human review when there is time please.

I'm not a reviewer (my java is nowhere near good enough), I was just trying to help the you and the ultimate approver by kicking off the co-pilot reviews, and as they're a quick/easy win to address.

psmedley

Added a few things I noticed

dag81 · 2026-02-18T08:31:02Z

Hi @psmedley - thank you for looking through - I didn't know whether you had to do something to get it assigned to someone or mark co-pilot as good / re-run it, etc.... So big thanks for having a look through - I guess this gets automatically picked up then when someone has time and assigns it to themselves irrelevant of the co-pilot run then?

psmedley · 2026-02-18T08:35:56Z

Hi @psmedley - thank you for looking through - I didn't know whether you had to do something to get it assigned to someone or mark co-pilot as good / re-run it, etc.... So big thanks for having a look through - I guess this gets automatically picked up then when someone has time and assigns it to themselves irrelevant of the co-pilot run then?

Yeah a reviewer will pick it up when they have time, I have a couple of PRs for Ring to extend functionality which are waiting. I added the 'Bug' tag to this one as the vesync binding is currently broken for many users - so this one should hopefully get looked at first. Bug fixes make it into 5.1.x builds, enhancements only go into 5.2.x (due midyear)

[veSync] Const creation Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Removal of unnecessary static imports Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Alignment to use const Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Const simplification Signed-off-by: David Goodyear <david.goodyear@gmail.com>

dag81 · 2026-03-28T00:22:31Z

@openhab/add-ons-maintainers any chance of a review on this change please - currently users need to use snapshot builds, until we get it merged, for any new accounts at least. (I've been running locally now on our home server for approx. 2 month's - perfectly stable and running BAU).

Copilot

Pull request overview

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

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

Copilot · 2026-03-28T10:03:25Z

+        final ContentResponse response = request.send();
+
+        final VeSyncAuthLoginWithAuthorizeCodeVeSyncResponse result = VeSyncConstants.GSON
+                .fromJson(response.getContentAsString(), VeSyncAuthLoginWithAuthorizeCodeVeSyncResponse.class);
+
+        if (result == null) {
+            return false;
+        }
+
+        if (result.isMsgSuccess()) {
+            loginData.token = result.result.token;
+            loginData.accountId = result.result.accountID;
+            return true;


In the regional-redirect login flow, the HTTP status isn’t validated and result.result is dereferenced without checking for null. A non-200 response or a response without result will cause incorrect behavior or an NPE. Check response.getStatus() first and ensure result.result is non-null before reading token/accountID (otherwise return false or throw AuthenticationException).

Yeah thats a good catch. Added the missing http response code check required.

Copilot · 2026-03-28T10:03:26Z

+            if (session != null && !url.startsWith(session.getServerUrl())) {
+                url = session.getServerUrl() + url;


session.getServerUrl() may be null (it’s not annotated and is populated dynamically), but it’s used in url.startsWith(...) and string concatenation. This can throw an NPE during requests (e.g., device discovery). Guard against null/blank server URLs and only prefix when the incoming url is relative.

Suggested change

if (session != null && !url.startsWith(session.getServerUrl())) {

url = session.getServerUrl() + url;

if (session != null) {

String serverUrl = session.getServerUrl();

if (serverUrl != null && !serverUrl.isBlank() && !url.startsWith("http://")

&& !url.startsWith("https://") && !url.startsWith(serverUrl)) {

url = serverUrl + url;

}

Adjusted with additional null check, to protect against NPE.

Copilot · 2026-03-28T10:03:26Z

+        String result = api.reqV2Authorized(url, macId, requestData);
+
+        VeSyncResponse responseFrame = VeSyncConstants.GSON.fromJson(result, VeSyncResponse.class);
+


fromJson(result, VeSyncResponse.class) can throw a JsonSyntaxException if the backend returns non-JSON (HTML error pages, proxies, etc.). Right now that would bubble up as a runtime exception and break command handling. Wrap the parse in a try/catch and skip token-refresh logic when parsing fails (keeping the original result).

Copilot · 2026-03-28T10:03:27Z

+    @SerializedName("result")
+    public VeSyncAuthTokenResponse.LoginTokenResponse result;
+
+    public class LoginTokenResponse {


LoginTokenResponse is a non-static inner class. Gson cannot reliably instantiate non-static inner classes during deserialization, which can cause the auth-token response parsing to fail at runtime. Make LoginTokenResponse a static nested class (or a top-level class) so it can be deserialized safely.

Suggested change

public class LoginTokenResponse {

public static class LoginTokenResponse {

I was going to move all the inner classes out to top level dto. However on looking through the files as they currently are it would make the files much less readable when comparing to the raw payloads. This could be done on another PR but will possibly bloat this one. I have however changed the inner classes to be all static on the responses as recommended.

lsiepel · 2026-03-28T14:50:31Z

+                logger.trace("Refreshing API token due to error response regarding the token");
+                final VeSyncBridgeConfiguration config = getConfigAs(VeSyncBridgeConfiguration.class);
+                final String passwordMd5 = VeSyncV2ApiHelper.calculateMd5(config.password);
+                api.login(config.username, passwordMd5, "Europe/London");


The zone is global? Or do you need to make this configurable?

I don't believe the timezone matters, python vesync default to America/New_york - https://github.com/webdjoe/pyvesync/blob/2f2511f27acc8162722f2bf2eee5e74134d07cdd/README.md?plain=1#L57

It is a parameter to the request, so it does something and i noticed different zones are used, so if you don;lt experience any issue, atleast align them.

Ideally it would be driven from the system and passed as you say Isiepel. Your right psmedley regarding timezone it doesn't technically get used (at least on the api's mapped by pyvesync - I suspect the app uses it when generating accounts possibly is my guess). However the API's are undocumented and mapped from the mapping work from pyvesync. Therefore I wouldn't plug new data generally into it, as they do appear to validate requests against what they expect the app to possibly send. I tested the old America/New York tz against my primary account (appears to be USA based due to its age), and a test one I setup for testing this binding (which logs in to the eu servers) I can see redirect's to eu based server, and it accepts the America/New York tz so have aligned on that one. If times are ever mapped rather than durations then this may need to be looked at but then pyvesync will then also map true tz data alongside the new api's as well, so it can be updated then based on the region openhab is running in. I have therefore kept as is just aligned for consistency as suggested, in case of future use.

lsiepel

Beside the comment i made and copilot did. I wonder if the differnt (hardcoded and/or default) timezone/locale are right. As this supports both us and europ, i see both of them mixed. Please make sure that it works for both areas as intended.

lsiepel · 2026-04-03T09:13:59Z

Besides fixing the open comments, i wonder if you have any thoughts about backporting this to 4.3.x ?

dag81 · 2026-04-03T10:26:56Z

Yeah I was going to try and look over this long weekend at the merge once the changes are good. I’ll try to back port the auth updates. The comment it made about the inner classes I moved out as I thought that was the preference, but can move back to inner if needed. What’s the pref @Isiepel on that bit?On 3 Apr 2026, at 10:14, lsiepel ***@***.***> wrote:lsiepel left a comment (openhab/openhab-addons#20236) Besides fixing the open comments, i wonder if you have any thoughts about backporting this to 4.3.x ? —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: ***@***.***>

lsiepel · 2026-04-03T11:00:21Z

Stand-alone /top level dto is preferred.

I'll look at refactoring this on a second pass after this is merged and backported. As I was about to do it, then realised it would create a lot more changes, that would introduce risk on what is a bug fix. It will need some careful renaming to keep everything easily traceable when done. So will move, but if okay I'll open a second less urgent PR for that against this binding.

[veSync] Timezone use alignment Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Add V1 failsafe fallback Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Revert and adjust V1 fallback behaviour Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Further inner class static updates on responses Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] HTTP Response code check Signed-off-by: David Goodyear <david.goodyear@gmail.com>

lsiepel

Thanks, LGTM

* [veSync] Initial upload of prototype POC code before cleanup Signed-off-by: David Goodyear <david.goodyear@gmail.com>

…20236) * [veSync] Initial upload of prototype POC code before cleanup Signed-off-by: David Goodyear <david.goodyear@gmail.com>

…20236) [veSync] Auth V2 Support so users can use the binding again (openhab#20236) Signed-off-by: David Goodyear <david.goodyear@gmail.com>

…20556) [veSync] Auth V2 Support so users can use the binding again (#20236) Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Initial upload of prototype POC code before cleanup

afcaf7b

[veSync] Initial upload of prototype POC code before cleanup Signed-off-by: David Goodyear <david.goodyear@gmail.com>

dag81 changed the title ~~[veSync] No Need to review yet WIP~~ [veSync] Auth V2 Support - No Need to review yet WIP Feb 16, 2026

dag81 mentioned this pull request Feb 16, 2026

[VeSync] Bridge configuration error login #19127

Closed

dag81 added 8 commits February 16, 2026 02:22

[veSync] Non-op cleanup's from checks

cb00640

[veSync] Non-op cleanup's from checks Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Constant cleanups

6439519

[veSync] Constant cleanups Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Refactored to remove inner class with cleanups

3d0c9d5

[veSync] Refactored to remove inner class with cleanups Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Bugfix previous constants update

6dc6508

[veSync] Bugfix previous constants update Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Cleanup commented out lines

dc9b776

[veSync] Cleanup commented out lines Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Cleanups - removal of unit test and cleanup

2534060

[veSync] Cleanups - removal of unit test and cleanup Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] SCA cleanups

feaf9c3

[veSync] SCA cleanups Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Token refresh support

43c5033

[veSync] Token refresh support Signed-off-by: David Goodyear <david.goodyear@gmail.com>

dag81 changed the title ~~[veSync] Auth V2 Support - No Need to review yet WIP~~ [veSync] Auth V2 Support Feb 17, 2026

dag81 marked this pull request as ready for review February 17, 2026 01:09

psmedley added the bug An unexpected problem or unintended behavior of an add-on label Feb 17, 2026

psmedley requested a review from Copilot February 17, 2026 08:13

Copilot started reviewing on behalf of psmedley February 17, 2026 08:13 View session

Copilot AI reviewed Feb 17, 2026

View reviewed changes

dag81 added 5 commits February 17, 2026 18:40

[veSync] Copilot suggestion minor mods

4e9da4f

[veSync] Copilot suggestion minor mods Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] LoginAuthV2Helper.java NPE

2b03071

[veSync] LoginAuthV2Helper.java NPE Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Refactored to remove inner class with cleanups

4c6e1e9

[veSync] Refactored to remove inner class with cleanups Signed-off-by: David Goodyear <david.goodyear@gmail.com>

Update VeSyncBridgeHandler.java

8b81e00

[veSync] Token refresh correction and NPE handling updates based on co-pilot feedback Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Further NPE bits from copilot flagged bits

485b0cc

[veSync] Further NPE bits from copilot flagged bits Signed-off-by: David Goodyear <david.goodyear@gmail.com>

psmedley reviewed Feb 18, 2026

View reviewed changes

dag81 added 2 commits February 18, 2026 23:19

[veSync] Const creation

c2debca

[veSync] Const creation Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Removal of unnecessary static imports

927032f

[veSync] Removal of unnecessary static imports Signed-off-by: David Goodyear <david.goodyear@gmail.com>

dag81 changed the title ~~[veSync] Auth V2 Support~~ [veSync] Auth V2 Support so users can use the binding again Feb 19, 2026

dag81 added 2 commits February 19, 2026 02:14

[veSync] Alignment to use const

6c9dd16

[veSync] Alignment to use const Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Const simplification

9b1e05e

[veSync] Const simplification Signed-off-by: David Goodyear <david.goodyear@gmail.com>

wborn requested a review from Copilot March 28, 2026 09:57

Copilot started reviewing on behalf of wborn March 28, 2026 09:58 View session

Copilot AI reviewed Mar 28, 2026

View reviewed changes

lsiepel reviewed Mar 28, 2026

View reviewed changes

lsiepel requested changes Mar 28, 2026

View reviewed changes

dag81 added 6 commits April 3, 2026 23:20

[veSync] Timezone use alignment

cf6de47

[veSync] Timezone use alignment Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Add V1 failsafe fallback

0c846b0

[veSync] Add V1 failsafe fallback Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Add V1 failsafe fallback

8f89d00

[veSync] Add V1 failsafe fallback Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Revert and adjust V1 fallback behaviour

9882afe

[veSync] Revert and adjust V1 fallback behaviour Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] Further inner class static updates on responses

0107dc7

[veSync] Further inner class static updates on responses Signed-off-by: David Goodyear <david.goodyear@gmail.com>

[veSync] HTTP Response code check

83f6dc6

[veSync] HTTP Response code check Signed-off-by: David Goodyear <david.goodyear@gmail.com>

lsiepel approved these changes Apr 5, 2026

View reviewed changes

lsiepel merged commit 4895202 into openhab:main Apr 5, 2026
2 checks passed

lsiepel added this to the 5.2 milestone Apr 5, 2026

lsiepel pushed a commit that referenced this pull request Apr 5, 2026

[veSync] Auth V2 Support so users can use the binding again (#20236)

1be519e

* [veSync] Initial upload of prototype POC code before cleanup Signed-off-by: David Goodyear <david.goodyear@gmail.com>

lsiepel added the backported A PR that has been cherry-picked to a patch release branch label Apr 5, 2026

lsiepel changed the title ~~[veSync] Auth V2 Support so users can use the binding again~~ [vesync] Add auth V2 support Apr 5, 2026

dag81 mentioned this pull request Apr 11, 2026

[veSync] Auth V2 Support so users can use the binding again (#20236) #20556

Merged

lsiepel pushed a commit that referenced this pull request Apr 12, 2026

[veSync] Auth V2 Support so users can use the binding again (#20236) (#…

baf6940

…20556) [veSync] Auth V2 Support so users can use the binding again (#20236) Signed-off-by: David Goodyear <david.goodyear@gmail.com>

lsiepel added the backported4 Backported patches to the 4.x branch. Used for constructing release notes. label Apr 12, 2026

		if (session != null && !url.startsWith(session.getServerUrl())) {
		url = session.getServerUrl() + url;

-            if (session != null && !url.startsWith(session.getServerUrl())) {
-                url = session.getServerUrl() + url;
+            if (session != null) {
+                String serverUrl = session.getServerUrl();
+                if (serverUrl != null && !serverUrl.isBlank() && !url.startsWith("http://")
+                        && !url.startsWith("https://") && !url.startsWith(serverUrl)) {
+                    url = serverUrl + url;
+                }

		String result = api.reqV2Authorized(url, macId, requestData);

		VeSyncResponse responseFrame = VeSyncConstants.GSON.fromJson(result, VeSyncResponse.class);

	public class LoginTokenResponse {
	public static class LoginTokenResponse {

Uh oh!

Conversation

dag81 commented Feb 16, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

dag81 commented Feb 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

psmedley commented Feb 18, 2026

Uh oh!

psmedley left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

dag81 commented Feb 18, 2026

Uh oh!

psmedley commented Feb 18, 2026

Uh oh!

dag81 commented Mar 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Mar 28, 2026

Choose a reason for hiding this comment

Uh oh!

dag81 Apr 4, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 28, 2026

Choose a reason for hiding this comment

Uh oh!

dag81 Apr 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Copilot AI Mar 28, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 28, 2026

Choose a reason for hiding this comment

Uh oh!

dag81 Apr 4, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

lsiepel Mar 28, 2026

Choose a reason for hiding this comment

Uh oh!

psmedley Mar 28, 2026

Choose a reason for hiding this comment

Uh oh!

lsiepel Mar 28, 2026

Choose a reason for hiding this comment

Uh oh!

dag81 Apr 3, 2026

Choose a reason for hiding this comment

Uh oh!

lsiepel left a comment

Choose a reason for hiding this comment

Uh oh!

lsiepel commented Apr 3, 2026

Uh oh!

dag81 commented Feb 16, 2026 •

edited

Loading

dag81 commented Feb 17, 2026 •

edited

Loading

dag81 commented Mar 28, 2026 •

edited

Loading

dag81 Apr 3, 2026 •

edited

Loading

lsiepel commented Apr 3, 2026 •

edited by dag81

Loading