Skip to content

Refactor to use Caller - #2090

Merged
adelowo merged 13 commits into
mainfrom
feat-caller
Mar 4, 2026
Merged

Refactor to use Caller#2090
adelowo merged 13 commits into
mainfrom
feat-caller

Conversation

@matoszz

@matoszz matoszz commented Mar 1, 2026

Copy link
Copy Markdown
Member

PR requires theopenlane/iam#448 + theopenlane/gqlgen-plugins#166 + theopenlane/entx#251 to be merged first

With the local directives that include these changes, all passing tests and functionality intact:

➜  core git:(feat-caller) task go:testsum:local                    
✓  internal/ent/eventqueue (1.867s)
✓  internal/ent/notifications (1.805s)
✓  internal/ent/interceptors (3.356s)
✓  internal/ent/privacy/rule (6.748s)
✓  internal/ent/schema (8.338s)
✓  internal/ent/validator (8.773s)
✓  internal/entitlements/entmapping (13.414s)
✓  internal/entitlements/reconciler (10.171s)
✓  internal/httpserve/authmanager (2.167s)
✓  internal/graphapi/common (3.61s)
✓  internal/graphsubscriptions (5.151s)
✓  internal/entdb (23.152s)
✓  internal/ent/hooks (25.291s)
✓  internal/httpserve/config (9.362s)
✓  internal/httpserve/handlers/internal/jsonschemautil (cached)
✓  internal/httpserve/handlers/test_webfinger (cached)
✓  internal/integrations (cached)
✓  internal/integrations/providers/schematest (cached)
✓  internal/integrations/activation (619ms)
✓  internal/httpserve/route (4.696s)
✓  internal/httpserve/handlers/scim (6.289s)
✓  internal/integrations/providers/apikey (8.36s)
✓  internal/integrations/providers/awssts (8.499s)
✓  internal/httpserve/server (8.621s)
✓  internal/integrations/providers/aws (10.513s)
✓  internal/integrations/providers/azuresecuritycenter (10.738s)
✓  internal/integrations/providers/github (10.49s)
✓  internal/integrations/providers/microsoftteams (10.724s)
✓  internal/integrations/providers/gcpscc (11.061s)
✓  internal/integrations/providers/slack (11.231s)
✓  internal/httpserve/serveropts (11.9s)
✓  internal/keymaker (10.802s)
✓  internal/mutations (8.617s)
✓  internal/objects (9.3s)
✓  internal/keystore (9.424s)
✓  internal/objects/resolver (6.871s)
✓  internal/objects/store (7.653s)
✓  internal/objects/upload (6.097s)
✓  internal/objects/validators (4.306s)
✓  internal/workflows (4.577s)
✓  internal/workflows/observability (cached)
✓  pkg/domain (cached)
✓  pkg/jsonx (cached)
✓  pkg/catalog (712ms)
✓  pkg/catalog/gencatalog (1.238s)
✓  pkg/celx (3.37s)
✓  pkg/entitlements (4.325s)
✓  pkg/gala (4.733s)
✓  pkg/logx (5.074s)
✓  pkg/logx/consolelog (5.546s)
✓  pkg/mapx (5.83s)
✓  pkg/middleware/cachecontrol (cached)
✓  pkg/middleware/graphapi (cached)
✓  pkg/middleware/impersonation (cached)
✓  pkg/middleware/mime (cached)
✓  pkg/middleware/secure (cached)
✓  pkg/objects (cached)
✓  pkg/objects/storage (cached)
✓  pkg/middleware/ratelimit (cached)
✓  pkg/middleware/auth (4.669s)
✓  pkg/objects/storage/providers/disk (2.48s)
✓  pkg/objects/storage/providers/database (4.049s)
✓  pkg/ssoutils (cached)
✓  pkg/summarizer (cached)
✓  pkg/slateparser (cached)
✓  pkg/objects/storage/proxy (1.642s)
✓  pkg/permissioncache (4.019s)
✓  pkg/objects/storage/providers/s3 (9.238s)
✓  pkg/objects/storage/providers/r2 (23.794s)
✓  internal/workflows/engine (1m15.884s)
✓  internal/httpserve/handlers (2m40.238s)
✓  internal/graphapi (3m47.434s)

=== Skipped
=== SKIP: pkg/summarizer TestLLM_Summarize (0.00s)

DONE 4419 tests, 1 skipped in 268.937s

Comment thread internal/ent/mixin/systemowned_mixin.go
Comment thread internal/ent/interceptors/filter.go
Comment thread internal/ent/interceptors/trustcenter.go
Comment thread internal/ent/interceptors/usersetting.go
Comment thread internal/ent/privacy/rule/modules.go
Comment thread internal/ent/schema/mixin_objectowned.go
Comment thread internal/ent/schemautil/trustcenter_scope.go
Comment thread internal/graphapi/common/helpers.go Outdated
Comment thread internal/graphapi/tools_test.go Outdated
Comment thread internal/integrations/providers/oauth/options.go Outdated
Comment thread internal/objects/store/permissions.go Outdated
Comment thread internal/objects/store/persistence.go Outdated
Comment thread pkg/middleware/impersonation/impersonation_test.go Outdated
Comment thread pkg/middleware/impersonation/impersonation_test.go
matoszz and others added 5 commits March 4, 2026 11:16
Signed-off-by: Sarah Funkhouser <147884153+golanglemonade@users.noreply.github.com>
Signed-off-by: Sarah Funkhouser <147884153+golanglemonade@users.noreply.github.com>
Signed-off-by: Sarah Funkhouser <147884153+golanglemonade@users.noreply.github.com>
Signed-off-by: Sarah Funkhouser <147884153+golanglemonade@users.noreply.github.com>
Signed-off-by: Sarah Funkhouser <147884153+golanglemonade@users.noreply.github.com>
@sonarqubecloud

sonarqubecloud Bot commented Mar 4, 2026

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
8.2% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@golanglemonade
golanglemonade marked this pull request as ready for review March 4, 2026 19:21
@golanglemonade
golanglemonade requested a review from a team as a code owner March 4, 2026 19:21
@golanglemonade golanglemonade changed the title Refcator to use Caller Refactor to use Caller Mar 4, 2026
@adelowo
adelowo merged commit 44eb720 into main Mar 4, 2026
20 of 21 checks passed
@adelowo
adelowo deleted the feat-caller branch March 4, 2026 19:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants