🚀 Auto-PR: Merge test → main#584
Open
proxy-smart-releaser[bot] wants to merge 82 commits intomainfrom
Open
Conversation
Test Suite: Passed: 0 Failed: 0 Run: https://github.com/Max-Health-Inc/proxy-smart/actions/runs/24935815110
Test Suite: Passed: 0 Failed: 0 Run: https://github.com/Max-Health-Inc/proxy-smart/actions/runs/24935893853
Test Suite: Passed: 0 Failed: 0 Run: https://github.com/Max-Health-Inc/proxy-smart/actions/runs/24935956351
Test Suite: Passed: 0 Failed: 0 Run: https://github.com/Max-Health-Inc/proxy-smart/actions/runs/24935893945
Test Suite: Passed: 0 Failed: 0 Run: https://github.com/Max-Health-Inc/proxy-smart/actions/runs/24936342247
Test Suite: Passed: 0 Failed: 0 Run: https://github.com/Max-Health-Inc/proxy-smart/actions/runs/24936391230
…pharmacogenomics clickable [skip northflank]
🧪 Auto-PR: Merge `develop` → `test`
Test Suite: Passed: 0 Failed: 0 Run: https://github.com/Max-Health-Inc/proxy-smart/actions/runs/24941698311
Test Suite: Passed: 0 Failed: 0 Run: https://github.com/Max-Health-Inc/proxy-smart/actions/runs/24941698399
…per types - Flip @typescript-eslint/no-explicit-any from warn to error in shared ESLint config - Backend (42 violations fixed across 14 files): - auth.ts: Add KeycloakJwtPayload interface for realm_access/resource_access claims - init.ts: Remove as any — RealmRepresentation has organizationsEnabled - runtime-config.ts: Remove as any — RealmRepresentation has loginTheme - org-branding.ts: Remove as any — OrganizationRepresentation has attributes - smart-scope-mappers.ts: Import ProtocolMapperRepresentation, type all mapper callbacks - smart-apps.ts: Use ClientRepresentation.protocolMappers directly, type getAttr properly - organizations.ts: Import OrganizationRepresentation for create/update calls - brand-bundle.ts: Import UserAccessBrandsBundle, use unknown-based assertions - access-control.ts: Add ErrorResponse to response schemas, remove as any - keycloak-config.ts: Add ErrorResponse to response schema - backend-services.ts: Make getKeysInterceptor async, use JSONWebKey[] type - oauth.ts: Type token response body fields instead of as any - dicomweb.ts: Type set parameter with proper Elysia status/headers types - app-factory.ts: Define DiscoveredApp interface for filesystem app objects - index.ts: Use Record<string, unknown> for listen options - UI (3 violations fixed in 1 file): - DicomServersManager.tsx: Import AddDicomServerRequest/UpdateDicomServerRequest types [skip northflank]
🧪 Auto-PR: Merge `develop` → `test`
Test Suite: Passed: 0 Failed: 0 Run: https://github.com/Max-Health-Inc/proxy-smart/actions/runs/24942622780
…h generated types - Export DiscoveredApp from app-factory.ts so tsc can name it in app's inferred type - Use AddDicomServerRequest and AddDicomServerRequestAuthTypeEnum in AddDicomServerDialog instead of inline type with plain string authType [skip northflank]
🧪 Auto-PR: Merge `develop` → `test`
Test Suite: Passed: 62 Failed: 0 Run: https://github.com/Max-Health-Inc/proxy-smart/actions/runs/24945746238
…ploy race Root cause: smart-compliance-tests.yml triggered directly on push to 'test' and 'main' branches, racing against the release workflows that deploy first. Tests hit the beta server before new code was deployed → 503 from Caddy. Fix: Remove 'test' and 'main' from push triggers. These branches are already handled by release-beta.yml and release-production.yml which call testing-strategy.yml → smart-compliance-tests.yml AFTER deployment completes. Direct push triggers remain for dev/*, develop/*, and develop (alpha). [skip northflank]
🧪 Auto-PR: Merge `develop` → `test`
Test Suite: Passed: 0 Failed: 0 Run: https://github.com/Max-Health-Inc/proxy-smart/actions/runs/24946133498
Test Suite: Passed: 62 Failed: 0 Run: https://github.com/Max-Health-Inc/proxy-smart/actions/runs/24946140337
Test Suite: Passed: 0 Failed: 0 Run: https://github.com/Max-Health-Inc/proxy-smart/actions/runs/24980720308
- Create custom keycloak.v2 child theme (proxy-smart) - Override social-providers.ftl with inline M logo SVG for maxhealth alias - Add branded button styles targeting #social-maxhealth - Copy theme into Keycloak image via Dockerfile [skip northflank]
🧪 Auto-PR: Merge `develop` → `test`
Test Suite: Passed: 0 Failed: 0 Run: https://github.com/Max-Health-Inc/proxy-smart/actions/runs/24997735764
Test Suite: Passed: 0 Failed: 0 Run: https://github.com/Max-Health-Inc/proxy-smart/actions/runs/24997921718
The Keycloak Docker build failed because keycloak/ was fully excluded except realm-export*.json. Add !keycloak/themes/ to allow the custom proxy-smart login theme to be copied into the image. Also adds proxy-smart to the login theme dropdown in BrandSettings. [skip northflank]
Test Suite: Passed: 0 Failed: 0 Run: https://github.com/Max-Health-Inc/proxy-smart/actions/runs/24999575059
Test Suite: Passed: 0 Failed: 0 Run: https://github.com/Max-Health-Inc/proxy-smart/actions/runs/24999792368
MCP clients (e.g. VS Code) obtain Keycloak tokens via OAuth DCR but may not refresh them when they expire. This causes 401s on all admin operations routed through the Keycloak admin client. When the user token passes JWT signature/claims validation but Keycloak rejects it (session expired, token revoked), the admin client now falls back to the admin-service client_credentials grant automatically. The caller's authorization is still fully enforced — their roles are checked during JWT validation. The service account is only used as a transport credential for the Keycloak admin API. - Add getServiceAccountAdmin() with token caching (4.5 min TTL) - Wrap KcAdminClient in a Proxy that catches 401s on namespace methods - On 401, retry the same operation with a fresh service-account token [skip northflank]
🧪 Auto-PR: Merge `develop` → `test`
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Automated Pull Request 🤖
This PR was automatically created to merge changes from
testintomain.Changes:
Review: Please review the changes before merging.
Last updated: 2026-04-27 20:06:52 UTC